Skip to content

Remove eval() from the Wrangler bundle#11452

Merged
penalosa merged 2 commits intomainfrom
penalosa/no-more-eval
Nov 28, 2025
Merged

Remove eval() from the Wrangler bundle#11452
penalosa merged 2 commits intomainfrom
penalosa/no-more-eval

Conversation

@penalosa
Copy link
Contributor

Patch @netlify/build-info to remove the dependency on @iarna/toml, which removes the eval() warning from Wrangler bundles


  • Tests
    • Tests included
    • Tests not necessary because: covered by existing tests
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: not user facing
  • Wrangler V3 Backport
    • Wrangler PR:
    • Not necessary because: not a dep in v3

@penalosa penalosa requested a review from a team as a code owner November 27, 2025 17:51
@changeset-bot
Copy link

changeset-bot bot commented Nov 27, 2025

🦋 Changeset detected

Latest commit: d3c8068

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

@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Nov 27, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 27, 2025

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@11452

wrangler

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

commit: d3c8068

Copy link
Contributor

@petebacondarwin petebacondarwin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we upstream this fix?

@ascorbic
Copy link
Contributor

Netlify needs it to read their config. An upstream fix would be to switch to smol-toml or some other lib

@penalosa penalosa merged commit 76f0540 into main Nov 28, 2025
38 of 40 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Nov 28, 2025
@penalosa penalosa deleted the penalosa/no-more-eval branch November 28, 2025 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants