Skip to content

chore(db): migrate Drizzle v1 compatibility#2794

Open
RSO wants to merge 2 commits intomainfrom
RSO/rambunctious-grain
Open

chore(db): migrate Drizzle v1 compatibility#2794
RSO wants to merge 2 commits intomainfrom
RSO/rambunctious-grain

Conversation

@RSO
Copy link
Copy Markdown
Contributor

@RSO RSO commented Apr 24, 2026

Summary

  • Pin Drizzle ORM and Kit to the v1 beta line and make the Postgres Kit config explicit for schema and migration-table handling.
  • Convert Postgres and Durable Object SQLite migration artifacts to the v3 folder layout expected by Drizzle v1.
  • Preserve existing relational query semantics by routing RQB v1 call sites through db._query and adapting runtime/migrator call shapes.

Verification

  • Reviewed generated v3 migration folders and Durable Object migration bundles for the converted artifacts.
  • N/A for UI/browser verification; this is a dependency, migration, and compatibility update.

Visual Changes

N/A

Reviewer Notes

  • RQB v2 adoption is intentionally out of scope; this PR keeps current query semantics via compatibility APIs.
  • Before production rollout, validate the Drizzle migration-table upgrade against a staging/restored production database snapshot and confirm no historical migrations reapply.


export default { fetch: app.fetch, queue };
export default {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CRITICAL: SessionIngestRPC is no longer exported from the worker entry module

cloud-agent-next binds to session-ingest with "entrypoint": "SessionIngestRPC" in services/cloud-agent-next/wrangler.jsonc. Wrangler resolves that entrypoint from src/index.ts, so after this refactor the service binding no longer has a target and the createSessionForCloudAgent / deleteSessionForCloudAgent RPC calls will fail.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented Apr 24, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • services/session-ingest/src/index.ts - previous CRITICAL resolved
  • services/session-ingest/src/session-ingest-rpc.ts

Reviewed by gpt-5.4-20260305 · 992,819 tokens

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