fix: #80507 show dry-run output for message send/poll (#94684)

lzyyzznl · web-flow · commit 508e3bf41353 · 2026-06-19T03:51:50.000Z
Summary: - The branch changes `formatMessageCliText` to render dry-run message output from `result.dryRun` instead of only `handledBy === "dry-run"`. - PR surface: Source 0. Total 0 across 1 file. - Reproducibility: yes. source-reproducible. The linked issue has captured CLI output, and current main shows ... e the formatter still checks `handledBy === "dry-run"`; I did not execute the CLI in this read-only review. Automerge notes: - No ClawSweeper repair was needed after automerge opt-in. Validation: - ClawSweeper review passed for head dce6d6a. - Required merge gates passed before the squash merge. Prepared head SHA: dce6d6a Review: #94684 (comment) Co-authored-by: lizeyu-xydt <li.zeyu@xydigit.com>
diff --git a/src/commands/message-format.ts b/src/commands/message-format.ts
@@ -247,7 +247,7 @@ export function formatMessageCliText(result: MessageActionRunResult): string[] {
   const width = getTerminalTableWidth();
   const opts: FormatOpts = { width };
 
-  if (result.handledBy === "dry-run") {
+  if (result.dryRun) {
     return [muted(`[dry-run] would run ${result.action} via ${result.channel}`)];
   }
 

Original file line number	Diff line number	Diff line change
`@@ -247,7 +247,7 @@ export function formatMessageCliText(result: MessageActionRunResult): string[] {`
`247`	`247`	`const width = getTerminalTableWidth();`
`248`	`248`	`const opts: FormatOpts = { width };`
`249`	`249`
`250`		`- if (result.handledBy === "dry-run") {`
	`250`	`+ if (result.dryRun) {`
`251`	`251`	return [muted(`[dry-run] would run ${result.action} via ${result.channel}`)];
`252`	`252`	`}`
`253`	`253`