Skip to content

fix(user): require nuclear delete confirmation first - #190

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/nuclear-delete-confirm-first
Jul 29, 2026
Merged

fix(user): require nuclear delete confirmation first#190
ralyodio merged 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/nuclear-delete-confirm-first

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • validate the explicit nuclear-delete confirmation immediately after auth
  • avoid service-role user lookups and delete RPC setup when confirmation is missing
  • add regression coverage for the no-confirmation path

Test

  • node node_modules\vitest\vitest.mjs run --config %TEMP%\qryptchat-vitest-minimal.config.mjs "src/app/api/user/nuclear-delete/route.test.js"
  • git diff --check

uGig billable bugfix candidate. Not counted as revenue until accepted, invoiced, and paid.

@ralyodio
ralyodio merged commit b533364 into profullstack:master Jul 29, 2026
8 checks 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.

2 participants