Conversation
…20260905 Fix G037 read-only transaction admission order
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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
1 participant
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.
Promote the reviewed G037 read-only transaction correction from protected develop to data after PR #2865 merged at
ac849063ae0f36f0df80e0c579065ad660605677.The controller starts its repeatable-read snapshot before role admission, fixing the PostgreSQL 25001 failure reproduced from the failed hosted probe. Real PostgreSQL regression tests passed in Linux CI; historical approval records remain immutable and are checked against their protected reviewed Git source separately from the current correction.
The preceding PR had 28 successful effective checks and one intentional skip, no unresolved reviews, and normal protected-merge readback. This promotion retains branch protection and the active write freeze. No production deployment or hosted mutation is included; one corrected read-only probe requires fresh exact-main authorization after the subsequent main promotion.
Windows Bun CI initially failed before retaining any sample (
RSS_SAMPLER_NATIVE_ROOT_UNOBSERVED). Its failure artifact was preserved. One same-source job retry produced complete raw evidence and a report that passed the official independent validator; source and absolute/relative/noise budgets were unchanged. This is compiler-cohort evidence only and establishes no G003 application improvement.