jimweller/review-full

<!-- markdownlint-disable-file MD041 -->

STARTER_CHARACTER = 🔎

Full Review

Review the entire codebase across 8 focus areas in parallel. Each agent reads from a single repomix-packed snapshot via MCP. Output goes to per-area files in .llmtmp/review-full/.

Arguments: $ARGUMENTS

If $ARGUMENTS is provided, treat it as a path relative to the repo root. Otherwise pack the whole repo.

Step 1: Resolve Target

$ARGUMENTS is a Claude Code template substitution, not a shell variable. It is replaced in the markdown before bash runs. Use plain assignment plus a conditional fallback. Do NOT use ${1:-...} or ${ARGUMENTS:-...} parameter expansion forms; both expand the substituted text inside ${...} and produce invalid bash when the argument contains slashes or is empty.

PROJECT_ROOT=$(git rev-parse --show-toplevel)
PROJECT_ROOT=$(cd -P "$PROJECT_ROOT" && pwd -P)
TARGET_PATH="$ARGUMENTS"
[ -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"

Step 2: Clean and Prepare Output

OUTPUT_DIR="$PROJECT_ROOT/.llmtmp/review-full"
rm -rf "$OUTPUT_DIR"
mkdir -p "$OUTPUT_DIR"

Step 3: Pack Codebase

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

The repomix file lives inside OUTPUT_DIR (.llmtmp/review-full/), which is gitignored and wiped at the start of every run by Step 2. No /tmp leakage and no trap needed. A trap does not work here because each bash code block in this skill runs in a separate shell process; an EXIT trap registered in Step 3 would fire as soon as Step 3's bash block ends, deleting the file before Step 4 (attach_packed_output) can read it.

Repomix reads .repomixignore from the project root automatically.

Verify the file was created. Confirm REPOMIX_FILE before proceeding.

Step 4: Attach Packed Output

Call attach_packed_output with filePath=REPOMIX_FILE. Record the returned outputId string. This is the codebase access handle the agents will use.

Do not delegate this step to a subagent. The orchestrator must own the outputId.

Step 5: Dispatch 8 Review Agents in Parallel

Send a single message with 8 Task tool uses, one per review area. Each Task uses the corresponding subagent_type:

| Agent | Subagent type | Output file | | --------------------- | --------------------- | ----------------- | | Architecture & Design | review-architecture | architecture.md | | Correctness & Bugs | review-correctness | correctness.md | | Operational Readiness | review-ops | ops.md | | Performance | review-performance | performance.md | | Code Quality | review-quality | quality.md | | Security | review-security | security.md | | SOLID Principles | review-solid | solid.md | | Testing | review-testing | testing.md |

Each Task prompt contains:

1. The outputId from Step 4
2. Instruction to use read_repomix_output and grep_repomix_output for navigation
3. Instruction: "Read CLAUDE.md and .llmdocs/architecture.md (if present) via the repomix output for project context."
4. Instruction: "Write findings to <OUTPUT_DIR>/<area>.md. Use H2 header followed by findings or 'No findings.'"

Step 6: Wait and Verify

After all 8 agents complete, verify each file in <OUTPUT_DIR>:

1. File exists and is non-empty
2. First non-blank line is an H2 header (## <Area>)
3. Body contains at least one finding OR exactly the literal No findings.

If a file fails any of those checks, treat the agent as having silently failed and re-dispatch that one agent. Do NOT accept a stub like ## Testing\n with no body or content that omits the H2 header.

Step 7: Summarize

Produce a summary in the conversation:

# 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> |
| 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/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.

Do not load the file contents into the conversation. The summary is the artifact; files are persistent for follow-up.