Content
56%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 strong on executable, concrete AWS CLI/boto3 guidance and has a sensible phased workflow with dry-run and validation emphasis. Its weaknesses are verbosity from redundant safety/practice sections and a monolithic structure that inlines scripts which belong in bundle files.
Suggestions
Move the named scripts (cleanup-unused-ebs.sh, cleanup-old-snapshots.sh, release-unused-eips.sh, calculate-savings.py, the Lambda handler) into a scripts/ directory and reference them from SKILL.md to cut the body's length substantially.
Merge the overlapping 'Safety Checklist', 'Best Practices', 'Risk Mitigation', and 'Always:' sections into one consolidated safety block, and drop the redundant 'When to Use This Skill' section and 'Kiro CLI Integration' filler.
Add concrete validation commands to each workflow phase (e.g. how to verify a resource is unused before deletion) and a fix/retry loop for failed deletions to raise workflow clarity.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~300-line body has several padded or duplicated sections: 'When to Use This Skill' merely repeats the frontmatter description, and 'Safety Checklist', 'Best Practices', 'Risk Mitigation', and 'Always:' blocks restate the same cautionary advice three times over. The 'Kiro CLI Integration' section adds little skill-specific value, and a full Lambda function is inlined. This matches anchor 2 (several unnecessary/padded sections) more than 3's 'some unnecessary explanation', since the padding is repeated across multiple sections rather than incidental. | 2 / 5 |
Actionability | Concrete, executable code dominates: dry-run bash scripts for EBS volumes, snapshots, and Elastic IPs; a boto3 savings calculator; an S3 lifecycle policy; and CloudWatch alarm commands. It falls short of 5 because listed cleanup targets (unused AMIs, orphaned ENIs, NAT Gateways, ELBs) have no scripts, and the Organizations loop assumes preconfigured named profiles without explaining how. | 4 / 5 |
Workflow Clarity | The four-phase workflow (Discovery → Validation → dry-run Execution → Verification) is clearly sequenced with explicit validation checkpoints and a safety checklist, so the destructive-operation cap at 3 does not apply. However, phases are high-level bullets without the underlying commands or fix/retry feedback loops, keeping it at anchor 4 rather than 5. | 4 / 5 |
Progressive Disclosure | The body is well-sectioned with clear headers, but it is monolithic: scripts explicitly named as files ('cleanup-unused-ebs.sh', 'calculate-savings.py', the Lambda handler) are fully inlined rather than placed in a scripts/ directory, and there are no one-level-deep internal references at all. This matches anchor 3 ('content that should be separate is inline') — structure is good, so it is above anchor 2's 'minimal structure'. | 3 / 5 |
Total | 13 / 20 Passed |