Content
82%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 dispatch-layer skill body: lean tables, verbatim prompts, and sensible ask-only-when-risky agent rules. The main gaps are the absence of complete executable command lines in the body, no explicit post-fix re-verification loop, and the referenced references/checkov.md being absent from the bundle.
Suggestions
Ship references/checkov.md with the skill (or remove the pointer) — the body delegates all mode logic, bootstrap steps, and fix patterns to it, so its absence leaves the skill unusable as delivered.
Add an explicit post-fix verification step to the fix mode (e.g., 'after applying patches, re-run the scan and confirm the targeted CKV_* checks pass') to close the feedback loop.
Include at least one complete copy-paste-ready command per common mode (e.g., the exact checkov invocation for static and secrets scans) so the top-level file is executable without the reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is almost entirely dense tables and verbatim prompt blocks (mode dispatch, intent classification, ask/never-ask rules) with zero concept explanation and no padding — every section carries skill-specific dispatch or agent-behavior guidance, matching the anchor-5 'lean and efficient; every token earns its place'. | 5 / 5 |
Actionability | Concrete flag-level guidance ('--download-external-modules true', '--framework secrets --enable-secret-scan-all-files', '--deep-analysis', '--scan-secrets-history'), specific file names (.checkov.baseline, .checkov.yaml), and verbatim wizard prompts, but no complete copy-paste-ready checkov command lines or bootstrap commands in the body — execution detail is deferred to references/checkov.md, fitting anchor 4 ('concrete commands with minor gaps') rather than 5. | 4 / 5 |
Workflow Clarity | Clear sequencing (parse mode → wizard Q1 then Q2 → mode section) with confirm checkpoints before destructive actions ('Apply these changes? (y/N) — show unified diff first', 'Write .checkov.yaml to <path>? (y/N)'), but there is no explicit post-fix verification step (re-scanning to confirm the CKV finding is resolved), which is the minor validation gap of anchor 4 rather than the full feedback loop of anchor 5. | 4 / 5 |
Progressive Disclosure | The design is exactly right — a lean overview/dispatch layer pointing to a single clearly signaled one-level-deep reference ('Read references/checkov.md before responding. It contains all mode logic, bootstrap steps, provider detection, and fix patterns') — but no references/ directory exists in the bundle, so the referenced file cannot be verified and navigation is unverifiable, leaving anchor 4 ('references mostly clear; minor organization gaps') rather than 5. | 4 / 5 |
Total | 17 / 20 Passed |