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.
The body is highly actionable with copy-paste CLI commands, explicit connection-state checkpoints, and a feedback loop for readiness polling, but it carries some definitional/conversational padding and has inconsistent step numbering plus an empty overview stub.
Suggestions
Remove the "Npm is a package manager for Javascript..." intro and the "auth plumbing"/"fastest way" commentary; Claude already knows what npm is.
Fix the step numbering: add a labeled "Step 1"/"Step 2" (or rename) so the "skip to Step 2" reference resolves, and add the missing "1a" alongside "1b".
Either fill in the "Npm Overview" bullets (Package/Version/User/Organization) with real content or delete the empty stub.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient CLI commands, but it opens with a definition Claude already knows ("Npm is a package manager for Javascript...") and includes padded commentary like "so you can focus on the integration logic rather than auth plumbing" and "This is the fastest way to get a connection"; fits "mostly efficient but includes some unnecessary explanation" rather than a 4 which would require only minor trimming. | 3 / 5 |
Actionability | It provides copy-paste-ready commands for every common case — install, login (including headless completion), connection ensure, polling get --wait, action list/run with --input JSON, and a full proxy-request flag table — covering the common cases fully. | 5 / 5 |
Workflow Clarity | The connect-then-poll flow has explicit state checkpoints (READY/BUILDING/CLIENT_ACTION_REQUIRED/CONFIGURATION_ERROR) with a feedback loop (poll again after user action), but step labels are inconsistent — "skip to Step 2" with no matching Step 2 header and a "1b" with no "1a" — so it stops short of the fully checklisted level 5. | 4 / 5 |
Progressive Disclosure | No bundle files exist, so the single SKILL.md is scored on its own sectioning; it is organized into logical sections (Install, Auth, Connecting, Searching, Running, Proxy, Best practices), but the empty "Npm Overview" stub (Package/Version/User/Organization with no content) is a minor organization gap that keeps it below a 5. | 4 / 5 |
Total | 16 / 20 Passed |