perf(wrangler): remove execa#12369
Conversation
🦋 Changeset detectedLatest commit: d5d3c20 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: |
3723c61 to
56b361e
Compare
|
The windows CI job appears to keep hanging but also this needs another rebase. |
|
Strange. When I'm next on a Windows machine I can try run them locally and see what happens if that helps |
56b361e to
9499795
Compare
This removes `execa` and uses the much smaller, more modern `tinyexec` instead. Reasoning: - `execa` 640KB vs `tinyexec` 26KB - `execa` 23 packages vs `tinyexec` 1 package - `tinyexec` is widely adopted by most of the modern CLIs today (tsdown, vite, vitest, storybook, etc.) Part of cloudflare#11854.
|
Hi @43081j — we're doing some cleanup of stale draft PRs. This PR has been quiet since 2026-02-13. Could you let us know within 14 days whether you're still planning to continue?
Thanks! |
workers-devprod
left a comment
There was a problem hiding this comment.
Codeowners reviews satisfied
|
Codeowners approval required for this PR:
Show detailed file reviewers |
|
@petebacondarwin i've revived it, thanks for the ping as i also forgot about this one. assuming CI is happy, let me know if there's any changes you or any other maintainers want |
|
Codeowners approval required for this PR:
Show detailed file reviewers |
| await x(name, ["--version"], { | ||
| nodeOptions: { stdio: "ignore" }, | ||
| throwOnError: true, | ||
| }); |
There was a problem hiding this comment.
It looks like the change here is resulting in the test failures.
There was a problem hiding this comment.
ok i figured this one out too - will try get to it at the weekend 👍
This removes
execaand uses the much smaller, more moderntinyexecinstead.
Reasoning:
execa640KB vstinyexec26KBexeca23 packages vstinyexec1 packagetinyexecis widely adopted by most of the modern CLIs today (tsdown,vite, vitest, storybook, etc.)
Part of #11854.