ultra-review

Files (4)

• SKILL.md
• unified-schema.md · references/
• review_verdict_update.py · scripts/
• review_wait.py · scripts/

SKILL.md (first lines)

# /ultra-review - Ultra Review System

Orchestrates parallel code review using specialized agents. All findings written to JSON files to prevent context window pollution.

## Workflow Tracking (MANDATORY)

**On command start**, create tasks for each major step using `TaskCreate`:

| Step | Subject | activeForm |
|------|---------|------------|
| 1 | Setup & Scope Detection | Detecting diff scope... |
| 2 | Agent Selection | Selecting review agents... |
| 3 | Background Execution | Running review agents... |
| 4 | Wait & Coordinate | Waiting for agent results... |
| 5 | Report to User | Generating review report... |

**Before each step**: `TaskUpdate` → `status: "in_progress"`
**After each step**: `TaskUpdate` → `status: "completed"`
**On context recovery**: `TaskList` → resume from last incomplete step

## Usage

```
/ultra-review              # Full review (smart skip based on diff content)
/ultra-review all          # Force ALL 6 agents, no auto-skip (pre-merge gate)
/ultra-review quick        # Quick review (review-code only)
/ultra-review security     # Security focus (review-code + review-errors)
/ultra-review tests        # Test quality focus (review-tests only)
/ultra-review recheck      # Re-check only P0/P1 files from last session
/ultra-review delta        # Review only changes since last review session
/ultra-review <agents...>  # Custom: e.g., /ultra-review tests errors types
```

### Scope Options (combinable with any mode)

```
/ultra-review --pr 123            # Review PR #123 diff (base...head)
/ultra-review --range main..HEAD  # Review specific commit range
/ultra-review --range abc123      # Review single commit
/ultra-review security --pr 42    # Security review scoped to PR #42
```

## Orchestration Flow

### Phase 1: Setup & Scope Detection

```python
# 1. Determine session context
BRANCH = git branch --show-current            # e.g., "feat/task-3-auth"
ITER = count existing sessions matching this branch + 1  # iteration number

# 2. Create session directory with full context
SESSION_ID = "<YYYYMMDD-HHmmss>-<branch>-iter<N>"
# e.g., "20260214-103000-feat-task-3-auth-iter2"
SESSION_PATH = ".ultra/reviews/{SESSION_ID}/"
mkdir -p SESSION_PATH

# 3. Update session index (placeholder — verdict filled in Phase 5)
# Append entry to .ultra/reviews/index.json with verdict="pending"
# See Session Index below for full schema

# 4. Cleanup (see Lifecycle Management below)

# 5. Detect diff scope (priority order)
```

**Scope resolution priority:**

| Flag | Diff Command | DIFF_RANGE |
|------|-------------|------------|
| `--pr NUMBER` | `gh pr diff NUMBER --name-only` | PR base...head |
| `--range RANGE` | `git diff RANGE --name-only` | Specified range |
| _(default)_ | `git diff --name-only HEAD` + `git diff --cached --name-only` | HEAD (unstaged + staged) |

**PR scope details** (`--pr NUMBER`):
```bash
# Get PR metadata for session naming
gh pr view NUMBER --json headRefName,baseRefName,title
# Get changed files
gh pr diff NUMBER --name-only
# Set DIFF_RANGE to base...head for agents
DIFF_RANGE="$(gh pr view NUMBER --json baseRefName -q .baseRefName)..HEAD"
```

**Range scope details** (`--range RANGE`):
```bash
git diff RANGE --name-only
DIFF_RANGE="RANGE"
```

### Phase 2: Agent Selection

Based on the mode argument and file classification:

**Mode: `full` (default)**
Select all applicable agents based on diff content:

| Agent | Skip Condition | Warning |
|-------|---------------|---------|
| review-code | Never skip | - |
| review-tests | No test files in diff AND no source files without tests | WARN: "Code changed without test updates" |
| review-errors | No error handling code detected | - |
| review-design | No type definitions AND no function-level changes in diff | - |
| review-comments | No comment changes in diff | - |
| review-ac-drift | No `.ultra/tasks/contexts/` directory OR no active task locatable | - |

**Auto-skip detection commands:**
```bash
# Has type definitions?
grep -l "interface \|type \|class \|enum " <diff_files>

# Has comment changes?
git diff HEAD -- <diff_files> | grep "^[+-].*//\|^[+-].*\*\|^[+-].*#"

# Has test files?
echo "<diff_files>" | grep -E "\.(test|spec)\.(ts|tsx|js|jsx)$|test_.*\.py$|.*_test\.go$"

# Has error handling?
grep -l "try\|catch\|\.catch\|throw\|Error(" <diff_files>

From rocky2431/ultra-builder-pro, MIT.

More from rocky2431/ultra-builder-pro