Skip to content

refactor: migrate process backend from execa to tinyexec#77

Open
davigiroux wants to merge 1 commit into
wevm:mainfrom
davigiroux:davigiroux/prool-tinyexec
Open

refactor: migrate process backend from execa to tinyexec#77
davigiroux wants to merge 1 commit into
wevm:mainfrom
davigiroux:davigiroux/prool-tinyexec

Conversation

@davigiroux

Copy link
Copy Markdown

Closes #34.

Wraps tinyexec in a small tagged-template adapter so the existing ($) => $\...`and$({ env })`...`callable form keep working. Instance files (anvil/alto/tempo) and theprool.processes.execa` namespace are unchanged. (Path 2 from the issue thread.)

  • Adapter splits static segments on whitespace and spreads array interpolations into argv. Relies on toArgs returning unquoted string[], which it already does.
  • env option merges into process.env.
  • Two new tests pin the array-spread contract and env merging. The five original process tests pass unchanged.
  • About 16 fewer transitive deps. No new deps added.

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedtinyexec@​1.1.2991001009080

View full report

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate from execa to tinyexec

1 participant