refactor

Author: abhraina-aws

packages/amazonq/src/lsp/chat/messages.ts

@@ -435,8 +435,9 @@ export function registerMessageListeners(
             case conversationClickRequestType.method:
             case listMcpServersRequestType.method:
             case mcpServerClickRequestType.method:
-            case tabBarActionRequestType.method:
-                // Special handling for tab bar actions
+            case tabBarActionRequestType.method: {
+                let success = false
+                // handling for show_logs button
                 if (message.params.action === 'show_logs') {
                     languageClient.info('[VSCode Client] Received show_logs action, showing disclaimer')
 
@@ -451,23 +452,22 @@ export function registerMessageListeners(
                         // Open the log directory in the OS file explorer directly
                         languageClient.info('[VSCode Client] Opening logs directory')
                         await vscode.commands.executeCommand('revealFileInOS', vscode.Uri.file(logPath))
-                        const result = { ...message.params, success: true }
-                        void webview?.postMessage({
-                            command: message.command,
-                            params: result,
-                        })
+                        success = true
                     } else {
                         // Fallback: show error if log path is not available
                         void vscode.window.showErrorMessage('Log location not available.')
                         languageClient.error('[VSCode Client] Log location not available')
-                        const result = { ...message.params, success: false }
-                        void webview?.postMessage({
-                            command: message.command,
-                            params: result,
-                        })
+                        success = false
                     }
+
+                    void webview?.postMessage({
+                        command: message.command,
+                        params: { ...message.params, success: success },
+                    })
+
                     break
                 }
+            }
             // eslint-disable-next-line no-fallthrough
             case listAvailableModelsRequestType.method:
                 await resolveChatResponse(message.command, message.params, languageClient, webview)