aidp 0.49.20 → 0.49.22

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3bcae0816a7d6d049e039d8670fd6b8e21e7e9228af2d0b996853c38f39b2bd6
-  data.tar.gz: f7aeae62f6ce57552d31885a69461ea00c50bf9d4f30056a3f76ea563b174eb6
+  metadata.gz: 6cb8dcb8dbf9eb841ad69223a465c30329b9e9ed29853ddb168e237a6f3bebfb
+  data.tar.gz: 39d511353116c51196a464c85775ea9227f6e40ccd5028a7174fe048d7adc567
 SHA512:
-  metadata.gz: 81497e0b2050c4e9d06e45a0251123e792993694b64e5015b6a0f907ab1dc22bd576266c11a2254afed433285570cf459152d357c01ea345e0cc99a7279f7d38
-  data.tar.gz: 873e5c878b464542b6a533222db9842bf7303e06d29d868b020588079c6eb957ff51b4b2238b027cf15ce2579588bba8ad8ef4f841f47305eb5cfd71a5a1dfed
+  metadata.gz: cb5c58bbf3c79069585f2341f9b4839d2d03489f6a50bebba5b05d971f6deb4184050d987b6bc07547e79938de204ca44a45dfe112622b12b8d42f488dcc28d7
+  data.tar.gz: 555d5538815206f1ce3fd104a6a5f8c93b243320993fb73d0fdc9ac05d1f01e59197886d2f806b5a02dce45d9dd705305107f3ef388086aed987c1cf646cf1c4

data/lib/aidp/shell_executor.rb

@@ -126,6 +126,11 @@ module Aidp
     # When suppress_output is true, output is redirected to /dev/null
     # unless explicit out:/err: options are provided.
     #
+    # Commands are always executed in argument form so a shell never
+    # interprets any value. A leading environment hash is forwarded
+    # unchanged; otherwise the [program, argv0] form forces direct
+    # execution even when a single command string is supplied.
+    #
     # @param args [Array] Arguments passed to Kernel.system
     # @param opts [Hash] Options passed to Kernel.system
     # @return [Boolean, nil] Same as Kernel.system
@@ -133,7 +138,19 @@ module Aidp
       if self.class.suppress_output && !opts.key?(:out) && !opts.key?(:err)
         opts = opts.merge(out: File::NULL, err: File::NULL)
       end
-      Kernel.system(*args, **opts)
+
+      if args.first.is_a?(Hash)
+        environment, *command = args
+        program, *rest = command.map(&:to_s)
+        raise ArgumentError, "command must not be blank" if program.nil? || program.empty?
+
+        Kernel.system(environment, [program, program], *rest, **opts)
+      else
+        program, *rest = args.map(&:to_s)
+        raise ArgumentError, "command must not be blank" if program.nil? || program.empty?
+
+        Kernel.system([program, program], *rest, **opts)
+      end
     end
   end
 end

data/lib/aidp/strategy_execution/cli_protocol.rb

@@ -36,7 +36,13 @@ module Aidp
             task_id: request[:task]&.dig(:id),
             artifact_dir: artifact_dir)
 
-          stdout, stderr, status = Open3.capture3(*parts, stdin_data: JSON.generate(payload), chdir: @project_dir)
+          program, *args = parts.map(&:to_s)
+          stdout, stderr, status = Open3.capture3(
+            [program, program],
+            *args,
+            stdin_data: JSON.generate(payload),
+            chdir: @project_dir
+          )
           unless status.success?
             Aidp.log_error("cli_protocol", "execution_failed",
               role: role,

data/lib/aidp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Aidp
-  VERSION = "0.49.20"
+  VERSION = "0.49.22"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aidp
 version: !ruby/object:Gem::Version
-  version: 0.49.20
+  version: 0.49.22
 platform: ruby
 authors:
 - Bart Agapinan