feat(ai-persistence-drizzle): schema-first ownership, Postgres provider, schema subpaths#994
Merged
Merged
Conversation
…grations Stop publishing SQL migrations and a default bundled schema from the drizzle adapter. Apps own DDL via tanstack-ai-drizzle-schema + their drizzle-kit journal; drizzlePersistence requires an injected schema. sqlitePersistence keeps local defaults via createDefaultSqliteSchema and optional runtime CREATE TABLE IF NOT EXISTS bootstrap.
…errupt indexes
- drizzlePersistence(db, { provider: 'sqlite' | 'pg', schema }): one entry,
overload-discriminated so db/provider/schema must agree at compile time,
with a runtime dialect guard (is PgDatabase / BaseSQLiteDatabase) and a
provider-aware schema assertion
- Dialect-neutral contract: TanstackAiTableShapes single source of truth,
projected as TanstackAiSqliteSchema and TanstackAiPgSchema
- Postgres: createDefaultPgSchema (jsonb/bigint), ensurePgTables bootstrap,
pg starter asset via tanstack-ai-drizzle-schema --dialect pg, and a
conformance suite against real Postgres (pglite)
- Schema subpath exports /sqlite-schema and /pg-schema so stock-table users
re-export one line into their drizzle-kit paths instead of copying a file
- Default lookup indexes on interrupts (thread_id, run_id) across defaults,
starters, and the ensure* bootstraps
- Stores stay a single SQLite-typed implementation; the pg pair is re-faced
at one documented seam in drizzlePersistence
- Docs (drizzle/sql-backends/migrations), middleware skill, changeset, and
Cloudflare D1 call site updated
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
🚀 Changeset Version Preview17 package(s) bumped directly, 36 bumped as dependents. 🟥 Major bumps
🟨 Minor bumps
🟩 Patch bumps
|
|
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
|
View your CI Pipeline Execution ↗ for commit 7b3a996
☁️ Nx Cloud last updated this comment at |
@tanstack/ai
@tanstack/ai-acp
@tanstack/ai-angular
@tanstack/ai-anthropic
@tanstack/ai-bedrock
@tanstack/ai-claude-code
@tanstack/ai-client
@tanstack/ai-code-mode
@tanstack/ai-code-mode-skills
@tanstack/ai-codex
@tanstack/ai-devtools-core
@tanstack/ai-durable-stream
@tanstack/ai-elevenlabs
@tanstack/ai-event-client
@tanstack/ai-fal
@tanstack/ai-gemini
@tanstack/ai-grok
@tanstack/ai-grok-build
@tanstack/ai-groq
@tanstack/ai-isolate-cloudflare
@tanstack/ai-isolate-node
@tanstack/ai-isolate-quickjs
@tanstack/ai-mcp
@tanstack/ai-memory
@tanstack/ai-mistral
@tanstack/ai-ollama
@tanstack/ai-openai
@tanstack/ai-opencode
@tanstack/ai-openrouter
@tanstack/ai-persistence
@tanstack/ai-persistence-cloudflare
@tanstack/ai-persistence-drizzle
@tanstack/ai-persistence-prisma
@tanstack/ai-preact
@tanstack/ai-react
@tanstack/ai-react-ui
@tanstack/ai-sandbox
@tanstack/ai-sandbox-cloudflare
@tanstack/ai-sandbox-daytona
@tanstack/ai-sandbox-docker
@tanstack/ai-sandbox-local-process
@tanstack/ai-sandbox-sprites
@tanstack/ai-sandbox-vercel
@tanstack/ai-solid
@tanstack/ai-solid-ui
@tanstack/ai-svelte
@tanstack/ai-utils
@tanstack/ai-vue
@tanstack/ai-vue-ui
@tanstack/openai-base
@tanstack/preact-ai-devtools
@tanstack/react-ai-devtools
@tanstack/solid-ai-devtools
commit: |
…/drizzle-schema-first
- ai-persistence: TEXT_MESSAGE_START event literal now carries the required ag-ui `role` field - ai-client resume-snapshot tests: construct the joinRun:undefined adapter without an invalid cast; remove a dead never-typed else-branch assertion - ts-react-chat: project hydrated history onto an explicit JSON-safe wire shape (server fns require serializable payloads; UIMessage part metadata / tool-call input/output are typed unknown/any); default zod-defaulted tool inputs at the destructure - docs: mark the router-loader resume snippet as a schematic fence for kiira - scan-dangling-dts: strip comments so JSDoc @example imports (rewritten to .js by declaration emit) are not scanned as real imports
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎯 Changes
Stacked on #984. Evolves the Drizzle persistence backend to schema-first ownership with SQLite + Postgres support behind one provider-discriminated entry:
drizzlePersistence(db, { provider: 'sqlite' | 'pg', schema })— overloads pair the db, provider, and schema dialects at compile time (@ts-expect-error-verified in type tests); a runtimeis()guard and a provider-aware schema assertion re-verify at construction.TanstackAiTableShapesis the single source of truth, projected onto SQLite (TanstackAiSqliteSchema) and Postgres (TanstackAiPgSchema) table types. Renames, drizzlecasing, and app-owned columns stay free.createDefaultPgSchema(jsonb/bigint),ensurePgTablesbootstrap,tanstack-ai-drizzle-schema --dialect pgstarter, and a full conformance suite against real Postgres (PGlite)./sqlite-schemaand/pg-schemaexports: stock-table users re-export one line into their drizzle-kit paths and migrations track package upgrades; the emit-CLI remains for owned/customized schemas (renames, extra columns, index tuning).interrupts.thread_id/interrupts.run_id(the columns the stores list by), emitted by the bootstraps and shipped in defaults + starters.drizzle,sql-backends,migrations,client-persistencefix), middleware skill, changeset, and the Cloudflare D1 call site updated.✅ Checklist
pnpm run test:pr.🚀 Release Impact