Content
56%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 Membrane commands and a well-validated connection workflow. Its weaknesses are conciseness (intro padding explaining concepts Claude knows plus an inlined 17-row action table) and progressive disclosure (everything inlined with no reference files to offload the bulk API reference).
Suggestions
Delete the introductory paragraph explaining what Semgrep is and who uses it; Claude already knows this — start directly at how to use the skill.
Move the 17-row 'Popular actions' table into a references file (e.g. references/actions.md) and keep only the 4-5 most common actions inline, linking out for the rest.
Fix the connection workflow numbering (label the 'ensure' step as 1a, add an explicit 'Step 2: Search for actions' heading) so the cross-reference and sequence are unambiguous.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The opening paragraph explains what Semgrep is and who uses it ('Semgrep is a static analysis tool for finding bugs... Developers and security engineers use it...') — concepts Claude already knows — and the 17-row popular-actions table inlines bulk API reference, giving 'several unnecessary explanations or padded sections'. It is not 3 (the padding is more than 'some' over-explanation) and not 1 (commands and code are largely efficient rather than severely padded). | 2 / 5 |
Actionability | Concrete, copy-paste-ready commands appear throughout (npm install, membrane login, connection ensure, action list/run, request) plus a proxy-options table and named action keys. It is mostly executable with minor gaps (placeholders like CONNECTION_ID/<actionId>/QUERY need substitution, and there is no single end-to-end worked example), fitting anchor 4 rather than the fully-covered anchor 5. | 4 / 5 |
Workflow Clarity | The connection flow is a clear sequence with explicit validation (READY/BUILDING/CLIENT_ACTION_REQUIRED/CONFIGURATION_ERROR state checks) and a feedback loop ('poll again... to check if the state moved to READY'). It misses a clean 5 because of inconsistent numbering ('#### 1b' with no 1a, a dangling 'Step 2' reference, and no unified connect→search→run numbering). | 4 / 5 |
Progressive Disclosure | Section headers are clear and well-organized, but with no bundle files the entire ~160-line guide is inlined, including a 17-row bulk action-reference table that 'should be separate' but is inline, and there are no one-level-deep references to offload detail. It is above 2 (good structure, not minimal/buried) but below 4 (the inlined reference bulk is more than a 'minor organization gap'). | 3 / 5 |
Total | 13 / 20 Passed |