Skip to content

Close publication connections before frontend builds - #79

Merged
RaviTejaKNTS merged 1 commit into
mainfrom
chore/check-handoff-file
Oct 21, 2026
Merged

RaviTejaKNTS merged 1 commit into
mainfrom
chore/check-handoff-file

Conversation

@RaviTejaKNTS

Copy link
Copy Markdown
Collaborator

Page publication could crash after its database commit when a checked-out PostgreSQL connection dropped during the frontend build. The job now closes the snapshot connection before building and uses fresh connections for completion and failure recording.

Client and pool error events enter the normal failure path. The client listener is attached inside the checkout callback, before the promise resolves. Failed connections are discarded, and a failed retry or rollback cannot hide the original error. Current-release checks, public verification, and version-specific queue acknowledgement remain in place.

Validation:

  • Independent Sol review approved the four-file change after the checkout race was fixed; 124 focused tests and rollback-error probes passed.
  • 73 focused frontend and publisher tests passed, including asynchronous disconnects, fresh completion connections, preserved queue events, and failure recording errors.
  • A real PostgreSQL protocol stub reproduced crashes during snapshot and retry checkout before the callback fix; both regressions pass with the fix.
  • npm run quality passed TypeScript and 116 core tests.
  • git diff --check passed.
  • The previous metadata publication recovered through the existing refresh queue; all 67 live ad-network pages passed verification.

After deployment, run the reviewed 67-page batch through the fixed publisher against the exact live main revision and confirm its public checks finish successfully.

@RaviTejaKNTS
RaviTejaKNTS merged commit 447f0eb into main Oct 21, 2026
4 checks passed

This branch was previously deployed

1 inactive deployment
production — 78b9f3c9 Deployed Oct 21, 2026 by RaviTejaKNTS via verify-production-build #79
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