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 strong, highly actionable workflow document with excellent sequencing, validation, and error-recovery structure. Its weaknesses are the absence of any progressive disclosure — everything, including reference material like command tables and output templates, lives inline in one long file — and minor redundancy in the Constraints and completion-checklist sections.
Suggestions
Move stable reference material into bundle files — e.g., references/output-templates.md for the No Fix Available report (Step 2.5) and Full Advisory (Phase 4a) formats, and references/feedback-format.md for the Step 6.2/6B.2 payload specifications — leaving SKILL.md as the core workflow with clearly signaled one-level-deep pointers.
Delete or shrink the 'Reference: common install/resolve commands' table in Step 4.4; Claude already knows these ecosystem commands, and the surrounding text already instructs to use the ecosystem-appropriate command.
Consolidate the Constraints section: its SCA bullets restate Step 4.2 and Step 2.5 nearly verbatim ('Breakability gates all SCA fixes... See Step 4.2', 'No fix path means no fix attempt — See Step 2.5'), so a one-line pointer per constraint would reclaim tokens without losing the gate.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense operational logic — decision trees, tables, exact tool calls — with almost no explanation of concepts Claude already knows, fitting the 'efficient; minor instances of over-explanation' anchor. The trimmable parts are the 'common install/resolve commands' table (npm/pip/maven commands Claude already knows) and the Constraints section, which largely restates earlier steps via 'See Step 4.2' / 'See Step 2.5' pointers. It is not a 3 because background-concept padding is absent and the bulk of every section earns its place. | 4 / 5 |
Actionability | Guidance is fully executable: exact MCP invocations with parameters ('mcp_snyk_snyk_code_scan: path: ...'), concrete git/gh commands ('git checkout -b fix/security-<identifier>', 'git push -u origin fix/security-<identifier>', 'gh pr create --title ... --base main'), and complete fill-in output templates (No Fix Available report, advisory format, feedback payloads). This matches the 'fully executable; copy-paste ready' anchor with the common cases covered. | 5 / 5 |
Workflow Clarity | The phased sequence (Parse → Scan → Analyze → Fix → Validate → Summary → PR) has explicit validation steps (Phase 5 re-scan, tests, lint), feedback loops for error recovery (max 3 attempts per instance, rollback triggers, per-item validation in batch mode), a batch-plan confirmation gate, an error-handling table, and completion checklists. This matches the score-5 anchor; the batch/destructive cap of 3 does not apply because validation is pervasive. | 5 / 5 |
Progressive Disclosure | This is a single-file skill with no references/ directory and all ~530 lines inline. Section headers are clear and the doc is well-navigable, but content that clearly belongs in separate reference files is inlined: the full advisory template (Phase 4a), the No Fix Available report template (Step 2.5), the install/resolve command reference table, and the detailed feedback payload formats. That fits the score-3 anchor ('some structure but could be better organized; content that should be separate is inline') rather than 4, where most content would be appropriately split across files. | 3 / 5 |
Total | 17 / 20 Passed |