Content
47%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 maximally concise but hollow: it is a pointer file that delegates everything to external files that are not bundled, creating a self-referential read of `.claude/skills/harvest/SKILL.md` and leaving the actual harvest workflow, staging process, and approval validation completely unspecified. As evaluated standalone, an agent receiving only this bundle could not execute the skill.
Suggestions
Inline or bundle the actual playbook content: either replace the pointer with the harvest workflow itself, or ship the referenced files (playbook, `.agents/rules/cog.md`, `.agents/agents/<name>.md`) inside references/ so the delegation resolves.
Resolve the self-reference: 'Read `.claude/skills/harvest/SKILL.md`' points at this very file — point at a distinct playbook path (e.g. `references/playbook.md`) or merge its content into the body.
Add the validation checkpoint the description promises: an explicit 'Never write durable knowledge without user approval' gate and a verify-before-staging step in the body's workflow, since the skill stages durable knowledge.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is six lean lines with zero padding or explanation of concepts Claude already knows — 'Read `.claude/skills/harvest/SKILL.md` and execute it exactly as written' plus one substitution rule. Every token earns its place, matching the 'lean and efficient' anchor exactly; it cannot be 4+verbose because there is no over-explanation anywhere. | 5 / 5 |
Actionability | The only guidance is an indirection: 'Read `.claude/skills/harvest/SKILL.md` and execute it exactly as written' and 'follow `.agents/rules/cog.md`' — no actual steps for capturing learnings, staging, or proposing patches appear in this body, and the referenced files are not part of the bundle (`.claude/skills/harvest/SKILL.md` is self-referential, implying recursion). This matches the 'minimal concrete guidance; missing the specific steps to execute' anchor and mirrors the bad example of a pointer-only skill; not 3 because there are no partially-executable steps at all — the work is entirely delegated elsewhere. | 2 / 5 |
Workflow Clarity | The sequence is roughly 'read playbook → execute it → then follow `.agents/rules/cog.md`' with one conditional substitution rule — a rough order with major gaps: no definition of what executing entails, no checkpoints, and no validation despite the skill involving staging durable knowledge for promotion (the description's approval gate never appears in the body). Fits 'rough sequence present but many gaps; validation absent'; not 3 because even the step list itself is incomplete rather than merely un-validated. | 2 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, assets/) yet the body's entire substance lives in external files — `.claude/skills/harvest/SKILL.md`, `.agents/rules/cog.md`, `.agents/agents/<name>.md` — none of which are shipped, and the primary reference is self-referential. The skill is effectively a pointer shell with buried references, matching 'minimal structure; references are buried'; not 1 because there is only one level of indirection stated, not a demonstrated 2+ level chain; not 3 because there is no in-body overview content at all to organize. | 2 / 5 |
Total | 11 / 20 Passed |