coding-style-guide-review

コーディングガイドラインレビュー基準

プロジェクト固有のコーディングガイドラインに基づいてコードレビューを行う。

ガイドラインの探索

レビュー前に、対象リポジトリのルートから以下のファイルを Glob で探索し、見つかったものを Read で読み込んでプロジェクト固有の規約を把握する。

探索対象ファイル

• CLAUDE.md
• CONTRIBUTING.md / .github/CONTRIBUTING.md
• CONVENTIONS.md / CODING_STANDARDS.md / CODING_GUIDELINES.md / STYLE_GUIDE.md
• code-review.md / coding-style-guide.md
• docs/ 配下のガイドライン関連ドキュメント

探索手順

1. Glob で上記ファイルの存在を確認
2. 見つかったファイルを Read で読み込み、規約を抽出
3. 規約が見つからない場合は、既存コードからパターンを推測（後述）

レビュー観点

命名規約

• 変数・関数・クラス・ファイル名の命名規則の一貫性
• プロジェクト固有の命名パターンへの準拠
• 言語ごとの慣習（camelCase, snake_case, PascalCase等）の統一

ディレクトリ・ファイル構成

• プロジェクトのディレクトリ構造規約への準拠
• ファイル配置の妥当性（適切なディレクトリに配置されているか）
• モジュール分割の粒度

インポート・依存関係

• インポート順序の規約準拠
• 不要なインポートの検出
• 循環依存の回避

コメント・ドキュメンテーション

• ドキュメンテーションコメント（JSDoc, docstring, GoDoc等）の規約準拠
• コメントの言語（日本語/英語）の統一
• TODO/FIXME等のタグ規約

エラーハンドリング

• プロジェクト固有のエラーハンドリングパターンへの準拠
• エラーメッセージの形式統一
• ログ出力の規約準拠

API設計

• API設計規約（URL命名、HTTPメソッド使用等）への準拠
• レスポンス形式の統一
• エラーレスポンスの形式統一

Git規約

• コミットメッセージの形式（Conventional Commits等）
• ブランチ命名規約

ガイドラインが見つからない場合

明示的なガイドラインファイルが存在しない場合:

1. 既存コードから暗黙的な規約を推測する
   • 既存ファイルの命名パターンを Glob で確認
   • 既存コードのスタイルを Read / Grep で確認
2. 推測した規約との不一致を「Low」重要度で報告
3. レビュー結果に「明示的なガイドラインなし（既存コードから推測）」と明記