Addy Osmani の agent-skills リポジトリにあるコードレビュースキル:マージ前(または他のエージェントや他人が書いたコードをレビューする際)に、エージェントへ正しさ、可読性、アーキテクチャ、セキュリティ、パフォーマンスの5つの観点で変更点をレビューさせます。まずテストを確認してから実装を見、各指摘事項に深刻度(Critical / 必須修正 / Nit / Optional / FYI)を付け、最終的に承認または修正要求の結論を出します。
skills.sh インストール数 79294(+975 / 24時間ごと)
セキュリティ監査:未監査
- まだスキャンされていません
理由
- プロセスは 5 ステップに分かれています。まず変更の意図を明確にし、次にテストを先にレビューし、ファイルごとに5つの観点で実装をレビューし、問題点を深刻度別に列挙し、作成者の検証説明を確認します。レポート作成前に呼び出し側、ガード、テストと照合して誤検出を排除するよう求めており、問題ゼロも正当な結論として許容されますが、無理に問題を作り出すことは禁止されています。
- テストの有効性は実験で判断します。変更内の条件を1つ反転させてテストを実行し、コピーからファイルを復元します。テストが依然としてパスする場合はケース不足を示すため、どのケースが欠けているかを指摘する必要があります。
- 変更規模の目安(約 100 行なら一度でレビュー可能、約 1000 行なら分割すべき)と4種類の分割方法を提供しています。依存関係のアップグレードでは更新ログを読み、一度に1パッケージのみアップグレードし、ロックファイルの差分をレビューします。
- 不要なコードを発見した場合は、まずリストを作成して削除するか確認し、勝手に削除しません。構造上の問題を提起する際は、具体的なリファクタリング方法も併せて提示します。
- リポジトリは MIT ライセンスで、同じリポジトリには他に 24 個のスキルがあります。skills.sh では 24 時間でインストール数 975、累計 79294 に達し、直近 8 週間の週間インストール数は 3503 から 5359 へ増加しています(これは注目度を示すに過ぎません)。
使い方
- インストールコマンドは監査合格後に表示されます。「この PR / diff をレビューして」と言う場合、機能実装完了後のマージ準備時、または他のエージェントが書いたコードを評価したい場合にトリガーされます。
- アカウントや外部サービスは不要です。「条件を反転させてテストを実行する」ステップでは、ローカルでプロジェクトのテストを実行できる環境が必要です。
リスク
- 本号では未監査のため、固定で C ランクとなり、インストールコマンドは表示されません。SKILL.md は主にレビューチェックリストであり、唯一ファイルを変更するのは「一時的に条件を1つ反転させてテストを実行し、コピーから復元する」処理ですが、これは用途と一致しています。ただし、ファイルが正しく復元されたことを確認する必要があります。skills.sh 上では Gen Agent Trust Hub と Socket の監査は Pass、Snyk は Warn です。
- セキュリティおよびパフォーマンスに関する部分は、同じリポジトリ内の security-and-hardening、performance-optimization スキルや、リポジトリルートディレクトリのチェックリストファイルを参照しています。このスキルのみをインストールした場合、これらのリンクが読み取れない可能性があります。
- 承認基準はやや緩やかです:コードの健全性全体が改善されていれば承認されます。「1営業日以内に返信する」といったチーム規範は、個人プロジェクトではあまり意味がありません。