pr-review

PR レビュースキル

リポジトリの規約に照らしてプルリクエストをレビューします。2 段階のプロセスです。自動検証の後、手動で内容をレビューします。

フェーズ 1：自動検証（必須ルール）

検証スクリプトを実行して、構造上の要件を確認します。

python .claude/skills/pr-review/scripts/validate_skills.py

スクリプトでは次の項目を確認します。

• すべてのスキルディレクトリに SKILL.md が存在する
• YAML フロントマターを解析できる
• 必須フィールド（name、description）が存在する
• name がディレクトリ名と一致する
• ハードコードされたシークレットが検出されない

ERROR レベルのチェックはすべて合格する必要があります。WARNING レベルの項目（license、metadata の欠落）は指摘する必要がありますが、ブロッカーではありません。

必須ルールの完全な仕様については、references/structure-rules.md を参照してください。

フェーズ 2：内容レビュー（推奨ガイドライン）

自動チェックに合格した後、品質ガイドラインに照らしてプルリクエストをレビューします。

1. スキルの範囲 — 既存のスキルと重複していませんか？境界は明確ですか？
2. 説明の品質 — description に明確なトリガー条件が含まれていますか？
3. ファイルサイズ — 参照ドキュメントはコンテキストウィンドウでの利用に適したサイズですか？
4. API キーの取り扱い — 外部 API を使用する場合、認証情報は環境変数から読み込まれていますか？
5. スクリプトの品質 — スクリプトに shebang、requirements.txt、エラー処理が含まれていますか？
6. 言語 — SKILL.md とコードは英語で記述されていますか？
7. README の同期 — 新しいスキルを追加する際、README.md と README_zh.md が更新されていますか？

推奨ガイドラインの詳細については、references/quality-guidelines.md を参照してください。

レビューチェックリストの概要

必須（ブロッカー）

• validate_skills.py が終了コード 0 で終了する
• PR タイトルが Conventional Commits の形式に従っている
• 1 つの PR に 1 つの目的がある

合格が望ましい（レビューで指摘）

• 既存のスキルと機能が重複していない
• 説明にトリガー条件が含まれている
• ファイルサイズが適切である
• API キーが環境変数経由で使用されている
• 新しいスキルについて README の表が更新されている（Source 列が Community に設定されている）