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 tight, well-structured body with concrete code examples and a clear decision tree for error-handling patterns. The main gaps are undefined helper functions in examples and minor redundant prose, rather than structural or verbosity problems.
Suggestions
Define or stub the helper functions used in examples (isTransient, sleep, logger, ctx) so the withRetry and logging snippets are fully copy-paste executable.
Remove or shorten the redundant opening line that restates the frontmatter description, since the body's sections already establish scope.
Add a brief note on what to do when retries are exhausted (e.g., log final failure with attempt count and surface a user-facing error) to give the retry workflow an explicit feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and mostly efficient with focused Bad/Good code pairs that earn their place, but the opening line 'Use this skill when reviewing or writing code that needs to handle unexpected failures' duplicates the frontmatter and a few explanatory sentences could be trimmed, so just below anchor 5. | 4 / 5 |
Actionability | Provides copy-paste-ready guidance (the withRetry function, structured logger.error calls, a concrete user-facing JSON response), but relies on undefined helpers like isTransient(), sleep(), and ctx without defining them, leaving minor gaps versus fully executable anchor 5. | 4 / 5 |
Workflow Clarity | The numbered 'ask in order' decision tree is a clearly sequenced workflow covering transient, user-input, permanent, boundary, and fallback cases; no validation checkpoints are required since the skill is advisory rather than destructive, but it lacks the explicit feedback loops of anchor 5. | 4 / 5 |
Progressive Disclosure | Well-organized into clearly headed sections (Don't swallow errors, Fail fast at boundaries, Logging, User-facing errors) with content appropriately contained in one file and no need for external references; just over the ~50-line simple-skill threshold, so it does not quite reach the no-reference anchor-5 exception. | 4 / 5 |
Total | 16 / 20 Passed |