Story 2: build the Supabase realtime queue engine - #1
Merged
Conversation
XonkelX
marked this pull request as ready for review
July 17, 2026 22:25
Owner
Author
|
Final architectural review complete: SQL security, RLS and grants, concurrency/lock ordering, idempotency and stable request IDs, anonymous-session handling, and the Realtime subscription lifecycle were reviewed. The committed migrations are remotely applied; remote Supabase authorization, concurrency, idempotency, privacy, multi-client Realtime, reconnect, and cleanup validation passed. Supabase remains Free with no paid feature enabled, and no application deployment was performed during Story 2. All three PR CI jobs are green. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements Story 2's persistent Supabase Realtime queue engine while preserving the Story 1 visual system.
Final architectural review
The complete Story 2 diff was reviewed for SQL security,
SECURITY DEFINERsearch paths, RLS, grants, publications, actor derivation, queue/entry binding, concurrency, lock order, idempotency, anonymous SSR sessions, Realtime lifecycle, privacy, CI, environment handling, and documentation.Review fixes in
95cac31:20260717230000_harden_request_id_serialization.sqlNo already-applied migration was rewritten.
Architecture and security
anonandauthenticatedauth.uid(); no client actor UUID or staff Boolean is acceptedSERVINGentryqueuesandqueue_entriesnpm auditreports 0 vulnerabilitiesSupabase Free project
FOGnext-queueus-east-1ACTIVE_HEALTHY20260717190000,20260717191000,20260717230000Validation
Complete local suite after the review fix:
formatandformat:check)git diff --check: passHosted validation:
USAGEon the managedextensionsschema; no assertions or mutations ran in that failed invocationHosted coverage includes persistence, staff claims, private-name isolation, direct-write denial, actor and command request binding, idempotent replay, simultaneous staff serialization, one-serving-entry enforcement, lifecycle commands, access throttling, offline recovery, public-display convergence, and anonymous authentication.
Cleanup and final state
Delivery status
This PR is ready for final review after CI. It remains open, unmerged, and undeployed until the explicit Phase 1 merge step.