chore(deps-dev): bump @stryker-mutator/core from 9.6.1 to 10.0.0 - #506
chore(deps-dev): bump @stryker-mutator/core from 9.6.1 to 10.0.0#506dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [@stryker-mutator/core](https://github.com/stryker-mutator/stryker-js/tree/HEAD/packages/core) from 9.6.1 to 10.0.0. - [Release notes](https://github.com/stryker-mutator/stryker-js/releases) - [Changelog](https://github.com/stryker-mutator/stryker-js/blob/master/packages/core/CHANGELOG.md) - [Commits](https://github.com/stryker-mutator/stryker-js/commits/v10.0.0/packages/core) --- updated-dependencies: - dependency-name: "@stryker-mutator/core" dependency-version: 10.0.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
|
🦞👀 Pull request received. I will update this pull request when review starts. |
|
Codex review: found issues before merge. Reviewed August 17, 2026, 9:28 AM ET / 13:28 UTC. ClawSweeper reviewWhat this changesThis PR updates the development-only mutation-testing engine from version 9.6.1 to 10.0.0 and refreshes its resolved dependency graph. Merge readinessKeep open: the Stryker 10 resolution brings in Babel 8, whose declared Node support excludes part of acpx’s documented Node 22.13+ range; resolve that compatibility choice before merging. Priority: P2 Review scores
Verification
How this fits togetheracpx runs Stryker as its mutation-testing tool during development and CI. The package manifest and pnpm lockfile select the toolchain that flowchart LR
A[Developer or CI] --> B[Package manifest]
B --> C[pnpm lockfile]
C --> D[Stryker mutation tool]
D --> E[Babel instrumenter]
E --> F[Mutation test result]
Decision needed
Why: The current manifest and repository policy promise Node >=22.13, while the newly locked transitive toolchain excludes part of that range; changing that promise is a maintainer compatibility-policy choice. Before merge
Findings
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Keep the documented Node 22.13 baseline by using a Stryker resolution compatible with it, or deliberately raise the project’s supported Node floor and validate clean install and mutation runs at the new minimum. Do we have a high-confidence way to reproduce the issue? Unclear: source proves the Node-engine contract mismatch, but confirming the exact install or runtime failure requires a clean run on Node 22.13–22.17. Is this the best way to solve the issue? No. The update is reasonable, but this resolution is not the best path while acpx promises Node >=22.13; preserve that support or make a deliberate, validated runtime-policy change. Full review comments:
Overall correctness: patch is incorrect AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 026544cd7550. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rank-up movesOptional improvements that raise the rating; they are not merge blockers.
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
|
|
Superseded by #515, which lands the dependency refresh as one coherent branch (avoids lockfile cascades) with build + smoke proof. |
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
Rebasing might not happen immediately, so don't worry if this takes some time.
Note: if you make any changes to this PR yourself, they will take precedence over the rebase.
Bumps @stryker-mutator/core from 9.6.1 to 10.0.0.
Release notes
Sourced from @stryker-mutator/core's releases.
Changelog
Sourced from @stryker-mutator/core's changelog.
Commits
cb3bd8fv10.0.0aadfdc9chore(deps): update typescript-eslint monorepo to v8.67.0 (#5984)51b9fa5chore(deps): update dependency@types/nodeto v24.13.3 (#6122)29ede69fix(deps): update mutation-testing-elements monorepo to v3.8.4 (#6118)291b7eafeat(babel): update babel to major version 8 (#6104)d62fd75fix(deps): update mutation-testing-elements monorepo to v3.8.0 (#6095)cc08738feat: empty-expression-mutator (#6012)5deb03echore(deps): update dependency@types/nodeto v24.13.2 (#6058)0ead8cefix(deps): update dependency ajv to v8.20.0 (#5992)6201b38chore(deps): update dependency@types/nodeto v24.12.4 (#6015)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)