Content
40%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 skill delivers abundant, mostly executable code but is severely over-long for a SKILL.md, re-teaching standard error-handling knowledge Claude already has across four languages. More importantly, all six advertised bundle files are missing from the repository, making the Resources section dead references, and there is no decision workflow for choosing among the patterns.
Suggestions
Create the referenced bundle files (references/exception-hierarchy-design.md, references/error-recovery-strategies.md, references/async-error-handling.md, assets/error-handling-checklist.md, assets/error-message-guide.md, scripts/error-analyzer.py) or remove them from the Resources section.
Trim or move per-language sections (Rust '?'/'From' mechanics, Go '%w' wrapping, Python context managers, Option/Maybe basics) into one-level-deep per-language reference files, keeping only a pattern-selection overview in SKILL.md.
Add a short decision workflow (e.g., categorize the error as recoverable/unrecoverable, then expected/unexpected, then map to exception vs Result vs panic, with a pointer to the recovery-strategies reference) so Claude has a sequence to follow instead of a catalog to browse.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | At roughly 640 lines, the body spends whole sections on concepts Claude already knows (Rust '?' operator and From impls, Go '%w' wrapping and errors.Is/As, Python context managers, Option/Maybe basics) and repeats near-identical custom-error-hierarchy patterns across four languages. Not a 1 because the prose itself is tight and the bulk is code rather than padded explanation. | 2 / 5 |
Actionability | Mostly executable, copy-paste-ready code (retry decorator with backoff, circuit breaker, Result helpers, error collector), but minor gaps remain: 'logger', 'NetworkError', and 'requests' are used without definition, 'datetime' is unimported in the first example, and the 'or'-chained multi-fallback example silently skips falsy exchange rates. Not a 5 because these undefined symbols and the falsy-value bug keep the examples from running as-is. | 4 / 5 |
Workflow Clarity | The body is a reference catalog organized by topic, with no decision procedure or sequenced steps telling Claude how to categorize an error and select and apply a pattern; the 'When to Use Each' bullets are the only decision guidance. Not a 1 because section ordering (concepts, per-language patterns, universal patterns, pitfalls) gives a coherent rough reading sequence, and no destructive or batch operations exist that would require validation checkpoints. | 2 / 5 |
Progressive Disclosure | The Resources section clearly signals six bundle files (references/, assets/, scripts/error-analyzer.py), but none of those files or directories exist, so every reference is dead; meanwhile four languages' worth of pattern detail that belongs in separate per-language files is inlined. Not a 1 because the file is well-sectioned with headers and references are one level deep and prominently listed rather than buried. | 2 / 5 |
Total | 10 / 20 Passed |