Skip to content

Cancel pending review when deleting a record - #50

Merged
sametbasbug merged 1 commit into
mainfrom
equinox/v0.5.1-delete-pending-cleanup
Aug 8, 2026
Merged

sametbasbug merged 1 commit into
mainfrom
equinox/v0.5.1-delete-pending-cleanup

Conversation

@sametbasbug

Copy link
Copy Markdown
Owner

Summary

  • close the moderation lane when a pending record is soft-deleted
  • cancel pending publication reviews through the existing append-only transition table
  • reject the attached pending revision and clear pending_revision_id
  • backfill already-deleted records that still carry a pending review/revision, including the state caught by the live v0.5.1 selene-lab acceptance
  • retain the existing media-orphaning trigger and deletion audit/idempotency behavior
  • add regression coverage that deletes a real approval-required pending reply and verifies no zombie pending metadata remains

Validation

  • v0.5.1 MCP parity/invariant test passes; only the two known unrelated local sign-in-trace ECONNRESET flakes remain in Slice 1
  • npm run check: 0 errors / 0 warnings / 2 pre-existing hints
  • production Worker dry-run passed

Close the moderation lane when a pending record is deleted: cancel open reviews through the append-only review transition table, reject the pending revision, clear pending_revision_id, and backfill already-deleted records with the same zombie-review state. Add regression coverage that mirrors the live v0.5.1 acceptance failure.
@sametbasbug
sametbasbug marked this pull request as ready for review August 8, 2026 20:18
@sametbasbug
sametbasbug merged commit ac7889b into main Aug 8, 2026
4 checks passed
@sametbasbug
sametbasbug deleted the equinox/v0.5.1-delete-pending-cleanup branch August 8, 2026 20:19
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