Terraform review
Review Terraform and OpenTofu configuration and plans for blast radius, state risk, IAM overreach, and safer apply order. Use whenever the user shares .tf files, a terraform plan, asks if apply is safe, or reviews infrastructure-as-code modules and state backends.From its SKILL.md
npx -y skills add shinzoxD/knackbox --skill terraform-reviewAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 0 stars0 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
What its file declares
Copied from the file, not written here
The file declares its own license as Apache-2.0. That is the author’s claim about this one file, and it is not the same thing as the license GitHub reports for the repository, which is listed with the other numbers below.
SKILL.md
2.2 KB, 414 tokens by cl100k_base, as published. Nobody here has run it
Terraform Review
Treat apply as a production change. Optimize for least privilege, least surprise, and reversible steps. Prefer plan evidence over vibes.
Workflow
- Identify providers, workspaces/envs, backend/state, and blast radius.
- Read plan summary: create/update/replace/destroy counts and critical resources.
- Check IAM, network exposure, secrets, and public endpoints.
- Flag force-new / replace chains and data-loss paths.
- Order safe apply (expand before destroy) and verification.
- Call out missing remote state locking, missing CI plan, or local-only state.
Severity tags
- [blocking] destroy/replace of stateful prod data, open 0.0.0.0/0 admin, secrets in VCS/state plaintext without process
- [important] overly broad IAM, missing tags/ownership, non-concurrent-safe patterns
- [nit] style, naming, minor refactors
Output format
## Terraform review
**Scope:** …
**Env assumption:** …
### Plan / change summary
…
### Findings
1. [blocking] …
### Apply order
1. …
### Verify after apply
…
### Do not apply if
…
Rules
- Never invent plan output; if missing, ask for
terraform plan(or equivalent). - Prefer modules and env separation over copy-paste with silent drift.
- Secrets: no committing
.tfvarssecrets; use secret managers / CI injection. - State: remote backend + locking for shared envs.
- Destructive changes need explicit user confirmation language.
- Distinguish OpenTofu/Terraform only when syntax or workflow differs.
Edge cases
- Import / move: review address changes carefully; wrong move corrupts state.
- for_each / count index churn: warn about replace storms.
- Partial apply failures: recovery notes and state surgery as last resort.
What ships with it: 1 file
1.4 KB alongside SKILL.md
benchmarks/
- prompts.json1.4 KB