feat: support postgres schema overrides by hongzexin · Pull Request #1 · chekdata/codex-lb

hongzexin · 2026-08-12T12:31:51Z

Summary

add optional PostgreSQL schema override support for shared-database prod installs
wire the schema setting through runtime, migrations, Helm values, and docs
add focused unit coverage and openspec change files for the new contract

Validation

/tmp/codex-lb-py313/bin/pytest tests/unit/test_db_migrate.py -k "postgres_search_path or ensure_postgres_schema_exists or sync_connection_creates_schema" -q
/tmp/codex-lb-py313/bin/pytest tests/unit/test_db_session.py -k postgres -q
/tmp/codex-lb-py313/bin/pytest tests/unit/test_durable_bridge_sessions.py -k current_postgres_schemas -q
/tmp/codex-lb-py313/bin/pytest tests/unit/test_helm_external_secrets.py -k postgres_schema -q

Source sidechat: 019dc7df-46ed-71f0-84cf-0cb475d1857f

hongzexin · 2026-08-12T14:00:10Z

@codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2b5ef94eba

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Address Codex review by scoping migration state and Alembic version tables to the configured schema, quoting PostgreSQL identifiers, and removing the unrelated image publisher from this PR.

hongzexin · 2026-08-12T14:37:48Z

@codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 63737df687

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Address Codex review by committing schema/search_path setup before Alembic transaction ownership and constraining historical enum catalog probes to the active schema.

hongzexin · 2026-08-12T14:46:59Z

@codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3f6b2a1536

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Address Codex review by resolving historical enum probes through pg_type_is_visible so configured and default search paths both target the actually visible type.

hongzexin · 2026-08-12T14:57:27Z

@codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 50ade0d763

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Source sidechat: 019dc7df-46ed-71f0-84cf-0cb475d1857f

hongzexin · 2026-08-12T15:06:53Z

@codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c36aed9b96

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Source sidechat: 019dc7df-46ed-71f0-84cf-0cb475d1857f

hongzexin · 2026-08-12T15:16:48Z

@codex review

chatgpt-codex-connector · 2026-08-12T15:20:37Z

Codex Review: Didn't find any major issues. Hooray!

Reviewed commit: 5cbd43b610

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

hongzexin added 3 commits August 12, 2026 20:17

feat: support postgres schema overrides

342e981

test: fix postgres schema CI coverage

83e6d8c

ci: publish CHEK internal images

2b5ef94

chatgpt-codex-connector Bot reviewed Aug 12, 2026

View reviewed changes

Comment thread app/db/migrate.py Outdated

Comment thread app/db/migration_url.py Outdated

Comment thread .github/workflows/internal-image.yml Outdated

fix(db): isolate postgres migration schema

63737df

Address Codex review by scoping migration state and Alembic version tables to the configured schema, quoting PostgreSQL identifiers, and removing the unrelated image publisher from this PR.

chatgpt-codex-connector Bot reviewed Aug 12, 2026

View reviewed changes

Comment thread app/db/alembic/env.py

Comment thread app/db/alembic/env.py

fix(db): commit and scope postgres schema setup

3f6b2a1

Address Codex review by committing schema/search_path setup before Alembic transaction ownership and constraining historical enum catalog probes to the active schema.

github-actions Bot added the db migration label Aug 12, 2026

chatgpt-codex-connector Bot reviewed Aug 12, 2026

View reviewed changes

Comment thread app/db/alembic/versions/20260310_000000_fix_postgresql_enum_value_casing.py Outdated

fix(db): follow postgres enum visibility

50ade0d

Address Codex review by resolving historical enum probes through pg_type_is_visible so configured and default search paths both target the actually visible type.

chatgpt-codex-connector Bot reviewed Aug 12, 2026

View reviewed changes

Comment thread deploy/helm/codex-lb/README.md Outdated

docs(db): move shared schema guidance to docs

c36aed9

Source sidechat: 019dc7df-46ed-71f0-84cf-0cb475d1857f

chatgpt-codex-connector Bot reviewed Aug 12, 2026

View reviewed changes

Comment thread app/core/config/settings.py Outdated

Comment thread app/db/alembic/env.py

fix(db): validate schema and scope offline migrations

5cbd43b

Source sidechat: 019dc7df-46ed-71f0-84cf-0cb475d1857f

hongzexin merged commit e8ef0a5 into main Aug 12, 2026
34 checks passed

hongzexin mentioned this pull request Aug 12, 2026

ci: publish CHEK internal images #2

Merged

Uh oh!

Conversation

hongzexin commented Aug 12, 2026

Summary

Validation

Uh oh!

hongzexin commented Aug 12, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

Uh oh!

Uh oh!

hongzexin commented Aug 12, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

Uh oh!

hongzexin commented Aug 12, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

hongzexin commented Aug 12, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

hongzexin commented Aug 12, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

Uh oh!

hongzexin commented Aug 12, 2026

Uh oh!

chatgpt-codex-connector Bot commented Aug 12, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant