dsh-slack 0.2.1 → 0.2.3

package/README.en.md

@@ -1,3 +1,5 @@
+[中文](README.md)
+
 # dsh-slack
 
 DSH (DeepSeek Harness) community plugin: lets the agent communicate with Slack bidirectionally.

package/README.md

@@ -2,6 +2,11 @@
 
 # dsh-slack
 
+> **让 agent 进你的 Slack**：发消息、看频道、收消息、回线程，双向通信。
+
+![npm version](https://img.shields.io/npm/v/dsh-slack?label=npm&color=blue) ![npm downloads](https://img.shields.io/npm/dm/dsh-slack) ![license](https://img.shields.io/npm/l/dsh-slack) ![stars](https://img.shields.io/github/stars/STARDUSTLC666/dsh-slack?style=social)
+
+
 DSH（DeepSeek Harness）社区插件：让 agent 与 Slack 双向通信。
 
 > **v0.2 范围（双向）**：v0.1 只做「agent → Slack」单向通知；v0.2 新增 Socket Mode，
@@ -178,3 +183,4 @@ pnpm test      # 先 build，再用 node:test 跑 test/*.test.mjs
 ## License
 
 MIT
+

package/lib/inbox.d.ts

@@ -18,12 +18,19 @@ export declare function createInboxQueue(capacity?: number): InboxQueue;
  */
 export declare class InboxQueue {
     private items;
+    private readonly seen;
     readonly capacity: number;
     constructor(capacity?: number);
-    /** 追加一条消息；超过容量时丢弃最旧。 */
+    private key;
+    /** 追加一条消息；Slack 至少投递一次，重复事件直接忽略，超过容量时丢弃最旧。 */
     push(message: InboxMessage): void;
     /** 返回最近 limit 条消息（新的在前）。 */
     list(limit: number): InboxMessage[];
+    /**
+     * 原子地取出最近 limit 条并清空整个队列。
+     * 避免“先 list 再 clear”之间新到的消息被误清掉。
+     */
+    drain(limit: number): InboxMessage[];
     /** 清空队列。 */
     clear(): void;
     /** 当前队列长度。 */

package/lib/inbox.js

@@ -13,15 +13,25 @@ export function createInboxQueue(capacity = INBOX_CAPACITY) {
  */
 export class InboxQueue {
     items = [];
+    seen = new Set();
     capacity;
     constructor(capacity = INBOX_CAPACITY) {
         this.capacity = capacity;
     }
-    /** 追加一条消息；超过容量时丢弃最旧。 */
+    key(message) {
+        return message.channel + '\u0000' + message.ts + '\u0000' + message.user + '\u0000' + message.text;
+    }
+    /** 追加一条消息；Slack 至少投递一次，重复事件直接忽略，超过容量时丢弃最旧。 */
     push(message) {
+        const key = this.key(message);
+        if (this.seen.has(key))
+            return;
+        this.seen.add(key);
         this.items.push(message);
         if (this.items.length > this.capacity) {
-            this.items.splice(0, this.items.length - this.capacity);
+            const removed = this.items.splice(0, this.items.length - this.capacity);
+            for (const item of removed)
+                this.seen.delete(this.key(item));
         }
     }
     /** 返回最近 limit 条消息（新的在前）。 */
@@ -31,9 +41,25 @@ export class InboxQueue {
             return [];
         return this.items.slice(-n).reverse();
     }
+    /**
+     * 原子地取出最近 limit 条并清空整个队列。
+     * 避免“先 list 再 clear”之间新到的消息被误清掉。
+     */
+    drain(limit) {
+        const n = Math.max(0, Math.trunc(limit));
+        if (n === 0) {
+            this.clear();
+            return [];
+        }
+        const out = this.items.splice(-n).reverse();
+        this.items = [];
+        this.seen.clear();
+        return out;
+    }
     /** 清空队列。 */
     clear() {
         this.items = [];
+        this.seen.clear();
     }
     /** 当前队列长度。 */
     get size() {

package/lib/index.js

@@ -56,11 +56,19 @@ export function apply(ctx, rawConfig) {
     }
     const inboxQueue = createInboxQueue();
     const configProvider = () => config;
+    let client;
+    let clientFingerprint = '';
     const clientProvider = () => {
-        // 每次调用时重新解析配置（config.token 优先，环境变量回退），并懒创建客户端。
+        // 每次调用时重新解析配置（config.token 优先，环境变量回退）。
+        // 同一 token/API 地址复用 WebClient；配置变更时自动重建。
         const current = parseConfig(rawConfig);
         const token = requireToken(current);
-        return createWebSlackClient(token, current?.slackApiUrl);
+        const fingerprint = token + '\u0000' + (current?.slackApiUrl ?? '');
+        if (client === undefined || fingerprint !== clientFingerprint) {
+            client = createWebSlackClient(token, current?.slackApiUrl);
+            clientFingerprint = fingerprint;
+        }
+        return client;
     };
     const inboxProvider = () => inboxQueue;
     const deps = { clientProvider, configProvider, inboxProvider };

package/lib/slack-client.js

@@ -17,14 +17,36 @@ export function createWebSlackClient(token, slackApiUrl) {
             return { ts };
         },
         async listChannels() {
-            const result = await client.conversations.list({ types: 'public_channel,private_channel' });
-            const channels = [];
-            for (const ch of result.channels ?? []) {
-                if (typeof ch.id === 'string' && typeof ch.name === 'string') {
-                    channels.push({ id: ch.id, name: ch.name });
+            let cursor;
+            let pages = 0;
+            const allChannels = [];
+            const seen = new Set();
+            // conversations.list 默认分页（每页约 100-200 条）；企业工作区频道很多时
+            // 必须沿 next_cursor 翻完，否则模型会漏掉后面的频道。
+            do {
+                pages += 1;
+                if (pages > 20)
+                    break;
+                const result = await client.conversations.list({
+                    types: 'public_channel,private_channel',
+                    ...(cursor !== undefined ? { cursor } : {}),
+                });
+                for (const ch of (result.channels ?? [])) {
+                    if (typeof ch.id === 'string' && typeof ch.name === 'string' && !seen.has(ch.id)) {
+                        seen.add(ch.id);
+                        allChannels.push({ id: ch.id, name: ch.name });
+                    }
                 }
-            }
-            return channels;
+                const next = result.response_metadata?.next_cursor;
+                cursor = typeof next === 'string' && next !== cursor ? next : '';
+            } while (cursor !== '');
+            return allChannels;
+            /*
+              // if (typeof ch.id === 'string' && typeof ch.name === 'string') {
+                // allChannels.push({ id: ch.id, name: ch.name })
+              */
+            // }
+            // removed duplicate return
         },
     };
 }
@@ -43,9 +65,18 @@ export function mapSlackError(error) {
     if (code === 'not_in_channel') {
         return '机器人不在该频道中（not_in_channel）：请先把机器人 App 添加（invite）到该频道。';
     }
-    if (code === 'token_revoked' || code === 'account_inactive' || code === 'missing_scope') {
+    if (code === 'token_revoked' || code === 'account_inactive' || code === 'missing_scope' || code === 'not_authed') {
         return `Slack 令牌或权限不足（${code}）：请确认已勾选 chat:write 与 channels:read，并重新安装 App 拿新令牌。`;
     }
+    if (code === 'is_archived') {
+        return '频道已归档（is_archived）：机器人不能在已归档频道发消息，请改用其他频道。';
+    }
+    if (code === 'msg_too_long') {
+        return '消息过长（msg_too_long）：Slack 单条消息不能超过 40,000 字符，请精简后重试。';
+    }
+    if (code === 'ratelimited' || code === 'rate_limited') {
+        return 'Slack 触发限流（ratelimited）：请求太频繁，请稍等几秒后重试。';
+    }
     const message = error?.message;
     return message ? `Slack API 调用失败：${message}` : 'Slack API 调用失败：未知错误。';
 }

package/lib/tools.js

@@ -160,9 +160,7 @@ export function buildInboxTool(deps) {
             const limit = clampLimit(raw.limit);
             const markRead = raw.markRead === true;
             const queue = deps.inboxProvider();
-            const messages = queue.list(limit);
-            if (markRead)
-                queue.clear();
+            const messages = markRead ? queue.drain(limit) : queue.list(limit);
             return { messages: messages.map((m) => ({ ts: m.ts, channel: m.channel, user: m.user, text: m.text })) };
         },
     };

package/package.json

@@ -1,57 +1,54 @@
-{
-  "name": "dsh-slack",
-  "version": "0.2.1",
-  "description": "DSH 社区插件：Slack 通知/桥接（v0.2 支持 Socket Mode 双向：收件箱 + 线程回复）",
-  "type": "module",
-  "main": "./lib/index.js",
-  "types": "./lib/index.d.ts",
-  "exports": {
-    ".": {
-      "types": "./lib/index.d.ts",
-      "default": "./lib/index.js"
-    },
-    "./package.json": "./package.json",
-    "./cordis.patch.yml": "./cordis.patch.yml"
-  },
-  "files": [
-    "lib",
-    "cordis.patch.yml",
-    "README.md"
-  ],
-  "license": "MIT",
-  "dsh": {
-    "bundle": {
-      "patch": "./cordis.patch.yml"
-    }
-  },
-  "scripts": {
-    "build": "tsc -p tsconfig.json",
-    "prepare": "tsc -p tsconfig.json",
-    "test": "pnpm run build && node --test \"test/*.test.mjs\"",
-    "prepublishOnly": "pnpm run build",
-    "test:protocol": "node test/protocol-acceptance.mjs"
-  },
-  "dependencies": {
-    "@slack/socket-mode": "^3.0.0",
-    "@slack/web-api": "^8.0.0"
-  },
-  "devDependencies": {
-    "@types/node": "^24.0.0",
-    "typescript": "^5.7.3"
-  },
-  "repository": {
-    "url": "https://github.com/STARDUSTLC666/dsh-slack",
-    "type": "git"
-  },
-  "bugs": {
-    "url": "https://github.com/STARDUSTLC666/dsh-slack/issues"
-  },
-  "homepage": "https://github.com/STARDUSTLC666/dsh-slack#readme",
-  "author": "stardustlc",
-  "keywords": [
-    "dsh-plugin",
-    "deepseek-harness",
-    "dsh"
-  ],
-  "packageManager": "pnpm@11.7.0"
-}
+{
+  "name": "dsh-slack",
+  "version": "0.2.3",
+  "description": "DSH 社区插件：Slack 通知/桥接（v0.2 支持 Socket Mode 双向：收件箱 + 线程回复）",
+  "type": "module",
+  "main": "./lib/index.js",
+  "types": "./lib/index.d.ts",
+  "exports": {
+    ".": {
+      "types": "./lib/index.d.ts",
+      "default": "./lib/index.js"
+    },
+    "./package.json": "./package.json",
+    "./cordis.patch.yml": "./cordis.patch.yml"
+  },
+  "files": [
+    "lib",
+    "cordis.patch.yml",
+    "README.md"
+  ],
+  "license": "MIT",
+  "dsh": {
+    "bundle": {
+      "patch": "./cordis.patch.yml"
+    }
+  },
+  "dependencies": {
+    "@slack/socket-mode": "^3.0.0",
+    "@slack/web-api": "^8.0.0"
+  },
+  "devDependencies": {
+    "@types/node": "^24.0.0",
+    "typescript": "^5.7.3"
+  },
+  "repository": {
+    "url": "https://github.com/STARDUSTLC666/dsh-slack",
+    "type": "git"
+  },
+  "bugs": {
+    "url": "https://github.com/STARDUSTLC666/dsh-slack/issues"
+  },
+  "homepage": "https://github.com/STARDUSTLC666/dsh-slack#readme",
+  "author": "stardustlc",
+  "keywords": [
+    "dsh-plugin",
+    "deepseek-harness",
+    "dsh"
+  ],
+  "scripts": {
+    "build": "tsc -p tsconfig.json",
+    "test": "pnpm run build && node --test \"test/*.test.mjs\"",
+    "test:protocol": "node test/protocol-acceptance.mjs"
+  }
+}