agent-messenger 1.14.0 → 2.0.0

package/src/platforms/slack/types.ts

@@ -150,6 +150,170 @@ export interface SlackChannelSection {
   date_updated: number
 }
 
+export interface SlackPin {
+  channel: string
+  message: SlackMessage
+  date_created: number
+  created_by: string
+}
+
+export interface SlackBookmark {
+  id: string
+  channel_id: string
+  title: string
+  link: string
+  emoji?: string
+  icon_url?: string
+  type: string
+  date_created: number
+  date_updated: number
+  created_by: string
+}
+
+export interface SlackScheduledMessage {
+  id: string
+  channel_id: string
+  post_at: number
+  date_created: number
+  text: string
+}
+
+export interface SlackReminder {
+  id: string
+  creator: string
+  text: string
+  user: string
+  recurring: boolean
+  time: number
+  complete_ts: number
+}
+
+export interface SlackUserProfile {
+  title?: string
+  phone?: string
+  skype?: string
+  real_name?: string
+  real_name_normalized?: string
+  display_name?: string
+  display_name_normalized?: string
+  status_text?: string
+  status_emoji?: string
+  status_expiration?: number
+  email?: string
+  first_name?: string
+  last_name?: string
+  image_24?: string
+  image_32?: string
+  image_48?: string
+  image_72?: string
+  image_192?: string
+  image_512?: string
+}
+
+export interface SlackUsergroup {
+  id: string
+  team_id: string
+  name: string
+  handle: string
+  description: string
+  is_external: boolean
+  is_usergroup: boolean
+  date_create: number
+  date_update: number
+  date_delete: number
+  auto_type: string | null
+  created_by: string
+  updated_by: string
+  deleted_by: string | null
+  prefs: {
+    channels: string[]
+    groups: string[]
+  }
+  users: string[]
+  user_count: number
+}
+
+// RTM event types
+
+export interface SlackRTMMessageEvent {
+  type: 'message'
+  subtype?: string
+  channel: string
+  user?: string
+  text?: string
+  ts: string
+  thread_ts?: string
+  edited?: { user: string; ts: string }
+  hidden?: boolean
+}
+
+export interface SlackRTMReactionEvent {
+  type: 'reaction_added' | 'reaction_removed'
+  user: string
+  reaction: string
+  item: { type: string; channel: string; ts: string }
+  item_user?: string
+  event_ts: string
+}
+
+export interface SlackRTMMemberEvent {
+  type: 'member_joined_channel' | 'member_left_channel'
+  user: string
+  channel: string
+  channel_type?: string
+  event_ts?: string
+}
+
+export interface SlackRTMChannelEvent {
+  type: 'channel_created' | 'channel_deleted' | 'channel_rename' | 'channel_archive' | 'channel_unarchive'
+  channel: { id: string; name?: string } | string
+}
+
+export interface SlackRTMPresenceEvent {
+  type: 'presence_change'
+  user: string
+  presence: 'active' | 'away'
+}
+
+export interface SlackRTMUserTypingEvent {
+  type: 'user_typing'
+  channel: string
+  user: string
+}
+
+export interface SlackRTMGenericEvent {
+  type: string
+  [key: string]: unknown
+}
+
+export type SlackRTMEvent =
+  | SlackRTMMessageEvent
+  | SlackRTMReactionEvent
+  | SlackRTMMemberEvent
+  | SlackRTMChannelEvent
+  | SlackRTMPresenceEvent
+  | SlackRTMUserTypingEvent
+  | SlackRTMGenericEvent
+
+export interface SlackListenerEventMap {
+  message: [event: SlackRTMMessageEvent]
+  reaction_added: [event: SlackRTMReactionEvent]
+  reaction_removed: [event: SlackRTMReactionEvent]
+  member_joined_channel: [event: SlackRTMMemberEvent]
+  member_left_channel: [event: SlackRTMMemberEvent]
+  presence_change: [event: SlackRTMPresenceEvent]
+  user_typing: [event: SlackRTMUserTypingEvent]
+  channel_created: [event: SlackRTMChannelEvent]
+  channel_deleted: [event: SlackRTMChannelEvent]
+  channel_rename: [event: SlackRTMChannelEvent]
+  channel_archive: [event: SlackRTMChannelEvent]
+  channel_unarchive: [event: SlackRTMChannelEvent]
+  slack_event: [event: SlackRTMGenericEvent]
+  connected: [info: { self: { id: string }; team: { id: string } }]
+  disconnected: []
+  error: [error: Error]
+}
+
 export interface WorkspaceCredentials {
   workspace_id: string
   workspace_name: string
@@ -244,6 +408,29 @@ export const SlackUserSchema = z.object({
     .optional(),
 })
 
+export const SlackUsergroupSchema = z.object({
+  id: z.string(),
+  team_id: z.string(),
+  name: z.string(),
+  handle: z.string(),
+  description: z.string(),
+  is_external: z.boolean(),
+  is_usergroup: z.boolean(),
+  date_create: z.number(),
+  date_update: z.number(),
+  date_delete: z.number(),
+  auto_type: z.string().nullable(),
+  created_by: z.string(),
+  updated_by: z.string(),
+  deleted_by: z.string().nullable(),
+  prefs: z.object({
+    channels: z.array(z.string()),
+    groups: z.array(z.string()),
+  }),
+  users: z.array(z.string()),
+  user_count: z.number(),
+})
+
 export const WorkspaceCredentialsSchema = z.object({
   workspace_id: z.string(),
   workspace_name: z.string(),

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

@@ -122,33 +122,33 @@ describe('SlackBotClient', () => {
     mockUsers.info.mockClear()
   })
 
-  describe('constructor', () => {
-    test('accepts bot tokens (xoxb-)', () => {
+  describe('login', () => {
+    test('accepts bot tokens (xoxb-)', async () => {
       // when/then: should not throw
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
       expect(client).toBeDefined()
     })
 
-    test('rejects user tokens (xoxp-)', () => {
+    test('rejects user tokens (xoxp-)', async () => {
       // when/then
-      expect(() => new SlackBotClient('xoxp-user-token')).toThrow(SlackBotError)
+      await expect(new SlackBotClient().login({ token: 'xoxp-user-token' })).rejects.toThrow(SlackBotError)
     })
 
-    test('rejects empty token', () => {
+    test('rejects empty token', async () => {
       // when/then
-      expect(() => new SlackBotClient('')).toThrow(SlackBotError)
+      await expect(new SlackBotClient().login({ token: '' })).rejects.toThrow(SlackBotError)
     })
 
-    test('rejects non-bot tokens', () => {
+    test('rejects non-bot tokens', async () => {
       // when/then
-      expect(() => new SlackBotClient('invalid-token')).toThrow(SlackBotError)
+      await expect(new SlackBotClient().login({ token: 'invalid-token' })).rejects.toThrow(SlackBotError)
     })
   })
 
   describe('testAuth', () => {
     test('returns auth info for valid token', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const result = await client.testAuth()
@@ -163,7 +163,7 @@ describe('SlackBotClient', () => {
   describe('postMessage', () => {
     test('sends message and returns timestamp', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const result = await client.postMessage('C123', 'Hello')
@@ -177,7 +177,7 @@ describe('SlackBotClient', () => {
   describe('getConversationHistory', () => {
     test('returns messages', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const messages = await client.getConversationHistory('C123')
@@ -191,7 +191,7 @@ describe('SlackBotClient', () => {
   describe('getMessage', () => {
     test('returns single message', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const message = await client.getMessage('C123', '1234567890.123456')
@@ -205,7 +205,7 @@ describe('SlackBotClient', () => {
   describe('addReaction', () => {
     test('adds reaction to message', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when/then: should not throw
       await client.addReaction('C123', '1234567890.123456', 'thumbsup')
@@ -216,7 +216,7 @@ describe('SlackBotClient', () => {
   describe('removeReaction', () => {
     test('removes reaction from message', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when/then: should not throw
       await client.removeReaction('C123', '1234567890.123456', 'thumbsup')
@@ -227,7 +227,7 @@ describe('SlackBotClient', () => {
   describe('listChannels', () => {
     test('returns list of channels', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const channels = await client.listChannels()
@@ -242,7 +242,7 @@ describe('SlackBotClient', () => {
   describe('getChannelInfo', () => {
     test('returns channel details', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const channel = await client.getChannelInfo('C123')
@@ -256,7 +256,7 @@ describe('SlackBotClient', () => {
   describe('resolveChannel', () => {
     test('returns channel ID unchanged when it starts with C', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const channel = await client.resolveChannel('C123ABC')
@@ -268,7 +268,7 @@ describe('SlackBotClient', () => {
 
     test('returns channel ID unchanged when it starts with D', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const channel = await client.resolveChannel('D123ABC')
@@ -280,7 +280,7 @@ describe('SlackBotClient', () => {
 
     test('returns channel ID unchanged when it starts with G', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const channel = await client.resolveChannel('G123ABC')
@@ -292,7 +292,7 @@ describe('SlackBotClient', () => {
 
     test('resolves channel name to ID', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const channel = await client.resolveChannel('general')
@@ -304,7 +304,7 @@ describe('SlackBotClient', () => {
 
     test('strips leading # from channel name', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const channel = await client.resolveChannel('#general')
@@ -316,7 +316,7 @@ describe('SlackBotClient', () => {
 
     test('returns channel ID unchanged when input is #C prefixed ID', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const channel = await client.resolveChannel('#C123ABC')
@@ -327,7 +327,7 @@ describe('SlackBotClient', () => {
 
     test('throws channel_not_found error when name is not found', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when/then
       try {
@@ -343,7 +343,7 @@ describe('SlackBotClient', () => {
   describe('listUsers', () => {
     test('returns list of users', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const users = await client.listUsers()
@@ -357,7 +357,7 @@ describe('SlackBotClient', () => {
   describe('getUserInfo', () => {
     test('returns user details', async () => {
       // given
-      const client = new SlackBotClient('xoxb-test-token')
+      const client = await new SlackBotClient().login({ token: 'xoxb-test-token' })
 
       // when
       const user = await client.getUserInfo('U123')

package/src/platforms/slackbot/client.ts

@@ -1,22 +1,40 @@
 import { WebClient } from '@slack/web-api'
 
+import { SlackBotCredentialManager } from './credential-manager'
 import { SlackBotError, type SlackChannel, type SlackMessage, type SlackUser } from './types'
 
 const MAX_RETRIES = 3
 const RATE_LIMIT_ERROR_CODE = 'slack_webapi_rate_limited_error'
 
 export class SlackBotClient {
-  private client: WebClient
+  private client: WebClient | null = null
 
-  constructor(token: string) {
-    if (!token) {
-      throw new SlackBotError('Token is required', 'missing_token')
-    }
-    if (!token.startsWith('xoxb-')) {
-      throw new SlackBotError('Token must be a bot token (xoxb-)', 'invalid_token_type')
+  async login(credentials?: { token: string }): Promise<this> {
+    if (credentials) {
+      const { token } = credentials
+      if (!token) {
+        throw new SlackBotError('Token is required', 'missing_token')
+      }
+      if (!token.startsWith('xoxb-')) {
+        throw new SlackBotError('Token must be a bot token (xoxb-)', 'invalid_token_type')
+      }
+      this.client = new WebClient(token)
+    } else {
+      const credManager = new SlackBotCredentialManager()
+      const creds = await credManager.getCredentials()
+      if (!creds) {
+        throw new SlackBotError('No credentials configured. Run "auth set <token>" first.', 'not_authenticated')
+      }
+      await this.login({ token: creds.token })
     }
+    return this
+  }
 
-    this.client = new WebClient(token)
+  private ensureAuth(): WebClient {
+    if (!this.client) {
+      throw new SlackBotError('Not authenticated. Call .login() first.', 'not_authenticated')
+    }
+    return this.client
   }
 
   private async withRetry<T>(operation: () => Promise<T>): Promise<T> {
@@ -55,7 +73,7 @@ export class SlackBotClient {
     team?: string
   }> {
     return this.withRetry(async () => {
-      const response = await this.client.auth.test()
+      const response = await this.ensureAuth().auth.test()
       this.checkResponse(response)
       return {
         user_id: response.user_id!,
@@ -69,7 +87,7 @@ export class SlackBotClient {
 
   async postMessage(channel: string, text: string, options?: { thread_ts?: string }): Promise<SlackMessage> {
     return this.withRetry(async () => {
-      const response = await this.client.chat.postMessage({
+      const response = await this.ensureAuth().chat.postMessage({
         channel,
         text,
         thread_ts: options?.thread_ts,
@@ -92,7 +110,7 @@ export class SlackBotClient {
     options?: { limit?: number; cursor?: string },
   ): Promise<SlackMessage[]> {
     return this.withRetry(async () => {
-      const response = await this.client.conversations.history({
+      const response = await this.ensureAuth().conversations.history({
         channel,
         limit: options?.limit || 20,
         cursor: options?.cursor,
@@ -131,7 +149,7 @@ export class SlackBotClient {
 
   async getMessage(channel: string, ts: string): Promise<SlackMessage | null> {
     return this.withRetry(async () => {
-      const response = await this.client.conversations.history({
+      const response = await this.ensureAuth().conversations.history({
         channel,
         oldest: ts,
         inclusive: true,
@@ -179,7 +197,7 @@ export class SlackBotClient {
     const normalizedEmoji = emoji.replace(/^:|:$/g, '')
 
     return this.withRetry(async () => {
-      const response = await this.client.reactions.add({
+      const response = await this.ensureAuth().reactions.add({
         channel,
         timestamp,
         name: normalizedEmoji,
@@ -193,7 +211,7 @@ export class SlackBotClient {
     const normalizedEmoji = emoji.replace(/^:|:$/g, '')
 
     return this.withRetry(async () => {
-      const response = await this.client.reactions.remove({
+      const response = await this.ensureAuth().reactions.remove({
         channel,
         timestamp,
         name: normalizedEmoji,
@@ -209,7 +227,7 @@ export class SlackBotClient {
     do {
       // Only wrap individual API call in withRetry, not the entire loop
       const response = await this.withRetry(async () => {
-        const res = await this.client.conversations.list({
+        const res = await this.ensureAuth().conversations.list({
           cursor,
           limit: options?.limit || 200,
           types: 'public_channel,private_channel',
@@ -255,7 +273,7 @@ export class SlackBotClient {
 
   async getChannelInfo(channel: string): Promise<SlackChannel> {
     return this.withRetry(async () => {
-      const response = await this.client.conversations.info({ channel })
+      const response = await this.ensureAuth().conversations.info({ channel })
       this.checkResponse(response)
 
       const ch = response.channel!
@@ -312,7 +330,7 @@ export class SlackBotClient {
     do {
       // Only wrap individual API call in withRetry, not the entire loop
       const response = await this.withRetry(async () => {
-        const res = await this.client.users.list({
+        const res = await this.ensureAuth().users.list({
           cursor,
           limit: options?.limit || 200,
         })
@@ -352,7 +370,7 @@ export class SlackBotClient {
 
   async getUserInfo(userId: string): Promise<SlackUser> {
     return this.withRetry(async () => {
-      const response = await this.client.users.info({ user: userId })
+      const response = await this.ensureAuth().users.info({ user: userId })
       this.checkResponse(response)
 
       const member = response.user!
@@ -378,7 +396,7 @@ export class SlackBotClient {
 
   async updateMessage(channel: string, ts: string, text: string): Promise<SlackMessage> {
     return this.withRetry(async () => {
-      const response = await this.client.chat.update({ channel, ts, text })
+      const response = await this.ensureAuth().chat.update({ channel, ts, text })
       this.checkResponse(response)
       const msg = (response as any).message
       return {
@@ -396,7 +414,7 @@ export class SlackBotClient {
     options?: { limit?: number; cursor?: string },
   ): Promise<SlackMessage[]> {
     return this.withRetry(async () => {
-      const response = await this.client.conversations.replies({
+      const response = await this.ensureAuth().conversations.replies({
         channel,
         ts,
         limit: options?.limit || 100,
@@ -435,14 +453,14 @@ export class SlackBotClient {
 
   async joinChannel(channel: string): Promise<void> {
     return this.withRetry(async () => {
-      const response = await this.client.conversations.join({ channel })
+      const response = await this.ensureAuth().conversations.join({ channel })
       this.checkResponse(response)
     })
   }
 
   async deleteMessage(channel: string, ts: string): Promise<void> {
     return this.withRetry(async () => {
-      const response = await this.client.chat.delete({ channel, ts })
+      const response = await this.ensureAuth().chat.delete({ channel, ts })
       this.checkResponse(response)
     })
   }

package/src/platforms/slackbot/commands/auth.test.ts

@@ -17,11 +17,7 @@ const mockTestAuth = mock(() =>
 
 mock.module('../client', () => ({
   SlackBotClient: class MockSlackBotClient {
-    constructor(token: string) {
-      if (!token.startsWith('xoxb-')) {
-        throw new Error('Token must be a bot token (xoxb-)')
-      }
-    }
+    async login(_credentials?: any) { return this }
     testAuth = mockTestAuth
   },
 }))

package/src/platforms/slackbot/commands/auth.ts

@@ -37,7 +37,7 @@ export async function setAction(token: string, options: ActionOptions): Promise<
       return { error: 'Token must be a bot token (xoxb-). User tokens (xoxp-) are not supported.' }
     }
 
-    const client = new SlackBotClient(token)
+    const client = await new SlackBotClient().login({ token })
     const authInfo = await client.testAuth()
 
     const botId = options.bot || authInfo.bot_id || authInfo.user || 'default'
@@ -100,7 +100,7 @@ export async function statusAction(options: ActionOptions): Promise<ActionResult
     } | null = null
 
     try {
-      const client = new SlackBotClient(creds.token)
+      const client = await new SlackBotClient().login({ token: creds.token })
       authInfo = await client.testAuth()
       valid = true
     } catch {

package/src/platforms/slackbot/commands/shared.ts

@@ -18,5 +18,5 @@ export async function getClient(options: BotOption): Promise<SlackBotClient> {
     process.exit(1)
   }
 
-  return new SlackBotClient(creds.token)
+  return new SlackBotClient().login({ token: creds.token })
 }