Content
57%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 thorough, well-organized AVM requirements reference with strong concrete HCL examples and a compliance checklist. Its main weaknesses are the lack of file-based progressive disclosure (everything inlined in one 615-line file), no explicit validate/retry workflow, and some non-essential padding such as the statistics summary.
Suggestions
Split the per-requirement detail into one-level-deep reference files (e.g. references/variables.md, references/outputs.md, references/code-style.md) and keep SKILL.md as an overview with clearly signaled links, to improve progressive disclosure and reduce token load.
Add an explicit develop/review workflow with validation checkpoints (e.g. 'run terraform fmt/validate, then tflint + checkov, fix and re-run until clean, then confirm against the Compliance Checklist') so the testing tools form a feedback loop rather than a bare list.
Remove the Summary Statistics section and the introductory sentence restating purpose, since Claude does not need requirement counts or a restated rationale to apply the rules.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient concrete rules with HCL examples, but the 615-line monolith includes unnecessary padding such as the Summary Statistics section ('Total Requirements: 37', 'MUST: 21') and an introductory sentence Claude does not need. It is not a 4 because that stats block and intro could be trimmed, and not a 2 because the bulk is concrete and unpadded rather than heavily padded. | 3 / 5 |
Actionability | Provides many copy-paste-ready HCL examples, a provider version-constraint table, specific file names (deprecated_variables.tf, .terraform-docs.yml), and severity/requirement-ID tagging. It is not a 5 because several sections (Variable Definition Order, Naming Rules, Branch Protection) give only abstract rules without executable examples, and not a 3 because most guidance is concrete and executable. | 4 / 5 |
Workflow Clarity | Numbered orderings exist for block-internal structure and a Compliance Checklist provides checkpoints, but there is no end-to-end develop/review workflow with an explicit validate-then-fix-then-retry feedback loop. It is not a 4 because validation checkpoints are not sequenced into a workflow, and not a 2 because the checklist and ordered lists supply some structure and checkpoints. | 3 / 5 |
Progressive Disclosure | The body is well-structured with a Table of Contents and clear section headers, but all 615 lines are inlined in SKILL.md with no bundle/reference files to defer to (references/scripts/assets directories are absent). It is not a 4 because content that clearly belongs in separate files (e.g. the 37-requirement spec) is not split out, and not a 2 because the headers and TOC provide real navigable structure rather than minimal structure. | 3 / 5 |
Total | 13 / 20 Passed |