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

Author: edelauna

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"],
 		},
 	},

packages/cloud/vitest.config.ts

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

packages/core/src/worktree/__tests__/worktree-include.integration.spec.ts

@@ -38,6 +38,7 @@ describe.sequential("WorktreeIncludeService integration", () => {
 			await execGit(repoDir, ["init"])
 			await execGit(repoDir, ["config", "user.name", "Test User"])
 			await execGit(repoDir, ["config", "user.email", "test@example.com"])
+			await execGit(repoDir, ["config", "commit.gpgSign", "false"])
 
 			await fs.writeFile(path.join(repoDir, "README.md"), "test")
 			await execGit(repoDir, ["add", "README.md"])

packages/core/src/worktree/__tests__/worktree-service.integration.spec.ts

@@ -28,6 +28,7 @@ describe.sequential("WorktreeService integration", () => {
 		await execGit(repoDir, ["init"])
 		await execGit(repoDir, ["config", "user.name", "Test User"])
 		await execGit(repoDir, ["config", "user.email", "test@example.com"])
+		await execGit(repoDir, ["config", "commit.gpgSign", "false"])
 		await fs.writeFile(path.join(repoDir, "README.md"), "base")
 		await execGit(repoDir, ["add", "README.md"])
 		await execGit(repoDir, ["commit", "-m", "init"])

packages/core/vitest.config.ts

@@ -8,6 +8,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"],
 		},
 	},

packages/ipc/tsconfig.json

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

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

@@ -37,7 +37,9 @@ describe("PostHogTelemetryClient", () => {
 			optOut: vi.fn(),
 			shutdown: vi.fn().mockResolvedValue(undefined),
 		}
-		;(PostHog as any).mockImplementation(() => mockPostHogClient)
+		;(PostHog as any).mockImplementation(function () {
+			return mockPostHogClient
+		})
 
 		// @ts-expect-error - Accessing private static property for testing
 		PostHogTelemetryClient._instance = undefined

packages/telemetry/vitest.config.ts

@@ -8,6 +8,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"],
 		},
 	},

src/__tests__/extension.spec.ts

@@ -158,17 +158,21 @@ vi.mock("../utils/autoImportSettings", () => ({
 }))
 
 vi.mock("../extension/api", () => ({
-	API: vi.fn().mockImplementation(() => ({})),
+	API: vi.fn().mockImplementation(function () {
+		return {}
+	}),
 }))
 
 vi.mock("../activate", () => ({
 	handleUri: vi.fn(),
 	registerCommands: vi.fn(),
 	registerCodeActions: vi.fn(),
 	registerTerminalActions: vi.fn(),
-	CodeActionProvider: vi.fn().mockImplementation(() => ({
-		providedCodeActionKinds: [],
-	})),
+	CodeActionProvider: vi.fn().mockImplementation(function () {
+		return {
+			providedCodeActionKinds: [],
+		}
+	}),
 }))
 
 vi.mock("../i18n", () => ({
@@ -192,7 +196,9 @@ vi.mock("../core/webview/ClineProvider", async () => {
 	}
 	return {
 		ClineProvider: Object.assign(
-			vi.fn().mockImplementation(() => mockInstance),
+			vi.fn().mockImplementation(function () {
+				return mockInstance
+			}),
 			{
 				// Static method used by extension.ts
 				getVisibleInstance: vi.fn().mockReturnValue(mockInstance),

src/__tests__/history-resume-delegation.spec.ts

@@ -11,10 +11,12 @@ vi.mock("vscode", () => {
 		onDidChangeActiveTextEditor: vi.fn(() => ({ dispose: vi.fn() })),
 	}
 	const workspace = {
-		getConfiguration: vi.fn(() => ({
-			get: vi.fn((_key: string, defaultValue: any) => defaultValue),
-			update: vi.fn(),
-		})),
+		getConfiguration: vi.fn(() => {
+			return {
+				get: vi.fn((_key: string, defaultValue: any) => defaultValue),
+				update: vi.fn(),
+			}
+		}),
 		workspaceFolders: [],
 	}
 	const env = { machineId: "test-machine", uriScheme: "vscode", appName: "VSCode", language: "en", sessionId: "sess" }