Request
Issue #1219
draft pr openA check exited non-zero in the sandbox. See the failing script + logs below.
An AI estimate of whether the change actually fixes the issue — the checks themselves only confirm nothing broke. It's a hint for review, not a pass/fail check.
Why?
The diff adds `normalizeInputStdio` in `lib/stdio/stdio-option.js` which checks whether `input`/`inputFile` is defined and `stdioArray[0]` is `'inherit'`, then rewrites `stdin` to `'pipe'` while preserving the other stdio values, exactly matching the requested silent replacement. New tests in `test/stdio/stdio-option.js` explicitly verify all three issue examples: `{input: 'foo', stdio: 'inherit'}`, `{input: 'foo', stdio: ['inherit', ...]}`, and `{input: 'foo', stdin: 'inherit'}`.
lintpassed · 23.0s
> execa@9.6.1 lint > xo npm warn Unknown env config "verify-deps-before-run". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown user config "dangerously-allow-all-builds". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown user config "verify-deps-before-run". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options.
testfailed · exit 1 · 386.4s
00 | 100 | 100 | reject.js | 100 | 100 | 100 | 100 | result.js | 100 | 100 | 100 | 100 | repo/lib/stdio | 100 | 100 | 100 | 100 | direction.js | 100 | 100 | 100 | 100 | duplicate.js | 100 | 100 | 100 | 100 | handle-async.js | 100 | 100 | 100 | 100 | handle-sync.js | 100 | 100 | 100 | 100 | handle.js | 100 | 100 | 100 | 100 | input-option.js | 100 | 100 | 100 | 100 | native.js | 100 | 100 | 100 | 100 | stdio-option.js | 100 | 100 | 100 | 100 | type.js | 100 | 100 | 100 | 100 | repo/lib/terminate | 100 | 100 | 100 | 100 | cancel.js | 100 | 100 | 100 | 100 | cleanup.js | 100 | 100 | 100 | 100 | graceful.js | 100 | 100 | 100 | 100 | kill.js | 100 | 100 | 100 | 100 | signal.js | 100 | 100 | 100 | 100 | timeout.js | 100 | 100 | 100 | 100 | repo/lib/transform | 100 | 100 | 100 | 100 | encoding-transform.js | 100 | 100 | 100 | 100 | generator.js | 100 | 100 | 100 | 100 | normalize.js | 100 | 100 | 100 | 100 | object-mode.js | 100 | 100 | 100 | 100 | run-async.js | 100 | 100 | 100 | 100 | run-sync.js | 100 | 100 | 100 | 100 | split.js | 100 | 100 | 100 | 100 | validate.js | 100 | 100 | 100 | 100 | repo/lib/utils | 100 | 100 | 100 | 100 | abort-signal.js | 100 | 100 | 100 | 100 | deferred.js | 100 | 100 | 100 | 100 | max-listeners.js | 100 | 100 | 100 | 100 | standard-stream.js | 100 | 100 | 100 | 100 | uint-array.js | 100 | 100 | 100 | 100 | repo/lib/verbose | 100 | 100 | 100 | 100 | complete.js | 100 | 100 | 100 | 100 | custom.js | 100 | 100 | 100 | 100 | default.js | 100 | 100 | 100 | 100 | error.js | 100 | 100 | 100 | 100 | info.js | 100 | 100 | 100 | 100 | ipc.js | 100 | 100 | 100 | 100 | log.js | 100 | 100 | 100 | 100 | output.js | 100 | 100 | 100 | 100 | start.js | 100 | 100 | 100 | 100 | values.js | 100 | 100 | 100 | 100 | ------------------------|---------|----------|---------|---------|------------------- npm warn Unknown env config "verify-deps-before-run". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown user config "dangerously-allow-all-builds". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown user config "verify-deps-before-run". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown env config "dangerously-allow-all-builds". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown env config "verify-deps-before-run". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown user config "dangerously-allow-all-builds". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown user config "verify-deps-before-run". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown env config "dangerously-allow-all-builds". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown env config "verify-deps-before-run". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown user config "dangerously-allow-all-builds". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options. npm warn Unknown user config "verify-deps-before-run". This will stop working in the next major version of npm. See `npm help npmrc` for supported config options.
typecheck and build are not defined separately in this repo.
Ran 6/6/2026, 4:19:47 AM · npm
Get the code
Pull this branch
gh pr checkout 3 --repo kamransiddiqui/execa
git fetch https://github.com/kamransiddiqui/execa.git gitgot/issue-1219 git checkout -b gitgot/issue-1219 FETCH_HEAD
Fork-mode request: the branch lives on kamransiddiqui/execa (your fork), not sindresorhus/execa, so the commands above target the fork.
Drive locally
Paste this into your local coding agent (Claude Code, Cursor, etc.) to continue iterating with full repo context.
You are continuing work on a draft PR for sindresorhus/execa, addressing issue #1219. First, check out the branch with gitgot's first-pass change (run from a clone of the repo): gh pr checkout 3 --repo kamransiddiqui/execa Then review the diff and: - run the project's lint/typecheck/test commands and fix anything broken - if the diff is wrong-headed, revert it and start fresh from the issue - keep the change tight; do not refactor unrelated code
- No comments on the PR yet. Use the form below to start the thread.