legion-llm 0.14.18 → 0.14.20

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eb744016e12f01c149812c027353b25ecba016826789d83a3a795a5b9cba2a19
-  data.tar.gz: 8ed359b0f6d0b9d147a04a419ad30bf26834a258b1448c625e0a6614187f5db3
+  metadata.gz: 96388b9de10bee10e82a3d0ec28ee9206710cfdc0c61684022fc94bb9250a9b0
+  data.tar.gz: 93362c4565301c19d55a32cb28b9e2e8fae145a893fb7330594188916280fa9e
 SHA512:
-  metadata.gz: 74db62246f4ff813981821a699374fa21e39d84b2d7496c28c969c3f2345ea777d7c74cd5a763222b5fa0014864eda6ae81332609a595b9f51a02162198af778
-  data.tar.gz: 46b000f6e78863046f6c05560ba24aa49a16d097f92cea59390735b9acb42b9fbed1f1f0d04715262b5a9f07b1b521c370484b11acb010822a6dc56737dde40c
+  metadata.gz: 717d93944bb01f208072e86b90ecf4649baa084da362cab3217f3c58c76c6e7da3c35ad088907b8ed10a607199ae4604fcc6f89862c2f625e1ce8da9160afdf7
+  data.tar.gz: b80edee9bd60d15b433e3a5b816eeb90ee75065b9018a7ec07acc7574191af17904752531a1361cdf41ebdd44f6fcc9dea08a4db69e3336c1ef3371e1e4a8c80

data/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Legion LLM Changelog
 
+## [0.14.20] - 2026-07-24
+
+### Fixed
+- **Curator `strip_thinking` no longer runs on tool messages.** Tool results (`role: :tool`) contain raw command output, not thinking content. The method was incorrectly applying `lstrip` and thinking-tag detection to tool results, stripping leading whitespace from Bash output on every curation pass.
+- **Curator `strip_thinking_tags` no longer lstrips content unconditionally.** Leading whitespace is only removed between consecutive tag removals, not from the original input.
+- **StreamAssembler allows interleaved thinking blocks.** Models that produce think→content→think→content sequences no longer have subsequent thinking phases silently dropped. Each thinking phase opens a new block instead of being discarded.
+- **Trigger match strips all tagged blocks and harness messages.** Tool injection no longer triggers on words from `<system-reminder>`, CLAUDE.md content, or harness prompts (SUGGESTION MODE, title generation). Only actual user conversational text is scanned for trigger words.
+
+### Changed
+- **Context curation token targets raised.** `target_context_tokens` 60K→120K, `summarize_threshold` 90K→120K, `target_tokens` 60K→90K.
+
+## [0.14.19] - 2026-07-24
+
+### Fixed
+- **Curator `strip_thinking` no longer runs on tool messages.** Tool results (`role: :tool`) contain raw command output, not thinking content. The method was incorrectly applying `lstrip` and thinking-tag detection to tool results, stripping leading whitespace from Bash output on every curation pass.
+
 ## [0.14.18] - 2026-07-24
 
 ### Changed

data/lib/legion/llm/api/stream_assembler.rb

@@ -352,24 +352,26 @@ module Legion
         end
 
         def handle_thinking_delta(text, signature)
-          # Thinking always comes before text per the canonical ordering.
-          # When emit_thinking_blocks is off we still accumulate the internal
-          # buffer (so finalize can attach it to the final message audit) but
-          # never emit blocks to the client.
           @full_thinking << text.to_s unless text.to_s.empty?
           @full_thinking_signature ||= signature.to_s unless signature.to_s.empty?
 
           return unless @emit_thinking_blocks
 
-          unless @thinking_block_open || @thinking_block_closed
+          # If thinking arrives after text has started, open a new thinking block
+          if @thinking_block_closed
+            close_text_block
+            @thinking_block_closed = false
+            @thinking_block_index = @next_block_index
+            @next_block_index += 1
+            guard { @emitter.on_thinking_open(block_index: @thinking_block_index) }
+            @thinking_block_open = true
+          elsif !@thinking_block_open
             @thinking_block_index = @next_block_index
             @next_block_index += 1
             guard { @emitter.on_thinking_open(block_index: @thinking_block_index) }
             @thinking_block_open = true
           end
 
-          return if @thinking_block_closed
-
           @phase = :mid_thinking
           guard { @emitter.on_thinking_delta(block_index: @thinking_block_index, text: text.to_s, signature: signature.to_s) }
         end

data/lib/legion/llm/context/curator.rb

@@ -123,6 +123,7 @@ module Legion
         # Heuristic: remove extended thinking blocks, keep conclusions.
         def strip_thinking(msg)
           return msg unless setting(:thinking_eviction, true)
+          return msg if msg[:role] == :tool
 
           content = msg[:content].to_s
           stripped = strip_thinking_tags(content)
@@ -136,6 +137,10 @@ module Legion
           chars_removed = content.length - stripped.length
           log.info "[llm][curator] action=strip_thinking conversation_id=#{@conversation_id} " \
                    "chars_removed=#{chars_removed} original_chars=#{content.length} stripped_chars=#{stripped.length}"
+          if content.length < 50
+            log.debug "[llm][curator] action=strip_thinking_debug conversation_id=#{@conversation_id} " \
+                      "original=#{content.inspect} stripped=#{stripped.inspect} role=#{msg[:role]}"
+          end
           msg.merge(content: stripped, curated: true, original_content: content)
         end
 
@@ -293,7 +298,7 @@ module Legion
         end
 
         def strip_thinking_tags(text)
-          result = text.lstrip
+          result = text
           loop do
             stripped = false
             THINKING_TAG_PAIRS.each do |open_tag, close_tag|

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

@@ -36,7 +36,7 @@ module Legion
 
         private
 
-        def execute_native_tool_loop
+        def execute_native_tool_loop # rubocop:disable Metrics/AbcSize
           messages = native_dispatch_messages.dup
           max_rounds = Legion::Settings[:llm][:max_tool_rounds].to_i
           max_rounds = 200 unless max_rounds.positive?
@@ -60,6 +60,17 @@ module Legion
               result = apply_synthesized_tool_calls(result, tool_calls) if tool_calls.any?
             end
             if tool_calls.empty?
+              result_text = result.respond_to?(:text) ? result.text : (result[:result] || result[:content])
+              result_thinking = if result.respond_to?(:thinking)
+                                  t = result.thinking
+                                  t.respond_to?(:content) ? t.content : t.to_s
+                                end
+              log.debug "[llm][native_tool_loop] action=exit_no_tool_calls round=#{round} " \
+                        "result_text_length=#{result_text.to_s.length} " \
+                        "result_text=#{result_text.to_s.inspect} " \
+                        "thinking_length=#{result_thinking.to_s.length} " \
+                        "thinking_first_200=#{result_thinking.to_s[0, 200].inspect} " \
+                        "stop_reason=#{result.respond_to?(:stop_reason) ? result.stop_reason : 'n/a'}"
               log.debug "[llm][executor] action=native_tool_loop.complete rounds=#{round} reason=no_tool_calls"
               @last_tool_loop_messages = messages
               return result
@@ -143,7 +154,7 @@ module Legion
           @native_tool_loop_round = nil
         end
 
-        def execute_native_streaming_tool_loop(&block)
+        def execute_native_streaming_tool_loop(&block) # rubocop:disable Metrics/AbcSize
           messages = native_dispatch_messages.dup
           max_rounds = Legion::Settings[:llm][:max_tool_rounds].to_i
           max_rounds = 200 unless max_rounds.positive?
@@ -167,6 +178,17 @@ module Legion
               result = apply_synthesized_tool_calls(result, tool_calls) if tool_calls.any?
             end
             if tool_calls.empty?
+              result_text = result.respond_to?(:text) ? result.text : (result[:result] || result[:content])
+              result_thinking = if result.respond_to?(:thinking)
+                                  t = result.thinking
+                                  t.respond_to?(:content) ? t.content : t.to_s
+                                end
+              log.debug "[llm][native_tool_loop] action=stream_exit_no_tool_calls round=#{round} " \
+                        "result_text_length=#{result_text.to_s.length} " \
+                        "result_text=#{result_text.to_s.inspect} " \
+                        "thinking_length=#{result_thinking.to_s.length} " \
+                        "thinking_first_200=#{result_thinking.to_s[0, 200].inspect} " \
+                        "stop_reason=#{result.respond_to?(:stop_reason) ? result.stop_reason : 'n/a'}"
               log.debug "[llm][executor] action=native_streaming_tool_loop.complete rounds=#{round} reason=no_tool_calls"
               @last_tool_loop_messages = messages
               return result

data/lib/legion/llm/inference/steps/trigger_match.rb

@@ -70,40 +70,56 @@ module Legion
             record_trigger_match_timeline(0, start_time)
           end
 
-          private
+          HARNESS_PREFIXES = [
+            '[SUGGESTION MODE',
+            '[REQUEST INTERRUPTED',
+            'Write the title in the predominant language'
+          ].freeze
 
           def extract_recent_text
             depth = trigger_scan_depth
             messages = @request.messages.last(depth)
+
             text = messages.filter_map do |msg|
               next unless msg.is_a?(Hash)
               next unless (msg[:role] || msg['role']).to_s == 'user'
 
               content = msg[:content] || msg['content']
-              if content.is_a?(Array)
-                content.filter_map do |c|
-                  if c.is_a?(String)
-                    c
-                  else
-                    (c.is_a?(Hash) ? (c[:text] || c['text']) : nil)
-                  end
-                end.join(' ')
-              else
-                content.to_s
-              end
+              raw = if content.is_a?(Array)
+                      content.filter_map do |c|
+                        if c.is_a?(String)
+                          c
+                        else
+                          (c.is_a?(Hash) ? (c[:text] || c['text']) : nil)
+                        end
+                      end.join(' ')
+                    else
+                      content.to_s
+                    end
+              next if harness_message?(raw)
+
+              raw
             end.join(' ')
 
             sanitize_trigger_text(text)
           end
 
           def sanitize_trigger_text(text)
-            stripped = strip_system_reminders(text)
-            session_text = extract_session_text(stripped)
-            session_text || stripped
+            session_text = extract_session_text(text)
+            return session_text if session_text
+
+            strip_tagged_blocks(text)
           end
 
-          def strip_system_reminders(text)
-            text.to_s.gsub(%r{<system-reminder\b[^>]*>.*?</system-reminder>}mi, ' ')
+          def strip_tagged_blocks(text)
+            text.to_s.gsub(%r{<[a-z][\w-]*\b[^>]*>.*?</[a-z][\w-]*>}mi, ' ')
+          end
+
+          def harness_message?(text)
+            return false if text.nil? || text.empty?
+
+            trimmed = text.lstrip
+            HARNESS_PREFIXES.any? { |prefix| trimmed.start_with?(prefix) }
           end
 
           def extract_session_text(text)

data/lib/legion/llm/settings.rb

@@ -437,7 +437,7 @@ module Legion
             # Claude Code linter/formatter file-dump note.
             'was modified, either by the user or by a linter'
           ],
-          target_context_tokens:    60_000,
+          target_context_tokens:    120_000,
           context_window_threshold: 0.90,
           archive_dropped_turns:    true,
           archive_preserve_recent:  10
@@ -446,8 +446,8 @@ module Legion
 
       def self.conversation_defaults
         {
-          summarize_threshold: 90_000,
-          target_tokens:       60_000,
+          summarize_threshold: 120_000,
+          target_tokens:       90_000,
           preserve_recent:     10,
           auto_compact:        true
         }

data/lib/legion/llm/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module LLM
-    VERSION = '0.14.18'
+    VERSION = '0.14.20'
   end
 end

metadata

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