@myclaw163/clawclaw-cli 0.6.58 → 0.6.60

package/src/commands/setup/hermes.test.ts

@@ -1,96 +1,96 @@
-import { describe, expect, it, vi, beforeEach } from 'vitest';
-import { runHermesSetup } from './hermes.js';
-import { spawnSync } from 'child_process';
-
-vi.mock('child_process', () => ({
-  spawnSync: vi.fn(),
-}));
-
-const mockedSpawnSync = vi.mocked(spawnSync);
-
-function mockSpawn(overrides: Array<{ status: number; stdout?: string; stderr?: string }>) {
-  mockedSpawnSync.mockImplementation((_cmd: any, _args?: any, _opts?: any): any => {
-    const next = overrides.shift();
-    if (!next) throw new Error('Unexpected spawnSync call');
-    return {
-      status: next.status,
-      stdout: next.stdout ?? '',
-      stderr: next.stderr ?? '',
-    };
-  });
-}
-
-beforeEach(() => {
-  vi.clearAllMocks();
-});
-
-describe('runHermesSetup', () => {
-  it('returns error when hermes CLI is not found', () => {
-    mockSpawn([{ status: 1, stderr: 'command not found' }]);
-    const r = runHermesSetup({});
-    expect(r.exitCode).toBe(1);
-    expect(r.output.some((l) => l.includes('not found'))).toBe(true);
-  });
-
-  it('reports already-enabled when clawclaw is in plugins list', () => {
-    mockSpawn([
-      { status: 0 },                                  // --version
-      { status: 0, stdout: 'clawclaw\nother-plugin' }, // plugins list
-    ]);
-    const r = runHermesSetup({});
-    expect(r.exitCode).toBe(0);
-    expect(r.output.some((l) => l.includes('already enabled'))).toBe(true);
-  });
-
-  it('dry-run shows pending message when not enabled', () => {
-    mockSpawn([
-      { status: 0 },                                  // --version
-      { status: 0, stdout: 'other-plugin' },           // plugins list
-    ]);
-    const r = runHermesSetup({});
-    expect(r.exitCode).toBe(2);
-    expect(r.output.some((l) => l.includes('Dry-run'))).toBe(true);
-  });
-
-  it('--print shows the command that would run', () => {
-    mockSpawn([
-      { status: 0 },                                  // --version
-      { status: 0, stdout: 'other-plugin' },           // plugins list
-    ]);
-    const r = runHermesSetup({ print: true });
-    expect(r.exitCode).toBe(0);
-    expect(r.output.some((l) => l.includes('hermes plugins enable clawclaw'))).toBe(true);
-  });
-
-  it('-y runs hermes plugins enable and succeeds', () => {
-    mockSpawn([
-      { status: 0 },                                  // --version
-      { status: 0, stdout: 'other-plugin' },           // plugins list
-      { status: 0, stdout: 'plugin enabled' },         // plugins enable
-    ]);
-    const r = runHermesSetup({ yes: true });
-    expect(r.exitCode).toBe(0);
-    expect(r.output.some((l) => l.includes('enabled'))).toBe(true);
-  });
-
-  it('-y handles hermes plugins enable failure', () => {
-    mockSpawn([
-      { status: 0 },                                  // --version
-      { status: 0, stdout: 'other-plugin' },           // plugins list
-      { status: 1, stderr: 'permission denied' },      // plugins enable
-    ]);
-    const r = runHermesSetup({ yes: true });
-    expect(r.exitCode).toBe(1);
-    expect(r.output.some((l) => l.includes('Failed'))).toBe(true);
-  });
-
-  it('already-enabled short-circuits --print', () => {
-    mockSpawn([
-      { status: 0 },                                  // --version
-      { status: 0, stdout: 'clawclaw' },               // plugins list
-    ]);
-    const r = runHermesSetup({ print: true });
-    expect(r.exitCode).toBe(0);
-    expect(r.output.some((l) => l.includes('already enabled'))).toBe(true);
-  });
-});
+import { describe, expect, it, vi, beforeEach } from 'vitest';
+import { runHermesSetup } from './hermes.js';
+import { spawnSync } from 'child_process';
+
+vi.mock('child_process', () => ({
+  spawnSync: vi.fn(),
+}));
+
+const mockedSpawnSync = vi.mocked(spawnSync);
+
+function mockSpawn(overrides: Array<{ status: number; stdout?: string; stderr?: string }>) {
+  mockedSpawnSync.mockImplementation((_cmd: any, _args?: any, _opts?: any): any => {
+    const next = overrides.shift();
+    if (!next) throw new Error('Unexpected spawnSync call');
+    return {
+      status: next.status,
+      stdout: next.stdout ?? '',
+      stderr: next.stderr ?? '',
+    };
+  });
+}
+
+beforeEach(() => {
+  vi.clearAllMocks();
+});
+
+describe('runHermesSetup', () => {
+  it('returns error when hermes CLI is not found', () => {
+    mockSpawn([{ status: 1, stderr: 'command not found' }]);
+    const r = runHermesSetup({});
+    expect(r.exitCode).toBe(1);
+    expect(r.output.some((l) => l.includes('not found'))).toBe(true);
+  });
+
+  it('reports already-enabled when clawclaw is in plugins list', () => {
+    mockSpawn([
+      { status: 0 },                                  // --version
+      { status: 0, stdout: 'clawclaw\nother-plugin' }, // plugins list
+    ]);
+    const r = runHermesSetup({});
+    expect(r.exitCode).toBe(0);
+    expect(r.output.some((l) => l.includes('already enabled'))).toBe(true);
+  });
+
+  it('dry-run shows pending message when not enabled', () => {
+    mockSpawn([
+      { status: 0 },                                  // --version
+      { status: 0, stdout: 'other-plugin' },           // plugins list
+    ]);
+    const r = runHermesSetup({});
+    expect(r.exitCode).toBe(2);
+    expect(r.output.some((l) => l.includes('Dry-run'))).toBe(true);
+  });
+
+  it('--print shows the command that would run', () => {
+    mockSpawn([
+      { status: 0 },                                  // --version
+      { status: 0, stdout: 'other-plugin' },           // plugins list
+    ]);
+    const r = runHermesSetup({ print: true });
+    expect(r.exitCode).toBe(0);
+    expect(r.output.some((l) => l.includes('hermes plugins enable clawclaw'))).toBe(true);
+  });
+
+  it('-y runs hermes plugins enable and succeeds', () => {
+    mockSpawn([
+      { status: 0 },                                  // --version
+      { status: 0, stdout: 'other-plugin' },           // plugins list
+      { status: 0, stdout: 'plugin enabled' },         // plugins enable
+    ]);
+    const r = runHermesSetup({ yes: true });
+    expect(r.exitCode).toBe(0);
+    expect(r.output.some((l) => l.includes('enabled'))).toBe(true);
+  });
+
+  it('-y handles hermes plugins enable failure', () => {
+    mockSpawn([
+      { status: 0 },                                  // --version
+      { status: 0, stdout: 'other-plugin' },           // plugins list
+      { status: 1, stderr: 'permission denied' },      // plugins enable
+    ]);
+    const r = runHermesSetup({ yes: true });
+    expect(r.exitCode).toBe(1);
+    expect(r.output.some((l) => l.includes('Failed'))).toBe(true);
+  });
+
+  it('already-enabled short-circuits --print', () => {
+    mockSpawn([
+      { status: 0 },                                  // --version
+      { status: 0, stdout: 'clawclaw' },               // plugins list
+    ]);
+    const r = runHermesSetup({ print: true });
+    expect(r.exitCode).toBe(0);
+    expect(r.output.some((l) => l.includes('already enabled'))).toBe(true);
+  });
+});

package/src/commands/setup/hermes.ts

@@ -1,76 +1,76 @@
-/**
- * `ccl setup hermes` — enable the clawclaw plugin in Hermes via `hermes plugins enable`.
- */
-
-import { Command } from 'commander';
-import { spawnSync } from 'child_process';
-
-export const PLUGIN_ID = 'clawclaw';
-
-export interface SetupHermesResult {
-  exitCode: number;
-  output: string[];
-}
-
-export function runHermesSetup(flags: { yes?: boolean; print?: boolean }): SetupHermesResult {
-  const out: string[] = [];
-
-  // Check hermes is available
-  const which = spawnSync('hermes', ['--version'], { stdio: 'pipe', timeout: 5000 });
-  if (which.status !== 0) {
-    out.push('hermes CLI not found. Install it first, then re-run.');
-    return { exitCode: 1, output: out };
-  }
-
-  // Check current plugin status
-  const list = spawnSync('hermes', ['plugins', 'list'], {
-    stdio: 'pipe',
-    timeout: 5000,
-    env: { ...process.env, HERMES_PLUGINS_DEBUG: '1' },
-  });
-  const listOutput = list.stdout?.toString() ?? '';
-  const alreadyEnabled = listOutput.includes(PLUGIN_ID) && !listOutput.includes(`${PLUGIN_ID} (disabled)`);
-
-  if (alreadyEnabled) {
-    out.push(`Plugin "${PLUGIN_ID}" is already enabled in Hermes.`);
-    return { exitCode: 0, output: out };
-  }
-
-  if (flags.print) {
-    out.push(`# Will run: hermes plugins enable ${PLUGIN_ID}`);
-    return { exitCode: 0, output: out };
-  }
-
-  if (!flags.yes) {
-    out.push(`Pending: hermes plugins enable ${PLUGIN_ID}`);
-    out.push(``);
-    out.push(`Dry-run only. Re-run with -y to apply.`);
-    return { exitCode: 2, output: out };
-  }
-
-  const result = spawnSync('hermes', ['plugins', 'enable', PLUGIN_ID], {
-    stdio: 'pipe',
-    timeout: 10000,
-  });
-
-  if (result.status !== 0) {
-    out.push(`Failed to enable plugin: ${result.stderr?.toString() ?? result.stdout?.toString() ?? 'unknown error'}`);
-    return { exitCode: 1, output: out };
-  }
-
-  out.push(`Plugin "${PLUGIN_ID}" enabled in Hermes.`);
-  out.push(`Verify: HERMES_PLUGINS_DEBUG=1 hermes plugins list`);
-  return { exitCode: 0, output: out };
-}
-
-export function createSetupHermesSubcommand(): Command {
-  return new Command('hermes')
-    .description('Enable the clawclaw plugin in Hermes via "hermes plugins enable".')
-    .option('-y, --yes', 'Apply changes (default is dry-run)')
-    .option('--print', 'Only print the command that would be run; do not execute')
-    .action((opts: { yes?: boolean; print?: boolean }) => {
-      const result = runHermesSetup({ yes: opts.yes, print: opts.print });
-      for (const line of result.output) console.log(line);
-      if (result.exitCode !== 0) process.exit(result.exitCode);
-    });
-}
+/**
+ * `ccl setup hermes` — enable the clawclaw plugin in Hermes via `hermes plugins enable`.
+ */
+
+import { Command } from 'commander';
+import { spawnSync } from 'child_process';
+
+export const PLUGIN_ID = 'clawclaw';
+
+export interface SetupHermesResult {
+  exitCode: number;
+  output: string[];
+}
+
+export function runHermesSetup(flags: { yes?: boolean; print?: boolean }): SetupHermesResult {
+  const out: string[] = [];
+
+  // Check hermes is available
+  const which = spawnSync('hermes', ['--version'], { stdio: 'pipe', timeout: 5000 });
+  if (which.status !== 0) {
+    out.push('hermes CLI not found. Install it first, then re-run.');
+    return { exitCode: 1, output: out };
+  }
+
+  // Check current plugin status
+  const list = spawnSync('hermes', ['plugins', 'list'], {
+    stdio: 'pipe',
+    timeout: 5000,
+    env: { ...process.env, HERMES_PLUGINS_DEBUG: '1' },
+  });
+  const listOutput = list.stdout?.toString() ?? '';
+  const alreadyEnabled = listOutput.includes(PLUGIN_ID) && !listOutput.includes(`${PLUGIN_ID} (disabled)`);
+
+  if (alreadyEnabled) {
+    out.push(`Plugin "${PLUGIN_ID}" is already enabled in Hermes.`);
+    return { exitCode: 0, output: out };
+  }
+
+  if (flags.print) {
+    out.push(`# Will run: hermes plugins enable ${PLUGIN_ID}`);
+    return { exitCode: 0, output: out };
+  }
+
+  if (!flags.yes) {
+    out.push(`Pending: hermes plugins enable ${PLUGIN_ID}`);
+    out.push(``);
+    out.push(`Dry-run only. Re-run with -y to apply.`);
+    return { exitCode: 2, output: out };
+  }
+
+  const result = spawnSync('hermes', ['plugins', 'enable', PLUGIN_ID], {
+    stdio: 'pipe',
+    timeout: 10000,
+  });
+
+  if (result.status !== 0) {
+    out.push(`Failed to enable plugin: ${result.stderr?.toString() ?? result.stdout?.toString() ?? 'unknown error'}`);
+    return { exitCode: 1, output: out };
+  }
+
+  out.push(`Plugin "${PLUGIN_ID}" enabled in Hermes.`);
+  out.push(`Verify: HERMES_PLUGINS_DEBUG=1 hermes plugins list`);
+  return { exitCode: 0, output: out };
+}
+
+export function createSetupHermesSubcommand(): Command {
+  return new Command('hermes')
+    .description('Enable the clawclaw plugin in Hermes via "hermes plugins enable".')
+    .option('-y, --yes', 'Apply changes (default is dry-run)')
+    .option('--print', 'Only print the command that would be run; do not execute')
+    .action((opts: { yes?: boolean; print?: boolean }) => {
+      const result = runHermesSetup({ yes: opts.yes, print: opts.print });
+      for (const line of result.output) console.log(line);
+      if (result.exitCode !== 0) process.exit(result.exitCode);
+    });
+}

package/src/commands/setup/index.ts

@@ -1,13 +1,13 @@
-import { Command } from 'commander';
-import { createSetupOpenclawSubcommand } from './openclaw.js';
-import { createSetupHermesSubcommand } from './hermes.js';
-import { createSetupCodexSubcommand } from './codex.js';
-
-export function createSetupCommand(): Command {
-  const setup = new Command('setup');
-  setup.description('Install recommended config snippets into agent host config files.');
-  setup.addCommand(createSetupOpenclawSubcommand());
-  setup.addCommand(createSetupHermesSubcommand());
-  setup.addCommand(createSetupCodexSubcommand());
-  return setup;
-}
+import { Command } from 'commander';
+import { createSetupOpenclawSubcommand } from './openclaw.js';
+import { createSetupHermesSubcommand } from './hermes.js';
+import { createSetupCodexSubcommand } from './codex.js';
+
+export function createSetupCommand(): Command {
+  const setup = new Command('setup');
+  setup.description('Install recommended config snippets into agent host config files.');
+  setup.addCommand(createSetupOpenclawSubcommand());
+  setup.addCommand(createSetupHermesSubcommand());
+  setup.addCommand(createSetupCodexSubcommand());
+  return setup;
+}

package/src/commands/setup/openclaw.test.ts

@@ -1,114 +1,114 @@
-import { describe, expect, it } from 'vitest';
-import { computeOpenclawPatch, resolveOpenclawConfigPath } from './openclaw.js';
-
-describe('computeOpenclawPatch', () => {
-  it('on empty config creates plugins.allow, entries.enabled, tools.alsoAllow', () => {
-    const r = computeOpenclawPatch({}, {});
-    expect(r.warnings).toEqual([]);
-    expect(r.next).toEqual({
-      plugins: { allow: ['clawclaw'], entries: { clawclaw: { enabled: true } } },
-      tools: { alsoAllow: ['clawclaw'] },
-    });
-    expect(r.changes).toHaveLength(3);
-  });
-
-  it('null/non-object current is treated as empty', () => {
-    const r1 = computeOpenclawPatch(null, {});
-    expect(r1.warnings).toEqual([]);
-    expect((r1.next as any).plugins.allow).toEqual(['clawclaw']);
-    const r2 = computeOpenclawPatch(['unexpected'], {});
-    expect(r2.warnings).toEqual([]);
-    expect((r2.next as any).plugins.allow).toEqual(['clawclaw']);
-  });
-
-  it('no-op when everything already present', () => {
-    const current = {
-      plugins: { allow: ['clawclaw'], entries: { clawclaw: { enabled: true } } },
-      tools: { profile: 'coding', alsoAllow: ['clawclaw'] },
-    };
-    const r = computeOpenclawPatch(current, {});
-    expect(r.changes).toEqual([]);
-    expect(r.warnings).toEqual([]);
-    expect(r.next).toEqual(current);
-  });
-
-  it('Set union into existing plugins.allow without dupes', () => {
-    const r = computeOpenclawPatch({ plugins: { allow: ['other'] } }, {});
-    expect((r.next as any).plugins.allow).toEqual(['other', 'clawclaw']);
-    expect(r.changes).toContain('add "clawclaw" to plugins.allow');
-  });
-
-  it('Set union into existing tools.alsoAllow without dupes', () => {
-    const r = computeOpenclawPatch({ tools: { profile: 'coding', alsoAllow: ['somethingElse'] } }, {});
-    expect((r.next as any).tools.alsoAllow).toEqual(['somethingElse', 'clawclaw']);
-    expect(r.changes).toContain('add "clawclaw" to tools.alsoAllow');
-  });
-
-  it('does not touch tools.profile, even when not coding', () => {
-    const r = computeOpenclawPatch({ tools: { profile: 'full' } }, {});
-    expect((r.next as any).tools.profile).toBe('full');
-  });
-
-  it('does not touch tools.profile when missing (no opinion-setting)', () => {
-    const r = computeOpenclawPatch({}, {});
-    expect((r.next as any).tools.profile).toBeUndefined();
-  });
-
-  it('warns and skips when plugins.allow is not an array', () => {
-    const r = computeOpenclawPatch({ plugins: { allow: 'oops' } }, {});
-    expect(r.warnings.some((w) => w.includes('plugins.allow'))).toBe(true);
-    // unchanged value
-    expect((r.next as any).plugins.allow).toBe('oops');
-  });
-
-  it('warns and skips when tools.alsoAllow is not an array', () => {
-    const r = computeOpenclawPatch({ tools: { alsoAllow: 'oops' } }, {});
-    expect(r.warnings.some((w) => w.includes('tools.alsoAllow'))).toBe(true);
-    expect((r.next as any).tools.alsoAllow).toBe('oops');
-  });
-
-  it('respects explicit enabled:false on entries.clawclaw', () => {
-    const r = computeOpenclawPatch(
-      { plugins: { entries: { clawclaw: { enabled: false } } } },
-      {},
-    );
-    expect((r.next as any).plugins.entries.clawclaw.enabled).toBe(false);
-    expect(r.warnings.some((w) => w.includes('explicitly false'))).toBe(true);
-    // and we did not mark a change for it
-    expect(r.changes.some((c) => c.includes('enabled = true'))).toBe(false);
-  });
-
-  it('--skip-plugins-allow leaves plugins untouched, still patches tools', () => {
-    const r = computeOpenclawPatch({}, { skipPluginsAllow: true });
-    expect((r.next as any).plugins).toBeUndefined();
-    expect((r.next as any).tools.alsoAllow).toEqual(['clawclaw']);
-  });
-
-  it('--skip-tools-allow leaves tools untouched, still patches plugins', () => {
-    const r = computeOpenclawPatch({}, { skipToolsAllow: true });
-    expect((r.next as any).tools).toBeUndefined();
-    expect((r.next as any).plugins.allow).toEqual(['clawclaw']);
-  });
-
-  it('preserves unrelated top-level keys', () => {
-    const r = computeOpenclawPatch(
-      { models: { mode: 'replace' }, channels: { telegram: { enabled: true } } },
-      {},
-    );
-    expect((r.next as any).models).toEqual({ mode: 'replace' });
-    expect((r.next as any).channels).toEqual({ telegram: { enabled: true } });
-  });
-});
-
-describe('resolveOpenclawConfigPath', () => {
-  it('uses OPENCLAW_HOME when set', () => {
-    const p = resolveOpenclawConfigPath({ OPENCLAW_HOME: '/tmp/oc' });
-    expect(p.replace(/\\/g, '/')).toBe('/tmp/oc/openclaw.json');
-  });
-
-  it('falls back to ~/.openclaw/openclaw.json', () => {
-    const p = resolveOpenclawConfigPath({});
-    expect(p.endsWith('openclaw.json')).toBe(true);
-    expect(p.includes('.openclaw')).toBe(true);
-  });
-});
+import { describe, expect, it } from 'vitest';
+import { computeOpenclawPatch, resolveOpenclawConfigPath } from './openclaw.js';
+
+describe('computeOpenclawPatch', () => {
+  it('on empty config creates plugins.allow, entries.enabled, tools.alsoAllow', () => {
+    const r = computeOpenclawPatch({}, {});
+    expect(r.warnings).toEqual([]);
+    expect(r.next).toEqual({
+      plugins: { allow: ['clawclaw'], entries: { clawclaw: { enabled: true } } },
+      tools: { alsoAllow: ['clawclaw'] },
+    });
+    expect(r.changes).toHaveLength(3);
+  });
+
+  it('null/non-object current is treated as empty', () => {
+    const r1 = computeOpenclawPatch(null, {});
+    expect(r1.warnings).toEqual([]);
+    expect((r1.next as any).plugins.allow).toEqual(['clawclaw']);
+    const r2 = computeOpenclawPatch(['unexpected'], {});
+    expect(r2.warnings).toEqual([]);
+    expect((r2.next as any).plugins.allow).toEqual(['clawclaw']);
+  });
+
+  it('no-op when everything already present', () => {
+    const current = {
+      plugins: { allow: ['clawclaw'], entries: { clawclaw: { enabled: true } } },
+      tools: { profile: 'coding', alsoAllow: ['clawclaw'] },
+    };
+    const r = computeOpenclawPatch(current, {});
+    expect(r.changes).toEqual([]);
+    expect(r.warnings).toEqual([]);
+    expect(r.next).toEqual(current);
+  });
+
+  it('Set union into existing plugins.allow without dupes', () => {
+    const r = computeOpenclawPatch({ plugins: { allow: ['other'] } }, {});
+    expect((r.next as any).plugins.allow).toEqual(['other', 'clawclaw']);
+    expect(r.changes).toContain('add "clawclaw" to plugins.allow');
+  });
+
+  it('Set union into existing tools.alsoAllow without dupes', () => {
+    const r = computeOpenclawPatch({ tools: { profile: 'coding', alsoAllow: ['somethingElse'] } }, {});
+    expect((r.next as any).tools.alsoAllow).toEqual(['somethingElse', 'clawclaw']);
+    expect(r.changes).toContain('add "clawclaw" to tools.alsoAllow');
+  });
+
+  it('does not touch tools.profile, even when not coding', () => {
+    const r = computeOpenclawPatch({ tools: { profile: 'full' } }, {});
+    expect((r.next as any).tools.profile).toBe('full');
+  });
+
+  it('does not touch tools.profile when missing (no opinion-setting)', () => {
+    const r = computeOpenclawPatch({}, {});
+    expect((r.next as any).tools.profile).toBeUndefined();
+  });
+
+  it('warns and skips when plugins.allow is not an array', () => {
+    const r = computeOpenclawPatch({ plugins: { allow: 'oops' } }, {});
+    expect(r.warnings.some((w) => w.includes('plugins.allow'))).toBe(true);
+    // unchanged value
+    expect((r.next as any).plugins.allow).toBe('oops');
+  });
+
+  it('warns and skips when tools.alsoAllow is not an array', () => {
+    const r = computeOpenclawPatch({ tools: { alsoAllow: 'oops' } }, {});
+    expect(r.warnings.some((w) => w.includes('tools.alsoAllow'))).toBe(true);
+    expect((r.next as any).tools.alsoAllow).toBe('oops');
+  });
+
+  it('respects explicit enabled:false on entries.clawclaw', () => {
+    const r = computeOpenclawPatch(
+      { plugins: { entries: { clawclaw: { enabled: false } } } },
+      {},
+    );
+    expect((r.next as any).plugins.entries.clawclaw.enabled).toBe(false);
+    expect(r.warnings.some((w) => w.includes('explicitly false'))).toBe(true);
+    // and we did not mark a change for it
+    expect(r.changes.some((c) => c.includes('enabled = true'))).toBe(false);
+  });
+
+  it('--skip-plugins-allow leaves plugins untouched, still patches tools', () => {
+    const r = computeOpenclawPatch({}, { skipPluginsAllow: true });
+    expect((r.next as any).plugins).toBeUndefined();
+    expect((r.next as any).tools.alsoAllow).toEqual(['clawclaw']);
+  });
+
+  it('--skip-tools-allow leaves tools untouched, still patches plugins', () => {
+    const r = computeOpenclawPatch({}, { skipToolsAllow: true });
+    expect((r.next as any).tools).toBeUndefined();
+    expect((r.next as any).plugins.allow).toEqual(['clawclaw']);
+  });
+
+  it('preserves unrelated top-level keys', () => {
+    const r = computeOpenclawPatch(
+      { models: { mode: 'replace' }, channels: { telegram: { enabled: true } } },
+      {},
+    );
+    expect((r.next as any).models).toEqual({ mode: 'replace' });
+    expect((r.next as any).channels).toEqual({ telegram: { enabled: true } });
+  });
+});
+
+describe('resolveOpenclawConfigPath', () => {
+  it('uses OPENCLAW_HOME when set', () => {
+    const p = resolveOpenclawConfigPath({ OPENCLAW_HOME: '/tmp/oc' });
+    expect(p.replace(/\\/g, '/')).toBe('/tmp/oc/openclaw.json');
+  });
+
+  it('falls back to ~/.openclaw/openclaw.json', () => {
+    const p = resolveOpenclawConfigPath({});
+    expect(p.endsWith('openclaw.json')).toBe(true);
+    expect(p.includes('.openclaw')).toBe(true);
+  });
+});