fix(chat): Add more logs and permission fix

Author: jguoamz

packages/core/src/codewhispererChat/controllers/chat/messenger/messenger.ts

@@ -250,9 +250,11 @@ export class Messenger {
                             const actionId =
                                 tool.type === ToolType.ExecuteBash ? 'run-shell-command' : 'generic-tool-execution'
 
-                            this.dispatcher.sendCustomFormActionMessage(
-                                new CustomFormActionMessage(tabID, { id: actionId })
-                            )
+                            if (!validation.requiresAcceptance) {
+                                this.dispatcher.sendCustomFormActionMessage(
+                                    new CustomFormActionMessage(tabID, { id: actionId })
+                                )
+                            }
                         } else {
                             // TODO: Handle the error
                         }

packages/core/src/codewhispererChat/tools/chatStream.ts

@@ -32,21 +32,31 @@ export class ChatStream extends Writable {
     }
 
     override _write(chunk: Buffer, encoding: BufferEncoding, callback: (error?: Error | null) => void): void {
-        const text = chunk.toString()
-        this.accumulatedLogs += text
-        this.logger.debug(`ChatStream received chunk: ${text}`)
-        this.messenger.sendPartialToolLog(
-            this.accumulatedLogs,
-            this.tabID,
-            this.triggerID,
-            this.toolUse,
-            this.validation,
-            this.changeList
-        )
-        callback()
+        try {
+            const text = chunk.toString()
+            this.accumulatedLogs += text
+            this.logger.debug(`ChatStream received chunk: ${text}`)
+            this.messenger.sendPartialToolLog(
+                this.accumulatedLogs,
+                this.tabID,
+                this.triggerID,
+                this.toolUse,
+                this.validation,
+                this.changeList
+            )
+            callback()
+        } catch (error) {
+            this.logger.error(`Error in ChatStream.write: ${error}`)
+            callback(error instanceof Error ? error : new Error(String(error)))
+        }
     }
 
     override _final(callback: (error?: Error | null) => void): void {
-        callback()
+        try {
+            callback()
+        } catch (error) {
+            this.logger.error(`Error in ChatStream.final: ${error}`)
+            callback(error instanceof Error ? error : new Error(String(error)))
+        }
     }
 }

packages/core/src/codewhispererChat/tools/fsRead.ts

@@ -50,15 +50,15 @@ export class FsRead {
     public queueDescription(updates: Writable): void {
         const fileName = path.basename(this.fsPath)
         const fileUri = vscode.Uri.file(this.fsPath)
-        updates.write(`Reading file: [${fileName}](${fileUri}), `)
+        updates.write(`Reading file: [${fileName}](${fileUri}) `)
 
         const [start, end] = this.readRange ?? []
 
         if (start && end) {
-            updates.write(`from line ${start} to ${end}`)
+            updates.write(`L${start} to L${end}`)
         } else if (start) {
             if (start > 0) {
-                updates.write(`from line ${start} to end of file`)
+                updates.write(`from L${start} to end of file`)
             } else {
                 updates.write(`${start} line from the end of file to end of file`)
             }

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

@@ -13,7 +13,7 @@ export interface MessageErrorInfo {
 }
 
 export function extractErrorInfo(error: any): MessageErrorInfo {
-    let errorMessage = 'Error reading chat stream.'
+    let errorMessage = 'Error reading chat stream: ' + error.message
     let statusCode = undefined
     let requestId = undefined