@code-yeongyu/senpi 2026.5.18 → 2026.5.19

package/dist/modes/interactive/interactive-mode.js

@@ -2989,7 +2989,7 @@ export class InteractiveMode {
         }
         this.showStatus(`Thinking blocks: ${this.hideThinkingBlock ? "hidden" : "visible"}`);
     }
-    openExternalEditor() {
+    async openExternalEditor() {
         // Determine editor (respect $VISUAL, then $EDITOR)
         const editorCmd = process.env.VISUAL || process.env.EDITOR;
         if (!editorCmd) {
@@ -3005,13 +3005,20 @@ export class InteractiveMode {
             this.ui.stop();
             // Split by space to support editor arguments (e.g., "code --wait")
             const [editor, ...editorArgs] = editorCmd.split(" ");
-            // Spawn editor synchronously with inherited stdio for interactive editing
-            const result = spawnSync(editor, [...editorArgs, tmpFile], {
-                stdio: "inherit",
-                shell: process.platform === "win32",
+            process.stdout.write(`Launching external editor: ${editorCmd}\nPi will resume when the editor exits.\n`);
+            // Do not use spawnSync here. On Windows, synchronous child_process calls can keep
+            // Node/libuv's console input read active after ui.stop() pauses stdin, racing
+            // vim/nvim for the console input buffer until Ctrl+C cancels the pending read.
+            const status = await new Promise((resolve) => {
+                const child = spawn(editor, [...editorArgs, tmpFile], {
+                    stdio: "inherit",
+                    shell: process.platform === "win32",
+                });
+                child.on("error", () => resolve(null));
+                child.on("close", (code) => resolve(code));
             });
             // On successful exit (status 0), replace editor content
-            if (result.status === 0) {
+            if (status === 0) {
                 const newContent = fs.readFileSync(tmpFile, "utf-8").replace(/\n$/, "");
                 this.editor.setText(newContent);
             }