Skip to content

Fix PostgreSQL search rollout safety - #655

Open
16francej wants to merge 1 commit into
mainfrom
codex/release-0-1-6
Open

Fix PostgreSQL search rollout safety#655
16francej wants to merge 1 commit into
mainfrom
codex/release-0-1-6

Conversation

@16francej

@16francej 16francej commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • mark entry_search_text(text) parallel-unsafe, matching PostgreSQL's requirements for its exception handler
  • extend the existing database safety regression to cover the search parser

Why

The v0.1.6 release candidate deployed successfully to Fly, but the live rollout gate rejected this function because it was declared PARALLEL SAFE. The declaration could also allow PostgreSQL to execute exception-handling PL/pgSQL in a parallel worker.

Validation

  • reproduced the regression against a real local PostgreSQL database before the fix
  • 57 PostgreSQL, search, and post-deploy smoke tests pass after the fix
  • root TypeScript typecheck, targeted ESLint, Prettier, and diff checks pass
  • deployed the exact patched source to qm-prod on Fly
  • qm check --live passes: machine health, S3 round-trip, portal HTTP 200, database rollout checks, and private live-session smoke
  • qm conformance passes

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@16francej
16francej force-pushed the codex/release-0-1-6 branch from 1dbf95b to 0ec2553 Compare August 21, 2026 22:32
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