-
Notifications
You must be signed in to change notification settings - Fork 1.1k
[vite-plugin] add a check for installed wrangler version #11176
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[vite-plugin] add a check for installed wrangler version #11176
Conversation
🦋 Changeset detectedLatest commit: c890500 The changes in this PR will be included in the next version bump. 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 |
create-cloudflare
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
|
Looks like Windows needs it to fail earlier, so I've moved it back up to the module level so it fails as soon as the config loads. The error isn't as nice (which is why I moved it into the config hook before), but it seems that's too late for Windows. |
|
omg thank you @petebacondarwin and @ascorbic !!! I was bitten by this one before and it's painful to know what's going on without an explicit check. |
856bc1c to
056b217
Compare
This change delays confusing "missing export" failures to allow us to run a more user friendly version skew check, for when the user installs a version of Wrangler that breaks the plugin
…being used internally is correct This is necessary because some package managers will incorrectly merge peer dependencies with actual dependencies (often ignoring peer dependency constraints).
…angler being used internally is correct
…angler being used internally is correct
…angler being used internally is correct
Co-authored-by: James Opstad <13586373+jamesopstad@users.noreply.github.com>
Co-authored-by: James Opstad <13586373+jamesopstad@users.noreply.github.com>
056b217 to
958ce08
Compare
|
Might be worth re-opening your PR and pushing my commits to it, so that CI works better |
|
Yeah. I'll try that in parallel to this... |
|
Opened #11265 |
* Avoid using object lookup for OAuth Error classes * fixup! Avoid using object lookup for OAuth Error classes
…0703) * feat: add streaming tail consumer configuration support Implement [[streaming_tail_consumer]] configuration similar to tail_consumer, sending to streamingTails instead of tails. Includes: - Configuration schema and validation for streaming_tail_consumers - Miniflare integration with streamingTails support - Development and deployment workflow updates - Print bindings integration for CLI output - Comprehensive test coverage 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> * fixups * Create clever-eggs-sit.md * fix types * Work in Vite + Vitest * fix tests * Add E2E test * Address comments --------- Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: Samuel Macleod <smacleod@cloudflare.com> Co-authored-by: Somhairle MacLeòid <samuel@macleod.space>
* Support the WRANGLER_HIDE_BANNER environment variable across Wrangler & C3 * Reduce C3 framework test timeout. Previously it was 25 mins, now it should be 3 mins * Remove dev server from C3 command. To make this consistent across frameworks we were injecting extra dev commands into package.json files, which meant that generated C3 projects often had both a `start` and a `develop` script. * Reduce logging in package-manager detection * Expand autoconfig & use it in C3 for gatsby * Expand framework configuration and support dry-running * Address comments * Address comments * longer timeout 😢
…rameworks (cloudflare#11013) * [C3] Bump sv in /packages/create-cloudflare/src/frameworks Bumps [sv](https://github.com/sveltejs/cli/tree/HEAD/packages/cli) from 0.9.8 to 0.9.9. - [Release notes](https://github.com/sveltejs/cli/releases) - [Changelog](https://github.com/sveltejs/cli/blob/main/packages/cli/CHANGELOG.md) - [Commits](https://github.com/sveltejs/cli/commits/sv@0.9.9/packages/cli) --- updated-dependencies: - dependency-name: sv dependency-version: 0.9.9 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * chore: update dependencies of "create-cloudflare" package The following dependency versions have been updated: | Dependency | From | To | | ---------- | ----- | ----- | | sv | 0.9.8 | 0.9.9 | --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Wrangler automated PR updater <wrangler@cloudflare.com>
…loudflare/src/frameworks (cloudflare#11015) * [C3] Bump create-docusaurus Bumps [create-docusaurus](https://github.com/facebook/docusaurus/tree/HEAD/packages/create-docusaurus) from 3.9.1 to 3.9.2. - [Release notes](https://github.com/facebook/docusaurus/releases) - [Changelog](https://github.com/facebook/docusaurus/blob/main/CHANGELOG.md) - [Commits](https://github.com/facebook/docusaurus/commits/v3.9.2/packages/create-docusaurus) --- updated-dependencies: - dependency-name: create-docusaurus dependency-version: 3.9.2 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * chore: update dependencies of "create-cloudflare" package The following dependency versions have been updated: | Dependency | From | To | | ----------------- | ----- | ----- | | create-docusaurus | 3.9.1 | 3.9.2 | --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Wrangler automated PR updater <wrangler@cloudflare.com>
…rc/frameworks (cloudflare#11222) * [C3] Bump nuxi in /packages/create-cloudflare/src/frameworks Bumps [nuxi](https://github.com/nuxt/cli/tree/HEAD/packages/nuxi) from 3.29.3 to 3.30.0. - [Release notes](https://github.com/nuxt/cli/releases) - [Commits](https://github.com/nuxt/cli/commits/v3.30.0/packages/nuxi) --- updated-dependencies: - dependency-name: nuxi dependency-version: 3.30.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * chore: update dependencies of "create-cloudflare" package The following dependency versions have been updated: | Dependency | From | To | | ---------- | ------ | ------ | | nuxi | 3.29.3 | 3.30.0 | --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Wrangler automated PR updater <wrangler@cloudflare.com>
…1235) * Bump the workerd-and-workers-types group with 2 updates Bumps the workerd-and-workers-types group with 2 updates: [workerd](https://github.com/cloudflare/workerd) and [@cloudflare/workers-types](https://github.com/cloudflare/workerd). Updates `workerd` from 1.20251109.0 to 1.20251111.0 - [Release notes](https://github.com/cloudflare/workerd/releases) - [Changelog](https://github.com/cloudflare/workerd/blob/main/Dockerfile.release) - [Commits](cloudflare/workerd@v1.20251109.0...v1.20251111.0) Updates `@cloudflare/workers-types` from 4.20251109.0 to 4.20251111.0 - [Release notes](https://github.com/cloudflare/workerd/releases) - [Changelog](https://github.com/cloudflare/workerd/blob/main/Dockerfile.release) - [Commits](https://github.com/cloudflare/workerd/commits) --- updated-dependencies: - dependency-name: workerd dependency-version: 1.20251111.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: workerd-and-workers-types - dependency-name: "@cloudflare/workers-types" dependency-version: 4.20251111.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: workerd-and-workers-types ... Signed-off-by: dependabot[bot] <support@github.com> * chore: update dependencies of "miniflare" package The following dependency versions have been updated: | Dependency | From | To | | ---------- | ------------ | ------------ | | workerd | 1.20251109.0 | 1.20251111.0 | --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Wrangler automated PR updater <wrangler@cloudflare.com>
* Fix typo that meant static routing exclude rules were being evaluated in both Vite and the Router Worker * Add changeset * Update changeset * Update changeset
* fix: use .dev.vars when envFiles is empty array * Add tests
…angler being used internally is correct
|
I've fixed this PR up on the internal branch one (#11265). So I think we can close this and finish the review there. |
Fixes [DEVX-2229]
This PR is mostly based on and supersedes #10848, but I've not stacked them because it includes updates from main to resolve conflicts. My changes are all in 8641dc6 so you may want to review that specifically.
Adds a check to ensure that the locally-installed version of wrangler meets the peer dependency requirements of the Vite plugin. This is required because of package managers with lax enforcement of peerdep ranges, which isn't acceptable for us as it can cause breakages.