code-review-and-qualitycommit 28f435eSKILL.md · コードレビュー
多軸コードレビュー
正しさ・可読性・アーキテクチャ・セキュリティ・性能の5軸で変更をレビューし、各指摘に重要度を付けます。
npx skills add addyosmani/agent-skills --skill code-review-and-qualityコマンドを選択しました。手動でコピーしてください。
- 権限
- read-repo
- write-files
- shell
- network
- 対応ツール
- Claude Code, Codex, Cursor, Copilot, Gemini CLI
- GitHub
- 112.3k
- 提供元
- コミュニティ · Addy Osmani
- 状態
- スキャン済み
- ライセンス
- MIT
- カテゴリ
- コードレビュー
基本ルール
変更が全体のコード健全性を確実に高めるなら、完璧でなくても承認すること。
実行フェーズ
フェーズ 01背景を理解
コードを見る前に、変更の目的、対応する仕様やタスク、期待される振る舞いの変化を把握します。
フェーズ 02テストから確認
テストが振る舞いと境界を検証しているか確認し、条件を1つ反転してテストを実行後に元に戻します。通ったままならテスト不足です。
フェーズ 035軸で実装を確認
ファイルごとに5軸で確認し、構造上の問題には具体的な改善手段を示します。
フェーズ 04重要度を付ける
Critical、Nit、Optional、FYI、または接頭辞なしで指摘を分類し、正しさとセキュリティの問題を先に書きます。
フェーズ 05検証内容を確認
実行したテスト、ビルドの成否、手動確認の有無を確かめ、不要コードは一覧にして削除前に確認します。
権限の説明（SKILL.md より）
| 機能 | 必要か | 説明 |
|---|---|---|
| ファイル書き込み | 一部で必要 | 変異実験のために条件を一時的に変えてコピーから戻す場合と、確認を得てから不要コードを削除する場合に限ります。 |
| コマンド実行 | 一部で必要 | 変更の検証と変異実験のためにテストスイートとビルドを実行します。 |
| 外部通信 | 不要 | レビューの手順自体にネットワークアクセスは不要です。 |
SKILL.md のフロントマター（YAML）
---
name: code-review-and-quality
description: Conducts multi-axis code review. Use before merging any change. Use when reviewing code written by yourself, another agent, or a human. Use when you need to assess code quality across multiple dimensions before it enters the main branch. Use when asked to review a diff or a pull request, even when the diff is pasted inline.
---関連スキル
ルールスキャンですべてのリスクを検出できるわけではありません。インストール前にこのスキルの SKILL.md を読み、権限を確認してください。