Content
32%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 content has a useful kernel — the file-existence commands and the Pattern Matching section are concrete — but it is wrapped in malformed markdown, templated filler sections, and generic instructions that never define an actual validation workflow. The dangling reference to a nonexistent playbook compounds the structural problems.
Suggestions
Replace the generic Instructions with a numbered validation workflow: (1) run the file-existence checks, (2) verify status markers in tracks.md/plan.md against the documented patterns, (3) validate track IDs against the <type>_<name>_<YYYYMMDD> format, (4) report or fix discrepancies — with explicit checkpoints.
Remove the templated 'Use this skill when' / 'Do not use this skill when' filler (the description already covers triggering) and the boilerplate bullets, and fix the malformed opening: wrap the shell commands in a proper fenced code block under a real heading.
Either add the referenced resources/implementation-playbook.md to the bundle or delete the reference to it; if the detailed validation criteria live elsewhere, point to a file that actually exists.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is noticeably padded: the templated 'Use this skill when' and 'Do not use this skill when' sections mechanically repeat the slug 'check if conductor directory exists' three times, and the Instructions bullets ('Clarify goals, constraints, and required inputs', 'Apply relevant best practices and validate outcomes') are generic boilerplate that adds no information. This matches 'several unnecessary explanations or padded sections' rather than the mostly-efficient anchor 3. | 2 / 5 |
Actionability | There is some genuinely concrete guidance — the shell commands ('ls -la conductor/tracks/', the required-files check listing index.md, product.md, tech-stack.md, workflow.md, tracks.md) and the pattern-matching examples (status markers, track ID format) — but the core validation procedure is missing: no criteria for what 'consistency' or 'correctness' means or what to do with the results. Concrete but incomplete matches anchor 3; a 4 would require the validation steps themselves to be executable. | 3 / 5 |
Workflow Clarity | The body opens with three loose, unfenced shell snippets (implying only a rough check-dir/check-tracks/check-files order) and then jumps to filler sections and generic instructions with no sequenced procedure and no validation checkpoints or error-recovery guidance — in a skill whose entire purpose is validation. This matches 'rough sequence present but many gaps; validation absent' rather than the listed-but-checkpoint-missing anchor 3. | 2 / 5 |
Progressive Disclosure | Although section headers exist, the structure is broken: the body starts with commands floating before any header behind a stray closing code fence, and the only external reference ('open resources/implementation-playbook.md') points to a file that does not exist in the bundle — a dangling reference, not merely an unclearly-signaled one. This matches 'minimal structure; references are buried' territory rather than the could-be-better-organized anchor 3. | 2 / 5 |
Total | 9 / 20 Passed |