code-review-and-quality(5次元コードレビューと品質ゲート)

Addy Osmani の agent-skills リポジトリにあるコードレビュースキル:マージ前(または他のエージェントや他人が書いたコードをレビューする際)に、エージェントへ正しさ、可読性、アーキテクチャ、セキュリティ、パフォーマンスの5つの観点で変更点をレビューさせます。まずテストを確認してから実装を見、各指摘事項に深刻度(Critical / 必須修正 / Nit / Optional / FYI)を付け、最終的に承認または修正要求の結論を出します。

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営業日以内に返信する」といったチーム規範は、個人プロジェクトではあまり意味がありません。

開く · リポジトリ

Best Skills 每日数据(LinklyAI / @BlueeonY,CC BY 4.0)

最新の AI ニュースを見る