Skip to content

[自動レビュー 2026-06-25] 日次コードレビューによる修正 - #12

Draft
stewroux wants to merge 1 commit into
mainfrom
claude/keen-fermi-lnqr3o
Draft

[自動レビュー 2026-06-25] 日次コードレビューによる修正#12
stewroux wants to merge 1 commit into
mainfrom
claude/keen-fermi-lnqr3o

Conversation

@stewroux

Copy link
Copy Markdown
Owner

概要

2026-06-25 の日次コードレビュー・セキュリティスキャンによる自動修正PRです。

修正内容

✅ CI: Node.js 18 → 20 にアップグレード (closes #10)

Node.js 18 は 2025年4月30日に EOL を迎えました。Node.js 20 LTS (Active) に更新。

# .github/workflows/ci.yml
- node-version: '18'
+ node-version: '20'

.env.example 追加

必要な環境変数を文書化。セキュリティの観点から GEMINI_API_KEY(サーバーサイド)推奨を注記。

✅ Claude Code セッションスタートフック追加

  • .claude/hooks/session-start.shnpm ci + lint を自動実行
  • .claude/settings.json — フックを登録

関連Issue(手動修正が必要)

Issue 深刻度 内容
#8 🔴 HIGH NEXT_PUBLIC_GEMINI_API_KEY がクライアントバンドルに露出 → API Routeへの移行を推奨
#9 🟡 MEDIUM APIキーを localStorage に保存 → XSSリスク、sessionStorage または API Route化を推奨
#10 🟡 MEDIUM CI Node.js 18 EOL → 本PRで修正済み
#11 🟠 LOW 空 catch ブロック + ズームボタン未実装スタブ

テスト計画

  • npm ci && npm run lint が通ること
  • npm run build が通ること
  • ブラウザで画像翻訳が正常に動作すること

🤖 Generated by daily code review agent (claude-sonnet-4-6)


Generated by Claude Code

… hook

- ci.yml: Node 18 is EOL since Apr 2025, upgrade to Node 20 LTS (fixes #10)
- .env.example: document required environment variables
- .claude/hooks/session-start.sh: install deps and run lint on session start
- .claude/settings.json: register session-start hook
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.

[CODE REVIEW] Node.js 18 in CI is EOL (since April 2025) — upgrade to Node 20

1 participant