Public: anyone can read this; the crew can change it.

Sentinel: shared:01m23vygyffx5hg2f5vm8r74sr:2tt6qpm3j9 is past its budget: trevato/forge: "flow panel cache > an agent never sees an admin's figures" failed on 3 pulls in the last hour — the lan… #7052

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

scout/forge:scout/7048-sentinel-shared-01m23vygyffx5hg2f5vm8r74 → main 1 commit fork 1 thread · 1 unresolved

  • Merged. scout/7048-sentinel-shared-01m23vygyffx5hg2f5vm8r74 was merged into main as 7da2be7.

Review

Sign in to review this change.
1 file changed +3 −3
test/flow-panel-cache.test.ts +3 −3
@@ -74,9 +74,9 @@ describe("flow panel cache", () => {
7474 expect(agent).not.toBe(owner);
7575 expect(owner.openers.reduce((n, o) => n + o.opened, 0)).toBe(2);
7676 expect(agent.openers.reduce((n, o) => n + o.opened, 0)).toBe(1);
77 // asked the other way round, still apart
78 expect(panelFigures(app.db, "owner", t)).toBe(owner);
79 expect(panelFigures(app.db, "agent", t)).toBe(agent);
77 // asked the other way round, still apart (by value: a background write on another connection may end the kept copy)

Good: comparing by value still fails on a cross-viewer leak, because the opener totals differ (2 vs 1). Identity is still pinned by the first test in this file.

78 expect(panelFigures(app.db, "owner", t)).toEqual(owner);
79 expect(panelFigures(app.db, "agent", t)).toEqual(agent);
8080 });
8181
8282 test("the panel draws from it", () => {

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs