Automate a review finding as a lint or test: This line runs far past the file's wrap width; rewrap it. #10149

The existing header-width rule now catches every comment line past 160 columns, not just the file's leading block. That is what #8957, #9222, #9627 and #9943 asked for. I ran test/header-width.test.ts (it passes), bun scripts/header-width.ts (exits 0), tsc --noEmit and scripts/gate.ts (both printed no errors). I did not run the rest of the suite.

What changed

• Coverage: longHeaderLines in scripts/header-width.ts now checks // lines, block-comment lines and the leading block, all at 160 columns. The same check runs in test/header-width.test.ts, which the gate already runs.
• Failure message: it names the rule: header-width: comment line over 160 columns, rewrap it: file:line. The test's own message does the same.
• Tests: they cover a long // line, a long block-comment line, code and string lines (not flagged) and a URL line (not flagged).
• Existing offenders: scripts/header-width-allow.json now lists all 34 files that already have a long comment line. The list can only shrink: a new offender fails, and so does a listed file that is now clean.

What I left out

• I didn't rewrap the existing long lines. That is a separate sweep, and the allow list tracks it.
• Trailing comments after code on the same line aren't checked.
• Comment lines containing a URL are exempt, since they can't be wrapped.
• 160 columns is the existing limit; I didn't make it per-file.

To try it, add a 200-character // … line to any file under src/, scripts/ or test/ and run bun test test/header-width.test.ts.

Closes #10140

The existing header-width rule now catches every comment line past 160 columns, not just the file's leading block. That is what #8957, #9222, #9627 and #9943 asked for. I ran test/header-width.test.ts (it passes), bun scripts/header-width.ts (exits 0), tsc --noEmit and scripts/gate.ts (both printed no errors). I did not run the rest of the suite.

What changed

• Coverage: longHeaderLines in scripts/header-width.ts now checks // lines, block-comment lines and the leading block, all at 160 columns. The same check runs in test/header-width.test.ts, which the gate already runs.
• Failure message: it names the rule: header-width: comment line over 160 columns, rewrap it: file:line. The test's own message does the same.
• Tests: they cover a long // line, a long block-comment line, code and string lines (not flagged) and a URL line (not flagged).
• Existing offenders: scripts/header-width-allow.json now lists all 34 files that already have a long comment line. The list can only shrink: a new offender fails, and so does a listed file that is now clean.

What I left out

• I didn't rewrap the existing long lines. That is a separate sweep, and the allow list tracks it.
• Trailing comments after code on the same line aren't checked.
• Comment lines containing a URL are exempt, since they can't be wrapped.
• 160 columns is the existing limit; I didn't make it per-file.

To try it, add a 200-character // … line to any file under src/, scripts/ or test/ and run bun test test/header-width.test.ts.

header-width now fails any comment line past the width and names the rule. The latest push fixes one of the two failing checks; the other I did not fix.

• Fixed: src/core/teams_words.ts is imported by the pull pages, the API and the terminal, so it no longer belongs in scripts/unreached-allow.json. I removed it, and test/unreached.test.ts passes (4 of 4). bun x tsc --noEmit is clean.
• Not fixed: "a child forge spawned into a cell serves its app at <app>--<child>--<parent>" is in test/spawn-cell-e2e.test.ts. It ran 153 s in the check and failed, and that file's cell tests are skipped in my sandbox, so I couldn't run it. This pull touches only scripts/header-width.ts, its allow list and its test, none of which that test uses. I read it as a timeout in the check's environment, not a regression from this change. If it fails again on the new push, it needs a person to look at it.
• Left out: I did not re-run the full suite here; the forge's check does that on push.