Content
96%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 strong skill body: executable commands with exit-code semantics, a gated multi-step workflow with explicit validation and escalation, and well-signaled one-level-deep references that all exist. The only deductions are inline time-sensitive details (pinned version, dated audit path) and a dangling audit reference, which cost it slightly on conciseness and progressive disclosure.
Suggestions
Move the pinned-version and dated-audit details (npx -y human-review@0.6.0, audit/human-review-2026-08/) into a dedicated 'Maintenance / pinned dependencies' section or a reference file, so time-sensitive values are isolated from the evergreen instructions.
Trim inline rationale clauses (e.g., the G7 typo explanation in the gate table) to their rule statements and move the 'why' to references/review_loop_discipline.md, which already covers loop discipline reasoning.
Either include the audit/human-review-2026-08/ artifact in the bundle or drop the pointer, since it currently leads nowhere for a reader of this skill.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and high-signal — a terse sh loop block, a gate-rule table, hard rules — with no explanation of concepts Claude already knows. However, it inlines time-sensitive details ('always pinned (`npx -y human-review@0.6.0`)', 'See `audit/human-review-2026-08/`') outside any old-patterns/deprecated section, and rationale clauses like 'a mistyped severity silently downgrades to NIT, so a real blocker can be lost to a typo' could be trimmed. Anchor 4 (efficient, minor trimming possible) fits better than 5. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands with exact script paths and flags: 'python3 $S/human_gate.py open plan.md --launch', 'collect plan.md --output json', 'close plan.md', plus '--sample' covering the whole loop in ~1s. Exit-code semantics ('exit 2 = NOT done', '0 clear·2 blocked·3 collect·4 none') make outcomes unambiguous — matches anchor 5. | 5 / 5 |
Workflow Clarity | The loop is a clear sequence (open → END TURN → status → collect → apply every item → close) with explicit validation checkpoints: the exit-2 gate, G4 (sidecar changed after last collect), G7 (integrity), and round-cap exhaustion that 'escalate[s], never pass[es]'. Batch feedback handling includes validation (quotes checked against raw and rendered text), so no cap applies; anchor 5's explicit validation and feedback loop are present. | 5 / 5 |
Progressive Disclosure | References are one level deep, clearly signaled, and real: 'Reasoning lives in `references/` — human-in-the-loop canon, feedback batching, loop discipline' (all three files exist) and 'Worked example and JSON contract are in `assets/`'. Minor gap: the cited 'audit/human-review-2026-08/' path is absent from this bundle, and some inline rationale (the derivation note, forcing-question commentary) could live in references. Anchor 4 fits; not 5 because of the dangling pointer. | 4 / 5 |
Total | 18 / 20 Passed |