惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

GbyAI
GbyAI
Martin Fowler
Martin Fowler
I
InfoQ
腾讯CDC
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
爱范儿
爱范儿
Microsoft Security Blog
Microsoft Security Blog
Google DeepMind News
Google DeepMind News
D
DataBreaches.Net
云风的 BLOG
云风的 BLOG
F
Fortinet All Blogs
N
Netflix TechBlog - Medium
博客园 - 聂微东
Microsoft Azure Blog
Microsoft Azure Blog
D
Docker
博客园 - 三生石上(FineUI控件)
Y
Y Combinator Blog
博客园 - Franky
Engineering at Meta
Engineering at Meta
B
Blog
罗磊的独立博客
Apple Machine Learning Research
Apple Machine Learning Research
Jina AI
Jina AI
V
Visual Studio Blog

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(imessage): strip leading echo corruption markers in t...
ly-wang19 · 2026-06-23 · via Recent Commits to openclaw:main

File tree

  • extensions/imessage/src/monitor

Original file line numberDiff line numberDiff line change

@@ -1,4 +1,5 @@

11

// Imessage plugin module implements echo cache behavior.

2+

import { stripLeadingEchoTextCorruptionMarkers } from "./echo-text-corruption.js";

23

import { hasPersistedIMessageEcho } from "./persisted-echo-cache.js";

34
45

type SentMessageLookup = {

@@ -35,20 +36,6 @@ export type SentMessageCache = {

3536

const SENT_MESSAGE_TEXT_TTL_MS = 4_000;

3637

const SENT_MESSAGE_ID_TTL_MS = 60_000;

3738
38-

function isLeadingEchoTextCorruptionMarker(code: number): boolean {

39-

return (

40-

code === 0x0000 || code === 0xfeff || code === 0xfffd || code === 0xfffe || code === 0xffff

41-

);

42-

}

43-
44-

function stripLeadingEchoTextCorruptionMarkers(text: string): string {

45-

let offset = 0;

46-

while (offset < text.length && isLeadingEchoTextCorruptionMarker(text.charCodeAt(offset))) {

47-

offset += 1;

48-

}

49-

return offset === 0 ? text : text.slice(offset);

50-

}

51-
5239

function normalizeEchoTextKey(text: string | undefined): string | null {

5340

if (!text) {

5441

return null;

Original file line numberDiff line numberDiff line change

@@ -0,0 +1,19 @@

1+

// Imessage plugin shared helper: strip leading attributedBody corruption markers from echo text.

2+

// The in-memory (echo-cache) and persisted (persisted-echo-cache) echo-dedupe paths must normalize

3+

// identically, so a reflected own-message echo whose attributedBody decoded with a leading

4+

// NUL/replacement marker still matches the clean stored send. Kept here (a leaf module with no

5+

// imports) so the persisted path can reuse it without an echo-cache <-> persisted-echo-cache cycle.

6+
7+

function isLeadingEchoTextCorruptionMarker(code: number): boolean {

8+

return (

9+

code === 0x0000 || code === 0xfeff || code === 0xfffd || code === 0xfffe || code === 0xffff

10+

);

11+

}

12+
13+

export function stripLeadingEchoTextCorruptionMarkers(text: string): string {

14+

let offset = 0;

15+

while (offset < text.length && isLeadingEchoTextCorruptionMarker(text.charCodeAt(offset))) {

16+

offset += 1;

17+

}

18+

return offset === 0 ? text : text.slice(offset);

19+

}

Original file line numberDiff line numberDiff line change

@@ -91,6 +91,21 @@ describe("iMessage sent-message echo cache", () => {

9191

expect(cache.has("acct:imessage:+1555", { text: "Delayed\u0000echo reply" })).toBe(false);

9292

});

9393
94+

it("matches a delayed reflected echo with leading corruption markers via the persisted cache", () => {

95+

// The persisted 12h cache is the only matcher once the 4s in-memory text TTL expires, so it

96+

// must strip leading attributedBody corruption markers exactly like the in-memory key (#93511).

97+

const scope = "acct:imessage:+1555";

98+

const markers = String.fromCharCode(0x0000, 0xfffd, 0xfffe, 0xffff, 0xfeff);

99+

rememberPersistedIMessageEcho({ scope, text: "Delayed echo reply" });

100+
101+

expect(hasPersistedIMessageEcho({ scope, text: "Delayed echo reply" })).toBe(true);

102+

expect(hasPersistedIMessageEcho({ scope, text: `${markers}Delayed echo reply` })).toBe(true);

103+

// Leading-only: a mid-string marker stays distinct.

104+

expect(

105+

hasPersistedIMessageEcho({ scope, text: `Delayed${String.fromCharCode(0x0000)}echo reply` }),

106+

).toBe(false);

107+

});

108+
94109

it("matches by outbound message id and ignores placeholder ids", () => {

95110

vi.useFakeTimers();

96111

vi.setSystemTime(new Date("2026-02-25T00:00:00Z"));

Original file line numberDiff line numberDiff line change

@@ -3,6 +3,7 @@ import { createHash } from "node:crypto";

33

import type { PluginStateSyncKeyedStore } from "openclaw/plugin-sdk/plugin-state-runtime";

44

import { logVerbose } from "openclaw/plugin-sdk/runtime-env";

55

import { getIMessageRuntime } from "../runtime.js";

6+

import { stripLeadingEchoTextCorruptionMarkers } from "./echo-text-corruption.js";

67
78

type PersistedEchoEntry = {

89

scope: string;

@@ -26,7 +27,14 @@ export const IMESSAGE_SENT_ECHOES_MAX_ENTRIES = 256;

2627

type PersistedEchoStore = PluginStateSyncKeyedStore<PersistedEchoEntry>;

2728
2829

function normalizeText(text: string | undefined): string | undefined {

29-

const normalized = text?.replace(/\r\n?/g, "\n").trim();

30+

if (!text) {

31+

return undefined;

32+

}

33+

// Match the in-memory echo-cache key so a reflected echo with a leading attributedBody

34+

// corruption marker still matches the clean stored send (the persisted sibling of #93511).

35+

const normalized = stripLeadingEchoTextCorruptionMarkers(

36+

text.replace(/\r\n?/g, "\n").trim(),

37+

).trim();

3038

return normalized || undefined;

3139

}

3240