@anh3d0nic/qwen-code-termux-ice 21.0.1 → 21.2.0

package/core-dist/src/tools/write-file.test.js

@@ -0,0 +1,613 @@
+/**
+ * @license
+ * Copyright 2025 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+import { describe, it, expect, beforeEach, afterEach, vi, } from 'vitest';
+import { getCorrectedFileContent, WriteFileTool } from './write-file.js';
+import { ToolErrorType } from './tool-error.js';
+import { ToolConfirmationOutcome } from './tools.js';
+import { ApprovalMode } from '../config/config.js';
+import path from 'node:path';
+import fs from 'node:fs';
+import os from 'node:os';
+import { GeminiClient } from '../core/client.js';
+import { createMockWorkspaceContext } from '../test-utils/mockWorkspaceContext.js';
+import { StandardFileSystemService } from '../services/fileSystemService.js';
+import { IdeClient } from '../ide/ide-client.js';
+const rootDir = path.resolve(os.tmpdir(), 'qwen-code-test-root');
+// --- MOCKS ---
+vi.mock('../core/client.js');
+vi.mock('../ide/ide-client.js', () => ({
+    IdeClient: {
+        getInstance: vi.fn(),
+    },
+}));
+let mockGeminiClientInstance;
+const mockIdeClient = {
+    openDiff: vi.fn(),
+    isDiffingEnabled: vi.fn(),
+};
+vi.mocked(IdeClient.getInstance).mockResolvedValue(mockIdeClient);
+// Mock Config
+const fsService = new StandardFileSystemService();
+const mockConfigInternal = {
+    getTargetDir: () => rootDir,
+    getApprovalMode: vi.fn(() => ApprovalMode.DEFAULT),
+    setApprovalMode: vi.fn(),
+    getGeminiClient: vi.fn(), // Initialize as a plain mock function
+    getBaseLlmClient: vi.fn(), // Initialize as a plain mock function
+    getFileSystemService: () => fsService,
+    getIdeMode: vi.fn(() => false),
+    getWorkspaceContext: () => createMockWorkspaceContext(rootDir),
+    getApiKey: () => 'test-key',
+    getModel: () => 'test-model',
+    getSandbox: () => false,
+    getDebugMode: () => false,
+    getQuestion: () => undefined,
+    getFullContext: () => false,
+    getToolDiscoveryCommand: () => undefined,
+    getToolCallCommand: () => undefined,
+    getMcpServerCommand: () => undefined,
+    getMcpServers: () => undefined,
+    getUserAgent: () => 'test-agent',
+    getUserMemory: () => '',
+    setUserMemory: vi.fn(),
+    getGeminiMdFileCount: () => 0,
+    setGeminiMdFileCount: vi.fn(),
+    getToolRegistry: () => ({
+        registerTool: vi.fn(),
+        discoverTools: vi.fn(),
+    }),
+    getDefaultFileEncoding: () => 'utf-8',
+};
+const mockConfig = mockConfigInternal;
+vi.mock('../telemetry/loggers.js', () => ({
+    logFileOperation: vi.fn(),
+}));
+// --- END MOCKS ---
+describe('WriteFileTool', () => {
+    let tool;
+    let tempDir;
+    beforeEach(() => {
+        vi.clearAllMocks();
+        // Create a unique temporary directory for files created outside the root
+        tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'write-file-test-external-'));
+        // Ensure the rootDir for the tool exists
+        if (!fs.existsSync(rootDir)) {
+            fs.mkdirSync(rootDir, { recursive: true });
+        }
+        // Setup GeminiClient mock
+        mockGeminiClientInstance = new (vi.mocked(GeminiClient))(mockConfig);
+        vi.mocked(GeminiClient).mockImplementation(() => mockGeminiClientInstance);
+        // Now that mockGeminiClientInstance is initialized, set the mock implementation for getGeminiClient
+        mockConfigInternal.getGeminiClient.mockReturnValue(mockGeminiClientInstance);
+        tool = new WriteFileTool(mockConfig);
+        // Reset mocks before each test
+        mockConfigInternal.getApprovalMode.mockReturnValue(ApprovalMode.DEFAULT);
+        mockConfigInternal.setApprovalMode.mockClear();
+    });
+    afterEach(() => {
+        // Clean up the temporary directories
+        if (fs.existsSync(tempDir)) {
+            fs.rmSync(tempDir, { recursive: true, force: true });
+        }
+        if (fs.existsSync(rootDir)) {
+            fs.rmSync(rootDir, { recursive: true, force: true });
+        }
+        vi.clearAllMocks();
+    });
+    describe('build', () => {
+        it('should return an invocation for a valid absolute path within root', () => {
+            const params = {
+                file_path: path.join(rootDir, 'test.txt'),
+                content: 'hello',
+            };
+            const invocation = tool.build(params);
+            expect(invocation).toBeDefined();
+            expect(invocation.params).toEqual(params);
+        });
+        it('should throw an error for a relative path', () => {
+            const params = { file_path: 'test.txt', content: 'hello' };
+            expect(() => tool.build(params)).toThrow(/File path must be absolute/);
+        });
+        it('should throw an error for a path outside root', () => {
+            const outsidePath = path.resolve(tempDir, 'outside-root.txt');
+            const params = {
+                file_path: outsidePath,
+                content: 'hello',
+            };
+            expect(() => tool.build(params)).toThrow(/File path must be within one of the workspace directories/);
+        });
+        it('should throw an error if path is a directory', () => {
+            const dirAsFilePath = path.join(rootDir, 'a_directory');
+            fs.mkdirSync(dirAsFilePath);
+            const params = {
+                file_path: dirAsFilePath,
+                content: 'hello',
+            };
+            expect(() => tool.build(params)).toThrow(`Path is a directory, not a file: ${dirAsFilePath}`);
+        });
+        it('should coerce null content into an empty string', () => {
+            const params = {
+                file_path: path.join(rootDir, 'test.txt'),
+                content: null,
+            }; // Intentionally non-conforming
+            expect(() => tool.build(params)).toBeDefined();
+        });
+        it('should throw error if the file_path is empty', () => {
+            const dirAsFilePath = path.join(rootDir, 'a_directory');
+            fs.mkdirSync(dirAsFilePath);
+            const params = {
+                file_path: '',
+                content: '',
+            };
+            expect(() => tool.build(params)).toThrow(`Missing or empty "file_path"`);
+        });
+    });
+    describe('getCorrectedFileContent', () => {
+        it('should return proposed content unchanged for a new file', async () => {
+            const filePath = path.join(rootDir, 'new_corrected_file.txt');
+            const proposedContent = 'Proposed new content.';
+            const result = await getCorrectedFileContent(mockConfig, filePath, proposedContent);
+            expect(result.correctedContent).toBe(proposedContent);
+            expect(result.originalContent).toBe('');
+            expect(result.fileExists).toBe(false);
+            expect(result.error).toBeUndefined();
+        });
+        it('should return proposed content unchanged for an existing file', async () => {
+            const filePath = path.join(rootDir, 'existing_corrected_file.txt');
+            const originalContent = 'Original existing content.';
+            const proposedContent = 'Proposed replacement content.';
+            fs.writeFileSync(filePath, originalContent, 'utf8');
+            const result = await getCorrectedFileContent(mockConfig, filePath, proposedContent);
+            expect(result.correctedContent).toBe(proposedContent);
+            expect(result.originalContent).toBe(originalContent);
+            expect(result.fileExists).toBe(true);
+            expect(result.error).toBeUndefined();
+        });
+        it('should return error if reading an existing file fails (e.g. permissions)', async () => {
+            const filePath = path.join(rootDir, 'unreadable_file.txt');
+            const proposedContent = 'some content';
+            fs.writeFileSync(filePath, 'content', { mode: 0o000 });
+            const readError = new Error('Permission denied');
+            vi.spyOn(fsService, 'readTextFile').mockImplementationOnce(() => Promise.reject(readError));
+            const result = await getCorrectedFileContent(mockConfig, filePath, proposedContent);
+            expect(fsService.readTextFile).toHaveBeenCalledWith(filePath);
+            expect(result.correctedContent).toBe(proposedContent);
+            expect(result.originalContent).toBe('');
+            expect(result.fileExists).toBe(true);
+            expect(result.error).toEqual({
+                message: 'Permission denied',
+                code: undefined,
+            });
+            fs.chmodSync(filePath, 0o600);
+        });
+    });
+    describe('shouldConfirmExecute', () => {
+        const abortSignal = new AbortController().signal;
+        it('should return false if _getCorrectedFileContent returns an error', async () => {
+            const filePath = path.join(rootDir, 'confirm_error_file.txt');
+            const params = { file_path: filePath, content: 'test content' };
+            fs.writeFileSync(filePath, 'original', { mode: 0o000 });
+            const readError = new Error('Simulated read error for confirmation');
+            vi.spyOn(fsService, 'readTextFile').mockImplementationOnce(() => Promise.reject(readError));
+            const invocation = tool.build(params);
+            const confirmation = await invocation.shouldConfirmExecute(abortSignal);
+            expect(confirmation).toBe(false);
+            fs.chmodSync(filePath, 0o600);
+        });
+        it('should request confirmation with diff for a new file', async () => {
+            const filePath = path.join(rootDir, 'confirm_new_file.txt');
+            const proposedContent = 'Proposed new content for confirmation.';
+            const params = { file_path: filePath, content: proposedContent };
+            const invocation = tool.build(params);
+            const confirmation = (await invocation.shouldConfirmExecute(abortSignal));
+            expect(confirmation).toEqual(expect.objectContaining({
+                title: `Confirm Write: ${path.basename(filePath)}`,
+                fileName: 'confirm_new_file.txt',
+                fileDiff: expect.stringContaining(proposedContent),
+            }));
+            expect(confirmation.fileDiff).toMatch(/--- confirm_new_file.txt\tCurrent/);
+            expect(confirmation.fileDiff).toMatch(/\+\+\+ confirm_new_file.txt\tProposed/);
+        });
+        it('should request confirmation with diff for an existing file', async () => {
+            const filePath = path.join(rootDir, 'confirm_existing_file.txt');
+            const originalContent = 'Original content for confirmation.';
+            const proposedContent = 'Proposed replacement for confirmation.';
+            fs.writeFileSync(filePath, originalContent, 'utf8');
+            const params = { file_path: filePath, content: proposedContent };
+            const invocation = tool.build(params);
+            const confirmation = (await invocation.shouldConfirmExecute(abortSignal));
+            expect(confirmation).toEqual(expect.objectContaining({
+                title: `Confirm Write: ${path.basename(filePath)}`,
+                fileName: 'confirm_existing_file.txt',
+                fileDiff: expect.stringContaining(proposedContent),
+            }));
+            expect(confirmation.fileDiff).toMatch(originalContent.replace(/[.*+?^${}()|[\\]\\]/g, '\\$&'));
+        });
+        describe('with IDE integration', () => {
+            beforeEach(() => {
+                // Enable IDE mode and set connection status for these tests
+                mockConfigInternal.getIdeMode.mockReturnValue(true);
+                mockIdeClient.isDiffingEnabled.mockReturnValue(true);
+                mockIdeClient.openDiff.mockResolvedValue({
+                    status: 'accepted',
+                    content: 'ide-modified-content',
+                });
+            });
+            it('should call openDiff and await it when in IDE mode and connected', async () => {
+                const filePath = path.join(rootDir, 'ide_confirm_file.txt');
+                const params = { file_path: filePath, content: 'test' };
+                const invocation = tool.build(params);
+                const confirmation = (await invocation.shouldConfirmExecute(abortSignal));
+                expect(mockIdeClient.openDiff).toHaveBeenCalledWith(filePath, 'test');
+                // Ensure the promise is awaited by checking the result
+                expect(confirmation.ideConfirmation).toBeDefined();
+                await confirmation.ideConfirmation; // Should resolve
+            });
+            it('should not call openDiff if not in IDE mode', async () => {
+                mockConfigInternal.getIdeMode.mockReturnValue(false);
+                const filePath = path.join(rootDir, 'ide_disabled_file.txt');
+                const params = { file_path: filePath, content: 'test' };
+                const invocation = tool.build(params);
+                await invocation.shouldConfirmExecute(abortSignal);
+                expect(mockIdeClient.openDiff).not.toHaveBeenCalled();
+            });
+            it('should not call openDiff if IDE is not connected', async () => {
+                mockIdeClient.isDiffingEnabled.mockReturnValue(false);
+                const filePath = path.join(rootDir, 'ide_disconnected_file.txt');
+                const params = { file_path: filePath, content: 'test' };
+                const invocation = tool.build(params);
+                await invocation.shouldConfirmExecute(abortSignal);
+                expect(mockIdeClient.openDiff).not.toHaveBeenCalled();
+            });
+            it('should update params.content with IDE content when onConfirm is called', async () => {
+                const filePath = path.join(rootDir, 'ide_onconfirm_file.txt');
+                const params = { file_path: filePath, content: 'original-content' };
+                const invocation = tool.build(params);
+                // This is the key part: get the confirmation details
+                const confirmation = (await invocation.shouldConfirmExecute(abortSignal));
+                // The `onConfirm` function should exist on the details object
+                expect(confirmation.onConfirm).toBeDefined();
+                // Call `onConfirm` to trigger the logic that updates the content
+                await confirmation.onConfirm(ToolConfirmationOutcome.ProceedOnce);
+                // Now, check if the original `params` object (captured by the invocation) was modified
+                expect(invocation.params.content).toBe('ide-modified-content');
+            });
+            it('should not await ideConfirmation promise', async () => {
+                const filePath = path.join(rootDir, 'ide_no_await_file.txt');
+                const params = { file_path: filePath, content: 'test' };
+                const invocation = tool.build(params);
+                let diffPromiseResolved = false;
+                const diffPromise = new Promise((resolve) => {
+                    setTimeout(() => {
+                        diffPromiseResolved = true;
+                        resolve({ status: 'accepted', content: 'ide-modified-content' });
+                    }, 50); // A small delay to ensure the check happens before resolution
+                });
+                mockIdeClient.openDiff.mockReturnValue(diffPromise);
+                const confirmation = (await invocation.shouldConfirmExecute(abortSignal));
+                // This is the key check: the confirmation details should be returned
+                // *before* the diffPromise is resolved.
+                expect(diffPromiseResolved).toBe(false);
+                expect(confirmation).toBeDefined();
+                expect(confirmation.ideConfirmation).toBe(diffPromise);
+                // Now, we can await the promise to let the test finish cleanly.
+                await diffPromise;
+                expect(diffPromiseResolved).toBe(true);
+            });
+        });
+    });
+    describe('execute', () => {
+        const abortSignal = new AbortController().signal;
+        beforeEach(() => {
+            // Ensure IDE mode is disabled for these tests
+            mockConfigInternal.getIdeMode.mockReturnValue(false);
+            mockIdeClient.isDiffingEnabled.mockReturnValue(false);
+        });
+        it('should return error if _getCorrectedFileContent returns an error during execute', async () => {
+            const filePath = path.join(rootDir, 'execute_error_file.txt');
+            const params = { file_path: filePath, content: 'test content' };
+            fs.writeFileSync(filePath, 'original', { mode: 0o000 });
+            vi.spyOn(fsService, 'readTextFile').mockImplementationOnce(() => {
+                const readError = new Error('Simulated read error for execute');
+                return Promise.reject(readError);
+            });
+            const invocation = tool.build(params);
+            const result = await invocation.execute(abortSignal);
+            expect(result.llmContent).toContain('Error checking existing file');
+            expect(result.returnDisplay).toMatch(/Error checking existing file: Simulated read error for execute/);
+            expect(result.error).toEqual({
+                message: 'Error checking existing file: Simulated read error for execute',
+                type: ToolErrorType.FILE_WRITE_FAILURE,
+            });
+            fs.chmodSync(filePath, 0o600);
+        });
+        it('should write a new file and return diff', async () => {
+            const filePath = path.join(rootDir, 'execute_new_file.txt');
+            const proposedContent = 'Proposed new content for execute.';
+            const params = { file_path: filePath, content: proposedContent };
+            const invocation = tool.build(params);
+            const confirmDetails = await invocation.shouldConfirmExecute(abortSignal);
+            if (typeof confirmDetails === 'object' &&
+                'onConfirm' in confirmDetails &&
+                confirmDetails.onConfirm) {
+                await confirmDetails.onConfirm(ToolConfirmationOutcome.ProceedOnce);
+            }
+            const result = await invocation.execute(abortSignal);
+            expect(result.llmContent).toMatch(/Successfully created and wrote to new file/);
+            expect(fs.existsSync(filePath)).toBe(true);
+            const writtenContent = await fsService.readTextFile(filePath);
+            expect(writtenContent).toBe(proposedContent);
+            const display = result.returnDisplay;
+            expect(display.fileName).toBe('execute_new_file.txt');
+            expect(display.fileDiff).toMatch(/--- execute_new_file.txt\tOriginal/);
+            expect(display.fileDiff).toMatch(/\+\+\+ execute_new_file.txt\tWritten/);
+            expect(display.fileDiff).toMatch(proposedContent.replace(/[.*+?^${}()|[\\]\\]/g, '\\$&'));
+        });
+        it('should overwrite an existing file and return diff', async () => {
+            const filePath = path.join(rootDir, 'execute_existing_file.txt');
+            const initialContent = 'Initial content for execute.';
+            const proposedContent = 'Proposed overwrite for execute.';
+            fs.writeFileSync(filePath, initialContent, 'utf8');
+            const params = { file_path: filePath, content: proposedContent };
+            const invocation = tool.build(params);
+            const confirmDetails = await invocation.shouldConfirmExecute(abortSignal);
+            if (typeof confirmDetails === 'object' &&
+                'onConfirm' in confirmDetails &&
+                confirmDetails.onConfirm) {
+                await confirmDetails.onConfirm(ToolConfirmationOutcome.ProceedOnce);
+            }
+            const result = await invocation.execute(abortSignal);
+            expect(result.llmContent).toMatch(/Successfully overwrote file/);
+            const writtenContent = await fsService.readTextFile(filePath);
+            expect(writtenContent).toBe(proposedContent);
+            const display = result.returnDisplay;
+            expect(display.fileName).toBe('execute_existing_file.txt');
+            expect(display.fileDiff).toMatch(initialContent.replace(/[.*+?^${}()|[\\]\\]/g, '\\$&'));
+            expect(display.fileDiff).toMatch(proposedContent.replace(/[.*+?^${}()|[\\]\\]/g, '\\$&'));
+        });
+        it('should create directory if it does not exist', async () => {
+            const dirPath = path.join(rootDir, 'new_dir_for_write');
+            const filePath = path.join(dirPath, 'file_in_new_dir.txt');
+            const content = 'Content in new directory';
+            const params = { file_path: filePath, content };
+            const invocation = tool.build(params);
+            // Simulate confirmation if your logic requires it before execute, or remove if not needed for this path
+            const confirmDetails = await invocation.shouldConfirmExecute(abortSignal);
+            if (typeof confirmDetails === 'object' &&
+                'onConfirm' in confirmDetails &&
+                confirmDetails.onConfirm) {
+                await confirmDetails.onConfirm(ToolConfirmationOutcome.ProceedOnce);
+            }
+            await invocation.execute(abortSignal);
+            expect(fs.existsSync(dirPath)).toBe(true);
+            expect(fs.statSync(dirPath).isDirectory()).toBe(true);
+            expect(fs.existsSync(filePath)).toBe(true);
+            expect(fs.readFileSync(filePath, 'utf8')).toBe(content);
+        });
+        it('should include modification message when proposed content is modified', async () => {
+            const filePath = path.join(rootDir, 'new_file_modified.txt');
+            const content = 'New file content modified by user';
+            const params = {
+                file_path: filePath,
+                content,
+                modified_by_user: true,
+            };
+            const invocation = tool.build(params);
+            const result = await invocation.execute(abortSignal);
+            expect(result.llmContent).toMatch(/User modified the `content`/);
+        });
+        it('should not include modification message when proposed content is not modified', async () => {
+            const filePath = path.join(rootDir, 'new_file_unmodified.txt');
+            const content = 'New file content not modified';
+            const params = {
+                file_path: filePath,
+                content,
+                modified_by_user: false,
+            };
+            const invocation = tool.build(params);
+            const result = await invocation.execute(abortSignal);
+            expect(result.llmContent).not.toMatch(/User modified the `content`/);
+        });
+        it('should not include modification message when modified_by_user is not provided', async () => {
+            const filePath = path.join(rootDir, 'new_file_unmodified.txt');
+            const content = 'New file content not modified';
+            const params = {
+                file_path: filePath,
+                content,
+            };
+            const invocation = tool.build(params);
+            const result = await invocation.execute(abortSignal);
+            expect(result.llmContent).not.toMatch(/User modified the `content`/);
+        });
+    });
+    describe('workspace boundary validation', () => {
+        it('should validate paths are within workspace root', () => {
+            const params = {
+                file_path: path.join(rootDir, 'file.txt'),
+                content: 'test content',
+            };
+            expect(() => tool.build(params)).not.toThrow();
+        });
+        it('should reject paths outside workspace root', () => {
+            const params = {
+                file_path: '/etc/passwd',
+                content: 'malicious',
+            };
+            expect(() => tool.build(params)).toThrow(/File path must be within one of the workspace directories/);
+        });
+    });
+    describe('specific error types for write failures', () => {
+        const abortSignal = new AbortController().signal;
+        it('should return PERMISSION_DENIED error when write fails with EACCES', async () => {
+            const filePath = path.join(rootDir, 'permission_denied_file.txt');
+            const content = 'test content';
+            // Mock FileSystemService writeTextFile to throw EACCES error
+            vi.spyOn(fsService, 'writeTextFile').mockImplementationOnce(() => {
+                const error = new Error('Permission denied');
+                error.code = 'EACCES';
+                return Promise.reject(error);
+            });
+            const params = { file_path: filePath, content };
+            const invocation = tool.build(params);
+            const result = await invocation.execute(abortSignal);
+            expect(result.error?.type).toBe(ToolErrorType.PERMISSION_DENIED);
+            expect(result.llmContent).toContain(`Permission denied writing to file: ${filePath} (EACCES)`);
+            expect(result.returnDisplay).toContain(`Permission denied writing to file: ${filePath} (EACCES)`);
+        });
+        it('should return NO_SPACE_LEFT error when write fails with ENOSPC', async () => {
+            const filePath = path.join(rootDir, 'no_space_file.txt');
+            const content = 'test content';
+            // Mock FileSystemService writeTextFile to throw ENOSPC error
+            vi.spyOn(fsService, 'writeTextFile').mockImplementationOnce(() => {
+                const error = new Error('No space left on device');
+                error.code = 'ENOSPC';
+                return Promise.reject(error);
+            });
+            const params = { file_path: filePath, content };
+            const invocation = tool.build(params);
+            const result = await invocation.execute(abortSignal);
+            expect(result.error?.type).toBe(ToolErrorType.NO_SPACE_LEFT);
+            expect(result.llmContent).toContain(`No space left on device: ${filePath} (ENOSPC)`);
+            expect(result.returnDisplay).toContain(`No space left on device: ${filePath} (ENOSPC)`);
+        });
+        it('should return TARGET_IS_DIRECTORY error when write fails with EISDIR', async () => {
+            const dirPath = path.join(rootDir, 'test_directory');
+            const content = 'test content';
+            // Mock fs.existsSync to return false to bypass validation
+            const originalExistsSync = fs.existsSync;
+            vi.spyOn(fs, 'existsSync').mockImplementation((path) => {
+                if (path === dirPath) {
+                    return false; // Pretend directory doesn't exist to bypass validation
+                }
+                return originalExistsSync(path);
+            });
+            // Mock FileSystemService writeTextFile to throw EISDIR error
+            vi.spyOn(fsService, 'writeTextFile').mockImplementationOnce(() => {
+                const error = new Error('Is a directory');
+                error.code = 'EISDIR';
+                return Promise.reject(error);
+            });
+            const params = { file_path: dirPath, content };
+            const invocation = tool.build(params);
+            const result = await invocation.execute(abortSignal);
+            expect(result.error?.type).toBe(ToolErrorType.TARGET_IS_DIRECTORY);
+            expect(result.llmContent).toContain(`Target is a directory, not a file: ${dirPath} (EISDIR)`);
+            expect(result.returnDisplay).toContain(`Target is a directory, not a file: ${dirPath} (EISDIR)`);
+            vi.spyOn(fs, 'existsSync').mockImplementation(originalExistsSync);
+        });
+        it('should return FILE_WRITE_FAILURE for generic write errors', async () => {
+            const filePath = path.join(rootDir, 'generic_error_file.txt');
+            const content = 'test content';
+            // Ensure fs.existsSync is not mocked for this test
+            vi.restoreAllMocks();
+            // Mock FileSystemService writeTextFile to throw generic error
+            vi.spyOn(fsService, 'writeTextFile').mockImplementationOnce(() => Promise.reject(new Error('Generic write error')));
+            const params = { file_path: filePath, content };
+            const invocation = tool.build(params);
+            const result = await invocation.execute(abortSignal);
+            expect(result.error?.type).toBe(ToolErrorType.FILE_WRITE_FAILURE);
+            expect(result.llmContent).toContain('Error writing to file: Generic write error');
+            expect(result.returnDisplay).toContain('Error writing to file: Generic write error');
+        });
+    });
+    describe('BOM preservation (Issue #1672)', () => {
+        const abortSignal = new AbortController().signal;
+        it('should preserve BOM when overwriting existing file with BOM', async () => {
+            const filePath = path.join(rootDir, 'bom_file.txt');
+            const originalContent = 'original content';
+            const newContent = 'new content';
+            // Create file with BOM
+            fs.writeFileSync(filePath, Buffer.concat([
+                Buffer.from([0xef, 0xbb, 0xbf]),
+                Buffer.from(originalContent, 'utf-8'),
+            ]));
+            // Spy on writeTextFile to verify BOM option
+            const writeSpy = vi.spyOn(fsService, 'writeTextFile');
+            const params = { file_path: filePath, content: newContent };
+            const invocation = tool.build(params);
+            await invocation.execute(abortSignal);
+            // Verify writeTextFile was called with bom: true
+            expect(writeSpy).toHaveBeenCalledWith(filePath, newContent, {
+                bom: true,
+                encoding: 'utf-8',
+            });
+            // Cleanup
+            if (fs.existsSync(filePath)) {
+                fs.unlinkSync(filePath);
+            }
+        });
+        it('should not add BOM when overwriting existing file without BOM', async () => {
+            const filePath = path.join(rootDir, 'no_bom_file.txt');
+            const originalContent = 'original content';
+            const newContent = 'new content';
+            // Create file without BOM
+            fs.writeFileSync(filePath, originalContent, 'utf-8');
+            // Spy on writeTextFile to verify BOM option
+            const writeSpy = vi.spyOn(fsService, 'writeTextFile');
+            const params = { file_path: filePath, content: newContent };
+            const invocation = tool.build(params);
+            await invocation.execute(abortSignal);
+            // Verify writeTextFile was called with bom: false
+            expect(writeSpy).toHaveBeenCalledWith(filePath, newContent, {
+                bom: false,
+                encoding: 'utf-8',
+            });
+            // Cleanup
+            if (fs.existsSync(filePath)) {
+                fs.unlinkSync(filePath);
+            }
+        });
+        it('should use default encoding for new files', async () => {
+            const filePath = path.join(rootDir, 'new_file.txt');
+            const newContent = 'new content';
+            // Ensure file does not exist
+            if (fs.existsSync(filePath)) {
+                fs.unlinkSync(filePath);
+            }
+            // Spy on writeTextFile to verify BOM option
+            const writeSpy = vi.spyOn(fsService, 'writeTextFile');
+            const params = { file_path: filePath, content: newContent };
+            const invocation = tool.build(params);
+            await invocation.execute(abortSignal);
+            // Verify writeTextFile was called with bom: false (default is utf-8)
+            expect(writeSpy).toHaveBeenCalledWith(filePath, newContent, {
+                bom: false,
+            });
+            // Cleanup
+            if (fs.existsSync(filePath)) {
+                fs.unlinkSync(filePath);
+            }
+        });
+        it('should use BOM for new files when defaultFileEncoding is utf-8-bom', async () => {
+            const filePath = path.join(rootDir, 'new_file_bom.txt');
+            const newContent = 'new content';
+            // Ensure file does not exist
+            if (fs.existsSync(filePath)) {
+                fs.unlinkSync(filePath);
+            }
+            // Mock config to return utf-8-bom
+            const originalGetDefaultFileEncoding = mockConfigInternal.getDefaultFileEncoding;
+            mockConfigInternal.getDefaultFileEncoding = () => 'utf-8-bom';
+            // Spy on writeTextFile to verify BOM option
+            const writeSpy = vi.spyOn(fsService, 'writeTextFile');
+            const params = { file_path: filePath, content: newContent };
+            const invocation = tool.build(params);
+            await invocation.execute(abortSignal);
+            // Verify writeTextFile was called with bom: true
+            expect(writeSpy).toHaveBeenCalledWith(filePath, newContent, {
+                bom: true,
+            });
+            // Restore mock
+            mockConfigInternal.getDefaultFileEncoding =
+                originalGetDefaultFileEncoding;
+            // Cleanup
+            if (fs.existsSync(filePath)) {
+                fs.unlinkSync(filePath);
+            }
+        });
+    });
+});
+//# sourceMappingURL=write-file.test.js.map