mirror of
https://github.com/google-gemini/gemini-cli.git
synced 2026-03-16 00:51:25 -07:00
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> Co-authored-by: Allen Hutchison <adh@google.com>
222 lines
7.8 KiB
TypeScript
222 lines
7.8 KiB
TypeScript
/**
|
|
* @license
|
|
* Copyright 2025 Google LLC
|
|
* SPDX-License-Identifier: Apache-2.0
|
|
*/
|
|
|
|
import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|
import { ModelRouterService } from './modelRouterService.js';
|
|
import { Config } from '../config/config.js';
|
|
|
|
import type { BaseLlmClient } from '../core/baseLlmClient.js';
|
|
import type { LocalLiteRtLmClient } from '../core/localLiteRtLmClient.js';
|
|
import type { RoutingContext, RoutingDecision } from './routingStrategy.js';
|
|
import { DefaultStrategy } from './strategies/defaultStrategy.js';
|
|
import { CompositeStrategy } from './strategies/compositeStrategy.js';
|
|
import { FallbackStrategy } from './strategies/fallbackStrategy.js';
|
|
import { OverrideStrategy } from './strategies/overrideStrategy.js';
|
|
import { ApprovalModeStrategy } from './strategies/approvalModeStrategy.js';
|
|
import { ClassifierStrategy } from './strategies/classifierStrategy.js';
|
|
import { NumericalClassifierStrategy } from './strategies/numericalClassifierStrategy.js';
|
|
import { logModelRouting } from '../telemetry/loggers.js';
|
|
import { ModelRoutingEvent } from '../telemetry/types.js';
|
|
import { GemmaClassifierStrategy } from './strategies/gemmaClassifierStrategy.js';
|
|
import { ApprovalMode } from '../policy/types.js';
|
|
|
|
vi.mock('../config/config.js');
|
|
vi.mock('../core/baseLlmClient.js');
|
|
vi.mock('./strategies/defaultStrategy.js');
|
|
vi.mock('./strategies/compositeStrategy.js');
|
|
vi.mock('./strategies/fallbackStrategy.js');
|
|
vi.mock('./strategies/overrideStrategy.js');
|
|
vi.mock('./strategies/approvalModeStrategy.js');
|
|
vi.mock('./strategies/classifierStrategy.js');
|
|
vi.mock('./strategies/numericalClassifierStrategy.js');
|
|
vi.mock('./strategies/gemmaClassifierStrategy.js');
|
|
vi.mock('../telemetry/loggers.js');
|
|
vi.mock('../telemetry/types.js');
|
|
|
|
describe('ModelRouterService', () => {
|
|
let service: ModelRouterService;
|
|
let mockConfig: Config;
|
|
let mockBaseLlmClient: BaseLlmClient;
|
|
let mockLocalLiteRtLmClient: LocalLiteRtLmClient;
|
|
let mockContext: RoutingContext;
|
|
let mockCompositeStrategy: CompositeStrategy;
|
|
|
|
beforeEach(() => {
|
|
vi.clearAllMocks();
|
|
|
|
mockConfig = new Config({} as never);
|
|
mockBaseLlmClient = {} as BaseLlmClient;
|
|
mockLocalLiteRtLmClient = {} as LocalLiteRtLmClient;
|
|
vi.spyOn(mockConfig, 'getBaseLlmClient').mockReturnValue(mockBaseLlmClient);
|
|
vi.spyOn(mockConfig, 'getLocalLiteRtLmClient').mockReturnValue(
|
|
mockLocalLiteRtLmClient,
|
|
);
|
|
vi.spyOn(mockConfig, 'getNumericalRoutingEnabled').mockResolvedValue(false);
|
|
vi.spyOn(mockConfig, 'getClassifierThreshold').mockResolvedValue(undefined);
|
|
vi.spyOn(mockConfig, 'getGemmaModelRouterSettings').mockReturnValue({
|
|
enabled: false,
|
|
classifier: {
|
|
host: 'http://localhost:1234',
|
|
model: 'gemma3-1b-gpu-custom',
|
|
},
|
|
});
|
|
vi.spyOn(mockConfig, 'getApprovalMode').mockReturnValue(
|
|
ApprovalMode.DEFAULT,
|
|
);
|
|
|
|
mockCompositeStrategy = new CompositeStrategy(
|
|
[
|
|
new FallbackStrategy(),
|
|
new OverrideStrategy(),
|
|
new ApprovalModeStrategy(),
|
|
new ClassifierStrategy(),
|
|
new NumericalClassifierStrategy(),
|
|
new DefaultStrategy(),
|
|
],
|
|
'agent-router',
|
|
);
|
|
vi.mocked(CompositeStrategy).mockImplementation(
|
|
() => mockCompositeStrategy,
|
|
);
|
|
|
|
service = new ModelRouterService(mockConfig);
|
|
|
|
mockContext = {
|
|
history: [],
|
|
request: [{ text: 'test prompt' }],
|
|
signal: new AbortController().signal,
|
|
};
|
|
});
|
|
|
|
it('should initialize with a CompositeStrategy', () => {
|
|
expect(CompositeStrategy).toHaveBeenCalled();
|
|
expect(service['strategy']).toBeInstanceOf(CompositeStrategy);
|
|
});
|
|
|
|
it('should initialize the CompositeStrategy with the correct child strategies in order', () => {
|
|
// This test relies on the mock implementation detail of the constructor
|
|
const compositeStrategyArgs = vi.mocked(CompositeStrategy).mock.calls[0];
|
|
const childStrategies = compositeStrategyArgs[0];
|
|
|
|
expect(childStrategies.length).toBe(6);
|
|
expect(childStrategies[0]).toBeInstanceOf(FallbackStrategy);
|
|
expect(childStrategies[1]).toBeInstanceOf(OverrideStrategy);
|
|
expect(childStrategies[2]).toBeInstanceOf(ApprovalModeStrategy);
|
|
expect(childStrategies[3]).toBeInstanceOf(ClassifierStrategy);
|
|
expect(childStrategies[4]).toBeInstanceOf(NumericalClassifierStrategy);
|
|
expect(childStrategies[5]).toBeInstanceOf(DefaultStrategy);
|
|
expect(compositeStrategyArgs[1]).toBe('agent-router');
|
|
});
|
|
|
|
it('should include GemmaClassifierStrategy when enabled', () => {
|
|
// Override the default mock for this specific test
|
|
vi.spyOn(mockConfig, 'getGemmaModelRouterSettings').mockReturnValue({
|
|
enabled: true,
|
|
classifier: {
|
|
host: 'http://localhost:1234',
|
|
model: 'gemma3-1b-gpu-custom',
|
|
},
|
|
});
|
|
|
|
// Clear previous mock calls from beforeEach
|
|
vi.mocked(CompositeStrategy).mockClear();
|
|
|
|
// Re-initialize the service to pick up the new config
|
|
service = new ModelRouterService(mockConfig);
|
|
|
|
const compositeStrategyArgs = vi.mocked(CompositeStrategy).mock.calls[0];
|
|
const childStrategies = compositeStrategyArgs[0];
|
|
|
|
expect(childStrategies.length).toBe(7);
|
|
expect(childStrategies[0]).toBeInstanceOf(FallbackStrategy);
|
|
expect(childStrategies[1]).toBeInstanceOf(OverrideStrategy);
|
|
expect(childStrategies[2]).toBeInstanceOf(ApprovalModeStrategy);
|
|
expect(childStrategies[3]).toBeInstanceOf(GemmaClassifierStrategy);
|
|
expect(childStrategies[4]).toBeInstanceOf(ClassifierStrategy);
|
|
expect(childStrategies[5]).toBeInstanceOf(NumericalClassifierStrategy);
|
|
expect(childStrategies[6]).toBeInstanceOf(DefaultStrategy);
|
|
expect(compositeStrategyArgs[1]).toBe('agent-router');
|
|
});
|
|
|
|
describe('route()', () => {
|
|
const strategyDecision: RoutingDecision = {
|
|
model: 'strategy-chosen-model',
|
|
metadata: {
|
|
source: 'test-router/fallback',
|
|
latencyMs: 10,
|
|
reasoning: 'Strategy reasoning',
|
|
},
|
|
};
|
|
|
|
it('should delegate routing to the composite strategy', async () => {
|
|
const strategySpy = vi
|
|
.spyOn(mockCompositeStrategy, 'route')
|
|
.mockResolvedValue(strategyDecision);
|
|
|
|
const decision = await service.route(mockContext);
|
|
|
|
expect(strategySpy).toHaveBeenCalledWith(
|
|
mockContext,
|
|
mockConfig,
|
|
mockBaseLlmClient,
|
|
mockLocalLiteRtLmClient,
|
|
);
|
|
expect(decision).toEqual(strategyDecision);
|
|
});
|
|
|
|
it('should log a telemetry event on a successful decision', async () => {
|
|
vi.spyOn(mockCompositeStrategy, 'route').mockResolvedValue(
|
|
strategyDecision,
|
|
);
|
|
|
|
await service.route(mockContext);
|
|
|
|
expect(ModelRoutingEvent).toHaveBeenCalledWith(
|
|
'strategy-chosen-model',
|
|
'test-router/fallback',
|
|
10,
|
|
'Strategy reasoning',
|
|
false,
|
|
undefined,
|
|
ApprovalMode.DEFAULT,
|
|
false,
|
|
undefined,
|
|
);
|
|
expect(logModelRouting).toHaveBeenCalledWith(
|
|
mockConfig,
|
|
expect.any(ModelRoutingEvent),
|
|
);
|
|
});
|
|
|
|
it('should log a telemetry event and return fallback on a failed decision', async () => {
|
|
const testError = new Error('Strategy failed');
|
|
vi.spyOn(mockCompositeStrategy, 'route').mockRejectedValue(testError);
|
|
vi.spyOn(mockConfig, 'getModel').mockReturnValue('default-model');
|
|
|
|
const decision = await service.route(mockContext);
|
|
|
|
expect(decision.model).toBe('default-model');
|
|
expect(decision.metadata.source).toBe('router-exception');
|
|
|
|
expect(ModelRoutingEvent).toHaveBeenCalledWith(
|
|
'default-model',
|
|
'router-exception',
|
|
expect.any(Number),
|
|
'An exception occurred during routing.',
|
|
true,
|
|
'Strategy failed',
|
|
ApprovalMode.DEFAULT,
|
|
false,
|
|
undefined,
|
|
);
|
|
expect(logModelRouting).toHaveBeenCalledWith(
|
|
mockConfig,
|
|
expect.any(ModelRoutingEvent),
|
|
);
|
|
});
|
|
});
|
|
});
|