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.
The body is a highly actionable, well-sequenced procedure with executable commands, explicit validation, bounded retry loops, and careful HALT semantics. Its weaknesses are moderate template verbosity and one broken reference (constitution.md) plus several reference files that SKILL.md never links, which slightly undermines navigation.
Suggestions
Fix the broken reference: either add the missing constitution.md file or point the Constitutional Rules section at the existing inline rules (or a real reference file) instead of a non-existent path.
Link the orphaned references (history-mining.md, intent-inference.md, mergiraf-integration.md, semantic-conflicts.md) from the relevant procedure steps so they are discoverable, or fold their key content into the corresponding steps.
Move the full Markdown templates for the Step 6 summary, per-file decision record, and HALT schema into a reference file, keeping only field lists or a compact example inline to reduce token cost.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence (never explains what a rebase or lockfile is), but the full Markdown templates for the Step 6 summary, per-file decision record, and HALT schema, plus advisory asides like 'These are configurable heuristics, not empirical laws', could be tightened or pushed to a reference. This matches 'efficient; minor instances of over-explanation that could be trimmed' rather than level 5's every-token-earns-its-place. | 4 / 5 |
Actionability | Every step ships an exact executable command with flags ('timeout 30 mergiraf solve -- <file> --compact --keep-backup=false', script invocations with --json/--top/--k), exit-code semantics (124 for timeout, 5 for the reprompt artifact), and complete lockfile-to-regeneration-command and context-to-continue-command tables. Fully copy-paste ready. | 5 / 5 |
Workflow Clarity | Steps 1–6 are clearly sequenced with explicit validation checkpoints (validate-and-reprompt.sh with a bounded retry budget, semantic-audit.sh where 'Any SUSPECT output requires review'), a feedback loop ('Loop to Step 1 if more conflicts appear'), HALT handling for un-inferable intent, and abort criteria. Matches the level-5 anchor including error-recovery loops. | 5 / 5 |
Progressive Disclosure | Good structure overall: the body is an actionable overview, references are one level deep and clearly signaled ('read ... only when needed', 'Full guidance: references/pr-decomposition.md'), and detailed material is split into references/ and scripts/. However, the body instructs 'Read ${CLAUDE_SKILL_DIR}/constitution.md' but no constitution.md exists in the bundle, and four of the eight reference files (history-mining.md, intent-inference.md, mergiraf-integration.md, semantic-conflicts.md) are not navigable from SKILL.md. This lands at level 4 (minor organization gaps) rather than 5 due to the broken/missing navigation targets. | 4 / 5 |
Total | 18 / 20 Passed |