mcpField
R
review-full

review-full

github.com/jimwellerテスト・品質44 回インストール
リポジトリ

review-full について

review-full は、repomix でまとめたコードスナップショットを使い、9 分野を複数のエージェントで並行レビューするスキルです。分野別の結果を `.llmtmp/review-full/` に保存し、要約を作成します。

主な機能

  1. 019 分野の並行レビュー
  2. 02repomix によるコードの集約
  3. 03分野別ファイルへの結果出力
  4. 04レビュー出力ファイルの検証

活用シーン

  1. 01リポジトリ全体のレビュー
  2. 02指定したコードパスのレビュー
  3. 03分野別の指摘事項の整理

MCP 対応

エージェントが MCP ツールを使って、集約されたリポジトリスナップショットを参照するよう指示します。

インストール

コマンドをコピーしてターミナルで実行します。

$npx skills add https://github.com/jimweller/clanker-skills --skill review-full

インストール後、Claude Code で/review-fullを入力して読み込まれたか確認します。

詳細

STARTER_CHARACTER = 🔎

全体レビュー

9 つの重点領域について、コードベース全体を並行してレビューします。各エージェントは、MCP 経由で単一の repomix パック済みスナップショットを読み込みます。出力は .llmtmp/review-full/ 内の領域ごとのファイルに保存されます。

引数

呼び出し時にユーザーがパスを指定した場合は、リポジトリのルートからの相対パスとして対象ディレクトリに設定します。それ以外の場合は、リポジトリ全体をパックします。

ステップ 1:対象を解決

PROJECT_ROOT=$(git rev-parse --show-toplevel)
PROJECT_ROOT=$(cd -P "$PROJECT_ROOT" && pwd -P)
TARGET_PATH="<user-provided path or empty>"
[ -z "$TARGET_PATH" ] && TARGET_PATH="$PROJECT_ROOT"

# Resolve to physical absolute path and verify it stays inside the repo (no traversal, no symlink escape).
TARGET_PATH=$(cd -P "$TARGET_PATH" 2>/dev/null && pwd -P) || { echo "TARGET_PATH does not exist"; exit 1; }
case "$TARGET_PATH" in
  "$PROJECT_ROOT"|"$PROJECT_ROOT"/*) ;;
  *) echo "TARGET_PATH escapes PROJECT_ROOT"; exit 1 ;;
esac

TARGET_NAME=$(basename "$TARGET_PATH")
[ "$TARGET_PATH" = "$PROJECT_ROOT" ] && TARGET_NAME="repo"

ステップ 2:出力をクリーンアップして準備

OUTPUT_DIR="$PROJECT_ROOT/.llmtmp/review-full"
mkdir -p "$OUTPUT_DIR"
find "$OUTPUT_DIR" -mindepth 1 -delete

rm -rf ではなく、mkdir -p の後に find -delete を実行します。一部の dotfiles 設定にあるような PATH 上の safe-rm shim は、パスが存在しない場合、-f を指定していても Trash に移動してゼロ以外で終了するため、初回実行時に消去処理が失敗します。

ステップ 3:コードベースをパック

REPOMIX_FILE="$OUTPUT_DIR/repomix.xml"
npx repomix -o "$REPOMIX_FILE" --quiet --output-show-line-numbers "$TARGET_PATH"

repomix ファイルは OUTPUT_DIR(.llmtmp/review-full/)内にあります。このディレクトリは gitignore 対象で、ステップ 2 により実行ごとに消去されます。/tmp への漏出はなく、trap も不要です。このスキルの各 bash コードブロックは別々のシェルプロセスで実行されるため、ここでは trap は機能しません。ステップ 3 で登録した EXIT trap は、ステップ 3 の bash ブロックが終了するとすぐに実行され、ステップ 4(attach_packed_output)が読み込む前にファイルを削除してしまいます。

Repomix はプロジェクトルートの .repomixignore を自動的に読み込みます。

ファイルが作成されたことを確認します。続行する前に REPOMIX_FILE を確認してください。

ステップ 4:パック済み出力を添付

filePath=REPOMIX_FILE を指定して attach_packed_output を呼び出します。返された outputId 文字列を記録します。これは、エージェントが使用するコードベースへのアクセスハンドルです。

このステップをサブエージェントに委任しないでください。オーケストレーターが outputId を管理する必要があります。

ステップ 5:9 つのレビューエージェントを並行して起動

9 件の Agent 呼び出しすべてを、run_in_background: false を指定して単一のツールブロック内で実行します。サブエージェントはデフォルトでバックグラウンド実行されます。バックグラウンドで一括起動すると、結果は後続のターンで返されるため、ステップ 6 ではエージェントがまだ書き込んでいないファイルを検証することになります。

エージェントサブエージェントの種類出力ファイル
アーキテクチャと設計reviewer-architecturearchitecture.md
正確性とバグreviewer-correctnesscorrectness.md
データと情報アーキテクチャreviewer-datadata.md
運用準備状況reviewer-opsops.md
パフォーマンスreviewer-performanceperformance.md
コード品質reviewer-qualityquality.md
セキュリティreviewer-securitysecurity.md
SOLID 原則reviewer-solidsolid.md
テストreviewer-testingtesting.md

各レビューのプロンプトには、次の内容が含まれます。

  1. ステップ 4 の outputId
  2. ナビゲーションに read_repomix_output と grep_repomix_output を使用する指示
  3. 指示:「プロジェクトのコンテキストを把握するため、repomix 出力から AGENTS.md(または CLAUDE.md)と .llmdocs/architecture.md(存在する場合)を読み込んでください。」
  4. 指示:「調査結果を <OUTPUT_DIR>/<area>.md に書き込んでください。H2 見出しの後に調査結果、または No findings. を記載してください。」

ステップ 6:待機して検証

9 つすべてのエージェントが完了したら、<OUTPUT_DIR> 内の各ファイルを検証します。

  1. ファイルが存在し、空でないこと
  2. 空白行を除いた先頭行が H2 見出し(## <Area>)であること
  3. 本文に少なくとも 1 つの調査結果が含まれるか、文字列 No findings. のみが記載されていること

いずれかのチェックに失敗したファイルは、エージェントが失敗を報告せずに終了したものとして扱い、そのエージェントを再度起動します。本文がない ## Testing\n のような空のひな形や、H2 見出しがない内容は受け入れないでください。

ステップ 7:要約

会話内で要約を作成します。

# Full Review Summary

Target: <TARGET_NAME>
Files reviewed: <count from repomix>

## Findings Counts

| Area                  | High | Medium | Low |
| --------------------- | ---- | ------ | --- |
| Architecture & Design | <N>  | <N>    | <N> |
| Correctness & Bugs    | <N>  | <N>    | <N> |
| Data & Information    | <N>  | <N>    | <N> |
| Operational Readiness | <N>  | <N>    | <N> |
| Performance           | <N>  | <N>    | <N> |
| Code Quality          | <N>  | <N>    | <N> |
| Security              | <N>  | <N>    | <N> |
| SOLID Principles      | <N>  | <N>    | <N> |
| Testing               | <N>  | <N>    | <N> |

## Output Files

- `.llmtmp/review-full/architecture.md`
- `.llmtmp/review-full/correctness.md`
- `.llmtmp/review-full/data.md`
- `.llmtmp/review-full/ops.md`
- `.llmtmp/review-full/performance.md`
- `.llmtmp/review-full/quality.md`
- `.llmtmp/review-full/security.md`
- `.llmtmp/review-full/solid.md`
- `.llmtmp/review-full/testing.md`

Read individual files for detailed findings.

ファイルの内容を会話に読み込まないでください。要約が成果物であり、ファイルは後続の作業用に保持されます。

コメント