Fix linter errors: Replace 'any' types with proper TypeScript types

- Replace 'any' with 'unknown as vscode.ExtensionContext' for mock contexts
- Use vi.mocked() instead of casting to 'any' for mock functions
- Replace 'any' with proper type annotations for function parameters
- Fixes all 16 ESLint warnings about @typescript-eslint/no-explicit-any

Addresses feedback from @cte in PR comment

Author: roomote

packages/cloud/src/TelemetryClient.ts

@@ -29,7 +29,7 @@ export class TelemetryClient extends BaseTelemetryClient {
 	 */
 	public initializeRetryQueue(context: vscode.ExtensionContext): void {
 		this.context = context
-		const retrySettings = context.globalState.get("telemetryRetrySettings") as any
+		const retrySettings = context.globalState.get("telemetryRetrySettings") as Record<string, unknown> | undefined
 		this.retryQueue = new TelemetryRetryQueue(context, retrySettings)
 
 		// Start periodic retry processing

packages/telemetry/src/__tests__/ResilientTelemetryClient.test.ts

@@ -47,7 +47,7 @@ describe("ResilientTelemetryClient", () => {
 				get: vi.fn().mockReturnValue([]),
 				update: vi.fn().mockResolvedValue(undefined),
 			},
-		} as any
+		} as unknown as vscode.ExtensionContext
 
 		resilientClient = new ResilientTelemetryClient(mockWrappedClient, mockContext)
 	})
@@ -97,7 +97,7 @@ describe("ResilientTelemetryClient", () => {
 			}
 
 			// Make wrapped client throw error
-			;(mockWrappedClient.capture as any).mockRejectedValue(new Error("Network error"))
+			vi.mocked(mockWrappedClient.capture).mockRejectedValue(new Error("Network error"))
 
 			await resilientClient.capture(event)
 
@@ -112,7 +112,7 @@ describe("ResilientTelemetryClient", () => {
 			}
 
 			// Make wrapped client fail
-			;(mockWrappedClient.capture as any).mockRejectedValue(new Error("Network error"))
+			vi.mocked(mockWrappedClient.capture).mockRejectedValue(new Error("Network error"))
 
 			await resilientClient.capture(highPriorityEvent)
 
@@ -125,7 +125,7 @@ describe("ResilientTelemetryClient", () => {
 				properties: { taskId: "test" },
 			}
 
-			;(mockWrappedClient.isTelemetryEnabled as any).mockReturnValue(false)
+			vi.mocked(mockWrappedClient.isTelemetryEnabled).mockReturnValue(false)
 
 			await resilientClient.capture(event)
 
@@ -137,7 +137,7 @@ describe("ResilientTelemetryClient", () => {
 
 	describe("delegation methods", () => {
 		it("should delegate setProvider to wrapped client", () => {
-			const mockProvider = {} as any
+			const mockProvider = {} as Parameters<typeof mockWrappedClient.setProvider>[0]
 			resilientClient.setProvider(mockProvider)
 
 			expect(mockWrappedClient.setProvider).toHaveBeenCalledWith(mockProvider)
@@ -157,7 +157,7 @@ describe("ResilientTelemetryClient", () => {
 		})
 
 		it("should return subscription from wrapped client", () => {
-			const mockSubscription = { type: "exclude", events: [] } as any
+			const mockSubscription = { type: "exclude", events: [] } as typeof mockWrappedClient.subscription
 			mockWrappedClient.subscription = mockSubscription
 
 			expect(resilientClient.subscription).toBe(mockSubscription)
@@ -219,7 +219,7 @@ describe("ResilientTelemetryClient", () => {
 				}
 
 				// Make wrapped client fail to trigger queueing
-				;(mockWrappedClient.capture as any).mockRejectedValue(new Error("Network error"))
+				vi.mocked(mockWrappedClient.capture).mockRejectedValue(new Error("Network error"))
 
 				await resilientClient.capture(event)
 

packages/telemetry/src/__tests__/TelemetryRetryQueue.test.ts

@@ -38,7 +38,7 @@ describe("TelemetryRetryQueue", () => {
 				get: vi.fn().mockReturnValue([]),
 				update: vi.fn().mockResolvedValue(undefined),
 			},
-		} as any
+		} as unknown as vscode.ExtensionContext
 
 		retryQueue = new TelemetryRetryQueue(mockContext)
 	})
@@ -97,7 +97,7 @@ describe("TelemetryRetryQueue", () => {
 			await retryQueue.enqueue(highEvent, "high")
 
 			// High priority event should be inserted before normal priority
-			const calls = (mockContext.globalState.update as any).mock.calls
+			const calls = vi.mocked(mockContext.globalState.update).mock.calls
 			const lastCall = calls[calls.length - 1]
 			const queue = lastCall[1]
 
@@ -131,7 +131,7 @@ describe("TelemetryRetryQueue", () => {
 			}
 
 			// Mock existing queue with one event
-			;(mockContext.globalState.get as any).mockReturnValue([
+			vi.mocked(mockContext.globalState.get).mockReturnValue([
 				{
 					id: "test-id",
 					event,
@@ -165,15 +165,15 @@ describe("TelemetryRetryQueue", () => {
 				priority: "normal",
 			}
 
-			;(mockContext.globalState.get as any).mockReturnValue([queuedEvent])
+			vi.mocked(mockContext.globalState.get).mockReturnValue([queuedEvent])
 
 			const sendFunction = vi.fn().mockResolvedValue(false) // Failure
 
 			await retryQueue.processQueue(sendFunction)
 
 			expect(sendFunction).toHaveBeenCalledWith(event)
 
-			const updateCalls = (mockContext.globalState.update as any).mock.calls
+			const updateCalls = vi.mocked(mockContext.globalState.update).mock.calls
 			const lastCall = updateCalls[updateCalls.length - 1]
 			const updatedQueue = lastCall[1]
 
@@ -196,7 +196,7 @@ describe("TelemetryRetryQueue", () => {
 				priority: "normal",
 			}
 
-			;(mockContext.globalState.get as any).mockReturnValue([queuedEvent])
+			vi.mocked(mockContext.globalState.get).mockReturnValue([queuedEvent])
 
 			const sendFunction = vi.fn().mockResolvedValue(false) // Failure
 
@@ -218,7 +218,7 @@ describe("TelemetryRetryQueue", () => {
 				priority: "normal" as const,
 			}))
 
-			;(mockContext.globalState.get as any).mockReturnValue(events)
+			vi.mocked(mockContext.globalState.get).mockReturnValue(events)
 
 			const sendFunction = vi.fn().mockResolvedValue(true)
 
@@ -236,7 +236,7 @@ describe("TelemetryRetryQueue", () => {
 				{ id: "2", event: {}, timestamp: 0, retryCount: 0, nextRetryAt: 0, priority: "normal" },
 			]
 
-			;(mockContext.globalState.get as any).mockReturnValue(events)
+			vi.mocked(mockContext.globalState.get).mockReturnValue(events)
 
 			const size = await retryQueue.getQueueSize()
 			expect(size).toBe(2)