A check that fails "(unnamed)" names the test file and the hook that failed #4620

A failure outside a named test now shows up as test/origins-reconnect.test.ts > beforeAll (timeout) instead of "(unnamed)". The new tests, the existing shards, shared-red and checks suites and the type check pass here. I did not run design-lint because nothing in src/web/ changed, and I have not seen the forge's check on this push.

I wrote the output format from what I believe bun prints, not from a real captured log. The test feeds a hand-made report in that shape, with a file header, an error: beforeAll() timed out after 5000ms line and (fail) (unnamed). It would fail on main, which still reports "(unnamed)". If bun words a hook error differently, the hook falls back to file > (unnamed), which still names the file.

• Where it lives: the renaming is one new function, nameUnnamed, in src/checks/bun-test.ts. It tracks the file header above each (fail) (unnamed) or … > (unnamed) line. If the error: line before it names a hook, the hook goes in the name, with (timeout) when bun says it timed out.
• src/checks/summary.ts: it already reads failures through parseBunTestFailure, which now uses the function. The summary's quoted test name is therefore the file and hook.
• scripts/shards.ts: tally renames before it echoes failures, so the shard log carries the same names.
• src/checks/shared_red.ts: I did not change it. Failures now carry a per-file name, so they group by file. Pulls that fail the same file's hook can count as the lane's red; only the bare "(unnamed)" is still excluded.
• Not done: an unhandled rejection or file-level throw that bun does not print as a (fail) (unnamed) line is not renamed.

The new tests are in test/shared-red.test.ts. One covers the summary and one covers the shard tally.

Closes #4582

A failure outside a named test now shows up as test/origins-reconnect.test.ts > beforeAll (timeout) instead of "(unnamed)". The new tests, the existing shards, shared-red and checks suites and the type check pass here. I did not run design-lint because nothing in src/web/ changed, and I have not seen the forge's check on this push.

I wrote the output format from what I believe bun prints, not from a real captured log. The test feeds a hand-made report in that shape, with a file header, an error: beforeAll() timed out after 5000ms line and (fail) (unnamed). It would fail on main, which still reports "(unnamed)". If bun words a hook error differently, the hook falls back to file > (unnamed), which still names the file.

• Where it lives: the renaming is one new function, nameUnnamed, in src/checks/bun-test.ts. It tracks the file header above each (fail) (unnamed) or … > (unnamed) line. If the error: line before it names a hook, the hook goes in the name, with (timeout) when bun says it timed out.
• src/checks/summary.ts: it already reads failures through parseBunTestFailure, which now uses the function. The summary's quoted test name is therefore the file and hook.
• scripts/shards.ts: tally renames before it echoes failures, so the shard log carries the same names.
• src/checks/shared_red.ts: I did not change it. Failures now carry a per-file name, so they group by file. Pulls that fail the same file's hook can count as the lane's red; only the bare "(unnamed)" is still excluded.
• Not done: an unhandled rejection or file-level throw that bun does not print as a (fail) (unnamed) line is not renamed.

The new tests are in test/shared-red.test.ts. One covers the summary and one covers the shard tally.

A failure outside a named test now carries its file name, and a hook timeout carries the hook too, e.g. a.test.ts > beforeEach/afterEach (timeout). Tests for shared-red, shards and checks and the type check pass. The (unnamed) rename can't tell a beforeAll timeout from a beforeEach one. I haven't seen the forge's check on this push.

I ran bun 1.3.14 here to see what it prints, and the guessed format was wrong. There is no error: beforeAll() timed out line. A timed-out hook prints (fail) (unnamed) [51ms], then the line ^ a beforeEach/afterEach hook timed out for this test. right after it. bun does not say whether it was beforeAll or beforeEach, so the name uses bun's own wording.

• Hook that threw: it prints (fail) s > (unnamed) with the error above it and no hook name. It now becomes c.test.ts > s > (unnamed), which names the file and keeps bun's describe prefix.
• Recap: once the summary starts (N pass, or N tests failed:), no file is current until the next file header. A recap's (fail) (unnamed) lines stay as bun printed them rather than taking the last file's name.
• Fixture: test/shared-red.test.ts now uses a fixture built from the real bun lines, plus a recap block. I wrote the recap from your description; I couldn't make bun print one here. The test also checks that recap lines are left alone.
• Import: the import in scripts/shards.ts now sits at the top of the file.

Not handled: the # Unhandled error between tests block, which bun prints for a file that throws while loading, has no (fail) line. It is still unnamed in the summary; onlyTestFailures already treats it as not a plain test failure.