fix(mattermost): suppress reasoning previews

extensions/mattermost/src/mattermost/monitor.test.ts

@@ -252,6 +252,29 @@ describe("shouldClearMattermostDraftPreview", () => {
 });
 
 describe("deliverMattermostReplyWithDraftPreview", () => {
+  it("suppresses reasoning-prefixed finals before preview finalization", async () => {
+    const draftStream = createDraftStreamMock();
+    const deliverFinal = vi.fn(async () => {});
+
+    await deliverMattermostReplyWithDraftPreview({
+      payload: { text: "  \n Reasoning:\n_hidden_" } as never,
+      info: { kind: "final" },
+      client: createMattermostClientMock(),
+      draftStream,
+      effectiveReplyToId: "thread-root-1",
+      resolvePreviewFinalText: (text) => text?.trim(),
+      previewState: { finalizedViaPreviewPost: false },
+      logVerboseMessage: vi.fn(),
+      deliverFinal,
+    });
+
+    expect(deliverFinal).not.toHaveBeenCalled();
+    expect(draftStream.flush).not.toHaveBeenCalled();
+    expect(draftStream.discardPending).not.toHaveBeenCalled();
+    expect(draftStream.clear).not.toHaveBeenCalled();
+    expect(updateMattermostPostSpy).not.toHaveBeenCalled();
+  });
+
   it("deletes the preview after a successful normal final send", async () => {
     const draftStream = createDraftStreamMock();
     const deliverFinal = vi.fn(async () => {});

extensions/mattermost/src/mattermost/monitor.ts

@@ -55,7 +55,10 @@ import {
   type MattermostWebSocketFactory,
 } from "./monitor-websocket.js";
 import { runWithReconnect } from "./reconnect.js";
-import { deliverMattermostReplyPayload } from "./reply-delivery.js";
+import {
+  deliverMattermostReplyPayload,
+  shouldSuppressMattermostReasoningReply,
+} from "./reply-delivery.js";
 import type {
   ChannelAccountSnapshot,
   ChatType,
@@ -289,7 +292,7 @@ type MattermostDraftPreviewDeliverParams = {
 export async function deliverMattermostReplyWithDraftPreview(
   params: MattermostDraftPreviewDeliverParams,
 ): Promise<void> {
-  if (params.payload.isReasoning) {
+  if (shouldSuppressMattermostReasoningReply(params.payload)) {
     return;
   }
 

extensions/mattermost/src/mattermost/reply-delivery.ts

@@ -26,7 +26,7 @@ type SendMattermostMessage = (
 
 const REASONING_PREFIX = "reasoning:";
 
-function shouldSuppressReasoningReply(payload: ReplyPayload): boolean {
+export function shouldSuppressMattermostReasoningReply(payload: ReplyPayload): boolean {
   if (payload.isReasoning === true) {
     return true;
   }
@@ -49,7 +49,7 @@ export async function deliverMattermostReplyPayload(params: {
   tableMode: MarkdownTableMode;
   sendMessage: SendMattermostMessage;
 }): Promise<void> {
-  if (shouldSuppressReasoningReply(params.payload)) {
+  if (shouldSuppressMattermostReasoningReply(params.payload)) {
     return;
   }
   const reply = resolveSendableOutboundReplyParts(params.payload, {