Content
40%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 is structurally broken and largely boilerplate: malformed code fencing, auto-generated filler in the use-case sections, generic non-actionable Instructions, and a reference to a file that does not exist. The only useful part is the concrete marker-pattern reference near the end.
Suggestions
Fix the malformed markdown: open a fenced code block before the ls commands and remove the stray closing fence, or move the diagnostic commands into a proper '## Validation steps' sequence.
Replace the auto-generated 'Use this skill when' / 'Do not use this skill when' filler (which repeats 'check if conductor directory exists') with real trigger conditions tied to the Conductor artifacts.
Either create the referenced resources/implementation-playbook.md (and place it under references/) or remove the dangling reference, since no references directory currently exists.
Turn the generic Instructions ('Clarify goals...', 'Apply relevant best practices') into a concrete validation workflow with explicit checkpoints (check files exist -> parse tracks.md -> report missing artifacts).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is short, but it wastes tokens on auto-generated boilerplate ('check if conductor directory exists' repeated in the Use/Do-not-use sections) and generic Instructions ('Clarify goals... Apply relevant best practices'), so it is mostly efficient but contains clear padding. | 3 / 5 |
Actionability | It includes concrete executable shell commands (ls of conductor files) and concrete marker patterns, but the commands are presented with broken fencing and no surrounding workflow, and the Instructions are vague boilerplate, leaving guidance incomplete. | 3 / 5 |
Workflow Clarity | There is no real sequenced workflow — the opening lines are loose ls commands with a stray closing fence, and the Instructions are circular ('Provide actionable steps and verification') with no validation checkpoints despite this being a validation skill. | 2 / 5 |
Progressive Disclosure | The body references 'resources/implementation-playbook.md' but no references/resources directory exists, so the reference is broken; structure is minimal and the single deferred reference is not actually loadable. | 2 / 5 |
Total | 10 / 20 Passed |