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 lean, well-structured, and actionable with concrete semgrep commands, threat-model checklists, an explicit human-verification gate, and a completion self-check. It falls just short of top marks because command coverage across tools is uneven, there is no re-scan feedback loop, and one reference duplicates inline content while others point outside the bundle.
Suggestions
Add copy-paste commands for the other tools in the table (e.g. `codeql database create`, `bandit -r .`, `gosec ./...`) so guidance is fully executable across the listed stack.
Make the validation loop explicit: after '人工验证', state 're-scan with the refined rule set until no new high-confidence findings remain' to add a fix→re-scan feedback loop.
Deduplicate the inline section-3 checklist against references/sast-review-checklist.md (link to it instead of repeating items), and flag the out-of-bundle sibling paths as non-loadable routing hints.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is terse — short checklists, a compact tool table, and minimal commands — and assumes Claude knows SAST concepts, but the '路由上下文' and '适用场景' routing metadata add a little non-execution overhead, fitting anchor 4 rather than the fully lean anchor 5. | 4 / 5 |
Actionability | Provides copy-paste semgrep commands and concrete verification checklists, but CodeQL/Bandit/gosec appear only as table entries without commands and human verification is checklist-based, so it is 'mostly executable with minor gaps' (anchor 4). | 4 / 5 |
Workflow Clarity | Clear NOW/NEXT/ACT preamble, a numbered 4-step workflow, an explicit '人工验证 (MUST)' checkpoint, and a final '任务完成自检' checklist give a strong sequence, but there is no explicit fix→re-scan feedback loop, capping it at anchor 4 rather than 5. | 4 / 5 |
Progressive Disclosure | Well-sectioned short body with a real one-level reference (references/sast-review-checklist.md, verified present), but that reference partly duplicates the inline checklist and several sibling paths (../field-journal, ../supply-chain-security) point outside the bundle, matching anchor 4. | 4 / 5 |
Total | 16 / 20 Passed |