Content
92%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A tight, highly executable skill body: copy-paste commands, a clearly sequenced pipeline with validation feedback loops, explicit safety bounds and halt conditions, and a verified single-script bundle. The only blemish is a brief historical aside about a retired predecessor loop that could be cut or condensed.
Suggestions
Trim the 'UX note' paragraph: keep the actionable half (inspect/resume via ~/.prp/<key>/state/prp-loop.state.json instead of watching live) and drop the description of the retired Ralph loop's Stop-hook mechanics, or move the comparison to a short 'old patterns' note.
Consider one line showing how to inspect loop progress (e.g., tailing the run.log or reading the state file's stage field) so operators can check on a detached run without resuming it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — commands, defaults, and halt conditions with zero concept explanation — but the "UX note" paragraph about the retired Ralph loop mixes historical comparison ("was single-session and interactive (a Stop-hook fed the prompt back...)" ) with useful guidance and could be trimmed to just the actionable half. Not 5 because of this minor over-explanation; not 3 because nothing else is padded. | 4 / 5 |
Actionability | Fully executable copy-paste commands for both start and resume ("uv run .agents/skills/prp-loop/scripts/prp_loop.py \"$ARGUMENTS\" --cli codex" and the --resume variant), plus the --until example, all defaults (--max-cycles, --max-implement-iterations, --validate), and the state-file path for inspection. | 5 / 5 |
Workflow Clarity | The five stages are sequenced (plan → implement → pr → review → cycle) with an explicit feedback loop ("if the verdict needs fixes, the complete report, plan, and live PR feed into a fresh prp-implement correction pass → push → re-review, up to --max-cycles"), explicit validation detection (VALIDATION: GREEN sentinel or --validate), and enumerated halt conditions. | 5 / 5 |
Progressive Disclosure | A short, single-purpose skill (~50-line body) with well-organized sections (Run it, What it does, Safety, Notes) and a single one-level-deep bundle reference (scripts/prp_loop.py, verified to exist and to implement --until/--validate/the GREEN sentinel and state file described). No inlined content that belongs in separate files; nothing needs external references. | 5 / 5 |
Total | 19 / 20 Passed |