Content
68%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-disclosed, mostly lean audit skill that delegates bulk commands to real reference files, with a clear eight-section sequence and useful severity thresholds. Main weaknesses: no explicit verification/triage step for scan findings, minor duplication between inline and referenced content, and half the bundle's reference files are orphaned from SKILL.md.
Suggestions
Add an explicit verification/triage checkpoint between scanning and scoring — e.g. a step 9 "Validate findings: confirm each grep hit is a true positive (reachable and in scope) before applying the deduction table" with a re-check loop for false positives.
Reference the three orphaned bundle files (signals.md, trust-review-policy.md, vibe-security-scan.md) from the body — e.g. a "Related lenses" section — so they are discoverable, and fix or drop signals.md's broken link to ../../common-architecture-audit/references/detection.md.
Remove the duplicate frontend-secrets grep in section 6 (implementation.md already covers it) and delete the "Canonical response anchors" section, whose lone "- -25" bullet adds no actionable guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean: sections 1-3, 5, and 7 are one-line pointers to reference files ("See [implementation examples](references/implementation.md) for secrets scanning commands") with brief scope notes, and sections 4/6/8 use compact command lists. Minor trimmable redundancy keeps it below the lean score-5 anchor: section 6's frontend grep duplicates content already in implementation.md's "Frontend (Bundled Secrets)" section, and the trailing "Canonical response anchors" section ("- -25") is padding that earns no tokens. | 4 / 5 |
Actionability | Mostly executable: concrete commands like `npm audit --audit-level=high`, `grep -rE "(REACT_APP_|NEXT_PUBLIC_|VITE_)" ...`, and a precise IDOR rule ("any `findById` without an `owner` filter is a P0"), with the deferred sections pointing to verified reference files containing full runnable commands. Minor gaps (the nonexistent-looking `pod audit`, the vague "Gradle scan") keep it below fully copy-paste-ready at 5. | 4 / 5 |
Workflow Clarity | Sections 1-8 provide a clear audit sequence and the Scoring Impact table gives thresholds, but validation checkpoints are missing or implicit: there is no verify step for triaging grep hits into true positives before applying deductions, and the CAUTION note ("rotate the credential NOW") has no error-recovery loop. This matches the score-3 anchor (steps listed, checkpoints missing/implicit), not score 4 which requires most checkpoints present. | 3 / 5 |
Progressive Disclosure | The body is well structured as an overview: bulk commands live in references/implementation.md, references/mobile-audit.md, and references/REMEDIATION.md, each clearly signaled with a one-level-deep markdown link and purpose statement. Scored against the actual bundle: three of six reference files (signals.md, trust-review-policy.md, vibe-security-scan.md) are never referenced from SKILL.md and are therefore undiscoverable, and signals.md links to a nonexistent ../.. path — organization gaps that hold this at the "minor gaps" score-4 anchor rather than the fully navigable score-5 anchor. | 4 / 5 |
Total | 15 / 20 Passed |