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.
A highly actionable, well-sequenced command skill with concrete commands, safety checks, and validation loops in every mode. Its weaknesses are repetition of the credential-boundary/TTL caveats across sections and a bundle that omits the reference file the body cites six times.
Suggestions
State the credential boundary and the TTL-unknown rule once in the dedicated 'Credential boundary' section and reference it from the modes instead of restating both in every mode.
Ship 'references/aws-mcp-profiles.md' with the sections the body cites (Profile Type Detection, Auth Flows, Config File Locations, Prod Safety, credential_process Pattern) or remove the six broken pointers.
Move the full example output tables and ini stanzas for discover/org-scan into the reference file, keeping one short example inline per mode to lighten the overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational, but the credential-boundary rules and 'TTL is unknown' caveats are restated in nearly every mode section, and 'Reference: references/aws-mcp-profiles.md' is repeated six times. Mostly efficient with some unnecessary repetition that could be tightened, and no padding of concepts Claude already knows, so it sits above anchor 2. | 3 / 5 |
Actionability | Fully executable guidance: exact commands ('aws configure list-profiles', 'aws sts get-caller-identity --profile <profile>', 'aws organizations list-accounts --output json'), exact config file paths, concrete output tables, ini stanzas, and patch diff examples cover the common cases. Copy-paste ready for an instruction-only skill. | 5 / 5 |
Workflow Clarity | Each mode has numbered steps with explicit validation checkpoints and error-recovery feedback: the prod switch halts without --confirm and shows a diff plus restart instructions, login validates with an STS check before reporting, and org-scan asks instead of inventing missing SSO settings. The destructive-switch cap does not apply because validation is present. | 5 / 5 |
Progressive Disclosure | References are one level deep and clearly signaled with section anchors, but the sole referenced file 'references/aws-mcp-profiles.md' does not exist in the bundle, so every pointer is broken. Substantial inline detail (full output tables, ini templates) also makes the overview heavy, fitting the 'some structure, references present but unreliable' anchor better than 4. | 3 / 5 |
Total | 16 / 20 Passed |