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/activate/__tests__/CodeActionProvider.spec.ts

@@ -18,19 +18,23 @@ vi.mock("../../i18n", () => ({
 }))
 
 vi.mock("vscode", () => ({
-	CodeAction: vi.fn().mockImplementation((title, kind) => ({
-		title,
-		kind,
-		command: undefined,
-	})),
+	CodeAction: vi.fn().mockImplementation(function (title, kind) {
+		return {
+			title,
+			kind,
+			command: undefined,
+		}
+	}),
 	CodeActionKind: {
 		QuickFix: { value: "quickfix" },
 		RefactorRewrite: { value: "refactor.rewrite" },
 	},
-	Range: vi.fn().mockImplementation((startLine, startChar, endLine, endChar) => ({
-		start: { line: startLine, character: startChar },
-		end: { line: endLine, character: endChar },
-	})),
+	Range: vi.fn().mockImplementation(function (startLine, startChar, endLine, endChar) {
+		return {
+			start: { line: startLine, character: startChar },
+			end: { line: endLine, character: endChar },
+		}
+	}),
 	DiagnosticSeverity: {
 		Error: 0,
 		Warning: 1,
@@ -78,7 +82,9 @@ describe("CodeActionProvider", () => {
 		})
 		;(EditorUtils.getFilePath as Mock).mockReturnValue("/test/file.ts")
 		;(EditorUtils.hasIntersectingRange as Mock).mockReturnValue(true)
-		;(EditorUtils.createDiagnosticData as Mock).mockImplementation((d) => d)
+		;(EditorUtils.createDiagnosticData as Mock).mockImplementation((d) => {
+			return d
+		})
 	})
 
 	describe("provideCodeActions", () => {
@@ -127,15 +133,15 @@ describe("CodeActionProvider", () => {
 		})
 
 		it("should handle errors gracefully", () => {
-			const consoleErrorSpy = vi.spyOn(console, "error").mockImplementation(() => {})
+			const consoleErrorSpy = vi.spyOn(console, "error").mockImplementation(function () {})
 
 			// Reset the workspace mock to return true for enableCodeActions
 			const mockGet = vi.fn().mockReturnValue(true)
 			const mockGetConfiguration = vi.fn().mockReturnValue({
 				get: mockGet,
 			})
 			;(vscode.workspace.getConfiguration as Mock).mockReturnValue(mockGetConfiguration())
-			;(EditorUtils.getEffectiveRange as Mock).mockImplementation(() => {
+			;(EditorUtils.getEffectiveRange as Mock).mockImplementation(function () {
 				throw new Error("Test error")
 			})