EDITOR=C:/Program in Windows Git-Bash #52265

Activity

1. github-actions

   assigned

   Hona

   on Sep 30, 2026
2. github-actions

   added

   needs:complianceThis means the issue will auto-close after 2 hours.This means the issue will auto-close after 2 hours.

   and removed

   needs:complianceThis means the issue will auto-close after 2 hours.This means the issue will auto-close after 2 hours.

   on Sep 30, 2026
3. github-actions commented on Sep 30, 2026

   

   github-actionsBot

   on Sep 30, 2026 – with GitHub Actions

   Contributor

   More actions

   Thanks for updating the issue with reproduction steps and system details!
4. github-actions commented on Sep 30, 2026

   

   github-actionsBot

   on Sep 30, 2026 – with GitHub Actions

   Contributor

   More actions

   Thanks for updating the issue with reproduction steps and system details!
5. Roffild commented on Sep 30, 2026

   

   Roffild

   on Sep 30, 2026

   Author

   More actions

   Google:

   import { parseArgsStringToArgv } from 'string-argv';
   // Или: const { parseArgsStringToArgv } = require('string-argv');
   
   const commandLine = '"C:/Program Files/Notepad++/notepad++.exe" -multiInst -nosession';
   
   const args = parseArgsStringToArgv(commandLine);
   const [executable, ...commandArgs] = args;
   
   console.log('Exec:', executable);
   console.log('Args:', commandArgs);
6. opencodejr commented on Oct 28, 2026

   

   opencodejrBot

   on Oct 28, 2026

   Contributor

   More actions

   Thanks for the report and for pointing at the code. I can confirm this.

   I don't have Windows here, so I reproduced it on Linux with the same kind of value: a fake editor at /tmp/opencode/Program Files/Fake Editor/editor.sh and EDITOR="'<that path>' -multiInst -nosession". Running /editor shows Failed to open editor: ENOENT: no such file or directory, posix_spawn ''/tmp/opencode/Program'. The command is split on every space and the quotes aren't handled, so only the part before the first space is run as the program. On Windows that part is passed to cmd, which gives the 'C:/Program' is not recognized... error you saw.

   This happens on 2.2.0 and still happens on the latest dev (86ce19d). The recording and script below show it. /export uses the same code to open the editor, so it should be affected too.

   Related:

   • fix(tui): $EDITOR with arguments and spaces #55423 and fix(tui): launch editors at quoted paths #51787: open PRs with fixes for this parsing
   • tui: /export and /editor fail when EDITOR/VISUAL contains a path with spaces #50722: the same bug reported on v1, closed because v1 only gets critical fixes

   Until it's fixed, pointing EDITOR at a path with no spaces should avoid it, for example a small wrapper script or a symlink to Notepad++.

   recording.mp4

   

   Reproduction script (drive.ts)

   // Repro for #52265: $EDITOR with a quoted path containing spaces + arguments
   // breaks /editor, because packages/tui/src/editor.ts splits $EDITOR on " ".
   import { Effect } from "effect"
   import { chmodSync, mkdirSync, writeFileSync } from "node:fs"
   import { OpenCodeDriver } from "opencode-drive"
   
   // A fake "editor" living in a directory whose name contains a space
   // (stands in for "C:/Program Files/Notepad++/notepad++.exe").
   const dir = "/tmp/opencode/Program Files/Fake Editor"
   mkdirSync(dir, { recursive: true })
   writeFileSync(`${dir}/editor.sh`, '#!/bin/sh\nfor f; do last="$f"; done\nprintf "EDITED BY FAKE EDITOR" > "$last"\n')
   chmodSync(`${dir}/editor.sh`, 0o755)
   delete process.env.VISUAL
   process.env.EDITOR = `'${dir}/editor.sh' -multiInst -nosession`
   
   export default OpenCodeDriver.use(
     { ...(process.env.OC_DEV ? { opencode: { dev: process.env.OC_DEV } } : {}), tui: { recording: true } },
     ({ ui }) =>
       Effect.gen(function* () {
         yield* ui.screenshot("01-home")
         yield* ui.type("/editor")
         yield* Effect.sleep("1 second")
         yield* ui.screenshot("02-typed")
         yield* ui.press("return")
         yield* Effect.sleep("4 seconds")
         yield* ui.screenshot("03-after-editor")
         const frame = yield* ui.capture()
         yield* Effect.log(JSON.stringify(frame).slice(0, 4000))
       }),
   )

   Run it with opencode-drive: opencode-drive run ./drive.ts. It drives opencode from your PATH; set OC_DEV to the path of an OpenCode checkout to run that from source instead.
7. opencodejr

   added

   reproduced

   on Oct 28, 2026