Content
81%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-engineered process skill: phased workflow with hard gates, feedback loops, concrete prompts, paths, and budgets, plus a cleanly organized one-level-deep reference bundle. Main issues are moderate redundancy (repeated gate emphasis, When-to-Use sections restating the description) and one referenced script that is absent from the bundle.
Suggestions
Fix or remove the dangling `./scripts/run-query-benchmarks.sh` reference in the schema-migration usage example — the scripts/ directory does not exist in the bundle, so the example's Step 2 is not executable as written.
Tighten conciseness by stating the Phase 1 entry gate once (e.g., in section 1.5) instead of three times, and trim the 'When to Use'/'When Not to Use' bullets that restate the frontmatter description verbatim.
Move the per-anti-pattern 'Why:' explanations into references/reference.md, keeping one-line imperatives in SKILL.md, to further reduce token load without losing rationale.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with operational specifics (token budgets "target 300 tokens inline, flex 200-500, hard cap 600", collision handling, naming conventions) and explains little Claude already knows, matching anchor 4. It is not a 5 because the Phase 1 gate emphasis is repeated in three places ("ENTRY GATE: Phase 2 does not start...", section 1.5, and bolded MANDATORY framing) and "When to Use"/"When Not to Use" substantially restate the frontmatter description. | 4 / 5 |
Actionability | Guidance is highly concrete for an instruction skill: exact AskUserQuestion prompt text ("Proceed with probe? [Yes / Revise hypothesis / Revise criteria / Abort]"), exact artifact path pattern `thinking/probes/{project}/{date}-{slug}-llm.md`, an explicit output format line, and a classification table — anchor 4. It falls short of anchor 5 partly because the migration example invokes `./scripts/run-query-benchmarks.sh`, which does not exist in the bundle. | 4 / 5 |
Workflow Clarity | The two-phase workflow has an explicit entry gate with a loop back to 1.1–1.4 on revision, pre-execution confirm/refute/surprise criteria as a validation checkpoint, a mandatory artifact-persistence step before the exit gate, an AskUserQuestion empty-answer fallback, and a self-transition feedback loop for partial results — matching anchor 5's 'explicit validation steps; feedback loops for error recovery'. Anchor 4 would require missing or merely implicit checkpoints, which is not the case here. | 5 / 5 |
Progressive Disclosure | Four references/ files exist in the bundle, are each signaled with a one-line purpose in a References section, and are one level deep (reference.md contains no nested file references) — close to anchor 5. The gap keeping it at 4 is the dangling `./scripts/run-query-benchmarks.sh` reference in the Usage Examples, which points at a nonexistent bundle file. | 4 / 5 |
Total | 17 / 20 Passed |