Problem
The skills/terraform-engineer/SKILL.md workflow says to run terraform plan -out=tfplan, review the output carefully, and then run terraform apply tfplan. It does not require explicit user approval between plan review and apply.
Impact
terraform apply can create, modify, or destroy infrastructure. A skill should not proceed from plan to apply without a clear approval checkpoint, especially when managing cloud resources.
Suggested fix
Add a required approval step after terraform plan and before terraform apply:
- Show a summarized plan with creates, updates, deletes, and any destructive actions.
- Ask for explicit user approval.
- Only run
terraform apply tfplan after approval.
- Refuse apply if destructive changes are present and the user has not explicitly accepted them.
Problem
The
skills/terraform-engineer/SKILL.mdworkflow says to runterraform plan -out=tfplan, review the output carefully, and then runterraform apply tfplan. It does not require explicit user approval between plan review and apply.Impact
terraform applycan create, modify, or destroy infrastructure. A skill should not proceed from plan to apply without a clear approval checkpoint, especially when managing cloud resources.Suggested fix
Add a required approval step after
terraform planand beforeterraform apply:terraform apply tfplanafter approval.