From fdcd83da08f3275444fdceec62526078df1095e2 Mon Sep 17 00:00:00 2001
From: Shakker <shakkerdroid@gmail.com>
Date: Tue, 12 May 2026 19:46:48 +0100
Subject: [PATCH] test: inscribe command config writes

---
 src/auto-reply/reply/commands-gating.test.ts  |  7 ++++---
 src/auto-reply/reply/commands-plugins.test.ts | 12 ++++++------
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/src/auto-reply/reply/commands-gating.test.ts b/src/auto-reply/reply/commands-gating.test.ts
index d79d52a6a15..a14b1d5cf76 100644
--- a/src/auto-reply/reply/commands-gating.test.ts
+++ b/src/auto-reply/reply/commands-gating.test.ts
@@ -576,8 +576,9 @@ describe("command gating", () => {
     expect(setResult?.shouldContinue).toBe(false);
     expect(setResult?.reply?.text).toContain("Config updated");
     expect(replaceConfigFileMock).toHaveBeenCalledTimes(1);
-    expect(replaceConfigFileMock).toHaveBeenCalledWith(
-      expect.objectContaining({ afterWrite: { mode: "auto" } }),
-    );
+    expect(replaceConfigFileMock).toHaveBeenCalledWith({
+      nextConfig: {},
+      afterWrite: { mode: "auto" },
+    });
   });
 });
diff --git a/src/auto-reply/reply/commands-plugins.test.ts b/src/auto-reply/reply/commands-plugins.test.ts
index 7780b4e4444..f44da133a60 100644
--- a/src/auto-reply/reply/commands-plugins.test.ts
+++ b/src/auto-reply/reply/commands-plugins.test.ts
@@ -133,22 +133,22 @@ function expectPluginEnabledInConfig(config: unknown, enabled: boolean) {
 }
 
 function expectLastReplaceConfig(enabled: boolean) {
-  expect(replaceConfigFileMock).toHaveBeenLastCalledWith(
-    expect.objectContaining({ afterWrite: { mode: "auto" } }),
-  );
   const calls = (replaceConfigFileMock as unknown as MockCalls).mock.calls;
   const [payload] = calls.at(-1) ?? [];
   const payloadRecord = requireRecord(payload, "replace config payload");
+  expect(Object.keys(payloadRecord).sort()).toEqual(["afterWrite", "nextConfig"]);
+  expect(payloadRecord.afterWrite).toEqual({ mode: "auto" });
   expectPluginEnabledInConfig(payloadRecord.nextConfig, enabled);
 }
 
 function expectLastRegistryRefresh(enabled: boolean) {
-  expect(refreshPluginRegistryAfterConfigMutationMock).toHaveBeenLastCalledWith(
-    expect.objectContaining({ reason: "policy-changed" }),
-  );
   const calls = (refreshPluginRegistryAfterConfigMutationMock as unknown as MockCalls).mock.calls;
   const [payload] = calls.at(-1) ?? [];
   const payloadRecord = requireRecord(payload, "registry refresh payload");
+  expect(Object.keys(payloadRecord).sort()).toEqual(["config", "logger", "reason"]);
+  expect(payloadRecord.reason).toBe("policy-changed");
+  const logger = getNestedRecord(payloadRecord, "logger", "registry refresh logger");
+  expect(logger.warn).toEqual(expect.any(Function));
   expectPluginEnabledInConfig(payloadRecord.config, enabled);
 }