fix(zalo): treat chat ids as opaque strings

goutamadwant · openclaw-clownfish[bot] · commit e90bdb95bb69 · 2026-06-17T17:01:26.000Z
diff --git a/extensions/zalo/src/channel.directory.test.ts b/extensions/zalo/src/channel.directory.test.ts
@@ -55,16 +55,20 @@ describe("zalo directory", () => {
     expect(zaloPlugin.messaging?.normalizeTarget?.("  zl:234  ")).toBe("234");
   });
 
-  it("recognizes numeric and hex-like Bot API chat ids as direct targets", () => {
+  it("recognizes opaque Bot API chat ids as direct targets", () => {
     const looksLikeId = zaloPlugin.messaging?.targetResolver?.looksLikeId;
     if (!looksLikeId) {
       throw new Error("expected Zalo target resolver");
     }
 
     expect(looksLikeId("123456", "123456")).toBe(true);
     expect(looksLikeId("3becaa50ae12474c1e03", "3becaa50ae12474c1e03")).toBe(true);
+    expect(looksLikeId("abc.xyz", "abc.xyz")).toBe(true);
     expect(looksLikeId("zalo:49270a5f8f1c66423f0d", "49270a5f8f1c66423f0d")).toBe(true);
+    expect(looksLikeId("zalo:abc.xyz", "abc.xyz")).toBe(true);
     expect(looksLikeId("zl:3becaa50ae12474c1e03", "3becaa50ae12474c1e03")).toBe(true);
-    expect(looksLikeId("support", "support")).toBe(false);
+    expect(looksLikeId("zl:abc.xyz", "abc.xyz")).toBe(true);
+    expect(looksLikeId("support", "support")).toBe(true);
+    expect(looksLikeId("zalo:  ", "")).toBe(false);
   });
 });
diff --git a/extensions/zalo/src/channel.ts b/extensions/zalo/src/channel.ts
@@ -71,11 +71,9 @@ function normalizeZaloMessagingTarget(raw: string): string | undefined {
   return trimmed.replace(/^(zalo|zl):/i, "").trim();
 }
 
-const zaloChatIdPattern = /^(?:\d{3,}|(?=[a-f0-9]{12,}$)(?=.*[a-f])[a-f0-9]+)$/i;
-
 function looksLikeZaloChatId(raw: string, normalized?: string): boolean {
   const target = normalizeZaloMessagingTarget(normalized ?? raw);
-  return target ? zaloChatIdPattern.test(target) : false;
+  return Boolean(target);
 }
 
 const loadZaloChannelRuntime = createLazyRuntimeModule(() => import("./channel.runtime.js"));

Original file line number	Diff line number	Diff line change
`@@ -71,11 +71,9 @@ function normalizeZaloMessagingTarget(raw: string): string \| undefined {`
`71`	`71`	`return trimmed.replace(/^(zalo\|zl):/i, "").trim();`
`72`	`72`	`}`
`73`	`73`
`74`		`-const zaloChatIdPattern = /^(?:\d{3,}\|(?=[a-f0-9]{12,}$)(?=.*[a-f])[a-f0-9]+)$/i;`
`75`		`-`
`76`	`74`	`function looksLikeZaloChatId(raw: string, normalized?: string): boolean {`
`77`	`75`	`const target = normalizeZaloMessagingTarget(normalized ?? raw);`
`78`		`- return target ? zaloChatIdPattern.test(target) : false;`
	`76`	`+ return Boolean(target);`
`79`	`77`	`}`
`80`	`78`
`81`	`79`	`const loadZaloChannelRuntime = createLazyRuntimeModule(() => import("./channel.runtime.js"));`