Content
85%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.
An exceptionally tight, well-sequenced workflow skill: lean prose, explicit validation gates, feedback loops, and hard stop conditions. Its one material defect is bundle integrity — every referenced rules/templates file is missing from the bundle (and the shared github-access rule lives outside it), so the otherwise excellent progressive-disclosure design fails at navigation time.
Suggestions
Ship the referenced bundle files alongside SKILL.md — rules/anti-patterns.md, rules/verdicts.md, rules/confidence-gate.md, rules/ci-verification.md, rules/regression-detection.md, rules/self-improvement-loop.md, and templates/plan-artifact.md are all cited but absent from the bundle, so every progressive-disclosure link dangles.
Remove or vendor the cross-bundle reference ../../../agents/shared/rules/github-access.md; a path escaping the skill directory breaks when the skill is installed standalone, and Phase 0 depends on it.
Inline one or two representative gh commands (e.g. fetching failed logs and watching PR checks) so the core diagnose-verify loop stays executable even when reference files are not loaded.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense directive prose with no padding and no explanations of concepts Claude already knows — e.g. "Investigation scales with uncertainty and blast radius; completion checks apply to every path. Maximum four fix-push cycles per invocation." Every sentence carries instruction; there is nothing to trim, matching the 'every token earns its place' anchor. | 5 / 5 |
Actionability | Guidance is concrete and decision-ready for an instruction-only skill — the mechanical-path checklist ("Logs identify the exact command... No assertion, public API, runtime behavior... needs a judgment call"), hard budgets ("resync and retry once; never force-push", "Maximum four fix-push cycles"), and enumerated report fields. It falls short of the 5 anchor because not a single executable command is shown ("Commands below show the `gh` path" yet no `gh` example appears), leaving the concrete command detail to reference files that are not present. | 4 / 5 |
Workflow Clarity | Phases 0-9 are clearly sequenced with explicit validation checkpoints (Phase 5 local verification, Phase 7 CI verification on the pushed revision, "Missing, pending, or inaccessible checks are not green"), feedback loops for error recovery ("A local failure returns to diagnosis", rebase-conflict abort, non-fast-forward retry-once), and explicit stop conditions (four cycles, two regressions, no useful next check). This matches the top anchor including its checklist-and-recovery-loop criteria. | 5 / 5 |
Progressive Disclosure | The on-page structure is well-designed — references are clearly signaled, one level deep, and load-pointed ("Read [anti-patterns](./rules/anti-patterns.md) first. Load other references only at the decision they support.") — but scoring against the actual bundle, none of the eight referenced files exist (rules/anti-patterns.md, rules/verdicts.md, rules/confidence-gate.md, rules/ci-verification.md, rules/regression-detection.md, rules/self-improvement-loop.md, templates/plan-artifact.md are all absent, and github-access.md resolves outside the bundle at ../../../agents/shared/rules/). Every navigation link dangles, so the structure cannot deliver easy navigation; this sits below the 4 anchor's 'minor organization gaps' and above the 2 anchor since the SKILL.md itself is well organized with nothing inlined that belongs elsewhere. | 3 / 5 |
Total | 17 / 20 Passed |