From 7123f89cf844c30fe75a05f16b6e1caeecd00f74 Mon Sep 17 00:00:00 2001 From: Dallin Romney Date: Sun, 9 Aug 2026 16:43:17 +0800 Subject: [PATCH] fix(slack): keep commentary progress on draft lane --- .../monitor/message-handler/dispatch.preview-fallback.test.ts | 1 + extensions/slack/src/monitor/message-handler/dispatch.ts | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/extensions/slack/src/monitor/message-handler/dispatch.preview-fallback.test.ts b/extensions/slack/src/monitor/message-handler/dispatch.preview-fallback.test.ts index 115aff7a7355..c6892e7d7d0d 100644 --- a/extensions/slack/src/monitor/message-handler/dispatch.preview-fallback.test.ts +++ b/extensions/slack/src/monitor/message-handler/dispatch.preview-fallback.test.ts @@ -3954,6 +3954,7 @@ describe("dispatchPreparedSlackMessage preview fallback", () => { ); expect(capturedReplyOptions?.commentaryProgressEnabled).toBe(true); + expect(capturedReplyOptions?.commentaryPayloadsEnabled).toBeUndefined(); expect(capturedReplyOptions?.suppressDefaultToolProgressMessages).toBe(true); expect(draftStream.update).toHaveBeenLastCalledWith("_Preparing the smallest fix_"); expect(draftStream.update.mock.calls.flat().join("\n")).not.toContain("pnpm test"); diff --git a/extensions/slack/src/monitor/message-handler/dispatch.ts b/extensions/slack/src/monitor/message-handler/dispatch.ts index 513a98b1f6ca..0946a83244bc 100644 --- a/extensions/slack/src/monitor/message-handler/dispatch.ts +++ b/extensions/slack/src/monitor/message-handler/dispatch.ts @@ -394,7 +394,6 @@ export async function dispatchPreparedSlackMessage(prepared: PreparedSlackMessag commentaryProgressEnabled: progress.commentaryProgressEnabled ? true : undefined, progressPreambleEnabled: progress.progressDraftActive && slackStreaming.mode === "progress" ? true : undefined, - commentaryPayloadsEnabled: progress.commentaryProgressEnabled ? true : undefined, onVerboseProgressVisibility: progress.commentaryProgressEnabled ? (isActive) => { progress.setShouldYieldDraftProgress(isActive);