Skip to content

deps: bump @stryker-mutator/core and vitest-runner to 9.6.1#842

Merged
Chris0Jeky merged 1 commit intomainfrom
deps/stryker-mutator-9.6.1
Apr 16, 2026
Merged

deps: bump @stryker-mutator/core and vitest-runner to 9.6.1#842
Chris0Jeky merged 1 commit intomainfrom
deps/stryker-mutator-9.6.1

Conversation

@Chris0Jeky
Copy link
Copy Markdown
Owner

Summary

  • Bumps @stryker-mutator/core from 8.7.1 to 9.6.1
  • Bumps @stryker-mutator/vitest-runner from 8.7.1 to 9.6.1
  • Both packages share a peer dependency and must be upgraded together

This combines the individual Dependabot PRs #828 (@stryker-mutator/core) and #829 (@stryker-mutator/vitest-runner), which each failed CI because they only bumped one side of the peer dependency pair.

Verification

  • npm ci -- clean install succeeds with both at 9.x
  • vue-tsc --noEmit -- typecheck passes
  • npm run build -- full production build passes
  • npx vitest --run -- all 2452 unit tests pass (205 test files)
  • Stryker config (stryker.config.mjs) requires no changes for 9.x
  • npm ls @stryker-mutator/core @stryker-mutator/vitest-runner confirms both at 9.6.1, deduped

Test plan

  • npm ci succeeds
  • Typecheck passes
  • Full build passes
  • All 2452 unit tests pass
  • CI pipeline passes

Upgrade both Stryker packages together since they share a peer
dependency and must stay on the same major version. Combines the
individual Dependabot PRs #828 and #829 which each failed CI
because they only bumped one side of the pair.
@Chris0Jeky
Copy link
Copy Markdown
Owner Author

Self-review

Changes reviewed: package.json and package-lock.json only.

package.json: Exactly 2 lines changed -- both @stryker-mutator/core and @stryker-mutator/vitest-runner bumped from ^8.7.1 to ^9.6.1. No other dependencies affected.

package-lock.json: Lockfile updates reflect the new Stryker 9.x dependency tree. Net reduction in transitive dependencies (452 additions vs 571 deletions) -- Stryker 9.x appears to have a leaner dependency graph.

Stryker config: stryker.config.mjs reviewed -- all options (packageManager, reporters, testRunner, vitest, coverageAnalysis, thresholds, mutate, tempDirName, cleanTempDir, timeoutMS, timeoutFactor) remain valid in 9.x. The @stryker-mutator/api/core type import resolves correctly. No config changes needed.

Verification results:

  • npm ci -- clean install succeeds
  • vue-tsc --noEmit -- typecheck passes
  • npm run build -- production build passes (97 precached PWA assets)
  • npx vitest --run -- all 2452 tests pass across 205 test files
  • npm ls confirms both packages at 9.6.1, properly deduped

Risk assessment: Low. This is a dev-only dependency (mutation testing tooling) that does not affect production builds or runtime behavior. The upgrade is straightforward with no breaking changes affecting our configuration.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the frontend mutation-testing toolchain by upgrading Stryker JS core and its Vitest runner together to satisfy their peer dependency relationship and restore CI stability for Dependabot-driven bumps.

Changes:

  • Bump @stryker-mutator/core from 8.7.1 to 9.6.1
  • Bump @stryker-mutator/vitest-runner from 8.7.1 to 9.6.1
  • Refresh package-lock.json to reflect the new Stryker dependency graph

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
frontend/taskdeck-web/package.json Updates Stryker core + vitest runner devDependency versions to 9.6.1 in tandem.
frontend/taskdeck-web/package-lock.json Locks the upgraded Stryker packages and their updated transitive dependencies.
Files not reviewed (1)
  • frontend/taskdeck-web/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request upgrades @stryker-mutator/core and @stryker-mutator/vitest-runner from version 8.7.1 to 9.6.1, which involves extensive updates to transitive dependencies in the lockfile. The review feedback suggests pinning these packages to exact versions rather than using caret ranges to prevent potential peer dependency conflicts, as the vitest-runner requires a specific version of the core package to function correctly.

Comment on lines +58 to +59
"@stryker-mutator/core": "^9.6.1",
"@stryker-mutator/vitest-runner": "^9.6.1",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The @stryker-mutator/vitest-runner package version 9.6.1 has a strict peer dependency on @stryker-mutator/core at exactly version 9.6.1 (as seen in the lockfile). Using the caret (^) range in package.json allows npm to install a newer version of the core package (e.g., 9.7.0) while keeping the runner at 9.6.1, which will lead to peer dependency conflicts and installation failures. It is recommended to use exact versions for these packages to ensure they remain in sync, especially since they must be upgraded together as noted in the PR description.

Suggested change
"@stryker-mutator/core": "^9.6.1",
"@stryker-mutator/vitest-runner": "^9.6.1",
"@stryker-mutator/core": "9.6.1",
"@stryker-mutator/vitest-runner": "9.6.1",

@Chris0Jeky Chris0Jeky merged commit 2e127ba into main Apr 16, 2026
30 checks passed
@Chris0Jeky Chris0Jeky deleted the deps/stryker-mutator-9.6.1 branch April 16, 2026 08:39
@github-project-automation github-project-automation Bot moved this from Pending to Done in Taskdeck Execution Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants