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.
The body delivers an explicit, well-gated generate-then-validate workflow with highly executable HCL and error-recovery detail, but it is inflated by standard Terraform idioms, duplicated lifecycle guidance, and date-stamped version numbers. Its navigation is undermined by references to bundle files (modern_features.md and two asset templates) that do not exist on disk.
Suggestions
Create the missing references/modern_features.md (or remove its three citations) and either add the assets/aws-web-app/ and assets/multi-env/ templates or drop them from the assets list, so every path in the Step 2.5 table, Version Awareness note, and References section resolves.
Trim Step 3's ten best-practice items down to the non-obvious, project-specific patterns (the prevent_destroy resource list, the S3 abort-incomplete-multipart-upload rule) and rely on references/terraform_best_practices.md for standard HCL syntax Claude already knows.
Move the date-stamped provider versions ("as of December 2025", "v6.23.0") into a reference file, and merge the two lifecycle sections ("Lifecycle Rules When Appropriate" and "Lifecycle Rules on Critical Resources") to remove duplicated guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Step 3's ten "Best Practices to Follow" re-teach standard HCL idioms Claude already knows ("Use descriptive snake_case names", variable/output/locals/dynamic-block syntax), lifecycle guidance appears twice ("Lifecycle Rules When Appropriate" then "Lifecycle Rules on Critical Resources"), and date-stamped versions ("Latest: v6.23.0 (Dec 2025)", "as of December 2025") sit outside any deprecated/old-patterns section. Not 2: most sections (workflow, anti-patterns, validation fixes) are project-specific and earn their tokens; not 4: the duplication and known-idiom examples go beyond minor trims. | 3 / 5 |
Actionability | Fully executable throughout: complete HCL blocks (provider pinning, validated variables, S3 lifecycle configuration), exact search-query formats ("datadog terraform provider v3.30 monitor resource documentation"), concrete tool calls (mcp__context7__resolve-library-id, Skill(devops-skills:terraform-validator)), and a Checkov fix table mapping CKV_AWS_300 to the abort_incomplete_multipart_upload rule. Not 4: common cases are covered copy-paste ready with specific remediations for each failure mode. | 5 / 5 |
Workflow Clarity | "Work through these steps in order. Do not skip any step" frames an explicit five-step sequence; Step 4 is a dedicated validation checkpoint with a fix-and-revalidate feedback loop ("Do NOT proceed to Step 5 until all checks pass") and an error-recovery table. Not 4: validation is explicit, looped, and gating — matching the anchor-5 pattern exactly. | 5 / 5 |
Progressive Disclosure | References are well signaled (a when-to-read table, explicit Read paths, one level deep), but scored against the actual bundle: references/modern_features.md is cited three times (Step 2.5 table, Version Awareness, References section) yet absent from references/, and the assets section lists aws-web-app/ and multi-env/ templates that do not exist alongside the present minimal-project/. Not 4: dangling paths are more than minor organization gaps; not 2: the structure, navigation instructions, and reference table are genuinely good. | 3 / 5 |
Total | 16 / 20 Passed |