ace-llm-providers-cli 0.32.3 → 0.33.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f6963c3de4deb7472734fbb346f1daa390982fb8442af80cae5f337dc093deeb
-  data.tar.gz: 3b09154bde9e388cf7ef4f52e2ef17c62f93fee36fa5f1fcea7f9a1533cf0521
+  metadata.gz: 7410921d087bbcfa634cec60984b2d578de92d92d84fc0730aa4cae054848b4b
+  data.tar.gz: '08b71084f7f13ac67941da17c20e84b34c1dd52673f037a929ac7e896e0c2310'
 SHA512:
-  metadata.gz: c8bd1e4a71471c32f71afc4d0912f446eeafc3eb1bfa73cb9db0f2db87c9fe978326d677c285a4b84aba75019b2655e663ee2de836fbc11c942ba9a3abcb6150
-  data.tar.gz: 98a449b5343719f31c42575f2db82881fd7b9efa6c4c7e0bbd3d4e5b8c4ffefe9739f5fcba09b8e79d8d8d417196c925cdc4b4e12c140a69e08547f71cb2f4ca
+  metadata.gz: 92613d00db61f1ad8e8a1e99285c478526b38dd998bd26355a543939ba2453da13f23df97479ae9a4186653fce4b31bd4383dc8fa23ff69c9a887ede90d8c339
+  data.tar.gz: 50d6485baf304a089b0311a3f0fe096fc90bb697a1b72b4c43f27d295a21631753e1e1d8be718c89722a71e51d3cbfa296926d49fe46875844d4dd53b4969e0a

data/.ace-defaults/llm/providers/pi.yml

@@ -3,11 +3,9 @@ class: Ace::LLM::Providers::CLI::PiClient
 gem: ace-llm-providers-cli
 models:
   # ZAI (default provider)
-  - zai/glm-4.7
-  - zai/glm-4.6
-  - zai/glm-4.5
-  - zai/glm-5.1
-  - zai/glm-5-turbo 
+  - zai/glm-5.3
+  - zai/glm-5.3-flash
+  - zai/glm-5.3-highspeed
   # Anthropic
   - anthropic/claude-opus-4-6
   - anthropic/claude-opus-4-5
@@ -26,10 +24,11 @@ models:
   - openai-codex/gpt-5.1
 aliases:
   global:
-    pi: pi:zai/glm-5.1
+    pi: pi:zai/glm-5.3-flash
   model:
-    glm: zai/glm-5.1
-    glmturbo: zai/glm-5-turbo
+    glm: zai/glm-5.3
+    glmflash: zai/glm-5.3-flash
+    glmhighspeed: zai/glm-5.3-highspeed
     gpt: openai-codex/gpt-5.4
 capabilities:
   - text_generation

data/CHANGELOG.md

@@ -7,6 +7,22 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+- Hardened `SafeCapture` process group termination for container PID namespaces where background processes reparent to PID 1, ensuring reliable cleanup of descendant processes without ESRCH failures.
+
+## [0.33.0] - 2026-09-04
+
+### Changed
+- Registered the live Pi/Z.AI GLM 5.3 family (`zai/glm-5.3`, `zai/glm-5.3-flash`, `zai/glm-5.3-highspeed`) in the shipped provider registry and removed the obsolete GLM 4.x/5.1/5-turbo entries and `glmturbo` alias.
+- Made `zai/glm-5.3-flash` the routine Pi default: the global `pi` alias and `PiClient` default model now target the flash model, with `glm`, `glmflash`, and `glmhighspeed` model aliases for the three live targets.
+
+### Technical
+- Added focused regression coverage for registry parity between project and shipped defaults, alias and full-slash-selector resolution without truncation, unknown-model fail-closed behavior, and `--no-fallback` preserving exactly one provider/model target.
+
+## [0.32.4] - 2026-09-02
+
+### Technical
+- Included in the coordinated all-package patch release preparation for ACE monorepo review.
 ## [0.32.3] - 2026-08-29
 
 ### Fixed

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

@@ -0,0 +1,68 @@
+# frozen_string_literal: true
+
+# Linux normally reparents orphaned grandchildren to PID 1. Container
+# entrypoints do not always reap them, leaving killed descendants as observable
+# zombies. This supervisor becomes a child subreaper before spawning the provider
+# command, so it owns and reaps the complete isolated command group before exit.
+module Ace
+  module LLM
+    module Providers
+      module CLI
+        module Molecules
+          module ProcessSupervisor
+            PR_SET_CHILD_SUBREAPER = 36
+
+            module_function
+
+            def run(command)
+              enable_child_subreaper
+              ready_fd = Integer(ENV.delete("ACE_SAFE_CAPTURE_READY_FD"))
+              command_pid = Process.spawn(*command, pgroup: true, ready_fd => :close)
+              command_pgid = command_pid
+              IO.for_fd(ready_fd).tap { |io| io.write("1"); io.close }
+
+              Signal.trap("TERM") { terminate_group(command_pgid) }
+
+              _pid, status = Process.wait2(command_pid)
+              terminate_group(command_pgid)
+              reap_group(command_pgid)
+
+              exit(status.exitstatus || 128 + status.termsig)
+            end
+
+            def enable_child_subreaper
+              previous_deprecated = Warning[:deprecated]
+              Warning[:deprecated] = false
+              require "fiddle"
+              Warning[:deprecated] = previous_deprecated
+
+              prctl = Fiddle::Function.new(
+                Fiddle::Handle::DEFAULT["prctl"],
+                [Fiddle::TYPE_INT, Fiddle::TYPE_LONG, Fiddle::TYPE_LONG, Fiddle::TYPE_LONG, Fiddle::TYPE_LONG],
+                Fiddle::TYPE_INT
+              )
+              raise "unable to become a child subreaper" unless prctl.call(PR_SET_CHILD_SUBREAPER, 1, 0, 0, 0).zero?
+            ensure
+              Warning[:deprecated] = previous_deprecated unless previous_deprecated.nil?
+            end
+
+            def terminate_group(pgid)
+              Process.kill("TERM", -pgid)
+              Process.kill("KILL", -pgid)
+            rescue Errno::ESRCH, Errno::EPERM
+              nil
+            end
+
+            def reap_group(pgid)
+              loop { Process.waitpid(-pgid) }
+            rescue Errno::ECHILD
+              nil
+            end
+          end
+        end
+      end
+    end
+  end
+end
+
+Ace::LLM::Providers::CLI::Molecules::ProcessSupervisor.run(ARGV) if $PROGRAM_NAME == __FILE__

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

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "open3"
+require "rbconfig"
 
 module Ace
   module LLM
@@ -13,8 +14,9 @@ module Ace
           # which causes "stream closed in another thread (IOError)" when the
           # timeout fires while Open3's internal reader threads hold pipe handles.
           #
-          # Uses Open3.popen3 + Process.kill so the child process is terminated
-          # directly — no thread interruption, no IOError.
+          # Uses Open3.popen3 and, on Linux, a dedicated child-subreaper supervisor
+          # so the complete command tree is terminated and reaped without thread
+          # interruption or closed-stream races.
           class SafeCapture
             # @param cmd [Array<String>] Command arguments
             # @param timeout [Integer] Timeout in seconds
@@ -23,7 +25,7 @@ module Ace
             # @param env [Hash, nil] Environment variables (merged with current env)
             # @param provider_name [String] Provider name for error messages
             # @param isolate_process_group [Boolean] Spawn subprocess in isolated process group
-            # @param cleanup_group_on_exit [Boolean] Best-effort cleanup of descendants on success
+            # @param cleanup_group_on_exit [Boolean] Clean up descendants on success
             # @return [Array(String, String, Process::Status)] [stdout, stderr, status]
             # @raise [Ace::LLM::ProviderError] on timeout
             def self.call(cmd, timeout:, stdin_data: nil, chdir: nil, env: nil, provider_name: "CLI",
@@ -35,9 +37,19 @@ module Ace
               opts[:pgroup] = true if isolate_process_group
 
               full_cmd = Array(command_prefix) + cmd
-              args = env ? [env, *full_cmd] : full_cmd
+              supervised = isolate_process_group && cleanup_group_on_exit && RUBY_PLATFORM.include?("linux")
+              full_cmd = supervisor_command(full_cmd) if supervised
+              ready_reader, ready_writer = IO.pipe if supervised
+              opts[ready_writer.fileno] = ready_writer if supervised
+              spawn_env = env&.dup || {}
+              spawn_env["ACE_SAFE_CAPTURE_READY_FD"] = ready_writer.fileno.to_s if supervised
+              args = spawn_env.empty? ? full_cmd : [spawn_env, *full_cmd]
 
               Open3.popen3(*args, **opts) do |stdin, stdout, stderr, wait_thr|
+                ready_writer&.close
+                ready_reader&.read(1)
+                ready_reader&.close
+
                 pid = wait_thr.pid
                 pgid = safe_getpgid(pid)
                 debug_log(provider_name, "spawn pid=#{pid} pgid=#{pgid || "n/a"}")
@@ -56,8 +68,13 @@ module Ace
 
                 unless wait_thr.join(normalized_timeout)
                   # Timeout: kill subprocess group (and descendants), then clean up
-                  terminate_subprocess_tree(pid: pid, pgid: pgid, provider_name: provider_name)
-                  wait_thr.join(5)
+                  terminate_subprocess_tree(
+                    pid: pid, pgid: pgid, provider_name: provider_name, supervised: supervised
+                  )
+                  unless wait_thr.join(5)
+                    terminate_group_or_pid("KILL", pid, pgid)
+                    wait_thr.join(5)
+                  end
 
                   stdout.close unless stdout.closed?
                   stderr.close unless stderr.closed?
@@ -70,12 +87,15 @@ module Ace
                 end
 
                 status = wait_thr.value
-                if isolate_process_group && cleanup_group_on_exit
+                if isolate_process_group && cleanup_group_on_exit && !supervised
                   terminate_descendants_after_success(pid: pid, pgid: pgid, provider_name: provider_name)
                 end
 
                 [out_reader.value, err_reader.value, status]
               end
+            ensure
+              ready_reader&.close unless ready_reader&.closed?
+              ready_writer&.close unless ready_writer&.closed?
             end
 
             class << self
@@ -99,21 +119,24 @@ module Ace
                 raise ArgumentError, "timeout must be a positive numeric value, got #{value.inspect}"
               end
 
-              def terminate_subprocess_tree(pid:, pgid:, provider_name:)
+              def supervisor_command(command)
+                supervisor = File.expand_path("process_supervisor.rb", __dir__)
+                [RbConfig.ruby, "-W0", supervisor, *command]
+              end
+
+              def terminate_subprocess_tree(pid:, pgid:, provider_name:, supervised: false)
                 debug_log(provider_name, "timeout cleanup pid=#{pid} pgid=#{pgid || "n/a"}")
                 terminate_group_or_pid("TERM", pid, pgid)
-                sleep(0.1)
-                terminate_group_or_pid("KILL", pid, pgid)
+                terminate_group_or_pid("KILL", pid, pgid) unless supervised
               end
 
               def terminate_descendants_after_success(pid:, pgid:, provider_name:)
                 return unless pgid
-                return unless group_alive?(pgid)
 
                 debug_log(provider_name, "post-exit cleanup pgid=#{pgid}")
-                terminate_group_or_pid("TERM", pid, pgid)
-                sleep(0.05)
-                terminate_group_or_pid("KILL", pid, pgid) if group_alive?(pgid)
+                safe_kill_group(pgid, "TERM")
+                safe_kill_group(pgid, "KILL")
+                reap_terminated_group(pgid)
               end
 
               def terminate_group_or_pid(signal, pid, pgid)
@@ -122,6 +145,13 @@ module Ace
                 else
                   Process.kill(signal, pid)
                 end
+                true
+              rescue Errno::ESRCH, Errno::EPERM
+                nil
+              end
+
+              def safe_kill_group(pgid, signal)
+                Process.kill(signal, -pgid)
               rescue Errno::ESRCH, Errno::EPERM
                 nil
               end
@@ -132,13 +162,10 @@ module Ace
                 nil
               end
 
-              def group_alive?(pgid)
-                Process.kill(0, -pgid)
-                true
-              rescue Errno::ESRCH
-                false
-              rescue Errno::EPERM
-                true
+              def reap_terminated_group(pgid)
+                loop { Process.waitpid(-pgid) }
+              rescue Errno::ECHILD
+                nil
               end
 
               def debug_log(provider_name, message)

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

@@ -27,7 +27,7 @@ module Ace
             "pi"
           end
 
-          DEFAULT_MODEL = "zai/glm-4.7"
+          DEFAULT_MODEL = "zai/glm-5.3-flash"
 
           def initialize(model: nil, **options)
             @model = model || DEFAULT_MODEL
@@ -67,7 +67,9 @@ module Ace
           # List available Pi models
           def list_models
             [
-              {id: "zai/glm-4.7", name: "GLM 4.7", description: "ZAI default model", context_size: 128_000},
+              {id: "zai/glm-5.3", name: "GLM 5.3", description: "ZAI flagship model", context_size: 1_000_000},
+              {id: "zai/glm-5.3-flash", name: "GLM 5.3 Flash", description: "ZAI routine default", context_size: 1_000_000},
+              {id: "zai/glm-5.3-highspeed", name: "GLM 5.3 Highspeed", description: "ZAI low-latency model", context_size: 1_000_000},
               {id: "anthropic/claude-opus-4-6", name: "Claude Opus 4.6", description: "Anthropic flagship", context_size: 200_000},
               {id: "anthropic/claude-sonnet-4-5", name: "Claude Sonnet 4.5", description: "Anthropic balanced", context_size: 200_000},
               {id: "anthropic/claude-haiku-4-5", name: "Claude Haiku 4.5", description: "Anthropic fast", context_size: 200_000},

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

@@ -4,7 +4,7 @@ module Ace
   module LLM
     module Providers
       module CLI
-        VERSION = "0.32.3"
+        VERSION = "0.33.0"
       end
     end
   end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: ace-llm-providers-cli
 version: !ruby/object:Gem::Version
-  version: 0.32.3
+  version: 0.33.0
 platform: ruby
 authors:
 - Michal Czyz
 bindir: exe
 cert_chain: []
-date: 2026-08-30 00:00:00.000000000 Z
+date: 2026-09-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ace-llm
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.38'
+        version: '0.39'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.38'
+        version: '0.39'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -111,6 +111,7 @@ files:
 - lib/ace/llm/providers/cli/codex_oai_client.rb
 - lib/ace/llm/providers/cli/gemini_client.rb
 - lib/ace/llm/providers/cli/molecules/health_checker.rb
+- lib/ace/llm/providers/cli/molecules/process_supervisor.rb
 - lib/ace/llm/providers/cli/molecules/safe_capture.rb
 - lib/ace/llm/providers/cli/molecules/session_finder.rb
 - lib/ace/llm/providers/cli/molecules/skill_name_reader.rb