Content
57%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.
Well-organized overview with a clear phased workflow and lean prose, but it is undermined by references to bundle files that are missing and by the absence of validation checkpoints in a batch scanning process. The skill reads as an index whose linked detail pages do not exist.
Suggestions
Add the missing referenced files (phases/01-04, specs/scoring-gates.md, specs/owasp-checklist.md, _shared/SKILL-DESIGN-SPEC.md) or inline the essential per-phase steps so the skill is self-contained.
Insert explicit validation checkpoints between phases, e.g. verify supply-chain-report.json / owasp-findings.json are well-formed and non-empty before advancing to the next phase, with a fix-and-retry loop on failure.
Expand the inline actionable detail for Phase 1 and Phase 3 (concrete commands and detection patterns) rather than relying solely on the absent phase files, since the body currently cannot execute those phases on its own.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's familiarity with OWASP/STRIDE, but the large ASCII architecture diagram and slight repetition between 'Key Design Principles' and 'Phase Sequence' could be trimmed. | 4 / 5 |
Actionability | Concrete commands appear (mkdir, WORK_DIR, ccw cli invocation, npm audit/pip-audit/safety check), but the per-phase executable detail is delegated to phases/*.md and specs/*.md files that do not exist, leaving key steps incomplete. | 3 / 5 |
Workflow Clarity | The 4-phase sequence is clearly laid out with a flow diagram and modes, but a security scan is a batch operation with no explicit validate/verify checkpoints between phases, capping the score at 3 per the destructive/batch guideline. | 3 / 5 |
Progressive Disclosure | Structure and signaling are good (overview plus a Reference Documents table with one-level-deep links), but every referenced path (phases/, specs/, _shared/) points to a file that is absent from the bundle, so navigation is broken in practice. | 3 / 5 |
Total | 13 / 20 Passed |