kodelyth-ecc 1.8.0 → 1.8.2

package/tests/mcp/client.test.js

@@ -1,113 +0,0 @@
-// Tests for scripts/mcp/client.js — registry + connect helpers.
-// Uses the LOCAL Kodelyth MCP server itself as a real test target so we
-// exercise the full stdio handshake without needing external dependencies.
-'use strict';
-
-const test   = require('node:test');
-const assert = require('node:assert/strict');
-const fs     = require('fs');
-const os     = require('os');
-const path   = require('path');
-
-// Isolate the registry to a tmp dir so we don't pollute ~/.kodelyth.
-const TMP = fs.mkdtempSync(path.join(os.tmpdir(), 'kodelyth-mcp-client-'));
-process.env.KODELYTH_MCP_CLIENT_DIR = TMP;
-
-const C = require('../../scripts/mcp/client.js');
-
-const ROOT = path.join(__dirname, '..', '..');
-const LOCAL_SERVER = path.join(ROOT, 'scripts', 'mcp', 'server.js');
-
-// ── Registry tests (no SDK required) ─────────────────────────────────────────
-
-test('registry: starts empty', () => {
-  const reg = C.loadRegistry();
-  assert.deepEqual(reg.servers, {});
-  assert.deepEqual(C.listServers(), []);
-});
-
-test('registry: addServer persists with sane defaults', () => {
-  const spec = C.addServer({
-    name: 'self',
-    command: process.execPath,
-    args: [LOCAL_SERVER],
-    env: { TEST_VAR: '1' },
-    description: 'local self-test target',
-  });
-  assert.equal(spec.name, 'self');
-  assert.equal(spec.command, process.execPath);
-  assert.deepEqual(spec.args, [LOCAL_SERVER]);
-  assert.equal(spec.env.TEST_VAR, '1');
-  assert.ok(spec.added_at);
-
-  // Persisted to disk:
-  assert.ok(fs.existsSync(C.REGISTRY_FILE));
-  const onDisk = JSON.parse(fs.readFileSync(C.REGISTRY_FILE, 'utf8'));
-  assert.equal(onDisk.servers.self.name, 'self');
-});
-
-test('registry: rejects bad names', () => {
-  assert.throws(() => C.addServer({ name: '',         command: 'x' }));
-  assert.throws(() => C.addServer({ name: 'has space',command: 'x' }));
-  assert.throws(() => C.addServer({ name: 'self',     command: ''  }));
-});
-
-test('registry: getServer / listServers reflect adds + removes', () => {
-  C.addServer({ name: 'second', command: '/usr/bin/true', args: [] });
-  const list = C.listServers();
-  assert.equal(list.length, 2, `expected 2, got ${list.length}`);
-  const names = list.map(s => s.name).sort();
-  assert.deepEqual(names, ['second', 'self']);
-
-  assert.ok(C.getServer('self'));
-  assert.equal(C.getServer('not-real'), null);
-
-  assert.equal(C.removeServer('second'), true);
-  assert.equal(C.removeServer('second'), false); // already gone
-  assert.equal(C.listServers().length, 1);
-});
-
-// ── End-to-end: connect to local kodelyth MCP server and call tools ──────────
-// These tests require @modelcontextprotocol/sdk (optionalDependency).
-// Skip gracefully when it isn't installed (e.g. CI without optional deps).
-let hasSdk = false;
-try { require('@modelcontextprotocol/sdk/client/index.js'); hasSdk = true; } catch (_) {}
-
-test('client: connects to local kodelyth MCP server and lists tools', { skip: !hasSdk }, async () => {
-  // self is already registered from the earlier test (running in same file).
-  const out = await C.listTools('self');
-  assert.ok(out.tools.length > 0, 'expected at least one tool from local kodelyth server');
-  const names = new Set(out.tools.map(t => t.name));
-  assert.ok(names.has('catalog_stats'), 'catalog_stats tool must be exposed');
-  assert.ok(names.has('list_agents'),   'list_agents tool must be exposed');
-});
-
-test('client: callTool on local server returns a real result', { skip: !hasSdk }, async () => {
-  const out = await C.callTool('self', 'catalog_stats', {});
-  assert.ok(Array.isArray(out.content));
-  assert.equal(out.content[0].type, 'text');
-  const data = JSON.parse(out.content[0].text);
-  assert.ok(data.agents > 0);
-  assert.ok(data.skills > 0);
-  assert.ok(data.commands > 0);
-});
-
-test('client: listResources surfaces kodelyth:// URIs', { skip: !hasSdk }, async () => {
-  const out = await C.listResources('self');
-  assert.ok(out.resources.length > 0);
-  assert.ok(out.resources.every(r => r.uri.startsWith('kodelyth://')),
-    'every resource uri should start with kodelyth://');
-});
-
-test('client: listPrompts surfaces routing-rule', { skip: !hasSdk }, async () => {
-  const out = await C.listPrompts('self');
-  const names = new Set(out.prompts.map(p => p.name));
-  assert.ok(names.has('routing-rule'));
-});
-
-test('client: connect fails clearly for unregistered server', async () => {
-  await assert.rejects(
-    () => C.connect('this-was-never-registered'),
-    err => err.code === 'MCP_SERVER_NOT_REGISTERED'
-  );
-});

package/tests/mcp/resources-prompts.test.js

@@ -1,70 +0,0 @@
-// Tests for scripts/mcp/resources.js and scripts/mcp/prompts.js
-'use strict';
-
-const test   = require('node:test');
-const assert = require('node:assert/strict');
-
-const RES     = require('../../scripts/mcp/resources');
-const PROMPTS = require('../../scripts/mcp/prompts');
-
-test('resources.buildList returns agents/skills/commands/rules/bundles', () => {
-  const list = RES.buildList();
-  assert.ok(list.length > 200, `expected >200 resources, got ${list.length}`);
-  const schemes = new Set(list.map(r => r.uri.split('://')[1].split('/')[0]));
-  for (const expected of ['agents', 'skills', 'commands', 'rules', 'bundles']) {
-    assert.ok(schemes.has(expected), `missing resource bucket: ${expected}`);
-  }
-  for (const r of list.slice(0, 5)) {
-    assert.match(r.uri, /^kodelyth:\/\//);
-    assert.equal(r.mimeType, 'text/markdown');
-  }
-});
-
-test('resources.readResource returns body for a known agent', () => {
-  const out = RES.readResource('kodelyth://agents/debug-detective');
-  assert.ok(out.contents.length === 1);
-  assert.equal(out.contents[0].mimeType, 'text/markdown');
-  assert.match(out.contents[0].text, /debug-detective/);
-});
-
-test('resources.readResource rejects unknown URI scheme', () => {
-  assert.throws(() => RES.readResource('weird://foo/bar'));
-});
-
-test('resources.readResource rejects unknown name', () => {
-  assert.throws(() => RES.readResource('kodelyth://agents/no-such-agent'));
-});
-
-test('PROMPT_DEFINITIONS exposes routing-rule and overviews', () => {
-  const names = PROMPTS.PROMPT_DEFINITIONS.map(p => p.name);
-  for (const expected of [
-    'routing-rule',
-    'agents-overview',
-    'skills-overview',
-    'commands-overview',
-    'handoff-chains',
-    'devil-mode',
-  ]) {
-    assert.ok(names.includes(expected), `missing prompt: ${expected}`);
-  }
-});
-
-test('routing-rule prompt returns the full intent routing rule body', () => {
-  const out = PROMPTS.PROMPT_HANDLERS['routing-rule']();
-  assert.ok(out.messages.length === 1);
-  assert.equal(out.messages[0].role, 'user');
-  assert.match(out.messages[0].content.text, /Agent Intent Routing/);
-});
-
-test('agents-overview returns a summary of all agents', () => {
-  const out = PROMPTS.PROMPT_HANDLERS['agents-overview']();
-  const text = out.messages[0].content.text;
-  assert.match(text, /Kodelyth ECC — Agents/);
-  assert.match(text, /debug-detective/);
-});
-
-test('devil-mode prompt returns the parallel command body', () => {
-  const out = PROMPTS.PROMPT_HANDLERS['devil-mode']();
-  const text = out.messages[0].content.text;
-  assert.match(text, /devil-mode/i);
-});

package/tests/mcp/tools.test.js

@@ -1,159 +0,0 @@
-// Tests for scripts/mcp/tools.js — pure-function MCP tool handlers.
-'use strict';
-
-const test   = require('node:test');
-const assert = require('node:assert/strict');
-const fs     = require('fs');
-const os     = require('os');
-const path   = require('path');
-
-// Isolate the memory store to a tmp dir so capture/recall don't pollute
-// the user's real ~/.kodelyth/memory.
-const TMP = fs.mkdtempSync(path.join(os.tmpdir(), 'kodelyth-mcp-'));
-process.env.KODELYTH_MEMORY_DIR = TMP;
-
-// Load AFTER setting env so the memory store picks up the tmp dir.
-const T = require('../../scripts/mcp/tools');
-
-function parseJSON(toolResult) {
-  const text = toolResult.content[0].text;
-  return JSON.parse(text);
-}
-
-test('TOOL_DEFINITIONS exposes the expected tool surface', () => {
-  const names = T.TOOL_DEFINITIONS.map(t => t.name);
-  for (const expected of [
-    'route_intent',
-    'list_agents',
-    'list_skills',
-    'list_commands',
-    'list_rules',
-    'list_bundles',
-    'get_agent',
-    'get_skill',
-    'get_command',
-    'get_rule',
-    'get_bundle',
-    'recall_memory',
-    'capture_memory',
-    'memory_stats',
-    'catalog_stats',
-    'audit_skill_match',
-  ]) {
-    assert.ok(names.includes(expected), `missing tool: ${expected}`);
-  }
-  // Every tool has a handler entry.
-  for (const t of T.TOOL_DEFINITIONS) {
-    assert.equal(typeof T.TOOL_HANDLERS[t.name], 'function', `${t.name} handler missing`);
-    assert.ok(t.description && t.description.length > 5, `${t.name} description too short`);
-    assert.ok(t.inputSchema && t.inputSchema.type === 'object', `${t.name} inputSchema malformed`);
-  }
-});
-
-test('route_intent rejects empty message', () => {
-  const r = T.tool_route_intent({});
-  assert.ok(r.isError, 'must error on missing message');
-});
-
-test('route_intent surfaces a relevant agent for a debug task', () => {
-  const r = T.tool_route_intent({ message: 'i have a hard intermittent bug in production' });
-  const data = parseJSON(r);
-  assert.ok(Array.isArray(data.suggestions));
-  assert.ok(data.suggestions.length > 0, 'should produce at least one suggestion');
-});
-
-test('route_intent honours top_k', () => {
-  const r = T.tool_route_intent({ message: 'review this code for security and quality issues', top_k: 2 });
-  const data = parseJSON(r);
-  assert.ok(data.suggestions.length <= 2);
-});
-
-test('list_agents returns the full catalog', () => {
-  const r = T.tool_list_agents();
-  const data = parseJSON(r);
-  assert.ok(data.count >= 60);
-  assert.ok(Array.isArray(data.agents) && data.agents.length === data.count);
-});
-
-test('get_agent returns body for known agent', () => {
-  const r = T.tool_get_agent({ name: 'debug-detective' });
-  assert.ok(!r.isError);
-  assert.match(r.content[0].text, /^# Agent: debug-detective/);
-});
-
-test('get_agent errors on unknown agent', () => {
-  const r = T.tool_get_agent({ name: 'no-such-agent' });
-  assert.ok(r.isError);
-});
-
-test('get_command tolerates leading slash', () => {
-  const r = T.tool_get_command({ name: '/code-review' });
-  assert.ok(!r.isError);
-  assert.match(r.content[0].text, /^# Command: \/code-review/);
-});
-
-test('get_rule returns the intent routing rule', () => {
-  const r = T.tool_get_rule({ name: 'agent-intent-routing' });
-  assert.ok(!r.isError);
-  assert.match(r.content[0].text, /Agent Intent Routing/);
-});
-
-test('get_bundle returns red-team cheat sheet', () => {
-  const r = T.tool_get_bundle({ name: 'red-team' });
-  assert.ok(!r.isError);
-  assert.ok(r.content[0].text.length > 100);
-});
-
-test('catalog_stats returns counts', () => {
-  const r = T.tool_catalog_stats();
-  const s = parseJSON(r);
-  assert.ok(s.agents > 0 && s.skills > 0 && s.commands > 0);
-});
-
-test('audit_skill_match suggests at least one skill for an observability task', () => {
-  const r = T.tool_audit_skill_match({ task: 'add structured logging and tracing to a fastapi service' });
-  const data = parseJSON(r);
-  assert.ok(Array.isArray(data.suggestions));
-  assert.ok(data.suggestions.length > 0, 'expected at least one skill suggestion');
-});
-
-test('capture_memory + recall_memory roundtrip in isolated tmp store', () => {
-  // Pre-seed a few unrelated memories so BM25 IDF can rise above the store's
-  // default minScore floor (with N=1 IDF stays ~0.29 which is filtered out).
-  for (const seed of [
-    { problem: 'fix a flaky CI build on github actions',  approach: 'set timeout to 60s and retry once' },
-    { problem: 'wire up redis caching for product list',  approach: 'use 60s ttl plus stale-while-revalidate' },
-    { problem: 'migrate postgres timestamps to utc',       approach: 'add at::timestamptz coercion in alembic migration' },
-  ]) {
-    T.tool_capture_memory({ ...seed, tags: ['seed'], language: 'misc' });
-  }
-
-  const cap = T.tool_capture_memory({
-    problem:  'mcp test problem keyword splatzilla',
-    approach: 'documenting that capture_memory works end-to-end',
-    tags:     ['mcp', 'test'],
-    language: 'javascript',
-  });
-  assert.ok(!cap.isError, 'capture must succeed');
-  const capData = parseJSON(cap);
-  assert.ok(capData.id && typeof capData.id === 'string');
-  assert.ok(capData.captured_at);
-
-  const rec = T.tool_recall_memory({ query: 'splatzilla', limit: 3 });
-  assert.ok(!rec.isError);
-  const recData = parseJSON(rec);
-  assert.ok(recData.count >= 1, 'should recall the memory we just captured');
-  assert.equal(recData.memories[0].problem, 'mcp test problem keyword splatzilla');
-});
-
-test('capture_memory rejects missing required fields', () => {
-  const r = T.tool_capture_memory({ problem: 'only problem, no approach' });
-  assert.ok(r.isError);
-});
-
-test('memory_stats reflects at least one stored memory', () => {
-  const r = T.tool_memory_stats();
-  assert.ok(!r.isError);
-  const s = parseJSON(r);
-  assert.ok(typeof s === 'object');
-});

package/tests/memory/auto-recall.test.js

@@ -1,132 +0,0 @@
-// Tests for hooks/memory/auto-recall.js — UserPromptSubmit auto chat detection
-'use strict';
-
-const test = require('node:test');
-const assert = require('node:assert/strict');
-const fs   = require('fs');
-const os   = require('os');
-const path = require('path');
-const { spawnSync } = require('child_process');
-
-const TMP = fs.mkdtempSync(path.join(os.tmpdir(), 'kodelyth-recall-'));
-process.env.KODELYTH_MEMORY_DIR = TMP;
-
-const HOOK = path.join(__dirname, '..', '..', 'hooks', 'memory', 'auto-recall.js');
-const store = require('../../scripts/memory/store');
-
-function runHook(payload) {
-  const res = spawnSync(process.execPath, [HOOK], {
-    input:    JSON.stringify(payload),
-    encoding: 'utf8',
-    env:      { ...process.env, KODELYTH_MEMORY_DIR: TMP },
-    timeout:  5000,
-  });
-  return { stdout: res.stdout || '', stderr: res.stderr || '', status: res.status };
-}
-
-// Seed memory once
-store.capture({
-  problem:  'Stripe webhook signature failed in production',
-  approach: 'Switched body parser from json to raw, validated with constructEvent',
-  tags:     ['payments', 'stripe', 'webhooks'],
-  project:  '/test/payments',
-  language: 'typescript',
-});
-store.capture({
-  problem:  'Redis cluster split-brain after AZ failover',
-  approach: 'Configured min-slaves-to-write to 2 to prevent stale primary writes',
-  tags:     ['redis', 'cluster', 'reliability'],
-  project:  '/test/payments',
-  language: 'typescript',
-});
-
-test('auto-recall: returns nothing for empty prompt', () => {
-  const r = runHook({ prompt: '', cwd: '/test/payments', session_id: 's1' });
-  assert.equal(r.status, 0);
-  assert.equal(r.stdout, '');
-});
-
-test('auto-recall: skips trivial prompts', () => {
-  for (const trivial of ['ok', 'yes', 'thanks', 'cool']) {
-    const r = runHook({ prompt: trivial, cwd: '/test/payments', session_id: 's-trivial' });
-    assert.equal(r.stdout, '', `should skip "${trivial}"`);
-  }
-});
-
-test('auto-recall: skips agent invocation prompts', () => {
-  for (const cmd of ['use debug-detective', '@code-reviewer', '/help', 'invoke security-reviewer']) {
-    const r = runHook({ prompt: cmd, cwd: '/test/payments', session_id: 's-cmd' });
-    assert.equal(r.stdout, '', `should skip "${cmd}"`);
-  }
-});
-
-test('auto-recall: skips short prompts', () => {
-  const r = runHook({ prompt: 'fix bug', cwd: '/test/payments', session_id: 's-short' });
-  assert.equal(r.stdout, '');
-});
-
-test('auto-recall: surfaces relevant memory for a real query', () => {
-  const r = runHook({
-    prompt: 'I need to add Stripe webhooks for subscription renewal events',
-    cwd:    '/test/payments',
-    session_id: 's-fire-1',
-  });
-  assert.equal(r.status, 0);
-  assert.ok(r.stdout.length > 0, 'should produce output');
-  const out = JSON.parse(r.stdout);
-  assert.ok(out.additionalContext.includes('Stripe webhook'));
-  assert.equal(out.meta.source, 'kodelyth-memory:auto-recall');
-  assert.equal(out.meta.recalledCount, 1);
-});
-
-test('auto-recall: suppresses repeat in same session', () => {
-  // First call surfaces the memory
-  runHook({
-    prompt: 'help with stripe webhook integration',
-    cwd:    '/test/payments',
-    session_id: 's-repeat',
-  });
-  // Second call same topic — should NOT surface again
-  const r2 = runHook({
-    prompt: 'about that stripe webhook problem, what about retries',
-    cwd:    '/test/payments',
-    session_id: 's-repeat',
-  });
-  assert.equal(r2.stdout, '', 'repeat should be suppressed');
-});
-
-test('auto-recall: surfaces different memory for different topic in same session', () => {
-  runHook({
-    prompt: 'how do I handle stripe webhook errors',
-    cwd:    '/test/payments',
-    session_id: 's-multi',
-  });
-  // Different topic — redis — should surface
-  const r2 = runHook({
-    prompt: 'redis cluster is acting weird after failover today',
-    cwd:    '/test/payments',
-    session_id: 's-multi',
-  });
-  assert.ok(r2.stdout.length > 0, 'different topic should surface');
-  const out = JSON.parse(r2.stdout);
-  assert.ok(out.additionalContext.toLowerCase().includes('redis'));
-});
-
-test('auto-recall: returns nothing when no memory matches', () => {
-  const r = runHook({
-    prompt: 'what is the meaning of life and quantum entanglement',
-    cwd:    '/test/payments',
-    session_id: 's-nomatch',
-  });
-  assert.equal(r.stdout, '');
-});
-
-test('auto-recall: never crashes on malformed payload', () => {
-  const res = spawnSync(process.execPath, [HOOK], {
-    input:    'not-json-at-all',
-    encoding: 'utf8',
-    env:      { ...process.env, KODELYTH_MEMORY_DIR: TMP },
-    timeout:  3000,
-  });
-  assert.equal(res.status, 0, 'must exit 0 even on garbage input');
-});