O

requesting-code-review

作成者 obra

タスク完了時や大きな機能の実装後、マージ前に、成果物が要件を満たしているか確認するために使います。

スター0
お気に入り0
コメント0
カテゴリーCode Review
インストールコマンド
npx skills add https://github.com/obra/superpowers --skill requesting-code-review
概要

概要

スキルでできること

requesting-code-review スキルは、変更をマージする前に AI のコードレビュー用サブエージェントにチェックしてもらうための、明確で再現性のあるワークフローを定義します。Git ベースのプロジェクト向けに設計されており、次のようなことに役立ちます。

  • レビューを依頼する タイミング を決める(タスク完了時、機能実装後、マージ前)
  • Git SHA と要件を使って、レビュアーに渡す 的確なコンテキスト をまとめる
  • レビュアーのフォーカスを、あなたのセッション履歴ではなく コードの差分 に絞る
  • フィードバックを重要度別に分類し、今すぐ修正すべきものと後回しにできるものを切り分ける

本質的には、requesting-code-review早く・こまめにレビューする ことを促し、問題がコードベース全体に波及する前に発見するためのスキルです。

想定ユーザー

このスキルは次のような人に向いています。

  • Git を使い、日常的に feature branch や PR を作成している
  • AI によるコードレビューを 構造的かつ再現性のある形 で依頼したい
  • code-reviewer のようなサブエージェント駆動開発を行っている
  • 本番投入前の品質(正しさ、アーキテクチャ、テスト、仕様との整合性)を重視している

特に有用なのは次のようなケースです。

  • しっかりしたセーフティネットが欲しい個人開発者
  • 専任のコードレビュアーがいない小規模チーム
  • コミット履歴と diff が主なソース・オブ・トゥルースになっているプロジェクト

不向きなケース

次のような場合、requesting-code-review は最適ではないかもしれません。

  • Git を使っていない、または commit SHA にアクセスできない
  • 特定の変更のレビュー ではなく、一般的なコード生成やリファクタリング支援を求めている
  • レビュー対象の変更について、明確な計画・仕様・要件を提示できない

このような場合は、レビュー特化のワークフローではなく、より一般的なコーディングや計画用スキルを検討するとよいでしょう。

解決したい課題

一貫したレビュー手順がないと、開発者は次のような問題に陥りがちです。

  • 重要なマイルストーンでレビュー依頼をし忘れる
  • コンテキストをほとんど共有しない(またはセッション履歴を過剰に共有してしまう)
  • フィードバックが構造化されておらず、具体的なアクションに落とし込みづらい

requesting-code-review スキルは、次のようにしてこれらの問題を解決します。

  • 必須・任意のレビューのチェックポイント を定義する
  • レビューの 入力形式(Git のレンジ、要件、サマリ)を標準化する
  • 専用の code-reviewer サブエージェントと組み合わせ、重要度順に整理されたフィードバック を返す

使い方

インストール

obra/superpowers リポジトリから requesting-code-review スキルをインストールするには、Skills CLI を使います。

npx skills add https://github.com/obra/superpowers --skill requesting-code-review

これにより、スキル定義と、code-reviewer サブエージェントのテンプレートを含む関連ファイルが取り込まれます。

インストール後、スキルのディレクトリを開き、まず次のファイルに目を通してください。

  • SKILL.md – コードレビュー依頼用の高レベルな説明とワークフローステップ
  • code-reviewer.md – 実際にコードレビューを行うエージェントのプロンプト/テンプレート

コアワークフローの全体像

requesting-code-review のワークフローは、主に 3 フェーズで構成されます。

  1. レビューのタイミングを決める
    • サブエージェント駆動ワークフローの各タスク完了後
    • 大きな機能を作り終えたタイミング
    • main ブランチにマージする前
  2. レビュー用のコンテキストを準備する
    • BASE_SHAHEAD_SHA を使って Git のコミットレンジを取得する
    • 何を実装し、どう動くべきかを要約する
    • code-reviewer.md に定義されているプレースホルダを埋める
  3. レビューを実行し、フィードバックへ対応する
    • superpowers:code-reviewer サブエージェントを実行する
    • Critical は即時対応、Important は先に片付け、Minor は記録のみにする

ステップ 1: 適切なレビュータイミングを見極める

SKILL.md によると、requesting-code-review は次のような場面で使います。

必須チェックポイント:

  • サブエージェント駆動開発における 各タスク完了後
  • 大きな機能を作り終えた後
  • main ブランチ(例: main, master)へ マージする前

任意だが有用なチェックポイント:

  • 開発が 行き詰まった ときに、新しい視点が欲しい場合
  • リファクタリング前 に、現状の動作をベースラインとして確認したい場合
  • 複雑なバグ修正 の後に、リグレッションや新たな問題がないか確かめたい場合

これらのタイミングでこのスキルを起動する習慣をつけることで、レビューが「後回しの作業」ではなく、自動的なプロセスとして定着します。

ステップ 2: Git のコミットレンジを取得する

レビュアーには、ノイズの少ない明確な diff を渡す必要があります。Git SHA を使ってレンジを指定します。

BASE_SHA=$(git rev-parse HEAD~1)  # or origin/main
HEAD_SHA=$(git rev-parse HEAD)
  • BASE_SHA は、以前のコミットや origin/main の先頭など、あなたの 開始点 を示すコミットを指すようにします。
  • HEAD_SHA は、あなたの 最新の変更 を含むコミットを指します。

ブランチ戦略に合わせて HEAD~1 を変えたり、別のベースを選んでも構いません。重要なのは、レビューしたい変更を正確にカバーするレンジを指定することです。

ステップ 3: レビューリクエストのテンプレートを準備する

code-reviewer.md は、Code Review Agent への指示方法を定義するファイルです。ここには、サブエージェントを実行する前に埋めるべきプレースホルダが用意されています。

主なプレースホルダ:

  • {WHAT_WAS_IMPLEMENTED} – 今回実装・変更した内容の簡潔な説明
  • {PLAN_OR_REQUIREMENTS} – 実装が満たすべき仕様、チケット、ビジネス要件
  • {BASE_SHA} – diff の開始点となるコミット SHA
  • {HEAD_SHA} – diff の終点となるコミット SHA
  • {DESCRIPTION} – 変更セットの短いサマリ(例: "Add verification function and tests for user signups")
  • {PLAN_REFERENCE} – 計画や要件ドキュメントへの参照

自分のツールチェーンの中で、これらのプレースホルダに値を埋めたうえで、Task tool を使って superpowers:code-reviewer サブエージェントを実行します。

ステップ 4: code-reviewer サブエージェントを実行する

Git SHA とテンプレートの準備ができたら、以下の手順で進めます。

  1. superpowers framework で説明されているように、オーケストレーション環境の Task tool を使い、type に superpowers:code-reviewer を指定します。

  2. プレースホルダを実際の値に置き換えた code-reviewer.md の内容を渡します。

  3. エージェントが指定レンジの Git diff にアクセスできるようにします。

    git diff --stat {BASE_SHA}..{HEAD_SHA}
    git diff {BASE_SHA}..{HEAD_SHA}
    

    テンプレートにはこれらのコマンドが示されており、レビュアーが BASE_SHAHEAD_SHA 間の変更だけに集中できるようになっています。

このスキルの設計により、レビューエージェントに見せるのは コミットと diff といった成果物のみ であり、広範なセッション履歴や無関係なコンテキストは共有されません。

ステップ 5: フィードバックを解釈し、対応する

code-reviewer.md テンプレートは、エージェントに次の観点でレビューするよう指示します。

  • コード品質(関心の分離、エラーハンドリング、DRY、エッジケース)
  • アーキテクチャ(設計、スケーラビリティ、パフォーマンス、セキュリティ)
  • テスト(カバレッジ、統合ポイント、テストケースの質)
  • 要件の確認(仕様との整合性、スコープ逸脱の有無、破壊的変更の明示)
  • 本番準備性(マイグレーション、後方互換性、ドキュメント)

フィードバックは次のカテゴリに分類されます。

  • Critical (Must Fix) – バグ、セキュリティ問題、データ損失リスク、機能の破壊
  • Important (Should Fix) – アーキテクチャ上の問題、欠けている機能、不十分なエラーハンドリング、テスト不足
  • Minor (Nice to Have) – スタイル、最適化、ドキュメント改善

SKILL.md で示される対応フローは次のとおりです。

  • Critical な問題は作業を進める前に必ず修正する
  • Important な問題も、さらなる開発やマージの前に解消しておく
  • Minor な事項は、後でのクリーンアップやフォローアップタスクとして記録しておく
  • レビュアーが誤っている・コンテキストを理解していないと判断した場合は、根拠を示してきちんと反論する

この重要度別トリアージに従うことで、AI のフィードバックを漠然とした提案ではなく、具体的なアクションリストに変換できます。

利用パターン例

典型的な requesting-code-review の利用例は次のような流れです。

  1. feature branch で「Task 2: Add verification function」を完了する。

  2. 次のように SHA を取得する。

    BASE_SHA=$(git rev-parse origin/main)
    HEAD_SHA=$(git rev-parse HEAD)
    
  3. code-reviewer.md のプレースホルダを次のように埋める。

    • {WHAT_WAS_IMPLEMENTED} = "Verification function for user email flow"
    • {PLAN_OR_REQUIREMENTS} = チケットや要件のリンク/要約
    • {BASE_SHA}{HEAD_SHA} = Git から取得した値
    • {DESCRIPTION} = "Implement email verification and add tests for edge cases"
  4. Task tool を使って superpowers:code-reviewer サブエージェントを実行する。

  5. Critical / Important / Minor に分類された構造化フィードバックを受け取る。

  6. Critical と Important を解消し、必要に応じてマージ前に再度同じプロセスを回す。


FAQ

requesting-code-review は GitHub リポジトリ専用ですか?

いいえ。requesting-code-reviewGit ベース であり、GitHub 専用ではありません。commit SHA と git diff コマンドに依存しているため、BASE_SHAHEAD_SHA を指定できる Git のリモート(GitHub, GitLab, Bitbucket, 自前ホスティングなど)であればどこでも使えます。

開発セッションの全履歴を共有する必要はありますか?

いいえ。requesting-code-review の重要な設計思想のひとつは、レビュアーに渡すコンテキストを 狭く適切な範囲に絞る ことです。共有するのは「何を実装したか」「要件は何か」「Git diff」のみであり、一般的なセッション履歴や思考プロセスは非公開のまま、レビュアーは実際のコード変更に集中できます。

ワークフローのどのタイミングで requesting-code-review を実行すべきですか?

requesting-code-review の利用タイミングは次のとおりです。

  • サブエージェント駆動開発の各タスク完了後
  • 大きな機能を作り終えた後
  • main ブランチへのマージ前

さらに、詰まったとき、大規模なリファクタリングの前、複雑なバグ修正の後などに実行しておくと、潜在的なリスクの早期検知に役立ちます。

既存の Pull Request レビューとはどう統合できますか?

requesting-code-review は、人間による PR レビューの 代替ではなく補完 として機能します。例えば次のような使い方ができます。

  • PR を作成する 前に このスキルを実行し、明らかな問題を事前に潰す
  • 人間のレビュアーと 並行して 実行し、レビュー範囲と一貫性を高める
  • ここで使われる Critical / Important / Minor の分類を、PR コメントの整理にも活用する

このスキルは Git のレンジを前提にしているため、PR ベースの開発フローにも自然に組み込めます。

code-reviewer エージェントの挙動はカスタマイズできますか?

はい。code-reviewer.md はカスタマイズ可能なテンプレートです。

  • コード品質、アーキテクチャ、テスト、セキュリティに関するチェック項目を調整する
  • ドメイン固有のルールやパフォーマンス予算など、プロジェクト特有の観点を追加する
  • 重要度のレベルや出力セクションを変更したい場合、出力フォーマットを調整する

ただし、レビューをフォーカスされ実行可能なものに保つため、タスクの明確化・Git レンジ・重要度カテゴリというコア構造は維持することをおすすめします。

レビュアーの指摘が間違っていた場合はどうすればよいですか?

このスキルでは、レビュアーが誤っている・コンテキストが不足していると感じたときに、理由を示してきちんと反論する ことを明示的に推奨しています。レビュー結果を絶対的な判定ではなく、強い提案として扱ってください。制約条件やトレードオフを説明したり、必要に応じて計画を更新したりすることで、次回以降のレビュー精度も高められます。

requesting-code-review はテストやコードを自動生成してくれますか?

いいえ。このスキルは 生成ではなくレビュー専用 です。できるのは次のようなことです。

  • 特定の変更にフォーカスしたレビューを依頼する
  • 品質・アーキテクチャ・テストに関する構造化されたフィードバックを受け取る

実際の修正やテスト作成は、あくまであなたの責任です。ただし、他のコーディングスキルやテスト生成スキルと組み合わせてツールチェーンを構成することは可能です。

すぐに試すにはどうすればよいですか?

  1. スキルをインストールします。

    npx skills add https://github.com/obra/superpowers --skill requesting-code-review
    
  2. SKILL.md を読み、レビューのチェックポイントを理解します。

  3. code-reviewer.md に目を通し、エージェントのチェックリストと出力フォーマットを確認します。

  4. 次のタスクや機能を実装し、BASE_SHAHEAD_SHA を取得してから、superpowers:code-reviewer サブエージェントを実行します。

あとは、テンプレートやワークフローをチームの開発習慣や品質基準に合わせて調整し、自分たちにフィットした requesting-code-review の運用方法に仕上げていきましょう。

評価とレビュー

まだ評価がありません
レビューを投稿
このスキルの評価やコメントを投稿するにはサインインしてください。
G
0/10000
新着レビュー
保存中...