Refactor services and components: improve logging, remove unused imports, and enhance search configuration handling

Author: Kingshuk-Microsoft

src/backend/v4/common/services/team_service.py

@@ -481,6 +481,12 @@ async def validate_team_search_indexes(
                     error_msg = "Team configuration references search indexes but no Azure Search endpoint is configured"
                     self.logger.warning(error_msg)
                     return False, [error_msg]
+                else:
+                    # No search endpoint and nothing to validate; skip further validation
+                    self.logger.info(
+                        "No Azure Search endpoint configured and no search indexes or RAG agents to validate - skipping search validation"
+                    )
+                    return True, []
 
             if not index_names:
                 self.logger.info(

src/backend/v4/magentic_agents/models/agent_models.py

@@ -47,7 +47,6 @@ class SearchConfig:
     @classmethod
     def from_env(cls, index_name: str) -> "SearchConfig":
         connection_name = config.AZURE_AI_SEARCH_CONNECTION_NAME
-        index_name = index_name
         endpoint = config.AZURE_AI_SEARCH_ENDPOINT
 
         # Raise exception if any required environment variable is missing
@@ -58,5 +57,6 @@ def from_env(cls, index_name: str) -> "SearchConfig":
 
         return cls(
             connection_name=connection_name,
-            endpoint=endpoint
+            endpoint=endpoint,
+            index_name=index_name
         )

src/frontend/src/components/content/HomeInput.tsx

@@ -15,7 +15,6 @@ import "./../../styles/HomeInput.css";
 import { HomeInputProps, iconMap, QuickTask } from "../../models/homeInput";
 import { TaskService } from "../../services/TaskService";
 import { NewTaskService } from "../../services/NewTaskService";
-import { RAIErrorData } from "../errors";
 
 import ChatInput from "@/coral/modules/ChatInput";
 import InlineToaster, { useInlineToaster } from "../toast/InlineToaster";

src/frontend/src/components/content/PlanChat.tsx

@@ -1,6 +1,4 @@
 import React from "react";
-import { 
-} from "@fluentui/react-icons";
 import { PlanChatProps, MPlanData } from "../../models/plan";
 import InlineToaster from "../toast/InlineToaster";
 import { AgentMessageData } from "@/models";

src/frontend/src/coral/components/Panels/PanelRightToolbar.tsx

@@ -21,7 +21,7 @@ const PanelRightToolbar: React.FC<PanelRightToolbarProps> = ({
     const onDismiss = () => {
         if (handleDismiss) {
             handleDismiss();
-        }else {
+        } else {
             eventBus.emit("setActivePanel", null); // Close the current panel
         }
     };

src/mcp_server/services/marketing_service.py

@@ -4,8 +4,6 @@
 
 
 from core.factory import Domain, MCPToolBase
-from utils.formatters import format_success_response
-
 
 class MarketingService(MCPToolBase):
     """Marketing tools for employee onboarding and management."""

src/mcp_server/services/product_service.py

@@ -4,8 +4,6 @@
 
 
 from core.factory import Domain, MCPToolBase
-from utils.formatters import format_success_response
-
 
 class ProductService(MCPToolBase):
     """Product tools for employee onboarding and management."""