Content
86%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 lean, highly actionable skill that teaches Knip-specific configuration without padding, with a clear seven-step sequence and appropriate external references. Its two weaknesses are minor: a small amount of citation/description redundancy, and the absence of an explicit verify-after-cleanup feedback loop.
Suggestions
Consolidate the three "Per the [Knip docs]." citations (after Step 1, Step 5, and in References) into a single mention in the References section, and drop or trim the "When to use" bullets that restate the frontmatter description.
Close the feedback loop after cleanup: add a final step to re-run `npx knip` and expect exit code 0 to confirm all flagged files, dependencies, and exports were actually removed (especially after `npm uninstall`).
In Step 7, show the exact `tags: ["public"]` knip.json snippet alongside the JSDoc `@public` mention so the public-API escape hatch is copy-paste ready instead of implied.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and tool-specific — config JSON, commands, and tables with no explanations of concepts Claude already knows (the "Entry vs project" note is genuinely non-obvious Knip semantics). Minor trimmable redundancy keeps it below a 5: "Per the [Knip docs]." appears after Step 1, Step 5, and again in References, and the "When to use" section largely restates the frontmatter description. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready guidance covering the common cases: "npm init @knip/config", "npm install -D knip typescript @types/node", a complete knip.json, "npx knip --max-show-issues 5", the baseline command "npx knip --no-exit-code > knip-baseline.txt", GitHub Actions YAML, and a per-issue triage table with concrete actions. The only terse spot ("tags: [\"public\"]" config) is a minor edge-case detail, not a common-case gap. | 5 / 5 |
Workflow Clarity | Seven clearly sequenced steps (install → configure entry → first run → plugins → limit output → CI gate → triage) with checkpoints present in the form of the Step 7 triage table ("Sometimes (public API)") and the anti-pattern guidance "Manual review first; --fix after baseline clean", so the destructive/batch cap at 3 does not bind. It falls short of a 5 because there is no explicit post-cleanup verification step (e.g., re-run knip to confirm a clean exit after deletions or npm uninstall). | 4 / 5 |
Progressive Disclosure | No bundle files exist and none are needed: the skill is well-organized into short numbered sections and tables, and both references ([Knip docs], [Knip plugins page]) are external URLs that are one level deep and clearly signaled both inline and in a dedicated References section. Nothing inlined here belongs in a separate file, and the plugin list is correctly delegated to the external plugins page rather than duplicated. | 5 / 5 |
Total | 18 / 20 Passed |