fix for lint errors

Author: ashishrp-aws

packages/core/src/codewhispererChat/clients/chat/v0/chat.ts

@@ -17,7 +17,7 @@ import { UserWrittenCodeTracker } from '../../../../codewhisperer/tracker/userWr
 import { DocumentReference, PromptMessage } from '../../../controllers/chat/model'
 import { FsWriteBackup } from '../../../../codewhispererChat/tools/fsWrite'
 import { randomUUID } from '../../../../shared/crypto'
-import { getLogger } from '../../../../shared'
+import { getLogger } from '../../../../shared/logger'
 
 export type ToolUseWithError = {
     toolUse: ToolUse

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

@@ -5,6 +5,7 @@
 
 import globals from '../extensionGlobals'
 import { CancellationToken, EventEmitter, Event } from 'vscode'
+import { getLogger } from '../logger'
 
 export const timeoutExpiredMessage = 'Timeout token expired'
 export const timeoutCancelledMessage = 'Timeout token cancelled'
@@ -385,7 +386,7 @@ export async function waitTimeout<T, R = void, B extends boolean = true>(
                         }
                     } catch (err) {
                         // If there's an error checking the condition, just continue
-                        console.error('Error checking cancellation condition:', err)
+                        getLogger().debug(`Error disposing token source: ${err}`)
                     }
                 }, checkInterval)
 

packages/core/src/test/codewhispererChat/storages/conversationTracker.test.ts

@@ -66,9 +66,9 @@ describe('ConversationTracker', () => {
             const tokenSource = new vscode.CancellationTokenSource()
 
             // @ts-ignore: Testing with invalid parameters
-            tracker.registerTrigger(null, tokenSource)
+            tracker.registerTrigger(undefined, tokenSource)
             // @ts-ignore: Testing with invalid parameters
-            tracker.registerTrigger('trigger1', null)
+            tracker.registerTrigger('trigger1', undefined)
 
             // @ts-ignore: Accessing private property for testing
             assert.strictEqual(tracker.triggerToToken.size, 0)
@@ -108,7 +108,7 @@ describe('ConversationTracker', () => {
 
             tracker.registerTrigger('trigger1', tokenSource)
             // @ts-ignore: Testing with invalid parameter
-            tracker.markTriggerCompleted(null)
+            tracker.markTriggerCompleted(undefined)
 
             assert.strictEqual(disposeSpy.called, false)
             // @ts-ignore: Accessing private property for testing
@@ -135,7 +135,7 @@ describe('ConversationTracker', () => {
 
         it('should return false if triggerID is missing', () => {
             // @ts-ignore: Testing with invalid parameter
-            const result = tracker.cancelTrigger(null)
+            const result = tracker.cancelTrigger(undefined)
             assert.strictEqual(result, false)
         })
     })
@@ -164,7 +164,7 @@ describe('ConversationTracker', () => {
 
         it('should return 0 if tabID is missing', () => {
             // @ts-ignore: Testing with invalid parameter
-            const result = tracker.cancelTabTriggers(null)
+            const result = tracker.cancelTabTriggers(undefined)
             assert.strictEqual(result, 0)
         })
     })
@@ -196,7 +196,7 @@ describe('ConversationTracker', () => {
 
         it('should return true if triggerID is missing', () => {
             // @ts-ignore: Testing with invalid parameter
-            const result = tracker.isTriggerCancelled(null)
+            const result = tracker.isTriggerCancelled(undefined)
             assert.strictEqual(result, true)
         })
     })
@@ -248,7 +248,7 @@ describe('ConversationTracker', () => {
 
         it('should return 0 if tabID is missing', () => {
             // @ts-ignore: Testing with invalid parameter
-            const result = tracker.clearTabTriggers(null)
+            const result = tracker.clearTabTriggers(undefined)
             assert.strictEqual(result, 0)
         })
     })