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.
An exceptionally well-engineered instruction-only skill: a phased workflow with explicit validation gates, feedback loops, and stop conditions, written with no wasted tokens. The two gaps are the absence of any concrete gh command examples in the body (despite promising "Commands below show the `gh` path") and referenced bundle files that are not present alongside SKILL.md, one of which lives outside the skill directory.
Suggestions
Include one or two concrete `gh` command examples (e.g., fetching failed-run logs via `gh run view --log-failed`) in the body, since it currently promises 'Commands below show the `gh` path' but shows none, and the referenced rule files are not in the bundle.
Ship the referenced files (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) with the skill, or inline the minimal essentials — as delivered, every reference is a broken path.
Replace the out-of-bundle relative reference ../../../agents/shared/rules/github-access.md with either a bundled copy or a self-contained access-resolution step, since a cross-tree path breaks when the skill is installed or moved.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, directive, and free of padding — e.g., "Inspect one representative of a repeated signature, then confirm the other jobs share its command/environment" and "On a non-fast-forward rejection, resync and retry once; never force-push" — with zero explanations of concepts Claude already knows. Every token carries an instruction or constraint, matching the 'lean and efficient; every token earns its place' anchor. | 5 / 5 |
Actionability | Guidance is highly concrete and directive ("Accept a PR URL, Actions run URL/ID, or check-run ID", "fetch logs once per run attempt, save them locally", "Maximum four fix-push cycles", "Abort a conflicting rebase and report the conflicting files"), but no executable commands appear in the body — it says "Commands below show the `gh` path" yet shows none, deferring all command syntax to referenced files that are not in this bundle. This sits between anchor 4 (mostly executable with minor gaps) and anchor 5 (fully executable, copy-paste ready); the missing concrete command examples keep it at 4. | 4 / 5 |
Workflow Clarity | A clearly sequenced Phase 0–9 workflow with explicit validation checkpoints and feedback loops: local verification before push (Phase 5), "A local failure returns to diagnosis", "Any contradictory evidence, failed local verification, wider-than-expected diff... moves a mechanical fix into the diagnostic path", post-push CI verification with "Missing, pending, or inaccessible checks are not green", and bounded stop/revert criteria. This matches anchor 5's explicit validation, error-recovery loops, and checklists for a complex process. | 5 / 5 |
Progressive Disclosure | The structure is exemplary in-text: references are one level deep, each loaded at the decision it supports ("Read [anti-patterns](./rules/anti-patterns.md) first. Load other references only at the decision they support"), covering verdicts, confidence gate, plan template, CI verification, and regression detection. However, none of the eight referenced files exist in the skill's bundle (no rules/, templates/, or references/ directories), and github-access.md is referenced via a fragile cross-tree relative path (../../../agents/shared/rules/github-access.md), so navigation cannot be verified — good structure with real organization gaps, matching anchor 4 rather than 5. | 4 / 5 |
Total | 18 / 20 Passed |