Repository navigation
Export kit driver as injectable string for drizzle-kit - #3
Merged
Merged
Conversation
- Move kit-only functions (prepareSqliteParams, normaliseSQLiteUrl) to kit.ts - Simplify API: createNodeSqlDriver(dbPath, prepareSqliteParams) - Add build script to bundle kit.ts as injectable string (dist/kit-string.ts) - Add prepublish task to generate kit-string at publish time - Configure CI workflow to run build:kit-string before publish
- Add hasColumns guard in proxy fallback path (consistent error handling) - Clarify migration-only usage of internal drizzle instance - Document Deno 2.x requirement for --unstable-bundle in README - Add local publishing instructions to README
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
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.
Summary
Adds a
./kit-stringexport containing the bundled driver code as a string. This enables drizzle-kit to inject thenode:sqlitedriver without adding a runtime dependency.Changes
API Refactoring
prepareSqliteParams,normaliseSQLiteUrl) fromshared.tstokit.tscreateNodeSqlDriver(dbPath, prepareSqliteParams)— caller provides the param preparation functionNew Export
./kit-stringexportsdrizzleKitDriverBlock: stringcontaining the bundled driver codedeno task build:kit-stringusingDeno.bundledist/kit-string.ts(gitignored, generated at publish time)CI/Publishing
build:kit-stringstep to publish workflowImproved Error Handling
hasColumns()guard in proxy fallback path with descriptive error messageUsage in drizzle-kit
Testing