fix(auto-reply): 限制空回复兜底仅用于私聊

Leon-llb · Leon-llb · commit 7b583d2f5c02 · 2026-04-09T11:09:54.000+08:00
避免通用 fallback 误伤 Slack 无回复也合法的 ack 场景
仅在 direct/private 类会话且确实无可发送输出时补发提示
补充频道消息回归测试，保持现有静默分支行为不变
diff --git a/src/auto-reply/reply/dispatch-from-config.test.ts b/src/auto-reply/reply/dispatch-from-config.test.ts
@@ -2984,4 +2984,20 @@ describe("before_dispatch hook", () => {
     expect(result.queuedFinal).toBe(true);
     expect(result.counts.final).toBe(1);
   });
+
+  it("does not send the fallback final reply for channel messages", async () => {
+    hookMocks.runner.runBeforeDispatch.mockResolvedValue({ handled: false });
+    const dispatcher = createDispatcher();
+
+    const result = await dispatchReplyFromConfig({
+      ctx: createHookCtx({ ChatType: "channel", Surface: "slack", Provider: "slack" }),
+      cfg: emptyConfig,
+      dispatcher,
+      replyResolver: async () => undefined,
+    });
+
+    expect(dispatcher.sendFinalReply).not.toHaveBeenCalled();
+    expect(result.queuedFinal).toBe(false);
+    expect(result.counts.final).toBe(0);
+  });
 });
diff --git a/src/auto-reply/reply/dispatch-from-config.ts b/src/auto-reply/reply/dispatch-from-config.ts
@@ -104,6 +104,10 @@ const normalizeMediaType = (value: string): string =>
   normalizeOptionalLowercaseString(value.split(";")[0]) ?? "";
 const EMPTY_RUN_FALLBACK_TEXT =
   "I couldn't produce a reply for that request. Please try again or rephrase your message.";
+const isDirectLikeChatType = (value?: string): boolean => {
+  const normalized = normalizeOptionalLowercaseString(value);
+  return normalized === "direct" || normalized === "dm" || normalized === "private";
+};
 
 const isInboundAudioContext = (ctx: FinalizedMsgContext): boolean => {
   const rawTypes = [
@@ -1044,6 +1048,7 @@ export async function dispatchReplyFromConfig(params: {
     counts.final += routedFinalCount;
 
     const shouldSendEmptyRunFallback =
+      isDirectLikeChatType(ctx.ChatType) &&
       !shouldRouteToOriginating &&
       replies.length === 0 &&
       blockCount === 0 &&
