LaunchAppTool: tokenize command string so inline flags work

Author: AdamBien

zsmith/src/main/java/airhacks/zsmith/tools/control/LaunchAppTool.java

@@ -2,6 +2,7 @@
 
 import java.io.IOException;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.List;
 import java.util.concurrent.TimeUnit;
 
@@ -89,10 +90,16 @@ public String execute(JSONObject input) {
 
     List<String> buildCommandLine(String arguments) {
         var parts = new ArrayList<String>();
-        parts.add(this.command);
+        parts.addAll(tokenize(this.command));
         if (arguments != null && !arguments.isBlank()) {
-            parts.addAll(List.of(arguments.trim().split("\\s+")));
+            parts.addAll(tokenize(arguments));
         }
         return parts;
     }
+
+    static List<String> tokenize(String input) {
+        return Arrays.stream(input.trim().split(" "))
+                .filter(s -> !s.isEmpty())
+                .toList();
+    }
 }

zsmith/src/test/java/LaunchAppToolTest.java

@@ -27,4 +27,9 @@ void main() {
     var badResult = badTool.execute(new JSONObject().put("arguments", "test"));
     assert badResult.contains("Error") : "expected error but got: " + badResult;
 
+    // command string may carry inline flags — they must be tokenised and executed
+    var inlineFlagsTool = new LaunchAppTool("echo_with_flag", "echo with -n", "echo -n");
+    var inlineResult = inlineFlagsTool.execute(new JSONObject().put("arguments", "hello"));
+    assert "hello".equals(inlineResult) : "expected 'hello' (echo -n suppresses newline) but got: " + inlineResult;
+
 }

zsmith/version.txt

@@ -1 +1 @@
-2026.05.05.01
+2026.05.07.01