@office-ai/aioncli-core 0.18.7 → 0.24.0

package/dist/src/core/coreToolScheduler.test.js

@@ -3,36 +3,36 @@
  * Copyright 2025 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { CoreToolScheduler, convertToFunctionResponse, truncateAndSaveToFile, } from './coreToolScheduler.js';
-import { DEFAULT_TRUNCATE_TOOL_OUTPUT_LINES, DEFAULT_TRUNCATE_TOOL_OUTPUT_THRESHOLD, BaseDeclarativeTool, BaseToolInvocation, ToolConfirmationOutcome, Kind, ApprovalMode, } from '../index.js';
+import { describe, it, expect, vi } from 'vitest';
+import { CoreToolScheduler } from './coreToolScheduler.js';
+import { DEFAULT_TRUNCATE_TOOL_OUTPUT_LINES, DEFAULT_TRUNCATE_TOOL_OUTPUT_THRESHOLD, BaseDeclarativeTool, BaseToolInvocation, ToolConfirmationOutcome, Kind, ApprovalMode, HookSystem, } from '../index.js';
+import { createMockMessageBus } from '../test-utils/mock-message-bus.js';
 import { MockModifiableTool, MockTool, MOCK_TOOL_SHOULD_CONFIRM_EXECUTE, } from '../test-utils/mock-tool.js';
 import * as modifiableToolModule from '../tools/modifiable-tool.js';
-import * as fs from 'node:fs/promises';
-import * as path from 'node:path';
-import { isShellInvocationAllowlisted } from '../utils/shell-utils.js';
+import { isShellInvocationAllowlisted } from '../utils/shell-permissions.js';
+import { DEFAULT_GEMINI_MODEL } from '../config/models.js';
 vi.mock('fs/promises', () => ({
     writeFile: vi.fn(),
 }));
 class TestApprovalTool extends BaseDeclarativeTool {
     config;
     static Name = 'testApprovalTool';
-    constructor(config) {
+    constructor(config, messageBus) {
         super(TestApprovalTool.Name, 'TestApprovalTool', 'A tool for testing approval logic', Kind.Edit, {
             properties: { id: { type: 'string' } },
             required: ['id'],
             type: 'object',
-        });
+        }, messageBus);
         this.config = config;
     }
-    createInvocation(params) {
-        return new TestApprovalInvocation(this.config, params);
+    createInvocation(params, messageBus, _toolName, _toolDisplayName) {
+        return new TestApprovalInvocation(this.config, params, messageBus);
     }
 }
 class TestApprovalInvocation extends BaseToolInvocation {
     config;
-    constructor(config, params) {
-        super(params);
+    constructor(config, params, messageBus) {
+        super(params, messageBus);
         this.config = config;
     }
     getDescription() {
@@ -68,8 +68,8 @@ class TestApprovalInvocation extends BaseToolInvocation {
 class AbortDuringConfirmationInvocation extends BaseToolInvocation {
     abortController;
     abortError;
-    constructor(abortController, abortError, params) {
-        super(params);
+    constructor(abortController, abortError, params, messageBus) {
+        super(params, messageBus);
         this.abortController = abortController;
         this.abortError = abortError;
     }
@@ -87,16 +87,16 @@ class AbortDuringConfirmationInvocation extends BaseToolInvocation {
 class AbortDuringConfirmationTool extends BaseDeclarativeTool {
     abortController;
     abortError;
-    constructor(abortController, abortError) {
+    constructor(abortController, abortError, messageBus) {
         super('abortDuringConfirmationTool', 'Abort During Confirmation Tool', 'A tool that aborts while confirming execution.', Kind.Other, {
             type: 'object',
             properties: {},
-        });
+        }, messageBus);
         this.abortController = abortController;
         this.abortError = abortError;
     }
-    createInvocation(params) {
-        return new AbortDuringConfirmationInvocation(this.abortController, this.abortError, params);
+    createInvocation(params, messageBus, _toolName, _toolDisplayName) {
+        return new AbortDuringConfirmationInvocation(this.abortController, this.abortError, params, messageBus);
     }
 }
 async function waitForStatus(onToolCallsUpdate, status, timeout = 5000) {
@@ -136,11 +136,13 @@ function createMockConfig(overrides = {}) {
         discoverTools: async () => { },
         getAllTools: () => [],
         getToolsByServer: () => [],
+        getExperiments: () => { },
     };
     const baseConfig = {
         getSessionId: () => 'test-session-id',
         getUsageStatisticsEnabled: () => true,
         getDebugMode: () => false,
+        isInteractive: () => true,
         getApprovalMode: () => ApprovalMode.DEFAULT,
         setApprovalMode: () => { },
         getAllowedTools: () => [],
@@ -151,6 +153,11 @@ function createMockConfig(overrides = {}) {
         getShellExecutionConfig: () => ({
             terminalWidth: 90,
             terminalHeight: 30,
+            sanitizationConfig: {
+                enableEnvironmentVariableRedaction: true,
+                allowedEnvironmentVariables: [],
+                blockedEnvironmentVariables: [],
+            },
         }),
         storage: {
             getProjectTempDir: () => '/tmp',
@@ -158,12 +165,12 @@ function createMockConfig(overrides = {}) {
         getTruncateToolOutputThreshold: () => DEFAULT_TRUNCATE_TOOL_OUTPUT_THRESHOLD,
         getTruncateToolOutputLines: () => DEFAULT_TRUNCATE_TOOL_OUTPUT_LINES,
         getToolRegistry: () => defaultToolRegistry,
-        getUseSmartEdit: () => false,
-        getUseModelRouter: () => false,
+        getActiveModel: () => DEFAULT_GEMINI_MODEL,
         getGeminiClient: () => null,
-        getEnableMessageBusIntegration: () => false,
-        getMessageBus: () => null,
+        getMessageBus: () => createMockMessageBus(),
+        getEnableHooks: () => false,
         getPolicyEngine: () => null,
+        getExperiments: () => { },
     };
     return { ...baseConfig, ...overrides };
 }
@@ -254,7 +261,7 @@ describe('CoreToolScheduler', () => {
         const onToolCallsUpdate = vi.fn();
         const mockConfig = createMockConfig({
             getToolRegistry: () => mockToolRegistry,
-            isInteractive: () => false,
+            getHookSystem: () => undefined,
         });
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
@@ -343,7 +350,7 @@ describe('CoreToolScheduler', () => {
         const onToolCallsUpdate = vi.fn();
         const mockConfig = createMockConfig({
             getToolRegistry: () => mockToolRegistry,
-            isInteractive: () => false,
+            getHookSystem: () => undefined,
         });
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
@@ -395,7 +402,7 @@ describe('CoreToolScheduler', () => {
     it('should mark tool call as cancelled when abort happens during confirmation error', async () => {
         const abortController = new AbortController();
         const abortError = new Error('Abort requested during confirmation');
-        const declarativeTool = new AbortDuringConfirmationTool(abortController, abortError);
+        const declarativeTool = new AbortDuringConfirmationTool(abortController, abortError, createMockMessageBus());
         const mockToolRegistry = {
             getTool: () => declarativeTool,
             getFunctionDeclarations: () => [],
@@ -436,34 +443,56 @@ describe('CoreToolScheduler', () => {
         const statuses = onToolCallsUpdate.mock.calls.flatMap((call) => call[0].map((toolCall) => toolCall.status));
         expect(statuses).not.toContain('error');
     });
-    describe('getToolSuggestion', () => {
-        it('should suggest the top N closest tool names for a typo', () => {
-            // Create mocked tool registry
-            const mockToolRegistry = {
-                getAllToolNames: () => ['list_files', 'read_file', 'write_file'],
-            };
-            const mockConfig = createMockConfig({
-                getToolRegistry: () => mockToolRegistry,
-                isInteractive: () => false,
-            });
-            // Create scheduler
-            const scheduler = new CoreToolScheduler({
-                config: mockConfig,
-                getPreferredEditor: () => 'vscode',
-            });
-            // Test that the right tool is selected, with only 1 result, for typos
-            // @ts-expect-error accessing private method
-            const misspelledTool = scheduler.getToolSuggestion('list_fils', 1);
-            expect(misspelledTool).toBe(' Did you mean "list_files"?');
-            // Test that the right tool is selected, with only 1 result, for prefixes
-            // @ts-expect-error accessing private method
-            const prefixedTool = scheduler.getToolSuggestion('github.list_files', 1);
-            expect(prefixedTool).toBe(' Did you mean "list_files"?');
-            // Test that the right tool is first
-            // @ts-expect-error accessing private method
-            const suggestionMultiple = scheduler.getToolSuggestion('list_fils');
-            expect(suggestionMultiple).toBe(' Did you mean one of: "list_files", "read_file", "write_file"?');
+    it('should error when tool requires confirmation in non-interactive mode', async () => {
+        const mockTool = new MockTool({
+            name: 'mockTool',
+            shouldConfirmExecute: MOCK_TOOL_SHOULD_CONFIRM_EXECUTE,
+        });
+        const declarativeTool = mockTool;
+        const mockToolRegistry = {
+            getTool: () => declarativeTool,
+            getFunctionDeclarations: () => [],
+            tools: new Map(),
+            discovery: {},
+            registerTool: () => { },
+            getToolByName: () => declarativeTool,
+            getToolByDisplayName: () => declarativeTool,
+            getTools: () => [],
+            discoverTools: async () => { },
+            getAllTools: () => [],
+            getToolsByServer: () => [],
+        };
+        const onAllToolCallsComplete = vi.fn();
+        const onToolCallsUpdate = vi.fn();
+        const mockConfig = createMockConfig({
+            getToolRegistry: () => mockToolRegistry,
+            isInteractive: () => false,
+        });
+        const scheduler = new CoreToolScheduler({
+            config: mockConfig,
+            onAllToolCallsComplete,
+            onToolCallsUpdate,
+            getPreferredEditor: () => 'vscode',
         });
+        const abortController = new AbortController();
+        const request = {
+            callId: '1',
+            name: 'mockTool',
+            args: {},
+            isClientInitiated: false,
+            prompt_id: 'prompt-id-1',
+        };
+        await scheduler.schedule([request], abortController.signal);
+        expect(onAllToolCallsComplete).toHaveBeenCalled();
+        const completedCalls = onAllToolCallsComplete.mock
+            .calls[0][0];
+        expect(completedCalls[0].status).toBe('error');
+        const erroredCall = completedCalls[0];
+        const errorResponse = erroredCall.response;
+        const errorParts = errorResponse.responseParts;
+        // @ts-expect-error - accessing internal structure of FunctionResponsePart
+        const errorMessage = errorParts[0].functionResponse.response.error;
+        expect(errorMessage).toContain('Tool execution for "mockTool" requires user confirmation, which is not supported in non-interactive mode.');
     });
 });
 describe('CoreToolScheduler with payload', () => {
@@ -488,8 +517,13 @@ describe('CoreToolScheduler with payload', () => {
         const onToolCallsUpdate = vi.fn();
         const mockConfig = createMockConfig({
             getToolRegistry: () => mockToolRegistry,
-            isInteractive: () => false,
         });
+        const mockMessageBus = createMockMessageBus();
+        mockConfig.getMessageBus = vi.fn().mockReturnValue(mockMessageBus);
+        mockConfig.getEnableHooks = vi.fn().mockReturnValue(false);
+        mockConfig.getHookSystem = vi
+            .fn()
+            .mockReturnValue(new HookSystem(mockConfig));
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
             onAllToolCallsComplete,
@@ -523,176 +557,9 @@ describe('CoreToolScheduler with payload', () => {
         });
     });
 });
-describe('convertToFunctionResponse', () => {
-    const toolName = 'testTool';
-    const callId = 'call1';
-    it('should handle simple string llmContent', () => {
-        const llmContent = 'Simple text output';
-        const result = convertToFunctionResponse(toolName, callId, llmContent);
-        expect(result).toEqual([
-            {
-                functionResponse: {
-                    name: toolName,
-                    id: callId,
-                    response: { output: 'Simple text output' },
-                },
-            },
-        ]);
-    });
-    it('should handle llmContent as a single Part with text', () => {
-        const llmContent = { text: 'Text from Part object' };
-        const result = convertToFunctionResponse(toolName, callId, llmContent);
-        expect(result).toEqual([
-            {
-                functionResponse: {
-                    name: toolName,
-                    id: callId,
-                    response: { output: 'Text from Part object' },
-                },
-            },
-        ]);
-    });
-    it('should handle llmContent as a PartListUnion array with a single text Part', () => {
-        const llmContent = [{ text: 'Text from array' }];
-        const result = convertToFunctionResponse(toolName, callId, llmContent);
-        expect(result).toEqual([
-            {
-                functionResponse: {
-                    name: toolName,
-                    id: callId,
-                    response: { output: 'Text from array' },
-                },
-            },
-        ]);
-    });
-    it('should handle llmContent with inlineData', () => {
-        const llmContent = {
-            inlineData: { mimeType: 'image/png', data: 'base64...' },
-        };
-        const result = convertToFunctionResponse(toolName, callId, llmContent);
-        expect(result).toEqual([
-            {
-                functionResponse: {
-                    name: toolName,
-                    id: callId,
-                    response: {
-                        output: 'Binary content of type image/png was processed.',
-                    },
-                },
-            },
-            llmContent,
-        ]);
-    });
-    it('should handle llmContent with fileData', () => {
-        const llmContent = {
-            fileData: { mimeType: 'application/pdf', fileUri: 'gs://...' },
-        };
-        const result = convertToFunctionResponse(toolName, callId, llmContent);
-        expect(result).toEqual([
-            {
-                functionResponse: {
-                    name: toolName,
-                    id: callId,
-                    response: {
-                        output: 'Binary content of type application/pdf was processed.',
-                    },
-                },
-            },
-            llmContent,
-        ]);
-    });
-    it('should handle llmContent as an array of multiple Parts (text and inlineData)', () => {
-        const llmContent = [
-            { text: 'Some textual description' },
-            { inlineData: { mimeType: 'image/jpeg', data: 'base64data...' } },
-            { text: 'Another text part' },
-        ];
-        const result = convertToFunctionResponse(toolName, callId, llmContent);
-        expect(result).toEqual([
-            {
-                functionResponse: {
-                    name: toolName,
-                    id: callId,
-                    response: { output: 'Tool execution succeeded.' },
-                },
-            },
-            ...llmContent,
-        ]);
-    });
-    it('should handle llmContent as an array with a single inlineData Part', () => {
-        const llmContent = [
-            { inlineData: { mimeType: 'image/gif', data: 'gifdata...' } },
-        ];
-        const result = convertToFunctionResponse(toolName, callId, llmContent);
-        expect(result).toEqual([
-            {
-                functionResponse: {
-                    name: toolName,
-                    id: callId,
-                    response: {
-                        output: 'Binary content of type image/gif was processed.',
-                    },
-                },
-            },
-            ...llmContent,
-        ]);
-    });
-    it('should handle llmContent as a generic Part (not text, inlineData, or fileData)', () => {
-        const llmContent = { functionCall: { name: 'test', args: {} } };
-        const result = convertToFunctionResponse(toolName, callId, llmContent);
-        expect(result).toEqual([
-            {
-                functionResponse: {
-                    name: toolName,
-                    id: callId,
-                    response: { output: 'Tool execution succeeded.' },
-                },
-            },
-        ]);
-    });
-    it('should handle empty string llmContent', () => {
-        const llmContent = '';
-        const result = convertToFunctionResponse(toolName, callId, llmContent);
-        expect(result).toEqual([
-            {
-                functionResponse: {
-                    name: toolName,
-                    id: callId,
-                    response: { output: '' },
-                },
-            },
-        ]);
-    });
-    it('should handle llmContent as an empty array', () => {
-        const llmContent = [];
-        const result = convertToFunctionResponse(toolName, callId, llmContent);
-        expect(result).toEqual([
-            {
-                functionResponse: {
-                    name: toolName,
-                    id: callId,
-                    response: { output: 'Tool execution succeeded.' },
-                },
-            },
-        ]);
-    });
-    it('should handle llmContent as a Part with undefined inlineData/fileData/text', () => {
-        const llmContent = {}; // An empty part object
-        const result = convertToFunctionResponse(toolName, callId, llmContent);
-        expect(result).toEqual([
-            {
-                functionResponse: {
-                    name: toolName,
-                    id: callId,
-                    response: { output: 'Tool execution succeeded.' },
-                },
-            },
-        ]);
-    });
-});
 class MockEditToolInvocation extends BaseToolInvocation {
-    constructor(params) {
-        super(params);
+    constructor(params, messageBus) {
+        super(params, messageBus);
     }
     getDescription() {
         return 'A mock edit tool invocation';
@@ -717,16 +584,16 @@ class MockEditToolInvocation extends BaseToolInvocation {
     }
 }
 class MockEditTool extends BaseDeclarativeTool {
-    constructor() {
-        super('mockEditTool', 'mockEditTool', 'A mock edit tool', Kind.Edit, {});
+    constructor(messageBus) {
+        super('mockEditTool', 'mockEditTool', 'A mock edit tool', Kind.Edit, {}, messageBus);
     }
-    createInvocation(params) {
-        return new MockEditToolInvocation(params);
+    createInvocation(params, messageBus, _toolName, _toolDisplayName) {
+        return new MockEditToolInvocation(params, messageBus);
     }
 }
 describe('CoreToolScheduler edit cancellation', () => {
     it('should preserve diff when an edit is cancelled', async () => {
-        const mockEditTool = new MockEditTool();
+        const mockEditTool = new MockEditTool(createMockMessageBus());
         const mockToolRegistry = {
             getTool: () => mockEditTool,
             getFunctionDeclarations: () => [],
@@ -744,8 +611,13 @@ describe('CoreToolScheduler edit cancellation', () => {
         const onToolCallsUpdate = vi.fn();
         const mockConfig = createMockConfig({
             getToolRegistry: () => mockToolRegistry,
-            isInteractive: () => false,
         });
+        const mockMessageBus = createMockMessageBus();
+        mockConfig.getMessageBus = vi.fn().mockReturnValue(mockMessageBus);
+        mockConfig.getEnableHooks = vi.fn().mockReturnValue(false);
+        mockConfig.getHookSystem = vi
+            .fn()
+            .mockReturnValue(new HookSystem(mockConfig));
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
             onAllToolCallsComplete,
@@ -814,6 +686,12 @@ describe('CoreToolScheduler YOLO mode', () => {
             getApprovalMode: () => ApprovalMode.YOLO,
             isInteractive: () => false,
         });
+        const mockMessageBus = createMockMessageBus();
+        mockConfig.getMessageBus = vi.fn().mockReturnValue(mockMessageBus);
+        mockConfig.getEnableHooks = vi.fn().mockReturnValue(false);
+        mockConfig.getHookSystem = vi
+            .fn()
+            .mockReturnValue(new HookSystem(mockConfig));
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
             onAllToolCallsComplete,
@@ -888,6 +766,12 @@ describe('CoreToolScheduler request queueing', () => {
             getApprovalMode: () => ApprovalMode.YOLO, // Use YOLO to avoid confirmation prompts
             isInteractive: () => false,
         });
+        const mockMessageBus = createMockMessageBus();
+        mockConfig.getMessageBus = vi.fn().mockReturnValue(mockMessageBus);
+        mockConfig.getEnableHooks = vi.fn().mockReturnValue(false);
+        mockConfig.getHookSystem = vi
+            .fn()
+            .mockReturnValue(new HookSystem(mockConfig));
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
             onAllToolCallsComplete,
@@ -910,6 +794,7 @@ describe('CoreToolScheduler request queueing', () => {
             prompt_id: 'prompt-2',
         };
         // Schedule the first call, which will pause execution.
+        // eslint-disable-next-line @typescript-eslint/no-floating-promises
         scheduler.schedule([request1], abortController.signal);
         // Wait for the first call to be in the 'executing' state.
         await waitForStatus(onToolCallsUpdate, 'executing');
@@ -979,9 +864,20 @@ describe('CoreToolScheduler request queueing', () => {
             getShellExecutionConfig: () => ({
                 terminalWidth: 80,
                 terminalHeight: 24,
+                sanitizationConfig: {
+                    enableEnvironmentVariableRedaction: true,
+                    allowedEnvironmentVariables: [],
+                    blockedEnvironmentVariables: [],
+                },
             }),
             isInteractive: () => false,
         });
+        const mockMessageBus = createMockMessageBus();
+        mockConfig.getMessageBus = vi.fn().mockReturnValue(mockMessageBus);
+        mockConfig.getEnableHooks = vi.fn().mockReturnValue(false);
+        mockConfig.getHookSystem = vi
+            .fn()
+            .mockReturnValue(new HookSystem(mockConfig));
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
             onAllToolCallsComplete,
@@ -1066,9 +962,14 @@ describe('CoreToolScheduler request queueing', () => {
             getShellExecutionConfig: () => ({
                 terminalWidth: 80,
                 terminalHeight: 24,
+                sanitizationConfig: {
+                    enableEnvironmentVariableRedaction: true,
+                    allowedEnvironmentVariables: [],
+                    blockedEnvironmentVariables: [],
+                },
             }),
             getToolRegistry: () => toolRegistry,
-            isInteractive: () => false,
+            getHookSystem: () => undefined,
         });
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
@@ -1117,8 +1018,13 @@ describe('CoreToolScheduler request queueing', () => {
         const mockConfig = createMockConfig({
             getToolRegistry: () => mockToolRegistry,
             getApprovalMode: () => ApprovalMode.YOLO,
-            isInteractive: () => false,
         });
+        const mockMessageBus = createMockMessageBus();
+        mockConfig.getMessageBus = vi.fn().mockReturnValue(mockMessageBus);
+        mockConfig.getEnableHooks = vi.fn().mockReturnValue(false);
+        mockConfig.getHookSystem = vi
+            .fn()
+            .mockReturnValue(new HookSystem(mockConfig));
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
             onAllToolCallsComplete,
@@ -1159,9 +1065,14 @@ describe('CoreToolScheduler request queueing', () => {
             setApprovalMode: (mode) => {
                 approvalMode = mode;
             },
-            isInteractive: () => false,
         });
-        const testTool = new TestApprovalTool(mockConfig);
+        const mockMessageBus = createMockMessageBus();
+        mockConfig.getMessageBus = vi.fn().mockReturnValue(mockMessageBus);
+        mockConfig.getEnableHooks = vi.fn().mockReturnValue(false);
+        mockConfig.getHookSystem = vi
+            .fn()
+            .mockReturnValue(new HookSystem(mockConfig));
+        const testTool = new TestApprovalTool(mockConfig, mockMessageBus);
         const toolRegistry = {
             getTool: () => testTool,
             getFunctionDeclarations: () => [],
@@ -1302,6 +1213,12 @@ describe('CoreToolScheduler Sequential Execution', () => {
             getApprovalMode: () => ApprovalMode.YOLO, // Use YOLO to avoid confirmation prompts
             isInteractive: () => false,
         });
+        const mockMessageBus = createMockMessageBus();
+        mockConfig.getMessageBus = vi.fn().mockReturnValue(mockMessageBus);
+        mockConfig.getEnableHooks = vi.fn().mockReturnValue(false);
+        mockConfig.getHookSystem = vi
+            .fn()
+            .mockReturnValue(new HookSystem(mockConfig));
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
             onAllToolCallsComplete,
@@ -1388,6 +1305,12 @@ describe('CoreToolScheduler Sequential Execution', () => {
             getApprovalMode: () => ApprovalMode.YOLO,
             isInteractive: () => false,
         });
+        const mockMessageBus = createMockMessageBus();
+        mockConfig.getMessageBus = vi.fn().mockReturnValue(mockMessageBus);
+        mockConfig.getEnableHooks = vi.fn().mockReturnValue(false);
+        mockConfig.getHookSystem = vi
+            .fn()
+            .mockReturnValue(new HookSystem(mockConfig));
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
             onAllToolCallsComplete,
@@ -1469,6 +1392,12 @@ describe('CoreToolScheduler Sequential Execution', () => {
         const mockConfig = createMockConfig({
             getToolRegistry: () => mockToolRegistry,
         });
+        const mockMessageBus = createMockMessageBus();
+        mockConfig.getMessageBus = vi.fn().mockReturnValue(mockMessageBus);
+        mockConfig.getEnableHooks = vi.fn().mockReturnValue(false);
+        mockConfig.getHookSystem = vi
+            .fn()
+            .mockReturnValue(new HookSystem(mockConfig));
         const scheduler = new CoreToolScheduler({
             config: mockConfig,
             onAllToolCallsComplete,
@@ -1499,118 +1428,4 @@ describe('CoreToolScheduler Sequential Execution', () => {
         modifyWithEditorSpy.mockRestore();
     });
 });
-describe('truncateAndSaveToFile', () => {
-    const mockWriteFile = vi.mocked(fs.writeFile);
-    const THRESHOLD = 40_000;
-    const TRUNCATE_LINES = 1000;
-    beforeEach(() => {
-        vi.clearAllMocks();
-    });
-    it('should return content unchanged if below threshold', async () => {
-        const content = 'Short content';
-        const callId = 'test-call-id';
-        const projectTempDir = '/tmp';
-        const result = await truncateAndSaveToFile(content, callId, projectTempDir, THRESHOLD, TRUNCATE_LINES);
-        expect(result).toEqual({ content });
-        expect(mockWriteFile).not.toHaveBeenCalled();
-    });
-    it('should truncate content by lines when content has many lines', async () => {
-        // Create content that exceeds 100,000 character threshold with many lines
-        const lines = Array(2000).fill('x'.repeat(100)); // 100 chars per line * 2000 lines = 200,000 chars
-        const content = lines.join('\n');
-        const callId = 'test-call-id';
-        const projectTempDir = '/tmp';
-        mockWriteFile.mockResolvedValue(undefined);
-        const result = await truncateAndSaveToFile(content, callId, projectTempDir, THRESHOLD, TRUNCATE_LINES);
-        expect(result.outputFile).toBe(path.join(projectTempDir, `${callId}.output`));
-        expect(mockWriteFile).toHaveBeenCalledWith(path.join(projectTempDir, `${callId}.output`), content);
-        // Should contain the first and last lines with 1/5 head and 4/5 tail
-        const head = Math.floor(TRUNCATE_LINES / 5);
-        const beginning = lines.slice(0, head);
-        const end = lines.slice(-(TRUNCATE_LINES - head));
-        const expectedTruncated = beginning.join('\n') + '\n... [CONTENT TRUNCATED] ...\n' + end.join('\n');
-        expect(result.content).toContain('Tool output was too large and has been truncated');
-        expect(result.content).toContain('Truncated part of the output:');
-        expect(result.content).toContain(expectedTruncated);
-    });
-    it('should wrap and truncate content when content has few but long lines', async () => {
-        const content = 'a'.repeat(200_000); // A single very long line
-        const callId = 'test-call-id';
-        const projectTempDir = '/tmp';
-        const wrapWidth = 120;
-        mockWriteFile.mockResolvedValue(undefined);
-        // Manually wrap the content to generate the expected file content
-        const wrappedLines = [];
-        for (let i = 0; i < content.length; i += wrapWidth) {
-            wrappedLines.push(content.substring(i, i + wrapWidth));
-        }
-        const expectedFileContent = wrappedLines.join('\n');
-        const result = await truncateAndSaveToFile(content, callId, projectTempDir, THRESHOLD, TRUNCATE_LINES);
-        expect(result.outputFile).toBe(path.join(projectTempDir, `${callId}.output`));
-        // Check that the file was written with the wrapped content
-        expect(mockWriteFile).toHaveBeenCalledWith(path.join(projectTempDir, `${callId}.output`), expectedFileContent);
-        // Should contain the first and last lines with 1/5 head and 4/5 tail of the wrapped content
-        const head = Math.floor(TRUNCATE_LINES / 5);
-        const beginning = wrappedLines.slice(0, head);
-        const end = wrappedLines.slice(-(TRUNCATE_LINES - head));
-        const expectedTruncated = beginning.join('\n') + '\n... [CONTENT TRUNCATED] ...\n' + end.join('\n');
-        expect(result.content).toContain('Tool output was too large and has been truncated');
-        expect(result.content).toContain('Truncated part of the output:');
-        expect(result.content).toContain(expectedTruncated);
-    });
-    it('should handle file write errors gracefully', async () => {
-        const content = 'a'.repeat(2_000_000);
-        const callId = 'test-call-id';
-        const projectTempDir = '/tmp';
-        mockWriteFile.mockRejectedValue(new Error('File write failed'));
-        const result = await truncateAndSaveToFile(content, callId, projectTempDir, THRESHOLD, TRUNCATE_LINES);
-        expect(result.outputFile).toBeUndefined();
-        expect(result.content).toContain('[Note: Could not save full output to file]');
-        expect(mockWriteFile).toHaveBeenCalled();
-    });
-    it('should save to correct file path with call ID', async () => {
-        const content = 'a'.repeat(200_000);
-        const callId = 'unique-call-123';
-        const projectTempDir = '/custom/temp/dir';
-        const wrapWidth = 120;
-        mockWriteFile.mockResolvedValue(undefined);
-        // Manually wrap the content to generate the expected file content
-        const wrappedLines = [];
-        for (let i = 0; i < content.length; i += wrapWidth) {
-            wrappedLines.push(content.substring(i, i + wrapWidth));
-        }
-        const expectedFileContent = wrappedLines.join('\n');
-        const result = await truncateAndSaveToFile(content, callId, projectTempDir, THRESHOLD, TRUNCATE_LINES);
-        const expectedPath = path.join(projectTempDir, `${callId}.output`);
-        expect(result.outputFile).toBe(expectedPath);
-        expect(mockWriteFile).toHaveBeenCalledWith(expectedPath, expectedFileContent);
-    });
-    it('should include helpful instructions in truncated message', async () => {
-        const content = 'a'.repeat(2_000_000);
-        const callId = 'test-call-id';
-        const projectTempDir = '/tmp';
-        mockWriteFile.mockResolvedValue(undefined);
-        const result = await truncateAndSaveToFile(content, callId, projectTempDir, THRESHOLD, TRUNCATE_LINES);
-        expect(result.content).toContain('read_file tool with the absolute file path above');
-        expect(result.content).toContain('read_file tool with offset=0, limit=100');
-        expect(result.content).toContain('read_file tool with offset=N to skip N lines');
-        expect(result.content).toContain('read_file tool with limit=M to read only M lines');
-    });
-    it('should sanitize callId to prevent path traversal', async () => {
-        const content = 'a'.repeat(200_000);
-        const callId = '../../../../../etc/passwd';
-        const projectTempDir = '/tmp/safe_dir';
-        const wrapWidth = 120;
-        mockWriteFile.mockResolvedValue(undefined);
-        // Manually wrap the content to generate the expected file content
-        const wrappedLines = [];
-        for (let i = 0; i < content.length; i += wrapWidth) {
-            wrappedLines.push(content.substring(i, i + wrapWidth));
-        }
-        const expectedFileContent = wrappedLines.join('\n');
-        await truncateAndSaveToFile(content, callId, projectTempDir, THRESHOLD, TRUNCATE_LINES);
-        const expectedPath = path.join(projectTempDir, 'passwd.output');
-        expect(mockWriteFile).toHaveBeenCalledWith(expectedPath, expectedFileContent);
-    });
-});
 //# sourceMappingURL=coreToolScheduler.test.js.map