ace-llm-providers-cli 0.36.1 → 0.36.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b9149be64f8f384f5636b393bfa3cf12f81aa9488e486225ed2966c731ef2a0f
-  data.tar.gz: 1fe4000a4a8000d49e04796a2eee4030b588cc3ef06babb2080b2e7feb66dedf
+  metadata.gz: 31bd7e17459ce8d8e8c1bc358819d49fbd919fa5c5fefbfab977d1a4993983c6
+  data.tar.gz: 47a14679feaea4018e4a437a36ddfb176a9392909db2c4eb68a49e7773927348
 SHA512:
-  metadata.gz: 7fc14db03c5a90eae90ec8855bb7bd801d5a75c5174488335b1201bd10d767c4f202eb192008b6a932c3c717e4913df2ae2cdd8650a16ebfd1921d3d50b57687
-  data.tar.gz: 1f2b33eb5461854fb293d583eebf87d723c0aa2ecd560995a6b05e56f00da0ba8570fedcd9531c853ccd24b055f64129721961d87988fd599378734c18507eca
+  metadata.gz: 8368b8e11eb5e0afa9e9e4fef56696bd16024b0ded23fca5e54689d86694a1aa045f6df1d701b1cadbe8281ccb04b992d861884bf5b4c1805da8d551979ecb48
+  data.tar.gz: e5353cb3b9734d28b9003d6017b3a31fbd61c8dd36d781c97b211f6e795323551b1ec0226ae527c4ca788df873983b001850afe0f159131605475700566901fe

data/CHANGELOG.md

@@ -7,6 +7,20 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.36.2] - 2026-10-16
+
+### Fixed
+
+- Terminate coding-agent subprocess groups when a caller interrupts a capture, before Open3 waits for the child.
+
+- `SafeCapture`: repair the `readiness.nil.` typo (missing `?`) introduced in
+  0.36.0 that raised `NoMethodError` on every successful supervised CLI
+  capture before any model call, breaking all CLI-delegating providers
+  (codex, agy, claude, pi). Regression tests pin readiness semantics
+  (String/nil/unsupervised); signal-classification tests now exercise an
+  unsupervised direct child, since the Linux supervisor converts child
+  signals to exit status by design.
+
 ## [0.36.1] - 2026-10-01
 
 ### Technical

data/lib/ace/llm/providers/cli/molecules/safe_capture.rb

@@ -97,7 +97,7 @@ module Ace
                 # StandardError describing a supervisor-reported child spawn
                 # failure, or nil when the child group started normally.
                 def child_spawn_failure
-                  return nil if supervised != true || readiness.nil.
+                  return nil if supervised != true || readiness.nil?
 
                   message = readiness.to_s
                   return nil unless message.start_with?("S")
@@ -203,6 +203,22 @@ module Ace
                     deadline_seconds: normalized_timeout,
                     elapsed_seconds: elapsed_since(started_at)
                   )
+                ensure
+                  # A caller may cancel a worker thread while it waits on a coding
+                  # agent. Clean up before Open3's block unwinds (which otherwise
+                  # waits for the still-running child). The capture boundary owns
+                  # the process group, so consumers must not duplicate this policy.
+                  if pid && (wait_thr.alive? || $!)
+                    terminate_subprocess_tree(
+                      pid: pid, pgid: pgid, provider_name: provider_name, supervised: spawn_result.supervised?
+                    )
+                    unless wait_thr.join(5)
+                      terminate_group_or_pid("KILL", pid, pgid)
+                      wait_thr.join(5)
+                    end
+                  end
+                  [stdin, stdout, stderr].each { |stream| stream.close unless stream.closed? }
+                  [out_reader, err_reader].compact.each { |reader| reader.join(1) }
                 end
               end
 
@@ -296,7 +312,7 @@ module Ace
               end
 
               def terminate_subprocess_tree(pid:, pgid:, provider_name:, supervised: false)
-                debug_log(provider_name, "timeout cleanup pid=#{pid} pgid=#{pgid || "n/a"}")
+                debug_log(provider_name, "capture cleanup pid=#{pid} pgid=#{pgid || "n/a"}")
                 terminate_group_or_pid("TERM", pid, pgid)
                 terminate_group_or_pid("KILL", pid, pgid) unless supervised
               end

data/lib/ace/llm/providers/cli/version.rb

@@ -4,7 +4,7 @@ module Ace
   module LLM
     module Providers
       module CLI
-        VERSION = "0.36.1"
+        VERSION = "0.36.2"
       end
     end
   end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: ace-llm-providers-cli
 version: !ruby/object:Gem::Version
-  version: 0.36.1
+  version: 0.36.2
 platform: ruby
 authors:
 - Michal Czyz
 bindir: exe
 cert_chain: []
-date: 2026-10-01 00:00:00.000000000 Z
+date: 2026-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ace-llm