mirror of
https://github.com/AutoMaker-Org/automaker.git
synced 2026-02-01 20:23:36 +00:00
* feat: add Gemini CLI provider for AI model execution - Add GeminiProvider class extending CliProvider for Gemini CLI integration - Add Gemini models (Gemini 3 Pro/Flash Preview, 2.5 Pro/Flash/Flash-Lite) - Add gemini-models.ts with model definitions and types - Update ModelProvider type to include 'gemini' - Add isGeminiModel() to provider-utils.ts for model detection - Register Gemini provider in provider-factory with priority 4 - Add Gemini setup detection routes (status, auth, deauth) - Add GeminiCliStatus to setup store for UI state management - Add Gemini to PROVIDER_ICON_COMPONENTS for UI icon display - Add GEMINI_MODELS to model-display for dropdown population - Support thinking levels: off, low, medium, high Based on https://github.com/google-gemini/gemini-cli * chore: update package-lock.json * feat(ui): add Gemini provider to settings and setup wizard - Add GeminiCliStatus component for CLI detection display - Add GeminiSettingsTab component for global settings - Update provider-tabs.tsx to include Gemini as 5th tab - Update providers-setup-step.tsx with Gemini provider detection - Add useGeminiCliStatus hook for querying CLI status - Add getGeminiStatus, authGemini, deauthGemini to HTTP API client - Add gemini query key for React Query - Fix GeminiModelId type to not double-prefix model IDs * feat(ui): add Gemini to settings sidebar navigation - Add 'gemini-provider' to SettingsViewId type - Add GeminiIcon and gemini-provider to navigation config - Add gemini-provider to NAV_ID_TO_PROVIDER mapping - Add gemini-provider case in settings-view switch - Export GeminiSettingsTab from providers index This fixes the missing Gemini entry in the AI Providers sidebar menu. * feat(ui): add Gemini model configuration in settings - Create GeminiModelConfiguration component for model selection - Add enabledGeminiModels and geminiDefaultModel state to app-store - Add setEnabledGeminiModels, setGeminiDefaultModel, toggleGeminiModel actions - Update GeminiSettingsTab to show model configuration when CLI is installed - Import GeminiModelId and getAllGeminiModelIds from types This adds the ability to configure which Gemini models are available in the feature modal, similar to other providers like Codex and OpenCode. * feat(ui): add Gemini models to all model dropdowns - Add GEMINI_MODELS to model-constants.ts for UI dropdowns - Add Gemini to ALL_MODELS array used throughout the app - Add GeminiIcon to PROFILE_ICONS mapping - Fix GEMINI_MODELS in model-display.ts to use correct model IDs - Update getModelDisplayName to handle Gemini models correctly Gemini models now appear in all model selection dropdowns including Model Defaults, Feature Defaults, and feature card settings. * fix(gemini): fix CLI integration and event handling - Fix model ID prefix handling: strip gemini- prefix in agent-service, add it back in buildCliArgs for CLI invocation - Fix event normalization to match actual Gemini CLI output format: - type: 'init' (not 'system') - type: 'message' with role (not 'assistant') - tool_name/tool_id/parameters/output field names - Add --sandbox false and --approval-mode yolo for faster execution - Remove thinking level selector from UI (Gemini CLI doesn't support it) - Update auth status to show errors properly * test: update provider-factory tests for Gemini provider - Add GeminiProvider import and spy mock - Update expected provider count from 4 to 5 - Add test for GeminiProvider inclusion - Add gemini key to checkAllProviders test * fix(gemini): address PR review feedback - Fix npm package name from @anthropic-ai/gemini-cli to @google/gemini-cli - Fix comments in gemini-provider.ts to match actual CLI output format - Convert sync fs operations to async using fs/promises * fix(settings): add Gemini and Codex settings to sync Add enabledGeminiModels, geminiDefaultModel, enabledCodexModels, and codexDefaultModel to SETTINGS_FIELDS_TO_SYNC for persistence across sessions. * fix(gemini): address additional PR review feedback - Use 'Speed' badge for non-thinking Gemini models (consistency) - Fix installCommand mapping in gemini-settings-tab.tsx - Add hasEnvApiKey to GeminiCliStatus interface for API parity - Clarify GeminiThinkingLevel comment (CLI doesn't support --thinking-level) * fix(settings): restore Codex and Gemini settings from server Add sanitization and restoration logic for enabledCodexModels, codexDefaultModel, enabledGeminiModels, and geminiDefaultModel in refreshSettingsFromServer() to match the fields in SETTINGS_FIELDS_TO_SYNC. * feat(gemini): normalize tool names and fix workspace restrictions - Add tool name mapping to normalize Gemini CLI tool names to standard names (e.g., write_todos -> TodoWrite, read_file -> Read) - Add normalizeGeminiToolInput to convert write_todos format to TodoWrite format (description -> content, handle cancelled status) - Pass --include-directories with cwd to fix workspace restriction errors when Gemini CLI has a different cached workspace from previous sessions --------- Co-authored-by: Claude <noreply@anthropic.com>
319 lines
12 KiB
TypeScript
319 lines
12 KiB
TypeScript
import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
|
|
import { ProviderFactory } from '@/providers/provider-factory.js';
|
|
import { ClaudeProvider } from '@/providers/claude-provider.js';
|
|
import { CursorProvider } from '@/providers/cursor-provider.js';
|
|
import { CodexProvider } from '@/providers/codex-provider.js';
|
|
import { OpencodeProvider } from '@/providers/opencode-provider.js';
|
|
import { GeminiProvider } from '@/providers/gemini-provider.js';
|
|
|
|
describe('provider-factory.ts', () => {
|
|
let consoleSpy: any;
|
|
let detectClaudeSpy: any;
|
|
let detectCursorSpy: any;
|
|
let detectCodexSpy: any;
|
|
let detectOpencodeSpy: any;
|
|
let detectGeminiSpy: any;
|
|
|
|
beforeEach(() => {
|
|
consoleSpy = {
|
|
warn: vi.spyOn(console, 'warn').mockImplementation(() => {}),
|
|
};
|
|
|
|
// Avoid hitting real CLI / filesystem checks during unit tests
|
|
detectClaudeSpy = vi
|
|
.spyOn(ClaudeProvider.prototype, 'detectInstallation')
|
|
.mockResolvedValue({ installed: true });
|
|
detectCursorSpy = vi
|
|
.spyOn(CursorProvider.prototype, 'detectInstallation')
|
|
.mockResolvedValue({ installed: true });
|
|
detectCodexSpy = vi
|
|
.spyOn(CodexProvider.prototype, 'detectInstallation')
|
|
.mockResolvedValue({ installed: true });
|
|
detectOpencodeSpy = vi
|
|
.spyOn(OpencodeProvider.prototype, 'detectInstallation')
|
|
.mockResolvedValue({ installed: true });
|
|
detectGeminiSpy = vi
|
|
.spyOn(GeminiProvider.prototype, 'detectInstallation')
|
|
.mockResolvedValue({ installed: true });
|
|
});
|
|
|
|
afterEach(() => {
|
|
consoleSpy.warn.mockRestore();
|
|
detectClaudeSpy.mockRestore();
|
|
detectCursorSpy.mockRestore();
|
|
detectCodexSpy.mockRestore();
|
|
detectOpencodeSpy.mockRestore();
|
|
detectGeminiSpy.mockRestore();
|
|
});
|
|
|
|
describe('getProviderForModel', () => {
|
|
describe('Claude models (claude-* prefix)', () => {
|
|
it('should return ClaudeProvider for claude-opus-4-5-20251101', () => {
|
|
const provider = ProviderFactory.getProviderForModel('claude-opus-4-5-20251101');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
|
|
it('should return ClaudeProvider for claude-sonnet-4-20250514', () => {
|
|
const provider = ProviderFactory.getProviderForModel('claude-sonnet-4-20250514');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
|
|
it('should return ClaudeProvider for claude-haiku-4-5', () => {
|
|
const provider = ProviderFactory.getProviderForModel('claude-haiku-4-5');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
|
|
it('should be case-insensitive for claude models', () => {
|
|
const provider = ProviderFactory.getProviderForModel('CLAUDE-OPUS-4-5-20251101');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
});
|
|
|
|
describe('Claude aliases', () => {
|
|
it("should return ClaudeProvider for 'haiku'", () => {
|
|
const provider = ProviderFactory.getProviderForModel('haiku');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
|
|
it("should return ClaudeProvider for 'sonnet'", () => {
|
|
const provider = ProviderFactory.getProviderForModel('sonnet');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
|
|
it("should return ClaudeProvider for 'opus'", () => {
|
|
const provider = ProviderFactory.getProviderForModel('opus');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
|
|
it('should be case-insensitive for aliases', () => {
|
|
const provider1 = ProviderFactory.getProviderForModel('HAIKU');
|
|
const provider2 = ProviderFactory.getProviderForModel('Sonnet');
|
|
const provider3 = ProviderFactory.getProviderForModel('Opus');
|
|
|
|
expect(provider1).toBeInstanceOf(ClaudeProvider);
|
|
expect(provider2).toBeInstanceOf(ClaudeProvider);
|
|
expect(provider3).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
});
|
|
|
|
describe('Cursor models (cursor-* prefix)', () => {
|
|
it('should return CursorProvider for cursor-auto', () => {
|
|
const provider = ProviderFactory.getProviderForModel('cursor-auto');
|
|
expect(provider).toBeInstanceOf(CursorProvider);
|
|
});
|
|
|
|
it('should return CursorProvider for cursor-sonnet-4.5', () => {
|
|
const provider = ProviderFactory.getProviderForModel('cursor-sonnet-4.5');
|
|
expect(provider).toBeInstanceOf(CursorProvider);
|
|
});
|
|
|
|
it('should return CursorProvider for cursor-gpt-5.2', () => {
|
|
const provider = ProviderFactory.getProviderForModel('cursor-gpt-5.2');
|
|
expect(provider).toBeInstanceOf(CursorProvider);
|
|
});
|
|
|
|
it('should be case-insensitive for cursor models', () => {
|
|
const provider = ProviderFactory.getProviderForModel('CURSOR-AUTO');
|
|
expect(provider).toBeInstanceOf(CursorProvider);
|
|
});
|
|
|
|
it('should return CursorProvider for known cursor model ID without prefix', () => {
|
|
const provider = ProviderFactory.getProviderForModel('auto');
|
|
expect(provider).toBeInstanceOf(CursorProvider);
|
|
});
|
|
});
|
|
|
|
describe('Unknown models', () => {
|
|
it('should default to ClaudeProvider for unknown model', () => {
|
|
const provider = ProviderFactory.getProviderForModel('unknown-model-123');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
|
|
it('should handle empty string by defaulting to ClaudeProvider', () => {
|
|
const provider = ProviderFactory.getProviderForModel('');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
|
|
it('should default to ClaudeProvider for completely unknown prefixes', () => {
|
|
const provider = ProviderFactory.getProviderForModel('random-xyz-model');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
});
|
|
|
|
describe('Cursor models via model ID lookup', () => {
|
|
it('should return CodexProvider for gpt-5.2 (Codex model, not Cursor)', () => {
|
|
// gpt-5.2 is in both CURSOR_MODEL_MAP and CODEX_MODEL_CONFIG_MAP
|
|
// It should route to Codex since Codex models take priority
|
|
const provider = ProviderFactory.getProviderForModel('gpt-5.2');
|
|
expect(provider).toBeInstanceOf(CodexProvider);
|
|
});
|
|
|
|
it('should return CursorProvider for grok (valid Cursor model)', () => {
|
|
const provider = ProviderFactory.getProviderForModel('grok');
|
|
expect(provider).toBeInstanceOf(CursorProvider);
|
|
});
|
|
|
|
it('should return CursorProvider for gemini-3-pro (valid Cursor model)', () => {
|
|
const provider = ProviderFactory.getProviderForModel('gemini-3-pro');
|
|
expect(provider).toBeInstanceOf(CursorProvider);
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('getAllProviders', () => {
|
|
it('should return array of all providers', () => {
|
|
const providers = ProviderFactory.getAllProviders();
|
|
expect(Array.isArray(providers)).toBe(true);
|
|
});
|
|
|
|
it('should include ClaudeProvider', () => {
|
|
const providers = ProviderFactory.getAllProviders();
|
|
const hasClaudeProvider = providers.some((p) => p instanceof ClaudeProvider);
|
|
expect(hasClaudeProvider).toBe(true);
|
|
});
|
|
|
|
it('should return exactly 5 providers', () => {
|
|
const providers = ProviderFactory.getAllProviders();
|
|
expect(providers).toHaveLength(5);
|
|
});
|
|
|
|
it('should include GeminiProvider', () => {
|
|
const providers = ProviderFactory.getAllProviders();
|
|
const hasGeminiProvider = providers.some((p) => p instanceof GeminiProvider);
|
|
expect(hasGeminiProvider).toBe(true);
|
|
});
|
|
|
|
it('should include CursorProvider', () => {
|
|
const providers = ProviderFactory.getAllProviders();
|
|
const hasCursorProvider = providers.some((p) => p instanceof CursorProvider);
|
|
expect(hasCursorProvider).toBe(true);
|
|
});
|
|
|
|
it('should create new instances each time', () => {
|
|
const providers1 = ProviderFactory.getAllProviders();
|
|
const providers2 = ProviderFactory.getAllProviders();
|
|
|
|
expect(providers1[0]).not.toBe(providers2[0]);
|
|
});
|
|
});
|
|
|
|
describe('checkAllProviders', () => {
|
|
it('should return installation status for all providers', async () => {
|
|
const statuses = await ProviderFactory.checkAllProviders();
|
|
|
|
expect(statuses).toHaveProperty('claude');
|
|
});
|
|
|
|
it('should call detectInstallation on each provider', async () => {
|
|
const statuses = await ProviderFactory.checkAllProviders();
|
|
|
|
expect(statuses.claude).toHaveProperty('installed');
|
|
});
|
|
|
|
it('should return correct provider names as keys', async () => {
|
|
const statuses = await ProviderFactory.checkAllProviders();
|
|
const keys = Object.keys(statuses);
|
|
|
|
expect(keys).toContain('claude');
|
|
expect(keys).toContain('cursor');
|
|
expect(keys).toContain('codex');
|
|
expect(keys).toContain('opencode');
|
|
expect(keys).toContain('gemini');
|
|
expect(keys).toHaveLength(5);
|
|
});
|
|
|
|
it('should include cursor status', async () => {
|
|
const statuses = await ProviderFactory.checkAllProviders();
|
|
|
|
expect(statuses.cursor).toHaveProperty('installed');
|
|
});
|
|
});
|
|
|
|
describe('getProviderByName', () => {
|
|
it("should return ClaudeProvider for 'claude'", () => {
|
|
const provider = ProviderFactory.getProviderByName('claude');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
|
|
it("should return ClaudeProvider for 'anthropic'", () => {
|
|
const provider = ProviderFactory.getProviderByName('anthropic');
|
|
expect(provider).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
|
|
it("should return CursorProvider for 'cursor'", () => {
|
|
const provider = ProviderFactory.getProviderByName('cursor');
|
|
expect(provider).toBeInstanceOf(CursorProvider);
|
|
});
|
|
|
|
it('should be case-insensitive', () => {
|
|
const provider1 = ProviderFactory.getProviderByName('CLAUDE');
|
|
const provider2 = ProviderFactory.getProviderByName('ANTHROPIC');
|
|
const provider3 = ProviderFactory.getProviderByName('CURSOR');
|
|
|
|
expect(provider1).toBeInstanceOf(ClaudeProvider);
|
|
expect(provider2).toBeInstanceOf(ClaudeProvider);
|
|
expect(provider3).toBeInstanceOf(CursorProvider);
|
|
});
|
|
|
|
it('should return null for unknown provider', () => {
|
|
const provider = ProviderFactory.getProviderByName('unknown');
|
|
expect(provider).toBeNull();
|
|
});
|
|
|
|
it('should return null for empty string', () => {
|
|
const provider = ProviderFactory.getProviderByName('');
|
|
expect(provider).toBeNull();
|
|
});
|
|
|
|
it('should create new instance each time', () => {
|
|
const provider1 = ProviderFactory.getProviderByName('claude');
|
|
const provider2 = ProviderFactory.getProviderByName('claude');
|
|
|
|
expect(provider1).not.toBe(provider2);
|
|
expect(provider1).toBeInstanceOf(ClaudeProvider);
|
|
expect(provider2).toBeInstanceOf(ClaudeProvider);
|
|
});
|
|
});
|
|
|
|
describe('getAllAvailableModels', () => {
|
|
it('should return array of models', () => {
|
|
const models = ProviderFactory.getAllAvailableModels();
|
|
expect(Array.isArray(models)).toBe(true);
|
|
});
|
|
|
|
it('should include models from all providers', () => {
|
|
const models = ProviderFactory.getAllAvailableModels();
|
|
expect(models.length).toBeGreaterThan(0);
|
|
});
|
|
|
|
it('should return models with required fields', () => {
|
|
const models = ProviderFactory.getAllAvailableModels();
|
|
|
|
models.forEach((model) => {
|
|
expect(model).toHaveProperty('id');
|
|
expect(model).toHaveProperty('name');
|
|
expect(typeof model.id).toBe('string');
|
|
expect(typeof model.name).toBe('string');
|
|
});
|
|
});
|
|
|
|
it('should include Claude models', () => {
|
|
const models = ProviderFactory.getAllAvailableModels();
|
|
|
|
// Claude models should include claude-* in their IDs
|
|
const hasClaudeModels = models.some((m) => m.id.toLowerCase().includes('claude'));
|
|
|
|
expect(hasClaudeModels).toBe(true);
|
|
});
|
|
|
|
it('should include Cursor models', () => {
|
|
const models = ProviderFactory.getAllAvailableModels();
|
|
|
|
// Cursor models should include cursor provider
|
|
const hasCursorModels = models.some((m) => m.provider === 'cursor');
|
|
|
|
expect(hasCursorModels).toBe(true);
|
|
});
|
|
});
|
|
});
|