Default D1 export to --local

[joshthoward]

Copy behavior of wrangler d1 execute for wrangler d1 export.

• Tests
  • TODO (before merge)
  • Tests included
  • Tests not necessary because:
• E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)
  • I don't know
  • Required
  • Not required because: Change is covered by unit tests
• Public documentation
  • TODO (before merge)
  • Cloudflare docs PR(s): Document change to default D1 export to --local cloudflare-docs#19088
  • Documentation not necessary because:

[changeset-bot]

🦋 Changeset detected

Latest commit: 4791a5c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
wrangler	Minor
@cloudflare/vitest-pool-workers	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

A wrangler prerelease is available for testing. You can install this latest build in your project with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-wrangler-7694

You can reference the automatically updated head of this PR with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/7694/npm-package-wrangler-7694

Or you can use npx with this latest build directly:

npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-wrangler-7694 dev path/to/script.js

Additional artifacts:

wget https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-cloudflare-workers-bindings-extension-7694 -O ./cloudflare-workers-bindings-extension.0.0.0-vf43a39eab.vsix && code --install-extension ./cloudflare-workers-bindings-extension.0.0.0-vf43a39eab.vsix

npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-create-cloudflare-7694 --no-auto-update

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-cloudflare-kv-asset-handler-7694

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-miniflare-7694

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-cloudflare-pages-shared-7694

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-cloudflare-unenv-preset-7694

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-cloudflare-vitest-pool-workers-7694

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-cloudflare-workers-editor-shared-7694

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-cloudflare-workers-shared-7694

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12678013135/npm-package-cloudflare-workflows-shared-7694

Note that these links will no longer work once the GitHub Actions artifact expires.

---

wrangler@3.100.0 includes the following runtime dependencies:

Package	Constraint	Resolved
miniflare	workspace:*	3.20241230.0
workerd	1.20241230.0	1.20241230.0
workerd --version	1.20241230.0	2024-12-30

Please ensure constraints are pinned, and miniflare/workerd minor versions match.

[isaac-mcfadyen]

IMO this is a very breaking change (all users using D1 export will suddenly be exporting their local DB instead of their remote DB).

Because of that, is this planned to be a major version bump? Landing this in a minor/patch has the potential to disrupt many user workflows.

[joshthoward]

This is not a breaking change. This is a feature enhancement where if --local or --remote are not specified wrangler d1 export will default to local rather than failing with a user error. All existing uses will continue working. This will get a minor version bump.

[rozenmd]

To be clear, wrangler d1 export did not default to either local or remote. If you didn't specify, it would error:

% npx wrangler d1 export northwind --output ./test.sql

 ⛅️ wrangler 3.100.0
--------------------


✘ [ERROR] You must specify either --local or --remote

IMO this is a feat/minor bump since existing user code cannot already be relying on this behaviour (no one has scripts that intentionally error).

[penalosa]

For posterity, merging with a failing C3 check because this PR doesn't touch C3 and the same test is failing (flaking, probably) on main