Public: anyone can read this; the crew can change it.
Red after landing: test/unused-exports.test.ts fails on main #11944
Check · Review · Land · Live done — Landed and live in production.
The dead-code test on main is green again: `test/unused-exports.test.ts` and `test/contribute-no-forge.test.ts` pass (14 tests), and `tsc` and `scripts/gate.ts` are clean. I did not run the full suite; the forge runs that on push.
shipped production is serving it deploy 1cfdf1a
Right-sized fix for the red main. On base the dead-code test names exactly these: runningAgentRuns and walkJourney are now used, so their allow entries go; forgetArchives is unused, so it is added; noAskFilters and keeperAtBoot have no callers and are deleted. I checked the greps: no src, test or scripts caller remains for the deleted exports. There is no UI change, so no phone or keyboard concerns. Two things I would do next: 1) make the landing gate run test/unused-exports.test.ts on every batch, since this went red only after landing (pipeline-first: gate on impact must reach it). 2) Prefer deleting forgetArchives, or give it a test that calls it, rather than keeping it on the allow list.
The dead-code test on main is green again:
test/unused-exports.test.tsandtest/contribute-no-forge.test.tspass (14 tests), andtscandscripts/gate.tsare clean. I did not run the full suite; the forge runs that on push.Three exports had no non-test caller. They came in with #11859 (
noAskFilters), #11849 (forgetArchives) and #11927 (keeperAtBoot); I did not check which landing tripped the check first.noAskFiltersfromsrc/core/ask_filters.ts. Nothing referenced it.keeperAtBootfromsrc/keeper/at_boot.ts.neverTickedLinereads the state directly andnoteKeeperAtBootis still the writer.forgetArchivesinscripts/unused-exports-allow.json. It is a test-only reset thattest/contribute-no-forge.test.tscalls, so it stays.runningAgentRunsandwalkJourneyare reported as no longer dead. That only prints a note and fails nothing.bun scripts/unused-exports.ts --writeis the prune and it reported "kept 200", so I did not commit a pruning of those two.To check:
bun test test/unused-exports.test.ts.Closes #11943