Content
77%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 highly actionable with a well-sequenced, validated workflow and concrete CLI examples. Its weaknesses are abstract meta-framework padding that hurts token efficiency, and references that point to non-existent bundle files.
Suggestions
Remove the meta-framework scaffolding (SSL-primitive action table, PREPARE/ACQUIRE/ACT scene labels, LOCAL_FS/PROCESS/NETWORK scope table) to reduce abstract padding; keep the concrete commands, flags, and vendor matrix.
Create the referenced files (resources/execution-protocol.md, vendor-matrix.md, prompt-tips.md, checklist.md) or remove the dangling references so navigation targets resolve.
Move the large inlined vendor-strategy and policy detail into the referenced resource files, leaving SKILL.md a lean overview with clearly signaled one-level-deep links.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Concrete command/flag/vendor content is efficient, but substantial meta-framework scaffolding (SSL-primitive action tables, PREPARE/ACQUIRE/ACT scene labels, LOCAL_FS/PROCESS/NETWORK scope table) is abstract padding Claude does not need for this task. | 3 / 5 |
Actionability | Provides copy-paste ready commands with full flag lists, reference-image examples, an explicit mandatory auto-forward instruction with the exact command, and a concrete output filename/tree layout covering common cases. | 5 / 5 |
Workflow Clarity | Clear Entry→Scenes→Transitions→Failure/recovery→Exit sequence with explicit validation checkpoints (Clarification Protocol checklist, VERIFY scene, cost guardrail confirmation, doctor precheck, exit-code table) and feedback loops for the batch/paid generation operations. | 5 / 5 |
Progressive Disclosure | Section structure and one-level-deep reference signaling are good, but the referenced files (resources/execution-protocol.md, vendor-matrix.md, prompt-tips.md, checklist.md, ../_shared/core/context-loading.md) do not exist in the bundle, so navigation targets are missing and heavy policy/vendor content remains inlined. | 3 / 5 |
Total | 16 / 20 Passed |