mirror of
https://github.com/Kaelio/ktx.git
synced 2026-06-28 08:49:38 +02:00
* feat(setup): add Claude Desktop target and MCP-first agent setup Adds `ktx mcp stdio` and a `claude-desktop` setup target that generates a local plugin ZIP wiring the analytics skill and a stdio MCP config. Replaces the CLI-only agent install mode with MCP+analytics (default) and an optional admin CLI skill, renames the research skill to analytics, and lets interactive setup pick project vs global scope when every target supports it. Extracts a shared MCP server factory used by both HTTP and stdio entrypoints. * Add MCP agent client setup support * Polish setup output formatting * Add MCP tool polish design spec Design for slimming the MCP-registered surface from 25 to 11 tools, introducing memory_ingest, applying the per-tool polish kit (annotations, outputSchema, .describe(), in-band error wrapping, union-drift fixes, type-narrowed jsonToolResult), emitting progress notifications on sql_execution + sl_query, and refining the ktx-analytics SKILL.md to match. * Refine MCP tool polish design spec after adversarial review iteration 1 * Refine MCP tool polish design spec after adversarial review iteration 2 * Refine MCP tool polish design spec after adversarial review iteration 3 * refactor(context): rename memory capture service to ingest * feat(mcp): slim research tool surface * refactor(mcp): remove admin ports from server factory * refactor(cli): rename text ingest memory port * docs: update analytics skill for memory ingest * chore: verify mcp surface rename * Add MCP tool polish v1 surface change plan * feat(context): polish mcp tool metadata * fix(context): enforce resolved semantic layer compute sources * feat(context): emit mcp query progress stages * fix(context): keep mcp progress event internal * Add MCP tool polish v1 metadata & progress plan * Fix CI snapshot and docs checks
102 lines
3.4 KiB
TypeScript
102 lines
3.4 KiB
TypeScript
import { Command } from '@commander-js/extra-typings';
|
|
import { describe, expect, it, vi } from 'vitest';
|
|
import type { KtxCliCommandContext } from '../cli-program.js';
|
|
import { registerMcpCommands } from './mcp-commands.js';
|
|
|
|
function makeContext(overrides: Partial<KtxCliCommandContext> = {}): KtxCliCommandContext {
|
|
let exitCode = 0;
|
|
return {
|
|
io: {
|
|
stdout: { write: vi.fn() },
|
|
stderr: { write: vi.fn() },
|
|
},
|
|
deps: {},
|
|
packageInfo: { name: '@ktx/cli', version: '0.0.0-test', contextPackageName: '@ktx/context' },
|
|
setExitCode: (code) => {
|
|
exitCode = code;
|
|
},
|
|
runInit: vi.fn(),
|
|
writeDebug: vi.fn(),
|
|
...overrides,
|
|
get exitCode() {
|
|
return exitCode;
|
|
},
|
|
} as KtxCliCommandContext;
|
|
}
|
|
|
|
describe('registerMcpCommands', () => {
|
|
it('registers the public mcp lifecycle commands', () => {
|
|
const program = new Command().exitOverride();
|
|
registerMcpCommands(program, makeContext());
|
|
const mcp = program.commands.find((command) => command.name() === 'mcp');
|
|
|
|
expect(mcp?.commands.map((command) => command.name()).sort()).toEqual([
|
|
'logs',
|
|
'serve-internal',
|
|
'start',
|
|
'status',
|
|
'stdio',
|
|
'stop',
|
|
]);
|
|
expect(
|
|
(mcp?.commands.find((command) => command.name() === 'serve-internal') as { _hidden?: boolean } | undefined)
|
|
?._hidden,
|
|
).toBe(true);
|
|
});
|
|
|
|
it('rejects non-loopback start without token before spawning', async () => {
|
|
const program = new Command().exitOverride();
|
|
const startDaemon = vi.fn();
|
|
const context = makeContext({ deps: { mcp: { startDaemon } } });
|
|
registerMcpCommands(program, context);
|
|
|
|
await expect(program.parseAsync(['mcp', 'start', '--host', '0.0.0.0'], { from: 'user' })).rejects.toThrow(
|
|
'Binding KTX MCP to 0.0.0.0 requires --token or KTX_MCP_TOKEN',
|
|
);
|
|
expect(startDaemon).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it('prints "already running" when startDaemon reports already-running', async () => {
|
|
const program = new Command().exitOverride().option('--project-dir <path>');
|
|
const startDaemon = vi.fn().mockResolvedValue({
|
|
status: 'already-running',
|
|
url: 'http://127.0.0.1:7878/mcp',
|
|
state: {
|
|
schemaVersion: 1,
|
|
pid: 4242,
|
|
host: '127.0.0.1',
|
|
port: 7878,
|
|
tokenAuth: false,
|
|
projectDir: '/tmp/ktx-already',
|
|
startedAt: '2026-05-14T00:00:00.000Z',
|
|
logPath: '/tmp/ktx-already/.ktx/logs/mcp.log',
|
|
},
|
|
});
|
|
const context = makeContext({ deps: { mcp: { startDaemon } } });
|
|
registerMcpCommands(program, context);
|
|
|
|
await program.parseAsync(['--project-dir', '/tmp/ktx-already', 'mcp', 'start'], { from: 'user' });
|
|
|
|
expect(startDaemon).toHaveBeenCalledTimes(1);
|
|
expect(context.io.stdout.write).toHaveBeenCalledWith(
|
|
'KTX MCP daemon already running: http://127.0.0.1:7878/mcp\n',
|
|
);
|
|
});
|
|
|
|
it('runs the stdio server with the resolved project directory', async () => {
|
|
const program = new Command().exitOverride().option('--project-dir <path>');
|
|
const runStdioServer = vi.fn().mockResolvedValue(undefined);
|
|
const context = makeContext({ deps: { mcp: { runStdioServer } } });
|
|
registerMcpCommands(program, context);
|
|
|
|
await expect(program.parseAsync(['--project-dir', '/tmp/ktx6', 'mcp', 'stdio'], { from: 'user' })).resolves.toBe(
|
|
program,
|
|
);
|
|
|
|
expect(runStdioServer).toHaveBeenCalledWith({
|
|
projectDir: '/tmp/ktx6',
|
|
cliVersion: '0.0.0-test',
|
|
io: context.io,
|
|
});
|
|
});
|
|
});
|