Content
96%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.
An exemplary audit skill body: tight rules, executable BAD/GOOD pairs for every pattern, a lookup table, and explicit verification with measurable thresholds. The only structural nit is that nothing is offloaded to reference files, leaving the SKILL.md slightly monolithic.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean: a one-line framing ("The #1 bug category in VS Code"), one rule per step, minimal BAD/GOOD snippets, and no explanation of concepts Claude already knows. Every token earns its place, matching the anchor-5 example's density. | 5 / 5 |
Actionability | Each step gives copy-paste-ready TypeScript showing both the anti-pattern and the fix, with exact APIs (addDisposableListener, Event.once, MutableDisposable, store.add(model.onWillDispose(...))), a decision table, and concrete commands including `npm run perf:chat-leak --messages 20 --verbose`. Not below 5 because the examples cover all the common cases the description claims. | 5 / 5 |
Workflow Clarity | "Work through each check in order" gives an explicit six-step sequence, and the Verification section provides feedback checkpoints (listener counts before/after, ensureNoDisposablesAreLeakedInTestSuite, object-count stabilization, a 2 MB/msg leak slope threshold). This matches the anchor-5 pattern of sequence plus explicit validation steps. | 5 / 5 |
Progressive Disclosure | Sections are clearly organized (When to Use, six audit steps, Quick Reference table, Verification) and the only external pointer (the `chat-perf` skill) is explicitly signaled with usage flags. It falls short of a 5 because at ~148 lines the skill is fully monolithic with no split of any material into reference files, and the under-50-line simple-skill exception does not apply. | 4 / 5 |
Total | 19 / 20 Passed |