Fix agent deletion to handle URL parameters and improve logging

- Add defensive programming to extract agent name from URL when passed to confirmDeleteAgent
- Prevent default event behavior during agent deletion confirmation
- Add comprehensive console logging throughout deletion process for debugging
- Fix code formatting issues (concatenated lines properly separated)
- Improve error handling with clearer error messages
- Add explicit checks for URL detection and agent name extraction

Author: anshrma

application_src/ui-react-cloudscape/src/components/AgentMapping.js

@@ -349,27 +349,65 @@ const AgentMapping = ({
     setShowDeleteConfirmation(true);
   };
 
-  const confirmDeleteAgent = async () => {
+  const confirmDeleteAgent = async (event) => {
+    // Prevent default form submission or page refresh
+    if (event) {
+      event.preventDefault();
+      event.stopPropagation();
+    }
+    
     if (!selectedAgentForDeletion) return;
 
+    // Extract agent name if URL was passed (defensive programming)
+    let agentNameToDelete = selectedAgentForDeletion;
+    
+    // If selectedAgentForDeletion looks like a URL, extract the agent name from agentMappingData
+    if (agentNameToDelete.startsWith('http://') || agentNameToDelete.startsWith('https://')) {
+      console.log('[AgentMapping] URL detected, extracting agent name from mapping data');
+      if (agentMappingData?.agent_mapping?.[agentNameToDelete]) {
+        agentNameToDelete = agentMappingData.agent_mapping[agentNameToDelete].agent_name;
+        console.log('[AgentMapping] Extracted agent name:', agentNameToDelete);
+      } else {
+        setError(`Cannot delete agent: unable to determine agent name from URL ${agentNameToDelete}`);
+        setIsDeletingAgent(false);
+        setSelectedAgentForDeletion(null);
+        return;
+      }
+    }
+    
+    console.log('[AgentMapping] Starting agent deletion:', agentNameToDelete);
     setIsDeletingAgent(true);
     setShowDeleteConfirmation(false);
 
     try {
       const configService = await import('../services/configuration');
 
-      // Step 1: Initiate agent deletion and wait for completionconst deletionResult = await configService.default.deleteAgentComplete(selectedAgentForDeletion, true);// Step 2: Wait a moment for infrastructure cleanup to propagateawait new Promise(resolve => setTimeout(resolve, 2000)); // 2 second delay
+      console.log('[AgentMapping] Calling deleteAgentComplete API with agent name:', agentNameToDelete);
+      // Step 1: Initiate agent deletion and wait for completion
+      const deletionResult = await configService.default.deleteAgentComplete(agentNameToDelete, true);
+      console.log('[AgentMapping] Delete API response:', deletionResult);
+
+      // Step 2: Wait a moment for infrastructure cleanup to propagate
+      await new Promise(resolve => setTimeout(resolve, 2000)); // 2 second delay
 
       // Step 3: Refresh supervisor cache first
       try {
         await configService.default.refreshSupervisorAgentCache();
       } catch (supervisorError) {
+        console.warn('[AgentMapping] Supervisor refresh failed:', supervisorError);
         // Continue even if supervisor refresh fails
       }
 
-      // Step 4: Refresh the local agent listawait onRefreshAgents();
+      // Step 4: Refresh the local agent list
+      await onRefreshAgents();
 
-      // Step 5: Finally reload the agent mapping to reflect changesawait loadAgentMappings();} catch (error) {setError(`Failed to delete agent "${selectedAgentForDeletion}": ${error.message}`);
+      // Step 5: Finally reload the agent mapping to reflect changes
+      await loadAgentMappings();
+      
+      console.log('[AgentMapping] Agent deletion completed successfully');
+    } catch (error) {
+      console.error('[AgentMapping] Agent deletion failed:', error);
+      setError(`Failed to delete agent "${selectedAgentForDeletion}": ${error.message}`);
     } finally {
       setIsDeletingAgent(false);
       setSelectedAgentForDeletion(null);

application_src/ui-react-cloudscape/src/components/AgentWizard.js

@@ -1080,7 +1080,8 @@ const AgentWizard = ({
     }
   };
 
-  const handleSaveConfiguration = async () => {// Use the same structure as handleCreateAgent to ensure all required fields are included
+  const handleSaveConfiguration = async () => {
+    // Use the same structure as handleCreateAgent to ensure all required fields are included
     const agentConfigRequest = {
       // Basic information - REQUIRED FIELDS - use actual form data
       agent_name: selectedAgentForConfig,
@@ -1091,7 +1092,18 @@ const AgentWizard = ({
       system_prompt_name: agentData.agent_basic_system_prompt_name || agentData.system_prompt_name || `${selectedAgentForConfig}_system_prompt`,
       system_prompt: agentData.agent_basic_system_prompt || agentData.system_prompt || `You are ${selectedAgentForConfig}, ${agentData.agent_basic_agent_description || agentData.agent_description || 'a helpful AI assistant'}`,
 
-      // Extract model configuration dynamically from form fields (same as other components)
+      // Initialize ALL required component fields with defaults - CRITICAL FIX for 422 error
+      memory: 'False',
+      memory_provider: 'default',
+      knowledge_base: 'False',
+      knowledge_base_provider: 'default',
+      knowledge_base_provider_type: 'custom',
+      observability: 'False',
+      observability_provider: 'default',
+      guardrail: 'False',
+      guardrail_provider: 'default',
+      
+      // Model configuration - extract dynamically from form fields
       ...((() => {
         const modelsConfig = {};
         const modelsProvider = agentData['models_provider'] || 'bedrock';