clawvault 2.5.4 → 2.6.1

package/bin/register-config-commands.js

@@ -1,153 +1,153 @@
-/**
- * Runtime config command registrations backed by .clawvault.json.
- */
-
-function stringifyValue(value) {
-  if (Array.isArray(value)) {
-    if (value.every((item) => typeof item === 'string' || typeof item === 'number' || typeof item === 'boolean')) {
-      return value.join(', ');
-    }
-    return JSON.stringify(value);
-  }
-  if (value && typeof value === 'object') {
-    return JSON.stringify(value);
-  }
-  if (value === null || value === undefined) {
-    return '(unset)';
-  }
-  return String(value);
-}
-
-function flattenConfig(value, prefix = '') {
-  if (!value || typeof value !== 'object' || Array.isArray(value)) {
-    return [{ key: prefix || '(root)', value: stringifyValue(value) }];
-  }
-
-  const rows = [];
-  const keys = Object.keys(value).sort((left, right) => left.localeCompare(right));
-  for (const key of keys) {
-    const next = prefix ? `${prefix}.${key}` : key;
-    const entry = value[key];
-    if (entry && typeof entry === 'object' && !Array.isArray(entry)) {
-      rows.push(...flattenConfig(entry, next));
-      continue;
-    }
-    rows.push({ key: next, value: stringifyValue(entry) });
-  }
-  return rows;
-}
-
-function printTable(rows) {
-  if (rows.length === 0) {
-    console.log('No config values found.');
-    return;
-  }
-  const keyWidth = Math.max(
-    'KEY'.length,
-    ...rows.map((row) => row.key.length)
-  );
-  const valueWidth = Math.max(
-    'VALUE'.length,
-    ...rows.map((row) => row.value.length)
-  );
-  const header = `${'KEY'.padEnd(keyWidth)}  ${'VALUE'.padEnd(valueWidth)}`;
-  const divider = `${'-'.repeat(keyWidth)}  ${'-'.repeat(valueWidth)}`;
-  console.log(header);
-  console.log(divider);
-  for (const row of rows) {
-    console.log(`${row.key.padEnd(keyWidth)}  ${row.value}`);
-  }
-}
-
-function normalizeConfigKey(key) {
-  return String(key || '').trim();
-}
-
-export function registerConfigCommands(program, { chalk, resolveVaultPath }) {
-  const config = program
-    .command('config')
-    .description('Read and modify runtime configuration');
-
-  config
-    .command('get <key>')
-    .description('Read a runtime config value (dot-notation supported)')
-    .option('-v, --vault <path>', 'Vault path')
-    .action(async (key, options) => {
-      try {
-        const { getConfigValue, SUPPORTED_CONFIG_KEYS } = await import('../dist/index.js');
-        const normalizedKey = normalizeConfigKey(key);
-        if (!SUPPORTED_CONFIG_KEYS.includes(normalizedKey)) {
-          throw new Error(`Unsupported config key: ${normalizedKey}`);
-        }
-
-        const value = getConfigValue(resolveVaultPath(options.vault), normalizedKey);
-        if (Array.isArray(value)) {
-          console.log(value.join(','));
-          return;
-        }
-        if (value && typeof value === 'object') {
-          console.log(JSON.stringify(value, null, 2));
-          return;
-        }
-        console.log(value === undefined || value === null ? '' : String(value));
-      } catch (err) {
-        console.error(chalk.red(`Error: ${err.message}`));
-        process.exit(1);
-      }
-    });
-
-  config
-    .command('set <key> <value>')
-    .description('Set a runtime config value')
-    .option('-v, --vault <path>', 'Vault path')
-    .action(async (key, value, options) => {
-      try {
-        const { setConfigValue, SUPPORTED_CONFIG_KEYS } = await import('../dist/index.js');
-        const normalizedKey = normalizeConfigKey(key);
-        if (!SUPPORTED_CONFIG_KEYS.includes(normalizedKey)) {
-          throw new Error(`Unsupported config key: ${normalizedKey}`);
-        }
-        const result = setConfigValue(resolveVaultPath(options.vault), normalizedKey, value);
-        console.log(chalk.green(`✓ Updated ${normalizedKey}`));
-        console.log(chalk.dim(`  Value: ${stringifyValue(result.value)}`));
-      } catch (err) {
-        console.error(chalk.red(`Error: ${err.message}`));
-        process.exit(1);
-      }
-    });
-
-  config
-    .command('list')
-    .description('List all config values')
-    .option('-v, --vault <path>', 'Vault path')
-    .action(async (options) => {
-      try {
-        const { listConfig } = await import('../dist/index.js');
-        const values = listConfig(resolveVaultPath(options.vault));
-        const rows = flattenConfig(values);
-        printTable(rows);
-      } catch (err) {
-        console.error(chalk.red(`Error: ${err.message}`));
-        process.exit(1);
-      }
-    });
-
-  config
-    .command('reset')
-    .description('Reset runtime config values to defaults')
-    .option('--confirm', 'Confirm reset (required)')
-    .option('-v, --vault <path>', 'Vault path')
-    .action(async (options) => {
-      try {
-        if (!options.confirm) {
-          throw new Error('Refusing to reset config without --confirm.');
-        }
-        const { resetConfig } = await import('../dist/index.js');
-        resetConfig(resolveVaultPath(options.vault));
-        console.log(chalk.green('✓ Config reset to defaults'));
-      } catch (err) {
-        console.error(chalk.red(`Error: ${err.message}`));
-        process.exit(1);
-      }
-    });
-}
+/**
+ * Runtime config command registrations backed by .clawvault.json.
+ */
+
+function stringifyValue(value) {
+  if (Array.isArray(value)) {
+    if (value.every((item) => typeof item === 'string' || typeof item === 'number' || typeof item === 'boolean')) {
+      return value.join(', ');
+    }
+    return JSON.stringify(value);
+  }
+  if (value && typeof value === 'object') {
+    return JSON.stringify(value);
+  }
+  if (value === null || value === undefined) {
+    return '(unset)';
+  }
+  return String(value);
+}
+
+function flattenConfig(value, prefix = '') {
+  if (!value || typeof value !== 'object' || Array.isArray(value)) {
+    return [{ key: prefix || '(root)', value: stringifyValue(value) }];
+  }
+
+  const rows = [];
+  const keys = Object.keys(value).sort((left, right) => left.localeCompare(right));
+  for (const key of keys) {
+    const next = prefix ? `${prefix}.${key}` : key;
+    const entry = value[key];
+    if (entry && typeof entry === 'object' && !Array.isArray(entry)) {
+      rows.push(...flattenConfig(entry, next));
+      continue;
+    }
+    rows.push({ key: next, value: stringifyValue(entry) });
+  }
+  return rows;
+}
+
+function printTable(rows) {
+  if (rows.length === 0) {
+    console.log('No config values found.');
+    return;
+  }
+  const keyWidth = Math.max(
+    'KEY'.length,
+    ...rows.map((row) => row.key.length)
+  );
+  const valueWidth = Math.max(
+    'VALUE'.length,
+    ...rows.map((row) => row.value.length)
+  );
+  const header = `${'KEY'.padEnd(keyWidth)}  ${'VALUE'.padEnd(valueWidth)}`;
+  const divider = `${'-'.repeat(keyWidth)}  ${'-'.repeat(valueWidth)}`;
+  console.log(header);
+  console.log(divider);
+  for (const row of rows) {
+    console.log(`${row.key.padEnd(keyWidth)}  ${row.value}`);
+  }
+}
+
+function normalizeConfigKey(key) {
+  return String(key || '').trim();
+}
+
+export function registerConfigCommands(program, { chalk, resolveVaultPath }) {
+  const config = program
+    .command('config')
+    .description('Read and modify runtime configuration');
+
+  config
+    .command('get <key>')
+    .description('Read a runtime config value (dot-notation supported)')
+    .option('-v, --vault <path>', 'Vault path')
+    .action(async (key, options) => {
+      try {
+        const { getConfigValue, SUPPORTED_CONFIG_KEYS } = await import('../dist/index.js');
+        const normalizedKey = normalizeConfigKey(key);
+        if (!SUPPORTED_CONFIG_KEYS.includes(normalizedKey)) {
+          throw new Error(`Unsupported config key: ${normalizedKey}`);
+        }
+
+        const value = getConfigValue(resolveVaultPath(options.vault), normalizedKey);
+        if (Array.isArray(value)) {
+          console.log(value.join(','));
+          return;
+        }
+        if (value && typeof value === 'object') {
+          console.log(JSON.stringify(value, null, 2));
+          return;
+        }
+        console.log(value === undefined || value === null ? '' : String(value));
+      } catch (err) {
+        console.error(chalk.red(`Error: ${err.message}`));
+        process.exit(1);
+      }
+    });
+
+  config
+    .command('set <key> <value>')
+    .description('Set a runtime config value')
+    .option('-v, --vault <path>', 'Vault path')
+    .action(async (key, value, options) => {
+      try {
+        const { setConfigValue, SUPPORTED_CONFIG_KEYS } = await import('../dist/index.js');
+        const normalizedKey = normalizeConfigKey(key);
+        if (!SUPPORTED_CONFIG_KEYS.includes(normalizedKey)) {
+          throw new Error(`Unsupported config key: ${normalizedKey}`);
+        }
+        const result = setConfigValue(resolveVaultPath(options.vault), normalizedKey, value);
+        console.log(chalk.green(`✓ Updated ${normalizedKey}`));
+        console.log(chalk.dim(`  Value: ${stringifyValue(result.value)}`));
+      } catch (err) {
+        console.error(chalk.red(`Error: ${err.message}`));
+        process.exit(1);
+      }
+    });
+
+  config
+    .command('list')
+    .description('List all config values')
+    .option('-v, --vault <path>', 'Vault path')
+    .action(async (options) => {
+      try {
+        const { listConfig } = await import('../dist/index.js');
+        const values = listConfig(resolveVaultPath(options.vault));
+        const rows = flattenConfig(values);
+        printTable(rows);
+      } catch (err) {
+        console.error(chalk.red(`Error: ${err.message}`));
+        process.exit(1);
+      }
+    });
+
+  config
+    .command('reset')
+    .description('Reset runtime config values to defaults')
+    .option('--confirm', 'Confirm reset (required)')
+    .option('-v, --vault <path>', 'Vault path')
+    .action(async (options) => {
+      try {
+        if (!options.confirm) {
+          throw new Error('Refusing to reset config without --confirm.');
+        }
+        const { resetConfig } = await import('../dist/index.js');
+        resetConfig(resolveVaultPath(options.vault));
+        console.log(chalk.green('✓ Config reset to defaults'));
+      } catch (err) {
+        console.error(chalk.red(`Error: ${err.message}`));
+        process.exit(1);
+      }
+    });
+}

package/bin/register-config-route-commands.test.js

@@ -1,121 +1,121 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-import { Command } from 'commander';
-import { registerConfigCommands } from './register-config-commands.js';
-import { registerRouteCommands } from './register-route-commands.js';
-import { chalkStub } from './test-helpers/cli-command-fixtures.js';
-
-const {
-  getConfigValueMock,
-  setConfigValueMock,
-  listConfigMock,
-  resetConfigMock,
-  addRouteRuleMock,
-  listRouteRulesMock,
-  removeRouteRuleMock,
-  testRouteRuleMock
-} = vi.hoisted(() => ({
-  getConfigValueMock: vi.fn(),
-  setConfigValueMock: vi.fn(),
-  listConfigMock: vi.fn(),
-  resetConfigMock: vi.fn(),
-  addRouteRuleMock: vi.fn(),
-  listRouteRulesMock: vi.fn(),
-  removeRouteRuleMock: vi.fn(),
-  testRouteRuleMock: vi.fn()
-}));
-
-vi.mock('../dist/index.js', () => ({
-  SUPPORTED_CONFIG_KEYS: [
-    'name',
-    'categories',
-    'theme',
-    'observe.model',
-    'observe.provider',
-    'observer.compression.provider',
-    'observer.compression.model',
-    'observer.compression.baseUrl',
-    'observer.compression.apiKey',
-    'context.maxResults',
-    'context.defaultProfile',
-    'graph.maxHops',
-    'inject.maxResults',
-    'inject.useLlm',
-    'inject.scope'
-  ],
-  getConfigValue: getConfigValueMock,
-  setConfigValue: setConfigValueMock,
-  listConfig: listConfigMock,
-  resetConfig: resetConfigMock,
-  addRouteRule: addRouteRuleMock,
-  listRouteRules: listRouteRulesMock,
-  removeRouteRule: removeRouteRuleMock,
-  testRouteRule: testRouteRuleMock
-}));
-
-function buildProgram() {
-  const program = new Command();
-  const resolveVaultPath = (value) => value ?? '/vault';
-  registerConfigCommands(program, { chalk: chalkStub, resolveVaultPath });
-  registerRouteCommands(program, { chalk: chalkStub, resolveVaultPath });
-  return program;
-}
-
-async function runCommand(args) {
-  const program = buildProgram();
-  await program.parseAsync(args, { from: 'user' });
-}
-
-beforeEach(() => {
-  vi.clearAllMocks();
-  getConfigValueMock.mockReturnValue('demo-vault');
-  setConfigValueMock.mockReturnValue({ value: 'demo-vault' });
-  listConfigMock.mockReturnValue({ name: 'demo-vault', context: { maxResults: 5 } });
-  resetConfigMock.mockReturnValue(undefined);
-  addRouteRuleMock.mockReturnValue({ pattern: 'Pedro', target: 'people/pedro', priority: 1 });
-  listRouteRulesMock.mockReturnValue([{ pattern: 'Pedro', target: 'people/pedro', priority: 1 }]);
-  removeRouteRuleMock.mockReturnValue(true);
-  testRouteRuleMock.mockReturnValue({ pattern: 'Pedro', target: 'people/pedro', priority: 1 });
-});
-
-describe('config and route command registrations', () => {
-  it('executes config get/set/list subcommands', async () => {
-    const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
-    try {
-      await runCommand(['config', 'get', 'name']);
-      expect(getConfigValueMock).toHaveBeenCalledWith('/vault', 'name');
-
-      await runCommand(['config', 'set', 'categories', 'people,projects']);
-      expect(setConfigValueMock).toHaveBeenCalledWith('/vault', 'categories', 'people,projects');
-
-      await runCommand(['config', 'list']);
-      expect(listConfigMock).toHaveBeenCalledWith('/vault');
-
-      const output = logSpy.mock.calls.map((call) => call.join(' ')).join('\n');
-      expect(output).toContain('context.maxResults');
-    } finally {
-      logSpy.mockRestore();
-    }
-  });
-
-  it('executes route add/remove/test subcommands', async () => {
-    const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
-    try {
-      await runCommand(['route', 'add', 'Pedro', 'people/pedro']);
-      expect(addRouteRuleMock).toHaveBeenCalledWith('/vault', 'Pedro', 'people/pedro');
-
-      await runCommand(['route', 'remove', 'Pedro']);
-      expect(removeRouteRuleMock).toHaveBeenCalledWith('/vault', 'Pedro');
-
-      await runCommand(['route', 'test', 'Talked to Pedro']);
-      expect(testRouteRuleMock).toHaveBeenCalledWith('/vault', 'Talked to Pedro');
-
-      await runCommand(['route', 'list']);
-      expect(listRouteRulesMock).toHaveBeenCalledWith('/vault');
-
-      const output = logSpy.mock.calls.map((call) => call.join(' ')).join('\n');
-      expect(output).toContain('Route matched');
-    } finally {
-      logSpy.mockRestore();
-    }
-  });
-});
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { Command } from 'commander';
+import { registerConfigCommands } from './register-config-commands.js';
+import { registerRouteCommands } from './register-route-commands.js';
+import { chalkStub } from './test-helpers/cli-command-fixtures.js';
+
+const {
+  getConfigValueMock,
+  setConfigValueMock,
+  listConfigMock,
+  resetConfigMock,
+  addRouteRuleMock,
+  listRouteRulesMock,
+  removeRouteRuleMock,
+  testRouteRuleMock
+} = vi.hoisted(() => ({
+  getConfigValueMock: vi.fn(),
+  setConfigValueMock: vi.fn(),
+  listConfigMock: vi.fn(),
+  resetConfigMock: vi.fn(),
+  addRouteRuleMock: vi.fn(),
+  listRouteRulesMock: vi.fn(),
+  removeRouteRuleMock: vi.fn(),
+  testRouteRuleMock: vi.fn()
+}));
+
+vi.mock('../dist/index.js', () => ({
+  SUPPORTED_CONFIG_KEYS: [
+    'name',
+    'categories',
+    'theme',
+    'observe.model',
+    'observe.provider',
+    'observer.compression.provider',
+    'observer.compression.model',
+    'observer.compression.baseUrl',
+    'observer.compression.apiKey',
+    'context.maxResults',
+    'context.defaultProfile',
+    'graph.maxHops',
+    'inject.maxResults',
+    'inject.useLlm',
+    'inject.scope'
+  ],
+  getConfigValue: getConfigValueMock,
+  setConfigValue: setConfigValueMock,
+  listConfig: listConfigMock,
+  resetConfig: resetConfigMock,
+  addRouteRule: addRouteRuleMock,
+  listRouteRules: listRouteRulesMock,
+  removeRouteRule: removeRouteRuleMock,
+  testRouteRule: testRouteRuleMock
+}));
+
+function buildProgram() {
+  const program = new Command();
+  const resolveVaultPath = (value) => value ?? '/vault';
+  registerConfigCommands(program, { chalk: chalkStub, resolveVaultPath });
+  registerRouteCommands(program, { chalk: chalkStub, resolveVaultPath });
+  return program;
+}
+
+async function runCommand(args) {
+  const program = buildProgram();
+  await program.parseAsync(args, { from: 'user' });
+}
+
+beforeEach(() => {
+  vi.clearAllMocks();
+  getConfigValueMock.mockReturnValue('demo-vault');
+  setConfigValueMock.mockReturnValue({ value: 'demo-vault' });
+  listConfigMock.mockReturnValue({ name: 'demo-vault', context: { maxResults: 5 } });
+  resetConfigMock.mockReturnValue(undefined);
+  addRouteRuleMock.mockReturnValue({ pattern: 'Pedro', target: 'people/pedro', priority: 1 });
+  listRouteRulesMock.mockReturnValue([{ pattern: 'Pedro', target: 'people/pedro', priority: 1 }]);
+  removeRouteRuleMock.mockReturnValue(true);
+  testRouteRuleMock.mockReturnValue({ pattern: 'Pedro', target: 'people/pedro', priority: 1 });
+});
+
+describe('config and route command registrations', () => {
+  it('executes config get/set/list subcommands', async () => {
+    const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+    try {
+      await runCommand(['config', 'get', 'name']);
+      expect(getConfigValueMock).toHaveBeenCalledWith('/vault', 'name');
+
+      await runCommand(['config', 'set', 'categories', 'people,projects']);
+      expect(setConfigValueMock).toHaveBeenCalledWith('/vault', 'categories', 'people,projects');
+
+      await runCommand(['config', 'list']);
+      expect(listConfigMock).toHaveBeenCalledWith('/vault');
+
+      const output = logSpy.mock.calls.map((call) => call.join(' ')).join('\n');
+      expect(output).toContain('context.maxResults');
+    } finally {
+      logSpy.mockRestore();
+    }
+  });
+
+  it('executes route add/remove/test subcommands', async () => {
+    const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+    try {
+      await runCommand(['route', 'add', 'Pedro', 'people/pedro']);
+      expect(addRouteRuleMock).toHaveBeenCalledWith('/vault', 'Pedro', 'people/pedro');
+
+      await runCommand(['route', 'remove', 'Pedro']);
+      expect(removeRouteRuleMock).toHaveBeenCalledWith('/vault', 'Pedro');
+
+      await runCommand(['route', 'test', 'Talked to Pedro']);
+      expect(testRouteRuleMock).toHaveBeenCalledWith('/vault', 'Talked to Pedro');
+
+      await runCommand(['route', 'list']);
+      expect(listRouteRulesMock).toHaveBeenCalledWith('/vault');
+
+      const output = logSpy.mock.calls.map((call) => call.join(' ')).join('\n');
+      expect(output).toContain('Route matched');
+    } finally {
+      logSpy.mockRestore();
+    }
+  });
+});