fix(vitest4): migrate tests and configs for vitest 4 compatibility

Author: edelauna

apps/cli/tsconfig.json

@@ -1,7 +1,6 @@
 {
 	"extends": "@roo-code/config-typescript/base.json",
 	"compilerOptions": {
-		"types": ["vitest/globals"],
 		"outDir": "dist",
 		"jsx": "react-jsx",
 		"jsxImportSource": "react",

apps/vscode-e2e/tsconfig.esm.json

@@ -1,7 +1,8 @@
 {
 	"extends": "@roo-code/config-typescript/base.json",
 	"compilerOptions": {
-		"outDir": "out"
+		"outDir": "out",
+		"types": ["node"]
 	},
 	"include": ["src"],
 	"exclude": ["node_modules"]

packages/build/tsconfig.json

@@ -1,7 +1,6 @@
 {
 	"extends": "@roo-code/config-typescript/base.json",
 	"compilerOptions": {
-		"types": ["vitest/globals"],
 		"outDir": "dist"
 	},
 	"include": ["src"],

packages/cloud/src/__mocks__/vscode.ts

@@ -1,19 +1,19 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 
-export const window = {
+export const window: { showInformationMessage: (...args: any[]) => any; showErrorMessage: (...args: any[]) => any } = {
 	showInformationMessage: vi.fn(),
 	showErrorMessage: vi.fn(),
 }
 
-export const env = {
+export const env: { openExternal: (...args: any[]) => any } = {
 	openExternal: vi.fn(),
 }
 
 export const Uri = {
 	parse: vi.fn((uri: string) => ({ toString: () => uri })),
 }
 
-export const commands = {
+export const commands: { executeCommand: (...args: any[]) => any } = {
 	executeCommand: vi.fn().mockResolvedValue(undefined),
 }
 

packages/cloud/src/__tests__/CloudService.test.ts

@@ -147,13 +147,21 @@ describe("CloudService", () => {
 			backfillMessages: vi.fn().mockResolvedValue(undefined),
 		}
 
-		vi.mocked(WebAuthService).mockImplementation(() => mockAuthService as unknown as WebAuthService)
+		vi.mocked(WebAuthService).mockImplementation(function () {
+			return mockAuthService as unknown as WebAuthService
+		})
 
-		vi.mocked(CloudSettingsService).mockImplementation(() => mockSettingsService as unknown as CloudSettingsService)
+		vi.mocked(CloudSettingsService).mockImplementation(function () {
+			return mockSettingsService as unknown as CloudSettingsService
+		})
 
-		vi.mocked(CloudShareService).mockImplementation(() => mockShareService as unknown as CloudShareService)
+		vi.mocked(CloudShareService).mockImplementation(function () {
+			return mockShareService as unknown as CloudShareService
+		})
 
-		vi.mocked(TelemetryClient).mockImplementation(() => mockTelemetryClient as unknown as TelemetryClient)
+		vi.mocked(TelemetryClient).mockImplementation(function () {
+			return mockTelemetryClient as unknown as TelemetryClient
+		})
 	})
 
 	afterEach(() => {
@@ -417,7 +425,9 @@ describe("CloudService", () => {
 			})
 
 			// Override the mock to return our properly typed instance
-			vi.mocked(CloudSettingsService).mockImplementation(() => mockCloudSettingsService)
+			vi.mocked(CloudSettingsService).mockImplementation(function () {
+				return mockCloudSettingsService
+			})
 
 			const cloudService = await CloudService.createInstance(mockContext)
 
@@ -450,9 +460,9 @@ describe("CloudService", () => {
 			}
 
 			// Override the mock to return a service that won't pass instanceof check
-			vi.mocked(CloudSettingsService).mockImplementation(
-				() => mockStaticSettingsService as unknown as CloudSettingsService,
-			)
+			vi.mocked(CloudSettingsService).mockImplementation(function () {
+				return mockStaticSettingsService as unknown as CloudSettingsService
+			})
 
 			// This should not throw even though the service doesn't pass instanceof check
 			const _cloudService = await CloudService.createInstance(mockContext)

packages/cloud/src/__tests__/CloudSettingsService.test.ts

@@ -11,7 +11,7 @@ vi.mock("../config", () => ({
 	getRooCodeApiUrl: vi.fn().mockReturnValue("https://app.roocode.com"),
 }))
 
-global.fetch = vi.fn()
+global.fetch = vi.fn() as unknown as typeof fetch
 
 describe("CloudSettingsService", () => {
 	let mockContext: ExtensionContext
@@ -27,7 +27,7 @@ describe("CloudSettingsService", () => {
 		stop: ReturnType<typeof vi.fn>
 	}
 	let cloudSettingsService: CloudSettingsService
-	let mockLog: ReturnType<typeof vi.fn>
+	let mockLog: (...args: unknown[]) => void
 
 	const mockSettings: OrganizationSettings = {
 		version: 1,
@@ -75,7 +75,9 @@ describe("CloudSettingsService", () => {
 		mockLog = vi.fn()
 
 		// Mock RefreshTimer constructor
-		vi.mocked(RefreshTimer).mockImplementation(() => mockRefreshTimer as unknown as RefreshTimer)
+		vi.mocked(RefreshTimer).mockImplementation(function () {
+			return mockRefreshTimer as unknown as RefreshTimer
+		})
 
 		cloudSettingsService = new CloudSettingsService(mockContext, mockAuthService as unknown as AuthService, mockLog)
 	})

packages/cloud/src/__tests__/WebAuthService.spec.ts

@@ -97,7 +97,9 @@ describe("WebAuthService", () => {
 			reset: vi.fn(),
 		}
 		const MockedRefreshTimer = vi.mocked(RefreshTimer)
-		MockedRefreshTimer.mockImplementation(() => mockTimer as unknown as RefreshTimer)
+		MockedRefreshTimer.mockImplementation(function () {
+			return mockTimer as unknown as RefreshTimer
+		})
 
 		// Setup config mocks - use production URL by default to maintain existing test behavior
 		vi.mocked(getClerkBaseUrl).mockReturnValue("https://clerk.roocode.com")

packages/cloud/src/retry-queue/__tests__/RetryQueue.test.ts

@@ -303,7 +303,7 @@ describe("RetryQueue", () => {
 		beforeEach(() => {
 			// Mock global fetch
 			fetchMock = vi.fn()
-			global.fetch = fetchMock
+			global.fetch = fetchMock as unknown as typeof fetch
 		})
 
 		afterEach(() => {

packages/cloud/tsconfig.json

@@ -1,7 +1,6 @@
 {
 	"extends": "@roo-code/config-typescript/base.json",
 	"compilerOptions": {
-		"types": ["vitest/globals", "node"],
 		"outDir": "./dist"
 	},
 	"include": ["src", "scripts", "*.config.ts"],

packages/config-typescript/base.json

@@ -15,6 +15,6 @@
 		"skipLibCheck": true,
 		"strict": true,
 		"target": "ES2022",
-		"types": ["node"]
+		"types": ["node", "vitest/globals"]
 	}
 }