fix(auto-reply): suppress repeated silent tokens (#86848) · openclaw/openclaw@8ec2b2d

Recent Commits to openclaw:main

Alix-007 · 2026-05-27 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -136,6 +136,7 @@ describe("AgentRuntimePlan", () => {`
`136`	`136`	`expect(plan.auth.authProfileProviderForAuth).toBe("openai-codex");`
`137`	`137`	`expect(plan.auth.harnessAuthProvider).toBe("openai-codex");`
`138`	`138`	`expect(plan.auth.forwardedAuthProfileId).toBe("openai-codex:work");`
	`139`	`+expect(plan.delivery.isSilentPayload({ text: "NO_REPLY\n\nNO_REPLY" })).toBe(true);`
`139`	`140`	`expect(plan.delivery.isSilentPayload({ text: '{"action":"NO_REPLY"}' })).toBe(true);`
`140`	`141`	`expect(`
`141`	`142`	`plan.delivery.isSilentPayload({`

Original file line number	Diff line number	Diff line change
`@@ -152,6 +152,11 @@ describe("normalizeReplyPayload", () => {`
`152`	`152`	`it("records skip reasons for silent/empty payloads", () => {`
`153`	`153`	`const cases = [`
`154`	`154`	`{ name: "silent", payload: { text: SILENT_REPLY_TOKEN }, reason: "silent" },`
	`155`	`+{`
	`156`	`+name: "repeated silent",`
	`157`	+payload: { text: `${SILENT_REPLY_TOKEN}\n\n${SILENT_REPLY_TOKEN}` },
	`158`	`+reason: "silent",`
	`159`	`+},`
`155`	`160`	`{ name: "empty", payload: { text: " " }, reason: "empty" },`
`156`	`161`	`] as const;`
`157`	`162`	`for (const testCase of cases) {`

Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,11 @@ describe("isSilentReplyText", () => {`
`22`	`22`	`expect(isSilentReplyText(" No_RePlY ")).toBe(true);`
`23`	`23`	`});`
`24`	`24`
	`25`	`+it("returns true for repeated token-only text separated by whitespace", () => {`
	`26`	`+expect(isSilentReplyText("NO_REPLY\n\nNO_REPLY")).toBe(true);`
	`27`	`+expect(isSilentReplyText(" no_reply \t No_RePlY ")).toBe(true);`
	`28`	`+});`
	`29`	`+`
`25`	`30`	`it("returns false for undefined/empty", () => {`
`26`	`31`	`expect(isSilentReplyText(undefined)).toBe(false);`
`27`	`32`	`expect(isSilentReplyText("")).toBe(false);`
`@@ -87,6 +92,11 @@ describe("custom silent tokens", () => {`
`87`	`92`	`check: () => isSilentReplyText("Checked inbox. HEARTBEAT_OK", "HEARTBEAT_OK"),`
`88`	`93`	`expected: false,`
`89`	`94`	`},`
	`95`	`+{`
	`96`	`+name: "repeated-token detection",`
	`97`	`+check: () => isSilentReplyText("HEARTBEAT_OK\nHEARTBEAT_OK", "HEARTBEAT_OK"),`
	`98`	`+expected: true,`
	`99`	`+},`
`90`	`100`	`{`
`91`	`101`	`name: "trailing token stripping",`
`92`	`102`	`check: () => stripSilentToken("done HEARTBEAT_OK", "HEARTBEAT_OK"),`

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ function getSilentExactRegex(token: string): RegExp {`
`13`	`13`	`return cached;`
`14`	`14`	`}`
`15`	`15`	`const escaped = escapeRegExp(token);`
`16`		-const regex = new RegExp(`^\\s${escaped}\\s$`, "i");
	`16`	+const regex = new RegExp(`^\\s${escaped}(?:\\s+${escaped})\\s*$`, "i");
`17`	`17`	`silentExactRegexByToken.set(token, regex);`
`18`	`18`	`return regex;`
`19`	`19`	`}`
`@@ -36,7 +36,7 @@ export function isSilentReplyText(`
`36`	`36`	`if (!text) {`
`37`	`37`	`return false;`
`38`	`38`	`}`
`39`		`-// Match only the exact silent token with optional surrounding whitespace.`
	`39`	`+// Match only token-only replies, including repeated tokens separated by whitespace.`
`40`	`40`	`// This prevents substantive replies ending with NO_REPLY from being suppressed (#19537).`
`41`	`41`	`return getSilentExactRegex(token).test(text);`
`42`	`42`	`}`

Original file line number	Diff line number	Diff line change
`@@ -77,6 +77,7 @@ describe("normalizeReplyPayloadsForDelivery", () => {`
`77`	`77`	`expect(`
`78`	`78`	`normalizeReplyPayloadsForDelivery([`
`79`	`79`	`{ text: "NO_REPLY" },`
	`80`	`+{ text: "NO_REPLY\n\nNO_REPLY" },`
`80`	`81`	`{ text: "Reasoning:\n_step_", isReasoning: true },`
`81`	`82`	`{ text: "final answer" },`
`82`	`83`	`]),`
`@@ -157,6 +158,7 @@ describe("normalizeReplyPayloadsForDelivery", () => {`
`157`	`158`	`normalizeReplyPayloadsForDelivery([`
`158`	`159`	`{ text: "NO_REPLY thanks for the update" },`
`159`	`160`	`{ text: "NO_REPLY" },`
	`161`	`+{ text: "NO_REPLY\n\nNO_REPLY" },`
`160`	`162`	`{ text: "thanks NO_REPLY" },`
`161`	`163`	`]),`
`162`	`164`	`).toEqual([`

Original file line number	Diff line number	Diff line change
`@@ -574,7 +574,12 @@ describe("plugin-sdk subpath exports", () => {`
`574`	`574`	`"createResolvedApproverActionAuthAdapter",`
`575`	`575`	`"resolveApprovalApprovers",`
`576`	`576`	`]);`
`577`		`-expectSourceMentions("reply-chunking", ["chunkText", "chunkTextWithMode"]);`
	`577`	`+expectSourceMentions("reply-chunking", [`
	`578`	`+"chunkText",`
	`579`	`+"chunkTextWithMode",`
	`580`	`+"isSilentReplyPayloadText",`
	`581`	`+"isSilentReplyText",`
	`582`	`+]);`
`578`	`583`	`expectSourceMentions("reply-history", [`
`579`	`584`	`"buildInboundHistoryFromEntries",`
`580`	`585`	`"buildInboundHistoryFromMap",`
`@@ -1421,6 +1426,19 @@ describe("plugin-sdk subpath exports", () => {`
`1421`	`1426`	`}`
`1422`	`1427`	`});`
`1423`	`1428`
	`1429`	`+it("keeps repeated silent-token semantics visible through the reply-chunking subpath", async () => {`
	`1430`	`+const replyChunkingSdk = await importResolvedPluginSdkSubpath(`
	`1431`	`+"openclaw/plugin-sdk/reply-chunking",`
	`1432`	`+);`
	`1433`	`+`
	`1434`	`+expect(replyChunkingSdk.isSilentReplyText("NO_REPLY\n\nNO_REPLY")).toBe(true);`
	`1435`	`+expect(replyChunkingSdk.isSilentReplyPayloadText("NO_REPLY\n\nNO_REPLY")).toBe(true);`
	`1436`	`+expect(replyChunkingSdk.isSilentReplyText("HEARTBEAT_OK\nHEARTBEAT_OK", "HEARTBEAT_OK")).toBe(`
	`1437`	`+true,`
	`1438`	`+);`
	`1439`	`+expect(replyChunkingSdk.isSilentReplyText("Visible update\n\nNO_REPLY")).toBe(false);`
	`1440`	`+});`
	`1441`	`+`
`1424`	`1442`	`it("keeps the Zalouser command-auth compatibility facade importable", () => {`
`1425`	`1443`	`expect(zalouserSdk.resolveSenderCommandAuthorization).toBe(`
`1426`	`1444`	`commandAuthSdk.resolveSenderCommandAuthorization,`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main