implement-review

Compare original and translation side by side

🇺🇸

Original

English

🇨🇳

Translation

Chinese

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

✅ 問題なし

✅ 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

✅ 問題なし

✅ 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