Content
77%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 highly actionable with executable commands, explicit exit-code-driven validation, and well-sequenced workflows, and it stays free of basic-concept padding. Its weaknesses are repeated explanations across sections (severity modifiers, public access block) and a broken progressive-disclosure structure — detailed check matrices are inlined while the referenced references/cspm-checks.md and scripts/cloud_posture_check.py files are missing from the bundle.
Suggestions
Remove the duplicated explanations: keep severity-modifier guidance in one place (the tool section) and have Anti-Patterns 3 and 7 cross-reference it instead of restating it; do the same for the bucket vs. account-level public access block explanation duplicated in Anti-Pattern 2.
Create the referenced files: add references/cspm-checks.md and scripts/cloud_posture_check.py to the bundle so the body's pointers resolve, and move the detailed severity matrices and per-port tables into that reference file to slim the SKILL.md overview.
Trim reference-style detail (the full IAM finding severity guide and database port table) down to the highest-frequency rules inline, pushing the exhaustive tables to the reference file the body already cites.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The dense severity matrices, port tables, and escalation combos are information Claude would not know, so most tokens earn their place — but there is noticeable repetition beyond minor trimming: severity modifiers are explained in the Security Group section ('Use --severity-modifier internet-facing when...') and re-explained in Anti-Patterns 3 and 7, and the bucket/account-level public access block guidance duplicates Anti-Pattern 2. This fits 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than the minor-trim anchor above it. | 3 / 5 |
Actionability | Guidance is fully executable and copy-paste ready: complete AWS CLI pipelines with jq ('aws iam get-policy-version ... | jq ... | python3 scripts/cloud_posture_check.py - --check iam --json'), a working CI/CD gate script with exit-code logic, concrete tool flags (--check, --severity-modifier, --json), a JSON baseline config, and a GOOD/BAD Terraform pair covering common cases. | 5 / 5 |
Workflow Clarity | The three workflows are clearly sequenced with explicit validation checkpoints: an exit-code table maps each code to a required action, Workflow 1 ends with an explicit decision rule ('Exit code 2 = block deployment and remediate'), and Workflow 3 shows the validate-then-block feedback loop in the pipeline script. Numbered day-by-day steps in Workflow 2 leave no sequence gaps. | 5 / 5 |
Progressive Disclosure | Section structure is good (ToC, clear headers, signaled pointer 'Full CSPM check reference: references/cspm-checks.md'), but the referenced files do not exist — neither references/ nor scripts/ is present in the bundle, so both cspm-checks.md and cloud_posture_check.py are broken paths. Additionally, substantial reference material (full severity matrices and per-port tables) is inlined in SKILL.md rather than split into the referenced file, fitting 'some structure but content that should be separate is inline' with references that cannot actually be followed. | 3 / 5 |
Total | 16 / 20 Passed |