feat(core): refactor shell execution to use node-pty (#6491)

Co-authored-by: Jacob Richman <[email protected]>

1 files changed, 3 insertions, 10 deletions

diff --git a/packages/cli/src/services/prompt-processors/shellProcessor.ts b/packages/cli/src/services/prompt-processors/shellProcessor.ts
index 7acf2415..5242fa7a 100644
--- a/packages/cli/src/services/prompt-processors/shellProcessor.ts
+++ b/packages/cli/src/services/prompt-processors/shellProcessor.ts
@@ -137,11 +137,12 @@ export class ShellProcessor implements IPromptProcessor {
 
       // Execute the resolved command (which already has ESCAPED input).
       if (injection.resolvedCommand) {
-        const { result } = ShellExecutionService.execute(
+        const { result } = await ShellExecutionService.execute(
           injection.resolvedCommand,
           config.getTargetDir(),
           () => {},
           new AbortController().signal,
+          config.getShouldUseNodePtyShell(),
         );
 
         const executionResult = await result;
@@ -154,15 +155,7 @@ export class ShellProcessor implements IPromptProcessor {
         }
 
         // Append the output, making stderr explicit for the model.
-        if (executionResult.stdout) {
-          processedPrompt += executionResult.stdout;
-        }
-        if (executionResult.stderr) {
-          if (executionResult.stdout) {
-            processedPrompt += '\n';
-          }
-          processedPrompt += `--- STDERR ---\n${executionResult.stderr}`;
-        }
+        processedPrompt += executionResult.output;
 
         // Append a status message if the command did not succeed.
         if (executionResult.aborted) {