Content
77%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 an exceptionally actionable, well-sequenced orchestration procedure with deterministic script-based validation and explicit handling of edge cases and tiers. Its weaknesses are length and inline density — repeated minimal-path rules and full output templates that belong in reference files — which cost both conciseness and progressive disclosure.
Suggestions
Move the two output templates (phase shape and minimal shape, Step 7) into a references/ file and keep only the formatting rules inline.
Consolidate the minimal-path rules stated across Steps 2, 4m, 5, 8, and 9 into Step 4m, leaving one-line pointers elsewhere.
Trim incident-reporting prose such as the 'any_of ... already produced a false incomplete once' backstory down to the rule itself.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 370-line body is dense with genuinely non-obvious project rules (status mapping, minimal-path routing), but includes unnecessary padding: the minimal-path handling is restated across Steps 2, 4m, 5, 7, 8, and 9; and the `any_of` section recounts a past incident ('hand-evaluating it has already produced a false incomplete once') plus rationale that could be trimmed. Mostly efficient but could be tightened, matching anchor 3; it is not severely padded enough for anchor 2's 'several unnecessary explanations'. | 3 / 5 |
Actionability | Guidance is fully executable: exact commands ('bash .claude/scripts/artifact-check.sh --phase [current-phase]', the Grep pattern for STATUS/CHECKPOINT markers), an explicit status-to-report mapping table, exact invocation constraints (no absolute path, no cd, no 2>&1, matching the permission grant), and copy-paste output templates for both the phase and minimal shapes. This matches anchor 5's 'copy-paste ready commands covering common cases'. | 5 / 5 |
Workflow Clarity | Nine clearly sequenced steps with explicit branching (the minimal path is routed deterministically at Step 2 into Step 4m, with explicit skip rules for Steps 5 and 8), deterministic validation via artifact-check.sh instead of ad-hoc globbing, explicit status mapping, and a defined recovery path for NO_CHECK/MANUAL steps (ask the user). Anchor 5's 'clear sequence with explicit validation steps and error-recovery guidance' fits; the skill is read-only so no destructive-operation cap applies. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned and its external references (.claude/docs/workflow-catalog.yaml, workflow-modes.md, config-resolution.md, settings-guidance.md § 4) are clearly signaled with paths, but no bundle files exist and all 370 lines live inline in SKILL.md. Content that could sit in reference files — the two full output templates, the minimal-path rules repeated across steps, the status mapping table — is inlined, matching anchor 3 ('some structure but could be better organized; content that should be separate is inline') rather than anchor 4's 'most content appropriately placed'. | 3 / 5 |
Total | 16 / 20 Passed |