refactor(imessage): tighten reflection guard coverage

Peter Steinberger · Peter Steinberger · commit 53eb0ed7c871 · 2026-06-30T12:22:04.000-07:00
diff --git a/extensions/imessage/src/monitor/reflection-guard.test.ts b/extensions/imessage/src/monitor/reflection-guard.test.ts
@@ -37,14 +37,11 @@ describe("detectReflectedContent", () => {
     expect(result.matchedLabels).toContain("thinking-tag");
   });
 
-  it("detects Anthropic namespaced <think> tags", () => {
-    const result = detectReflectedContent("<antml:think>secret</antml:think>visible");
-    expect(result.isReflection).toBe(true);
-    expect(result.matchedLabels).toContain("thinking-tag");
-  });
-
-  it("detects MiniMax namespaced <mm:think> tags", () => {
-    const result = detectReflectedContent("<mm:think>secret</mm:think>visible");
+  it.each([
+    { provider: "Anthropic", tag: "antml:think" },
+    { provider: "MiniMax", tag: "mm:think" },
+  ])("detects $provider namespaced reasoning tags", ({ tag }) => {
+    const result = detectReflectedContent(`<${tag}>secret</${tag}>visible`);
     expect(result.isReflection).toBe(true);
     expect(result.matchedLabels).toContain("thinking-tag");
   });
diff --git a/extensions/imessage/src/monitor/reflection-guard.ts b/extensions/imessage/src/monitor/reflection-guard.ts
@@ -4,11 +4,8 @@ import { findCodeRegions, isInsideCode } from "openclaw/plugin-sdk/text-chunking
 const INTERNAL_SEPARATOR_RE = /(?:#\+){2,}#?/;
 const ASSISTANT_ROLE_MARKER_RE = /\bassistant\s+to\s*=\s*\w+/i;
 // Require closing `>` to avoid false-positives on phrases like "<thought experiment>".
-// Reasoning tags may carry a model-specific namespace prefix (e.g. Anthropic's
-// `antml:`, MiniMax's `mm:`). Accept the known prefixes so namespaced variants
-// like `<mm:think>` are still detected as reflected assistant content instead of
-// being treated as normal inbound text (matches the shared reasoning-tag contract
-// in src/shared/text/reasoning-tags.ts).
+// Keep known provider namespaces aligned with the public reasoning-tag contract
+// so reflected hidden reasoning cannot re-enter the agent loop.
 const THINKING_TAG_RE =
   /<\s*\/?\s*(?:(?:antml:|mm:)?(?:think(?:ing)?|thought)|antthinking)\b[^<>]*>/i;
 const RELEVANT_MEMORIES_TAG_RE = /<\s*\/?\s*relevant[-_]memories\b[^<>]*>/i;
