Skip to content

fix(wrangler): handling of process.env.NODE_ENV in bundling mode#7932

Merged
vicb merged 1 commit intomainfrom
bundle/process-env
Jan 28, 2025
Merged

fix(wrangler): handling of process.env.NODE_ENV in bundling mode#7932
vicb merged 1 commit intomainfrom
bundle/process-env

Conversation

@vicb
Copy link
Copy Markdown
Contributor

@vicb vicb commented Jan 28, 2025

Relates to #7886.

Before this PR:

  • The first check process.env.NODE_ENV should have checked the runtime value (as done a few lines below)
  • This check should only (de)activate define["process.env.NODE_ENV"] and not define as a whole.

This fix a few points from the linked issue, not all.

Notably there is still an issue when process is imported (import process from "node:process";) as "process" will be renamed by the hybrid plugin and not imported!


  • 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:
  • Public documentation
    • TODO (before merge)
    • Cloudflare docs PR(s):
    • Documentation not necessary because: bug fix

@vicb vicb requested a review from a team as a code owner January 28, 2025 12:14
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 28, 2025

🦋 Changeset detected

Latest commit: a1ceab2

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

@vicb vicb force-pushed the bundle/process-env branch from 788069b to a1ceab2 Compare January 28, 2025 12:14
@github-actions
Copy link
Copy Markdown
Contributor

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/13010184680/npm-package-wrangler-7932

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

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

Or you can use npx with this latest build directly:

npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13010184680/npm-package-wrangler-7932 dev path/to/script.js
Additional artifacts:

cloudflare-workers-bindings-extension:

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

create-cloudflare:

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

@cloudflare/kv-asset-handler:

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

miniflare:

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

@cloudflare/pages-shared:

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

@cloudflare/unenv-preset:

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

@cloudflare/vite-plugin:

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13010184680/npm-package-cloudflare-vite-plugin-7932

@cloudflare/vitest-pool-workers:

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

@cloudflare/workers-editor-shared:

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

@cloudflare/workers-shared:

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

@cloudflare/workflows-shared:

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

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


wrangler@3.106.0 includes the following runtime dependencies:

Package Constraint Resolved
miniflare workspace:* 3.20250124.0
workerd 1.20250124.0 1.20250124.0
workerd --version 1.20250124.0 2025-01-24

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

@vicb vicb added the e2e Run wrangler + vite-plugin e2e tests on a PR label Jan 28, 2025
@vicb
Copy link
Copy Markdown
Contributor Author

vicb commented Jan 28, 2025

Thanks for the review @penalosa !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

e2e Run wrangler + vite-plugin e2e tests on a PR

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants