refactor(logging): Replace debug prints with logger calls in Mistral chat provider

- Updated debug output from `puts` to `RubyLLM.logger.debug` for better logging management.
- Commented out skip conditions in tests for Mistral provider to avoid flaky behavior during testing.

Author: OriPekelman

lib/ruby_llm/providers/mistral/chat.rb

@@ -25,7 +25,7 @@ def render_payload(messages, tools:, temperature:, model:, stream: nil,
               Array(tools)
             end
 
-          puts "\n[DEBUG] Available tools: #{tools_array&.map { |t| t.name.to_s }}" if ENV["DEBUG"]
+          RubyLLM.logger.debug  "[DEBUG] Available tools: #{tools_array&.map { |t| t.name.to_s }}"
 
           # Use "any" for tool_choice when tools are available
           effective_tool_choice = if tool_choice
@@ -36,7 +36,7 @@ def render_payload(messages, tools:, temperature:, model:, stream: nil,
               "none"
             end
 
-          puts "[DEBUG] Tool choice: #{effective_tool_choice.inspect}" if ENV["DEBUG"]
+          RubyLLM.logger.debug "[DEBUG] Tool choice: #{effective_tool_choice.inspect}"
 
           payload = {
             model: model,
@@ -55,7 +55,7 @@ def render_payload(messages, tools:, temperature:, model:, stream: nil,
             frequency_penalty: frequency_penalty,
           }.compact
 
-          puts "[DEBUG] Full payload: #{payload.inspect}" if ENV["DEBUG"]
+          RubyLLM.logger.debug  "[DEBUG] Full payload: #{payload.inspect}"
 
           payload
         end
@@ -110,7 +110,7 @@ def render_tool_call(tool_call)
               arguments: tool_call.arguments,
             },
           }
-          puts "[DEBUG] Rendered tool call: #{tool_call_spec.inspect}" if ENV["DEBUG"]
+          RubyLLM.logger.debug "[DEBUG] Rendered tool call: #{tool_call_spec.inspect}"
           tool_call_spec
         end
 
@@ -132,7 +132,7 @@ def render_tool(tool)
               }
             }
           }
-          puts "[DEBUG] Rendered tool spec: #{tool_spec.inspect}" if ENV["DEBUG"]
+          RubyLLM.logger.debug "[DEBUG] Rendered tool spec: #{tool_spec.inspect}"
           tool_spec
         end
 
@@ -144,7 +144,7 @@ def param_schema(param)
         end
 
         def parse_completion_response(response)
-          puts "\n[DEBUG] Raw response: #{response.body.inspect}" if ENV["DEBUG"]
+          RubyLLM.logger.debug "\n[DEBUG] Raw response: #{response.body.inspect}"
 
           if response.body["error"]
             error_message = response.body.dig("error", "message")
@@ -166,10 +166,10 @@ def parse_completion_response(response)
           message = choice.dig("message")
           return unless message
 
-          puts "[DEBUG] Message from model: #{message.inspect}" if ENV["DEBUG"]
+          RubyLLM.logger.debug "[DEBUG] Message from model: #{message.inspect}"
 
           tool_calls = Mistral::Tools.parse_tool_calls(message["tool_calls"])
-          puts "[DEBUG] Parsed tool calls: #{tool_calls.inspect}" if ENV["DEBUG"]
+          RubyLLM.logger.debug  "[DEBUG] Parsed tool calls: #{tool_calls.inspect}"
 
           content = message["content"]
 

spec/ruby_llm/chat_spec.rb

@@ -31,7 +31,7 @@
 
       it "#{provider}/#{model} successfully uses the system prompt" do
         skip 'System prompt can be flaky for Ollama models' if provider == :ollama
-        skip 'Mistral API does not allow system messages after assistant messages' if provider == :mistral
+        #skip 'Mistral API does not allow system messages after assistant messages' if provider == :mistral
         chat = RubyLLM.chat(model: model, provider: provider).with_temperature(0.0)
 
         # Use a distinctive and unusual instruction that wouldn't happen naturally

spec/ruby_llm/chat_tools_spec.rb

@@ -36,7 +36,7 @@ def execute
       model = model_info[:model]
       provider = model_info[:provider]
       it "#{provider}/#{model} can use tools" do # rubocop:disable RSpec/MultipleExpectations
-        skip 'Mistral does not reliably support tool usage' if provider == :mistral
+        #skip 'Mistral does not reliably support tool usage' if provider == :mistral
         chat = RubyLLM.chat(model: model, provider: provider)
                       .with_tool(Weather)
 
@@ -69,7 +69,7 @@ def execute
       provider = model_info[:provider]
       it "#{provider}/#{model} can use tools without parameters" do
         skip 'Ollama models do not reliably use tools without parameters' if provider == :ollama
-        skip 'Mistral does not reliably support tool usage' if provider == :mistral
+        #skip 'Mistral does not reliably support tool usage' if provider == :mistral
         chat = RubyLLM.chat(model: model, provider: provider)
                       .with_tool(BestLanguageToLearn)
         response = chat.ask("What's the best language to learn?")

spec/ruby_llm/embeddings_spec.rb

@@ -27,7 +27,7 @@
       end
 
       it "#{provider}/#{model} can handle a single text with custom dimensions" do # rubocop:disable RSpec/MultipleExpectations
-        skip "Mistral embed does not support custom dimensions" if model == "mistral-embed"
+        skip "Mistral embed does not support custom dimensions" if provider == :mistral
         embedding = RubyLLM.embed(test_text, model: model, dimensions: test_dimensions)
         expect(embedding.vectors).to be_an(Array)
         expect(embedding.vectors.length).to eq(test_dimensions)
@@ -43,7 +43,7 @@
       end
 
       it "#{provider}/#{model} can handle multiple texts with custom dimensions" do # rubocop:disable RSpec/MultipleExpectations
-        skip "Mistral embed does not support custom dimensions" if model == "mistral-embed"
+        skip "Mistral embed does not support custom dimensions" if provider == :mistral
         embeddings = RubyLLM.embed(test_texts, model: model, dimensions: test_dimensions)
         expect(embeddings.vectors).to be_an(Array)
         embeddings.vectors.each do |vector|