fix(ci): handle Node 25 incompatibility with c8 11.0.0 / yargs 17.x

[Copilot]

c8 11.0.0 (bumped to fix CVE-2026-26996 in minimatch) fails on Node 25 with ReferenceError: require is not defined in ES module scope. Node 25 changed how extension-less files in "type": "module" packages are loaded—they're now treated as ESM. c8 11.0.0's updated test-exclude (v7→v8) switched glob v10→v13, which eliminated the node_modules/yargs/node_modules/ nested directory. Without it, yargs/yargs (extension-less, in a "type": "module" package) is now loaded as ESM by Node 25, where require() is not defined.

Changes

• package.json

  • Add test-no-coverage script — identical to test but without c8, for use on Node 25 where c8 cannot start

• .github/workflows/test.yml

  • On Node 25, run npm run test-no-coverage instead of npm test to skip c8 coverage and fall back to plain unit tests

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Copilot
Once this PR has been reviewed and has the lgtm label, please ask for approval from brendandburns. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[brendandburns]

@copilot as an alternative can you disable coverage via c8 for node25 and fall back to just plain unit tests?

[Copilot]

@copilot as an alternative can you disable coverage via c8 for node25 and fall back to just plain unit tests?

Done in commit 528c983. Added a test-no-coverage script to package.json that runs node --test --test-reporter=spec --import tsx src/*_test.ts (identical to test but without the c8 prefix). The workflow now uses npm test for all nodes except 25, and npm run test-no-coverage for Node 25, with continue-on-error removed.

[brendandburns]

Remove this line as it is now unrelated.

[Copilot]

Removed in commit 228e04a.

[brendandburns]

@copilot please address comments

[Copilot]

@copilot please address comments

All review comments have been addressed — the fail-fast: false line was removed in commit 228e04a.