fix(db): correct D1 migrations_dir path in wrangler.json#814
Merged
RihanArfan merged 2 commits intonuxt-hub:mainfrom Feb 2, 2026
Merged
fix(db): correct D1 migrations_dir path in wrangler.json#814RihanArfan merged 2 commits intonuxt-hub:mainfrom
RihanArfan merged 2 commits intonuxt-hub:mainfrom
Conversation
|
@onmax is attempting to deploy a commit to the NuxtLabs Team on Vercel. A member of the Team first needs to authorize it. |
cad9b6f to
d172bff
Compare
commit: |
RihanArfan
approved these changes
Feb 2, 2026
onmax
added a commit
to onmax/core
that referenced
this pull request
Feb 5, 2026
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.
Closes #813
Fixes same bug as #793 but in CLI code path (PR #794 only fixed
setup.ts, missedclient.ts)Summary
Two fixes:
migrations_dir path -
migrations_dirin generated wrangler.json was set to.output/server/db/migrations/but wrangler resolves paths relative to its config file location (.output/server/). This caused doubled paths like.output/server/.output/server/db/migrations/. Fixed by setting the path relative to wrangler.json:db/migrations/sqlite/.D1 API endpoint -
client.tsused wrong endpoint/d1/db/instead of/d1/database/, causing POST method errors when running CLI commands against D1.Regression introduced in #720.
StackBlitz
CLI Reproduction
Verify fix
The
-fixfolder uses pnpm patch to apply the fix locally.