Content
93%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 dense, command-oriented body that assumes Claude's competence and offloads detail to a well-structured set of one-level-deep reference files. The only gap is that feedback loops for destructive operations are implied via the troubleshooting table rather than made explicit.
Suggestions
For destructive operations (cdk rollback --orphan, removalPolicy: DESTROY + autoDeleteObjects, cdk refactor), add an explicit validate→fix→retry loop, e.g. 'cdk diff → review replacements → deploy → verify resource still exists'.
Add an explicit post-action verification step after destructive commands (e.g., confirm the orphaned resource state with aws cloudformation describe-stack-resources after cdk rollback --orphan).
Promote the error→diagnose→fix pattern in the Troubleshooting table into a short stated loop ('diagnose → fix root cause → re-deploy → confirm resolved') so the feedback loop is explicit rather than inferred.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and command-dense throughout — tables pack a quick command plus a reference link, Critical Warnings give terse cause→fix with exact commands, and no space is spent explaining what CDK or CloudFormation is. Every token earns its place. | 5 / 5 |
Actionability | Copy-paste-ready commands and code abound: 'cdk bootstrap aws://$ACCOUNT/$REGION', 'cdk synth --strict → cdk diff → cdk deploy', 'cdk rollback $STACK --orphan <LogicalId>', and 'CrossStackReferences.of($RESOURCE).produce(ReferenceStrength.BOTH)'. The troubleshooting table gives a concrete fix command per error. | 5 / 5 |
Workflow Clarity | Sequenced workflows with validation checkpoints are present ('cdk synth --strict → cdk diff → cdk deploy', 'Always diff before deploy to prod', drift '--fail in CI'), and the deadly-embrace fix is a three-deploy sequence — but feedback loops for destructive ops (rollback --orphan, autoDeleteObjects) are implied via the troubleshooting table rather than stated as explicit validate→fix→retry loops, and a few destructive commands lack an explicit post-action verify step. | 4 / 5 |
Progressive Disclosure | SKILL.md is a clear overview with well-organized sections (Overview, Critical Warnings, Common Workflows, Troubleshooting, Construct Patterns, Security), and detail is offloaded one level deep to 9 real, clearly-signaled reference files — all referenced paths verified to exist in references/. | 5 / 5 |
Total | 19 / 20 Passed |