legion-llm 0.9.28 → 0.9.30

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 178958a3403cbac0fad20d83f2726914d420137db2a1c340c33c4c7305457fcd
-  data.tar.gz: df951b9e05e0a0bfaff3701b6a3c5bd8452edea2298fe91e6a98165ce96961d1
+  metadata.gz: 221d8482faa90a3fa8e4bbefe90999838a56b96bc34d6506956f6b8ac90ab290
+  data.tar.gz: a3c3ce4cde046e8cd1af18874e04da1698d4527549fc70df274d14c0830f3803
 SHA512:
-  metadata.gz: 7ff1622a50cdafc4d09577e8dc5f9e90632f7f93e528f41c1b630ce5daeeeceab6c91cbf5d8ec92183e0b6f27c17a62d09934b6bb0a413da9577cea5a47c942f
-  data.tar.gz: 49651eb56bbe046223674a626ce4339363ba10550793c1d95ca46d4bf2c4b7f2eb430397b7f0d91368791cecb683ae2919f8627edb45d88d5c847f4d0cb4ee12
+  metadata.gz: fd2fdd7bce06cd6efa9a8b583072b947f14ffdaa5a1898b65f2027da8693c79bdd56b3553d04f6f6329f2f3a9d71c0699947f371aa5715708fd538a144819363
+  data.tar.gz: 7c3c36ca8b7db0075b18ea656e3c13c9235d873362d8febbe663c96eb07a1fa562a16d754f1bfed42d909698b466c303d7395e79caa0093ad1d1008c64389f69

data/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Legion LLM Changelog
 
+## [0.9.30] - 2026-05-16
+
+### Fixed
+- Tools: API-submitted client tools are no longer advertised to providers unless callers explicitly opt into client-tool passthrough, preventing incomplete tool calls from ending streaming responses.
+
+## [0.9.29] - 2026-05-16
+
+### Added
+- Routing: generated discovery rules now expose `:tools` capability rules for tool-capable models and normalize provider aliases such as `function_calling`/`functions`.
+
+### Changed
+- Routing: automatic routing honors top-level `llm.tier_order` before routing-specific tier priority settings.
+- Routing: stream requests with injected native tools now require both `streaming` and `tools` model capabilities before selecting a target.
+
+### Fixed
+- Router: required model capabilities filter out non-tool-capable candidates instead of selecting a local model that later rejects tool payloads.
+- Executor: streaming provider calls now use the escalation chain, so provider errors like "does not support tools" can move to the next routed model.
+- Executor: synthetic routing requirements no longer make model-only or explicit-provider requests bypass provider inference or registry defaults.
+
 ## [0.9.28] - 2026-05-15
 
 ### Added

data/lib/legion/llm/api/anthropic/messages.rb

@@ -149,7 +149,7 @@ module Legion
                   name:        tname,
                   description: tdesc,
                   parameters:  tschema,
-                  source:      { type: :client, executable: false }
+                  source:      { type: :client, executable: true }
                 )
               rescue StandardError => e
                 log.warn("[llm][api][anthropic][messages] build_tool_classes failed name=#{tname} error=#{e.message}")

data/lib/legion/llm/api/native/inference.rb

@@ -28,6 +28,7 @@ module Legion
               conversation_id = body[:conversation_id]
               request_id      = body[:request_id] || SecureRandom.uuid
               include_thinking = body[:include_thinking] == true
+              client_tool_passthrough = body[:client_tool_passthrough] == true
 
               unless messages.is_a?(Array)
                 halt 400, { 'Content-Type' => 'application/json' },
@@ -103,6 +104,9 @@ module Legion
 
               extra = {}
               extra[:tier] = tier.to_sym if tier
+              metadata = { requested_tools: requested_tools }
+              metadata[:client_tool_passthrough] = true if client_tool_passthrough
+              metadata[:client_tool_request_count] = tools.size if tools.any?
 
               pipeline_request = Legion::LLM::Inference::Request.build(
                 id:              request_id,
@@ -112,7 +116,7 @@ module Legion
                 tools:           tool_declarations,
                 caller:          effective_caller,
                 conversation_id: conversation_id,
-                metadata:        { requested_tools: requested_tools },
+                metadata:        metadata,
                 stream:          streaming,
                 cache:           { strategy: :default, cacheable: true },
                 extra:           extra

data/lib/legion/llm/api/native/tiers.rb

@@ -160,8 +160,12 @@ module Legion
           end
 
           def self.tier_priority
+            return Legion::LLM::Router.tier_priority if defined?(Legion::LLM::Router)
+
             routing_config = Legion::LLM::Settings.value(:routing) || {}
-            Array(routing_config[:tier_priority] || %w[local fleet openai_compat cloud frontier])
+            top_level = Legion::LLM::Settings.value(:tier_order, default: nil)
+            Array(top_level || routing_config[:tier_order] || routing_config[:tier_priority] ||
+                  %w[local direct fleet openai_compat cloud frontier])
           end
 
           def self.privacy_mode?

data/lib/legion/llm/api/openai/chat_completions.rb

@@ -123,7 +123,7 @@ module Legion
                 name:        t[:name].to_s,
                 description: t[:description].to_s,
                 parameters:  t[:parameters] || {},
-                source:      { type: :client, executable: false }
+                source:      { type: :client, executable: true }
               )
             rescue StandardError => e
               tool_name = t.is_a?(Hash) ? t[:name] : nil

data/lib/legion/llm/discovery/rule_generator.rb

@@ -27,6 +27,14 @@ module Legion
         }.freeze
 
         DEFAULT_TIER_PRIORITY = %i[local direct fleet openai_compat cloud frontier].freeze
+        CAPABILITY_ALIASES = {
+          function_calling: :tools,
+          functions:        :tools,
+          tool:             :tools,
+          tool_use:         :tools,
+          stream:           :streaming,
+          stream_chat:      :streaming
+        }.freeze
 
         module_function
 
@@ -52,7 +60,10 @@ module Legion
                 capability = embedding_model?(model_data) ? :embed : :chat
                 priority = tier_weight(model_tier) - order
                 rules << build_rule(provider, instance_id, model_data, capability, model_tier, priority)
-                rules << build_rule(provider, instance_id, model_data, :stream, model_tier, priority) if capability == :chat
+                if capability == :chat
+                  rules << build_rule(provider, instance_id, model_data, :stream, model_tier, priority) if supports_streaming?(model_data)
+                  rules << build_rule(provider, instance_id, model_data, :tools, model_tier, priority) if supports_tools?(model_data)
+                end
                 order += 1
               end
             end
@@ -125,11 +136,39 @@ module Legion
           return nil unless model_data.is_a?(Hash)
 
           caps = model_data[:capabilities] || model_data['capabilities']
-          return caps if caps.is_a?(Array) && caps.any?
+          normalized = normalize_capabilities(caps)
+          return normalized if normalized.any?
 
           nil
         end
 
+        def supports_streaming?(model_data)
+          capabilities = extract_capabilities(model_data)
+          return true if capabilities.nil?
+
+          capabilities.include?(:streaming)
+        end
+
+        def supports_tools?(model_data)
+          capabilities = extract_capabilities(model_data)
+          return false if capabilities.nil?
+
+          capabilities.include?(:tools)
+        end
+
+        def normalize_capabilities(capabilities)
+          Array(capabilities).compact.each_with_object([]) do |capability, normalized|
+            next unless capability.respond_to?(:to_s)
+
+            capability_sym = capability.to_s.downcase.strip.to_sym
+            next if capability_sym.to_s.empty?
+
+            normalized << capability_sym
+            alias_sym = CAPABILITY_ALIASES[capability_sym]
+            normalized << alias_sym if alias_sym
+          end.uniq
+        end
+
         def extract_field(model_data, field)
           return nil unless model_data.is_a?(Hash)
 
@@ -145,7 +184,9 @@ module Legion
         end
 
         def tier_priority
-          configured = Legion::LLM::Settings.value(:routing, :tier_priority, default: DEFAULT_TIER_PRIORITY)
+          configured = Legion::LLM::Settings.value(:tier_order, default: nil)
+          configured = Legion::LLM::Settings.value(:routing, :tier_order, default: nil) if blank_array?(configured)
+          configured = Legion::LLM::Settings.value(:routing, :tier_priority, default: DEFAULT_TIER_PRIORITY) if blank_array?(configured)
           normalized = Array(configured).filter_map do |tier|
             tier.to_sym if tier.respond_to?(:to_sym)
           end
@@ -156,6 +197,10 @@ module Legion
           DEFAULT_TIER_PRIORITY
         end
 
+        def blank_array?(value)
+          Array(value).empty?
+        end
+
         def extension_providers
           ext = Legion::Settings[:extensions]
           return ext[:llm] if ext.is_a?(Hash) && ext[:llm].is_a?(Hash)

data/lib/legion/llm/inference/executor.rb

@@ -376,6 +376,7 @@ module Legion
 
         def routing_request_state
           routing_explicit = @request.extra[:routing_explicit]
+          request_intent = @request.extra[:intent]
           instance = @request.routing[:instance] || @request.routing[:instance_id] || @request.routing[:provider_instance]
           tier = @request.extra[:tier]
           {
@@ -385,7 +386,8 @@ module Legion
             offering_id:       @request.routing[:offering_id] || @request.routing[:id],
             offering_metadata: normalize_offering_metadata(@request.routing[:offering_metadata] ||
                                                            @request.routing[:offering]),
-            intent:            @request.extra[:intent],
+            intent:            routing_intent_for_request(request_intent),
+            intent_explicit:   routing_intent_present?(request_intent),
             tier:              tier,
             auto_route:        @request.extra[:auto_route],
             provider_explicit: routing_field_explicit?(routing_explicit, :provider, @request.routing[:provider]),
@@ -394,6 +396,65 @@ module Legion
           }
         end
 
+        def routing_intent_present?(intent)
+          intent.is_a?(Hash) && intent.any?
+        end
+
+        def routing_intent_for_request(intent)
+          normalized = if intent.is_a?(Hash)
+                         intent.transform_keys { |key| key.respond_to?(:to_sym) ? key.to_sym : key }
+                       else
+                         {}
+                       end
+          required = normalize_required_capabilities(
+            normalized.delete(:required_capabilities) || normalized.delete(:requires)
+          )
+
+          if @request.stream == true
+            normalized[:capability] = :stream if stream_routable_capability?(normalized[:capability])
+            required << :streaming
+          end
+
+          required << :tools if native_tools_requested_for_routing?
+          normalized[:required_capabilities] = required.uniq if required.any?
+          normalized
+        end
+
+        def stream_routable_capability?(capability)
+          capability.nil? || %i[chat completion stream].include?(capability.to_s.downcase.to_sym)
+        end
+
+        def native_tools_requested_for_routing?
+          Array(@request.tools).any? ||
+            requested_deferred_tool_names.any? ||
+            @triggered_tools.any? ||
+            Tools::Special.pinned_definitions.any?
+        rescue StandardError => e
+          handle_exception(e, level: :warn, handled: true, operation: 'llm.pipeline.routing_tools_required')
+          false
+        end
+
+        def normalize_required_capabilities(capabilities)
+          aliases = {
+            function_calling: :tools,
+            functions:        :tools,
+            tool:             :tools,
+            tool_use:         :tools,
+            stream:           :streaming,
+            stream_chat:      :streaming
+          }
+          Array(capabilities).compact.each_with_object([]) do |capability, normalized|
+            next unless capability.respond_to?(:to_s)
+
+            capability_sym = capability.to_s.downcase.strip.to_sym
+            next if capability_sym.to_s.empty?
+
+            normalized << capability_sym
+            alias_sym = aliases[capability_sym]
+            normalized << alias_sym if alias_sym
+          end.uniq
+        end
+
         def apply_proactive_tier_assignment(state)
           # Forced assignments carry security/privacy constraints and override
           # caller-supplied tier/intent. Advisory assignments only fill blanks.
@@ -416,7 +477,7 @@ module Legion
 
           explicit_route = state[:provider_explicit] || state[:instance_explicit] || state[:tier_explicit]
           auto_route = state[:auto_route] == true
-          intent_route = state[:intent] && Router.routing_enabled?
+          intent_route = state[:intent_explicit] && state[:intent] && Router.routing_enabled?
           return state unless explicit_route || auto_route || intent_route
 
           resolution = routing_resolution_for(state)
@@ -426,7 +487,7 @@ module Legion
         end
 
         def routing_resolution_for(state)
-          if state[:auto_route] == true || (state[:intent] && pipeline_escalation_enabled?)
+          if state[:auto_route] == true || (state[:intent_explicit] && state[:intent] && pipeline_escalation_enabled?)
             @escalation_chain = Router.resolve_chain(
               intent:                 state[:intent],
               tier:                   state[:tier],
@@ -527,13 +588,14 @@ module Legion
           end
         end
 
-        def run_provider_call_with_escalation
+        def run_provider_call_with_escalation(stream_block: nil)
           @escalation_chain ||= build_default_escalation_chain
           chain = @escalation_chain
           threshold = pipeline_escalation_quality_threshold
           quality_check = @request.extra[:quality_check]
           succeeded = false
           tried = []
+          @last_escalation_error = nil
           log.debug "[llm][executor] action=escalation.enter chain_size=#{chain.size} threshold=#{threshold}"
 
           primary_tier = @escalation_chain.primary&.tier
@@ -541,10 +603,12 @@ module Legion
           chain.each do |resolution|
             next if tried.any? { |t| t[:provider] == resolution.provider && t[:instance] == resolution.instance && t[:model] == resolution.model }
 
-            succeeded = run_escalation_resolution(resolution, threshold, quality_check, tried, primary_tier)
+            succeeded = run_escalation_resolution(resolution, threshold, quality_check, tried, primary_tier,
+                                                  stream_block: stream_block)
             break if succeeded
           end
           return if succeeded
+          raise @last_escalation_error if chain.size <= 1 && @last_escalation_error
 
           emit_error_audit(
             EscalationExhausted.new("All #{@escalation_history.size} attempts failed"),
@@ -553,7 +617,7 @@ module Legion
           raise EscalationExhausted, "All #{@escalation_history.size} escalation attempts failed"
         end
 
-        def run_escalation_resolution(resolution, threshold, quality_check, tried, primary_tier)
+        def run_escalation_resolution(resolution, threshold, quality_check, tried, primary_tier, stream_block: nil)
           move_type = if tried.empty?
                         :primary
                       elsif resolution.tier == primary_tier
@@ -570,7 +634,7 @@ module Legion
           @resolved_tier = resolution.tier
           @resolved_offering_id = resolution.offering_id
           @resolved_offering_metadata = resolution.offering_metadata
-          succeeded = attempt_escalation(resolution, threshold, quality_check, start_time)
+          succeeded = attempt_escalation(resolution, threshold, quality_check, start_time, stream_block: stream_block)
           tried << { provider: resolution.provider, instance: resolution.instance, model: resolution.model } unless succeeded
           succeeded
         rescue Legion::LLM::AuthError, Legion::LLM::PrivacyModeError => e
@@ -605,14 +669,19 @@ module Legion
           false
         end
 
-        def attempt_escalation(resolution, threshold, quality_check, start_time)
+        def attempt_escalation(resolution, threshold, quality_check, start_time, stream_block: nil)
           @current_escalation_context = {
             attempt:      @escalation_history.size + 1,
             max_attempts: @escalation_chain&.max_attempts
           }.compact
-          execute_provider_request
+          if stream_block
+            execute_provider_request_stream(&stream_block)
+            result = Quality::Checker::QualityResult.new(passed: true, failures: [])
+          else
+            execute_provider_request
+            result = Quality::Checker.check(@raw_response, quality_threshold: threshold, quality_check: quality_check)
+          end
           duration_ms = ((Time.now - start_time) * 1000).round
-          result = Quality::Checker.check(@raw_response, quality_threshold: threshold, quality_check: quality_check)
           outcome = result.passed ? :success : :quality_failure
           @timeline.record(
             category: :provider, key: 'escalation:attempt', direction: :internal,
@@ -649,6 +718,7 @@ module Legion
         end
 
         def record_escalation_failure(err, resolution, start_time, outcome:, operation:, handled: false)
+          @last_escalation_error = err
           duration_ms = ((Time.now - start_time) * 1000).round
           handle_exception(err, level: :warn, handled: handled, operation: operation,
                                provider: resolution.provider, model: resolution.model, duration_ms: duration_ms)
@@ -689,7 +759,7 @@ module Legion
         end
 
         def build_fallback_resolutions(exclude_provider: nil, exclude_instance: nil, primary_tier: nil)
-          tier_rank = Router::TIER_RANK
+          tier_rank = Router.tier_rank
           primary_rank = primary_tier ? (tier_rank[primary_tier.to_sym] || 99) : 99
 
           candidates = Call::Registry.all_instances.filter_map do |entry|
@@ -869,6 +939,23 @@ module Legion
           !(@request.respond_to?(:suppress_tools) && @request.suppress_tools)
         end
 
+        def client_tool_passthrough_enabled?
+          return false unless @request.respond_to?(:metadata)
+
+          metadata = @request.metadata || {}
+          value = metadata[:client_tool_passthrough] || metadata['client_tool_passthrough']
+          value == true
+        end
+
+        def non_executable_client_tool?(definition)
+          source = definition.respond_to?(:source) ? definition.source : {}
+          return false unless source.is_a?(Hash)
+
+          source_type = source[:type] || source['type']
+          executable = source.key?(:executable) ? source[:executable] : source['executable']
+          source_type.respond_to?(:to_sym) && source_type.to_sym == :client && executable != true
+        end
+
         def add_pinned_special_tool_definitions(definitions)
           Tools::Special.pinned_definitions.each do |definition|
             next if definitions.any? { |existing| existing.name == definition.name }
@@ -887,6 +974,14 @@ module Legion
                        else
                          Types::ToolDefinition.from_tool_class(tool)
                        end
+          if non_executable_client_tool?(definition) && !client_tool_passthrough_enabled?
+            log.info(
+              "[llm][tools][inject] action=client_tool_skipped request_id=#{request_log_value(:id, 'unknown')} " \
+              "conversation_id=#{request_log_value(:conversation_id, 'none') || 'none'} name=#{definition.name} " \
+              'reason=client_passthrough_not_enabled'
+            )
+            return
+          end
           return if gaia_tool_suppressed?(definition.name)
           return if definitions.any? { |existing| existing.name == definition.name }
 
@@ -1168,10 +1263,15 @@ module Legion
 
         private :async_post_enabled?
 
-        def step_provider_call_stream(&)
+        def step_provider_call_stream(&block)
+          if pipeline_escalation_enabled?
+            run_provider_call_with_escalation(stream_block: block)
+            return
+          end
+
           providers_tried = []
           begin
-            execute_provider_request_stream(&)
+            execute_provider_request_stream(&block)
           rescue Legion::LLM::AuthError, Faraday::UnauthorizedError, Faraday::ForbiddenError => e
             try_fallback_or_raise(e, providers_tried, operation: 'provider_call_stream.auth',
                                                       reason: 'auth_failed', error_class: Legion::LLM::AuthError)

data/lib/legion/llm/inventory.rb

@@ -35,6 +35,15 @@ module Legion
         chat:      %i[chat completion tools json_schema]
       }.freeze
 
+      CAPABILITY_ALIASES = {
+        function_calling: :tools,
+        functions:        :tools,
+        tool:             :tools,
+        tool_use:         :tools,
+        stream:           :streaming,
+        stream_chat:      :streaming
+      }.freeze
+
       class << self
         def offerings(filters = {})
           log.debug "[llm][inventory] action=offerings.enter filters=#{filters.keys}"
@@ -216,7 +225,16 @@ module Legion
 
         def normalize_capabilities(capabilities, type)
           raw = capabilities || DEFAULT_CAPABILITIES.fetch(type, DEFAULT_CAPABILITIES[:inference])
-          Array(raw).map(&:to_s).uniq.sort
+          Array(raw).compact.each_with_object([]) do |capability, normalized|
+            next unless capability.respond_to?(:to_s)
+
+            capability_sym = capability.to_s.downcase.strip.to_sym
+            next if capability_sym.to_s.empty?
+
+            normalized << capability_sym
+            alias_sym = CAPABILITY_ALIASES[capability_sym]
+            normalized << alias_sym if alias_sym
+          end.uniq.map(&:to_s).sort
         end
 
         def normalize_type(value)

data/lib/legion/llm/router.rb

@@ -19,6 +19,14 @@ module Legion
       PROVIDER_ORDER = %i[ollama vllm bedrock azure gemini anthropic openai].freeze
       TIER_EXTERNAL = Set[:cloud, :frontier, :openai_compat].freeze
       TIER_RANK = { local: 0, direct: 1, fleet: 2, openai_compat: 3, cloud: 4, frontier: 5 }.freeze
+      CAPABILITY_ALIASES = {
+        function_calling: :tools,
+        functions:        :tools,
+        tool:             :tools,
+        tool_use:         :tools,
+        stream:           :streaming,
+        stream_chat:      :streaming
+      }.freeze
 
       OLLAMA_MODEL_PATTERN = %r{[:/]}
 
@@ -120,6 +128,22 @@ module Legion
           @auto_rules_populated = false
         end
 
+        def tier_priority
+          configured = Legion::LLM::Settings.value(:tier_order, default: nil)
+          configured = routing_settings[:tier_order] if configured.nil? || Array(configured).empty?
+          configured = routing_settings[:tier_priority] if configured.nil? || Array(configured).empty?
+          normalized = Array(configured).filter_map { |tier| tier.to_sym if tier.respond_to?(:to_sym) }
+          normalized = TIER_RANK.keys if normalized.empty?
+          (normalized + TIER_RANK.keys).uniq
+        rescue StandardError => e
+          handle_exception(e, level: :warn, handled: true, operation: 'router.tier_priority')
+          TIER_RANK.keys
+        end
+
+        def tier_rank
+          tier_priority.each_with_index.to_h
+        end
+
         # Check whether a tier can be used right now.
         # :local          — always available
         # :direct         — always available (remote self-hosted instances)
@@ -227,16 +251,19 @@ module Legion
           # 3. Filter by schedule
           scheduled = matched.select(&:within_schedule?)
 
-          # 4. Reject rules excluded by active constraints
-          unconstrained = scheduled.reject { |r| excluded_by_constraint?(r, constraints) }
+          # 4. Reject rules that cannot satisfy required model capabilities
+          capable = scheduled.select { |r| satisfies_required_capabilities?(r, intent) }
 
-          # 4.5 Reject Ollama rules where model is not pulled or doesn't fit
+          # 5. Reject rules excluded by active constraints
+          unconstrained = capable.reject { |r| excluded_by_constraint?(r, constraints) }
+
+          # 5.5 Reject Ollama rules where model is not pulled or doesn't fit
           discovered = unconstrained.reject { |r| excluded_by_discovery?(r) }
 
-          # 4.55 Reject local-tier rules where model exceeds available memory
+          # 5.55 Reject local-tier rules where model exceeds available memory
           memory_checked = discovered.reject { |r| excluded_by_memory?(r) }
 
-          # 4.6 Reject rules matching caller-provided exclude list
+          # 5.6 Reject rules matching caller-provided exclude list
           normalized_exclude = exclude.is_a?(Hash) ? exclude : {}
           not_excluded = if normalized_exclude.empty?
                            memory_checked
@@ -244,10 +271,10 @@ module Legion
                            memory_checked.reject { |r| excluded_by_caller?(r, normalized_exclude) }
                          end
 
-          # 4.7 Reject rules for models denied by health tracker
+          # 5.7 Reject rules for models denied by health tracker
           not_denied = not_excluded.reject { |r| excluded_by_denial?(r) }
 
-          # 5. Filter by tier availability
+          # 6. Filter by tier availability
           final = not_denied.select { |r| tier_available?(r.target[:tier] || r.target['tier']) }
 
           log.debug("Router: #{final.size} candidates after filtering (started with #{rules.size})")
@@ -255,6 +282,37 @@ module Legion
           final
         end
 
+        def satisfies_required_capabilities?(rule, intent)
+          required = required_capabilities(intent)
+          return true if required.empty?
+
+          rule_capabilities = normalize_capabilities(rule.target[:model_capabilities] || rule.target['model_capabilities'] ||
+                                                     rule.target[:capabilities] || rule.target['capabilities'])
+          return false if rule_capabilities.empty?
+
+          required.all? { |capability| rule_capabilities.include?(capability) }
+        end
+
+        def required_capabilities(intent)
+          return [] unless intent.is_a?(Hash)
+
+          normalized = intent.transform_keys { |key| key.respond_to?(:to_sym) ? key.to_sym : key }
+          normalize_capabilities(normalized[:required_capabilities] || normalized[:requires])
+        end
+
+        def normalize_capabilities(capabilities)
+          Array(capabilities).compact.each_with_object([]) do |capability, normalized|
+            next unless capability.respond_to?(:to_s)
+
+            capability_sym = capability.to_s.downcase.strip.to_sym
+            next if capability_sym.to_s.empty?
+
+            normalized << capability_sym
+            alias_sym = CAPABILITY_ALIASES[capability_sym]
+            normalized << alias_sym if alias_sym
+          end.uniq
+        end
+
         def excluded_by_constraint?(rule, constraints)
           return false if constraints.empty?
 
@@ -474,8 +532,15 @@ module Legion
             seen[pname] ||= entry
           end
 
-          # Build resolutions in PROVIDER_ORDER
-          PROVIDER_ORDER.filter_map do |pname|
+          # Build resolutions ordered by configured tier priority, preserving provider preference inside a tier.
+          provider_index = PROVIDER_ORDER.each_with_index.to_h
+          sorted_entries = seen.values.sort_by do |entry|
+            tier = registry_tier(entry[:provider], entry[:metadata])
+            [tier_rank.fetch(tier, 99), provider_index.fetch(entry[:provider], PROVIDER_ORDER.size)]
+          end
+
+          sorted_entries.filter_map do |entry|
+            pname = entry[:provider]
             entry = seen[pname]
             next unless entry
 

data/lib/legion/llm/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module LLM
-    VERSION = '0.9.28'
+    VERSION = '0.9.30'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legion-llm
 version: !ruby/object:Gem::Version
-  version: 0.9.28
+  version: 0.9.30
 platform: ruby
 authors:
 - Esity