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, fully executable procedure: copy-paste commands, explicit validation checkpoints, and failure paths for every environmental constraint. The only costs are length — rationale-heavy sections inflate the token budget — and the absence of any bundle split for that explanatory material.
Suggestions
Trim the 'When to run this instead of retro' and 'Where this sits' sections to two or three lines each, moving the orchestration context to the existing workflow documentation the body already assumes.
Move the four rules under Step 1 — particularly the `.closingIssuesReferences` and label-policy rationale — into a references/ file, keeping only the rule statements and the one-line 'why' in SKILL.md.
Condense the `transcripts: no` and hook-fallback paragraphs into a short table of condition → consequence, preserving the rules while cutting roughly half the tokens.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence — no basic concepts are explained, and every rule carries non-obvious domain rationale (e.g. why a shell-rolled issue lookup silently reads a stranger's label). It sits at 4 rather than 5 because sections like 'When to run this instead of retro' and the four-rule block carry explanatory context that could be trimmed by a third without losing actionability; not 3, since none of it is padding Claude already knows. | 4 / 5 |
Actionability | Every step is copy-paste-ready bash: the three pre-flight probes, `bun run --cwd tools/pr-metrics card -- --resolve-issue` in both committed and markdown forms, the guarded `git add` of the single card, and the append-to-body sequence with the duplicate-check grep. Fallback commands for missing `gh` and missing network are given explicitly, covering the common failure cases — the 5 anchor. | 5 / 5 |
Workflow Clarity | Sequence is explicit (probe environment → write/commit card → render onto PR → finish report) with real checkpoints: the pre-flight network/gh/transcripts probe, the duplicate-block check before appending (with the correct match pattern explained), and per-step degradation paths ('record which and carry on'). Not 4: validation and error recovery are explicit at every fragile point, matching the 5 anchor's feedback-loop pattern. | 5 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent) and the body is self-contained with clear section headers, so navigation is unambiguous — but at ~160 lines, explanatory blocks like the four rules and the 'When to run this instead of retro' context could live in a reference file. That puts it at 4 (good structure, minor organization gaps) rather than 5, which is reserved for a lean overview with well-signaled references or a sub-50-line body. | 4 / 5 |
Total | 18 / 20 Passed |