Skip to content

fix(admin): purge cascade map learns crm_auto_lead_cursors (163) - #406

Merged
vjvarada merged 2 commits into
mainfrom
ws-26-purge-map-autolead-cursor
Aug 9, 2026
Merged

fix(admin): purge cascade map learns crm_auto_lead_cursors (163)#406
vjvarada merged 2 commits into
mainfrom
ws-26-purge-map-autolead-cursor

Conversation

@vjvarada

@vjvarada vjvarada commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Cross-PR seam: #399's cascade-map assertion × #403's new email_accounts child. Each was green alone, red combined on main — this is the one-line map addition (+ count comment) that makes them agree. Local: test_admin_member_purge.py + test_crm_auto_lead.py, 114 passed.

🤖 Generated with Claude Code

vjvarada and others added 2 commits August 10, 2026 01:51
…#399 x #403 cross-PR seam

Each PR was green alone: #399 added the offboarding purge whose cascade map is derived-and-asserted from the migrations; #403 added a table FKing email_accounts. Merged together the map assertion went red on main. The auto-lead cursor is per-mailbox state and goes with the mailbox.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…#399 x #403 seam

R5 ratchet: the per-mailbox cursor's tenant derives transitively through email_accounts; a direct organization_id would duplicate a derivable fact. Baseline count 113 -> 114 with the trajectory line updated.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@vjvarada
vjvarada merged commit 7b0ac58 into main Aug 9, 2026
6 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.

1 participant