agent-messenger 2.37.1 → 2.38.0

package/src/platforms/discord/index.ts

@@ -8,6 +8,7 @@ export type {
   DiscordConfig,
   DiscordCredentials,
   DiscordDMChannel,
+  DiscordEmoji,
   DiscordFile,
   DiscordGatewayChannelEvent,
   DiscordGatewayEvent,
@@ -28,6 +29,7 @@ export type {
   DiscordReadState,
   DiscordReadStateSnapshot,
   DiscordRelationship,
+  DiscordSticker,
   DiscordRole,
   DiscordRoleColors,
   DiscordRoleTags,

package/src/platforms/discord/types.ts

@@ -8,7 +8,14 @@ export interface DiscordGuild {
   id: string
   name: string
   icon?: string
+  /** Only present on GET /users/@me/guilds. */
   owner?: boolean
+  /** Only present on GET /guilds/{id}. */
+  owner_id?: string
+  premium_tier?: number
+  premium_subscription_count?: number
+  emojis?: DiscordEmoji[]
+  stickers?: DiscordSticker[]
 }
 
 export interface DiscordChannel {
@@ -116,6 +123,30 @@ export interface DiscordFile {
   width?: number
 }
 
+export interface DiscordEmoji {
+  id: string
+  name: string
+  animated?: boolean
+  managed?: boolean
+  available?: boolean
+  roles?: string[]
+  user?: DiscordUser
+}
+
+export interface DiscordSticker {
+  id: string
+  name: string
+  description?: string | null
+  tags: string
+  /** STANDARD = 1, GUILD = 2. */
+  type: number
+  /** PNG = 1, APNG = 2, LOTTIE = 3, GIF = 4. */
+  format_type: number
+  available?: boolean
+  guild_id?: string
+  user?: DiscordUser
+}
+
 export interface DiscordMention {
   id: string
   channel_id: string

package/src/platforms/kakaotalk/client.test.ts

@@ -763,6 +763,72 @@ describe('KakaoTalkClient', () => {
       client.close()
     })
 
+    it('advances LCHATLIST pagination when cursor fields are plain numbers', async () => {
+      // LOCO sends lastTokenId/lastChatId as plain numbers when they fit in a safe
+      // integer (bson promotes them). The mocked session ignores its arguments, so
+      // asserting only on the collected chats would pass even while every request
+      // re-sent cursor 0. Assert on the cursor that actually reaches getChatList.
+      mockLogin.mockResolvedValue({ ...DEFAULT_LOGIN_RESULT, lastTokenId: 7, lastChatId: 200, eof: false })
+      mockGetChatList
+        .mockResolvedValueOnce({
+          statusCode: 0,
+          body: {
+            status: 0,
+            chatDatas: [
+              {
+                c: 300,
+                t: 1,
+                k: ['Dave'],
+                i: [4],
+                a: 1,
+                n: 0,
+                o: 1698000000,
+                l: null,
+                ll: makeLong(100),
+              },
+            ],
+            lastTokenId: 8,
+            lastChatId: 300,
+            eof: false,
+          },
+        })
+        .mockResolvedValueOnce({
+          statusCode: 0,
+          body: {
+            status: 0,
+            chatDatas: [
+              {
+                c: 400,
+                t: 1,
+                k: ['Erin'],
+                i: [5],
+                a: 1,
+                n: 0,
+                o: 1697000000,
+                l: null,
+                ll: makeLong(90),
+              },
+            ],
+            lastTokenId: 9,
+            lastChatId: 400,
+            eof: true,
+          },
+        })
+
+      const client = await new KakaoTalkClient().login({ oauthToken: 'token', userId: 'user1', deviceUuid: 'device1' })
+      const chats = await client.getChats({ all: true })
+
+      expect(chats.map((chat) => chat.chat_id)).toEqual(['100', '200', '300', '400'])
+      expect(mockGetChatList).toHaveBeenCalledTimes(2)
+      const cursors = mockGetChatList.mock.calls.map((call) => (call as unknown[]).map((v) => String(v)))
+      expect(cursors).toEqual([
+        ['7', '200'],
+        ['8', '300'],
+      ])
+
+      client.close()
+    })
+
     it('fails closed when LCHATLIST has a nonzero response status', async () => {
       mockLogin.mockResolvedValue({ ...DEFAULT_LOGIN_RESULT, eof: false })
       mockGetChatList.mockResolvedValueOnce({

package/src/platforms/kakaotalk/client.ts

@@ -163,6 +163,18 @@ function bsonToLong(v: unknown): Long | undefined {
   return undefined
 }
 
+// LCHATLIST cursor fields (lastTokenId/lastChatId) arrive as plain numbers when the
+// value fits in a safe integer, because bson promotes them, and as { low, high }
+// otherwise. bsonToLong only accepts the object form, so a numeric cursor became
+// undefined and getChatList() re-requested the first page until MAX_PAGES.
+// Kept separate from bsonToLong on purpose: its other callers (log ids, open link
+// ids) rely on a numeric 0 staying absent rather than becoming a Long.
+function cursorToLong(v: unknown): Long | undefined {
+  if (typeof v === 'number' && Number.isSafeInteger(v)) return Long.fromNumber(v)
+  if (typeof v === 'bigint') return Long.fromBigInt(v)
+  return bsonToLong(v)
+}
+
 function longToString(v: unknown): string {
   if (v && typeof v === 'object' && 'high' in v && 'low' in v) {
     const { high, low } = v as { high: number; low: number }
@@ -1049,8 +1061,8 @@ export class KakaoTalkClient {
             if (cursor.eof && !snapshotEmpty) break
             if (cursor.eof && snapshotEmpty && pages > 0) break
 
-            const lastTokenId = bsonToLong(cursor.lastTokenId)
-            const lastChatId = bsonToLong(cursor.lastChatId)
+            const lastTokenId = cursorToLong(cursor.lastTokenId)
+            const lastChatId = cursorToLong(cursor.lastChatId)
 
             const response = await session.getChatList(lastTokenId, lastChatId)
             if (requireCompleteList) assertLocoOk(response, 'LCHATLIST')