fss-link 1.0.49 → 1.0.51

package/dist/commands/mcp/remove.test.ts

@@ -1,69 +0,0 @@
-/**
- * @license
- * Copyright 2025 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-
-import { vi, describe, it, expect, beforeEach } from 'vitest';
-import yargs from 'yargs';
-import { loadSettings, SettingScope } from '../../config/settings.js';
-import { removeCommand } from './remove.js';
-
-vi.mock('fs/promises', () => ({
-  readFile: vi.fn(),
-  writeFile: vi.fn(),
-}));
-
-vi.mock('../../config/settings.js', async () => {
-  const actual = await vi.importActual('../../config/settings.js');
-  return {
-    ...actual,
-    loadSettings: vi.fn(),
-  };
-});
-
-const mockedLoadSettings = loadSettings as vi.Mock;
-
-describe('mcp remove command', () => {
-  let parser: yargs.Argv;
-  let mockSetValue: vi.Mock;
-  let mockSettings: Record<string, unknown>;
-
-  beforeEach(() => {
-    vi.resetAllMocks();
-    const yargsInstance = yargs([]).command(removeCommand);
-    parser = yargsInstance;
-    mockSetValue = vi.fn();
-    mockSettings = {
-      mcpServers: {
-        'test-server': {
-          command: 'echo "hello"',
-        },
-      },
-    };
-    mockedLoadSettings.mockReturnValue({
-      forScope: () => ({ settings: mockSettings }),
-      setValue: mockSetValue,
-    });
-  });
-
-  it('should remove a server from project settings', async () => {
-    await parser.parseAsync('remove test-server');
-
-    expect(mockSetValue).toHaveBeenCalledWith(
-      SettingScope.Workspace,
-      'mcpServers',
-      {},
-    );
-  });
-
-  it('should show a message if server not found', async () => {
-    const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
-    await parser.parseAsync('remove non-existent-server');
-
-    expect(mockSetValue).not.toHaveBeenCalled();
-    expect(consoleSpy).toHaveBeenCalledWith(
-      'Server "non-existent-server" not found in project settings.',
-    );
-  });
-});

package/dist/commands/mcp/remove.ts

@@ -1,60 +0,0 @@
-/**
- * @license
- * Copyright 2025 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-
-// File for 'gemini mcp remove' command
-import type { CommandModule } from 'yargs';
-import { loadSettings, SettingScope } from '../../config/settings.js';
-
-async function removeMcpServer(
-  name: string,
-  options: {
-    scope: string;
-  },
-) {
-  const { scope } = options;
-  const settingsScope =
-    scope === 'user' ? SettingScope.User : SettingScope.Workspace;
-  const settings = loadSettings(process.cwd());
-
-  const existingSettings = settings.forScope(settingsScope).settings;
-  const mcpServers = existingSettings.mcpServers || {};
-
-  if (!mcpServers[name]) {
-    console.log(`Server "${name}" not found in ${scope} settings.`);
-    return;
-  }
-
-  delete mcpServers[name];
-
-  settings.setValue(settingsScope, 'mcpServers', mcpServers);
-
-  console.log(`Server "${name}" removed from ${scope} settings.`);
-}
-
-export const removeCommand: CommandModule = {
-  command: 'remove <name>',
-  describe: 'Remove a server',
-  builder: (yargs) =>
-    yargs
-      .usage('Usage: gemini mcp remove [options] <name>')
-      .positional('name', {
-        describe: 'Name of the server',
-        type: 'string',
-        demandOption: true,
-      })
-      .option('scope', {
-        alias: 's',
-        describe: 'Configuration scope (user or project)',
-        type: 'string',
-        default: 'project',
-        choices: ['user', 'project'],
-      }),
-  handler: async (argv) => {
-    await removeMcpServer(argv['name'] as string, {
-      scope: argv['scope'] as string,
-    });
-  },
-};

package/dist/commands/mcp.test.ts

@@ -1,55 +0,0 @@
-/**
- * @license
- * Copyright 2025 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-
-import { describe, it, expect, vi } from 'vitest';
-import { mcpCommand } from './mcp.js';
-import { type Argv } from 'yargs';
-import yargs from 'yargs';
-
-describe('mcp command', () => {
-  it('should have correct command definition', () => {
-    expect(mcpCommand.command).toBe('mcp');
-    expect(mcpCommand.describe).toBe('Manage MCP servers');
-    expect(typeof mcpCommand.builder).toBe('function');
-    expect(typeof mcpCommand.handler).toBe('function');
-  });
-
-  it('should have exactly one option (help flag)', () => {
-    // Test to ensure that the global 'gemini' flags are not added to the mcp command
-    const yargsInstance = yargs();
-    const builtYargs = mcpCommand.builder(yargsInstance);
-    const options = builtYargs.getOptions();
-
-    // Should have exactly 1 option (help flag)
-    expect(Object.keys(options.key).length).toBe(1);
-    expect(options.key).toHaveProperty('help');
-  });
-
-  it('should register add, remove, and list subcommands', () => {
-    const mockYargs = {
-      command: vi.fn().mockReturnThis(),
-      demandCommand: vi.fn().mockReturnThis(),
-      version: vi.fn().mockReturnThis(),
-    };
-
-    mcpCommand.builder(mockYargs as unknown as Argv);
-
-    expect(mockYargs.command).toHaveBeenCalledTimes(3);
-
-    // Verify that the specific subcommands are registered
-    const commandCalls = mockYargs.command.mock.calls;
-    const commandNames = commandCalls.map((call) => call[0].command);
-
-    expect(commandNames).toContain('add <name> <commandOrUrl> [args...]');
-    expect(commandNames).toContain('remove <name>');
-    expect(commandNames).toContain('list');
-
-    expect(mockYargs.demandCommand).toHaveBeenCalledWith(
-      1,
-      'You need at least one command before continuing.',
-    );
-  });
-});

package/dist/commands/mcp.ts

@@ -1,27 +0,0 @@
-/**
- * @license
- * Copyright 2025 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-
-// File for 'gemini mcp' command
-import type { CommandModule, Argv } from 'yargs';
-import { addCommand } from './mcp/add.js';
-import { removeCommand } from './mcp/remove.js';
-import { listCommand } from './mcp/list.js';
-
-export const mcpCommand: CommandModule = {
-  command: 'mcp',
-  describe: 'Manage MCP servers',
-  builder: (yargs: Argv) =>
-    yargs
-      .command(addCommand)
-      .command(removeCommand)
-      .command(listCommand)
-      .demandCommand(1, 'You need at least one command before continuing.')
-      .version(false),
-  handler: () => {
-    // yargs will automatically show help if no subcommand is provided
-    // thanks to demandCommand(1) in the builder.
-  },
-};

package/dist/config/apiValidation.test.ts

@@ -1,118 +0,0 @@
-/**
- * @license
- * Copyright 2025 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { validateAuthMethod } from './auth.js';
-import { AuthType } from 'fss-link-core';
-
-describe('API Key Validation Edge Cases', () => {
-  let originalEnv: NodeJS.ProcessEnv;
-
-  beforeEach(() => {
-    // Save original environment
-    originalEnv = { ...process.env };
-  });
-
-  afterEach(() => {
-    // Restore original environment
-    process.env = originalEnv;
-  });
-
-  describe('Required API Key Validation', () => {
-    it('should require GEMINI_API_KEY for Gemini auth', () => {
-      delete process.env['GEMINI_API_KEY'];
-      const result = validateAuthMethod(AuthType.USE_GEMINI);
-      expect(result).toBe('GEMINI_API_KEY environment variable not found. Add that to your environment and try again (no reload needed if using .env)!');
-    });
-
-    it('should pass with valid GEMINI_API_KEY', () => {
-      process.env['GEMINI_API_KEY'] = 'test-gemini-key';
-      const result = validateAuthMethod(AuthType.USE_GEMINI);
-      expect(result).toBeNull();
-    });
-
-    it('should require OPENAI_API_KEY for OpenAI auth', () => {
-      delete process.env['OPENAI_API_KEY'];
-      const result = validateAuthMethod(AuthType.USE_OPENAI);
-      expect(result).toBe('OPENAI_API_KEY environment variable not found. You can enter it interactively or add it to your .env file.');
-    });
-
-    it('should pass with valid OPENAI_API_KEY', () => {
-      process.env['OPENAI_API_KEY'] = 'test-openai-key';
-      const result = validateAuthMethod(AuthType.USE_OPENAI);
-      expect(result).toBeNull();
-    });
-  });
-
-  describe('Optional API Key Validation', () => {
-    it('should not require API key for Ollama', () => {
-      delete process.env['OLLAMA_API_KEY']; // Even if this existed, it shouldn't be required
-      const result = validateAuthMethod(AuthType.OLLAMA);
-      expect(result).toBeNull();
-    });
-
-    it('should not require API key for LM Studio', () => {
-      delete process.env['LM_STUDIO_API_KEY']; // Even if this existed, it shouldn't be required
-      const result = validateAuthMethod(AuthType.LM_STUDIO);
-      expect(result).toBeNull();
-    });
-
-    it('should not require API key for Qwen OAuth', () => {
-      // OAuth flow handles its own authentication
-      const result = validateAuthMethod(AuthType.QWEN_OAUTH);
-      expect(result).toBeNull();
-    });
-  });
-
-  describe('Complex Vertex AI Validation', () => {
-    it('should require either project/location or API key for Vertex AI', () => {
-      delete process.env['GOOGLE_CLOUD_PROJECT'];
-      delete process.env['GOOGLE_CLOUD_LOCATION'];
-      delete process.env['GOOGLE_API_KEY'];
-      
-      const result = validateAuthMethod(AuthType.USE_VERTEX_AI);
-      expect(result).toContain('When using Vertex AI, you must specify either');
-    });
-
-    it('should pass with project and location', () => {
-      process.env['GOOGLE_CLOUD_PROJECT'] = 'test-project';
-      process.env['GOOGLE_CLOUD_LOCATION'] = 'us-central1';
-      delete process.env['GOOGLE_API_KEY'];
-      
-      const result = validateAuthMethod(AuthType.USE_VERTEX_AI);
-      expect(result).toBeNull();
-    });
-
-    it('should pass with Google API key', () => {
-      delete process.env['GOOGLE_CLOUD_PROJECT'];
-      delete process.env['GOOGLE_CLOUD_LOCATION'];
-      process.env['GOOGLE_API_KEY'] = 'test-google-key';
-      
-      const result = validateAuthMethod(AuthType.USE_VERTEX_AI);
-      expect(result).toBeNull();
-    });
-  });
-
-  describe('Edge Case Scenarios', () => {
-    it('should handle empty string API keys as missing', () => {
-      process.env['GEMINI_API_KEY'] = '';
-      const result = validateAuthMethod(AuthType.USE_GEMINI);
-      expect(result).toBe('GEMINI_API_KEY environment variable not found. Add that to your environment and try again (no reload needed if using .env)!');
-    });
-
-    it('should handle whitespace-only API keys as missing', () => {
-      process.env['OPENAI_API_KEY'] = '   ';
-      // const _result = validateAuthMethod(AuthType.USE_OPENAI);
-      // The current implementation might not handle this edge case
-      // This test documents the expected behavior
-    });
-
-    it('should reject invalid auth methods', () => {
-      const result = validateAuthMethod('invalid_auth_method');
-      expect(result).toBe('Invalid auth method selected.');
-    });
-  });
-});

package/dist/config/auth.test.ts

@@ -1,79 +0,0 @@
-/**
- * @license
- * Copyright 2025 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-
-import { AuthType } from 'fss-link-core';
-import { vi } from 'vitest';
-import { validateAuthMethod } from './auth.js';
-
-vi.mock('./settings.js', async (importOriginal) => {
-  const actual = await importOriginal<typeof import('./settings.js')>();
-  return {
-    ...actual,
-    loadEnvironment: vi.fn(),
-  };
-});
-
-describe('validateAuthMethod', () => {
-  const originalEnv = process.env;
-
-  beforeEach(() => {
-    vi.resetModules();
-    process.env = {};
-  });
-
-  afterEach(() => {
-    process.env = originalEnv;
-  });
-
-  it('should return null for LOGIN_WITH_GOOGLE', () => {
-    expect(validateAuthMethod(AuthType.LOGIN_WITH_GOOGLE)).toBeNull();
-  });
-
-  it('should return null for CLOUD_SHELL', () => {
-    expect(validateAuthMethod(AuthType.CLOUD_SHELL)).toBeNull();
-  });
-
-  describe('USE_GEMINI', () => {
-    it('should return null if GEMINI_API_KEY is set', () => {
-      process.env['GEMINI_API_KEY'] = 'test-key';
-      expect(validateAuthMethod(AuthType.USE_GEMINI)).toBeNull();
-    });
-
-    it('should return an error message if GEMINI_API_KEY is not set', () => {
-      expect(validateAuthMethod(AuthType.USE_GEMINI)).toBe(
-        'GEMINI_API_KEY environment variable not found. Add that to your environment and try again (no reload needed if using .env)!',
-      );
-    });
-  });
-
-  describe('USE_VERTEX_AI', () => {
-    it('should return null if GOOGLE_CLOUD_PROJECT and GOOGLE_CLOUD_LOCATION are set', () => {
-      process.env['GOOGLE_CLOUD_PROJECT'] = 'test-project';
-      process.env['GOOGLE_CLOUD_LOCATION'] = 'test-location';
-      expect(validateAuthMethod(AuthType.USE_VERTEX_AI)).toBeNull();
-    });
-
-    it('should return null if GOOGLE_API_KEY is set', () => {
-      process.env['GOOGLE_API_KEY'] = 'test-api-key';
-      expect(validateAuthMethod(AuthType.USE_VERTEX_AI)).toBeNull();
-    });
-
-    it('should return an error message if no required environment variables are set', () => {
-      expect(validateAuthMethod(AuthType.USE_VERTEX_AI)).toBe(
-        'When using Vertex AI, you must specify either:\n' +
-          '• GOOGLE_CLOUD_PROJECT and GOOGLE_CLOUD_LOCATION environment variables.\n' +
-          '• GOOGLE_API_KEY environment variable (if using express mode).\n' +
-          'Update your environment and try again (no reload needed if using .env)!',
-      );
-    });
-  });
-
-  it('should return an error message for an invalid auth method', () => {
-    expect(validateAuthMethod('invalid-method')).toBe(
-      'Invalid auth method selected.',
-    );
-  });
-});

package/dist/config/auth.ts

@@ -1,100 +0,0 @@
-/**
- * @license
- * Copyright 2025 Google LLC
- * SPDX-License-Identifier: Apache-2.0
- */
-
-import { AuthType } from 'fss-link-core';
-import { loadEnvironment } from './settings.js';
-import { getModelManager } from './modelManager.js';
-
-export const validateAuthMethod = (authMethod: string): string | null => {
-  loadEnvironment();
-  
-  // Initialize environment from database if we have an active model
-  const modelManager = getModelManager();
-  modelManager.initializeFromStore();
-  
-  if (
-    authMethod === AuthType.LOGIN_WITH_GOOGLE ||
-    authMethod === AuthType.CLOUD_SHELL
-  ) {
-    return null;
-  }
-
-  if (authMethod === AuthType.USE_GEMINI) {
-    if (!process.env['GEMINI_API_KEY']) {
-      return 'GEMINI_API_KEY environment variable not found. Add that to your environment and try again (no reload needed if using .env)!';
-    }
-    return null;
-  }
-
-  if (authMethod === AuthType.USE_VERTEX_AI) {
-    const hasVertexProjectLocationConfig =
-      !!process.env['GOOGLE_CLOUD_PROJECT'] &&
-      !!process.env['GOOGLE_CLOUD_LOCATION'];
-    const hasGoogleApiKey = !!process.env['GOOGLE_API_KEY'];
-    if (!hasVertexProjectLocationConfig && !hasGoogleApiKey) {
-      return (
-        'When using Vertex AI, you must specify either:\n' +
-        '• GOOGLE_CLOUD_PROJECT and GOOGLE_CLOUD_LOCATION environment variables.\n' +
-        '• GOOGLE_API_KEY environment variable (if using express mode).\n' +
-        'Update your environment and try again (no reload needed if using .env)!'
-      );
-    }
-    return null;
-  }
-
-  if (authMethod === AuthType.USE_OPENAI) {
-    if (!process.env['OPENAI_API_KEY']) {
-      return 'OPENAI_API_KEY environment variable not found. You can enter it interactively or add it to your .env file.';
-    }
-    return null;
-  }
-
-  if (authMethod === AuthType.QWEN_OAUTH) {
-    // FSS Link OAuth doesn't require any environment variables for basic setup
-    // The OAuth flow will handle authentication
-    return null;
-  }
-
-  if (authMethod === AuthType.OLLAMA) {
-    // Ollama doesn't require API key - local endpoint
-    return null;
-  }
-
-  if (authMethod === AuthType.LM_STUDIO) {
-    // LM Studio doesn't require API key - local endpoint
-    return null;
-  }
-
-  return 'Invalid auth method selected.';
-};
-
-export const setOpenAIApiKey = (apiKey: string): void => {
-  process.env['OPENAI_API_KEY'] = apiKey;
-};
-
-export const setOpenAIBaseUrl = (baseUrl: string): void => {
-  process.env['OPENAI_BASE_URL'] = baseUrl;
-};
-
-export const setOpenAIModel = (model: string): void => {
-  process.env['OPENAI_MODEL'] = model;
-};
-
-export const setOllamaBaseUrl = (baseUrl: string): void => {
-  process.env['OLLAMA_BASE_URL'] = baseUrl || 'http://localhost:11434/v1';
-};
-
-export const setOllamaModel = (model: string): void => {
-  process.env['OLLAMA_MODEL'] = model;
-};
-
-export const setLMStudioBaseUrl = (baseUrl: string): void => {
-  process.env['LM_STUDIO_BASE_URL'] = baseUrl || 'http://localhost:1234/v1';
-};
-
-export const setLMStudioModel = (model: string): void => {
-  process.env['LM_STUDIO_MODEL'] = model;
-};