fix(lambda): redirect view logs action to terminal (aws#6103)

## Problem

When a build error occurs, the error details are correctly shown under
the "Terminal" tab. However, the toast notification says "build failed"
with a button to "See logs." Clicking this button incorrectly redirects
the user to the "Output" tab, which only shows a generic message (Error:
sam build exited with a non-zero exit code: 1 [NonZeroExitCode]),
instead of the detailed logs from the "Terminal" tab. This creates
confusion and makes troubleshooting more difficult.

## Solution
Redirect view logs action to open the terminal instead of output channel

---

<!--- REMINDER: Ensure that your PR meets the guidelines in
CONTRIBUTING.md -->

License: I confirm that my contribution is made under the terms of the
Apache 2.0 license.

Author: mbfreder

packages/core/src/shared/sam/build.ts

@@ -19,7 +19,14 @@ import globals from '../extensionGlobals'
 import { TreeNode } from '../treeview/resourceTreeDataProvider'
 import { telemetry } from '../telemetry/telemetry'
 import { getSpawnEnv } from '../env/resolveEnv'
-import { getErrorCode, getProjectRoot, getSamCliPathAndVersion, isDotnetRuntime, updateRecentResponse } from './utils'
+import {
+    getErrorCode,
+    getProjectRoot,
+    getSamCliPathAndVersion,
+    getTerminalFromError,
+    isDotnetRuntime,
+    updateRecentResponse,
+} from './utils'
 import { getConfigFileUri, validateSamBuildConfig } from './config'
 import { runInTerminal } from './processTerminal'
 
@@ -240,7 +247,7 @@ export async function runBuild(arg?: TreeNode): Promise<SamBuildResult> {
         }
     } catch (error) {
         throw ToolkitError.chain(error, 'Failed to build SAM template', {
-            details: { ...resolveBuildArgConflict(buildFlags) },
+            details: { terminal: getTerminalFromError(error), ...resolveBuildArgConflict(buildFlags) },
             code: getErrorCode(error),
         })
     }

packages/core/src/shared/sam/processTerminal.ts

@@ -16,14 +16,15 @@ import { throwIfErrorMatches } from './utils'
 
 let oldTerminal: ProcessTerminal | undefined
 export async function runInTerminal(proc: ChildProcess, cmd: string) {
-    const handleResult = (result?: ChildProcessResult) => {
+    const handleResult = (result?: ChildProcessResult, terminal?: vscode.Terminal) => {
         if (result && result.exitCode !== 0) {
-            throwIfErrorMatches(result)
+            throwIfErrorMatches(result, terminal)
 
             // If no specific error matched, throw the default non-zero exit code error.
             const defaultMessage = `sam ${cmd} exited with a non-zero exit code: ${result.exitCode}`
             throw ToolkitError.chain(result.error, defaultMessage, {
                 code: 'NonZeroExitCode',
+                details: { terminal: terminal },
             })
         }
     }
@@ -56,7 +57,7 @@ export async function runInTerminal(proc: ChildProcess, cmd: string) {
             ? ToolkitError.chain(result.error, 'SAM CLI was cancelled before exiting', { cancelled: true })
             : new CancellationError('user')
     } else {
-        return handleResult(result)
+        return handleResult(result, terminal)
     }
 }
 

packages/core/src/shared/sam/utils.ts

@@ -121,17 +121,23 @@ export function getErrorCode(error: unknown): string | undefined {
     return error instanceof ToolkitError ? error.code : undefined
 }
 
-export function throwIfErrorMatches(result: ChildProcessResult) {
+export function throwIfErrorMatches(result: ChildProcessResult, terminal?: vscode.Terminal) {
     const errorMessage = getSamCliErrorMessage(result.stderr)
     for (const errorType in SamCliErrorTypes) {
         if (errorMessage.includes(SamCliErrorTypes[errorType as keyof typeof SamCliErrorTypes])) {
             throw ToolkitError.chain(result.error, errorMessage, {
                 code: errorType,
+                details: { terminal: terminal },
             })
         }
     }
 }
 
+export function getTerminalFromError(error: any): vscode.Terminal {
+    return error.details?.['terminal'] as unknown as vscode.Terminal
+    //return vscode.window.activeTerminal as vscode.Terminal
+}
+
 export enum SamCliErrorTypes {
     DockerUnreachable = 'Docker is unreachable.',
     ResolveS3AndS3Set = 'Cannot use both --resolve-s3 and --s3-bucket parameters in non-guided deployments.',

packages/core/src/shared/utilities/logAndShowUtils.ts

@@ -32,11 +32,18 @@ export async function logAndShowError(
         return
     }
     const logsItem = localize('AWS.generic.message.viewLogs', 'View Logs...')
+    const viewInTerminalItem = localize('AWS.generic.message.viewInTerminal', 'View Logs In Terminal')
     const logId = getLogger().error(`${topic}: %s`, error)
     const message = resolveErrorMessageToDisplay(error, defaultMessage)
 
     if (error instanceof ToolkitError && error.documentationUri) {
         await showMessageWithUrl(message, error.documentationUri, 'View Documentation', 'error')
+    } else if (error instanceof ToolkitError && (error.details?.['terminal'] as unknown as vscode.Terminal)) {
+        await vscode.window.showErrorMessage(message, viewInTerminalItem).then(async (resp) => {
+            if (resp === viewInTerminalItem) {
+                ;(error.details?.['terminal'] as unknown as vscode.Terminal).show()
+            }
+        })
     } else {
         await vscode.window.showErrorMessage(message, logsItem).then(async (resp) => {
             if (resp === logsItem) {

packages/core/src/test/shared/sam/build.test.ts

@@ -515,7 +515,7 @@ describe('SAM runBuild', () => {
             }
         })
 
-        it('should throw ToolkitError when sync command fail', async () => {
+        it('should throw ToolkitError when build command fail', async () => {
             const prompterTester = PrompterTester.init()
                 .handleQuickPick('Specify parameter source for build', async (quickPick) => {
                     await quickPick.untilReady()
@@ -532,7 +532,7 @@ describe('SAM runBuild', () => {
                     value: sandbox.stub().resolves({
                         exitCode: -1,
                         stdout: 'Mock build command execution failure',
-                        stderr: '',
+                        stderr: 'Docker is unreachable.',
                     }),
                 },
             })
@@ -544,6 +544,8 @@ describe('SAM runBuild', () => {
             } catch (error: any) {
                 assert(error instanceof ToolkitError)
                 assert.strictEqual(error.message, 'Failed to build SAM template')
+                assert(error.details?.['terminal'] as unknown as vscode.Terminal)
+                assert.strictEqual((error.details?.['terminal'] as unknown as vscode.Terminal).name, 'SAM build')
             }
             prompterTester.assertCallAll()
         })

packages/toolkit/.changes/next-release/Bug Fix-85d04649-0aaa-4e00-8b6f-1c6885012932.json

@@ -0,0 +1,4 @@
+{
+	"type": "Bug Fix",
+	"description": "'View Logs...' action redirects to output channel instead of Terminal"
+}