fix(feishu): bound btw out-of-band lanes

ngutman · ngutman · commit 9a9a99815430 · 2026-04-10T17:22:16.000+03:00
diff --git a/extensions/feishu/src/sequential-key.test.ts b/extensions/feishu/src/sequential-key.test.ts
@@ -30,7 +30,7 @@ describe("getFeishuSequentialKey", () => {
     [createTextEvent({ text: "hello" }), "feishu:default:oc_dm_chat"],
     [createTextEvent({ text: "/status" }), "feishu:default:oc_dm_chat"],
     [createTextEvent({ text: "/stop" }), "feishu:default:oc_dm_chat:control"],
-    [createTextEvent({ text: "/btw what changed?" }), "feishu:default:oc_dm_chat:btw:om_message_1"],
+    [createTextEvent({ text: "/btw what changed?" }), "feishu:default:oc_dm_chat:btw"],
   ])("resolves sequential key %#", (event, expected) => {
     expect(
       getFeishuSequentialKey({
@@ -40,6 +40,24 @@ describe("getFeishuSequentialKey", () => {
     ).toBe(expected);
   });
 
+  it("keeps /btw on a stable per-chat lane across different message ids", () => {
+    const first = createTextEvent({ text: "/btw one", messageId: "om_message_1" });
+    const second = createTextEvent({ text: "/btw two", messageId: "om_message_2" });
+
+    expect(
+      getFeishuSequentialKey({
+        accountId: "default",
+        event: first,
+      }),
+    ).toBe("feishu:default:oc_dm_chat:btw");
+    expect(
+      getFeishuSequentialKey({
+        accountId: "default",
+        event: second,
+      }),
+    ).toBe("feishu:default:oc_dm_chat:btw");
+  });
+
   it("falls back to a stable btw lane when the message id is unavailable", () => {
     const event = createTextEvent({ text: "/btw what changed?" });
     delete (event.message as { message_id?: string }).message_id;
diff --git a/extensions/feishu/src/sequential-key.ts b/extensions/feishu/src/sequential-key.ts
@@ -18,8 +18,7 @@ export function getFeishuSequentialKey(params: {
   }
 
   if (isBtwRequestText(text)) {
-    const messageId = event.message.message_id?.trim();
-    return messageId ? `${baseKey}:btw:${messageId}` : `${baseKey}:btw`;
+    return `${baseKey}:btw`;
   }
 
   return baseKey;

Original file line number	Diff line number	Diff line change
`@@ -18,8 +18,7 @@ export function getFeishuSequentialKey(params: {`
`18`	`18`	`}`
`19`	`19`
`20`	`20`	`if (isBtwRequestText(text)) {`
`21`		`- const messageId = event.message.message_id?.trim();`
`22`		- return messageId ? `${baseKey}:btw:${messageId}` : `${baseKey}:btw`;
	`21`	+ return `${baseKey}:btw`;
`23`	`22`	`}`
`24`	`23`
`25`	`24`	`return baseKey;`