D1 export api change

[geelen]

What this PR solves / how to test

Internal API change for D1 import/export.

Author has addressed the following

• Tests
  • TODO (before merge)
  • Included
  • Not necessary because: behaviour unchanged
• E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)
  • I don't know
  • Required / Maybe required
  • Not required because: behaviour unchanged
• Changeset (Changeset guidelines)
  • TODO (before merge)
  • Included
  • Not necessary because:
• Public documentation
  • TODO (before merge)
  • Cloudflare docs PR(s):
  • Not necessary because: covered by D1 OpenAPI description

[changeset-bot]

🦋 Changeset detected

Latest commit: 5febe40

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

This PR includes changesets to release 2 packages

Name	Type
wrangler	Patch
@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/9991041208/npm-package-wrangler-6203

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

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

Or you can use npx with this latest build directly:

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

Additional artifacts:

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

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

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

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

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

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

---

[email protected] includes the following runtime dependencies:

Package	Constraint	Resolved
miniflare	workspace:*	3.20240712.0
workerd	1.20240712.0	1.20240712.0
workerd --version	1.20240712.0	2024-07-12

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

[petebacondarwin]

As D1 is now GA, I assume that the API cannot be broken by changing the fields that are expected in requests and returned in responses, right? Otherwise previous Wrangler versions will now start failing. Can you confirm this is the case.

[petebacondarwin]

Is there a link to an internal document or ticket describing this API change?

[geelen]

Yeah we'll be providing backwards-compatible API request/responses for versions of Wrangler that predate this change being released (based on user-agent). Internal ticket is CFSQL-942.

[petebacondarwin]

In order for this to trigger a release of Wrangler (and this code) it needs a "patch" changeset.

[petebacondarwin]

Looks like you need to update some tests.

[geelen]

@petebacondarwin added changeset and passing all tests

[petebacondarwin]

I tried re-running the CI jobs a number of times and I am seeing lots of OOM errors.
I can't tell if that is due to changes here, a new problem with Wrangler, or just flakes.

[CarmenPopoviciu]

@geelen can you please rebase your changes on top of main and resolve the conflicts? Maybe that fixes the tests failure we're seeing? 🤞

[RamIdeas]

@geelen can you please rebase your changes on top of main and resolve the conflicts? Maybe that fixes the tests failure we're seeing? 🤞

@geelen I rebased from main and resolved the conflicts on your behalf to get a head start on whether that will fix the OOM test failures