Content
63%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 delivers genuinely useful conventions — typed error hierarchies, the { error: { code, message } } envelope, retry policy, and a pre-merge checklist — with executable examples in three languages. Its weaknesses are token efficiency (large amounts of standard boilerplate inline) and the lack of progressive disclosure, since everything lives in one long file rather than being split into references.
Suggestions
Split the per-language pattern libraries into references/ files (e.g. references/typescript.md, references/python.md, references/go.md) and keep SKILL.md to the core principles, envelope contract, and merge checklist, signaling each reference clearly from a short overview section.
Trim code Claude can already write from memory — the full ErrorBoundary class and standard exception hierarchies — down to the conventions that matter (base AppError shape with code/statusCode fields, error-code naming) to cut a large fraction of the token cost.
Fix the Go and TypeScript examples so they are copy-paste runnable: add the missing fmt, log/slog, and zod imports and resolve the undefined sql.ErrNoRows and chi.URLParam references in the Go snippet.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The content is mostly efficient but spends many lines on boilerplate Claude already knows — a full React ErrorBoundary class, standard exception hierarchies, and generic API-handler scaffolding — where the conventions (error codes, envelope shape, principles) are what actually add value. | 3 / 5 |
Actionability | Concrete, near-complete code in all three languages plus a working withRetry helper, a message map, and a merge checklist. Minor gaps keep it below fully copy-paste ready: the Go example uses fmt, slog, and chi without imports, and z.ZodError appears with no import. | 4 / 5 |
Workflow Clarity | Not a multi-step process skill, but it is clearly organized: activation triggers, five numbered principles, per-language patterns, and an explicit 8-item validation checklist ("Before merging any code that touches error handling") serving as an end-state checkpoint. | 4 / 5 |
Progressive Disclosure | Headers and sectioning are good, but this is a ~378-line monolithic SKILL.md with no reference files. The per-language pattern libraries clearly belong in separate files (e.g. references/typescript.md), with SKILL.md as the overview. | 3 / 5 |
Total | 14 / 20 Passed |