Content
63%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, action-dense body with genuinely useful AWS footguns, checklists, and code — but it is undermined by referencing files that do not exist, a duplicated mode listing, and a truncated terraform-mode section. The inline bulk should largely live in the reference files the skill claims to have.
Suggestions
Create `references/aws-cloudfront.md` and `references/aws-waf.md` (or remove the Reference pointers) — every reference path in the body currently resolves to a nonexistent file.
De-duplicate the seven-mode listing: keep it once (Activation) and have the wizard reference it instead of restating it verbatim.
Complete the truncated "Non-negotiable patterns in every generated module:" section under Mode: terraform, whose content currently leaks into the Common mistakes section.
Move the orgs-mode SCP JSON, OU tables, and AFT example into a reference file to shrink the SKILL.md overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly dense and efficient (tables, checklists, decision trees, assumes AWS competence), but the seven-mode list is duplicated in full between the Interactive Wizard and Activation sections, and time-sensitive version pins (">= 1.7.0", "TLSv1.2_2021") sit in the main flow rather than a deprecated/old-patterns section. Not 4 because the duplication is a noticeable tightening opportunity, not a minor one; not 2 because there is no conceptual padding or explanation of things Claude already knows. | 3 / 5 |
Actionability | Largely concrete and executable: a Terraform resource table, a complete SCP JSON policy, an AFT module block, the `aws iam simulate-custom-policy` command, and a Functions-vs-Lambda@Edge decision tree. Not 5 because the WAF baseline rule-group block is a Terraform fragment (`rate_based_statement { limit = 2000; ... }` is not valid HCL), the "Provide: problem diagnosis, Terraform snippet, validation steps, rollback plan" steps direct without giving a template, and both `references/` paths are dead. | 4 / 5 |
Workflow Clarity | Each mode has a numbered sequence, with real checkpoints: "Count first, monitor 24–48h, then Block" and "Validate SCP effect before attaching" (with a simulate command). Not 5 because there are no error-recovery loops (e.g. what to do on `WAFInvalidParameterException`), the review mode's input-gathering is only sketched, and the terraform mode's "Non-negotiable patterns in every generated module:" heading is truncated with no content following it. | 4 / 5 |
Progressive Disclosure | The body is well-sectioned with clearly signaled references ("**Reference:** `references/aws-cloudfront.md`"), but neither referenced file exists — the bundle contains no references/, scripts/, or assets/ directories — so the primary navigation paths are broken. Additionally, reference-grade material (full SCP JSON, OU design tables, AFT module, provider-pinning patterns) is inlined in a ~400-line SKILL.md rather than split out. Not 2 because the in-body structure and mode organization are genuinely good; not 4 because dead reference links are a real navigation defect. | 3 / 5 |
Total | 14 / 20 Passed |