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 body is a lean, well-sectioned overview whose entire substance is deferred to 'resources/implementation-playbook.md' — a file that is absent from the bundle. What remains is a list of vague, high-level directives with no code, commands, or validation steps, so the skill delivers almost no actionable guidance as shipped.
Suggestions
Create resources/implementation-playbook.md with the promised detailed patterns and examples (gather, TaskGroup, queues, timeouts, cancellation), or remove the dangling reference and inline a compact quick-start instead.
Add at least one executable code example to the body (e.g., asyncio.gather with timeouts or a worker-queue pattern) so the skill is actionable without the external file.
Rewrite the Instructions as a concrete sequenced workflow with a verification step (e.g., 'run the test suite with pytest-asyncio to confirm tasks are cancelled and timeouts fire'), replacing vague directives like 'Include testing and debugging guidance'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is short and well-sectioned, but carries noticeable redundancy: the frontmatter description is restated verbatim under the title, 'resources/implementation-playbook.md' is referenced three separate times (instruction bullet, standalone sentence, and Resources section), and meta-bullets like 'Include testing and debugging guidance for async code paths' add no instruction. This fits anchor 3 ('mostly efficient but... could be tightened') better than anchor 4, where trimming would be minor. | 3 / 5 |
Actionability | There is no code, command, or executable example anywhere in the body — only high-level directives like 'Pick concurrency patterns (tasks, gather, queues, pools) with cancellation rules' and 'Add timeouts, backpressure, and structured error handling', matching anchor 2 ('minimal concrete guidance; high-level hints but missing the specific steps'). It is not a 1 because specific primitives are named rather than pure abstraction, and not a 3 because the promised concrete guidance lives in a file that does not exist in the bundle. | 2 / 5 |
Workflow Clarity | The Instructions section gives a rough sequence (clarify workload → pick patterns → add timeouts/backpressure → testing → open playbook), but the steps are poorly defined — 'Include testing and debugging guidance for async code paths' is incoherent as a workflow step — and there are no validation checkpoints or error-recovery guidance, matching anchor 2 ('rough sequence present but many gaps; steps poorly defined; validation absent'). It does not reach anchor 3, whose steps are at least concretely defined. | 2 / 5 |
Progressive Disclosure | Although sections (Use when / Do not use / Instructions / Resources) are well organized and the reference is clearly signaled, the referenced file 'resources/implementation-playbook.md' does not exist in the bundle — there is no resources/ directory at all. Per the guideline to score against the actual bundle structure, the entire disclosure chain to the detailed content is broken, a more fundamental structural failure than anchor 3's 'references present but not clearly signaled'; anchor 2 ('minimal structure... references buried/failing to deliver) is the best fit, while anchor 1's deeply-nested or monolithic condition does not apply. | 2 / 5 |
Total | 9 / 20 Passed |