Skip to content

perf(wrangler): remove execa#12369

Open
43081j wants to merge 10 commits intocloudflare:mainfrom
43081j:execaless
Open

perf(wrangler): remove execa#12369
43081j wants to merge 10 commits intocloudflare:mainfrom
43081j:execaless

Conversation

@43081j
Copy link
Copy Markdown
Contributor

@43081j 43081j commented Feb 2, 2026

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 #11854.


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

Open with Devin

@43081j 43081j requested review from a team as code owners February 2, 2026 17:32
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Feb 2, 2026

🦋 Changeset detected

Latest 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

@github-project-automation github-project-automation Bot moved this to Untriaged in workers-sdk Feb 2, 2026
@43081j 43081j changed the title perf: remove execa perf(wrangler): remove execa Feb 2, 2026
devin-ai-integration[bot]

This comment was marked as resolved.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Feb 2, 2026

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: 5a4f119

devin-ai-integration[bot]

This comment was marked as resolved.

Copy link
Copy Markdown
Contributor

@ascorbic ascorbic left a comment

Choose a reason for hiding this comment

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

Thanks, @43081j!

@github-project-automation github-project-automation Bot moved this from Untriaged to Approved in workers-sdk Feb 3, 2026
devin-ai-integration[bot]

This comment was marked as resolved.

@petebacondarwin petebacondarwin force-pushed the execaless branch 4 times, most recently from 3723c61 to 56b361e Compare February 3, 2026 20:50
@petebacondarwin
Copy link
Copy Markdown
Contributor

The windows CI job appears to keep hanging but also this needs another rebase.

@43081j
Copy link
Copy Markdown
Contributor Author

43081j commented Feb 3, 2026

Strange. When I'm next on a Windows machine I can try run them locally and see what happens if that helps

@petebacondarwin petebacondarwin moved this from Approved to In Progress in workers-sdk Feb 4, 2026
@petebacondarwin petebacondarwin marked this pull request as draft February 4, 2026 09:42
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.
@petebacondarwin
Copy link
Copy Markdown
Contributor

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?

  • If yes: please leave a comment with a status update or push a rebase, and we'll keep it open.
  • If no: no need to reply — we'll close it after 14 days, and you can always reopen later if you pick it back up.

Thanks!

Copy link
Copy Markdown
Contributor

@workers-devprod workers-devprod left a comment

Choose a reason for hiding this comment

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

Codeowners reviews satisfied

@github-project-automation github-project-automation Bot moved this from In Progress to Approved in workers-sdk Apr 28, 2026
@43081j 43081j marked this pull request as ready for review April 28, 2026 13:43
@workers-devprod
Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

@43081j
Copy link
Copy Markdown
Contributor Author

43081j commented Apr 28, 2026

@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

devin-ai-integration[bot]

This comment was marked as resolved.

@workers-devprod
Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

devin-ai-integration[bot]

This comment was marked as resolved.

Comment on lines +110 to +113
await x(name, ["--version"], {
nodeOptions: { stdio: "ignore" },
throwOnError: true,
});
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It looks like the change here is resulting in the test failures.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ok i figured this one out too - will try get to it at the weekend 👍

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

Labels

None yet

Projects

Status: Approved

Development

Successfully merging this pull request may close these issues.

5 participants