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 tightly written, well-sequenced procedural skill body that assumes Claude's competence and supplies concrete parameters, clear phases, and failure-mode recovery. The main gap is that the detailed procedures it points to live in context files that are not present in the local bundle, slightly weakening both actionability and progressive disclosure.
Suggestions
Consolidate the repeated 'size is an output, never a target' emphasis into a single canonical statement and reference it, to tighten conciseness.
Inline at least a minimal executable snippet for the materialize step (the count-only entity find with format: "none" and the radius filter) so the core action is copy-paste-ready, not only described.
Either ship the referenced context files (context/discovery.md, scoring.md, profiling.md, record.md, adjacencies.md) in the bundle or note explicitly that they are owned by the parent audience-analyze plugin, so the one-level-deep references resolve verifiably.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense procedural guidance that assumes Claude's competence and does not explain basic concepts, with concrete parameters ('format: "none"', 'entity_type: "person"'); it earns 4 rather than 5 because some emphases repeat across sections ('size is an output' appears in Purpose, step 1, How to behave, and Refuse cleanly) and a few long parentheticals could be trimmed. | 4 / 5 |
Actionability | It gives mostly executable guidance — a concrete command ('cat "<Bundle root>/context/scoring.md"'), specific parameters, role bounds ('5–20 defining, 0–3 must-haves, 0–5 exclusions'), and exact build semantics ('defining any-of, must-haves all-of, exclusions none-of'); it stops short of anchor 5 because the detailed procedures are deferred to referenced context files rather than fully copy-paste-ready inline. | 4 / 5 |
Workflow Clarity | The flow is clearly sequenced into four numbered phases with checkpoints ('let the operator amend before any search', 'end the turn at the pivot question', 'Exclusions are explicit-include only') plus a Failure modes section with recovery guidance; it is not 5 because the main flow lacks an explicit validate→fix→retry loop of the kind anchor 5 illustrates, though the operation is read-only so the destructive-cap does not apply. | 4 / 5 |
Progressive Disclosure | Structure is good — a clear overview (Purpose, Works with, Language) with one-level-deep, well-signaled references to context files ('context/discovery.md', 'context/scoring.md', 'context/profiling.md'); it does not reach 5 because no bundle files exist in references/scripts/assets to confirm the referenced paths resolve, and references use a '<Bundle root>' placeholder rather than verified local paths. | 4 / 5 |
Total | 16 / 20 Passed |