Repository navigation
fix(queue-storage): refresh SET-only lane repairs on 0.6 - #668
Conversation
Codex Review Summary: ✅ CompletedThis 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. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the
|
Lane repair could reassign a legacy sequence to the schema owner and then lose the authority needed to finish its runtime grants when the migrator had SET-only membership. This backports the repair from #637: discovery accepts that membership, repair temporarily activates the new owner when needed, and restores the exact previous role.
The repeatable
definer_lane_provisioningpatch now detects an older installed repair helper and refreshes it instead of skipping it merely because the function exists. The schema remains v40. The body marker identifies this helper generation; it is not a full definition or ACL integrity check. The upgrade guide runs migration before lane diagnostics and documents explicit helper/schema grants.Validation: