Content
22%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill is essentially a hollow wrapper that defers all substantive content to an external resource file. The instructions section contains only generic, non-actionable platitudes that could apply to any skill. There are no concrete examples, code snippets, patterns, or specific guidance about error handling in the main file.
Suggestions
Add at least 2-3 concrete, executable code examples of core error handling patterns (e.g., Result types, exception hierarchies, retry logic) directly in the SKILL.md rather than deferring everything to the resource file.
Replace the generic instructions ('Apply relevant best practices and validate outcomes') with specific, actionable guidance such as 'Use custom exception hierarchies for domain errors' or 'Wrap external API calls in retry logic with exponential backoff'.
Add a clear workflow for implementing error handling in a new feature, with explicit steps like: 1) Identify failure modes, 2) Choose error strategy, 3) Implement with specific pattern, 4) Verify error paths with tests.
Remove the 'Do not use this skill when' section as it adds no value, and trim the 'Use this skill when' list to focus on the most important 2-3 triggers.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The content is relatively short but includes filler sections like 'Use this skill when' and 'Do not use this skill when' that add little value for Claude. The instructions are generic platitudes ('Clarify goals, constraints, and required inputs') rather than substantive guidance. | 2 / 3 |
Actionability | The instructions are entirely vague and abstract — 'Apply relevant best practices and validate outcomes' provides zero concrete guidance. There are no code examples, no specific commands, no patterns, and no executable content. Everything meaningful is deferred to an external file. | 1 / 3 |
Workflow Clarity | The four bullet points in Instructions are generic meta-instructions that could apply to any skill. There is no clear workflow, no sequencing of steps for error handling implementation, and no validation checkpoints. | 1 / 3 |
Progressive Disclosure | There is a reference to a resource file (resources/implementation-playbook.md) which is one level deep and clearly signaled. However, the SKILL.md itself contains almost no substantive overview content — it's essentially an empty shell pointing to another file, which undermines the purpose of progressive disclosure. | 2 / 3 |
Total | 6 / 12 Passed |