fix: escape column names and handle mismatched data types in D1 SQL dump

[invisal]

Fixes #8377, fixes #6357, fixes #5920
Might also related to #8226

This PR improves the D1 SQL dump logic

• Properly escape column and table names to prevent syntax errors.
• Handle SQLite's loose typing by escaping values based on value type instead of enforcing strict column types.
• Add tests to verify SQL dump output and edge cases.

---

• Tests
  • Tests included
  • Tests not necessary because:
• Public documentation
  • Cloudflare docs PR(s):
  • Documentation not necessary because: It is a bugfix
• Wrangler V3 Backport
  • Wrangler PR:
  • Not necessary because: not a Wrangler change

[changeset-bot]

🦋 Changeset detected

Latest commit: e5c988a

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

This PR includes changesets to release 5 packages

Name	Type
miniflare	Patch
@cloudflare/pages-shared	Patch
@cloudflare/vite-plugin	Patch
@cloudflare/vitest-pool-workers	Patch
wrangler	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

[invisal]

SQLite escapes double quotes in identifiers by doubling them using sqlite3_mprintf("\"%w\"", azCol[i])
https://github.com/sqlite/sqlite/blob/master/src/shell.c.in#L2914

The %w format doubles " as "":
https://github.com/sqlite/sqlite/blob/master/src/printf.c#L34

[pkg-pr-new]

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@9866

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@9866

miniflare

npm i https://pkg.pr.new/miniflare@9866

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@9866

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@9866

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@9866

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@9866

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@9866

wrangler

npm i https://pkg.pr.new/wrangler@9866

commit: e5c988a

[alsuren]

Looks good. Thanks for fixing this.

Part of me is tempted to assert that we can round-trip our export through the sqlite cli, reimport it and still get the same results. I doubt that our CI boxes have the sqlite cli installed though, so the way you've done it is probably the right balance.

[holopin-bot]

Congratulations @invisal, the maintainer of this repository has issued you a holobyte! Here it is: https://holopin.io/holobyte/cmd8uaetj1763207jvmdf7wiq6

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!