@didim365/agent-cli-core 0.1.1 → 0.1.3

package/dist/src/services/loopDetectionService.test.js

@@ -1,881 +0,0 @@
-/**
- * @license
- * Copyright 2025 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-import { LlmEventType } from '../providers/events.js';
-import * as loggers from '../telemetry/loggers.js';
-import { LoopType } from '../telemetry/types.js';
-import { LoopDetectionService } from './loopDetectionService.js';
-import { createAvailabilityServiceMock } from '../availability/testUtils.js';
-vi.mock('../telemetry/loggers.js', () => ({
-    logLoopDetected: vi.fn(),
-    logLoopDetectionDisabled: vi.fn(),
-    logLlmLoopCheck: vi.fn(),
-}));
-const TOOL_CALL_LOOP_THRESHOLD = 5;
-const CONTENT_LOOP_THRESHOLD = 10;
-const CONTENT_CHUNK_SIZE = 50;
-describe('LoopDetectionService', () => {
-    let service;
-    let mockConfig;
-    beforeEach(() => {
-        mockConfig = {
-            getTelemetryEnabled: () => true,
-            isInteractive: () => false,
-            getModelAvailabilityService: vi
-                .fn()
-                .mockReturnValue(createAvailabilityServiceMock()),
-        };
-        service = new LoopDetectionService(mockConfig);
-        vi.clearAllMocks();
-    });
-    const createToolCallRequestEvent = (name, args) => ({
-        type: LlmEventType.ToolCallRequest,
-        name,
-        args,
-        callId: 'test-id',
-        isClientInitiated: false,
-        promptId: 'test-prompt-id',
-    });
-    const createContentEvent = (content) => ({
-        type: LlmEventType.TextDelta,
-        text: content,
-    });
-    const createRepetitiveContent = (id, length) => {
-        const baseString = `This is a unique sentence, id=${id}. `;
-        let content = '';
-        while (content.length < length) {
-            content += baseString;
-        }
-        return content.slice(0, length);
-    };
-    describe('Tool Call Loop Detection', () => {
-        it(`should not detect a loop for fewer than TOOL_CALL_LOOP_THRESHOLD identical calls`, () => {
-            const event = createToolCallRequestEvent('testTool', { param: 'value' });
-            for (let i = 0; i < TOOL_CALL_LOOP_THRESHOLD - 1; i++) {
-                expect(service.addAndCheck(event)).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it(`should detect a loop on the TOOL_CALL_LOOP_THRESHOLD-th identical call`, () => {
-            const event = createToolCallRequestEvent('testTool', { param: 'value' });
-            for (let i = 0; i < TOOL_CALL_LOOP_THRESHOLD - 1; i++) {
-                service.addAndCheck(event);
-            }
-            expect(service.addAndCheck(event)).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should detect a loop on subsequent identical calls', () => {
-            const event = createToolCallRequestEvent('testTool', { param: 'value' });
-            for (let i = 0; i < TOOL_CALL_LOOP_THRESHOLD; i++) {
-                service.addAndCheck(event);
-            }
-            expect(service.addAndCheck(event)).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should not detect a loop for different tool calls', () => {
-            const event1 = createToolCallRequestEvent('testTool', {
-                param: 'value1',
-            });
-            const event2 = createToolCallRequestEvent('testTool', {
-                param: 'value2',
-            });
-            const event3 = createToolCallRequestEvent('anotherTool', {
-                param: 'value1',
-            });
-            for (let i = 0; i < TOOL_CALL_LOOP_THRESHOLD - 2; i++) {
-                expect(service.addAndCheck(event1)).toBe(false);
-                expect(service.addAndCheck(event2)).toBe(false);
-                expect(service.addAndCheck(event3)).toBe(false);
-            }
-        });
-        it('should not reset tool call counter for other event types', () => {
-            const toolCallEvent = createToolCallRequestEvent('testTool', {
-                param: 'value',
-            });
-            const otherEvent = {
-                type: 'thought',
-            };
-            // Send events just below the threshold
-            for (let i = 0; i < TOOL_CALL_LOOP_THRESHOLD - 1; i++) {
-                expect(service.addAndCheck(toolCallEvent)).toBe(false);
-            }
-            // Send a different event type
-            expect(service.addAndCheck(otherEvent)).toBe(false);
-            // Send the tool call event again, which should now trigger the loop
-            expect(service.addAndCheck(toolCallEvent)).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should not detect a loop when disabled for session', () => {
-            service.disableForSession();
-            expect(loggers.logLoopDetectionDisabled).toHaveBeenCalledTimes(1);
-            const event = createToolCallRequestEvent('testTool', { param: 'value' });
-            for (let i = 0; i < TOOL_CALL_LOOP_THRESHOLD; i++) {
-                expect(service.addAndCheck(event)).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should stop reporting a loop if disabled after detection', () => {
-            const event = createToolCallRequestEvent('testTool', { param: 'value' });
-            for (let i = 0; i < TOOL_CALL_LOOP_THRESHOLD; i++) {
-                service.addAndCheck(event);
-            }
-            expect(service.addAndCheck(event)).toBe(true);
-            service.disableForSession();
-            // Should now return false even though a loop was previously detected
-            expect(service.addAndCheck(event)).toBe(false);
-        });
-    });
-    describe('Content Loop Detection', () => {
-        const generateRandomString = (length) => {
-            let result = '';
-            const characters = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789';
-            const charactersLength = characters.length;
-            for (let i = 0; i < length; i++) {
-                result += characters.charAt(Math.floor(Math.random() * charactersLength));
-            }
-            return result;
-        };
-        it('should not detect a loop for random content', () => {
-            service.reset('');
-            for (let i = 0; i < 1000; i++) {
-                const content = generateRandomString(10);
-                const isLoop = service.addAndCheck(createContentEvent(content));
-                expect(isLoop).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should detect a loop when a chunk of content repeats consecutively', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            let isLoop = false;
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD; i++) {
-                isLoop = service.addAndCheck(createContentEvent(repeatedContent));
-            }
-            expect(isLoop).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should not detect a loop if repetitions are very far apart', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            const fillerContent = generateRandomString(500);
-            let isLoop = false;
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD; i++) {
-                isLoop = service.addAndCheck(createContentEvent(repeatedContent));
-                isLoop = service.addAndCheck(createContentEvent(fillerContent));
-            }
-            expect(isLoop).toBe(false);
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should detect a loop with longer repeating patterns (e.g. ~150 chars)', () => {
-            service.reset('');
-            const longPattern = createRepetitiveContent(1, 150);
-            expect(longPattern.length).toBe(150);
-            let isLoop = false;
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD + 2; i++) {
-                isLoop = service.addAndCheck(createContentEvent(longPattern));
-                if (isLoop)
-                    break;
-            }
-            expect(isLoop).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should detect the specific user-provided loop example', () => {
-            service.reset('');
-            const userPattern = `I will not output any text.
-  I will just end the turn.
-  I am done.
-  I will not do anything else.
-  I will wait for the user's next command.
-`;
-            let isLoop = false;
-            // Loop enough times to trigger the threshold
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD + 5; i++) {
-                isLoop = service.addAndCheck(createContentEvent(userPattern));
-                if (isLoop)
-                    break;
-            }
-            expect(isLoop).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should detect the second specific user-provided loop example', () => {
-            service.reset('');
-            const userPattern = 'I have added all the requested logs and verified the test file. I will now mark the task as complete.\n  ';
-            let isLoop = false;
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD + 5; i++) {
-                isLoop = service.addAndCheck(createContentEvent(userPattern));
-                if (isLoop)
-                    break;
-            }
-            expect(isLoop).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should detect a loop of alternating short phrases', () => {
-            service.reset('');
-            const alternatingPattern = 'Thinking... Done. ';
-            let isLoop = false;
-            // Needs more iterations because the pattern is short relative to chunk size,
-            // so it takes a few slides of the window to find the exact alignment.
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD * 3; i++) {
-                isLoop = service.addAndCheck(createContentEvent(alternatingPattern));
-                if (isLoop)
-                    break;
-            }
-            expect(isLoop).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should detect a loop of repeated complex thought processes', () => {
-            service.reset('');
-            const thoughtPattern = 'I need to check the file. The file does not exist. I will create the file. ';
-            let isLoop = false;
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD + 5; i++) {
-                isLoop = service.addAndCheck(createContentEvent(thoughtPattern));
-                if (isLoop)
-                    break;
-            }
-            expect(isLoop).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-    });
-    describe('Content Loop Detection with Code Blocks', () => {
-        it('should not detect a loop when repetitive content is inside a code block', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            service.addAndCheck(createContentEvent('```\n'));
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD; i++) {
-                const isLoop = service.addAndCheck(createContentEvent(repeatedContent));
-                expect(isLoop).toBe(false);
-            }
-            const isLoop = service.addAndCheck(createContentEvent('\n```'));
-            expect(isLoop).toBe(false);
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should not detect loops when content transitions into a code block', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            // Add some repetitive content outside of code block
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 2; i++) {
-                service.addAndCheck(createContentEvent(repeatedContent));
-            }
-            // Now transition into a code block - this should prevent loop detection
-            // even though we were already close to the threshold
-            const codeBlockStart = '```javascript\n';
-            const isLoop = service.addAndCheck(createContentEvent(codeBlockStart));
-            expect(isLoop).toBe(false);
-            // Continue adding repetitive content inside the code block - should not trigger loop
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD; i++) {
-                const isLoopInside = service.addAndCheck(createContentEvent(repeatedContent));
-                expect(isLoopInside).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should skip loop detection when already inside a code block (this.inCodeBlock)', () => {
-            service.reset('');
-            // Start with content that puts us inside a code block
-            service.addAndCheck(createContentEvent('Here is some code:\n```\n'));
-            // Verify we are now inside a code block and any content should be ignored for loop detection
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD + 5; i++) {
-                const isLoop = service.addAndCheck(createContentEvent(repeatedContent));
-                expect(isLoop).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should correctly track inCodeBlock state with multiple fence transitions', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            // Outside code block - should track content
-            service.addAndCheck(createContentEvent('Normal text '));
-            // Enter code block (1 fence) - should stop tracking
-            const enterResult = service.addAndCheck(createContentEvent('```\n'));
-            expect(enterResult).toBe(false);
-            // Inside code block - should not track loops
-            for (let i = 0; i < 5; i++) {
-                const insideResult = service.addAndCheck(createContentEvent(repeatedContent));
-                expect(insideResult).toBe(false);
-            }
-            // Exit code block (2nd fence) - should reset tracking but still return false
-            const exitResult = service.addAndCheck(createContentEvent('```\n'));
-            expect(exitResult).toBe(false);
-            // Enter code block again (3rd fence) - should stop tracking again
-            const reenterResult = service.addAndCheck(createContentEvent('```python\n'));
-            expect(reenterResult).toBe(false);
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should detect a loop when repetitive content is outside a code block', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            service.addAndCheck(createContentEvent('```'));
-            service.addAndCheck(createContentEvent('\nsome code\n'));
-            service.addAndCheck(createContentEvent('```'));
-            let isLoop = false;
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD; i++) {
-                isLoop = service.addAndCheck(createContentEvent(repeatedContent));
-            }
-            expect(isLoop).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should handle content with multiple code blocks and no loops', () => {
-            service.reset('');
-            service.addAndCheck(createContentEvent('```\ncode1\n```'));
-            service.addAndCheck(createContentEvent('\nsome text\n'));
-            const isLoop = service.addAndCheck(createContentEvent('```\ncode2\n```'));
-            expect(isLoop).toBe(false);
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should handle content with mixed code blocks and looping text', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            service.addAndCheck(createContentEvent('```'));
-            service.addAndCheck(createContentEvent('\ncode1\n'));
-            service.addAndCheck(createContentEvent('```'));
-            let isLoop = false;
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD; i++) {
-                isLoop = service.addAndCheck(createContentEvent(repeatedContent));
-            }
-            expect(isLoop).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should not detect a loop for a long code block with some repeating tokens', () => {
-            service.reset('');
-            const repeatingTokens = 'for (let i = 0; i < 10; i++) { console.log(i); }';
-            service.addAndCheck(createContentEvent('```\n'));
-            for (let i = 0; i < 20; i++) {
-                const isLoop = service.addAndCheck(createContentEvent(repeatingTokens));
-                expect(isLoop).toBe(false);
-            }
-            const isLoop = service.addAndCheck(createContentEvent('\n```'));
-            expect(isLoop).toBe(false);
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should reset tracking when a code fence is found', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                service.addAndCheck(createContentEvent(repeatedContent));
-            }
-            // This should not trigger a loop because of the reset
-            service.addAndCheck(createContentEvent('```'));
-            // We are now in a code block, so loop detection should be off.
-            // Let's add the repeated content again, it should not trigger a loop.
-            let isLoop = false;
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD; i++) {
-                isLoop = service.addAndCheck(createContentEvent(repeatedContent));
-                expect(isLoop).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should reset tracking when a table is detected', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                service.addAndCheck(createContentEvent(repeatedContent));
-            }
-            // This should reset tracking and not trigger a loop
-            service.addAndCheck(createContentEvent('| Column 1 | Column 2 |'));
-            // Add more repeated content after table - should not trigger loop
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                const isLoop = service.addAndCheck(createContentEvent(repeatedContent));
-                expect(isLoop).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should reset tracking when a list item is detected', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                service.addAndCheck(createContentEvent(repeatedContent));
-            }
-            // This should reset tracking and not trigger a loop
-            service.addAndCheck(createContentEvent('* List item'));
-            // Add more repeated content after list - should not trigger loop
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                const isLoop = service.addAndCheck(createContentEvent(repeatedContent));
-                expect(isLoop).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should reset tracking when a heading is detected', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                service.addAndCheck(createContentEvent(repeatedContent));
-            }
-            // This should reset tracking and not trigger a loop
-            service.addAndCheck(createContentEvent('## Heading'));
-            // Add more repeated content after heading - should not trigger loop
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                const isLoop = service.addAndCheck(createContentEvent(repeatedContent));
-                expect(isLoop).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should reset tracking when a blockquote is detected', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                service.addAndCheck(createContentEvent(repeatedContent));
-            }
-            // This should reset tracking and not trigger a loop
-            service.addAndCheck(createContentEvent('> Quote text'));
-            // Add more repeated content after blockquote - should not trigger loop
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                const isLoop = service.addAndCheck(createContentEvent(repeatedContent));
-                expect(isLoop).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should reset tracking for various list item formats', () => {
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            // Test different list formats - make sure they start at beginning of line
-            const listFormats = [
-                '* Bullet item',
-                '- Dash item',
-                '+ Plus item',
-                '1. Numbered item',
-                '42. Another numbered item',
-            ];
-            listFormats.forEach((listFormat, index) => {
-                service.reset('');
-                // Build up to near threshold
-                for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                    service.addAndCheck(createContentEvent(repeatedContent));
-                }
-                // Reset should occur with list item - add newline to ensure it starts at beginning
-                service.addAndCheck(createContentEvent('\n' + listFormat));
-                // Should not trigger loop after reset - use different content to avoid any cached state issues
-                const newRepeatedContent = createRepetitiveContent(index + 100, CONTENT_CHUNK_SIZE);
-                for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                    const isLoop = service.addAndCheck(createContentEvent(newRepeatedContent));
-                    expect(isLoop).toBe(false);
-                }
-            });
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should reset tracking for various table formats', () => {
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            const tableFormats = [
-                '| Column 1 | Column 2 |',
-                '|---|---|',
-                '|++|++|',
-                '+---+---+',
-            ];
-            tableFormats.forEach((tableFormat, index) => {
-                service.reset('');
-                // Build up to near threshold
-                for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                    service.addAndCheck(createContentEvent(repeatedContent));
-                }
-                // Reset should occur with table format - add newline to ensure it starts at beginning
-                service.addAndCheck(createContentEvent('\n' + tableFormat));
-                // Should not trigger loop after reset - use different content to avoid any cached state issues
-                const newRepeatedContent = createRepetitiveContent(index + 200, CONTENT_CHUNK_SIZE);
-                for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                    const isLoop = service.addAndCheck(createContentEvent(newRepeatedContent));
-                    expect(isLoop).toBe(false);
-                }
-            });
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should reset tracking for various heading levels', () => {
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            const headingFormats = [
-                '# H1 Heading',
-                '## H2 Heading',
-                '### H3 Heading',
-                '#### H4 Heading',
-                '##### H5 Heading',
-                '###### H6 Heading',
-            ];
-            headingFormats.forEach((headingFormat, index) => {
-                service.reset('');
-                // Build up to near threshold
-                for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                    service.addAndCheck(createContentEvent(repeatedContent));
-                }
-                // Reset should occur with heading - add newline to ensure it starts at beginning
-                service.addAndCheck(createContentEvent('\n' + headingFormat));
-                // Should not trigger loop after reset - use different content to avoid any cached state issues
-                const newRepeatedContent = createRepetitiveContent(index + 300, CONTENT_CHUNK_SIZE);
-                for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 1; i++) {
-                    const isLoop = service.addAndCheck(createContentEvent(newRepeatedContent));
-                    expect(isLoop).toBe(false);
-                }
-            });
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-    });
-    describe('Edge Cases', () => {
-        it('should handle empty content', () => {
-            const event = createContentEvent('');
-            expect(service.addAndCheck(event)).toBe(false);
-        });
-    });
-    describe('Divider Content Detection', () => {
-        it('should not detect a loop for repeating divider-like content', () => {
-            service.reset('');
-            const dividerContent = '-'.repeat(CONTENT_CHUNK_SIZE);
-            let isLoop = false;
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD + 5; i++) {
-                isLoop = service.addAndCheck(createContentEvent(dividerContent));
-                expect(isLoop).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-        it('should not detect a loop for repeating complex box-drawing dividers', () => {
-            service.reset('');
-            const dividerContent = '╭─'.repeat(CONTENT_CHUNK_SIZE / 2);
-            let isLoop = false;
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD + 5; i++) {
-                isLoop = service.addAndCheck(createContentEvent(dividerContent));
-                expect(isLoop).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-    });
-    describe('Reset Functionality', () => {
-        it('tool call should reset content count', () => {
-            const contentEvent = createContentEvent('Some content.');
-            const toolEvent = createToolCallRequestEvent('testTool', {
-                param: 'value',
-            });
-            for (let i = 0; i < 9; i++) {
-                service.addAndCheck(contentEvent);
-            }
-            service.addAndCheck(toolEvent);
-            // Should start fresh
-            expect(service.addAndCheck(createContentEvent('Fresh content.'))).toBe(false);
-        });
-    });
-    describe('General Behavior', () => {
-        it('should return false for unhandled event types', () => {
-            const otherEvent = {
-                type: 'unhandled_event',
-            };
-            expect(service.addAndCheck(otherEvent)).toBe(false);
-            expect(service.addAndCheck(otherEvent)).toBe(false);
-        });
-    });
-    describe('addAndCheck - LlmEvent support (M2.2.2.3)', () => {
-        it('should detect tool call loop via LlmEvent ToolCallRequest', () => {
-            const llmToolCallEvent = {
-                type: LlmEventType.ToolCallRequest,
-                callId: 'call-1',
-                name: 'testTool',
-                args: { param: 'value' },
-            };
-            for (let i = 0; i < TOOL_CALL_LOOP_THRESHOLD - 1; i++) {
-                expect(service.addAndCheck(llmToolCallEvent)).toBe(false);
-            }
-            expect(service.addAndCheck(llmToolCallEvent)).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should detect content loop via LlmEvent TextDelta', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            const llmTextEvent = {
-                type: LlmEventType.TextDelta,
-                text: repeatedContent,
-            };
-            let isLoop = false;
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD; i++) {
-                isLoop = service.addAndCheck(llmTextEvent);
-            }
-            expect(isLoop).toBe(true);
-            expect(loggers.logLoopDetected).toHaveBeenCalledTimes(1);
-        });
-        it('should reset content tracking when ToolCallRequest arrives', () => {
-            service.reset('');
-            const repeatedContent = createRepetitiveContent(1, CONTENT_CHUNK_SIZE);
-            const llmTextEvent = {
-                type: LlmEventType.TextDelta,
-                text: repeatedContent,
-            };
-            const llmToolCallEvent = {
-                type: LlmEventType.ToolCallRequest,
-                callId: 'call-1',
-                name: 'someTool',
-                args: { key: 'val' },
-            };
-            // Build up near content loop threshold
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 2; i++) {
-                service.addAndCheck(llmTextEvent);
-            }
-            // Tool call should reset content tracking
-            service.addAndCheck(llmToolCallEvent);
-            // Content should start fresh — no loop detection
-            for (let i = 0; i < CONTENT_LOOP_THRESHOLD - 2; i++) {
-                expect(service.addAndCheck(llmTextEvent)).toBe(false);
-            }
-        });
-        it('should return false for non-monitored LlmEvent types', () => {
-            const finishedEvent = {
-                type: LlmEventType.Finished,
-                finishReason: 'end_turn',
-            };
-            const errorEvent = {
-                type: LlmEventType.Error,
-                error: 'some error',
-            };
-            const retryEvent = {
-                type: LlmEventType.Retry,
-            };
-            expect(service.addAndCheck(finishedEvent)).toBe(false);
-            expect(service.addAndCheck(errorEvent)).toBe(false);
-            expect(service.addAndCheck(retryEvent)).toBe(false);
-        });
-        it('should respect disabledForSession flag', () => {
-            service.disableForSession();
-            const llmToolCallEvent = {
-                type: LlmEventType.ToolCallRequest,
-                callId: 'call-1',
-                name: 'testTool',
-                args: { param: 'value' },
-            };
-            for (let i = 0; i < TOOL_CALL_LOOP_THRESHOLD + 5; i++) {
-                expect(service.addAndCheck(llmToolCallEvent)).toBe(false);
-            }
-            expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        });
-    });
-});
-describe('LoopDetectionService LLM Checks', () => {
-    let service;
-    let mockConfig;
-    let mockGeminiClient;
-    let mockBaseLlmClient;
-    let abortController;
-    beforeEach(() => {
-        mockGeminiClient = {
-            getHistory: vi.fn().mockReturnValue([]),
-        };
-        mockBaseLlmClient = {
-            generateJson: vi.fn(),
-        };
-        const mockAvailability = createAvailabilityServiceMock();
-        vi.mocked(mockAvailability.snapshot).mockReturnValue({ available: true });
-        mockConfig = {
-            getGeminiClient: () => mockGeminiClient,
-            getBaseLlmClient: () => mockBaseLlmClient,
-            getDebugMode: () => false,
-            getTelemetryEnabled: () => true,
-            getModel: vi.fn().mockReturnValue('cognitive-loop-v1'),
-            modelConfigService: {
-                getResolvedConfig: vi.fn().mockImplementation((key) => {
-                    if (key.model === 'loop-detection') {
-                        return { model: 'gemini-2.5-flash', generateContentConfig: {} };
-                    }
-                    return {
-                        model: 'cognitive-loop-v1',
-                        generateContentConfig: {},
-                    };
-                }),
-            },
-            isInteractive: () => false,
-            getModelAvailabilityService: vi.fn().mockReturnValue(mockAvailability),
-        };
-        service = new LoopDetectionService(mockConfig);
-        abortController = new AbortController();
-        vi.clearAllMocks();
-    });
-    afterEach(() => {
-        vi.restoreAllMocks();
-    });
-    const advanceTurns = async (count) => {
-        for (let i = 0; i < count; i++) {
-            await service.turnStarted(abortController.signal);
-        }
-    };
-    it('should not trigger LLM check before LLM_CHECK_AFTER_TURNS', async () => {
-        await advanceTurns(29);
-        expect(mockBaseLlmClient.generateJson).not.toHaveBeenCalled();
-    });
-    it('should trigger LLM check on the 30th turn', async () => {
-        mockBaseLlmClient.generateJson = vi
-            .fn()
-            .mockResolvedValue({ unproductive_state_confidence: 0.1 });
-        await advanceTurns(30);
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledTimes(1);
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledWith(expect.objectContaining({
-            modelConfigKey: { model: 'loop-detection' },
-            systemInstruction: expect.any(String),
-            contents: expect.any(Array),
-            schema: expect.any(Object),
-            promptId: expect.any(String),
-        }));
-    });
-    it('should detect a cognitive loop when confidence is high', async () => {
-        // First check at turn 30
-        mockBaseLlmClient.generateJson = vi.fn().mockResolvedValue({
-            unproductive_state_confidence: 0.85,
-            unproductive_state_analysis: 'Repetitive actions',
-        });
-        await advanceTurns(30);
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledTimes(1);
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledWith(expect.objectContaining({
-            modelConfigKey: { model: 'loop-detection' },
-        }));
-        // The confidence of 0.85 will result in a low interval.
-        // The interval will be: 5 + (15 - 5) * (1 - 0.85) = 5 + 10 * 0.15 = 6.5 -> rounded to 7
-        await advanceTurns(6); // advance to turn 36
-        mockBaseLlmClient.generateJson = vi.fn().mockResolvedValue({
-            unproductive_state_confidence: 0.95,
-            unproductive_state_analysis: 'Repetitive actions',
-        });
-        const finalResult = await service.turnStarted(abortController.signal); // This is turn 37
-        expect(finalResult).toBe(true);
-        expect(loggers.logLoopDetected).toHaveBeenCalledWith(mockConfig, expect.objectContaining({
-            'event.name': 'loop_detected',
-            loop_type: LoopType.LLM_DETECTED_LOOP,
-            confirmed_by_model: 'cognitive-loop-v1',
-        }));
-    });
-    it('should not detect a loop when confidence is low', async () => {
-        mockBaseLlmClient.generateJson = vi.fn().mockResolvedValue({
-            unproductive_state_confidence: 0.5,
-            unproductive_state_analysis: 'Looks okay',
-        });
-        await advanceTurns(30);
-        const result = await service.turnStarted(abortController.signal);
-        expect(result).toBe(false);
-        expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-    });
-    it('should adjust the check interval based on confidence', async () => {
-        // Confidence is 0.0, so interval should be MAX_LLM_CHECK_INTERVAL (15)
-        mockBaseLlmClient.generateJson = vi
-            .fn()
-            .mockResolvedValue({ unproductive_state_confidence: 0.0 });
-        await advanceTurns(30); // First check at turn 30
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledTimes(1);
-        await advanceTurns(14); // Advance to turn 44
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledTimes(1);
-        await service.turnStarted(abortController.signal); // Turn 45
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledTimes(2);
-    });
-    it('should handle errors from generateJson gracefully', async () => {
-        mockBaseLlmClient.generateJson = vi
-            .fn()
-            .mockRejectedValue(new Error('API error'));
-        await advanceTurns(30);
-        const result = await service.turnStarted(abortController.signal);
-        expect(result).toBe(false);
-        expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-    });
-    it('should not trigger LLM check when disabled for session', async () => {
-        service.disableForSession();
-        expect(loggers.logLoopDetectionDisabled).toHaveBeenCalledTimes(1);
-        await advanceTurns(30);
-        const result = await service.turnStarted(abortController.signal);
-        expect(result).toBe(false);
-        expect(mockBaseLlmClient.generateJson).not.toHaveBeenCalled();
-    });
-    it('should prepend user message if history starts with a function call', async () => {
-        const functionCallHistory = [
-            {
-                role: 'model',
-                parts: [{ functionCall: { name: 'someTool', args: {} } }],
-            },
-            {
-                role: 'model',
-                parts: [{ text: 'Some follow up text' }],
-            },
-        ];
-        vi.mocked(mockGeminiClient.getHistory).mockReturnValue(functionCallHistory);
-        mockBaseLlmClient.generateJson = vi
-            .fn()
-            .mockResolvedValue({ unproductive_state_confidence: 0.1 });
-        await advanceTurns(30);
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledTimes(1);
-        const calledArg = vi.mocked(mockBaseLlmClient.generateJson).mock
-            .calls[0][0];
-        expect(calledArg.contents[0]).toEqual({
-            role: 'user',
-            parts: [{ text: 'Recent conversation history:' }],
-        });
-        // Verify the original history follows
-        expect(calledArg.contents[1]).toEqual(functionCallHistory[0]);
-    });
-    it('should detect a loop when confidence is exactly equal to the threshold (0.9)', async () => {
-        mockBaseLlmClient.generateJson = vi
-            .fn()
-            .mockResolvedValueOnce({
-            unproductive_state_confidence: 0.9,
-            unproductive_state_analysis: 'Flash says loop',
-        })
-            .mockResolvedValueOnce({
-            unproductive_state_confidence: 0.9,
-            unproductive_state_analysis: 'Main says loop',
-        });
-        await advanceTurns(30);
-        // It should have called generateJson twice
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledTimes(2);
-        expect(mockBaseLlmClient.generateJson).toHaveBeenNthCalledWith(1, expect.objectContaining({
-            modelConfigKey: { model: 'loop-detection' },
-        }));
-        expect(mockBaseLlmClient.generateJson).toHaveBeenNthCalledWith(2, expect.objectContaining({
-            modelConfigKey: { model: 'loop-detection-double-check' },
-        }));
-        // And it should have detected a loop
-        expect(loggers.logLoopDetected).toHaveBeenCalledWith(mockConfig, expect.objectContaining({
-            'event.name': 'loop_detected',
-            loop_type: LoopType.LLM_DETECTED_LOOP,
-            confirmed_by_model: 'cognitive-loop-v1',
-        }));
-    });
-    it('should not detect a loop when Flash is confident (0.9) but Main model is not (0.89)', async () => {
-        mockBaseLlmClient.generateJson = vi
-            .fn()
-            .mockResolvedValueOnce({
-            unproductive_state_confidence: 0.9,
-            unproductive_state_analysis: 'Flash says loop',
-        })
-            .mockResolvedValueOnce({
-            unproductive_state_confidence: 0.89,
-            unproductive_state_analysis: 'Main says no loop',
-        });
-        await advanceTurns(30);
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledTimes(2);
-        expect(mockBaseLlmClient.generateJson).toHaveBeenNthCalledWith(1, expect.objectContaining({
-            modelConfigKey: { model: 'loop-detection' },
-        }));
-        expect(mockBaseLlmClient.generateJson).toHaveBeenNthCalledWith(2, expect.objectContaining({
-            modelConfigKey: { model: 'loop-detection-double-check' },
-        }));
-        // Should NOT have detected a loop
-        expect(loggers.logLoopDetected).not.toHaveBeenCalled();
-        // But should have updated the interval based on the main model's confidence (0.89)
-        // Interval = 5 + (15-5) * (1 - 0.89) = 5 + 10 * 0.11 = 5 + 1.1 = 6.1 -> 6
-        // Advance by 6 turns
-        await advanceTurns(6);
-        // Next turn (37) should trigger another check
-        await service.turnStarted(abortController.signal);
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledTimes(3);
-    });
-    it('should only call Flash model if main model is unavailable', async () => {
-        // Mock availability to return unavailable for the main model
-        const availability = mockConfig.getModelAvailabilityService();
-        vi.mocked(availability.snapshot).mockReturnValue({
-            available: false,
-            reason: 'quota',
-        });
-        mockBaseLlmClient.generateJson = vi.fn().mockResolvedValueOnce({
-            unproductive_state_confidence: 0.9,
-            unproductive_state_analysis: 'Flash says loop',
-        });
-        await advanceTurns(30);
-        // It should have called generateJson only once
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledTimes(1);
-        expect(mockBaseLlmClient.generateJson).toHaveBeenCalledWith(expect.objectContaining({
-            modelConfigKey: { model: 'loop-detection' },
-        }));
-        // And it should have detected a loop
-        expect(loggers.logLoopDetected).toHaveBeenCalledWith(mockConfig, expect.objectContaining({
-            'event.name': 'loop_detected',
-            loop_type: LoopType.LLM_DETECTED_LOOP,
-            confirmed_by_model: 'gemini-2.5-flash',
-        }));
-    });
-});
-//# sourceMappingURL=loopDetectionService.test.js.map