code-reviewer

コードレビュアー

プルリクエストの複雑さとリスクを分析し、ソースコードからコードスメルやSOLID原則違反をスキャンして、判定、スコア、優先順位付きアクション項目を含む構造化されたレビューレポートに結果をまとめる、自動コードレビューツールです。TypeScript、JavaScript、Python、Go、Swift、Kotlinに対応しています。

主な機能

• PRリスク分析 — Gitの差分をスキャンして、ハードコードされたシークレット、SQLインジェクション、デバッグ文、リンターの回避、any型、TODO/FIXMEを検出し、複雑さを1〜10のスコアで評価します。
• コード品質チェック — 固定しきい値に基づいて、長い関数、大きなファイル、ゴッドクラス、深いネスト、過剰なパラメーター、高い循環的複雑度、エラーハンドリングの不足、未使用のインポート、マジックナンバーを検出します。
• SOLID原則違反とアンチパターンの検出 — 構造、ロジック、セキュリティ、パフォーマンス、テスト、非同期処理に関するアンチパターンを修正案とともに指摘します。
• レビューレポート — PRと品質チェックの結果を統合し、判定（承認／変更を要求／ブロック）、0〜100のスコア、優先順位付きのアクション項目を、テキスト、Markdown、JSON形式で出力します。
• レビュー順の優先順位付け — セキュリティ上重要なコードから確認できるように、ファイルを並べ替えます。
• コミットの規約チェック — ブランチ全体でConventional Commits形式を検証します。

使用する場面

• PRをレビューする際、手動でロジックを確認する前に迅速な事前チェックが必要な場合。
• コード品質を分析したり、コードベースを監査してコードスメルやSOLID原則違反を調べたりする場合。
• 構造化されたレビューレポートやレビューチェックリストを作成する場合。
• 品質スコアやレビュー判定に基づいて、CI/CDでマージを制御する場合。

最初に確認する項目

レビューを作成する前に、以下の入力を確認してください。不明または曖昧な項目があれば、推測せずに質問してください。

• レビュー対象 — Gitの差分／ブランチ（baseとhead）、またはソースディレクトリ（--base／--headに設定する値、またはアナライザーがスキャンするパス）
• 言語 — TypeScript、JavaScript、Python、Go、Swift、Kotlin（--languageを介してcode_quality_checker.pyが適用するルールを選択します）
• 出力とゲート — レポート形式（text/markdown/JSON）と合格／不合格のスコアしきい値（成果物の形式とCI判定を設定します）

確認事項のルール：出力に最も影響する2〜3項目だけを質問してください。ユーザーが「そのまま下書きを作成して」と言った場合は作業を進め、成果物の冒頭に前提条件を列挙してください。

ツール

ツール	目的	コマンド
pr_analyzer.py	Gitの差分からPRの複雑さとリスクを評価します	python scripts/pr_analyzer.py . --base main --head feature-branch --json
code_quality_checker.py	ソースコードのコードスメルとSOLID原則違反を検出します	python scripts/code_quality_checker.py ./src --language typescript --json
review_report_generator.py	PRと品質チェックの結果をまとめてレビューレポートを作成します	python scripts/review_report_generator.py . --format markdown --output review.md

参考資料

タスクに合った参考資料を読み込んでください。このファイルは簡潔に保ち、必要に応じて詳細を確認してください。

• references/tool_reference.md — 各ツールの詳しい使い方、検出項目、しきい値／判定テーブル、すべてのCLIフラグ、JSON出力例、対応言語の拡張子、トラブルシューティングの表、成功基準を掲載しています。スクリプトの実行、設定、デバッグ時に参照してください。
• references/code_review_checklist.md — レビュー前、正確性、セキュリティ、パフォーマンス、保守性、テスト、言語固有のチェック項目を体系的にまとめたチェックリストです。手動レビューを行う際に参照してください。
• references/coding_standards.md — TypeScript、JavaScript、Python、Go、Swift、Kotlinの言語固有の標準（型、null安全性、非同期処理、エラーハンドリング、クラス設計）を掲載しています。スタイルや慣用的な書き方への準拠を評価する際に参照してください。
• references/common_antipatterns.md — アンチパターンのカタログ（構造、ロジック、セキュリティ、パフォーマンス、テスト、非同期処理）を、例、検出の手がかり、修正案とともに掲載しています。指摘されたコードスメルを説明または修正する際に参照してください。

対象範囲と制限事項

対象:

• git diff における静的なパターンベースのリスク検出（シークレット、SQL インジェクション、デバッグ文、リンターのバイパス）
• 構造的なコード品質分析：関数の長さ、クラスの規模、循環的複雑度、パラメーター数、SOLID 原則違反
• PR メタデータの評価：リスク優先度に基づくファイル分類、コミットメッセージの検証、複雑度スコアリング
• テキスト、markdown、JSON 形式での判定、スコア、優先順位付きアクション項目を含む統合レビュー報告

対象外:

• ランタイム分析または動的分析 -- テストの実行には senior-qa、エンドツーエンドテストには qa-browser-automation を使用してください
• セキュリティ脆弱性スキャン（CVE データベース、依存関係の監査） -- SAST/DAST およびサプライチェーン分析には senior-security または senior-secops を使用してください
• パフォーマンスのプロファイリングまたはベンチマーク -- パフォーマンス最適化のワークフローには senior-backend または senior-fullstack を使用してください
• アーキテクチャレベルのレビュー（システム設計、サービス境界、API コントラクトの検証） -- アーキテクチャ決定記録と設計レビューには senior-architect を使用してください

統合ポイント

スキル	統合内容	データフロー
senior-security	PR Analyzer の重大な検出結果をセキュリティレビューワークフローに渡し、より詳細な SAST/DAST 分析を行います	pr_analyzer.py --json の出力 risks.critical[] → セキュリティ評価の入力
senior-qa	レビュー報告の判定に基づいてテスト実行を制御し、判定が block の場合はテストスイートをブロックします	review_report_generator.py --json の出力 summary.verdict → QA パイプラインのゲート
senior-architect	複雑度の高い PR（スコア 7 以上）をアーキテクチャレビューにエスカレーションします	pr_analyzer.py の出力 summary.complexity_score → アーキテクチャレビューのトリガー
senior-fullstack	コード品質スコアをフルスタック品質アナライザーと組み合わせ、プロジェクトのエンドツーエンドの健全性を評価します	code_quality_checker.py --json の出力 → code_quality_analyzer.py のメトリクスと統合
tdd-guide	レビューの検出結果をテストカバレッジと照合し、品質チェッカーが指摘した未テストのコードパスにフラグを付けます	ファイルごとの品質チェッカーの smells[] → TDD カバレッジのギャップ分析
senior-devops	レビュー報告を CI/CD パイプラインに統合し、自動化された品質ゲートとして使用します	review_report_generator.py --json の出力 summary.score → パイプラインの合格／不合格のしきい値