ace-llm 0.39.1 → 0.40.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aa342251e0d5d1ccc1ee95cb264d88fa552b27b42e8d88d65cc29e06ec18dc93
-  data.tar.gz: f24622f0ebc04e007f687af00faf50d99d0ced4875969f3d8775ada565515131
+  metadata.gz: 48321c690b49c8d136a85e6f131328bbb4a6cfc55e504d1467c56d4d2a554092
+  data.tar.gz: 0af124c46785c2278cd5b1d10b4e5fee3b7f67ff6e446cd28e9bbe091abea429
 SHA512:
-  metadata.gz: aa3f5270880edd4b6821117530ed34df37e2a65d6a7dafec625a103572b78efc346dfc083eb898ae3af5100beeaf63f97ecad51b93009121ab178e57a3771bc5
-  data.tar.gz: 2bf24d295ac1b61b1beee10c03c217598dc0f2df1f084c3195851d0bd55aa86e7941e587aefed0a077a52eb92480e5abb22dbd863b1e171c02aaad10d00f815d
+  metadata.gz: e13d32931c1e249cba948d8113e124fc7412462fbafd2d397f389c38effe2beadc8f7413ce7e196893680e9c27842cae15218bba6da1db5c3f5e9a085ec34aff
+  data.tar.gz: 73b5a3eab3ffeceab03a5c67631162ab245ff512e01262029b253a04bc7acad610926ce4e13c62944f618c9c03b19964ecf0af2ffd9adfec8d7fce23a5bca9f5

data/.ace-defaults/llm/presets/codex/ro.yml

@@ -1,5 +1,4 @@
 timeout: 600
 cli_args:
-  - --full-auto
   - --sandbox
   - read-only

data/.ace-defaults/llm/presets/codex/rw.yml

@@ -1,3 +1,4 @@
 timeout: 600
 cli_args:
-  - --full-auto
+  - --sandbox
+  - workspace-write

data/.ace-defaults/llm/presets/pi/ro.yml

@@ -1 +1,5 @@
 timeout: 600
+cli_args:
+  - --tools
+  - read,grep,find,ls
+  - --no-extensions

data/.ace-defaults/llm/thinking/pi/high.yml

@@ -0,0 +1,3 @@
+cli_args:
+  - --thinking
+  - high

data/.ace-defaults/llm/thinking/pi/low.yml

@@ -0,0 +1,3 @@
+cli_args:
+  - --thinking
+  - low

data/.ace-defaults/llm/thinking/pi/max.yml

@@ -0,0 +1,3 @@
+cli_args:
+  - --thinking
+  - max

data/.ace-defaults/llm/thinking/pi/medium.yml

@@ -0,0 +1,3 @@
+cli_args:
+  - --thinking
+  - medium

data/.ace-defaults/llm/thinking/pi/xhigh.yml

@@ -0,0 +1,3 @@
+cli_args:
+  - --thinking
+  - xhigh

data/CHANGELOG.md

@@ -7,6 +7,23 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.40.0] - 2026-09-24
+
+### Fixed
+
+- Update Codex CLI presets for the new `codex exec` interface: replace the
+  removed `--full-auto` flag with explicit `--sandbox read-only` (ro) and
+  `--sandbox workspace-write` (rw); `yolo` keeps
+  `--dangerously-bypass-approvals-and-sandbox`.
+- Preserve resolved provider, model, preset and reasoning metadata for the executed request, including fallback targets.
+- Accept `max` reasoning in selectors and Pi reasoning configuration.
+
+## [0.39.2] - 2026-09-13
+
+### Fixed
+
+- Preserve explicit Codex model targets and native errors without fallback substitution, including Ruby model overrides; retain fallback behavior for aliases, roles and other providers.
+
 
 ## [0.39.1] - 2026-09-02
 

data/README.md

@@ -35,7 +35,7 @@ Deterministic coverage lives in `test/fast/` and `test/feat/`. Scenario assets s
 
 ## How It Works
 
-1. Select a model -- by alias, `provider:model`, with a thinking level suffix (`:low`/`:medium`/`:high`), or an `@preset` -- and submit a prompt.
+1. Select a model -- by alias, `provider:model`, with a configured thinking level suffix (`:low`/`:medium`/`:high`/`:xhigh`/`:max`), or an `@preset` -- and submit a prompt.
 2. The provider router resolves the target through [ace-llm-providers-cli](../ace-llm-providers-cli) adapters, applying fallback and retry rules from the [config cascade](../ace-support-config).
 3. The response is returned as text, markdown, or JSON with optional token usage metadata.
 
@@ -43,7 +43,7 @@ Deterministic coverage lives in `test/fast/` and `test/feat/`. Scenario assets s
 
 **Switch providers with aliases** - use short names like `gflash`, `sonnet`, `opus` instead of full `provider:model` notation. Aliases resolve through versioned YAML in [`.ace-defaults/`](docs/usage.md).
 
-**Control reasoning depth** - append a thinking level (`codex:gpt:high`, `claude:sonnet:low`) to tune reasoning budgets. Supported CLI providers: `claude`, `codex` (levels: `low`, `medium`, `high`, `xhigh`).
+**Control reasoning depth** - append a thinking level (`codex:gpt:high`, `claude:sonnet:low`, `pi:glm5:max`) to tune reasoning budgets. Supported CLI providers: `claude`, `codex` (`low`, `medium`, `high`, `xhigh`) and `pi` (including `max`). Each provider/level pair requires a matching thinking configuration.
 
 **Run preset-driven prompts** - apply execution profiles with `@preset` or `--preset`. Built-in presets for CLI providers: `@ro` (read-only), `@rw` (read-write), `@yolo` (full autonomy). Supported by: `claude`, `codex`, `gemini`, `opencode`, `pi`, `agy`.
 

data/lib/ace/llm/molecules/fallback_orchestrator.rb

@@ -33,6 +33,7 @@ module Ace
         # @raise [Error] If all providers and retries exhausted
         def execute(primary_provider:, registry:)
           @start_time = Time.now
+          @primary_provider = primary_provider
           @visited_providers.clear
 
           # If fallback disabled, just execute with primary
@@ -84,6 +85,7 @@ module Ace
             client = get_client(provider_name, registry)
             return yield client, provider_name
           rescue => error
+            raise if error.is_a?(Ace::LLM::ConfigurationError) && provider_name != @primary_provider
             last_error = error
 
             # Handle the error - returns :retry or :stop_and_fallback

data/lib/ace/llm/molecules/provider_model_parser.rb

@@ -10,10 +10,11 @@ module Ace
       # ProviderModelParser handles parsing and validation of provider:model syntax
       # for the unified LLM query interface.
       class ProviderModelParser
-        THINKING_LEVELS = %w[low medium high xhigh].freeze
+        THINKING_LEVELS = %w[low medium high xhigh max].freeze
 
         # Result object for parsed provider:model combinations.
-        ParseResult = Struct.new(:provider, :model, :preset, :thinking_level, :valid, :error, :original_input, :role_fallbacks) do
+        ParseResult = Struct.new(:provider, :model, :preset, :thinking_level, :valid, :error,
+          :original_input, :role_fallbacks, :explicit_model) do
           def valid?
             valid
           end
@@ -109,6 +110,11 @@ module Ace
           provider_target, preset_name, preset_error = split_preset_suffix(original_input)
           return create_error_result(original_input, preset_error) if preset_error
 
+          raw_provider, raw_model = provider_target.split(":", 2)
+          raw_model, = split_thinking_suffix(raw_model) if raw_model
+          raw_selector = "#{raw_provider}:#{raw_model}"
+          explicit_model = !raw_model.to_s.empty? && @alias_resolver.resolve(raw_selector) == raw_selector
+
           provider_target = @alias_resolver.resolve(provider_target).to_s.strip
           return create_error_result(original_input, "Invalid target: provider/model portion cannot be empty") if provider_target.empty?
 
@@ -146,7 +152,8 @@ module Ace
             return create_error_result(original_input, "Invalid target: resolved model cannot be empty")
           end
 
-          ParseResult.new(resolved_provider, model, preset_name, thinking_level, true, nil, original_input)
+          ParseResult.new(resolved_provider, model, preset_name, thinking_level, true, nil,
+            original_input, nil, explicit_model)
         end
 
         def build_role_fallbacks(candidates, caller_preset, caller_thinking)

data/lib/ace/llm/molecules/role_resolver.rb

@@ -8,7 +8,7 @@ module Ace
     module Molecules
       # RoleResolver maps role names to an available concrete selector.
       class RoleResolver
-        THINKING_LEVELS = %w[low medium high xhigh].freeze
+        THINKING_LEVELS = %w[low medium high xhigh max].freeze
 
         def initialize(registry: nil, configuration: nil)
           @registry = registry || ClientRegistry.new

data/lib/ace/llm/molecules/thinking_level_loader.rb

@@ -8,7 +8,7 @@ module Ace
     module Molecules
       # Loads provider-scoped thinking-level overrides from llm/thinking/*.yml.
       class ThinkingLevelLoader
-        ALLOWED_LEVELS = %w[low medium high xhigh].freeze
+        ALLOWED_LEVELS = %w[low medium high xhigh max].freeze
 
         class << self
           def load_for_provider(provider, level)

data/lib/ace/llm/query_interface.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "shellwords"
+
 require_relative "molecules/client_registry"
 require_relative "molecules/provider_model_parser"
 require_relative "molecules/preset_loader"
@@ -90,6 +92,9 @@ module Ace
         end
 
         fallback_config = load_fallback_config(fallback, fallback_providers, parser: parser)
+        if parse_result.provider == "codex" && (model || parse_result.explicit_model)
+          fallback_config = fallback_config.merge(enabled: false)
+        end
         timeout_value = first_non_nil(timeout, execution_overrides["timeout"], Molecules::ConfigLoader.get("llm.timeout"), 120)
         resolved_timeout = normalize_timeout(timeout_value)
         parser_for_options = Molecules::ProviderModelParser.new(registry: registry)
@@ -142,10 +147,12 @@ module Ace
 
         result = {
           text: text_content,
-          model: final_model,
-          provider: parse_result.provider,
-          preset: resolved_preset,
-          thinking_level: parse_result.thinking_level,
+          model: response.dig(:execution, :model) || final_model,
+          provider: response.dig(:execution, :provider) || parse_result.provider,
+          preset: response.fetch(:execution, {}).fetch(:preset, resolved_preset),
+          thinking_level: response.fetch(:execution, {}).fetch(:thinking_level, parse_result.thinking_level),
+          execution: response[:execution],
+          requested_selector: provider_model,
           usage: response[:usage],
           metadata: response[:metadata]
         }
@@ -361,7 +368,8 @@ module Ace
         registry:, fallback_config:, timeout:, debug:, role_fallbacks: nil, preset: nil, thinking_level: nil, option_builder:)
         if fallback_config.disabled?
           client = registry.get_client(provider, model: model, timeout: timeout)
-          return client.generate(messages, **generation_opts)
+          response = client.generate(messages, **generation_opts)
+          return response.merge(execution: execution_identity(provider, model, preset, thinking_level, response, generation_opts))
         end
 
         primary_provider_string = model ? "#{provider}:#{model}" : provider
@@ -388,9 +396,60 @@ module Ace
           else
             option_builder.call(target_selector)
           end
-          client.generate(messages, **opts)
+          parsed = Molecules::ProviderModelParser.new(registry: registry).parse(target_selector)
+          raise ConfigurationError, parsed.error unless parsed.valid?
+          primary = target_selector.to_s == primary_provider_string
+          response = client.generate(messages, **opts)
+          response.merge(execution: execution_identity(
+            parsed.provider, parsed.model,
+            primary ? preset : (parsed.preset || preset),
+            primary ? thinking_level : parsed.thinking_level,
+            response,
+            opts
+          ))
+        end
+      end
+
+      # Transport identity records what ACE executed, not a provider attestation.
+      def self.execution_identity(provider, model, preset, thinking_level, response, options)
+        model ||= response.dig(:metadata, :model) || response.dig(:metadata, "model")
+        finish_reason = response.dig(:metadata, :finish_reason) || response.dig(:metadata, "finish_reason") ||
+          response.dig(:metadata, :stop_reason) || response.dig(:metadata, "stop_reason")
+        status = if Ace::LLM::SUCCESSFUL_FINISH_REASONS.include?(finish_reason.to_s.downcase)
+          "succeeded"
+        else
+          "incomplete"
+        end
+        {provider: provider, model: model, preset: preset, thinking_level: effective_thinking_level(provider, thinking_level, options),
+         identity_source: "resolved_request", status: status}
+      end
+      private_class_method :execution_identity
+
+      def self.effective_thinking_level(provider, default, options)
+        return default unless %w[pi codex].include?(provider)
+
+        args = Array(options[:cli_args]).flat_map { |arg| Shellwords.split(arg.to_s) }
+        effective = default
+        args.each_with_index do |arg, index|
+          if provider == "pi"
+            candidate = if arg == "--thinking"
+              args[index + 1]
+            elsif arg.start_with?("--thinking=")
+              arg.delete_prefix("--thinking=")
+            end
+            effective = candidate if %w[off minimal low medium high xhigh max].include?(candidate)
+          else
+            config = if %w[-c --config].include?(arg)
+              args[index + 1]
+            elsif arg.match?(/\A(?:-c|--config)=?model_reasoning_effort=/)
+              arg.sub(/\A(?:-c|--config)=?/, "")
+            end
+            effective = config.split("=", 2).last if config&.start_with?("model_reasoning_effort=")
+          end
         end
+        effective
       end
+      private_class_method :effective_thinking_level
 
       def self.extract_text_content(response)
         if response[:text]

data/lib/ace/llm/version.rb

@@ -2,6 +2,6 @@
 
 module Ace
   module LLM
-    VERSION = "0.39.1"
+    VERSION = "0.40.0"
   end
 end

data/lib/ace/llm.rb

@@ -55,6 +55,8 @@ module Ace
     class ConfigurationError < Error; end
     class AuthenticationError < Error; end
 
+    SUCCESSFUL_FINISH_REASONS = %w[success stop end_turn complete completed stop_sequence].freeze
+
     # Define module namespaces
     module Atoms; end
     module Molecules; end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: ace-llm
 version: !ruby/object:Gem::Version
-  version: 0.39.1
+  version: 0.40.0
 platform: ruby
 authors:
 - Michal Czyz
 bindir: exe
 cert_chain: []
-date: 2026-09-05 00:00:00.000000000 Z
+date: 2026-09-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ace-support-cli
@@ -266,6 +266,11 @@ files:
 - ".ace-defaults/llm/thinking/codex/low.yml"
 - ".ace-defaults/llm/thinking/codex/medium.yml"
 - ".ace-defaults/llm/thinking/codex/xhigh.yml"
+- ".ace-defaults/llm/thinking/pi/high.yml"
+- ".ace-defaults/llm/thinking/pi/low.yml"
+- ".ace-defaults/llm/thinking/pi/max.yml"
+- ".ace-defaults/llm/thinking/pi/medium.yml"
+- ".ace-defaults/llm/thinking/pi/xhigh.yml"
 - ".ace-defaults/nav/protocols/guide-sources/ace-llm.yml"
 - CHANGELOG.md
 - LICENSE