Content
85%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 well-structured specialist skill: a checkpointed five-step workflow, executable tool commands, a concrete finding template, and an exemplary reference table that pushes detail to one-level-deep bundle files. The only trim opportunities are the redundant knowledge-reference name list and the high-level Scope/Report steps.
Suggestions
Trim or merge the 'Knowledge Reference' name list into references/sast-tools.md and the other reference files, since most of those tools are already documented there with commands.
Add one concrete example under the Scope and Report steps (e.g., a scope confirmation checklist item or a one-line pointer to the report-template.md structure) so every workflow step has in-body executable guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is directive and lean with no concept explanations, but the bare "Knowledge Reference" list of ~25 tool/standard names partially duplicates what references/sast-tools.md already covers, and the maintainer footer lines add tokens without guidance value. | 4 / 5 |
Actionability | The Scan step provides five copy-paste-ready commands (semgrep, bandit, gitleaks, npm audit, trivy) and the example finding entry is a complete fill-in template. Minor gaps: the Scope step ("Map attack surface and critical paths") and Report step give no in-body specifics, deferring entirely to references. | 4 / 5 |
Workflow Clarity | The five-step Core Workflow (Scope → Scan → Review → Test and classify → Report) is clearly sequenced with explicit validation checkpoints at nearly every step: "Confirm written authorization", "Verify written scope authorization before active testing", "Validate findings", "Confirm findings with stakeholder before finalizing", plus MUST DO/MUST NOT checklists. | 5 / 5 |
Progressive Disclosure | The Reference Guide table maps six topics to exact file paths (all verified to exist in references/) with explicit "Load When" conditions — a clear one-level-deep overview with well-signaled navigation and no content that belongs in a separate file inlined. | 5 / 5 |
Total | 18 / 20 Passed |