mirror of
https://github.com/davidkaya/aryx.git
synced 2026-07-24 13:38:43 +02:00
- load runtime tools dynamically from Copilot CLI capabilities via tools.list - merge runtime tools with configured MCP and LSP tools in the approval catalog - keep a fallback builtin runtime tool list when capabilities are unavailable - move approval-tool pruning to the app service so dynamic tools are not dropped on load - update approval UI and docs to use the corrected runtime-tool model Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
245 lines
6.8 KiB
TypeScript
245 lines
6.8 KiB
TypeScript
import { describe, expect, test } from 'bun:test';
|
|
|
|
import {
|
|
listApprovalToolDefinitions,
|
|
normalizeWorkspaceSettings,
|
|
validateLspProfileDefinition,
|
|
validateMcpServerDefinition,
|
|
type LspProfileDefinition,
|
|
type McpServerDefinition,
|
|
} from '@shared/domain/tooling';
|
|
|
|
const TIMESTAMP = '2026-03-23T00:00:00.000Z';
|
|
|
|
describe('tooling settings helpers', () => {
|
|
test('normalizes persisted MCP and LSP definitions into trimmed stable settings', () => {
|
|
const workspaceSettings = normalizeWorkspaceSettings({
|
|
tooling: {
|
|
mcpServers: [
|
|
{
|
|
id: 'mcp-local',
|
|
name: ' Local MCP ',
|
|
transport: 'local',
|
|
command: ' node ',
|
|
args: [' --stdio ', ' --stdio ', ''],
|
|
cwd: ' C:\\workspace\\repo ',
|
|
tools: [' git.status ', '', ' git.status '],
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
},
|
|
{
|
|
id: 'mcp-remote',
|
|
name: ' Remote MCP ',
|
|
transport: 'http',
|
|
url: ' https://example.com/mcp ',
|
|
tools: [],
|
|
timeoutMs: 1000,
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
},
|
|
],
|
|
lspProfiles: [
|
|
{
|
|
id: 'lsp-ts',
|
|
name: ' TypeScript ',
|
|
command: ' typescript-language-server ',
|
|
args: [' --stdio ', ' --stdio ', ''],
|
|
languageId: ' typescript ',
|
|
fileExtensions: ['ts', ' .tsx ', ''],
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
},
|
|
],
|
|
},
|
|
});
|
|
|
|
expect(workspaceSettings).toEqual({
|
|
theme: 'dark',
|
|
tooling: {
|
|
mcpServers: [
|
|
{
|
|
id: 'mcp-local',
|
|
name: 'Local MCP',
|
|
transport: 'local',
|
|
command: 'node',
|
|
args: ['--stdio'],
|
|
cwd: 'C:\\workspace\\repo',
|
|
tools: ['git.status'],
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
},
|
|
{
|
|
id: 'mcp-remote',
|
|
name: 'Remote MCP',
|
|
transport: 'http',
|
|
url: 'https://example.com/mcp',
|
|
tools: [],
|
|
timeoutMs: 1000,
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
},
|
|
],
|
|
lspProfiles: [
|
|
{
|
|
id: 'lsp-ts',
|
|
name: 'TypeScript',
|
|
command: 'typescript-language-server',
|
|
args: ['--stdio'],
|
|
languageId: 'typescript',
|
|
fileExtensions: ['.ts', '.tsx'],
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
},
|
|
],
|
|
},
|
|
});
|
|
});
|
|
|
|
test('validates required MCP transport settings', () => {
|
|
const localServer: McpServerDefinition = {
|
|
id: 'mcp-local',
|
|
name: 'Local MCP',
|
|
transport: 'local',
|
|
command: '',
|
|
args: [],
|
|
tools: ['*'],
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
};
|
|
const remoteServer: McpServerDefinition = {
|
|
id: 'mcp-remote',
|
|
name: 'Remote MCP',
|
|
transport: 'sse',
|
|
url: 'not-a-url',
|
|
tools: ['*'],
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
};
|
|
|
|
expect(validateMcpServerDefinition(localServer)).toBe('MCP server "Local MCP" needs a command.');
|
|
expect(validateMcpServerDefinition(remoteServer)).toBe('MCP server "Remote MCP" has an invalid URL.');
|
|
});
|
|
|
|
test('validates required LSP profile settings', () => {
|
|
const profile: LspProfileDefinition = {
|
|
id: 'lsp-empty',
|
|
name: 'TypeScript',
|
|
command: '',
|
|
args: [],
|
|
languageId: 'typescript',
|
|
fileExtensions: [],
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
};
|
|
|
|
expect(validateLspProfileDefinition(profile)).toBe('LSP profile "TypeScript" needs a command.');
|
|
expect(
|
|
validateLspProfileDefinition({
|
|
...profile,
|
|
command: 'typescript-language-server',
|
|
args: ['--stdio'],
|
|
}),
|
|
).toBe('LSP profile "TypeScript" needs at least one file extension.');
|
|
});
|
|
|
|
test('requires the stdio flag for the TypeScript language server profile', () => {
|
|
expect(
|
|
validateLspProfileDefinition({
|
|
id: 'lsp-ts',
|
|
name: 'Typescript LSP',
|
|
command: 'typescript-language-server',
|
|
args: [],
|
|
languageId: 'typescript',
|
|
fileExtensions: ['.ts', '.tsx'],
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
}),
|
|
).toBe('LSP profile "Typescript LSP" needs the "--stdio" argument.');
|
|
});
|
|
|
|
test('lists builtin, MCP, and LSP approval tools using runtime tool identifiers', () => {
|
|
const tools = listApprovalToolDefinitions({
|
|
mcpServers: [
|
|
{
|
|
id: 'mcp-git',
|
|
name: 'Git MCP',
|
|
transport: 'local',
|
|
command: 'node',
|
|
args: ['server.js'],
|
|
tools: ['git.status', 'git.diff'],
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
},
|
|
{
|
|
id: 'mcp-extra',
|
|
name: 'Extra MCP',
|
|
transport: 'local',
|
|
command: 'node',
|
|
args: ['extra.js'],
|
|
tools: ['git.status'],
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
},
|
|
],
|
|
lspProfiles: [
|
|
{
|
|
id: 'ts',
|
|
name: 'TypeScript',
|
|
command: 'typescript-language-server',
|
|
args: ['--stdio'],
|
|
languageId: 'typescript',
|
|
fileExtensions: ['.ts', '.tsx'],
|
|
createdAt: TIMESTAMP,
|
|
updatedAt: TIMESTAMP,
|
|
},
|
|
],
|
|
});
|
|
|
|
expect(tools).toContainEqual({
|
|
id: 'web_fetch',
|
|
label: 'web_fetch',
|
|
description: 'Fetch content from a URL.',
|
|
kind: 'builtin',
|
|
providerIds: ['builtin:web_fetch'],
|
|
providerNames: ['Built-in'],
|
|
});
|
|
expect(tools).toContainEqual({
|
|
id: 'git.status',
|
|
label: 'git.status',
|
|
kind: 'mcp',
|
|
providerIds: ['mcp-git', 'mcp-extra'],
|
|
providerNames: ['Git MCP', 'Extra MCP'],
|
|
});
|
|
expect(tools).toContainEqual({
|
|
id: 'lsp_ts_hover',
|
|
label: 'TypeScript · Hover',
|
|
kind: 'lsp',
|
|
providerIds: ['ts'],
|
|
providerNames: ['TypeScript'],
|
|
});
|
|
});
|
|
|
|
test('prefers dynamically reported runtime tools over the fallback builtin catalog', () => {
|
|
const tools = listApprovalToolDefinitions(
|
|
{ mcpServers: [], lspProfiles: [] },
|
|
[
|
|
{
|
|
id: 'fetch',
|
|
label: 'fetch',
|
|
description: 'Dynamic runtime tool from Copilot CLI.',
|
|
},
|
|
],
|
|
);
|
|
|
|
expect(tools).toContainEqual({
|
|
id: 'fetch',
|
|
label: 'fetch',
|
|
description: 'Dynamic runtime tool from Copilot CLI.',
|
|
kind: 'builtin',
|
|
providerIds: ['builtin:fetch'],
|
|
providerNames: ['Built-in'],
|
|
});
|
|
expect(tools.some((tool) => tool.id === 'web_fetch')).toBe(false);
|
|
});
|
|
});
|