@zhin.js/adapter-slack 8.0.0 → 8.0.1

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## 8.0.1
+
+### Patch Changes
+
+- f7ee28d: Require a platform message timestamp for each Slack message chunk instead of fabricating a successful receipt. Stop subsequent chunks when delivery cannot be confirmed, and propagate file upload failures instead of silently reporting partial delivery as success.
+- Updated dependencies [f7ee28d]
+  - @zhin.js/adapter@1.2.2
+  - @zhin.js/core@1.5.16
+  - zhin.js@7.0.1
+  - @zhin.js/agent@1.2.1
+
 ## 8.0.0
 
 ### Patch Changes

package/README.md

@@ -138,3 +138,9 @@ HTTP 模式下 Runtime Host（`http`）须已 listen；Slack App 的 Event Subsc
 ## 许可证
 
 MIT
+
+## 发送失败与回执
+
+成功发送返回平台 `ts` 组成的消息引用，不以本地时间替代缺失回执。
+文件上传失败会让本次发送失败；分段消息中某段缺少有效回执时，停止后续分段。
+前面已成功的文件或分段不会自动回滚；发送失败可能表示部分已送达，请先核对目标会话再重试。

package/lib/endpoint.js

@@ -138,9 +138,8 @@ export class SlackEndpoint extends Endpoint {
         const channel = conversation.id;
         const threadTs = conversation.threadId;
         const result = await sendSlackContent(this.#client, payload, { channel, threadTs }, this.#logger);
-        if (result.ts)
-            this.trackMessageChannel(result.ts, channel);
-        return formatSlackMessageRef(channel, result.ts || String(Date.now()));
+        this.trackMessageChannel(result.ts, channel);
+        return formatSlackMessageRef(channel, result.ts);
     }
     /** Test / internal: admit a message event when open. */
     admit(event) {

package/lib/slack-outbound.js

@@ -7,10 +7,11 @@ export async function sendSlackContent(client, content, opts, logger) {
     const textDelivery = applyTextMrkdwnBlocks(wire.text, blocks);
     for (const pf of wire.files) {
         try {
-            await uploadFile(client, opts.channel, pf, opts.threadTs, logger);
+            await uploadFile(client, opts.channel, pf, opts.threadTs);
         }
         catch (e) {
             logger.error('Failed to upload file:', e);
+            throw e;
         }
     }
     return postSlackMessage(client, {
@@ -62,7 +63,7 @@ async function postSlackMessage(client, opts) {
             ...(threadTs ? { thread_ts: threadTs } : {}),
             ...(attachments.length > 0 ? { attachments } : {}),
         });
-        return { ts: result.ts ?? '' };
+        return { ts: requireMessageTimestamp(result) };
     }
     for (let offset = 0; offset < blocks.length; offset += SLACK_MAX_BLOCKS_PER_MESSAGE) {
         const chunk = blocks.slice(offset, offset + SLACK_MAX_BLOCKS_PER_MESSAGE);
@@ -73,7 +74,7 @@ async function postSlackMessage(client, opts) {
             ...(threadTs ? { thread_ts: threadTs } : {}),
             ...(offset === 0 && attachments.length > 0 ? { attachments } : {}),
         });
-        const ts = result.ts ?? '';
+        const ts = requireMessageTimestamp(result);
         if (!firstTs)
             firstTs = ts;
         if (!threadTs)
@@ -81,29 +82,30 @@ async function postSlackMessage(client, opts) {
     }
     return { ts: firstTs };
 }
-async function uploadFile(client, channel, file, threadTs, logger) {
+function requireMessageTimestamp(result) {
+    if (typeof result?.ts !== 'string' || !result.ts.trim()) {
+        throw new Error('Slack chat.postMessage returned no valid ts; delivery is unconfirmed');
+    }
+    return result.ts;
+}
+async function uploadFile(client, channel, file, threadTs) {
     if (!client.filesUploadV2)
-        return;
-    try {
-        let buffer = file.buffer;
-        if (!buffer && file.path) {
-            const { readFile } = await import('node:fs/promises');
-            buffer = await readFile(file.path);
-        }
-        if (!buffer && file.url) {
-            const res = await fetch(file.url);
-            if (!res.ok)
-                throw new Error(`fetch ${file.url}: ${res.status}`);
-            buffer = Buffer.from(await res.arrayBuffer());
-        }
-        if (!buffer)
-            return;
-        await client.filesUploadV2(threadTs
-            ? { channel_id: channel, file: buffer, filename: file.name ?? 'file', thread_ts: threadTs }
-            : { channel_id: channel, file: buffer, filename: file.name ?? 'file' });
+        throw new Error('Slack client does not support file uploads');
+    let buffer = file.buffer;
+    if (!buffer && file.path) {
+        const { readFile } = await import('node:fs/promises');
+        buffer = await readFile(file.path);
     }
-    catch (e) {
-        logger?.error('File upload failed:', e);
+    if (!buffer && file.url) {
+        const res = await fetch(file.url);
+        if (!res.ok)
+            throw new Error(`fetch ${file.url}: ${res.status}`);
+        buffer = Buffer.from(await res.arrayBuffer());
     }
+    if (!buffer)
+        throw new Error('Slack file contains no uploadable content');
+    await client.filesUploadV2(threadTs
+        ? { channel_id: channel, file: buffer, filename: file.name ?? 'file', thread_ts: threadTs }
+        : { channel_id: channel, file: buffer, filename: file.name ?? 'file' });
 }
 export { keyboardToBlockKitBlocks };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhin.js/adapter-slack",
-  "version": "8.0.0",
+  "version": "8.0.1",
   "description": "Zhin.js Slack adapter for Plugin Runtime (Socket Mode / HTTP Events)",
   "type": "module",
   "main": "./lib/index.js",
@@ -34,8 +34,8 @@
   "dependencies": {
     "@slack/socket-mode": "^2.0.7",
     "@slack/web-api": "^7.19.0",
-    "@zhin.js/adapter": "1.2.1",
-    "@zhin.js/core": "1.5.15",
+    "@zhin.js/adapter": "1.2.2",
+    "@zhin.js/core": "1.5.16",
     "@zhin.js/feature-kit": "1.0.13",
     "@zhin.js/host-http": "1.0.17",
     "@zhin.js/im-contract": "1.0.4",
@@ -43,13 +43,13 @@
   },
   "peerDependencies": {
     "zod": "^4.0.0",
-    "@zhin.js/adapter": "1.2.1",
-    "@zhin.js/agent": "1.2.0",
+    "@zhin.js/adapter": "1.2.2",
+    "@zhin.js/agent": "1.2.1",
     "@zhin.js/command": "1.0.16",
-    "@zhin.js/core": "1.5.15",
+    "@zhin.js/core": "1.5.16",
     "@zhin.js/host-http": "1.0.17",
     "@zhin.js/permission": "1.0.4",
-    "zhin.js": "7.0.0"
+    "zhin.js": "7.0.1"
   },
   "peerDependenciesMeta": {
     "@zhin.js/agent": {
@@ -70,8 +70,8 @@
     "typescript": "^6.0.3",
     "vitest": "^4.1.10",
     "zod": "^4.4.3",
-    "@zhin.js/agent": "1.2.0",
-    "zhin.js": "7.0.0"
+    "@zhin.js/agent": "1.2.1",
+    "zhin.js": "7.0.1"
   },
   "files": [
     "adapters",

package/src/endpoint.ts

@@ -233,8 +233,8 @@ export class SlackEndpoint extends Endpoint<SlackWebClientLike> implements Slack
       { channel, threadTs },
       this.#logger,
     );
-    if (result.ts) this.trackMessageChannel(result.ts, channel);
-    return formatSlackMessageRef(channel, result.ts || String(Date.now()));
+    this.trackMessageChannel(result.ts, channel);
+    return formatSlackMessageRef(channel, result.ts);
   }
 
   /** Test / internal: admit a message event when open. */

package/src/slack-outbound.ts

@@ -36,9 +36,10 @@ export async function sendSlackContent(
 
   for (const pf of wire.files) {
     try {
-      await uploadFile(client, opts.channel, pf, opts.threadTs, logger);
+      await uploadFile(client, opts.channel, pf, opts.threadTs);
     } catch (e) {
       logger.error('Failed to upload file:', e);
+      throw e;
     }
   }
 
@@ -113,7 +114,7 @@ async function postSlackMessage(
       ...(threadTs ? { thread_ts: threadTs } : {}),
       ...(attachments.length > 0 ? { attachments } : {}),
     });
-    return { ts: result.ts ?? '' };
+    return { ts: requireMessageTimestamp(result) };
   }
 
   for (let offset = 0; offset < blocks.length; offset += SLACK_MAX_BLOCKS_PER_MESSAGE) {
@@ -125,7 +126,7 @@ async function postSlackMessage(
       ...(threadTs ? { thread_ts: threadTs } : {}),
       ...(offset === 0 && attachments.length > 0 ? { attachments } : {}),
     });
-    const ts = result.ts ?? '';
+    const ts = requireMessageTimestamp(result);
     if (!firstTs) firstTs = ts;
     if (!threadTs) threadTs = ts;
   }
@@ -133,35 +134,37 @@ async function postSlackMessage(
   return { ts: firstTs };
 }
 
+function requireMessageTimestamp(result: { ts?: string } | null | undefined): string {
+  if (typeof result?.ts !== 'string' || !result.ts.trim()) {
+    throw new Error('Slack chat.postMessage returned no valid ts; delivery is unconfirmed');
+  }
+  return result.ts;
+}
+
 async function uploadFile(
   client: SlackChatClient,
   channel: string,
   file: { buffer?: Buffer; url?: string; path?: string; name?: string },
   threadTs?: string,
-  logger?: Logger,
 ): Promise<void> {
-  if (!client.filesUploadV2) return;
-  try {
-    let buffer = file.buffer;
-    if (!buffer && file.path) {
-      const { readFile } = await import('node:fs/promises');
-      buffer = await readFile(file.path);
-    }
-    if (!buffer && file.url) {
-      const res = await fetch(file.url);
-      if (!res.ok) throw new Error(`fetch ${file.url}: ${res.status}`);
-      buffer = Buffer.from(await res.arrayBuffer());
-    }
-    if (!buffer) return;
-
-    await client.filesUploadV2(
-      threadTs
-        ? { channel_id: channel, file: buffer, filename: file.name ?? 'file', thread_ts: threadTs }
-        : { channel_id: channel, file: buffer, filename: file.name ?? 'file' },
-    );
-  } catch (e) {
-    logger?.error('File upload failed:', e);
+  if (!client.filesUploadV2) throw new Error('Slack client does not support file uploads');
+  let buffer = file.buffer;
+  if (!buffer && file.path) {
+    const { readFile } = await import('node:fs/promises');
+    buffer = await readFile(file.path);
+  }
+  if (!buffer && file.url) {
+    const res = await fetch(file.url);
+    if (!res.ok) throw new Error(`fetch ${file.url}: ${res.status}`);
+    buffer = Buffer.from(await res.arrayBuffer());
   }
+  if (!buffer) throw new Error('Slack file contains no uploadable content');
+
+  await client.filesUploadV2(
+    threadTs
+      ? { channel_id: channel, file: buffer, filename: file.name ?? 'file', thread_ts: threadTs }
+      : { channel_id: channel, file: buffer, filename: file.name ?? 'file' },
+  );
 }
 
 export { keyboardToBlockKitBlocks };