Skip to content

@types/nodeをLTSバージョンに固定しhonoを更新#119

Merged
poad merged 1 commit into
mainfrom
fix/vulnerability
Feb 22, 2026
Merged

@types/nodeをLTSバージョンに固定しhonoを更新#119
poad merged 1 commit into
mainfrom
fix/vulnerability

Conversation

@poad

@poad poad commented Feb 22, 2026

Copy link
Copy Markdown
Owner

概要

依存パッケージの脆弱性対応として、@types/node を LTS バージョンに固定し、hono を最新版に更新しました。

変更点

追加・変更・削除したファイル (リポジトリルートからの相対パス) 変更内容 事由
.github/dependabot.yml @types/node の奇数バージョン(25.x, 27.x, 29.x)を無視する設定を追加 非LTSバージョンの自動更新を防止し、安定したLTSバージョンのみを使用するため
example/package.json @types/node を 25.3.0 から ^24.10.13 に変更 LTS バージョン (Node.js 24) に固定するため
package/package.json @types/node を 25.3.0 から ^24.10.13 に変更、hono を ^4.12.0 から ^4.12.1 に更新 LTS バージョンへの固定および最新版への更新
pnpm-lock.yaml ロックファイルを更新 依存関係の変更を反映
pnpm-workspace.yaml 不要な overrides(@eslint/js, eslint, ajv など)を削除 依存関係が解決されたため不要になった

確認事項

  • (Typescriptの場合) pnpm audit --fix で脆弱性を修正済みか?
  • (Typescriptの場合) pnpm lint-fix でコードスタイルは修正済みか?
  • (Markdownの場合)npx -y markdownlint-cli2@latest . --fix で Markdown の lint は修正済みか?

特記事項

  • Node.js の奇数バージョンは非LTSリリースであるため、dependabot で自動更新されないように設定しました
  • 今回の変更により、依存関係の脆弱性が解消されています

@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 1 package(s) with unknown licenses.
See the Details below.

License Issues

package/package.json

PackageVersionLicenseIssue Type
hono^4.12.1NullUnknown License

OpenSSF Scorecard

PackageVersionScoreDetails
npm/@types/node ^24.10.13 UnknownUnknown
npm/hono 4.12.1 UnknownUnknown
npm/@types/node ^24.10.13 UnknownUnknown
npm/hono ^4.12.1 UnknownUnknown
npm/@types/node 24.10.13 🟢 6.8
Details
CheckScoreReason
Maintained🟢 1030 commit(s) and 1 issue activity found in the last 90 days -- score normalized to 10
Code-Review🟢 7Found 23/30 approved changesets -- score normalized to 7
Packaging⚠️ -1packaging workflow not detected
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Security-Policy🟢 10security policy file detected
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
License🟢 9license file detected
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
Vulnerabilities🟢 100 existing vulnerabilities detected
Binary-Artifacts🟢 10no binaries found in the repo
Pinned-Dependencies🟢 8dependency not pinned by hash detected -- score normalized to 8
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
Fuzzing⚠️ 0project is not fuzzed
npm/hono 4.12.1 UnknownUnknown
npm/undici-types 7.16.0 🟢 8.3
Details
CheckScoreReason
Maintained🟢 1030 commit(s) and 18 issue activity found in the last 90 days -- score normalized to 10
Security-Policy🟢 9security policy file detected
Dependency-Update-Tool🟢 10update tool detected
Code-Review🟢 10all changesets reviewed
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Binary-Artifacts🟢 8binaries present in source code
Pinned-Dependencies🟢 4dependency not pinned by hash detected -- score normalized to 4
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Vulnerabilities🟢 100 existing vulnerabilities detected
Signed-Releases⚠️ -1no releases found
Fuzzing🟢 10project is fuzzed
License🟢 10license file detected
Packaging🟢 10packaging workflow detected
SAST🟢 9SAST tool detected but not run on all commits
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
CI-Tests🟢 1030 out of 30 merged PRs checked by a CI test -- score normalized to 10
Contributors🟢 10project has 80 contributing companies or organizations

Scanned Files

  • example/package.json
  • package/package.json
  • pnpm-lock.yaml

@amazon-q-developer amazon-q-developer 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.

レビュー完了

このPRは、依存関係のセキュリティ対応として@types/nodeをLTSバージョンに固定し、honoを最新版に更新する変更です。コードレビューの結果、マージをブロックする重大な欠陥は見つかりませんでした。

変更内容の確認

@types/node: 25.3.0(非LTS)→ ^24.10.13(LTS)への変更は適切です
hono: 4.12.0 → 4.12.1 へのパッチ更新は問題ありません
dependabot設定: 非LTSバージョン(25.x, 27.x, 29.x)の自動更新を防止する設定は適切です
pnpm-workspace.yaml: 解決済みのoverridesを削除し、必要なminimatchのoverrideのみを保持しています

すべての変更は、PRの目的である脆弱性対応とLTSバージョンへの固定という目標に沿っています。


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

@poad
poad marked this pull request as ready for review February 22, 2026 06:32
@poad
poad merged commit 54849c3 into main Feb 22, 2026
5 checks passed
@poad
poad deleted the fix/vulnerability branch February 22, 2026 06:33
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.

1 participant