Content
82%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 dense, highly actionable safety contract: exact commands, explicit user gates, exit-code-driven CI polling, and a thorough verification checklist. Its weaknesses are a corrupted step 6/step 7 boundary (duplicated and garbled markup that obscures the terminal flow) and structural redundancy between the Rules and Instructions sections.
Suggestions
Fix the corrupted markup at the step 6 'Absent' bullet and step 7 heading: step 7's full text is currently nested inline inside step 6's parenthetical continuation, and the heading renders as '7. **7. **TOTAL STOP**' — rewrite step 6 to end cleanly and restore step 7 as a standalone step.
Consolidate the heavy overlap between Rules (4 and 9) and Instructions (steps 3 and 6): keep the rules as one-line invariants and let the steps carry the procedural detail, trimming roughly a third of the duplicated force-refusal and CI-await prose.
Consider moving the CI-await contract (rule 9 + step 6, ~40 lines of polling, tunables, and investigate-handoff logic) into a references/ file linked from a short 'Await CI' pointer, which would cut the SKILL.md body roughly in half while keeping the core push flow front-and-center.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | There is no filler or re-teaching of known concepts — every line is contract detail (commands, exit codes, exact user-facing strings). However, redundancy is noticeable: Rule 4 restates Step 3 at length, Rule 9 restates Step 6, and the duplicated step 6/step 7 text wastes tokens. This sits between anchors 3 and 4; the repetition is deliberate safety emphasis rather than unnecessary explanation, so anchor 4 fits best. | 4 / 5 |
Actionability | Fully executable guidance throughout: exact commands ('git rev-parse --abbrev-ref HEAD', 'git rev-list --left-right --count HEAD...@{u}', 'export SHA=$(git rev-parse HEAD)'), exact quoted user prompts, explicit exit-code semantics (0/1/2/other) for the CI status command, and a computable cap formula ('ITER_CAP = max(1, ceil(<budget> / <poll interval>))'). Copy-paste ready and covering the common cases, matching anchor 5. | 5 / 5 |
Workflow Clarity | Steps 1-7 are clearly sequenced with explicit validation checkpoints (post-push verification that @{u} equals HEAD, the CI poll loop with terminal-state handling, and a full verification checklist with feedback paths for every failure mode). Two genuine defects hold it below anchor 5: step 6's 'Absent' bullet embeds the entire step 7 text inside a broken parenthetical continuation (lines 70-71), and step 7's heading is mangled as '7. **7. **TOTAL STOP**' with unbalanced bold markers (lines 98-99), making the terminal flow ambiguous on a first read. | 4 / 5 |
Progressive Disclosure | A single self-contained file, well-sectioned (Overview/Context/Rules/Instructions/Verification), with no bundle files and no buried or nested references — consistent with its declared 'Self-contained' design. It exceeds the 50-line simple-skill exception at ~117 lines, and the CI-await subsection (~40 lines of polling and handoff logic) is substantial enough that it could plausibly live in a separate reference file, which is the minor organization gap that fits anchor 4 rather than 5. | 4 / 5 |
Total | 17 / 20 Passed |