refactor: promote _is_reasoning_model and _resolve_model_name to public API

Remove underscore prefix from helper functions in agent_builder.py to
make them part of the supported public API, eliminating fragile
cross-module dependency on private internals from map_handler.py.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: Prachig-Microsoft

src/ContentProcessor/src/libs/agent_framework/agent_builder.py

@@ -34,13 +34,13 @@
 _REASONING_MODEL_PREFIXES = ("o1", "o3", "o4", "gpt-5")
 
 
-def _is_reasoning_model(model_name: str) -> bool:
+def is_reasoning_model(model_name: str) -> bool:
     """Check if a model is a reasoning model based on its deployment name."""
     name = model_name.lower()
     return any(name.startswith(prefix) for prefix in _REASONING_MODEL_PREFIXES)
 
 
-def _resolve_model_name(client: Any, model_id: str | None = None) -> str | None:
+def resolve_model_name(client: Any, model_id: str | None = None) -> str | None:
     """Extract model/deployment name from the client or explicit model_id."""
     if model_id:
         return model_id
@@ -55,7 +55,7 @@ def _strip_unsupported_reasoning_params(
     options: dict[str, Any], model_name: str | None
 ) -> dict[str, Any]:
     """Remove temperature and top_p for reasoning models that don't support them."""
-    if model_name and _is_reasoning_model(model_name):
+    if model_name and is_reasoning_model(model_name):
         removed = []
         for param in ("temperature", "top_p"):
             if param in options:
@@ -525,7 +525,7 @@ def build(self) -> Agent:
         if self._additional_chat_options:
             options.update(self._additional_chat_options)
 
-        model_name = _resolve_model_name(self._chat_client, self._model_id)
+        model_name = resolve_model_name(self._chat_client, self._model_id)
         _strip_unsupported_reasoning_params(options, model_name)
 
         default_options: ChatOptions | None = ChatOptions(**options) if options else None
@@ -866,7 +866,7 @@ def create_agent(
         if additional_chat_options:
             options.update(additional_chat_options)
 
-        model_name = _resolve_model_name(chat_client, model_id)
+        model_name = resolve_model_name(chat_client, model_id)
         _strip_unsupported_reasoning_params(options, model_name)
 
         default_options: ChatOptions | None = ChatOptions(**options) if options else None

src/ContentProcessor/src/libs/pipeline/handlers/map_handler.py

@@ -17,7 +17,7 @@
 from agent_framework import Content, Message
 from pdf2image import convert_from_bytes
 
-from libs.agent_framework.agent_builder import AgentBuilder, _is_reasoning_model, _resolve_model_name
+from libs.agent_framework.agent_builder import AgentBuilder, is_reasoning_model, resolve_model_name
 from libs.agent_framework.agent_framework_helper import AgentFrameworkHelper
 from libs.agent_framework.azure_openai_response_retry import ContextTrimConfig
 from libs.application.application_context import AppContext
@@ -256,8 +256,8 @@ async def execute(self, context: MessageContext) -> StepResult:
         )
 
         # logprobs is not supported by reasoning models (o1, o3, gpt-5.x)
-        model_name = _resolve_model_name(agent_client)
-        if model_name and _is_reasoning_model(model_name):
+        model_name = resolve_model_name(agent_client)
+        if model_name and is_reasoning_model(model_name):
             run_options = {}
         else:
             run_options = {"logprobs": True, "top_logprobs": 5}