Skip to content

Bulk copy pushed turns#201

Merged
upsetbit merged 1 commit into
masterfrom
codex/issue-106-copyfrom-turns
Jun 6, 2026
Merged

Bulk copy pushed turns#201
upsetbit merged 1 commit into
masterfrom
codex/issue-106-copyfrom-turns

Conversation

@upsetbit
Copy link
Copy Markdown
Contributor

@upsetbit upsetbit commented Jun 6, 2026

Closes #106.

Summary

  • replace per-turn Postgres inserts in replaceTurns with tx.CopyFrom
  • keep the existing delete-then-replace transaction semantics
  • extend the server e2e Push test to cover default turn kind and tool evidence metadata

Local validation

  • PROSA_TEST_PG_URL=postgres://prosa:prosa@localhost:55432/prosa?sslmode=disable go test ./internal/server/handlers -run 'TestSessionsConnectEndToEnd|TestSessionsPush' -count=1 -v\n- PROSA_TEST_PG_URL=postgres://prosa:prosa@localhost:55432/prosa?sslmode=disable go test -race -count=1 ./internal/server/...\n- devbox run -- just lint\n- PROSA_TEST_PG_URL=postgres://prosa:prosa@localhost:55432/prosa?sslmode=disable just test-race

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

@upsetbit upsetbit merged commit b325f90 into master Jun 6, 2026
5 checks passed
@upsetbit upsetbit deleted the codex/issue-106-copyfrom-turns branch June 6, 2026 06:10
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.

Switch server replaceTurns to pgx.CopyFrom

1 participant