Content
68%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 tight, highly actionable guardrails document whose rules name concrete tools, commands, and AWS-specific mechanisms rather than vague directives. Its weaknesses are thematic rather than sequenced organization with no validation checkpoint for the destructive infrastructure operations it governs, and reliance on sibling-skill references that are not present in this bundle.
Suggestions
Add a lightweight validation checkpoint for infrastructure changes (e.g., review 'cdk diff' output or a CloudFormation change set before deploying) so the destructive-operation cap on workflow clarity is lifted.
Include a minimal least-privilege policy snippet using the named 'aws:ViaAWSMCPService' / 'aws:CalledViaAWSMCP' condition keys to make the read-only-in-production rule copy-paste ready.
Trim known-concept rationale ('IaC is reviewable, repeatable, and safely destroyable') and the six-pillar enumeration to tighten token efficiency.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a lean set of directives with no concept tutorials, and includes genuinely non-obvious specifics like the '{{resolve:secretsmanager:secret-id:SecretString:json-key}}' dynamic reference and the 'aws:ViaAWSMCPService' condition keys. Not 5: a few phrases explain what Claude already knows ('IaC is reviewable, repeatable, and safely destroyable') and the six-pillar enumeration could be trimmed. | 4 / 5 |
Actionability | Concrete, named guidance throughout — 'retrieve_skill', 'search_documentation' / 'read_documentation', forbidden commands ('secretsmanager get-secret-value' / 'batch-get-secret-value'), a copy-pasteable CloudFormation dynamic reference, and specific standing-cost examples (NAT gateways, RDS/Aurora provisioned instances, always-on load balancers). Not 5: it stops short of executable examples for the common cases (no CDK/CLI snippet, no least-privilege policy example using the named condition keys). | 4 / 5 |
Workflow Clarity | The rules are grouped thematically rather than sequenced; only 'Before beginning a task, check whether a relevant AWS skill exists' provides explicit ordering, and verification guidance ('Verify, don't guess') is a rule rather than a checkpoint in a flow. The skill governs potentially destructive infrastructure operations yet includes no post-change validation step, so workflow clarity is capped at 3. Not 4: there is no clear overall sequence with checkpoints. | 3 / 5 |
Progressive Disclosure | Sections are well organized (Core rules, Security defaults, Secret safety, Cost awareness) with no nested references, and the single external pointer ('See the aws-mcp-server skill for how to drive it') is clearly signaled. Not 5: no bundle files exist to resolve the referenced sibling skills, and the body slightly exceeds the ~50-line simple-skill threshold while inlining several rule sets that could live in references. | 4 / 5 |
Total | 15 / 20 Passed |