Content
67%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 a well-organized dispatch layer: lean tables, concrete flags, verbatim wizard prompts, and confirmation checkpoints for the destructive fix and scaffold operations. Its decisive flaw is structural — the load-bearing reference file it tells the agent to read before responding is missing from the bundle, so no mode logic, bootstrap steps, or fix patterns are actually available.
Suggestions
Ship the missing reference: the body instructs 'Read references/checkov.md before responding' and 'proceed into the relevant mode section in references/checkov.md', but no references/ directory exists in the bundle — either include the file or inline the mode logic into SKILL.md.
Merge the mode dispatch table and the wizard Q1 prompt: both enumerate the same 8 modes with near-identical descriptions, so a single table plus short prompt labels would cut ~15 lines of duplication.
Add a brief error-recovery note in the dispatch section (e.g., what to do when bootstrap or a scan fails, such as falling back to static mode) to close the workflow-clarity gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense, table-driven body with zero concept explanations Claude already knows ('Do not pepper the user with questions', 'Never ask about: output format (default to cli)...'). The one trim opportunity is that the mode dispatch table and the wizard Q1 prompt enumerate the same 8 modes twice with near-identical descriptions — minor duplication, not padding. | 4 / 5 |
Actionability | Concrete Checkov flags ('--download-external-modules true', '--framework secrets --enable-secret-scan-all-files', '--scan-secrets-history', '--deep-analysis') plus verbatim prompt text ('Enter 1–8 or mode name:') make the guidance mostly executable. Not 5 because complete, copy-paste-ready checkov command lines are deferred to references/checkov.md rather than shown, leaving a gap in the body itself. | 4 / 5 |
Workflow Clarity | Sequencing is explicit — parse the first word of $ARGUMENTS, empty input routes to the wizard (Q1 then Q2), classify intent before asking anything — and the risky operations carry confirmation checkpoints ('Apply these changes? (y/N) — show unified diff first'; 'Write .checkov.yaml to <path>? (y/N)'), so the destructive-operation cap does not apply. Not 5 because error-recovery loops for bootstrap or scan failures are not surfaced in the body. | 4 / 5 |
Progressive Disclosure | The body correctly defers all mode logic to a single, clearly signaled, one-level-deep reference ('Read `references/checkov.md` before responding'), but that file does not exist in the bundle — no references/, scripts/, or assets/ directories are present — so the disclosure chain dead-ends and the skill's core content is unreachable. The design would score 5; the actual bundle structure, which the guidelines direct scoring against, breaks it. | 2 / 5 |
Total | 14 / 20 Passed |