test: add safeWriteJson mock for transactional file operations

- Add vitest imports and mock safeWriteJson to support atomic transaction testing. This ensures file operations in tests are properly isolated and can be verified without actual filesystem interactions.
- Add fs/promises.access mock to prevent errors when checking file existence.
- Satisfy requirement that clineMessages must be defined for say/ask() to be valid

Signed-off-by: Eric Wheeler <[email protected]>

Author: Eric Wheeler

src/core/task/__tests__/Task.spec.ts

@@ -5,6 +5,7 @@ import * as path from "path"
 
 import * as vscode from "vscode"
 import { Anthropic } from "@anthropic-ai/sdk"
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest"
 
 import type { GlobalState, ProviderSettings, ModelInfo } from "@roo-code/types"
 import { TelemetryService } from "@roo-code/telemetry"
@@ -59,6 +60,7 @@ vi.mock("fs/promises", async (importOriginal) => {
 		}),
 		unlink: vi.fn().mockResolvedValue(undefined),
 		rmdir: vi.fn().mockResolvedValue(undefined),
+		access: vi.fn().mockResolvedValue(undefined),
 	}
 
 	return {
@@ -164,6 +166,10 @@ vi.mock("../../../utils/fs", () => ({
 	}),
 }))
 
+vi.mock("../../../utils/safeWriteJson", () => ({
+	safeWriteJson: vi.fn().mockResolvedValue(undefined),
+}))
+
 const mockMessages = [
 	{
 		ts: Date.now(),
@@ -1037,6 +1043,16 @@ describe("Cline", () => {
 					startTask: false,
 				})
 
+				// Initialize child messages
+				child.clineMessages = [
+					{
+						ts: Date.now(),
+						type: "say",
+						say: "api_req_started",
+						text: "Preparing request...",
+					},
+				]
+
 				// Mock the child's API stream
 				const childMockStream = {
 					async *[Symbol.asyncIterator]() {
@@ -1169,6 +1185,16 @@ describe("Cline", () => {
 
 				vi.spyOn(child1.api, "createMessage").mockReturnValue(mockStream)
 
+				// Initialize with a starting message
+				child1.clineMessages = [
+					{
+						ts: Date.now(),
+						type: "say",
+						say: "api_req_started",
+						text: "Preparing request...",
+					},
+				]
+
 				// Make an API request with the first child task
 				const child1Iterator = child1.attemptApiRequest(0)
 				await child1Iterator.next()
@@ -1192,6 +1218,16 @@ describe("Cline", () => {
 
 				vi.spyOn(child2.api, "createMessage").mockReturnValue(mockStream)
 
+				// Initialize with a starting message
+				child2.clineMessages = [
+					{
+						ts: Date.now(),
+						type: "say",
+						say: "api_req_started",
+						text: "Preparing request...",
+					},
+				]
+
 				// Make an API request with the second child task
 				const child2Iterator = child2.attemptApiRequest(0)
 				await child2Iterator.next()