Skip to content

fix(core): parameterize v1 migration messages - #41877

Merged
thdxr merged 1 commit into
v2from
v1-sql-escaping
Aug 11, 2026
Merged

fix(core): parameterize v1 migration messages#41877
thdxr merged 1 commit into
v2from
v1-sql-escaping

Conversation

@opencode-agent

Copy link
Copy Markdown
Contributor

Summary

  • write migrated V1 and previous-V2 session messages through the typed Drizzle insert builder
  • keep JSON payloads bound as SQL parameters so apostrophes cannot break the statement
  • cover apostrophes in both migration sources

Fixes #41869

Tests

  • bun test test/v1-migration.test.ts --timeout 30000
  • bun typecheck

Requested by: @thdxr (Dax via Slack)

@thdxr
thdxr merged commit 055a380 into v2 Aug 11, 2026
10 checks passed
@thdxr
thdxr deleted the v1-sql-escaping branch August 11, 2026 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant