diff --git a/internal/agent/core/continuation_placeholder_test.go b/internal/agent/core/continuation_placeholder_test.go new file mode 100644 index 00000000..889ff479 --- /dev/null +++ b/internal/agent/core/continuation_placeholder_test.go @@ -0,0 +1,42 @@ +package core + +// 「本批全是输出通道发送」之后补的占位文案必须同时给出**两种授权**。 +// +// 现场(2026-10-27,用户原话:"那你去啊?说完就停了"): +// 模型经 output_send__qq 说了一句"我这就去办",内核随后补的占位只有 +// 终止许可("若你的回复已完成,直接返回纯文本即可结束本轮")⇒ 模型照做, +// 本轮结束,活没干。这不是模型偷懒,是内核把它劝退了。 +// +// 但也不能只写"继续":注释里记着一次事故 —— 通用「请继续」被读成 +// "再发一条消息",生产实测单轮 34 次 output_send__qq、持续 514 秒。 +// +// 所以判据是两条同时成立: +// ① 明确"没做完就继续做"(允许继续调工具) +// ② 明确"全部完成就收尾,且不要再发第二条消息"(护栏不削弱) + +import ( + "strings" + "testing" +) + +func TestReplyDeliveredPlaceholderAllowsBothContinuingAndStopping(t *testing.T) { + p := replyDeliveredPlaceholder + if !strings.Contains(p, "没做完") || !strings.Contains(p, "继续做完") { + t.Fatalf("占位文案必须允许「未做完就继续」,实际: %q", p) + } + if !strings.Contains(p, "结束本轮") { + t.Fatalf("占位文案必须保留终止许可(否则模型会无限发送),实际: %q", p) + } + if !strings.Contains(p, "不要再发第二条消息") { + t.Fatalf("占位文案必须明确禁止重复发送(防 34 连发事故复发),实际: %q", p) + } + if strings.Contains(p, "无需再调用任何工具") { + t.Fatalf("不得出现「无需再调用任何工具」——那会把「承诺后未执行」也一并劝退,实际: %q", p) + } + if continuationFor(true) != p { + t.Fatal("replyOnly 时必须使用本条占位") + } + if continuationFor(false) != continuationPlaceholder { + t.Fatal("非 replyOnly 时必须用通用续写占位") + } +} diff --git a/internal/agent/core/process.go b/internal/agent/core/process.go index 8285b44c..30f95252 100644 --- a/internal/agent/core/process.go +++ b/internal/agent/core/process.go @@ -34,8 +34,18 @@ const continuationPlaceholder = "请根据以上工具结果继续。" // 还是再发一条消息。两者叠加成自我强化的发送循环:生产实测单轮 34 次 // output_send__qq、持续 514 秒,直到 QQ 插件自己的循环保险拒绝发送才停下。 // -// 所以这里换成一条明确的终止许可:已回复完就直接返回纯文本收尾。 -const replyDeliveredPlaceholder = "若你的回复已完成,直接返回纯文本即可结束本轮,无需再调用任何工具。" +// 所以这里必须保留「终止许可」,否则发送风暴回来。 +// +// ★ 但原措辞只有终止、没有"继续干活"的授权(2026-10-27 现场): +// 模型先说一句"我这就去办"(经 output_send 交付),随后被这条占位劝退 +// ⇒ 活没干就结束本轮。用户看到的正是"说完就停了"。 +// +// 现在把两件事分开说: +// +// · 有没做完的操作 → 现在继续做完(继续指**非发送类**工具) +// · 真的做完了 → 返回纯文本收尾,**不要再发第二条消息**(护栏仍在) +const replyDeliveredPlaceholder = "回复已发送。若还有没做完的操作(需要调用工具才能完成的)," + + "现在继续做完;若这一轮的工作已经全部完成,直接返回纯文本结束本轮——不要再发第二条消息。" // continuationFor 选择工具轮之后补位的 user 占位文案。 // replyOnly 表示上一批工具调用全部是输出通道发送(即模型刚交付了回复)。