fix(chat): use adapter to parse channel id (#286)

* fix(chat): use adapter to parse channel id

* test: fix cases use adapter to parse channel id

* chore: update changeset

Author: yjl9903

.changeset/thin-sloths-joke.md

@@ -0,0 +1,5 @@
+---
+"chat": patch
+---
+
+Use adapter to parse channel id for `thread.channelId`

packages/chat/src/chat.ts

@@ -2077,10 +2077,8 @@ export class Chat<
     initialMessage: Message,
     isSubscribedContext = false
   ): Thread<TState> {
-    // Parse thread ID to get channel info
-    // Format: "adapter:channel:thread"
-    const parts = threadId.split(":");
-    const channelId = parts[1] || "";
+    // Parse thread ID to get channel ID with adapter
+    const channelId = adapter.channelIdFromThreadId(threadId);
 
     // Check if this is a DM
     const isDM = adapter.isDM?.(threadId) ?? false;

packages/integration-tests/src/replay-actions-reactions.test.ts

@@ -70,7 +70,7 @@ describe("Replay Tests - Actions & Reactions", () => {
         userId: "U00FAKEUSER1",
         userName: "testuser",
         adapterName: "slack",
-        channelId: "C00FAKECHAN1",
+        channelId: "slack:C00FAKECHAN1",
         isDM: false,
       });
 
@@ -91,7 +91,7 @@ describe("Replay Tests - Actions & Reactions", () => {
         added: true,
         userId: "U00FAKEUSER1",
         adapterName: "slack",
-        channelId: "C00FAKECHAN1",
+        channelId: "slack:C00FAKECHAN1",
         messageId: "1767326126.896109",
         isDM: false,
       });
@@ -110,7 +110,7 @@ describe("Replay Tests - Actions & Reactions", () => {
         userId: "U00FAKEUSER1",
         userName: "testuser",
         adapterName: "slack",
-        channelId: "C00FAKECHAN1",
+        channelId: "slack:C00FAKECHAN1",
         isDM: false,
       });
 
@@ -129,7 +129,7 @@ describe("Replay Tests - Actions & Reactions", () => {
         userId: "U00FAKEUSER1",
         userName: "testuser",
         adapterName: "slack",
-        channelId: "C00FAKECHAN1",
+        channelId: "slack:C00FAKECHAN1",
         isDM: false,
       });
 

packages/integration-tests/src/replay-channel.test.ts

@@ -116,7 +116,7 @@ describe("Replay Tests - Channel", () => {
         userId: "U00FAKEUSER1",
         userName: "testuser",
         adapterName: "slack",
-        channelId: "C00FAKECHAN1",
+        channelId: "slack:C00FAKECHAN1",
         isDM: false,
       });
     });

packages/integration-tests/src/replay-modal-private-metadata.test.ts

@@ -64,7 +64,7 @@ describe("Replay Tests - Modal privateMetadata", () => {
         userId: "U0A8WUV28QM",
         userName: "sd0a90bkva4s_user",
         adapterName: "slack",
-        channelId: "C0A9D9RTBMF",
+        channelId: "slack:C0A9D9RTBMF",
         isDM: false,
       });
 

packages/integration-tests/src/replay-modals.test.ts

@@ -80,7 +80,7 @@ describe("Replay Tests - Modals", () => {
         userId: "U00FAKEUSER2",
         userName: "jane.smith",
         adapterName: "slack",
-        channelId: "C00FAKECHAN2",
+        channelId: "slack:C00FAKECHAN2",
         isDM: false,
       });
 
@@ -254,7 +254,7 @@ describe("Replay Tests - Modals", () => {
       expect(capturedAction?.threadId).toBe(
         "slack:C00FAKECHAN3:1771126602.612659"
       );
-      expect(capturedAction?.thread?.channelId).toBe("C00FAKECHAN3");
+      expect(capturedAction?.thread?.channelId).toBe("slack:C00FAKECHAN3");
     });
 
     it("should allow editing relatedMessage from ephemeral modal submission", async () => {