Content
88%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 tightly written, highly actionable autonomous-loop skill with explicit sequencing, validation checkpoints, and error-recovery feedback loops. The only real weakness is mild repetition of the scope boundary and merge logic, which slightly inflates length without adding clarity.
Suggestions
Consolidate the 'never edit packages/*/src' rule: state it once in the scope-boundary section and reference it briefly elsewhere rather than re-deriving it three times.
The squash-merge-on-green-CI preference list appears in both the step-0 guard and step-3 build sections; factor it into one place and cross-reference to remove redundancy.
Consider moving the 'value bar for a QA task' checklist into a short reference file so the main loop body stays a lean overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense, competent prose that assumes Claude's intelligence (no explaining what tests/CI are), but the scope boundary 'never edits packages/*/src' and the squash-merge-on-green-CI logic are each restated several times and could be tightened. | 4 / 5 |
Actionability | Fully executable guidance throughout — `gh pr list --base auto-qa --state open`, `git checkout -b claude/qa-<slug> origin/auto-qa`, `pnpm build && pnpm check && pnpm test`, `gh pr merge <num> --squash --auto` — copy-paste ready commands covering the common cases. | 5 / 5 |
Workflow Clarity | Clear serial sequence (guard → interrupts → pick issue → build & record) with explicit validation checkpoints — re-verify before building, quality gates, squash-merge on green CI only — and a feedback loop for red CI (fix/re-push, 3-attempt limit, then label needs-attention). | 5 / 5 |
Progressive Disclosure | Well-organized into clearly headed sections (scope, guard, interrupts, pick, build, boundaries) in a single self-contained file with no bundle references; minor gap in that some content (value bar, boundaries) is inline rather than split out, but no nested or buried references. | 4 / 5 |
Total | 18 / 20 Passed |