handle executable paths with spaces in CLI arguments, close Effect-TS#5845 (Effect-TS#5853)

Author: Masty88

.changeset/cute-lizards-knock.md

@@ -0,0 +1,5 @@
+---
+"@effect/cli": patch
+---
+
+handle executable paths with spaces in CLI arguments

packages/cli/src/internal/cliApp.ts

@@ -102,7 +102,8 @@ export const run = dual<
                             InternalCommand.getHelp(e.command, config)
                           ),
                           execute,
-                          config
+                          config,
+                          args
                         )
                       )
                       : Option.none()
@@ -115,7 +116,7 @@ export const run = dual<
             })
           }
           case "BuiltIn": {
-            return handleBuiltInOption(self, executable, filteredArgs, directive.option, execute, config).pipe(
+            return handleBuiltInOption(self, executable, filteredArgs, directive.option, execute, config, args).pipe(
               Effect.catchSome((e) =>
                 InternalValidationError.isValidationError(e)
                   ? Option.some(Effect.zipRight(printDocs(e.error), Effect.fail(e)))
@@ -151,7 +152,8 @@ const handleBuiltInOption = <R, E, A>(
   args: ReadonlyArray<string>,
   builtIn: BuiltInOptions.BuiltInOptions,
   execute: (a: A) => Effect.Effect<void, E, R>,
-  config: CliConfig.CliConfig
+  config: CliConfig.CliConfig,
+  originalArgs: ReadonlyArray<string>
 ): Effect.Effect<
   void,
   E | ValidationError.ValidationError,
@@ -161,7 +163,10 @@ const handleBuiltInOption = <R, E, A>(
     case "SetLogLevel": {
       console.log("\n========== ACTUAL TEST ==========")
       console.log("executable:", executable)
-      console.log("Has spaces in paths?", executable.includes("Program Files") || executable.includes("Program Files (x86)"))
+      console.log(
+        "Has spaces in paths?",
+        executable.includes("Program Files") || executable.includes("Program Files (x86)")
+      )
 
       const nextArgs = executable.split(/\s+/)
       console.log("After split:", nextArgs.length, "elements")
@@ -176,7 +181,7 @@ const handleBuiltInOption = <R, E, A>(
         if (isLogLevelArg(args[i]) || isLogLevelArg(args[i - 1])) {
           continue
         }
-        nextArgs.push(args[i])
+        filteredArgs.push(args[i])
       }
 
       console.log("Final nextArgs length:", nextArgs.length)
@@ -285,10 +290,11 @@ const handleBuiltInOption = <R, E, A>(
             active: "yes",
             inactive: "no"
           }).pipe(Effect.flatMap((shouldRunCommand) => {
-            const finalArgs = pipe(
-              Arr.drop(args, 1),
-              Arr.prependAll(executable.split(/\s+/))
-            )
+            // Use first 2 elements from originalArgs (runtime + script) to preserve paths with spaces
+            // This mimics executable.split() behavior but without breaking Windows paths
+            const baseArgs = Arr.take(originalArgs, 2)
+            const wizardArgs = Arr.drop(args, 1)
+            const finalArgs = Arr.appendAll(baseArgs, wizardArgs)
             return shouldRunCommand
               ? Console.log().pipe(Effect.zipRight(run(self, finalArgs, execute)))
               : Effect.void

packages/cli/test/CliApp.test.ts

@@ -1,3 +1,4 @@
+import * as Args from "@effect/cli/Args"
 import type * as CliApp from "@effect/cli/CliApp"
 import * as CliConfig from "@effect/cli/CliConfig"
 import * as Command from "@effect/cli/Command"
@@ -113,5 +114,21 @@ describe("CliApp", () => {
         yield* cli(["node", "logging.js", "--log-level=debug"])
         expect(logLevel).toEqual(LogLevel.Debug)
       }).pipe(runEffect))
+
+    it("should handle paths with spaces when using --log-level", () =>
+      Effect.gen(function*() {
+        let executedValue: string | undefined = undefined
+        const cmd = Command.make("test", { value: Args.text() }, ({ value }) =>
+          Effect.sync(() => {
+            executedValue = value
+          }))
+        const cli = Command.run(cmd, {
+          name: "Test",
+          version: "1.0.0"
+        })
+        // Simulate Windows path with spaces (e.g., "C:\Program Files\nodejs\node.exe")
+        yield* cli(["C:\\Program Files\\node.exe", "C:\\My Scripts\\test.js", "--log-level", "info", "hello"])
+        expect(executedValue).toEqual("hello")
+      }).pipe(runEffect))
   })
 })