fix-review について
監査指摘への修正コミットが問題を解消し、新たな不具合や脆弱性を招いていないか確認するコードレビュースキルです。変更と指摘を照合し、関連箇所を調べ、対象コミットに結び付いた根拠を記録します。
主な機能
- 01修正と監査指摘の照合
- 02根本原因への対処確認
- 03不具合や脆弱性の確認
- 04テスト範囲のレビュー
- 05コミットに基づく記録
活用シーン
- 01セキュリティ修正コミットのレビュー
- 02監査指摘が完全に解消されたかの確認
- 03同じ根本原因を持つ関連箇所の調査
- 04未検証のテストと制約の記録
インストール
コマンドをコピーしてターミナルで実行します。
$
npx skills add https:/ / github.com/ sickn33/ agentic-awesome-skills --skill fix-reviewインストール後、Claude Code で/fix-reviewを入力して読み込まれたか確認します。
詳細
修正のレビュー
概要
修正コミットが監査で指摘された問題に適切に対処し、新たなバグやセキュリティ上の脆弱性を生じさせていないことを確認します。
このスキルを使用する場面
修正コミットが、監査で指摘された問題に新たなバグを生じさせることなく対処しているか確認する必要がある場合に、このスキルを使用します。
次のような場合に使用します:
- セキュリティ監査で指摘された問題に対処するコミットをレビューする
- 修正によって新たな脆弱性が生じていないことを確認する
- コードの変更によって特定された問題が適切に解決されていることを確認する
- 問題への対処が完全かつ正確であることを検証する
手順
このスキルは、修正コミットが監査で指摘された問題に適切に対処しているか確認するのに役立ちます:
- 修正コミットをレビューする:監査で指摘された問題を修正したとするコミットを分析します
- 解決を確認する:元の問題が適切に解決されていることを確認します
- リグレッションを確認する:新たなバグや脆弱性が生じていないことを確認します
- 完全性を検証する:指摘事項のすべての側面が解決されていることを確認します
レビュープロセス
修正コミットをレビューする際は、次の手順に従います:
- 修正を元の監査指摘と比較します
- 症状だけでなく、根本原因に対処していることを確認します
- 潜在的な副作用や新たな問題がないか確認します
- テストが修正対象のシナリオを網羅していることを検証します
- 同じ根本原因に関係する到達可能な経路を検索し、確認した範囲と未検証の範囲を記録します
証拠記録
レビューの対象を、正確な完全版の head SHA、元の指摘事項、影響を受ける範囲に紐付けます。修正前のトリガー、修正後の動作、実際に実行したテスト、未解決の派生ケース、制約を記録します。実行できないテストは合格ではなく、未検証です。この指示だけでは、リポジトリの Tessl またはメンテナーのマージゲートを満たしません。
ベストプラクティス
- コードベース全体の状況を踏まえて修正をレビューします
- 修正対象の問題がテストで網羅されていることを確認します
- 修正が必要な類似パターンがないか確認します
- セキュリティのベストプラクティスに従って修正されていることを確認します
- 解決方法を記録します
リソース
詳細については、ソースリポジトリを参照してください。
例
ユーザーのリクエスト:
セキュリティ監査で指摘された問題に対処するコミットをレビューしてください。
制限事項
- このスキルは、タスクが上記の説明範囲に明確に該当する場合にのみ使用してください。
- 出力を、環境固有の検証、テスト、または専門家によるレビューの代わりとして扱わないでください。
- 必要な入力、権限、安全上の境界、または成功基準が不足している場合は、作業を中断して確認を求めてください。
コメント
Markdown 対応