-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Labels
priority:highHigh priorityHigh prioritysecuritySecurity vulnerability fixesSecurity vulnerability fixes
Description
Problem
npm audit reports vulnerabilities in the Vitest 3.x dependency chain. Vitest 4.x resolves these.
Scope
- Upgrade
vitest,@vitest/coverage-v8,@vitest/uifrom^3.2.4to^4.x - Review Vitest 4 migration guide for breaking changes
- Update
vitest.config.tsif config format changed (especiallymemoryLimit, worker pool,singleThread) - Run all test groups and verify they pass:
npm run test:corenpm run test:handlersnpm run test:repositoriesnpm run test:adaptersnpm run test:implementationsnpm run test:clinpm run test:integrationnpm run test:worker-handler
- Verify
npm auditis clean (0 vulnerabilities) - Verify
npm run buildsucceeds
Risk
Semver major upgrade — config format, worker pool behavior, and API surface may change. The memoryLimit and singleThread options are critical for our memory management strategy.
Acceptance Criteria
- All test groups pass
-
npm auditreports 0 vulnerabilities -
npm run buildsucceeds - Memory management still effective (workers restart at threshold)
Dependencies
- Depends on NPM_TOKEN rotation and release.yml removal (need green CI to verify)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
priority:highHigh priorityHigh prioritysecuritySecurity vulnerability fixesSecurity vulnerability fixes