Skip to content

fix(stacker): avoid duplicate stack deletions - #81

Merged
Majorfi merged 2 commits into
mainfrom
fix/issue-80-stack-delete
Sep 7, 2026
Merged

fix(stacker): avoid duplicate stack deletions#81
Majorfi merged 2 commits into
mainfrom
fix/issue-80-stack-delete

Conversation

@Majorfi

@Majorfi Majorfi commented Sep 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • deduplicate child stack IDs before issuing delete requests
  • exclude successfully deleted stacks from the in-memory snapshot while retaining failed or refused deletions
  • treat already-gone stack responses as benign without reporting deletions that did not happen
  • mask API keys in client initialization and user lookup errors
  • add regression coverage for stack deletion, replacement, dry-run, and logging behavior

Testing

  • go test ./...
  • go vet ./...

Closes #80

@Majorfi
Majorfi merged commit 0e429c8 into main Sep 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Errors on stack delete calls

1 participant