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.
Highly actionable content with complete executable examples, a clearly sequenced workflow with verification steps, and good single-file organization. The main weakness is redundancy: the error-signaling pattern and several Common Issues entries duplicate the Instructions and Critical sections.
Suggestions
Consolidate the throw new Error('__exit__') guidance into one canonical statement (e.g., the Critical section or step 2) and reference it elsewhere instead of restating it in step 5 and two Common Issues entries.
Make step 5's verification checklist executable by naming the concrete commands to run (e.g., the project's build or typecheck command) rather than advisory checks like "Verify the function exports correctly".
Trim Common Issues entries that merely restate Instructions (telemetry, option declaration, subcommand registration) to the ones covering genuinely new failure modes (module export errors, .ts vs .js import extensions).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and project-specific, assuming Claude's competence with TypeScript and Commander, but the throw new Error('__exit__') pattern is restated in four places (Critical, step 2, step 5, and two Common Issues entries), and Common Issues substantially restate the Instructions. It fits anchor 3 (mostly efficient but could be tightened) better than anchor 4, where over-explanation would be only minor. | 3 / 5 |
Actionability | All three examples (simple command, subcommand, option parsing) include complete, copy-paste-ready code with exact export signatures, import paths with .js extensions, and registration snippets; the guidance is fully executable. | 5 / 5 |
Workflow Clarity | Five clearly sequenced steps end in a "Verify before proceeding" checklist, and the Common Issues section provides cause/fix feedback loops. However, the verification checkpoints are advisory rather than executable (no build or typecheck command to run), leaving minor validation gaps that keep it below anchor 5. | 4 / 5 |
Progressive Disclosure | No bundle files exist, so everything is in SKILL.md, which is appropriately sized and well-organized into Critical, Instructions, Examples, and Common Issues sections with clear headers. Minor gaps (stray --- separators between examples, no explicit navigation cues) keep it at anchor 4 rather than 5. | 4 / 5 |
Total | 16 / 20 Passed |