address comments

Author: xzrderek

eval_protocol/mcp/execution/base_policy.py

@@ -59,20 +59,6 @@ def __init__(
         # Initialize conversation state tracking for proper OpenAI trajectories
         self.initialized = False
 
-    def _supports_reasoning_details(self) -> bool:
-        """
-        Returns True if this policy is configured for a provider/model that expects
-        top-level reasoning_details to be preserved (e.g., Gemini 3 via OpenRouter).
-        """
-        model_id = getattr(self, "model_id", "") or ""
-        base_url = getattr(self, "base_url", "") or ""
-
-        if isinstance(model_id, str) and "openrouter" in model_id:
-            return True
-        if isinstance(base_url, str) and "openrouter.ai" in base_url:
-            return True
-        return False
-
     @abstractmethod
     async def _make_llm_call(self, messages: List[Dict], tools: List[Dict]) -> Dict:
         """
@@ -214,7 +200,7 @@ async def _generate_live_tool_calls(
             assistant_message_for_history["tool_calls"] = message["tool_calls"]
 
         rd = message.get("reasoning_details", None)
-        if rd is not None and self._supports_reasoning_details():
+        if rd is not None:
             assistant_message_for_history["reasoning_details"] = rd
 
         # Add to actual conversation history

eval_protocol/mcp/execution/policy.py

@@ -146,10 +146,7 @@ def _clean_messages_for_api(self, messages: List[Dict]) -> List[Dict]:
             Clean messages with only OpenAI API compatible fields
         """
         # Standard OpenAI message fields
-        allowed_fields = {"role", "content", "tool_calls", "tool_call_id", "name"}
-
-        if self._supports_reasoning_details():
-            allowed_fields.add("reasoning_details")
+        allowed_fields = {"role", "content", "tool_calls", "tool_call_id", "name", "reasoning_details"}
 
         clean_messages = []
         for msg in messages:
@@ -242,10 +239,9 @@ async def _make_llm_call(self, messages: List[Dict[str, Any]], tools: List[Dict[
                 ),
             }
 
-            if self._supports_reasoning_details():
-                rd = getattr(message_obj, "reasoning_details", None)
-                if rd is not None:
-                    message_dict["reasoning_details"] = rd
+            rd = getattr(message_obj, "reasoning_details", None)
+            if rd is not None:
+                message_dict["reasoning_details"] = rd
 
             return {
                 "choices": [