Content
75%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 well-structured, disciplined instruction-only skill with an explicit fail-fast validation loop and clear handoff boundaries. Main deductions are a duplicate step number in the Procedure, some cross-section redundancy, and reference paths that dangle relative to the shipped bundle.
Suggestions
Fix the duplicate step numbering in Procedure (two consecutive steps are both numbered 6, lines 63 and 65) so the sequence reads 1-8 without ambiguity.
Verify or repair the out-of-package reference paths (../../references/*.md and Plugins/harness-engineering/references/deferred-context-index.md) so every cited contract resolves from the skill directory, and link the unmentioned references/task-profile.json or remove it.
Merge the overlapping Outputs and Output Format sections and deduplicate side-effect classification between Preconditions and Safety Boundaries to tighten token efficiency.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and directive with no concept explanations Claude already knows (e.g., "Prove the failure before fixing it", "A failing label is not a reproduction"). It falls short of anchor 5 due to redundancy across sections: side-effect classes appear in both Preconditions and Safety Boundaries, and Outputs overlaps Output Format ("Use concise sections: Reproduction, Root Cause, Patch, Validation..." restates the Outputs field list). | 4 / 5 |
Actionability | For an instruction-only skill the guidance is largely concrete: named validation gates ("run strict audit, OpenClaw, OpenAI format lint, progressive disclosure lint, Plugin Eval"), exact output fields ("schema_version: 1", "pass|fail|blocked"), and a numbered procedure with specific directives like "Add or name regression protection that would fail before the fix and pass after it". It is not anchor 5 because several steps defer to contracts whose execution details are not in the bundle ("Apply the git staging contract", "Apply the visual reference contract"), leaving minor gaps in copy-paste executability. | 4 / 5 |
Workflow Clarity | The Procedure is a clear 1-8 sequence with explicit validation checkpoints and a fail-fast feedback loop ("stop at the first failed gate, classify it, fix or block it, then rerun before broader validation"), which satisfies the validation requirement for mutation-class operations. It misses anchor 5 due to a sequencing defect — two steps are both numbered 6 (the git staging step and the media-storage step) — and some steps are conditional abstractions ("Apply the visual reference contract when...") rather than fully specified actions. | 4 / 5 |
Progressive Disclosure | SKILL.md is a lean overview with a dedicated, purpose-labeled References section ("Contract and eval routing: references/contract.yaml, references/evals.yaml" — both files exist in the bundle), and the Stage Arc Boundary section clearly signals when to load its contract. Scored against the actual bundle structure it is not anchor 5 because four of the six reference entries (../../references/stage-arc-boundary-contract.md, ../../references/subagent-call-contract.md, ../../references/visual-reference-contract.md, and Plugins/harness-engineering/references/deferred-context-index.md) do not resolve within this package, and references/task-profile.json exists but is never linked. | 4 / 5 |
Total | 16 / 20 Passed |