chore(deps): update dependency vitest to v4 [security] (#443)

* chore(deps): update dependency vitest to v4 [security]

* fix(deps): co-bump @vitest/coverage-v8 and @vitest/ui to v4.1.0

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

* fixup! fix(deps): co-bump @vitest/coverage-v8 and @vitest/ui to v4.1.0

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

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Elliott de Launay <edelauna@gmail.com>

Author: renovate[bot]

apps/cli/package.json

@@ -43,11 +43,11 @@
 		"@roo-code/config-typescript": "workspace:^",
 		"@types/node": "24.2.1",
 		"@types/react": "18.3.23",
-		"@vitest/coverage-v8": "3.2.6",
+		"@vitest/coverage-v8": "4.1.0",
 		"ink-testing-library": "4.0.0",
 		"rimraf": "6.0.1",
 		"tsup": "8.5.0",
 		"tsx": "4.19.4",
-		"vitest": "3.2.6"
+		"vitest": "4.1.0"
 	}
 }

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/cli/vitest.config.ts

@@ -16,6 +16,7 @@ export default defineConfig({
 		coverage: {
 			provider: "v8",
 			reporter: ["text", "lcov"],
+			include: ["src/**/*.ts", "src/**/*.tsx"],
 			exclude: ["**/*.test.ts", "**/*.test.tsx", "**/*.spec.ts", "**/*.spec.tsx", "**/vitest.config.ts"],
 		},
 	},

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/package.json

@@ -19,6 +19,6 @@
 		"@roo-code/config-eslint": "workspace:^",
 		"@roo-code/config-typescript": "workspace:^",
 		"@types/node": "20.19.41",
-		"vitest": "3.2.6"
+		"vitest": "4.1.0"
 	}
 }

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/package.json

@@ -24,7 +24,7 @@
 		"@types/node": "24.2.1",
 		"@types/vscode": "1.100.0",
 		"globals": "16.3.0",
-		"@vitest/coverage-v8": "3.2.6",
-		"vitest": "3.2.6"
+		"@vitest/coverage-v8": "4.1.0",
+		"vitest": "4.1.0"
 	}
 }

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)
 	})