Skip to content

fix(ci): register missing mutation coverage tests - #17083

Closed
diegosouzapw wants to merge 2 commits into
release/v3.8.52from
fix/release-v3.8.52-basereds-mutation-20261018
Closed

diegosouzapw wants to merge 2 commits into
release/v3.8.52from
fix/release-v3.8.52-basereds-mutation-20261018

Conversation

@diegosouzapw

Copy link
Copy Markdown
Collaborator

Summary

Register four existing covering test files in Stryker's explicit tap.testFiles
list. The strict gate found seven missing test/module associations across five
mutated modules on release base a016744417facc740a9360816a00dca07d72d6c6.
No runtime behavior, mutation targets, thresholds, exemptions or test assertions
are changed.

⚠️ base-red inherited: #15306. This fixes the independently reproduced mutation
registration failure, not all release failures.

Validation on VPS .15

Node 24.20.0; isolated worktree and temporary test data; reused diagnostic
dependencies, not a fresh-install certificate.

  • RED: node scripts/check/check-mutation-test-coverage.mjs --strict — exit 1;
    6,948 unit files, 31 mutated modules, seven missing associations.
  • GREEN: same command after the four additions — exit 0, zero drift.
  • node --import tsx/esm --import ./tests/_setup/isolateDataDir.ts --test --test-concurrency=1 tests/unit/build/check-mutation-test-coverage.test.ts tests/unit/chat-empty-stream-recovery.test.ts tests/unit/content-policy-refusal.test.ts tests/unit/provider-error-rules-opt-in-flags-16797.test.ts tests/unit/stream-timing-late-output.test.ts
    — 65 PASS, zero failures/skips/cancellations.
  • Normal pre-commit and commit-msg hooks passed on .15.
  • Independent Standards and Spec reviews: zero findings in the four-entry diff.

RED/GREEN and regression receipts preserve source/diff/lockfile identities,
resource reserves, actual exits and raw-log digests. Source remained unchanged
during each check. The exact configuration content reviewed and tested is SHA256
f691461e8b62dd516276a217bea7da168fc24c736d1afd3f92c0a999a43a4cf6.

Full mutation scoring, full CI/release admission, build and deployment are
separate and are not claimed by these focused results. Owner-authorized merge
remains conditional on all applicable exact-candidate requirements.

@diegosouzapw
diegosouzapw marked this pull request as ready for review October 18, 2026 18:31
@diegosouzapw

Copy link
Copy Markdown
Collaborator Author

Exact-candidate checkpoint, 2026-10-18: HOLD — not merge-ready.

Current head cd0e8eae661f846d2b165f2cdea7895788d96ec0 was tested by CI on
merge 3bcf3a742623a3c52e785652af4a1f9b8c227062, base a016744417.
The candidate changes only the Stryker registration and its changelog fragment.

An independent focused reproduction on the owner-approved validation VPS,
against the clean pure base a016, ran43 tests in9 files:35 passed,8 failed.
It reproduces the same four documentation hash failures, two manual-model
fixture failures, duplicate /api/models custom-model read, and DeepSeek quota
spacing failure (24ms vs required180ms). Dependencies were reused; this is
diagnostic evidence, not a clean-install/full-suite certificate.

Relay's403/429 failure did not reproduce in that focused run. The context-retention
and BurnRateChart failures remain unclassified; no flaky/infra waiver is claimed.
Full CI logs and raw reproduction receipts are preserved privately. No rerun
to hide a red, baseline change or merge bypass was performed.

Coordination: #17090 from another workstream addresses the three catalog failures
and registers the same four Stryker paths (different order). Its work must be
reconciled before landing competing changes. No edit or merge of #17090 is
claimed here. Documentation recovery continues separately in #16980.

The earlier focused registration tests remain valid for their limited scope.
Required aggregate CI/Quality admission is still pending/failing. Refs #15306.

@github-actions

Copy link
Copy Markdown
Contributor

CI Coverage Report

  • Coverage job: skipped
  • PR test policy: success

Coverage artifact was not available for this run.

@diegosouzapw

Copy link
Copy Markdown
Collaborator Author

Superseded by #17090, which registered the same four tests (chat-empty-stream-recovery, content-policy-refusal, provider-error-rules-opt-in-flags-16797, stream-timing-late-output) in stryker.conf.json tap.testFiles. Nothing left to land.

@diegosouzapw diegosouzapw mentioned this pull request Oct 19, 2026
@diegosouzapw
diegosouzapw deleted the fix/release-v3.8.52-basereds-mutation-20261018 branch October 22, 2026 04:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant