✨ feat: add environment variable support in shell execution (#1369)

Expanded shell options to include env variables and isolateEnv settings.

Author: pelikhan

packages/cli/src/nodehost.ts

@@ -552,6 +552,8 @@ export class NodeHost extends EventTarget implements RuntimeHost {
             cancellationToken,
             stdin: input,
             ignoreError,
+            env,
+            isolateEnv,
         } = options || {}
         const trace = options?.trace?.startTraceDetails(label || command)
         try {
@@ -583,6 +585,8 @@ export class NodeHost extends EventTarget implements RuntimeHost {
                     stdin: input ? undefined : "ignore",
                     stdout: ["pipe"],
                     stderr: ["pipe"],
+                    env,
+                    extendEnv: !isolateEnv,
                 }
             )
             trace?.itemValue(`exit code`, `${exitCode}`)

packages/core/src/git.ts

@@ -108,9 +108,12 @@ export class GitClient implements Git {
         args: string | string[],
         options?: { label?: string }
     ): Promise<string> {
-        const opts = {
+        const opts: ShellOptions = {
             ...(options || {}),
             cwd: this.cwd,
+            env: {
+                LC_ALL: "en_US",
+            },
         }
         const eargs = Array.isArray(args) ? args : shellParse(args)
         dbg(`exec`, shellQuote(eargs))

packages/core/src/types/prompt_template.d.ts

@@ -4236,7 +4236,9 @@ interface SgHost {
 
 interface ShellOptions {
     cwd?: string
+
     stdin?: string
+
     /**
      * Process timeout in  milliseconds, default is 60s
      */
@@ -4250,6 +4252,16 @@ interface ShellOptions {
      * Ignore exit code errors
      */
     ignoreError?: boolean
+
+    /**
+     * Additional environment variables to set for the process.
+     */
+    env?: Record<string, string>
+
+    /**
+     * Inject the content of 'env' exclusively
+     */
+    isolateEnv?: boolean
 }
 
 interface ShellOutput {