2025-08-06 11:52:29 -04:00
|
|
|
/**
|
|
|
|
|
* @license
|
|
|
|
|
* Copyright 2025 Google LLC
|
|
|
|
|
* SPDX-License-Identifier: Apache-2.0
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
import yargs from 'yargs';
|
|
|
|
|
import { addCommand } from './add.js';
|
|
|
|
|
import { loadSettings, SettingScope } from '../../config/settings.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 add command', () => {
|
|
|
|
|
let parser: yargs.Argv;
|
|
|
|
|
let mockSetValue: vi.Mock;
|
|
|
|
|
|
|
|
|
|
beforeEach(() => {
|
|
|
|
|
vi.resetAllMocks();
|
|
|
|
|
const yargsInstance = yargs([]).command(addCommand);
|
|
|
|
|
parser = yargsInstance;
|
|
|
|
|
mockSetValue = vi.fn();
|
|
|
|
|
mockedLoadSettings.mockReturnValue({
|
|
|
|
|
forScope: () => ({ settings: {} }),
|
|
|
|
|
setValue: mockSetValue,
|
|
|
|
|
});
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
it('should add a stdio server to project settings', async () => {
|
|
|
|
|
await parser.parseAsync(
|
|
|
|
|
'add my-server /path/to/server arg1 arg2 -e FOO=bar',
|
|
|
|
|
);
|
|
|
|
|
|
|
|
|
|
expect(mockSetValue).toHaveBeenCalledWith(
|
|
|
|
|
SettingScope.Workspace,
|
|
|
|
|
'mcpServers',
|
|
|
|
|
{
|
|
|
|
|
'my-server': {
|
|
|
|
|
command: '/path/to/server',
|
|
|
|
|
args: ['arg1', 'arg2'],
|
|
|
|
|
env: { FOO: 'bar' },
|
|
|
|
|
},
|
|
|
|
|
},
|
|
|
|
|
);
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
it('should add an sse server to user settings', async () => {
|
|
|
|
|
await parser.parseAsync(
|
|
|
|
|
'add --transport sse sse-server https://example.com/sse-endpoint --scope user -H "X-API-Key: your-key"',
|
|
|
|
|
);
|
|
|
|
|
|
|
|
|
|
expect(mockSetValue).toHaveBeenCalledWith(SettingScope.User, 'mcpServers', {
|
|
|
|
|
'sse-server': {
|
|
|
|
|
url: 'https://example.com/sse-endpoint',
|
|
|
|
|
headers: { 'X-API-Key': 'your-key' },
|
|
|
|
|
},
|
|
|
|
|
});
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
it('should add an http server to project settings', async () => {
|
|
|
|
|
await parser.parseAsync(
|
|
|
|
|
'add --transport http http-server https://example.com/mcp -H "Authorization: Bearer your-token"',
|
|
|
|
|
);
|
|
|
|
|
|
|
|
|
|
expect(mockSetValue).toHaveBeenCalledWith(
|
|
|
|
|
SettingScope.Workspace,
|
|
|
|
|
'mcpServers',
|
|
|
|
|
{
|
|
|
|
|
'http-server': {
|
|
|
|
|
httpUrl: 'https://example.com/mcp',
|
|
|
|
|
headers: { Authorization: 'Bearer your-token' },
|
|
|
|
|
},
|
|
|
|
|
},
|
|
|
|
|
);
|
|
|
|
|
});
|
2025-08-15 15:36:38 -04:00
|
|
|
|
|
|
|
|
it('should handle MCP server args with -- separator', async () => {
|
|
|
|
|
await parser.parseAsync(
|
|
|
|
|
'add my-server npx -- -y http://example.com/some-package',
|
|
|
|
|
);
|
|
|
|
|
|
|
|
|
|
expect(mockSetValue).toHaveBeenCalledWith(
|
|
|
|
|
SettingScope.Workspace,
|
|
|
|
|
'mcpServers',
|
|
|
|
|
{
|
|
|
|
|
'my-server': {
|
|
|
|
|
command: 'npx',
|
|
|
|
|
args: ['-y', 'http://example.com/some-package'],
|
|
|
|
|
},
|
|
|
|
|
},
|
|
|
|
|
);
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
it('should handle unknown options as MCP server args', async () => {
|
|
|
|
|
await parser.parseAsync(
|
|
|
|
|
'add test-server npx -y http://example.com/some-package',
|
|
|
|
|
);
|
|
|
|
|
|
|
|
|
|
expect(mockSetValue).toHaveBeenCalledWith(
|
|
|
|
|
SettingScope.Workspace,
|
|
|
|
|
'mcpServers',
|
|
|
|
|
{
|
|
|
|
|
'test-server': {
|
|
|
|
|
command: 'npx',
|
|
|
|
|
args: ['-y', 'http://example.com/some-package'],
|
|
|
|
|
},
|
|
|
|
|
},
|
|
|
|
|
);
|
|
|
|
|
});
|
2025-08-06 11:52:29 -04:00
|
|
|
});
|