fix(agents): recognize bare-ok broadcast sends · openclaw...

Recent Commits to openclaw:main

vincentkoc · 2026-06-16 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -205,6 +205,21 @@ describe("isDeliveredMessagingToolResult", () => {`
`205`	`205`	`result: { results: [{ channel: "googlechat", to: "space-1", ok: true }] },`
`206`	`206`	`}),`
`207`	`207`	`).toBe(false);`
	`208`	`+expect(`
	`209`	`+isDeliveredMessagingToolResult({`
	`210`	`+args: { action: "broadcast" },`
	`211`	`+result: {`
	`212`	`+results: [`
	`213`	`+{`
	`214`	`+channel: "googlechat",`
	`215`	`+to: "space-1",`
	`216`	`+ok: true,`
	`217`	`+payload: { ok: true, to: "spaces/AAA" },`
	`218`	`+},`
	`219`	`+],`
	`220`	`+},`
	`221`	`+}),`
	`222`	`+).toBe(true);`
`208`	`223`	`});`
`209`	`224`
`210`	`225`	`it("rejects successful broadcast wrappers around suppressed sends", () => {`
`@@ -222,6 +237,15 @@ describe("isDeliveredMessagingToolResult", () => {`
`222`	`237`	`result: { results: [{ ok: true, result: { deliveryStatus: "suppressed" } }] },`
`223`	`238`	`}),`
`224`	`239`	`).toBe(false);`
	`240`	`+expect(`
	`241`	`+isDeliveredMessagingToolResult({`
	`242`	`+toolName: "message",`
	`243`	`+args: { action: "broadcast" },`
	`244`	`+result: {`
	`245`	`+results: [{ ok: true, payload: { ok: true, deliveryStatus: "suppressed" } }],`
	`246`	`+},`
	`247`	`+}),`
	`248`	`+).toBe(false);`
`225`	`249`	`});`
`226`	`250`
`227`	`251`	`it("accepts failed broadcast entries with partial-delivery evidence", () => {`

Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,7 @@ const RESULT_ENVELOPE_KEYS = [`
`24`	`24`	`"sendResult",`
`25`	`25`	`"toolResult",`
`26`	`26`	`];`
	`27`	`+const BROADCAST_SEND_ENVELOPE_KEYS = ["payload", "result", "sendResult", "toolResult"];`
`27`	`28`	`const PARTIAL_DELIVERY_ENVELOPE_KEYS = [...RESULT_ENVELOPE_KEYS, "error", "cause"];`
`28`	`29`	`const SESSIONS_SEND_DELIVERY_STATUSES = new Set(["accepted", "ok"]);`
`29`	`30`	`const CONVERSATION_CREATE_ACTIONS = new Set([`
`@@ -233,6 +234,20 @@ function deliveryEnvelopeIndicatesNoOp(value: unknown, depth = 0): boolean {`
`233`	`234`	`return RESULT_ENVELOPE_KEYS.some((key) => deliveryEnvelopeIndicatesNoOp(record[key], depth + 1));`
`234`	`235`	`}`
`235`	`236`
	`237`	`+function broadcastEntryHasSuccessfulBareOkSend(`
	`238`	`+record: Record<string, unknown>,`
	`239`	`+depth: number,`
	`240`	`+): boolean {`
	`241`	`+return BROADCAST_SEND_ENVELOPE_KEYS.some((key) => {`
	`242`	`+const value = record[key];`
	`243`	`+return (`
	`244`	`+deliveryEnvelopeIndicatesOk(value, depth + 1) &&`
	`245`	`+!deliveryEnvelopeIndicatesNonDelivery(value, depth + 1) &&`
	`246`	`+!deliveryEnvelopeIndicatesNoOp(value, depth + 1)`
	`247`	`+);`
	`248`	`+});`
	`249`	`+}`
	`250`	`+`
`236`	`251`	`function deliveryEnvelopeIndicatesSuccessfulBroadcast(value: unknown, depth = 0): boolean {`
`237`	`252`	`if (!value \|\| typeof value !== "object" \|\| depth > 4) {`
`238`	`253`	`return false;`
`@@ -246,7 +261,8 @@ function deliveryEnvelopeIndicatesSuccessfulBroadcast(value: unknown, depth = 0)`
`246`	`261`	`(item as Record<string, unknown>).ok === true &&`
`247`	`262`	`!deliveryEnvelopeIndicatesNonDelivery(item) &&`
`248`	`263`	`!deliveryEnvelopeIndicatesNoOp(item) &&`
`249`		`-deliveryEnvelopeIndicatesDelivered(item, depth + 1),`
	`264`	`+(deliveryEnvelopeIndicatesDelivered(item, depth + 1) \|\|`
	`265`	`+broadcastEntryHasSuccessfulBareOkSend(item as Record<string, unknown>, depth + 1)),`
`250`	`266`	`);`
`251`	`267`	`}`
`252`	`268`	`const record = value as Record<string, unknown>;`

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

推荐订阅源

Recent Commits to openclaw:main