[roodboi]

Summary

Two small follow-ups to #489 in workflowJobBunVersion (scripts/ci-bun-crash-census.ts).

Root cause

1. Commented jobs: line. The parser found the jobs section with line.trimEnd() === "jobs:". A valid workflow whose jobs: line has a trailing YAML comment (jobs: # all jobs) therefore had no jobs, and every job read as having no Bun version.
2. Dead blank-line clause. The job scan skipped lines with line.trim() === "" || line.startsWith("#"). Removing the blank-line half changed no result: a blank or whitespace-only line has no non-space first character (so it does not end the jobs) and matches none of the job-key, name: or bun-version patterns (so it does not change a block). No test can fail without it, so the JSDoc claim that blank lines are "skipped" described code that did nothing.

Fix

• Strip a whitespace-led trailing comment (/\s+#.*$/) before matching jobs:. A # with no space before it (jobs:#) is not a YAML comment and still does not match.
• Remove the redundant blank-line clause and correct the JSDoc to say why blank lines need no skip.

Tests

• New: a trailing comment on the jobs: line still opens the jobs (HACK-1462). It covers jobs: # all jobs with a whitespace-only line inside the job, plus the jobs:# negative case.
• Existing: blank and column-0 comment lines inside jobs: do not end the job scan already has blank lines between jobs and still holds the documented behavior.

Evidence

Focused suite tests/ci-bun-crash-census.test.ts: 20 pass, 0 fail. Mutation results:

Mutation	Result
M1: match jobs: with plain trimEnd() (pre-fix)	new test fails (19 pass, 1 fail)
M2: comment regex \s*# (no whitespace needed)	new test fails on the jobs:# case
M3: a blank line ends the jobs scan	4 tests fail, including the blank-line test and the real ci.yml test
M4: put back the removed blank-line clause	20 pass: the clause makes no difference to behavior

Gates: root bun run check (includes privacy:check) passes with exit 0. bun run --cwd packages/cli typecheck has 0 errors. Biome reports no issues on both changed files. gitleaks 8.30.1 found no leaks in origin/next..HEAD. The CLI surface and agent guidance did not change, so neither needs regeneration. For the full suite, see hosted CI test at this head.

Gaps

• Job-key lines ( test: # note) and name: lines with trailing comments are still not handled. They are outside this unit's scope and are reported separately.

Refs HACK-1462

🤖 Generated with Claude Code

---

^{Need help on this PR? Tag @codesmith-bot with what you need.}

[linear-code]

HACK-1462