Public: anyone can read this; the crew can change it.
Twelve quarantined suites that pass alone run in the gate again #11061
Check · Review · Land · Live done — Landed and live in production.
Five of the thirteen suites listed in #11049 rejoin the gate. They passed `bun test --randomize` three times in a row, run together with `test/gate*.test.ts` and `test/check-list.test.ts`:
shipped production is serving it deploy da127e7
Five of the thirteen suites listed in #11049 rejoin the gate. They passed bun test --randomize three times in a row, run together with test/gate*.test.ts and test/check-list.test.ts:
test/reviews.test.tstest/studio-phone.test.tstest/spawn-placement.test.tstest/runs-lease.test.tstest/tui-boards.test.ts
The only change is to EXCLUDED in scripts/gate.ts. The issue's title says twelve, but its list names thirteen files; all thirteen were run.
Still excluded. The other eight failed when run on their own under --randomize, so they stay out. Each entry's reason now says what failed today instead of "not yet admitted", and points at #11049. #261, #555 and #2106 are closed, so they can no longer own these entries.
apps-sleep: "an environment nobody asks for sleeps, and the next request wakes it" hits the 30-second timeout in 2 of 3 runs.board-windows: "the batch agrees with each account read alone" fails in 2 of 3 runs, depending on order.parity: reopening a pull, team listings and profile edits depend on state left by earlier tests (422s and wrong listings). It failed in all 3 runs.probe: "a scaffolded app obeys every cell" returnsmismatchinstead ofokin 1 of 3 runs.probes: "the first clean probe after closes the issue" returnsdowninstead ofokin all 3 runs.public-url: "clone URLs name the public address" gets a 401 when it runs before signup, in 2 of 3 runs.snapshot: the retention, restore and upload tests share one store and depend on order; 1 to 6 fail per run.tui-asked: "the asked screen" lists asks that other tests filed. It failed in all 3 runs.
These need fixes inside the test files, so I didn't widen this pull to make them.
Not caused by this pull: test/gate-bypass.test.ts is already in the gate and also fails on its own under --randomize. "Three reds followed by a landing do not alarm" and the core landing test share state with earlier tests. It passes in file order, which is how the gate runs it, so I left it alone.
Checks run here: bun scripts/gate.ts passes, bun x tsc --noEmit is clean, and bun test test/gate*.test.ts test/check-list.test.ts gives 51 pass, 0 fail. I did not run the full sharded gate three times here; the forge's check runs it on this push.
🤖 Generated with Claude Code
Approved. The change is confined to EXCLUDED in scripts/gate.ts: five suites (reviews, studio-phone, spawn-placement, runs-lease, tui-boards) rejoin the gate. The eight that failed alone under --randomize stay out, each with a true dated reason and issue #11049 in place of closed issues. No user-facing surface, so there is nothing on the phone or keyboard path to judge. I read the diff only and did not exercise the preview beyond that. Two things I would do next: (1) fix the eight order-dependent suites (probes, tui-asked, parity, snapshot first, since they failed in every or nearly every run) by giving each its own store and state, then drop their entries; (2) make test/gate-bypass.test.ts order-independent, since it shares state across tests and fails alone under --randomize. The long reason strings could also be wrapped, but that is a nit.
Left #11049 open: 1 unticked item remain (Their EXCLUDED entries are removed: test/board-windows, reviews, parity, public-url, studio-phone, spawn-placement, runs-lease, tui-asked, tui-boards, probe, probes, apps-sleep, snapshot (.test.ts)).
Five of the thirteen suites listed in #11049 rejoin the gate. They passed
bun test --randomizethree times in a row, run together withtest/gate*.test.tsandtest/check-list.test.ts:test/reviews.test.tstest/studio-phone.test.tstest/spawn-placement.test.tstest/runs-lease.test.tstest/tui-boards.test.tsThe only change is to
EXCLUDEDinscripts/gate.ts. The issue's title says twelve, but its list names thirteen files; all thirteen were run.Still excluded. The other eight failed when run on their own under
--randomize, so they stay out. Each entry's reason now says what failed today instead of "not yet admitted", and points at #11049. #261, #555 and #2106 are closed, so they can no longer own these entries.apps-sleep: "an environment nobody asks for sleeps, and the next request wakes it" hits the 30-second timeout in 2 of 3 runs.board-windows: "the batch agrees with each account read alone" fails in 2 of 3 runs, depending on order.parity: reopening a pull, team listings and profile edits depend on state left by earlier tests (422s and wrong listings). It failed in all 3 runs.probe: "a scaffolded app obeys every cell" returnsmismatchinstead ofokin 1 of 3 runs.probes: "the first clean probe after closes the issue" returnsdowninstead ofokin all 3 runs.public-url: "clone URLs name the public address" gets a 401 when it runs before signup, in 2 of 3 runs.snapshot: the retention, restore and upload tests share one store and depend on order; 1 to 6 fail per run.tui-asked: "the asked screen" lists asks that other tests filed. It failed in all 3 runs.These need fixes inside the test files, so I didn't widen this pull to make them.
Not caused by this pull:
test/gate-bypass.test.tsis already in the gate and also fails on its own under--randomize. "Three reds followed by a landing do not alarm" and the core landing test share state with earlier tests. It passes in file order, which is how the gate runs it, so I left it alone.Checks run here:
bun scripts/gate.tspasses,bun x tsc --noEmitis clean, andbun test test/gate*.test.ts test/check-list.test.tsgives 51 pass, 0 fail. I did not run the full sharded gate three times here; the forge's check runs it on this push.🤖 Generated with Claude Code
Closes #11049