Remove lint consistent-type-imports override (to off) for miniflare

[dario-piotrowicz]

We have a set of lint overrides for miniflare that we should get rid of:

workers-sdk/.oxlintrc.jsonc

Line 123 in fb67a18

// TODO: remove these overrides. This is a legacy of Miniflare's history, not intentional

I think it makes sense to try to gradually get rid of those instead of trying to remove them all in one go.

In this PR I am getting rid of the consistent-type-imports one, re-aligning miniflare with the rest of the codebase.

---

• Tests
  • Tests included/updated
  • Automated tests not possible - manual testing has been completed as follows:
  • Additional testing not necessary because: simple code refactoring
• Public documentation
  • Cloudflare docs PR(s):
  • Documentation not necessary because: simple code refactoring

A picture of a cute animal (not mandatory, but encouraged)

---

[changeset-bot]

⚠️ No Changeset found

Latest commit: dd441ff

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

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

[ask-bonk]

UnknownError: ProviderInitError

github run

[workers-devprod]

Codeowners approval required for this PR:

• @cloudflare/workers-kv
• ✅ @cloudflare/wrangler

Show detailed file reviewers

• packages/miniflare/src/workers/kv/constants.ts: [@cloudflare/workers-kv]
• packages/miniflare/src/workers/kv/namespace.worker.ts: [@cloudflare/workers-kv]
• packages/miniflare/src/workers/kv/sites.worker.ts: [@cloudflare/workers-kv]
• packages/miniflare/test/plugins/kv/index.spec.ts: [@cloudflare/workers-kv]

[ask-bonk]

@dario-piotrowicz Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

[pkg-pr-new]

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

wrangler

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

commit: dd441ff

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 5 additional findings.

[penalosa]

Only a skim review, seems completely mechanical

[dario-piotrowicz]

For some reason the devprod bot is not approving the PR... I've copied this PR to #13250 hoping that the bot might work there 😖