Skip to content

chore: vitest ツールチェーン更新で audit 脆弱性を解消#1

Merged
kyosuke merged 2 commits into
mainfrom
chore/bump-vitest-deps
Jun 5, 2026
Merged

chore: vitest ツールチェーン更新で audit 脆弱性を解消#1
kyosuke merged 2 commits into
mainfrom
chore/bump-vitest-deps

Conversation

@kyosuke

@kyosuke kyosuke commented Jun 5, 2026

Copy link
Copy Markdown
Owner

概要

npm audit が報告した 6 件の脆弱性(vite パストラバーサル / ws メモリ開示 / postcss XSS)を解消する。いずれも vitest と @cloudflare/vitest-pool-workers 経由の推移的依存だったため、直接依存を更新して対応した。

変更内容

パッケージ
vitest ^4.1.2 ^4.1.8
@cloudflare/vitest-pool-workers ^0.14.1 ^0.16.13
wrangler ^4.80.0 ^4.98.0

@cloudflare/vitest-pool-workers の 0.14→0.16 は破壊的変更扱いだが、peer の vitest ^4.1.0 要件を満たしている。

確認

  • npm audit: 0 vulnerabilities
  • npm run test:node: 11 passed
  • npm run test:workers: 4 passed

🤖 Generated with Claude Code

vite/ws/postcss の脆弱性は vitest と vitest-pool-workers 経由の推移的依存
だったため、直接依存を更新して解消する。pool-workers の 0.14→0.16 は
破壊的変更扱いだが peer の vitest ^4.1.0 を満たし、全テストが通過する。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

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

Copy link
Copy Markdown

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 updates several development dependencies, including @cloudflare/vitest-pool-workers, vitest, and wrangler. Feedback on the changes highlights that these dependency updates raise the required Node.js version to >=22.0.0 in the lockfile, which conflicts with the "engines": { "node": ">=18" } configuration currently specified in package.json. It is recommended to update the Node.js engine requirement in package.json to >=22 to maintain consistency and prevent installation or runtime issues in older Node.js environments.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread package.json

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Vitest + Cloudflare Workers testing toolchain dependencies to eliminate transitive npm audit vulnerabilities reported via vitest / @cloudflare/vitest-pool-workers (and related packages like wrangler, miniflare, vite, ws, postcss).

Changes:

  • Bump vitest from ^4.1.2 to ^4.1.8.
  • Bump @cloudflare/vitest-pool-workers from ^0.14.1 to ^0.16.13.
  • Bump wrangler from ^4.80.0 to ^4.98.0 and refresh the lockfile accordingly.

Reviewed changes

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

File Description
package.json Updates direct devDependencies for Vitest + Workers test tooling.
package-lock.json Updates the resolved dependency graph to match the new toolchain versions and remove audited vulnerabilities.

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

Comment thread package.json
Node 18 は EOL、Node 20 もメンテナンス終了済みで、現在維持されている
最古の LTS は v22。devDeps(wrangler/miniflare/kv-asset-handler)も
node>=22 を要求しており、サポート対象 Node を実態に合わせる。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@kyosuke kyosuke merged commit 8e9c4a7 into main Jun 5, 2026
1 check passed
@kyosuke kyosuke deleted the chore/bump-vitest-deps branch June 5, 2026 14:43
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.

2 participants