Back to Details

implement-review

Compare original and translation side by side

🇺🇸

Original

English
🇨🇳

Translation

Chinese

implement-review

implement-review

コード変更の品質・アーキテクチャ準拠・セキュリティをレビューします。
Review code changes for quality, architecture compliance, and security.

フロー

Flow

Step 1: 変更内容を確認する

Step 1: Confirm the changes

bash
git diff main...HEAD --stat
git diff main...HEAD
ベースブランチはリポジトリの規約に従う（
main
/ 
develop
等）。 または特定のファイル・ディレクトリを対象にする（ユーザーが指定した場合）。
bash
git diff main...HEAD --stat
git diff main...HEAD
Follow the repository's conventions for the base branch (e.g., 
main
/ 
develop
). Or target specific files/directories (if specified by the user).

Step 2: コード品質レビュー

Step 2: Code Quality Review

Agent ツールに委譲してコード品質を確認:
確認項目:
  • アーキテクチャ準拠（コンポーネント階層、配置先）
  • 命名規則
  • Import ルール
  • 不要な再描画リスク
  • props の最小化
  • テストカバレッジ
Delegate to Agent tools to check code quality:
Check items:
  • Architecture compliance (component hierarchy, placement)
  • Naming conventions
  • Import rules
  • Risk of unnecessary re-renders
  • Minimization of props
  • Test coverage

Step 3: セキュリティレビュー（必須）

Step 3: Security Review (Required)

Agent ツールに委譲してセキュリティを確認:
確認項目:
  • OWASP Top 10（XSS, CSRF, インジェクション等）
  • API キー・シークレットのハードコーディング
  • 入力バリデーション（システム境界での検証）
  • 認証・認可の実装
  • 機密データの取り扱い
Delegate to Agent tools to check security:
Check items:
  • OWASP Top 10 (XSS, CSRF, Injection, etc.)
  • Hardcoding of API keys/secrets
  • Input validation (validation at system boundaries)
  • Implementation of authentication/authorization
  • Handling of sensitive data

Step 4: レポートを生成する

Step 4: Generate Report

以下の形式でレポートをまとめる:
undefined
Compile the report in the following format:
undefined

Code Review Results

Code Review Results

✅ 問題なし

✅ No Issues

  • ...
  • ...

⚠️ 要改善（非ブロッキング）

⚠️ Needs Improvement (Non-blocking)

  • src/components/Foo.tsx:42
    — 説明
  • src/components/Foo.tsx:42
    — Description

❌ 要修正（ブロッキング）

❌ Needs Fix (Blocking)

  • src/components/Bar.tsx:10
    — 説明
  • src/components/Bar.tsx:10
    — Description

Security Audit Results

Security Audit Results

✅ 問題なし

✅ No Issues

  • ...
  • ...

❌ セキュリティ問題

❌ Security Issues

  • 深刻度: HIGH/MEDIUM/LOW
  • src/api/endpoint.ts:20
    — 説明
  • 推奨修正: ...
undefined
  • Severity: HIGH/MEDIUM/LOW
  • src/api/endpoint.ts:20
    — Description
  • Recommended Fix: ...
undefined

注意事項

Notes

  • このスキルは読み取り専用のレビューのみ行う（自動修正しない）
  • セキュリティ問題は必ず修正してからマージするよう案内する
  • This skill only performs read-only reviews (no automatic fixes)
  • Guide users to fix security issues before merging