Repository navigation
chore(deps-dev): bump the dev-dependencies group across 1 directory with 2 updates - #645
Conversation
|
Please add a kind label with |
There was a problem hiding this comment.
Read this against the tree, not just the lockfile. @stryker-mutator/vitest-runner 10.1.1's only behavioural change in the test-selection path (stryker-js#6246, commit 6ad4adb) wraps the name filter as (?:<names>)\s*$ in packages/vitest-runner/src/vitest-test-runner.ts:242 — it still joins test names with plain spaces, so the separator bug this repo works around (stryker-js#6210) is not fixed upstream and the shim in vitest.mutation.config.mjs is still required.
Checked the interaction: the shim rewrites each literal space in the assigned pattern to (?: > | ); the new \s*$ suffix contains no literal space, so it survives the rewrite, and end-anchoring against Vitest's fullTestName ("suite > test") still matches. The anchor actually helps — it stops a mutant's filter for "calc add" from also running "calc add negative".
The mutation check is path-filtered off on this PR, so CI didn't exercise this; the above is from reading both sources. Looks correct to me. Not approving — that signature belongs to a human.
— hive: agent=reviewer backend=copilot model=claude-fable-5 copilot=1.0.88
…ith 2 updates Bumps the dev-dependencies group with 2 updates in the / directory: [@stryker-mutator/core](https://github.com/stryker-mutator/stryker-js/tree/HEAD/packages/core) and [@stryker-mutator/vitest-runner](https://github.com/stryker-mutator/stryker-js/tree/HEAD/packages/vitest-runner). Updates `@stryker-mutator/core` from 10.0.0 to 10.1.1 - [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.1.1/packages/core) Updates `@stryker-mutator/vitest-runner` from 10.0.0 to 10.1.1 - [Release notes](https://github.com/stryker-mutator/stryker-js/releases) - [Changelog](https://github.com/stryker-mutator/stryker-js/blob/master/packages/vitest-runner/CHANGELOG.md) - [Commits](https://github.com/stryker-mutator/stryker-js/commits/v10.1.1/packages/vitest-runner) --- updated-dependencies: - dependency-name: "@stryker-mutator/core" dependency-version: 10.1.1 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: dev-dependencies - dependency-name: "@stryker-mutator/vitest-runner" dependency-version: 10.1.1 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: dev-dependencies ... Signed-off-by: dependabot[bot] <support@github.com>
94f0152 to
de131d0
Compare
There was a problem hiding this comment.
Looks correct to me — dev-dependencies only, checks green, lockfile consistent with the package.json ranges.
One interaction worth knowing about, verified rather than guessed: this bumps @stryker-mutator/vitest-runner 10.0.0→10.1.1, and 10.1.1 ships the fix for stryker-mutator/stryker-js#6210 — the test-name-separator bug that vitest.mutation.config.mjs (the stryker-vitest5-test-name-separator plugin) works around. In 10.1.1 the runner builds testNamePattern from the " > "-joined full test name with a \s*$ anchor (packages/vitest-runner/src/vitest-test-runner.ts:303-308 at v10.1.1), so the shim's space→(?: > | ) rewrite now operates on patterns that already contain " > ". I checked the combination: the rewritten regex still matches the vitest-5 fullTestName (backtracking through the alternation), so the nightly mutation job — which is skipped on this PR and will first exercise 10.1.1 after merge — should keep selecting tests correctly.
Suggestion (not a defect): once this lands, the shim and its "unfixed in 10.0.0, until that fix ships" comment in vitest.mutation.config.mjs can be retired in a follow-up, confirmed against a nightly mutation run on 10.1.1.
— hive: agent=reviewer backend=copilot model=claude-fable-5 copilot=1.0.88
Bumps the dev-dependencies group with 2 updates in the / directory: @stryker-mutator/core and @stryker-mutator/vitest-runner.
Updates
@stryker-mutator/corefrom 10.0.0 to 10.1.1Release notes
Sourced from @stryker-mutator/core's releases.
Changelog
Sourced from @stryker-mutator/core's changelog.
Commits
9591277v10.1.1d6148ffv10.1.0c931a93fix(core): prevent test runners racing to populate shared caches on startup (...7284a4afeat(core): report mutantdurationin the mutation report (#6252)8419413chore(deps): update dependency prettier to v3.9.4 (#6089)aa7f3bbfeat(core): cli option for command runner (#6006)201dce8feat: add opt-in mutation-phase test timing diagnostics with sidecar reporter...61c7a59chore(deps): update dependency@types/semverto v7.8.0 (#6165)Updates
@stryker-mutator/vitest-runnerfrom 10.0.0 to 10.1.1Release notes
Sourced from @stryker-mutator/vitest-runner's releases.
Changelog
Sourced from @stryker-mutator/vitest-runner's changelog.
Commits
9591277v10.1.1d6148ffv10.1.0cf5695efix(vitest-runner): report mutant collection failures instead of surviving mu...3073ea2feat(vitest-runner): support vitest pool options (#6268)6ad4adbfix(vitest-runner): do not select tests whose name only starts with a filtere...10e2ccafeat(vitest-runner): support vitest v5 (#6214)7284a4afeat(core): report mutantdurationin the mutation report (#6252)d763db3chore(deps): update dependency vite to v8.3.2 (#6091)8419413chore(deps): update dependency prettier to v3.9.4 (#6089)201dce8feat: add opt-in mutation-phase test timing diagnostics with sidecar reporter...