format and lint fix

Signed-off-by: ashu8912 <aghildiyal@microsoft.com>

Author: ashu8912

ai-assistant/src/ai/mcp/electron-client.ts

@@ -8,33 +8,6 @@ interface MCPTool {
   server?: string; // Add server information
 }
 
-interface MCPResponse {
-  success: boolean;
-  tools?: MCPTool[];
-  result?: any;
-  error?: string;
-  toolCallId?: string;
-}
-
-interface ElectronMCPApi {
-  executeTool: (
-    toolName: string,
-    args: Record<string, any>,
-    toolCallId?: string
-  ) => Promise<MCPResponse>;
-  getStatus: () => Promise<{ isInitialized: boolean; hasClient: boolean }>;
-  resetClient: () => Promise<MCPResponse>;
-  getConfig: () => Promise<{ success: boolean; config?: any; error?: string }>;
-  updateConfig: (config: any) => Promise<MCPResponse>;
-  getToolsConfig: () => Promise<{ success: boolean; config?: any; error?: string }>;
-  updateToolsConfig: (config: any) => Promise<MCPResponse>;
-  setToolEnabled: (serverName: string, toolName: string, enabled: boolean) => Promise<MCPResponse>;
-  getToolStats: (
-    serverName: string,
-    toolName: string
-  ) => Promise<{ success: boolean; stats?: any; error?: string }>;
-}
-
 // Type augmentation is handled by src/types/electron.d.ts
 
 class ElectronMCPClient {

ai-assistant/src/components/settings/MCPServerEditor.tsx

@@ -9,7 +9,6 @@ import {
   DialogTitle,
   FormControlLabel,
   IconButton,
-  Paper,
   Switch,
   TextField,
   Typography,
@@ -244,7 +243,11 @@ export default function MCPServerEditor({
                       sx={{ flex: 2 }}
                       placeholder="e.g., HEADLAMP_CURRENT_CLUSTER"
                     />
-                    <IconButton size="small" onClick={() => handleRemoveEnvVar(index)} color="error">
+                    <IconButton
+                      size="small"
+                      onClick={() => handleRemoveEnvVar(index)}
+                      color="error"
+                    >
                       <Icon icon="mdi:delete" />
                     </IconButton>
                   </Box>

ai-assistant/src/components/settings/MCPServerFormDialog.tsx

@@ -4,7 +4,6 @@ import {
   Alert,
   Box,
   Button,
-  Chip,
   DialogActions,
   DialogContent,
   DialogTitle,
@@ -76,9 +75,7 @@ export default function MCPServerFormDialog({
         name: server.name,
         command: server.command,
         args: server.args.join(' '),
-        env: server.env
-          ? Object.entries(server.env).map(([key, value]) => ({ key, value }))
-          : [],
+        env: server.env ? Object.entries(server.env).map(([key, value]) => ({ key, value })) : [],
         enabled: server.enabled,
       }));
       setServers(formData);
@@ -415,7 +412,9 @@ export default function MCPServerFormDialog({
                 control={
                   <Switch
                     checked={currentServer.enabled}
-                    onChange={e => setCurrentServer({ ...currentServer, enabled: e.target.checked })}
+                    onChange={e =>
+                      setCurrentServer({ ...currentServer, enabled: e.target.checked })
+                    }
                   />
                 }
                 label="Enable this server"

ai-assistant/src/components/settings/MCPSettings.tsx

@@ -41,6 +41,7 @@ interface MCPSettingsProps {
 }
 
 export function MCPSettings({ config, onConfigChange }: MCPSettingsProps) {
+  console.log('config in MCPSettings', config);
   const [jsonEditorOpen, setJsonEditorOpen] = useState(false);
   const [serverEditorOpen, setServerEditorOpen] = useState(false);
   const [editingServer, setEditingServer] = useState<MCPServer | undefined>(undefined);
@@ -217,19 +218,20 @@ export function MCPSettings({ config, onConfigChange }: MCPSettingsProps) {
 
   const handleSaveServer = (server: MCPServer) => {
     if (!pendingConfig) return;
-    
+
     let newServers: MCPServer[];
 
     if (editingServer) {
       // Check if the server actually changed
       const originalServer = pendingConfig.servers.find(s => s.name === editingServer.name);
-      const serverChanged = !originalServer || JSON.stringify(originalServer) !== JSON.stringify(server);
-      
+      const serverChanged =
+        !originalServer || JSON.stringify(originalServer) !== JSON.stringify(server);
+
       if (!serverChanged) {
         // No changes, just close the dialog
         return;
       }
-      
+
       // Update existing server
       newServers = pendingConfig.servers.map(s => (s.name === editingServer.name ? server : s));
     } else {
@@ -374,10 +376,7 @@ export function MCPSettings({ config, onConfigChange }: MCPSettingsProps) {
                       </TableCell>
                       <TableCell align="right">
                         <Tooltip title="Edit">
-                          <IconButton
-                            size="small"
-                            onClick={() => handleOpenServerEditor(server)}
-                          >
+                          <IconButton size="small" onClick={() => handleOpenServerEditor(server)}>
                             <Icon icon="mdi:pencil" />
                           </IconButton>
                         </Tooltip>

ai-assistant/src/langchain/LangChainManager.ts

@@ -169,14 +169,17 @@ export default class LangChainManager extends AIManager {
       const assistantPrompt: Prompt = {
         role: 'assistant',
         content: fullContent,
-        toolCalls: toolCalls.length > 0 ? toolCalls.map(tc => ({
-          type: 'function',
-          id: tc.id,
-          function: {
-            name: tc.name,
-            arguments: JSON.stringify(tc.args || {}),
-          },
-        })) : undefined,
+        toolCalls:
+          toolCalls.length > 0
+            ? toolCalls.map(tc => ({
+                type: 'function',
+                id: tc.id,
+                function: {
+                  name: tc.name,
+                  arguments: JSON.stringify(tc.args || {}),
+                },
+              }))
+            : undefined,
       };
 
       // If there are tool calls, handle them with streaming
@@ -209,7 +212,7 @@ export default class LangChainManager extends AIManager {
         }
 
         // Clear progress steps for non-tool responses
-          }
+      }
 
       return assistantPrompt;
     } catch (error) {
@@ -864,7 +867,7 @@ The user is waiting for you to explain what the tools discovered. Provide a dire
         this.history.push(assistantPrompt);
 
         // Clear progress steps for non-tool responses
-    
+
         return assistantPrompt;
       }
     } catch (error) {
@@ -1103,12 +1106,10 @@ The user is waiting for you to explain what the tools discovered. Provide a dire
         })
       );
 
-      let approvedToolIds: string[] = [];
+      const approvedToolIds: string[] = [];
 
       // Separate built-in tools from MCP tools (same pattern as handleToolCalls)
-      const builtInToolsForApproval = toolsForApproval.filter(tool =>
-        isBuiltInTool(tool.name)
-      );
+      const builtInToolsForApproval = toolsForApproval.filter(tool => isBuiltInTool(tool.name));
       const mcpToolsForApproval = toolsForApproval.filter(tool => !isBuiltInTool(tool.name));
 
       // Auto-approve all built-in tools (no user interaction needed)
@@ -1141,9 +1142,7 @@ The user is waiting for you to explain what the tools discovered. Provide a dire
       // Match by checking if the approved ID contains the tool name as a suffix
       const approvedTools = recommendedTools.filter(tool => {
         const expectedIdPrefix = `orchestrated-${tool.name}-`;
-        return approvedToolIds.some(
-          id => id === tool.name || id.startsWith(expectedIdPrefix)
-        );
+        return approvedToolIds.some(id => id === tool.name || id.startsWith(expectedIdPrefix));
       });
 
       // Group tools by execution strategy (parallel vs sequential)
@@ -1411,7 +1410,7 @@ ${Object.entries(toolResults)
       this.history.push(assistantPrompt);
 
       // Clear progress steps when all tools are disabled
-  
+
       return assistantPrompt;
     }
 
@@ -1464,12 +1463,12 @@ Without access to the Kubernetes API, I cannot fetch current pod, deployment, se
         this.history[this.history.length - 1] = updatedPrompt;
 
         // Clear progress steps when tools are disabled
-    
+
         return updatedPrompt;
       }
 
       // Clear progress steps when no tools to execute
-  
+
       return assistantPrompt;
     }
 
@@ -1625,7 +1624,10 @@ Without access to the Kubernetes API, I cannot fetch current pod, deployment, se
    * Handle tool calls for streaming scenario - executes tools without generating response
    * (response will be streamed separately by processToolResponsesStream)
    */
-  private async handleToolCallsForStreaming(toolCalls: any[], assistantPrompt: Prompt): Promise<void> {
+  private async handleToolCallsForStreaming(
+    toolCalls: any[],
+    assistantPrompt: Prompt
+  ): Promise<void> {
     const enabledToolIds = this.toolManager.getToolNames();
 
     // Convert tool calls to expected format
@@ -1699,10 +1701,7 @@ Without access to the Kubernetes API, I cannot fetch current pod, deployment, se
 
       // Request approval for MCP tools
       if (mcpTools.length > 0) {
-        const approvedMCPToolIds = await inlineToolApprovalManager.requestApproval(
-          mcpTools,
-          this
-        );
+        const approvedMCPToolIds = await inlineToolApprovalManager.requestApproval(mcpTools, this);
         approvedToolIds.push(...approvedMCPToolIds);
       }
 
@@ -2097,14 +2096,14 @@ Format your response to make the errors prominent and actionable.`,
       this.history.push(assistantPrompt);
 
       // Clear progress steps after streaming response
-  
+
       return assistantPrompt;
     } catch (error) {
       const errorPrompt = this.handleToolResponseError(error);
       yield errorPrompt.content;
 
       // Clear progress steps even on error
-  
+
       return errorPrompt;
     }
   }
@@ -2790,7 +2789,9 @@ Return the complete arguments object:`;
             key.toLowerCase().includes('max'))
         ) {
           sanitized[key] = parseFloat(value);
-          console.log(`[ArgSanitize] Converted ${key} from string "${value}" to number ${sanitized[key]}`);
+          console.log(
+            `[ArgSanitize] Converted ${key} from string "${value}" to number ${sanitized[key]}`
+          );
           continue;
         }
       }

ai-assistant/src/langchain/tools/ToolOrchestrator.ts

@@ -42,10 +42,7 @@ const ToolRecommendationSchema = z.object({
           tool_name: z.string().optional().describe('Alternative field for tool name'),
           description: z.string().describe('What this tool will do'),
           arguments: z
-            .union([
-              z.record(z.string(), z.any()),
-              z.string(),
-            ])
+            .union([z.record(z.string(), z.any()), z.string()])
             .default({})
             .describe('Arguments needed for this tool'),
           priority: z

ai-assistant/src/utils.tsx

@@ -7,10 +7,7 @@ import type {
 import React from 'react';
 import { useBetween } from 'use-between';
 import { SavedConfigurations, StoredProviderConfig } from './utils/ProviderConfigManager';
-import {
-  getAllAvailableToolsIncludingMCP,
-  initializeToolsState,
-} from './utils/ToolConfigManager';
+import { getAllAvailableToolsIncludingMCP, initializeToolsState } from './utils/ToolConfigManager';
 
 export const PLUGIN_NAME = '@headlamp-k8s/ai-assistant';
 export const getSettingsURL = () => `/settings/plugins/${encodeURIComponent(PLUGIN_NAME)}`;