import { Command } from "commander"; import { afterEach, beforeEach, describe, expect, it, vi } from "vitest"; import type { ConfigFileSnapshot, OpenClawConfig } from "../config/types.js"; /** * Test for issue #6070: * `openclaw config set/unset` must update snapshot.resolved (user config after $include/${ENV}, * but before runtime defaults), so runtime defaults don't leak into the written config. */ const mockReadConfigFileSnapshot = vi.fn<() => Promise>(); const mockWriteConfigFile = vi.fn<(cfg: OpenClawConfig) => Promise>(async () => {}); vi.mock("../config/config.js", () => ({ readConfigFileSnapshot: () => mockReadConfigFileSnapshot(), writeConfigFile: (cfg: OpenClawConfig) => mockWriteConfigFile(cfg), })); const mockLog = vi.fn(); const mockError = vi.fn(); const mockExit = vi.fn((code: number) => { const errorMessages = mockError.mock.calls.map((c) => c.join(" ")).join("; "); throw new Error(`__exit__:${code} - ${errorMessages}`); }); vi.mock("../runtime.js", () => ({ defaultRuntime: { log: (...args: unknown[]) => mockLog(...args), error: (...args: unknown[]) => mockError(...args), exit: (code: number) => mockExit(code), }, })); function buildSnapshot(params: { resolved: OpenClawConfig; config: OpenClawConfig; }): ConfigFileSnapshot { return { path: "/tmp/openclaw.json", exists: true, raw: JSON.stringify(params.resolved), parsed: params.resolved, resolved: params.resolved, valid: true, config: params.config, issues: [], warnings: [], legacyIssues: [], }; } function setSnapshot(resolved: OpenClawConfig, config: OpenClawConfig) { mockReadConfigFileSnapshot.mockResolvedValueOnce(buildSnapshot({ resolved, config })); } async function runConfigCommand(args: string[]) { const { registerConfigCli } = await import("./config-cli.js"); const program = new Command(); program.exitOverride(); registerConfigCli(program); await program.parseAsync(args, { from: "user" }); } describe("config cli", () => { beforeEach(() => { vi.clearAllMocks(); }); afterEach(() => { vi.restoreAllMocks(); }); describe("config set - issue #6070", () => { it("preserves existing config keys when setting a new value", async () => { const resolved: OpenClawConfig = { agents: { list: [{ id: "main" }, { id: "oracle", workspace: "~/oracle-workspace" }], }, gateway: { port: 18789 }, tools: { allow: ["group:fs"] }, logging: { level: "debug" }, }; const runtimeMerged: OpenClawConfig = { ...resolved, agents: { ...resolved.agents, defaults: { model: "gpt-5.2", } as never, } as never, }; setSnapshot(resolved, runtimeMerged); await runConfigCommand(["config", "set", "gateway.auth.mode", "token"]); expect(mockWriteConfigFile).toHaveBeenCalledTimes(1); const written = mockWriteConfigFile.mock.calls[0]?.[0]; expect(written.gateway?.auth).toEqual({ mode: "token" }); expect(written.gateway?.port).toBe(18789); expect(written.agents).toEqual(resolved.agents); expect(written.tools).toEqual(resolved.tools); expect(written.logging).toEqual(resolved.logging); expect(written.agents).not.toHaveProperty("defaults"); }); it("does not inject runtime defaults into the written config", async () => { const resolved: OpenClawConfig = { gateway: { port: 18789 }, }; const runtimeMerged = { ...resolved, agents: { defaults: { model: "gpt-5.2", contextWindow: 128_000, maxTokens: 16_000, }, } as never, messages: { ackReaction: "✅" } as never, sessions: { persistence: { enabled: true } } as never, } as unknown as OpenClawConfig; setSnapshot(resolved, runtimeMerged); await runConfigCommand(["config", "set", "gateway.auth.mode", "token"]); expect(mockWriteConfigFile).toHaveBeenCalledTimes(1); const written = mockWriteConfigFile.mock.calls[0]?.[0]; expect(written).not.toHaveProperty("agents.defaults.model"); expect(written).not.toHaveProperty("agents.defaults.contextWindow"); expect(written).not.toHaveProperty("agents.defaults.maxTokens"); expect(written).not.toHaveProperty("messages.ackReaction"); expect(written).not.toHaveProperty("sessions.persistence"); expect(written.gateway?.port).toBe(18789); expect(written.gateway?.auth).toEqual({ mode: "token" }); }); }); describe("config set parsing flags", () => { it("falls back to raw string when parsing fails and strict mode is off", async () => { const resolved: OpenClawConfig = { gateway: { port: 18789 } }; setSnapshot(resolved, resolved); await runConfigCommand(["config", "set", "gateway.auth.mode", "{bad"]); expect(mockWriteConfigFile).toHaveBeenCalledTimes(1); const written = mockWriteConfigFile.mock.calls[0]?.[0]; expect(written.gateway?.auth).toEqual({ mode: "{bad" }); }); it("throws when strict parsing is enabled via --strict-json", async () => { await expect( runConfigCommand(["config", "set", "gateway.auth.mode", "{bad", "--strict-json"]), ).rejects.toThrow("__exit__:1"); expect(mockWriteConfigFile).not.toHaveBeenCalled(); expect(mockReadConfigFileSnapshot).not.toHaveBeenCalled(); }); it("keeps --json as a strict parsing alias", async () => { await expect( runConfigCommand(["config", "set", "gateway.auth.mode", "{bad", "--json"]), ).rejects.toThrow("__exit__:1"); expect(mockWriteConfigFile).not.toHaveBeenCalled(); expect(mockReadConfigFileSnapshot).not.toHaveBeenCalled(); }); it("shows --strict-json and keeps --json as a legacy alias in help", async () => { const { registerConfigCli } = await import("./config-cli.js"); const program = new Command(); registerConfigCli(program); const configCommand = program.commands.find((command) => command.name() === "config"); const setCommand = configCommand?.commands.find((command) => command.name() === "set"); const helpText = setCommand?.helpInformation() ?? ""; expect(helpText).toContain("--strict-json"); expect(helpText).toContain("--json"); expect(helpText).toContain("Legacy alias for --strict-json"); }); }); describe("config unset - issue #6070", () => { it("preserves existing config keys when unsetting a value", async () => { const resolved: OpenClawConfig = { agents: { list: [{ id: "main" }] }, gateway: { port: 18789 }, tools: { profile: "coding", alsoAllow: ["agents_list"], }, logging: { level: "debug" }, }; const runtimeMerged: OpenClawConfig = { ...resolved, agents: { ...resolved.agents, defaults: { model: "gpt-5.2", }, } as never, }; setSnapshot(resolved, runtimeMerged); await runConfigCommand(["config", "unset", "tools.alsoAllow"]); expect(mockWriteConfigFile).toHaveBeenCalledTimes(1); const written = mockWriteConfigFile.mock.calls[0]?.[0]; expect(written.tools).not.toHaveProperty("alsoAllow"); expect(written.agents).not.toHaveProperty("defaults"); expect(written.agents?.list).toEqual(resolved.agents?.list); expect(written.gateway).toEqual(resolved.gateway); expect(written.tools?.profile).toBe("coding"); expect(written.logging).toEqual(resolved.logging); }); }); });