Skip to content

Fix op checkpoint JSONB binding#2292

Closed
uncfreak1255-code wants to merge 20 commits into
garrytan:masterfrom
uncfreak1255-code:codex/op-checkpoint-jsonb-direct-fix
Closed

Fix op checkpoint JSONB binding#2292
uncfreak1255-code wants to merge 20 commits into
garrytan:masterfrom
uncfreak1255-code:codex/op-checkpoint-jsonb-direct-fix

Conversation

@uncfreak1255-code

Copy link
Copy Markdown

Summary

  • Fix recordCompleted so op_checkpoints.completed_keys is written as a real JSONB array on Postgres, not a double-encoded scalar string.
  • Add PGLite and real-Postgres regression coverage for the completed_keys shape.
  • Tighten the JSONB static guard for executeRawDirect and fix the admin build gate that was timing out in verify.

Proof

  • bun run verify -> 30/30 checks green
  • bun test test/op-checkpoint.test.ts -> 31 pass, 0 fail
  • Fresh Postgres pgvector/pgvector:pg16 E2E: DATABASE_URL=postgresql://postgres:postgres@localhost:5435/gbrain_test bun test test/e2e/op-checkpoint-jsonb-postgres.test.ts -> 1 pass, 0 fail
  • Direct Postgres readback: array|chunk-a|2

Notes

Compared with upstream PR #2288, this keeps the fix narrow but avoids passing a bare top-level JS array into $3::jsonb; the value is wrapped in an object and extracted server-side with ($3::jsonb)->'completed_keys'.

uncfreak1255-code and others added 20 commits June 17, 2026 15:48
…onb-regression-tests

test: cover migrate source JSONB config shape
…-pr23

docs: keep release privacy guidance generic
…e-jsonb-ci-lane

Add migrate source JSONB regression to Postgres E2E CI
…ricing-clean

[codex] Pin DeepSeek V4 pricing and aliases
…epseek-budget-gate

Add monthly Claude and DeepSeek budget gate
@uncfreak1255-code

Copy link
Copy Markdown
Author

Closing this one because it was opened from the fork master lineage and includes unrelated fork history. I am rebuilding the branch from upstream/master and will reopen a narrow PR.

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