Update mcp_client.py

mcp_client.py

@@ -257,23 +257,28 @@ class UniversalMCPClient:
         # Add any remaining kwargs (highest precedence)
         params.update(kwargs)
         
-        # Add reasoning effort if specified (GPT OSS feature)
-        reasoning_effort = kwargs.pop("reasoning_effort", self.generation_params.get("reasoning_effort", AppConfig.DEFAULT_REASONING_EFFORT))
-        if reasoning_effort:
-            # For GPT OSS models, we can set reasoning in system prompt
-            system_message = None
-            for msg in messages:
-                if msg.get("role") == "system":
-                    system_message = msg
-                    break
-            
-            if system_message:
-                system_message["content"] += f"\n\nReasoning: {reasoning_effort}"
-            else:
-                messages.insert(0, {
-                    "role": "system", 
-                    "content": f"You are a helpful AI assistant. Reasoning: {reasoning_effort}"
-                })
+        # Add reasoning effort only for GPT-OSS models
+        if AppConfig.is_gpt_oss_model(self.current_model):
+            reasoning_effort = kwargs.pop("reasoning_effort", self.generation_params.get("reasoning_effort", AppConfig.DEFAULT_REASONING_EFFORT))
+            if reasoning_effort:
+                # For GPT OSS models, we can set reasoning in system prompt
+                system_message = None
+                for msg in messages:
+                    if msg.get("role") == "system":
+                        system_message = msg
+                        break
+                
+                if system_message:
+                    system_message["content"] += f"\n\nReasoning: {reasoning_effort}"
+                else:
+                    messages.insert(0, {
+                        "role": "system", 
+                        "content": f"You are a helpful AI assistant. Reasoning: {reasoning_effort}"
+                    })
+        else:
+            # Ensure reasoning_effort is not sent in params for non-GPT-OSS
+            if "reasoning_effort" in params:
+                params.pop("reasoning_effort", None)
         
         try:
             logger.info(f"🤖 Calling {model_endpoint} via {self.current_provider}")