Use PGlite by default for type generation - #59
Open
vik319 wants to merge 1 commit into
Open
Conversation
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.
Fixes #2.
Running
db:generatenow uses the existing in-memory PGlite implementation by default, so type and SQL generation no longer requires a running PostgreSQL server. Both the API and CLI use this default;--no-pgliteretains the external database option. The CLI awaits generation, and configuration imports use a file URL so the built Node CLI can run on Windows.The regression test covers both the default and explicit PGlite modes and checks that the generated schema contains the migrated table type. The README documents the default and opt-out.
Validation on Windows with Bun 1.4.2:
bun run db:generatein a separate migration fixture succeeded withDATABASE_URLpointing to localhost port 1; generated SQL and TypeScript were inspected.git diff --checkpassed.The existing baseline also prints Windows path warnings during tests. Linux/macOS and the external PostgreSQL opt-out were not tested.