From 367877bb45b25e981d6cc1625f3569db9d4223dd Mon Sep 17 00:00:00 2001 From: Kjell Tore Guttormsen Date: Fri, 1 May 2026 17:00:09 +0200 Subject: [PATCH] docs(ultraplan-local): wire ultrareview-local + 4 agents into plugin CLAUDE.md --- plugins/ultraplan-local/CLAUDE.md | 22 +++++++++++++++++++++- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/plugins/ultraplan-local/CLAUDE.md b/plugins/ultraplan-local/CLAUDE.md index 27d973b..d82a43f 100644 --- a/plugins/ultraplan-local/CLAUDE.md +++ b/plugins/ultraplan-local/CLAUDE.md @@ -14,6 +14,7 @@ Deep implementation planning and research with an explicit brief step, specializ | `/ultraresearch-local` | Research — deep local + external research, produces structured research brief | opus | | `/ultraplan-local` | Plan — brief-reviewer, explore, plan, review. Requires `--brief` or `--project`. Auto-discovers `architecture/overview.md` if present | opus | | `/ultraexecute-local` | Execute — disciplined plan/session-spec executor with failure recovery | opus | +| `/ultrareview-local` | Review — independent post-hoc review of delivered code against the brief. Produces `review.md` with severity-tagged findings (Handover 6) | opus | ### /ultrabrief-local modes @@ -66,12 +67,27 @@ If `{project_dir}/architecture/overview.md` exists (typically produced by the se | `--fg` | Force foreground — run all steps sequentially, ignore Execution Strategy | | `--session N` | Execute only session N from plan's Execution Strategy | +### /ultrareview-local modes + +| Flag | Behavior | +|------|----------| +| _(default)_ | Run brief-conformance + code-correctness reviewers in parallel, coordinator dedup + verdict, write `{project_dir}/review.md` | +| `--project ` | **Required.** Path to ultraplan-local project folder containing `brief.md`. Review is written to `{dir}/review.md` | +| `--since ` | Override "before" SHA for the diff range. Validated via `git rev-parse --verify` | +| `--quick` | Skip brief-conformance reviewer; skip coordinator's reasonableness filter — fast correctness-only pass | +| `--validate` | Schema-only check on existing `{dir}/review.md`. No LLM calls | +| `--dry-run` | Print discovered scope + triage map; skip writes | +| `--fg` | No-op alias (foreground is default) | + +The triage gate is deterministic — path-pattern classifier produces `{file → deep-review|summary-only|skip}`. Hard refuse-with-suggestion above 100 files / 100K diff tokens. + ## Agents | Agent | Model | Role | |-------|-------|------| | planning-orchestrator | opus | Inline reference documentation for the planning pipeline workflow (brief-driven) | | research-orchestrator | opus | Inline reference documentation for the research pipeline workflow | +| review-orchestrator | opus | Inline reference documentation for the review pipeline workflow | | architecture-mapper | sonnet | Codebase structure, tech stack, patterns | | dependency-tracer | sonnet | Import chains, data flow, side effects | | task-finder | sonnet | Task-relevant files, functions, reuse candidates | @@ -81,6 +97,9 @@ If `{project_dir}/architecture/overview.md` exists (typically produced by the se | research-scout | sonnet | External docs for unfamiliar tech (conditional, planning only) | | convention-scanner | sonnet | Coding conventions: naming, style, error handling, test patterns | | brief-reviewer | sonnet | Task brief quality (5 dimensions: completeness, consistency, testability, scope clarity, research plan validity) | +| brief-conformance-reviewer | sonnet | Brief conformance review (SC + Non-Goal traceability) | +| code-correctness-reviewer | sonnet | Code correctness review (7 dimensions) | +| review-coordinator | sonnet | Judge Agent — dedup + reasonableness filter + verdict | | plan-critic | sonnet | Adversarial plan review (9 dimensions) | | scope-guardian | sonnet | Scope alignment (creep + gaps) | | session-decomposer | sonnet | Splits plans into headless sessions with dependency graph | @@ -177,5 +196,6 @@ Stats: - **Task brief** — produced by `/ultrabrief-local`. Declares intent, goal, and research plan. Drives planning. - **Research brief** — produced by `/ultraresearch-local`. Answers a specific research question. Feeds planning. - **Architecture note** — opt-in, produced by `/ultra-cc-architect-local` from the separate `ultra-cc-architect` plugin. Proposes which Claude Code features fit the task with brief-anchored rationale + explicit gaps. When present, enriches planning. +- **Review** — produced by `/ultrareview-local`. Independent post-hoc review of delivered code against the task brief. **Handover 6 (review → plan)** routes BLOCKER + MAJOR findings into `/ultraplan-local --brief review.md` for a remediation plan. The plan's optional `source_findings:` frontmatter list is the audit trail back to the consumed findings. MINOR + SUGGESTION are skipped for v1.0 plan-input. -A project typically has 1 task brief, 0–N research briefs, and 0 or 1 architecture note. +A project typically has 1 task brief, 0–N research briefs, 0 or 1 architecture note, and 0–N reviews (one per review iteration).