From 5cd3415a330f81edc42f9ab476f0b61919ceef25 Mon Sep 17 00:00:00 2001
From: Daniel Chalef <131175+danielchalef@users.noreply.github.com>
Date: Thu, 30 Oct 2025 17:03:12 -0700
Subject: [PATCH] refactor: Remove duplicate is_reasoning_model calculation in
 factories.py

---
 mcp_server/src/services/factories.py | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/mcp_server/src/services/factories.py b/mcp_server/src/services/factories.py
index 23b8f256..02f19e35 100644
--- a/mcp_server/src/services/factories.py
+++ b/mcp_server/src/services/factories.py
@@ -138,12 +138,6 @@ class LLMClientFactory:
                 )
 
                 # Only pass reasoning/verbosity parameters for reasoning models (gpt-5 family)
-                is_reasoning_model = (
-                    config.model.startswith('gpt-5')
-                    or config.model.startswith('o1')
-                    or config.model.startswith('o3')
-                )
-
                 if is_reasoning_model:
                     return OpenAIClient(config=llm_config, reasoning='minimal', verbosity='low')
                 else: