code-review について
開発者やコーディングエージェントが、変更を提出する前に確認するためのコードレビュー用スキルです。正確性、品質、テスト、安全性のチェックリストに加え、手順と補助 lint スクリプトを備えています。
主な機能
- 01正確性とエッジケースのチェックリスト
- 02コード品質のレビュー
- 03テストカバレッジと実行手順
- 04安全性と入力検証の確認
- 05Python 用 lint スクリプト
活用シーン
- 01提出前のコード変更レビュー
- 02テストカバレッジとエラーケースの確認
- 03Python コードの一般的な lint 問題の検査
インストール
コマンドをコピーしてターミナルで実行します。
$
npx skills add https:/ / github.com/ langchain-ai/ deepagents --skill code-reviewインストール後、Claude Code で/code-reviewを入力して読み込まれたか確認します。
同梱ファイルlint_check.py
詳細
コードレビューのスキル
変更を実装した後、提出前に作業内容を検証するためにこのスキルを使用します。
レビューチェックリスト
1. 正確性
- 変更によって元の課題/タスクが解決されています
- 既存の機能に意図しない副作用がありません
- エッジケースに対処しています
- エラー処理が適切です(過剰ではありません)
2. コード品質
- コードが既存のスタイルやパターンに沿っています
- 不必要な複雑さや抽象化がありません
- 変数名と関数名が明確です
- デッドコード、コメントアウトされたコード、TODO が残っていません
3. テスト
- 新しい機能にテストが用意されています
- 既存のテストが引き続き成功します
- 正常系とエラーケースの両方をテストしています
- テストが脆くありません(実装の詳細をテストしていません)
4. 安全性
- ハードコードされた秘密情報や認証情報がありません
- 境界でユーザー入力を検証しています
- SQL インジェクション、XSS、コマンドインジェクションにつながる要因がありません
- ファイル操作で安全なパスを使用しています
手順
- 変更した各ファイルを最初から最後まで読みます(差分だけではなく)
- テストスイートを実行します:
execute("python -m pytest -v") - 利用可能な場合はリンターを実行します:
execute("ruff check .") - 同梱されている lint チェックを実行します:
execute("python /skills/code-review/lint_check.py .") - レビューチェックリストの各項目に照らして確認します
- 問題が見つかった場合は修正し、再レビューします
- すべて合格したら、レビュー完了です
ヘルパースクリプト
/skills/code-review/lint_check.py— Python ファイルをスキャンし、ドックストリングの欠落、長い関数(50 行超)、裸のexcept:節を検出します。execute("python /skills/code-review/lint_check.py [path ...]")で実行します。
コメント
Markdown 対応