fix(agenticChat): UX fixes for MCP (#1661)

* fix: setting mute as false for mcp tools

* fix: set muted as false for mcp

Author: laileni-aws

chat-client/src/client/mynahUi.ts

@@ -995,11 +995,9 @@ export const createMynahUi = (
      * Creates a properly formatted chat item for MCP tool summary with accordion view
      */
     const createMcpToolSummaryItem = (message: ChatMessage, isPartialResult?: boolean): Partial<ChatItem> => {
-        const muted = message.summary?.content?.header?.status !== undefined
         return {
             type: ChatItemType.ANSWER,
             messageId: message.messageId,
-            muted,
             summary: {
                 content: message.summary?.content
                     ? {
@@ -1028,7 +1026,7 @@ export const createMynahUi = (
                             : undefined,
                         fullWidth: true,
                         padding: false,
-                        muted: true,
+                        muted: false,
                         wrapCodes: item.header?.body === 'Parameters' ? true : false,
                         codeBlockActions: { copy: null, 'insert-to-cursor': null },
                     })) || [],

server/aws-lsp-codewhisperer/src/language-server/agenticChat/agenticChatController.ts

@@ -941,14 +941,18 @@ export class AgenticChatController implements ChatHandlers {
         toolUse: ToolUse,
         resultStream: AgenticChatResultStream,
         promptBlockId: number,
-        session: ChatSessionService
+        session: ChatSessionService,
+        toolName: string
     ) {
         const deferred = this.#createDeferred()
         session.setDeferredToolExecution(toolUse.toolUseId!, deferred.resolve, deferred.reject)
-        this.#log(`Prompting for tool approval for tool: ${toolUse.name}`)
+        this.#log(`Prompting for tool approval for tool: ${toolName ?? toolUse.name}`)
         await deferred.promise
         // Note: we want to overwrite the button block because it already exists in the stream.
-        await resultStream.overwriteResultBlock(this.#getUpdateToolConfirmResult(toolUse, true), promptBlockId)
+        await resultStream.overwriteResultBlock(
+            this.#getUpdateToolConfirmResult(toolUse, true, toolName),
+            promptBlockId
+        )
     }
 
     /**
@@ -1054,7 +1058,13 @@ export class AgenticChatController implements ChatHandlers {
                                 )
                             }
                             if (requiresAcceptance) {
-                                await this.waitForToolApproval(toolUse, chatResultStream, cachedButtonBlockId, session)
+                                await this.waitForToolApproval(
+                                    toolUse,
+                                    chatResultStream,
+                                    cachedButtonBlockId,
+                                    session,
+                                    toolUse.name
+                                )
                             }
                             if (isExecuteBash) {
                                 this.#telemetryController.emitInteractWithAgenticChat(
@@ -1098,7 +1108,8 @@ export class AgenticChatController implements ChatHandlers {
                                         toolUse,
                                         chatResultStream,
                                         cachedButtonBlockId,
-                                        session
+                                        session,
+                                        toolName
                                     )
                                 }
 
@@ -1230,7 +1241,7 @@ export class AgenticChatController implements ChatHandlers {
                     // Handle ToolApprovalException for any tool
                     if (err instanceof ToolApprovalException && cachedButtonBlockId) {
                         await chatResultStream.overwriteResultBlock(
-                            this.#getUpdateToolConfirmResult(toolUse, false),
+                            this.#getUpdateToolConfirmResult(toolUse, false, toolUse.name),
                             cachedButtonBlockId
                         )
                         if (err.shouldShowMessage) {
@@ -1525,8 +1536,13 @@ export class AgenticChatController implements ChatHandlers {
      * @param toolType Optional tool type for specialized handling
      * @returns ChatResult with appropriate confirmation UI
      */
-    #getUpdateToolConfirmResult(toolUse: ToolUse, isAccept: boolean, toolType?: string): ChatResult {
-        const toolName = toolType || toolUse.name
+    #getUpdateToolConfirmResult(
+        toolUse: ToolUse,
+        isAccept: boolean,
+        originalToolName: string,
+        toolType?: string
+    ): ChatResult {
+        const toolName = originalToolName ?? (toolType || toolUse.name)
 
         // Handle bash commands with special formatting
         if (toolName === 'executeBash') {
@@ -1585,15 +1601,15 @@ export class AgenticChatController implements ChatHandlers {
                 break
 
             default:
-                // Default tool (not MCP)
+                // Default tool (not only MCP)
                 return {
                     type: 'tool',
                     messageId: toolUse.toolUseId!,
                     summary: {
                         content: {
                             header: {
                                 icon: 'tools',
-                                body: `${toolUse.name}`,
+                                body: `${toolName}`,
                                 status: {
                                     status: isAccept ? 'success' : 'error',
                                     icon: isAccept ? 'ok' : 'cancel',