@didim365/agent-cli-core 0.1.0

package/dist/src/tools/modifiable-tool.js

@@ -0,0 +1,110 @@
+/**
+ * @license
+ * Copyright 2025 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+import { openDiff } from '../utils/editor.js';
+import os from 'node:os';
+import path from 'node:path';
+import fs from 'node:fs';
+import * as Diff from 'diff';
+import { DEFAULT_DIFF_OPTIONS } from './diffOptions.js';
+import { isNodeError } from '../utils/errors.js';
+import { debugLogger } from '../utils/debugLogger.js';
+/**
+ * Type guard to check if a declarative tool is modifiable.
+ */
+export function isModifiableDeclarativeTool(tool) {
+    return 'getModifyContext' in tool;
+}
+function createTempFilesForModify(currentContent, proposedContent, file_path) {
+    const diffDir = fs.mkdtempSync(path.join(os.tmpdir(), 'gemini-cli-tool-modify-'));
+    try {
+        fs.chmodSync(diffDir, 0o700);
+    }
+    catch (e) {
+        debugLogger.error(`Error setting permissions on temp diff directory: ${diffDir}`, e);
+        throw e;
+    }
+    const ext = path.extname(file_path);
+    const fileName = path.basename(file_path, ext);
+    const timestamp = Date.now();
+    const tempOldPath = path.join(diffDir, `gemini-cli-modify-${fileName}-old-${timestamp}${ext}`);
+    const tempNewPath = path.join(diffDir, `gemini-cli-modify-${fileName}-new-${timestamp}${ext}`);
+    fs.writeFileSync(tempOldPath, currentContent, {
+        encoding: 'utf8',
+        mode: 0o600,
+    });
+    fs.writeFileSync(tempNewPath, proposedContent, {
+        encoding: 'utf8',
+        mode: 0o600,
+    });
+    return { oldPath: tempOldPath, newPath: tempNewPath, dirPath: diffDir };
+}
+function getUpdatedParams(tmpOldPath, tempNewPath, originalParams, modifyContext) {
+    let oldContent = '';
+    let newContent = '';
+    try {
+        oldContent = fs.readFileSync(tmpOldPath, 'utf8');
+    }
+    catch (err) {
+        if (!isNodeError(err) || err.code !== 'ENOENT')
+            throw err;
+        oldContent = '';
+    }
+    try {
+        newContent = fs.readFileSync(tempNewPath, 'utf8');
+    }
+    catch (err) {
+        if (!isNodeError(err) || err.code !== 'ENOENT')
+            throw err;
+        newContent = '';
+    }
+    const updatedParams = modifyContext.createUpdatedParams(oldContent, newContent, originalParams);
+    const updatedDiff = Diff.createPatch(path.basename(modifyContext.getFilePath(originalParams)), oldContent, newContent, 'Current', 'Proposed', DEFAULT_DIFF_OPTIONS);
+    return { updatedParams, updatedDiff };
+}
+function deleteTempFiles(oldPath, newPath, dirPath) {
+    try {
+        fs.unlinkSync(oldPath);
+    }
+    catch {
+        debugLogger.error(`Error deleting temp diff file: ${oldPath}`);
+    }
+    try {
+        fs.unlinkSync(newPath);
+    }
+    catch {
+        debugLogger.error(`Error deleting temp diff file: ${newPath}`);
+    }
+    try {
+        fs.rmdirSync(dirPath);
+    }
+    catch {
+        debugLogger.error(`Error deleting temp diff directory: ${dirPath}`);
+    }
+}
+/**
+ * Triggers an external editor for the user to modify the proposed content,
+ * and returns the updated tool parameters and the diff after the user has modified the proposed content.
+ */
+export async function modifyWithEditor(originalParams, modifyContext, editorType, _abortSignal, overrides) {
+    const hasCurrentOverride = overrides !== undefined && 'currentContent' in overrides;
+    const hasProposedOverride = overrides !== undefined && 'proposedContent' in overrides;
+    const currentContent = hasCurrentOverride
+        ? (overrides.currentContent ?? '')
+        : await modifyContext.getCurrentContent(originalParams);
+    const proposedContent = hasProposedOverride
+        ? (overrides.proposedContent ?? '')
+        : await modifyContext.getProposedContent(originalParams);
+    const { oldPath, newPath, dirPath } = createTempFilesForModify(currentContent ?? '', proposedContent ?? '', modifyContext.getFilePath(originalParams));
+    try {
+        await openDiff(oldPath, newPath, editorType);
+        const result = getUpdatedParams(oldPath, newPath, originalParams, modifyContext);
+        return result;
+    }
+    finally {
+        deleteTempFiles(oldPath, newPath, dirPath);
+    }
+}
+//# sourceMappingURL=modifiable-tool.js.map

package/dist/src/tools/modifiable-tool.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"modifiable-tool.js","sourceRoot":"","sources":["../../../src/tools/modifiable-tool.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAGH,OAAO,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAC9C,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAC7B,OAAO,EAAE,oBAAoB,EAAE,MAAM,kBAAkB,CAAC;AACxD,OAAO,EAAE,WAAW,EAAE,MAAM,oBAAoB,CAAC;AAMjD,OAAO,EAAE,WAAW,EAAE,MAAM,yBAAyB,CAAC;AAkCtD;;GAEG;AACH,MAAM,UAAU,2BAA2B,CACzC,IAAwB;IAExB,OAAO,kBAAkB,IAAI,IAAI,CAAC;AACpC,CAAC;AAED,SAAS,wBAAwB,CAC/B,cAAsB,EACtB,eAAuB,EACvB,SAAiB;IAEjB,MAAM,OAAO,GAAG,EAAE,CAAC,WAAW,CAC5B,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,yBAAyB,CAAC,CAClD,CAAC;IAEF,IAAI,CAAC;QACH,EAAE,CAAC,SAAS,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC;IAC/B,CAAC;IAAC,OAAO,CAAC,EAAE,CAAC;QACX,WAAW,CAAC,KAAK,CACf,qDAAqD,OAAO,EAAE,EAC9D,CAAC,CACF,CAAC;QACF,MAAM,CAAC,CAAC;IACV,CAAC;IAED,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC;IACpC,MAAM,QAAQ,GAAG,IAAI,CAAC,QAAQ,CAAC,SAAS,EAAE,GAAG,CAAC,CAAC;IAC/C,MAAM,SAAS,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IAC7B,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,CAC3B,OAAO,EACP,qBAAqB,QAAQ,QAAQ,SAAS,GAAG,GAAG,EAAE,CACvD,CAAC;IACF,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,CAC3B,OAAO,EACP,qBAAqB,QAAQ,QAAQ,SAAS,GAAG,GAAG,EAAE,CACvD,CAAC;IAEF,EAAE,CAAC,aAAa,CAAC,WAAW,EAAE,cAAc,EAAE;QAC5C,QAAQ,EAAE,MAAM;QAChB,IAAI,EAAE,KAAK;KACZ,CAAC,CAAC;IACH,EAAE,CAAC,aAAa,CAAC,WAAW,EAAE,eAAe,EAAE;QAC7C,QAAQ,EAAE,MAAM;QAChB,IAAI,EAAE,KAAK;KACZ,CAAC,CAAC;IAEH,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,OAAO,EAAE,WAAW,EAAE,OAAO,EAAE,OAAO,EAAE,CAAC;AAC1E,CAAC;AAED,SAAS,gBAAgB,CACvB,UAAkB,EAClB,WAAmB,EACnB,cAA0B,EAC1B,aAAwC;IAExC,IAAI,UAAU,GAAG,EAAE,CAAC;IACpB,IAAI,UAAU,GAAG,EAAE,CAAC;IAEpB,IAAI,CAAC;QACH,UAAU,GAAG,EAAE,CAAC,YAAY,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;IACnD,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,IAAI,CAAC,WAAW,CAAC,GAAG,CAAC,IAAI,GAAG,CAAC,IAAI,KAAK,QAAQ;YAAE,MAAM,GAAG,CAAC;QAC1D,UAAU,GAAG,EAAE,CAAC;IAClB,CAAC;IAED,IAAI,CAAC;QACH,UAAU,GAAG,EAAE,CAAC,YAAY,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC;IACpD,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,IAAI,CAAC,WAAW,CAAC,GAAG,CAAC,IAAI,GAAG,CAAC,IAAI,KAAK,QAAQ;YAAE,MAAM,GAAG,CAAC;QAC1D,UAAU,GAAG,EAAE,CAAC;IAClB,CAAC;IAED,MAAM,aAAa,GAAG,aAAa,CAAC,mBAAmB,CACrD,UAAU,EACV,UAAU,EACV,cAAc,CACf,CAAC;IACF,MAAM,WAAW,GAAG,IAAI,CAAC,WAAW,CAClC,IAAI,CAAC,QAAQ,CAAC,aAAa,CAAC,WAAW,CAAC,cAAc,CAAC,CAAC,EACxD,UAAU,EACV,UAAU,EACV,SAAS,EACT,UAAU,EACV,oBAAoB,CACrB,CAAC;IAEF,OAAO,EAAE,aAAa,EAAE,WAAW,EAAE,CAAC;AACxC,CAAC;AAED,SAAS,eAAe,CACtB,OAAe,EACf,OAAe,EACf,OAAe;IAEf,IAAI,CAAC;QACH,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC;IACzB,CAAC;IAAC,MAAM,CAAC;QACP,WAAW,CAAC,KAAK,CAAC,kCAAkC,OAAO,EAAE,CAAC,CAAC;IACjE,CAAC;IAED,IAAI,CAAC;QACH,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC;IACzB,CAAC;IAAC,MAAM,CAAC;QACP,WAAW,CAAC,KAAK,CAAC,kCAAkC,OAAO,EAAE,CAAC,CAAC;IACjE,CAAC;IAED,IAAI,CAAC;QACH,EAAE,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;IACxB,CAAC;IAAC,MAAM,CAAC;QACP,WAAW,CAAC,KAAK,CAAC,uCAAuC,OAAO,EAAE,CAAC,CAAC;IACtE,CAAC;AACH,CAAC;AAED;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,gBAAgB,CACpC,cAA0B,EAC1B,aAAwC,EACxC,UAAsB,EACtB,YAAyB,EACzB,SAAkC;IAElC,MAAM,kBAAkB,GACtB,SAAS,KAAK,SAAS,IAAI,gBAAgB,IAAI,SAAS,CAAC;IAC3D,MAAM,mBAAmB,GACvB,SAAS,KAAK,SAAS,IAAI,iBAAiB,IAAI,SAAS,CAAC;IAE5D,MAAM,cAAc,GAAG,kBAAkB;QACvC,CAAC,CAAC,CAAC,SAAS,CAAC,cAAc,IAAI,EAAE,CAAC;QAClC,CAAC,CAAC,MAAM,aAAa,CAAC,iBAAiB,CAAC,cAAc,CAAC,CAAC;IAE1D,MAAM,eAAe,GAAG,mBAAmB;QACzC,CAAC,CAAC,CAAC,SAAS,CAAC,eAAe,IAAI,EAAE,CAAC;QACnC,CAAC,CAAC,MAAM,aAAa,CAAC,kBAAkB,CAAC,cAAc,CAAC,CAAC;IAE3D,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,wBAAwB,CAC5D,cAAc,IAAI,EAAE,EACpB,eAAe,IAAI,EAAE,EACrB,aAAa,CAAC,WAAW,CAAC,cAAc,CAAC,CAC1C,CAAC;IAEF,IAAI,CAAC;QACH,MAAM,QAAQ,CAAC,OAAO,EAAE,OAAO,EAAE,UAAU,CAAC,CAAC;QAC7C,MAAM,MAAM,GAAG,gBAAgB,CAC7B,OAAO,EACP,OAAO,EACP,cAAc,EACd,aAAa,CACd,CAAC;QAEF,OAAO,MAAM,CAAC;IAChB,CAAC;YAAS,CAAC;QACT,eAAe,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;IAC7C,CAAC;AACH,CAAC"}

package/dist/src/tools/modifiable-tool.test.d.ts

@@ -0,0 +1,6 @@
+/**
+ * @license
+ * Copyright 2025 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+export {};

package/dist/src/tools/modifiable-tool.test.js

@@ -0,0 +1,237 @@
+/**
+ * @license
+ * Copyright 2025 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+import { vi, describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { modifyWithEditor, isModifiableDeclarativeTool, } from './modifiable-tool.js';
+import { DEFAULT_GUI_EDITOR } from '../utils/editor.js';
+import fs from 'node:fs';
+import fsp from 'node:fs/promises';
+import os from 'node:os';
+import * as path from 'node:path';
+import { debugLogger } from '../utils/debugLogger.js';
+// Mock dependencies
+const mockOpenDiff = vi.hoisted(() => vi.fn());
+const mockCreatePatch = vi.hoisted(() => vi.fn());
+vi.mock('../utils/editor.js', async (importOriginal) => {
+    const actual = await importOriginal();
+    return {
+        ...actual,
+        openDiff: mockOpenDiff,
+    };
+});
+vi.mock('diff', () => ({
+    createPatch: mockCreatePatch,
+}));
+describe('modifyWithEditor', () => {
+    let testProjectDir;
+    let mockModifyContext;
+    let mockParams;
+    let currentContent;
+    let proposedContent;
+    let modifiedContent;
+    let abortSignal;
+    beforeEach(async () => {
+        vi.resetAllMocks();
+        testProjectDir = await fsp.mkdtemp(path.join(os.tmpdir(), 'modifiable-tool-test-'));
+        abortSignal = new AbortController().signal;
+        currentContent = 'original content\nline 2\nline 3';
+        proposedContent = 'modified content\nline 2\nline 3';
+        modifiedContent = 'user modified content\nline 2\nline 3\nnew line';
+        mockParams = {
+            filePath: path.join(testProjectDir, 'test.txt'),
+            someOtherParam: 'value',
+        };
+        mockModifyContext = {
+            getFilePath: vi.fn().mockReturnValue(mockParams.filePath),
+            getCurrentContent: vi.fn().mockResolvedValue(currentContent),
+            getProposedContent: vi.fn().mockResolvedValue(proposedContent),
+            createUpdatedParams: vi
+                .fn()
+                .mockImplementation((oldContent, modifiedContent, originalParams) => ({
+                ...originalParams,
+                modifiedContent,
+                oldContent,
+            })),
+        };
+        mockOpenDiff.mockImplementation(async (_oldPath, newPath) => {
+            await fsp.writeFile(newPath, modifiedContent, 'utf8');
+        });
+        mockCreatePatch.mockReturnValue('mock diff content');
+    });
+    afterEach(async () => {
+        vi.restoreAllMocks();
+        await fsp.rm(testProjectDir, { recursive: true, force: true });
+    });
+    describe('successful modification', () => {
+        const assertMode = (mode, expected) => {
+            if (process.platform === 'win32') {
+                // Windows reports POSIX modes as 0o666 regardless of requested bits.
+                // At minimum confirm the owner read/write bits are present.
+                expect(mode & 0o600).toBe(0o600);
+                return;
+            }
+            expect(mode & 0o777).toBe(expected);
+        };
+        it('should successfully modify content with VSCode editor', async () => {
+            const result = await modifyWithEditor(mockParams, mockModifyContext, DEFAULT_GUI_EDITOR, abortSignal);
+            expect(mockModifyContext.getCurrentContent).toHaveBeenCalledWith(mockParams);
+            expect(mockModifyContext.getProposedContent).toHaveBeenCalledWith(mockParams);
+            expect(mockModifyContext.getFilePath).toHaveBeenCalledWith(mockParams);
+            expect(mockOpenDiff).toHaveBeenCalledOnce();
+            const [oldFilePath, newFilePath] = mockOpenDiff.mock.calls[0];
+            expect(mockModifyContext.createUpdatedParams).toHaveBeenCalledWith(currentContent, modifiedContent, mockParams);
+            expect(mockCreatePatch).toHaveBeenCalledWith(path.basename(mockParams.filePath), currentContent, modifiedContent, 'Current', 'Proposed', expect.objectContaining({
+                context: 3,
+                ignoreWhitespace: false,
+            }));
+            // Check that temp files are deleted.
+            await expect(fsp.access(oldFilePath)).rejects.toThrow();
+            await expect(fsp.access(newFilePath)).rejects.toThrow();
+            expect(result).toEqual({
+                updatedParams: {
+                    ...mockParams,
+                    modifiedContent,
+                    oldContent: currentContent,
+                },
+                updatedDiff: 'mock diff content',
+            });
+        });
+        it('should create temp directory and files with restrictive permissions', async () => {
+            mockOpenDiff.mockImplementation(async (oldPath, newPath) => {
+                const diffDir = path.dirname(oldPath);
+                expect(diffDir).toBe(path.dirname(newPath));
+                const dirStats = await fsp.stat(diffDir);
+                const oldStats = await fsp.stat(oldPath);
+                const newStats = await fsp.stat(newPath);
+                assertMode(dirStats.mode, 0o700);
+                assertMode(oldStats.mode, 0o600);
+                assertMode(newStats.mode, 0o600);
+                await fsp.writeFile(newPath, modifiedContent, 'utf8');
+            });
+            await modifyWithEditor(mockParams, mockModifyContext, DEFAULT_GUI_EDITOR, abortSignal);
+            const [oldFilePath] = mockOpenDiff.mock.calls[0];
+            const diffDir = path.dirname(oldFilePath);
+            // Temp directory should be cleaned up after modification
+            await expect(fsp.stat(diffDir)).rejects.toThrow();
+        });
+    });
+    it('should handle missing old temp file gracefully', async () => {
+        mockOpenDiff.mockImplementation(async (oldPath, newPath) => {
+            await fsp.writeFile(newPath, modifiedContent, 'utf8');
+            await fsp.unlink(oldPath);
+        });
+        const result = await modifyWithEditor(mockParams, mockModifyContext, DEFAULT_GUI_EDITOR, abortSignal);
+        expect(mockCreatePatch).toHaveBeenCalledWith(path.basename(mockParams.filePath), '', modifiedContent, 'Current', 'Proposed', expect.objectContaining({
+            context: 3,
+            ignoreWhitespace: false,
+        }));
+        expect(result.updatedParams).toBeDefined();
+        expect(result.updatedDiff).toBe('mock diff content');
+    });
+    it('should handle missing new temp file gracefully', async () => {
+        mockOpenDiff.mockImplementation(async (_oldPath, newPath) => {
+            await fsp.unlink(newPath);
+        });
+        const result = await modifyWithEditor(mockParams, mockModifyContext, DEFAULT_GUI_EDITOR, abortSignal);
+        expect(mockCreatePatch).toHaveBeenCalledWith(path.basename(mockParams.filePath), currentContent, '', 'Current', 'Proposed', expect.objectContaining({
+            context: 3,
+            ignoreWhitespace: false,
+        }));
+        expect(result.updatedParams).toBeDefined();
+        expect(result.updatedDiff).toBe('mock diff content');
+    });
+    it('should honor override content values when provided', async () => {
+        const overrideCurrent = 'override current content';
+        const overrideProposed = 'override proposed content';
+        mockModifyContext.getCurrentContent = vi.fn();
+        mockModifyContext.getProposedContent = vi.fn();
+        await modifyWithEditor(mockParams, mockModifyContext, DEFAULT_GUI_EDITOR, abortSignal, {
+            currentContent: overrideCurrent,
+            proposedContent: overrideProposed,
+        });
+        expect(mockModifyContext.getCurrentContent).not.toHaveBeenCalled();
+        expect(mockModifyContext.getProposedContent).not.toHaveBeenCalled();
+        expect(mockCreatePatch).toHaveBeenCalledWith(path.basename(mockParams.filePath), overrideCurrent, modifiedContent, 'Current', 'Proposed', expect.any(Object));
+    });
+    it('should treat null override as explicit empty content', async () => {
+        mockModifyContext.getCurrentContent = vi.fn();
+        mockModifyContext.getProposedContent = vi.fn();
+        await modifyWithEditor(mockParams, mockModifyContext, DEFAULT_GUI_EDITOR, abortSignal, {
+            currentContent: null,
+            proposedContent: 'override proposed content',
+        });
+        expect(mockModifyContext.getCurrentContent).not.toHaveBeenCalled();
+        expect(mockModifyContext.getProposedContent).not.toHaveBeenCalled();
+        expect(mockCreatePatch).toHaveBeenCalledWith(path.basename(mockParams.filePath), '', modifiedContent, 'Current', 'Proposed', expect.any(Object));
+    });
+    it('should clean up temp files even if editor fails', async () => {
+        const editorError = new Error('Editor failed to open');
+        mockOpenDiff.mockRejectedValue(editorError);
+        const writeSpy = vi.spyOn(fs, 'writeFileSync');
+        await expect(modifyWithEditor(mockParams, mockModifyContext, DEFAULT_GUI_EDITOR, abortSignal)).rejects.toThrow('Editor failed to open');
+        expect(writeSpy).toHaveBeenCalledTimes(2);
+        const oldFilePath = writeSpy.mock.calls[0][0];
+        const newFilePath = writeSpy.mock.calls[1][0];
+        await expect(fsp.access(oldFilePath)).rejects.toThrow();
+        await expect(fsp.access(newFilePath)).rejects.toThrow();
+        writeSpy.mockRestore();
+    });
+    it('should handle temp file cleanup errors gracefully', async () => {
+        const consoleErrorSpy = vi
+            .spyOn(debugLogger, 'error')
+            .mockImplementation(() => { });
+        vi.spyOn(fs, 'unlinkSync').mockImplementation(() => {
+            throw new Error('Failed to delete file');
+        });
+        vi.spyOn(fs, 'rmdirSync').mockImplementation(() => {
+            throw new Error('Failed to delete directory');
+        });
+        await modifyWithEditor(mockParams, mockModifyContext, DEFAULT_GUI_EDITOR, abortSignal);
+        expect(consoleErrorSpy).toHaveBeenCalledTimes(3);
+        expect(consoleErrorSpy).toHaveBeenCalledWith(expect.stringContaining('Error deleting temp diff file:'));
+        expect(consoleErrorSpy).toHaveBeenCalledWith(expect.stringContaining('Error deleting temp diff directory:'));
+        consoleErrorSpy.mockRestore();
+    });
+    it('should create temp files with correct naming with extension', async () => {
+        const testFilePath = path.join(testProjectDir, 'subfolder', 'test-file.txt');
+        mockModifyContext.getFilePath = vi.fn().mockReturnValue(testFilePath);
+        await modifyWithEditor(mockParams, mockModifyContext, DEFAULT_GUI_EDITOR, abortSignal);
+        expect(mockOpenDiff).toHaveBeenCalledOnce();
+        const [oldFilePath, newFilePath] = mockOpenDiff.mock.calls[0];
+        expect(oldFilePath).toMatch(/gemini-cli-modify-test-file-old-\d+\.txt$/);
+        expect(newFilePath).toMatch(/gemini-cli-modify-test-file-new-\d+\.txt$/);
+        const diffDirPrefix = path.join(os.tmpdir(), 'gemini-cli-tool-modify-');
+        expect(path.dirname(oldFilePath).startsWith(diffDirPrefix)).toBe(true);
+        expect(path.dirname(newFilePath).startsWith(diffDirPrefix)).toBe(true);
+    });
+    it('should create temp files with correct naming without extension', async () => {
+        const testFilePath = path.join(testProjectDir, 'subfolder', 'test-file');
+        mockModifyContext.getFilePath = vi.fn().mockReturnValue(testFilePath);
+        await modifyWithEditor(mockParams, mockModifyContext, DEFAULT_GUI_EDITOR, abortSignal);
+        expect(mockOpenDiff).toHaveBeenCalledOnce();
+        const [oldFilePath, newFilePath] = mockOpenDiff.mock.calls[0];
+        expect(oldFilePath).toMatch(/gemini-cli-modify-test-file-old-\d+$/);
+        expect(newFilePath).toMatch(/gemini-cli-modify-test-file-new-\d+$/);
+        const diffDirPrefix = path.join(os.tmpdir(), 'gemini-cli-tool-modify-');
+        expect(path.dirname(oldFilePath).startsWith(diffDirPrefix)).toBe(true);
+        expect(path.dirname(newFilePath).startsWith(diffDirPrefix)).toBe(true);
+    });
+});
+describe('isModifiableTool', () => {
+    it('should return true for objects with getModifyContext method', () => {
+        const mockTool = {
+            name: 'test-tool',
+            getModifyContext: vi.fn(),
+        };
+        expect(isModifiableDeclarativeTool(mockTool)).toBe(true);
+    });
+    it('should return false for objects without getModifyContext method', () => {
+        const mockTool = {
+            name: 'test-tool',
+        };
+        expect(isModifiableDeclarativeTool(mockTool)).toBe(false);
+    });
+});
+//# sourceMappingURL=modifiable-tool.test.js.map

package/dist/src/tools/modifiable-tool.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"modifiable-tool.test.js","sourceRoot":"","sources":["../../../src/tools/modifiable-tool.test.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,EAAE,EAAE,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AAKzE,OAAO,EACL,gBAAgB,EAChB,2BAA2B,GAC5B,MAAM,sBAAsB,CAAC;AAC9B,OAAO,EAAE,kBAAkB,EAAE,MAAM,oBAAoB,CAAC;AACxD,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,GAAG,MAAM,kBAAkB,CAAC;AACnC,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,KAAK,IAAI,MAAM,WAAW,CAAC;AAClC,OAAO,EAAE,WAAW,EAAE,MAAM,yBAAyB,CAAC;AAEtD,oBAAoB;AACpB,MAAM,YAAY,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;AAC/C,MAAM,eAAe,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;AAElD,EAAE,CAAC,IAAI,CAAC,oBAAoB,EAAE,KAAK,EAAE,cAAc,EAAE,EAAE;IACrD,MAAM,MAAM,GAAG,MAAM,cAAc,EAAuC,CAAC;IAC3E,OAAO;QACL,GAAG,MAAM;QACT,QAAQ,EAAE,YAAY;KACvB,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,EAAE,CAAC,IAAI,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,CAAC;IACrB,WAAW,EAAE,eAAe;CAC7B,CAAC,CAAC,CAAC;AAQJ,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,IAAI,cAAsB,CAAC;IAC3B,IAAI,iBAA4C,CAAC;IACjD,IAAI,UAAsB,CAAC;IAC3B,IAAI,cAAsB,CAAC;IAC3B,IAAI,eAAuB,CAAC;IAC5B,IAAI,eAAuB,CAAC;IAC5B,IAAI,WAAwB,CAAC;IAE7B,UAAU,CAAC,KAAK,IAAI,EAAE;QACpB,EAAE,CAAC,aAAa,EAAE,CAAC;QAEnB,cAAc,GAAG,MAAM,GAAG,CAAC,OAAO,CAChC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,uBAAuB,CAAC,CAChD,CAAC;QACF,WAAW,GAAG,IAAI,eAAe,EAAE,CAAC,MAAM,CAAC;QAE3C,cAAc,GAAG,kCAAkC,CAAC;QACpD,eAAe,GAAG,kCAAkC,CAAC;QACrD,eAAe,GAAG,iDAAiD,CAAC;QACpE,UAAU,GAAG;YACX,QAAQ,EAAE,IAAI,CAAC,IAAI,CAAC,cAAc,EAAE,UAAU,CAAC;YAC/C,cAAc,EAAE,OAAO;SACxB,CAAC;QAEF,iBAAiB,GAAG;YAClB,WAAW,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,eAAe,CAAC,UAAU,CAAC,QAAQ,CAAC;YACzD,iBAAiB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,cAAc,CAAC;YAC5D,kBAAkB,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,eAAe,CAAC;YAC9D,mBAAmB,EAAE,EAAE;iBACpB,EAAE,EAAE;iBACJ,kBAAkB,CAAC,CAAC,UAAU,EAAE,eAAe,EAAE,cAAc,EAAE,EAAE,CAAC,CAAC;gBACpE,GAAG,cAAc;gBACjB,eAAe;gBACf,UAAU;aACX,CAAC,CAAC;SACN,CAAC;QAEF,YAAY,CAAC,kBAAkB,CAAC,KAAK,EAAE,QAAQ,EAAE,OAAO,EAAE,EAAE;YAC1D,MAAM,GAAG,CAAC,SAAS,CAAC,OAAO,EAAE,eAAe,EAAE,MAAM,CAAC,CAAC;QACxD,CAAC,CAAC,CAAC;QAEH,eAAe,CAAC,eAAe,CAAC,mBAAmB,CAAC,CAAC;IACvD,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,KAAK,IAAI,EAAE;QACnB,EAAE,CAAC,eAAe,EAAE,CAAC;QACrB,MAAM,GAAG,CAAC,EAAE,CAAC,cAAc,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACjE,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;QACvC,MAAM,UAAU,GAAG,CAAC,IAAY,EAAE,QAAgB,EAAQ,EAAE;YAC1D,IAAI,OAAO,CAAC,QAAQ,KAAK,OAAO,EAAE,CAAC;gBACjC,qEAAqE;gBACrE,4DAA4D;gBAC5D,MAAM,CAAC,IAAI,GAAG,KAAK,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;gBACjC,OAAO;YACT,CAAC;YACD,MAAM,CAAC,IAAI,GAAG,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACtC,CAAC,CAAC;QAEF,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;YACrE,MAAM,MAAM,GAAG,MAAM,gBAAgB,CACnC,UAAU,EACV,iBAAiB,EACjB,kBAAkB,EAClB,WAAW,CACZ,CAAC;YAEF,MAAM,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC,oBAAoB,CAC9D,UAAU,CACX,CAAC;YACF,MAAM,CAAC,iBAAiB,CAAC,kBAAkB,CAAC,CAAC,oBAAoB,CAC/D,UAAU,CACX,CAAC;YACF,MAAM,CAAC,iBAAiB,CAAC,WAAW,CAAC,CAAC,oBAAoB,CAAC,UAAU,CAAC,CAAC;YAEvE,MAAM,CAAC,YAAY,CAAC,CAAC,oBAAoB,EAAE,CAAC;YAC5C,MAAM,CAAC,WAAW,EAAE,WAAW,CAAC,GAAG,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;YAE9D,MAAM,CAAC,iBAAiB,CAAC,mBAAmB,CAAC,CAAC,oBAAoB,CAChE,cAAc,EACd,eAAe,EACf,UAAU,CACX,CAAC;YAEF,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAC1C,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,QAAQ,CAAC,EAClC,cAAc,EACd,eAAe,EACf,SAAS,EACT,UAAU,EACV,MAAM,CAAC,gBAAgB,CAAC;gBACtB,OAAO,EAAE,CAAC;gBACV,gBAAgB,EAAE,KAAK;aACxB,CAAC,CACH,CAAC;YAEF,qCAAqC;YACrC,MAAM,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,CAAC;YACxD,MAAM,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,CAAC;YAExD,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;gBACrB,aAAa,EAAE;oBACb,GAAG,UAAU;oBACb,eAAe;oBACf,UAAU,EAAE,cAAc;iBAC3B;gBACD,WAAW,EAAE,mBAAmB;aACjC,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qEAAqE,EAAE,KAAK,IAAI,EAAE;YACnF,YAAY,CAAC,kBAAkB,CAAC,KAAK,EAAE,OAAO,EAAE,OAAO,EAAE,EAAE;gBACzD,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;gBACtC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC;gBAE5C,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;gBACzC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;gBACzC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;gBAEzC,UAAU,CAAC,QAAQ,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;gBACjC,UAAU,CAAC,QAAQ,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;gBACjC,UAAU,CAAC,QAAQ,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;gBAEjC,MAAM,GAAG,CAAC,SAAS,CAAC,OAAO,EAAE,eAAe,EAAE,MAAM,CAAC,CAAC;YACxD,CAAC,CAAC,CAAC;YAEH,MAAM,gBAAgB,CACpB,UAAU,EACV,iBAAiB,EACjB,kBAAkB,EAClB,WAAW,CACZ,CAAC;YAEF,MAAM,CAAC,WAAW,CAAC,GAAG,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;YACjD,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC;YAC1C,yDAAyD;YACzD,MAAM,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,CAAC;QACpD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAC9D,YAAY,CAAC,kBAAkB,CAAC,KAAK,EAAE,OAAO,EAAE,OAAO,EAAE,EAAE;YACzD,MAAM,GAAG,CAAC,SAAS,CAAC,OAAO,EAAE,eAAe,EAAE,MAAM,CAAC,CAAC;YACtD,MAAM,GAAG,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;QAC5B,CAAC,CAAC,CAAC;QAEH,MAAM,MAAM,GAAG,MAAM,gBAAgB,CACnC,UAAU,EACV,iBAAiB,EACjB,kBAAkB,EAClB,WAAW,CACZ,CAAC;QAEF,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAC1C,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,QAAQ,CAAC,EAClC,EAAE,EACF,eAAe,EACf,SAAS,EACT,UAAU,EACV,MAAM,CAAC,gBAAgB,CAAC;YACtB,OAAO,EAAE,CAAC;YACV,gBAAgB,EAAE,KAAK;SACxB,CAAC,CACH,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,WAAW,EAAE,CAAC;QAC3C,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC;IACvD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAC9D,YAAY,CAAC,kBAAkB,CAAC,KAAK,EAAE,QAAQ,EAAE,OAAO,EAAE,EAAE;YAC1D,MAAM,GAAG,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;QAC5B,CAAC,CAAC,CAAC;QAEH,MAAM,MAAM,GAAG,MAAM,gBAAgB,CACnC,UAAU,EACV,iBAAiB,EACjB,kBAAkB,EAClB,WAAW,CACZ,CAAC;QAEF,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAC1C,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,QAAQ,CAAC,EAClC,cAAc,EACd,EAAE,EACF,SAAS,EACT,UAAU,EACV,MAAM,CAAC,gBAAgB,CAAC;YACtB,OAAO,EAAE,CAAC;YACV,gBAAgB,EAAE,KAAK;SACxB,CAAC,CACH,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,WAAW,EAAE,CAAC;QAC3C,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC;IACvD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;QAClE,MAAM,eAAe,GAAG,0BAA0B,CAAC;QACnD,MAAM,gBAAgB,GAAG,2BAA2B,CAAC;QACrD,iBAAiB,CAAC,iBAAiB,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;QAC9C,iBAAiB,CAAC,kBAAkB,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;QAE/C,MAAM,gBAAgB,CACpB,UAAU,EACV,iBAAiB,EACjB,kBAAkB,EAClB,WAAW,EACX;YACE,cAAc,EAAE,eAAe;YAC/B,eAAe,EAAE,gBAAgB;SAClC,CACF,CAAC;QAEF,MAAM,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QACnE,MAAM,CAAC,iBAAiB,CAAC,kBAAkB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QACpE,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAC1C,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,QAAQ,CAAC,EAClC,eAAe,EACf,eAAe,EACf,SAAS,EACT,UAAU,EACV,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,CACnB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACpE,iBAAiB,CAAC,iBAAiB,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;QAC9C,iBAAiB,CAAC,kBAAkB,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;QAE/C,MAAM,gBAAgB,CACpB,UAAU,EACV,iBAAiB,EACjB,kBAAkB,EAClB,WAAW,EACX;YACE,cAAc,EAAE,IAAI;YACpB,eAAe,EAAE,2BAA2B;SAC7C,CACF,CAAC;QAEF,MAAM,CAAC,iBAAiB,CAAC,iBAAiB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QACnE,MAAM,CAAC,iBAAiB,CAAC,kBAAkB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QACpE,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAC1C,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,QAAQ,CAAC,EAClC,EAAE,EACF,eAAe,EACf,SAAS,EACT,UAAU,EACV,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,CACnB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,MAAM,WAAW,GAAG,IAAI,KAAK,CAAC,uBAAuB,CAAC,CAAC;QACvD,YAAY,CAAC,iBAAiB,CAAC,WAAW,CAAC,CAAC;QAE5C,MAAM,QAAQ,GAAG,EAAE,CAAC,KAAK,CAAC,EAAE,EAAE,eAAe,CAAC,CAAC;QAE/C,MAAM,MAAM,CACV,gBAAgB,CACd,UAAU,EACV,iBAAiB,EACjB,kBAAkB,EAClB,WAAW,CACZ,CACF,CAAC,OAAO,CAAC,OAAO,CAAC,uBAAuB,CAAC,CAAC;QAE3C,MAAM,CAAC,QAAQ,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QAC1C,MAAM,WAAW,GAAG,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAW,CAAC;QACxD,MAAM,WAAW,GAAG,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAW,CAAC;QAExD,MAAM,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,CAAC;QACxD,MAAM,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,CAAC;QAExD,QAAQ,CAAC,WAAW,EAAE,CAAC;IACzB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QACjE,MAAM,eAAe,GAAG,EAAE;aACvB,KAAK,CAAC,WAAW,EAAE,OAAO,CAAC;aAC3B,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QAChC,EAAE,CAAC,KAAK,CAAC,EAAE,EAAE,YAAY,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE;YACjD,MAAM,IAAI,KAAK,CAAC,uBAAuB,CAAC,CAAC;QAC3C,CAAC,CAAC,CAAC;QACH,EAAE,CAAC,KAAK,CAAC,EAAE,EAAE,WAAW,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE;YAChD,MAAM,IAAI,KAAK,CAAC,4BAA4B,CAAC,CAAC;QAChD,CAAC,CAAC,CAAC;QAEH,MAAM,gBAAgB,CACpB,UAAU,EACV,iBAAiB,EACjB,kBAAkB,EAClB,WAAW,CACZ,CAAC;QAEF,MAAM,CAAC,eAAe,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QACjD,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAC1C,MAAM,CAAC,gBAAgB,CAAC,gCAAgC,CAAC,CAC1D,CAAC;QACF,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAC1C,MAAM,CAAC,gBAAgB,CAAC,qCAAqC,CAAC,CAC/D,CAAC;QAEF,eAAe,CAAC,WAAW,EAAE,CAAC;IAChC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;QAC3E,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAC5B,cAAc,EACd,WAAW,EACX,eAAe,CAChB,CAAC;QACF,iBAAiB,CAAC,WAAW,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,eAAe,CAAC,YAAY,CAAC,CAAC;QAEtE,MAAM,gBAAgB,CACpB,UAAU,EACV,iBAAiB,EACjB,kBAAkB,EAClB,WAAW,CACZ,CAAC;QAEF,MAAM,CAAC,YAAY,CAAC,CAAC,oBAAoB,EAAE,CAAC;QAC5C,MAAM,CAAC,WAAW,EAAE,WAAW,CAAC,GAAG,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;QAC9D,MAAM,CAAC,WAAW,CAAC,CAAC,OAAO,CAAC,2CAA2C,CAAC,CAAC;QACzE,MAAM,CAAC,WAAW,CAAC,CAAC,OAAO,CAAC,2CAA2C,CAAC,CAAC;QAEzE,MAAM,aAAa,GAAG,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,yBAAyB,CAAC,CAAC;QACxE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACvE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACzE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAC9E,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,cAAc,EAAE,WAAW,EAAE,WAAW,CAAC,CAAC;QACzE,iBAAiB,CAAC,WAAW,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,eAAe,CAAC,YAAY,CAAC,CAAC;QAEtE,MAAM,gBAAgB,CACpB,UAAU,EACV,iBAAiB,EACjB,kBAAkB,EAClB,WAAW,CACZ,CAAC;QAEF,MAAM,CAAC,YAAY,CAAC,CAAC,oBAAoB,EAAE,CAAC;QAC5C,MAAM,CAAC,WAAW,EAAE,WAAW,CAAC,GAAG,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;QAC9D,MAAM,CAAC,WAAW,CAAC,CAAC,OAAO,CAAC,sCAAsC,CAAC,CAAC;QACpE,MAAM,CAAC,WAAW,CAAC,CAAC,OAAO,CAAC,sCAAsC,CAAC,CAAC;QAEpE,MAAM,aAAa,GAAG,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,yBAAyB,CAAC,CAAC;QACxE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACvE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACzE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACrE,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,WAAW;YACjB,gBAAgB,EAAE,EAAE,CAAC,EAAE,EAAE;SAC0B,CAAC;QAEtD,MAAM,CAAC,2BAA2B,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC3D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QACzE,MAAM,QAAQ,GAAG;YACf,IAAI,EAAE,WAAW;SACkC,CAAC;QAEtD,MAAM,CAAC,2BAA2B,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/src/tools/read-file.d.ts

@@ -0,0 +1,36 @@
+/**
+ * @license
+ * Copyright 2025 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+import type { MessageBus } from '../confirmation-bus/message-bus.js';
+import type { ToolInvocation, ToolResult } from './tools.js';
+import { BaseDeclarativeTool } from './tools.js';
+import type { Config } from '../config/config.js';
+/**
+ * Parameters for the ReadFile tool
+ */
+export interface ReadFileToolParams {
+    /**
+     * The path to the file to read
+     */
+    file_path: string;
+    /**
+     * The line number to start reading from (optional)
+     */
+    offset?: number;
+    /**
+     * The number of lines to read (optional)
+     */
+    limit?: number;
+}
+/**
+ * Implementation of the ReadFile tool logic
+ */
+export declare class ReadFileTool extends BaseDeclarativeTool<ReadFileToolParams, ToolResult> {
+    private config;
+    static readonly Name = "read_file";
+    constructor(config: Config, messageBus: MessageBus);
+    protected validateToolParamValues(params: ReadFileToolParams): string | null;
+    protected createInvocation(params: ReadFileToolParams, messageBus: MessageBus, _toolName?: string, _toolDisplayName?: string): ToolInvocation<ReadFileToolParams, ToolResult>;
+}

package/dist/src/tools/read-file.js

@@ -0,0 +1,133 @@
+/**
+ * @license
+ * Copyright 2025 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+import path from 'node:path';
+import { makeRelative, shortenPath } from '../utils/paths.js';
+import { BaseDeclarativeTool, BaseToolInvocation, Kind } from './tools.js';
+import { processSingleFileContent, getSpecificMimeType, } from '../utils/fileUtils.js';
+import { FileOperation } from '../telemetry/metrics.js';
+import { getProgrammingLanguage } from '../telemetry/telemetry-utils.js';
+import { logFileOperation } from '../telemetry/loggers.js';
+import { FileOperationEvent } from '../telemetry/types.js';
+import { READ_FILE_TOOL_NAME } from './tool-names.js';
+class ReadFileToolInvocation extends BaseToolInvocation {
+    config;
+    resolvedPath;
+    constructor(config, params, messageBus, _toolName, _toolDisplayName) {
+        super(params, messageBus, _toolName, _toolDisplayName);
+        this.config = config;
+        this.resolvedPath = path.resolve(this.config.getTargetDir(), this.params.file_path);
+    }
+    getDescription() {
+        const relativePath = makeRelative(this.resolvedPath, this.config.getTargetDir());
+        return shortenPath(relativePath);
+    }
+    toolLocations() {
+        return [{ path: this.resolvedPath, line: this.params.offset }];
+    }
+    async execute() {
+        const result = await processSingleFileContent(this.resolvedPath, this.config.getTargetDir(), this.config.getFileSystemService(), this.params.offset, this.params.limit);
+        if (result.error) {
+            return {
+                llmContent: result.llmContent,
+                returnDisplay: result.returnDisplay || 'Error reading file',
+                error: {
+                    message: result.error,
+                    type: result.errorType,
+                },
+            };
+        }
+        let llmContent;
+        if (result.isTruncated) {
+            const [start, end] = result.linesShown;
+            const total = result.originalLineCount;
+            const nextOffset = this.params.offset
+                ? this.params.offset + end - start + 1
+                : end;
+            llmContent = `
+IMPORTANT: The file content has been truncated.
+Status: Showing lines ${start}-${end} of ${total} total lines.
+Action: To read more of the file, you can use the 'offset' and 'limit' parameters in a subsequent 'read_file' call. For example, to read the next section of the file, use offset: ${nextOffset}.
+
+--- FILE CONTENT (truncated) ---
+${result.llmContent}`;
+        }
+        else {
+            llmContent = result.llmContent || '';
+        }
+        const lines = typeof result.llmContent === 'string'
+            ? result.llmContent.split('\n').length
+            : undefined;
+        const mimetype = getSpecificMimeType(this.resolvedPath);
+        const programming_language = getProgrammingLanguage({
+            file_path: this.resolvedPath,
+        });
+        logFileOperation(this.config, new FileOperationEvent(READ_FILE_TOOL_NAME, FileOperation.READ, lines, mimetype, path.extname(this.resolvedPath), programming_language));
+        return {
+            llmContent,
+            returnDisplay: result.returnDisplay || '',
+        };
+    }
+}
+/**
+ * Implementation of the ReadFile tool logic
+ */
+export class ReadFileTool extends BaseDeclarativeTool {
+    config;
+    static Name = READ_FILE_TOOL_NAME;
+    constructor(config, messageBus) {
+        super(ReadFileTool.Name, 'ReadFile', `Reads and returns the content of a specified file. If the file is large, the content will be truncated. The tool's response will clearly indicate if truncation has occurred and will provide details on how to read more of the file using the 'offset' and 'limit' parameters. Handles text, images (PNG, JPG, GIF, WEBP, SVG, BMP), audio files (MP3, WAV, AIFF, AAC, OGG, FLAC), and PDF files. For text files, it can read specific line ranges.`, Kind.Read, {
+            properties: {
+                file_path: {
+                    description: 'The path to the file to read.',
+                    type: 'string',
+                },
+                offset: {
+                    description: "Optional: For text files, the 0-based line number to start reading from. Requires 'limit' to be set. Use for paginating through large files.",
+                    type: 'number',
+                },
+                limit: {
+                    description: "Optional: For text files, maximum number of lines to read. Use with 'offset' to paginate through large files. If omitted, reads the entire file (if feasible, up to a default limit).",
+                    type: 'number',
+                },
+            },
+            required: ['file_path'],
+            type: 'object',
+        }, messageBus, true, false);
+        this.config = config;
+    }
+    validateToolParamValues(params) {
+        if (params.file_path.trim() === '') {
+            return "The 'file_path' parameter must be non-empty.";
+        }
+        const workspaceContext = this.config.getWorkspaceContext();
+        const projectTempDir = this.config.storage.getProjectTempDir();
+        const resolvedPath = path.resolve(this.config.getTargetDir(), params.file_path);
+        const resolvedProjectTempDir = path.resolve(projectTempDir);
+        const isWithinTempDir = resolvedPath.startsWith(resolvedProjectTempDir + path.sep) ||
+            resolvedPath === resolvedProjectTempDir;
+        if (!workspaceContext.isPathWithinWorkspace(resolvedPath) &&
+            !isWithinTempDir) {
+            const directories = workspaceContext.getDirectories();
+            return `File path must be within one of the workspace directories: ${directories.join(', ')} or within the project temp directory: ${projectTempDir}`;
+        }
+        if (params.offset !== undefined && params.offset < 0) {
+            return 'Offset must be a non-negative number';
+        }
+        if (params.limit !== undefined && params.limit <= 0) {
+            return 'Limit must be a positive number';
+        }
+        const fileService = this.config.getFileService();
+        const fileFilteringOptions = this.config.getFileFilteringOptions();
+        if (fileService.shouldIgnoreFile(resolvedPath, fileFilteringOptions)) {
+            return `File path '${resolvedPath}' is ignored by configured ignore patterns.`;
+        }
+        return null;
+    }
+    createInvocation(params, messageBus, _toolName, _toolDisplayName) {
+        return new ReadFileToolInvocation(this.config, params, messageBus, _toolName, _toolDisplayName);
+    }
+}
+//# sourceMappingURL=read-file.js.map

package/dist/src/tools/read-file.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"read-file.js","sourceRoot":"","sources":["../../../src/tools/read-file.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAGH,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAE9D,OAAO,EAAE,mBAAmB,EAAE,kBAAkB,EAAE,IAAI,EAAE,MAAM,YAAY,CAAC;AAG3E,OAAO,EACL,wBAAwB,EACxB,mBAAmB,GACpB,MAAM,uBAAuB,CAAC;AAE/B,OAAO,EAAE,aAAa,EAAE,MAAM,yBAAyB,CAAC;AACxD,OAAO,EAAE,sBAAsB,EAAE,MAAM,iCAAiC,CAAC;AACzE,OAAO,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAC3D,OAAO,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAC3D,OAAO,EAAE,mBAAmB,EAAE,MAAM,iBAAiB,CAAC;AAsBtD,MAAM,sBAAuB,SAAQ,kBAGpC;IAGW;IAFO,YAAY,CAAS;IACtC,YACU,MAAc,EACtB,MAA0B,EAC1B,UAAsB,EACtB,SAAkB,EAClB,gBAAyB;QAEzB,KAAK,CAAC,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,gBAAgB,CAAC,CAAC;QAN/C,WAAM,GAAN,MAAM,CAAQ;QAOtB,IAAI,CAAC,YAAY,GAAG,IAAI,CAAC,OAAO,CAC9B,IAAI,CAAC,MAAM,CAAC,YAAY,EAAE,EAC1B,IAAI,CAAC,MAAM,CAAC,SAAS,CACtB,CAAC;IACJ,CAAC;IAED,cAAc;QACZ,MAAM,YAAY,GAAG,YAAY,CAC/B,IAAI,CAAC,YAAY,EACjB,IAAI,CAAC,MAAM,CAAC,YAAY,EAAE,CAC3B,CAAC;QACF,OAAO,WAAW,CAAC,YAAY,CAAC,CAAC;IACnC,CAAC;IAEQ,aAAa;QACpB,OAAO,CAAC,EAAE,IAAI,EAAE,IAAI,CAAC,YAAY,EAAE,IAAI,EAAE,IAAI,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC;IACjE,CAAC;IAED,KAAK,CAAC,OAAO;QACX,MAAM,MAAM,GAAG,MAAM,wBAAwB,CAC3C,IAAI,CAAC,YAAY,EACjB,IAAI,CAAC,MAAM,CAAC,YAAY,EAAE,EAC1B,IAAI,CAAC,MAAM,CAAC,oBAAoB,EAAE,EAClC,IAAI,CAAC,MAAM,CAAC,MAAM,EAClB,IAAI,CAAC,MAAM,CAAC,KAAK,CAClB,CAAC;QAEF,IAAI,MAAM,CAAC,KAAK,EAAE,CAAC;YACjB,OAAO;gBACL,UAAU,EAAE,MAAM,CAAC,UAAU;gBAC7B,aAAa,EAAE,MAAM,CAAC,aAAa,IAAI,oBAAoB;gBAC3D,KAAK,EAAE;oBACL,OAAO,EAAE,MAAM,CAAC,KAAK;oBACrB,IAAI,EAAE,MAAM,CAAC,SAAS;iBACvB;aACF,CAAC;QACJ,CAAC;QAED,IAAI,UAAqB,CAAC;QAC1B,IAAI,MAAM,CAAC,WAAW,EAAE,CAAC;YACvB,MAAM,CAAC,KAAK,EAAE,GAAG,CAAC,GAAG,MAAM,CAAC,UAAW,CAAC;YACxC,MAAM,KAAK,GAAG,MAAM,CAAC,iBAAkB,CAAC;YACxC,MAAM,UAAU,GAAG,IAAI,CAAC,MAAM,CAAC,MAAM;gBACnC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,MAAM,GAAG,GAAG,GAAG,KAAK,GAAG,CAAC;gBACtC,CAAC,CAAC,GAAG,CAAC;YACR,UAAU,GAAG;;wBAEK,KAAK,IAAI,GAAG,OAAO,KAAK;qLACqI,UAAU;;;EAG7L,MAAM,CAAC,UAAU,EAAE,CAAC;QAClB,CAAC;aAAM,CAAC;YACN,UAAU,GAAG,MAAM,CAAC,UAAU,IAAI,EAAE,CAAC;QACvC,CAAC;QAED,MAAM,KAAK,GACT,OAAO,MAAM,CAAC,UAAU,KAAK,QAAQ;YACnC,CAAC,CAAC,MAAM,CAAC,UAAU,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM;YACtC,CAAC,CAAC,SAAS,CAAC;QAChB,MAAM,QAAQ,GAAG,mBAAmB,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;QACxD,MAAM,oBAAoB,GAAG,sBAAsB,CAAC;YAClD,SAAS,EAAE,IAAI,CAAC,YAAY;SAC7B,CAAC,CAAC;QACH,gBAAgB,CACd,IAAI,CAAC,MAAM,EACX,IAAI,kBAAkB,CACpB,mBAAmB,EACnB,aAAa,CAAC,IAAI,EAClB,KAAK,EACL,QAAQ,EACR,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,YAAY,CAAC,EAC/B,oBAAoB,CACrB,CACF,CAAC;QAEF,OAAO;YACL,UAAU;YACV,aAAa,EAAE,MAAM,CAAC,aAAa,IAAI,EAAE;SAC1C,CAAC;IACJ,CAAC;CACF;AAED;;GAEG;AACH,MAAM,OAAO,YAAa,SAAQ,mBAGjC;IAIW;IAHV,MAAM,CAAU,IAAI,GAAG,mBAAmB,CAAC;IAE3C,YACU,MAAc,EACtB,UAAsB;QAEtB,KAAK,CACH,YAAY,CAAC,IAAI,EACjB,UAAU,EACV,ubAAub,EACvb,IAAI,CAAC,IAAI,EACT;YACE,UAAU,EAAE;gBACV,SAAS,EAAE;oBACT,WAAW,EAAE,+BAA+B;oBAC5C,IAAI,EAAE,QAAQ;iBACf;gBACD,MAAM,EAAE;oBACN,WAAW,EACT,8IAA8I;oBAChJ,IAAI,EAAE,QAAQ;iBACf;gBACD,KAAK,EAAE;oBACL,WAAW,EACT,uLAAuL;oBACzL,IAAI,EAAE,QAAQ;iBACf;aACF;YACD,QAAQ,EAAE,CAAC,WAAW,CAAC;YACvB,IAAI,EAAE,QAAQ;SACf,EACD,UAAU,EACV,IAAI,EACJ,KAAK,CACN,CAAC;QA/BM,WAAM,GAAN,MAAM,CAAQ;IAgCxB,CAAC;IAEkB,uBAAuB,CACxC,MAA0B;QAE1B,IAAI,MAAM,CAAC,SAAS,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC;YACnC,OAAO,8CAA8C,CAAC;QACxD,CAAC;QAED,MAAM,gBAAgB,GAAG,IAAI,CAAC,MAAM,CAAC,mBAAmB,EAAE,CAAC;QAC3D,MAAM,cAAc,GAAG,IAAI,CAAC,MAAM,CAAC,OAAO,CAAC,iBAAiB,EAAE,CAAC;QAC/D,MAAM,YAAY,GAAG,IAAI,CAAC,OAAO,CAC/B,IAAI,CAAC,MAAM,CAAC,YAAY,EAAE,EAC1B,MAAM,CAAC,SAAS,CACjB,CAAC;QACF,MAAM,sBAAsB,GAAG,IAAI,CAAC,OAAO,CAAC,cAAc,CAAC,CAAC;QAC5D,MAAM,eAAe,GACnB,YAAY,CAAC,UAAU,CAAC,sBAAsB,GAAG,IAAI,CAAC,GAAG,CAAC;YAC1D,YAAY,KAAK,sBAAsB,CAAC;QAE1C,IACE,CAAC,gBAAgB,CAAC,qBAAqB,CAAC,YAAY,CAAC;YACrD,CAAC,eAAe,EAChB,CAAC;YACD,MAAM,WAAW,GAAG,gBAAgB,CAAC,cAAc,EAAE,CAAC;YACtD,OAAO,8DAA8D,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,0CAA0C,cAAc,EAAE,CAAC;QACxJ,CAAC;QACD,IAAI,MAAM,CAAC,MAAM,KAAK,SAAS,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACrD,OAAO,sCAAsC,CAAC;QAChD,CAAC;QACD,IAAI,MAAM,CAAC,KAAK,KAAK,SAAS,IAAI,MAAM,CAAC,KAAK,IAAI,CAAC,EAAE,CAAC;YACpD,OAAO,iCAAiC,CAAC;QAC3C,CAAC;QAED,MAAM,WAAW,GAAG,IAAI,CAAC,MAAM,CAAC,cAAc,EAAE,CAAC;QACjD,MAAM,oBAAoB,GAAG,IAAI,CAAC,MAAM,CAAC,uBAAuB,EAAE,CAAC;QACnE,IAAI,WAAW,CAAC,gBAAgB,CAAC,YAAY,EAAE,oBAAoB,CAAC,EAAE,CAAC;YACrE,OAAO,cAAc,YAAY,6CAA6C,CAAC;QACjF,CAAC;QAED,OAAO,IAAI,CAAC;IACd,CAAC;IAES,gBAAgB,CACxB,MAA0B,EAC1B,UAAsB,EACtB,SAAkB,EAClB,gBAAyB;QAEzB,OAAO,IAAI,sBAAsB,CAC/B,IAAI,CAAC,MAAM,EACX,MAAM,EACN,UAAU,EACV,SAAS,EACT,gBAAgB,CACjB,CAAC;IACJ,CAAC"}

package/dist/src/tools/read-file.test.d.ts

@@ -0,0 +1,6 @@
+/**
+ * @license
+ * Copyright 2025 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+export {};