ace-llm-providers-cli 0.36.0 → 0.36.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fd7b7ee1d350ec4ea77298f5ecd0f137537897291ce0ba1f13949357fb1a0d94
-  data.tar.gz: 2ee0526ca547c242ec6346f7db4e1816a31b652ba8c06d8ba98ddff0cf6acfe0
+  metadata.gz: 31bd7e17459ce8d8e8c1bc358819d49fbd919fa5c5fefbfab977d1a4993983c6
+  data.tar.gz: 47a14679feaea4018e4a437a36ddfb176a9392909db2c4eb68a49e7773927348
 SHA512:
-  metadata.gz: 60660511b2ef85e79054367ccb21560663cdca66714d6c3f501ad0d04e917f058330b4ba8e93c285133832e91edf949a1cd80c6bf348c93ec2657d4e5fd1b044
-  data.tar.gz: 67f8baa381e615f7c53eba0df32db3fab86e417f0ef048f99929c8a12f3a9e83f93e9a31df8470e61a8fc35b6e8c7e17681f9c2c905f21dcccf2967ca0ceca59
+  metadata.gz: 8368b8e11eb5e0afa9e9e4fef56696bd16024b0ded23fca5e54689d86694a1aa045f6df1d701b1cadbe8281ccb04b992d861884bf5b4c1805da8d551979ecb48
+  data.tar.gz: e5353cb3b9734d28b9003d6017b3a31fbd61c8dd36d781c97b211f6e795323551b1ec0226ae527c4ca788df873983b001850afe0f159131605475700566901fe

data/CHANGELOG.md

@@ -7,6 +7,25 @@ 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
+- Observe descendant exit with a bounded monotonic wait in cleanup tests, retaining the process-absence predicate and adding a live-child negative control.
+
 ## [0.36.0] - 2026-09-29
 
 ### Added

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
 
@@ -243,7 +259,9 @@ module Ace
                   io.close unless io.closed?
                   reader.join(1)
                   reader.kill if reader.alive?
-                  buffer.dup
+                  # The reader may have been killed mid-chunk; finalize the
+                  # partial buffer with the same UTF-8 scrubbing contract.
+                  buffer.dup.force_encoding(Encoding::UTF_8).scrub!
                 end
               end
 
@@ -261,6 +279,12 @@ module Ace
                   end
                   buffer << chunk
                 end
+                # Pipes yield raw bytes; CLI output can contain non-UTF-8
+                # fragments. Finalize as valid UTF-8 (invalid bytes become
+                # replacement characters) so downstream prompt concatenation
+                # cannot raise Encoding::CompatibilityError.
+                buffer.force_encoding(Encoding::UTF_8)
+                buffer.scrub!
                 buffer
               rescue IOError, SystemCallError
                 buffer
@@ -288,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.0"
+        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.0
+  version: 0.36.2
 platform: ruby
 authors:
 - Michal Czyz
 bindir: exe
 cert_chain: []
-date: 2026-09-30 00:00:00.000000000 Z
+date: 2026-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ace-llm