eating a space prior to tool calls

Author: SamSaffron

lib/completions/xml_tool_processor.rb

@@ -30,7 +30,7 @@ def <<(text)
           if @in_tool
             @function_buffer = @buffer[index..-1]
             text_index = text.rindex("<function_calls>")
-            result << text[0..text_index - 1].strip if text_index && text_index > 0
+            result << text[0..text_index - 1].rstrip if text_index && text_index > 0
           end
         else
           add_to_function_buffer(text)

spec/lib/completions/xml_tool_processor_spec.rb

@@ -86,9 +86,8 @@
   end
 
   it "is usable for simple single message mode" do
-    xml = (<<~XML).strip
-      hello
-      <function_calls>
+    xml = (<<~XML)
+       world <function_calls>
       <invoke>
       <tool_name>hello</tool_name>
       <parameters>
@@ -99,6 +98,7 @@
     XML
 
     result = []
+    result << (processor << "hello")
     result << (processor << xml)
     result << (processor.finish)
 
@@ -111,7 +111,7 @@
           test: "value",
         },
       )
-    expect(result).to eq([["hello"], [tool_call]])
+    expect(result).to eq([["hello"], [" world"], [tool_call]])
     expect(processor.should_cancel?).to eq(false)
   end