Content
71%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 index-style skill body: concrete commands and defaults, explicitly sequenced workflows with mandatory safety gates, and exemplary progressive disclosure into a verified flat reference bundle. The main cost is token efficiency — repeated boilerplate across the six safety sections and a verbose install-mode guardrail could be condensed without losing clarity.
Suggestions
Collapse the six safety sections' repeated 'You MUST follow the ... procedure in template-safety-guidance.md before advising or editing' sentence into a single stated rule with per-section links, saving ~10 lines of duplicated boilerplate.
Condense the MCP-vs-local-install guardrail to its two resolution rules and the 'never fetch or write customer data' constraint; the examples of mistaken local-file matches can be trimmed.
Inline one or two explicit validation checkpoints (e.g. 'run cfn-lint, fix, re-run until clean') in the validate-before-deploy section so the feedback loop is visible in the body, not only in the referenced guide.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and never explains concepts Claude already knows, but it could be tightened: the six safety sections each repeat the identical formula 'You MUST follow the ... procedure in [template-safety-guidance.md](references/template-safety-guidance.md) before advising or editing', and the ~20-line MCP-vs-local-install guardrail could be halved. This is 'mostly efficient but includes some... could be tightened' (3) rather than the 4 anchor, where over-explanation would be only minor. | 3 / 5 |
Actionability | Concrete, executable specifics throughout: '--deployment-config "{\"mode\": \"EXPRESS\"}" on create-stack', 'cdk deploy --express', 'wc -c' for size limits, 'list-imports' before touching exports, '!Sub "${AWS::StackName}-..."', '{{resolve:secretsmanager:...}}', and named S3 properties ('PublicAccessBlockConfiguration (all four true)'). Mostly executable with minor gaps (no complete example template inline — detail is correctly delegated to SOPs), so 4 rather than the copy-paste-comprehensive 5. | 4 / 5 |
Workflow Clarity | The authoring task is explicitly sequenced (retrieve embedded context → best-practices checklist → property lookup → persist context → attribution marker) with mandatory pre-edit gates ('Before touching any Export, you MUST check list-imports') and a destructive-operation guard ('Run destructive operations... only on direct user instruction'). It is not 5 because the validate→fix→retry feedback loops themselves live in the referenced SOPs rather than being stated as explicit checkpoints in the body. | 4 / 5 |
Progressive Disclosure | A clear overview with well-signaled one-level-deep references: all ten inline links (e.g. references/validation-tool-selection.md, references/persist-template-context.script.md, references/template-safety-guidance.md) resolve to real bundle files, the Decision Guide table maps user intents to SOPs, and cross-links between reference files are flat sibling links — the validation-tool-selection guide acts as a clean sub-index for the validator SOPs. Content is appropriately split with key points inline and detail in files, matching the 5 anchor; it does not fall to 4 since navigation is easy and no reference is buried or nested. | 5 / 5 |
Total | 16 / 20 Passed |