Content
65%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, code-rich Terraform skill body that suffers from monolithic length, redundant security coverage, and broken script references to files absent from the bundle. Workflow clarity is capped by missing validation feedback loops around destructive operations.
Suggestions
Split the secondary topics (OpenTofu Compatibility, Infracost Integration, Terragrunt Patterns, Multi-Cloud Provider Configuration, CI/CD Integration Patterns) into reference files under references/ and keep SKILL.md as a concise overview that links to them one level deep.
Either ship the referenced scripts (tf_module_analyzer.py, tf_security_scanner.py, convert.sh) in a scripts/ directory or remove the run-instructions that depend on them, so the body never points to nonexistent bundle files.
Add explicit validation feedback loops to the destructive workflows, e.g. after terraform import run terraform plan and only proceed when the diff is clean; before terraform apply -auto-approve require a passing plan and state backup.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 741-line body is mostly concrete checklists, code, and decision trees, but it is monolithic, repeats security checks across the review, security, and proactive-triggers sections, and includes tool-intro prose ("OpenTofu is an open-source fork of Terraform...", "Infracost estimates cloud costs...") that explains concepts Claude already knows. | 3 / 5 |
Actionability | Provides fully executable, copy-paste-ready guidance throughout: complete HCL provider/version blocks, GitHub Actions plan/apply and drift-detection YAML, terraform CLI and import commands, and concrete severity/fix tables covering the common cases. | 5 / 5 |
Workflow Clarity | The /terraform:review, :module, and :security workflows are clearly numbered with checklists, but there are no explicit validate->fix->retry feedback loops, and destructive or batch operations (terraform apply -auto-approve, terraform state mv/rm, import) lack structured validation checkpoints, which caps the score. | 3 / 5 |
Progressive Disclosure | Internal section structure is good (clear headers, tables, code fences), but nearly everything is inlined into one massive file and the body instructs running scripts/tf_module_analyzer.py, scripts/tf_security_scanner.py, and scripts/convert.sh that do not exist in the bundle, so the references are broken. | 3 / 5 |
Total | 14 / 20 Passed |