[KM-564] edit agent logic

src/agents/chat_handler.py

@@ -28,10 +28,10 @@ from typing import TYPE_CHECKING, Any
 from langchain_core.messages import BaseMessage
 
 from src.middlewares.logging import get_logger
-
 from src.retrieval.base import RetrievalResult
+
 from .chatbot import ChatbotAgent, DocumentChunk
-from .intent_router import IntentRouter
+from .orchestration import OrchestratorAgent
 
 if TYPE_CHECKING:
     from ..catalog.reader import CatalogReader
@@ -54,7 +54,7 @@ class ChatHandler:
 
     def __init__(
         self,
-        intent_router: IntentRouter | None = None,
+        intent_router: OrchestratorAgent | None = None,
         answer_agent: ChatbotAgent | None = None,
         catalog_reader: CatalogReader | None = None,
         query_service: QueryService | None = None,
@@ -70,9 +70,9 @@ class ChatHandler:
     # Lazy default-dep builders
     # ------------------------------------------------------------------
 
-    def _get_intent_router(self) -> IntentRouter:
+    def _get_intent_router(self) -> OrchestratorAgent:
         if self._intent_router is None:
-            self._intent_router = IntentRouter()
+            self._intent_router = OrchestratorAgent()
         return self._intent_router
 
     def _get_answer_agent(self) -> ChatbotAgent:

src/agents/{intent_router.py → orchestration.py}

@@ -1,11 +1,13 @@
-"""IntentRouter — classifies a user message and emits source_hint.
+"""OrchestratorAgent — classifies a user message and emits source_hint.
 
 Output: needs_search (bool) + source_hint ∈ { chat, unstructured, structured }
 + rewritten_query (standalone form of the user's question, history-resolved).
 
-Replaces the previous orchestration.py once the chat endpoint is rewired.
-The default LLM is constructed lazily so the module is import-safe even
-without `.env` populated.
+Phase 2 replaces the previous intent-classification body. The class name
+is preserved so existing import sites (`from src.agents.orchestration
+import OrchestratorAgent`) keep working. The default LLM chain is
+constructed lazily so the module is import-safe even without `.env`
+populated.
 """
 
 from __future__ import annotations
@@ -21,7 +23,7 @@ from pydantic import BaseModel, Field
 
 from src.middlewares.logging import get_logger
 
-logger = get_logger("intent_router")
+logger = get_logger("orchestrator")
 
 SourceHint = Literal["chat", "unstructured", "structured"]
 
@@ -75,7 +77,7 @@ def _build_default_chain() -> Runnable:
     return prompt | llm.with_structured_output(IntentRouterDecision)
 
 
-class IntentRouter:
+class OrchestratorAgent:
     """Classifies a user message into chat / unstructured / structured.
 
     Inject `structured_chain` for tests; default builds the production