mcpField
C
code-review

code-review

github.com/langchain-aiテスト・品質30.8k213 回インストール今日更新
リポジトリ

code-review について

開発者やコーディングエージェントが、変更を提出する前に確認するためのコードレビュー用スキルです。正確性、品質、テスト、安全性のチェックリストに加え、手順と補助 lint スクリプトを備えています。

主な機能

  1. 01正確性とエッジケースのチェックリスト
  2. 02コード品質のレビュー
  3. 03テストカバレッジと実行手順
  4. 04安全性と入力検証の確認
  5. 05Python 用 lint スクリプト

活用シーン

  1. 01提出前のコード変更レビュー
  2. 02テストカバレッジとエラーケースの確認
  3. 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、コマンドインジェクションにつながる要因がありません
  • ファイル操作で安全なパスを使用しています

手順

  1. 変更した各ファイルを最初から最後まで読みます(差分だけではなく)
  2. テストスイートを実行します:execute("python -m pytest -v")
  3. 利用可能な場合はリンターを実行します:execute("ruff check .")
  4. 同梱されている lint チェックを実行します:execute("python /skills/code-review/lint_check.py .")
  5. レビューチェックリストの各項目に照らして確認します
  6. 問題が見つかった場合は修正し、再レビューします
  7. すべて合格したら、レビュー完了です

ヘルパースクリプト

  • /skills/code-review/lint_check.py — Python ファイルをスキャンし、ドックストリングの欠落、長い関数(50 行超)、裸の except: 節を検出します。execute("python /skills/code-review/lint_check.py [path ...]") で実行します。

コメント