fix(mattermost): address text slash review feedback

amknight · amknight · commit 548c8abdc5d2 · 2026-05-26T19:09:17.000+10:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -31,7 +31,6 @@ Docs: https://docs.openclaw.ai
 - Agents/Codex: honor yolo app-server approval policy only for the full `never` plus `danger-full-access` case. (#85909) Thanks @earlvanze.
 - Gateway/Gmail: clear Gmail watcher renewal intervals on re-entry so hot reloads do not leak lifecycle timers. (#82947) Thanks @SebTardif.
 - Logging: exit cleanly on broken stdout/stderr pipes without masking existing failure exit codes. (#80059) Thanks @pavelzak.
-- Mattermost: show `/new` and `/reset` acknowledgements (and other typed text-slash commands) in Mattermost direct chats and channels when the active harness defaults source replies to message-tool-only delivery (e.g. Codex), so authorized control commands no longer silently change session state without confirmation. Fixes #86664. Thanks @amknight.
 - Gateway/security: escape transcript metadata field names while extracting oversized session line prefixes. (#85934) Thanks @SebTardif.
 - Plugins/security: validate manifest model pattern regexes with the safe-regex compiler so unsafe patterns are ignored before matching. (#86046) Thanks @SebTardif.
 - Discord: route gateway metadata REST lookups through the configured Discord proxy so proxied accounts do not fall back to direct `discord.com` connections before opening the WebSocket. Fixes #80227. Thanks @Clivilwalker.
diff --git a/extensions/mattermost/src/mattermost/monitor.inbound-system-event.test.ts b/extensions/mattermost/src/mattermost/monitor.inbound-system-event.test.ts
@@ -575,7 +575,7 @@ describe("mattermost inbound user posts", () => {
           id: "post-reset",
           channel_id: "dm-1",
           user_id: "user-1",
-          message: "/reset",
+          message: " /reset",
           create_at: 1_714_000_000_000,
         }),
       },
@@ -590,6 +590,7 @@ describe("mattermost inbound user posts", () => {
     expect(mockState.dispatchReplyFromConfig).toHaveBeenCalledTimes(1);
     const ctx = mockState.dispatchReplyFromConfig.mock.calls.at(0)?.[0].ctx;
     expect(ctx?.BodyForAgent).toBe("/reset");
+    expect(ctx?.CommandBody).toBe("/reset");
     expect(ctx?.CommandAuthorized).toBe(true);
     expect(ctx?.CommandSource).toBe("text");
   });
