loop-review-skill-until-fixed-point
Loop Review Skill Until Fixed Point
Run /review-skill on a target document repeatedly until fixed point—when all reviewers return NO ISSUES.
Core Concept
/review-skill <target> --auto
│
▼
┌─────────┐
│ Issues? │
└────┬────┘
│
yes ─┴─ no
│ │
▼ ▼
repeat done (fixed point)
Fixed point = the document is both correct AND unambiguous. No reviewer can find anything to flag.
Arguments
| Arg | Required | Description |
|---|---|---|
<target> |
yes | Path to SKILL.md to review |
State
max_iterations: 10 # Safety limit
iteration_count: 0 # Current iteration
target: "<from args>" # Target SKILL.md path
history: [] # Minimal: just iteration outcomes
History Entry Schema
Each iteration appends a minimal entry:
- iteration: 1
fixed_point: false
- iteration: 2
fixed_point: false
- iteration: 3
fixed_point: true # All reviewers returned NO ISSUES
The orchestrator does NOT track per-reviewer metrics. That detail stays inside /review-skill where it belongs. This prevents context leak between iterations.
Phase: Initialize
- Parse target path from arguments
- Set
iteration_count = 0 - Set
max_iterations = 10 - Confirm target exists
Phase: Loop
Isolation requirement: Each iteration must be context-isolated. The orchestrator must NOT carry synthesis, triage details, or "recurring theme" narratives between iterations. This prevents context leak that would bias subsequent reviews.
while iteration_count < max_iterations:
iteration_count += 1
1. Run: /review-skill <target> --auto
(This is a FRESH invocation—no prior iteration context)
2. Capture only the exit status: "fixed point" or "issues remain"
3. Append to history file: { iteration, fixed_point: bool }
4. Output: "Iteration {N}: {'fixed point' | 'issues remain'}"
5. If fixed point → exit loop
6. Else → continue loop (do NOT summarize or analyze findings here)
The orchestrator is intentionally stateless. All synthesis, triage, and issue-addressing happens INSIDE /review-skill --auto. The orchestrator only observes the outcome.
Exit Conditions
| Condition | Action |
|---|---|
/review-skill reports fixed point |
Exit with success |
iteration_count >= max_iterations |
Safety limit hit, ask user how to proceed |
Phase: Report
Present final state:
## Loop Complete
| Metric | Value |
| ------------------- | ----------------- |
| Target | {target} |
| Iterations | {iteration_count} |
| Fixed point reached | yes/no |
If fixed point reached:
{target} reached fixed point after {N} iterations.
If max iterations hit:
Safety limit reached after {max_iterations} iterations without convergence.
Begin now.
More from corygabrielsen/skills
mission-control
Coordinate complex multi-step work using task graphs and parallel background agents. Use when work requires decomposition, delegation, and long-running operations that may survive context compaction.
11loop-codex-review
Automated code review loop with progressive reasoning levels. Runs n parallel Codex reviews (configurable via -n), Claude addresses issues, climbs from low→xhigh reasoning until fixed point (all n clean). Human approval at each iteration.
11review-pr
Thorough, educational PR review process. Gathers context, explains changes, identifies issues systematically, and drafts constructive feedback collaboratively before submitting.
11orthogonalize-pr
Identify and separate orthogonal (independent) change sets within a decomposed PR. Creates branches for each set that could theoretically be reviewed or merged separately.
10decontextualize
Course-correct when artifacts overfit to conversation context. Invoke when examples, names, or language are too specific to current work.
10fork
Branch off a conversation to handle tangents. Outputs context summary and ready-to-paste command for a new terminal session.
9