@xmanrui/dsh-im 2.4.0 → 2.6.0

package/src/channels/shared/text-harness-bridge.mjs

@@ -27,6 +27,7 @@ import {
 } from './batch-input.mjs';
 import {
   hasInboundImages,
+  imagePromptDiagnostic,
   imagePromptUserMessage,
   promptContentForMessage,
 } from './image-prompt.mjs';
@@ -45,6 +46,13 @@ import {
   createTextDeliveryBlock,
   providerMessageIdsFor,
 } from './semantic/delivery.mjs';
+import {
+  channelDeliveryFailure,
+  clearLastMessageFailure,
+  messageFailureText,
+  setLastMessageFailure,
+} from './message-failure.mjs';
+import { beginStatusReaction } from './status-reaction.mjs';
 
 const INTERACTION_RESOLVED_TEXT = '这个问题已在其他客户端处理，无需再次回答。';
 const FILE_ONLY_COMPLETION_TEXT = '任务已完成。';
@@ -106,6 +114,10 @@ export function createTextBridgeStatus() {
     lastReplyAt: null,
     lastRejectedAt: null,
     lastError: null,
+    lastMessageError: null,
+    reactionsAdded: 0,
+    reactionsRemoved: 0,
+    reactionErrors: 0,
   };
 }
 
@@ -170,6 +182,27 @@ export class TextHarnessBridge {
       return Promise.resolve();
     }
     this.#acceptedMessageIds.add(messageId);
+    const statusReaction = beginStatusReaction({
+      adapter: this.#bot,
+      target: normalized.kind === 'direct' || normalized.addressed === true
+        ? normalized.reactionTarget
+        : null,
+      reactions: this.#descriptor.reactions,
+      status: this.#status,
+      logger: this.#logger,
+      label: this.#descriptor.key,
+    });
+    normalized.statusReaction = statusReaction;
+
+    const processing = this.#acceptAcceptedMessage(normalized, messageId, senderId);
+    void processing.then(
+      () => statusReaction.success(),
+      () => statusReaction.error(),
+    );
+    return processing;
+  }
+
+  #acceptAcceptedMessage(normalized, messageId, senderId) {
     if (normalized.kind === 'direct') {
       rememberConnectionTestTarget(
         this.#state,
@@ -177,7 +210,7 @@ export class TextHarnessBridge {
       );
     }
 
-    const key = `${kind}:${conversationId}`;
+    const key = `${normalized.kind}:${normalized.conversationId}`;
     const pending = this.#pendingInteractions.get(key);
     const text = cleanText(normalized.content);
     const batchCommand = isBatchInputCommand(text);
@@ -318,10 +351,15 @@ export class TextHarnessBridge {
       if (reply) await this.#bot.sendText(message.replyTarget, reply);
       this.#status.lastError = null;
     })().catch(async (error) => {
-      if (this.#signal?.aborted) return;
+      if (this.#signal?.aborted) {
+        message.statusReaction?.clear();
+        return;
+      }
+      message.statusReaction?.error();
       this.#status.lastError = error?.message ?? String(error);
+      const failure = setLastMessageFailure(this.#status, error);
       this.#logger.error?.(
-        `[dsh-im:${this.#descriptor.key}] failed to process a batch input message:`,
+        `[dsh-im:${this.#descriptor.key}] failed to process a batch input message [${failure.referenceId}]:`,
         error,
       );
     }).finally(() => {
@@ -397,10 +435,18 @@ export class TextHarnessBridge {
       }
       this.#status.lastError = null;
     } catch (error) {
-      if (error?.code === 'turn-stopped' || this.#signal?.aborted) return;
+      if (error?.code === 'turn-stopped' || this.#signal?.aborted) {
+        message.statusReaction?.clear();
+        return;
+      }
+      message.statusReaction?.error();
       this.#status.lastError = error?.message ?? String(error);
-      this.#logger.error?.(`[dsh-im:${this.#descriptor.key}] failed to process a command:`, error);
-      await this.#bot.sendText(target, t('消息处理失败，请稍后重试。')).catch(() => undefined);
+      const failure = setLastMessageFailure(this.#status, error);
+      this.#logger.error?.(
+        `[dsh-im:${this.#descriptor.key}] failed to process a command [${failure.referenceId}]:`,
+        error,
+      );
+      await this.#bot.sendText(target, messageFailureText(failure)).catch(() => undefined);
     }
   }
 
@@ -426,9 +472,13 @@ export class TextHarnessBridge {
       sendFile: typeof this.#bot.sendFile === 'function'
         ? (file) => this.#bot.sendFile(target, file)
         : undefined,
-      sendFailureNotice: (artifact, error) => this.#bot.sendText(
+      onFailure: (artifact, error) => setLastMessageFailure(this.#status, error, {
+        userMessage: artifactFailureText(artifact?.fileName, error, this.#descriptor),
+        reason: error?.code,
+      }),
+      sendFailureNotice: (_artifact, _error, failure) => this.#bot.sendText(
         target,
-        artifactFailureText(artifact?.fileName, error, this.#descriptor),
+        messageFailureText(failure),
       ),
       logger: this.#logger,
     });
@@ -436,7 +486,11 @@ export class TextHarnessBridge {
       + delivery.artifactsSent;
     this.#status.artifactSendErrors = (this.#status.artifactSendErrors ?? 0)
       + delivery.artifactSendErrors;
-    return { receipt: delivery.receipt, userVisible: delivery.userVisible };
+    return {
+      receipt: delivery.receipt,
+      userVisible: delivery.userVisible,
+      artifactSendErrors: delivery.artifactSendErrors,
+    };
   }
 
   async #process(message, messageId, senderId, conversationKey, {
@@ -643,25 +697,37 @@ export class TextHarnessBridge {
             reason: result?.reason,
           });
         } catch (error) {
-          textDeliveryError = error;
+          textDeliveryError = channelDeliveryFailure(error);
         }
       }
+      const finalDeliveryUnknown = textReceipt?.deliveryOutcome === 'unknown';
       if (textReceipt?.deliveryOutcome === 'failed') {
-        const reason = textReceipt.reason ?? 'text-delivery-failed';
-        textDeliveryError = new Error(`Final text delivery failed (${reason})`);
-        textDeliveryError.code = reason;
+        textDeliveryError = channelDeliveryFailure(
+          new Error(`Final text delivery failed (${textReceipt.reason ?? 'unknown'})`),
+          { uncertain: false },
+        );
+      } else if (finalDeliveryUnknown) {
+        textDeliveryError = channelDeliveryFailure(
+          new Error(`Final text delivery outcome is unknown (${textReceipt.reason ?? 'unknown'})`),
+        );
       }
       // A failed final text must not discard an already registered result file.
       // Settle the independent attachment path before surfacing the text error.
       const delivery = await this.#deliverArtifacts(target, messageId, artifacts, textReceipt);
-      if (textDeliveryError && !delivery.userVisible) {
+      if (textDeliveryError && (!delivery.userVisible || finalDeliveryUnknown)) {
         textDeliveryError.deliveryReceipt = delivery.receipt;
         throw textDeliveryError;
       }
+      if (textDeliveryError && delivery.artifactSendErrors === 0) {
+        setLastMessageFailure(this.#status, textDeliveryError);
+      }
       if (delivery.userVisible) {
         this.#status.messagesReplied += 1;
         this.#status.lastReplyAt = new Date().toISOString();
         this.#status.lastError = null;
+        if (!textDeliveryError && delivery.artifactSendErrors === 0) {
+          clearLastMessageFailure(this.#status);
+        }
       }
       return delivery.receipt;
     } catch (error) {
@@ -673,6 +739,7 @@ export class TextHarnessBridge {
         ? this.#batches.fail(conversationKey, batchSubmission.token)
         : null;
       if (turnStopped) {
+        message.statusReaction?.clear();
         if (stream) {
           try {
             await stream.finish(t('已停止。'));
@@ -683,15 +750,22 @@ export class TextHarnessBridge {
         return;
       }
       if (this.#signal?.aborted) {
+        message.statusReaction?.clear();
         stream?.cancel?.();
         return;
       }
+      message.statusReaction?.error();
       this.#status.lastError = error?.message ?? String(error);
       const presentStreamFailure = async (text) => {
-        if (typeof stream?.fail !== 'function') return false;
+        const method = typeof stream?.fail === 'function'
+          ? 'fail'
+          : (typeof stream?.finish === 'function' ? 'finish' : null);
+        if (!method) return false;
         try {
-          const result = await stream.fail(text);
-          return Boolean(result) && result.deliveryOutcome !== 'failed';
+          const result = await stream[method](text);
+          return method === 'fail'
+            ? Boolean(result) && result.deliveryOutcome !== 'failed'
+            : result !== false && result?.deliveryOutcome !== 'failed';
         } catch (streamError) {
           this.#logger.warn?.(
             `[dsh-im:${this.#descriptor.key}] unable to finalize the failed stream:`,
@@ -700,58 +774,25 @@ export class TextHarnessBridge {
           return false;
         }
       };
-      if (failedBatch?.retained) {
-        this.#logger.error?.(
-          `[dsh-im:${this.#descriptor.key}] failed to submit a batch input:`,
-          error,
-        );
-        if (await presentStreamFailure(failedBatch.message)) return;
-        stream?.cancel?.();
-        try {
-          await this.#bot.sendText(target, failedBatch.message);
-        } catch (sendError) {
-          this.#logger.error?.(
-            `[dsh-im:${this.#descriptor.key}] failed to send the batch retry notice:`,
-            sendError,
-          );
-        }
-        return;
-      }
       const imageErrorMessage = imagePromptUserMessage(error);
-      if (imageErrorMessage) {
-        if (await presentStreamFailure(imageErrorMessage)) return;
-        stream?.cancel?.();
-        try {
-          await this.#bot.sendText(target, imageErrorMessage);
-        } catch (sendError) {
-          this.#logger.error?.(
-            `[dsh-im:${this.#descriptor.key}] failed to send the image error reply:`,
-            sendError,
-          );
-        }
-        return;
-      }
       const fileErrorMessage = inboundFileUserMessage(error);
-      if (fileErrorMessage) {
-        if (await presentStreamFailure(fileErrorMessage)) return;
-        stream?.cancel?.();
-        try {
-          await this.#bot.sendText(target, fileErrorMessage);
-        } catch (sendError) {
-          this.#logger.error?.(
-            `[dsh-im:${this.#descriptor.key}] failed to send the file error reply:`,
-            sendError,
-          );
-        }
-        return;
-      }
-      this.#logger.error?.(`[dsh-im:${this.#descriptor.key}] failed to process a message:`, error);
-      if (await presentStreamFailure('消息处理失败，请稍后重试。')) {
+      const failure = setLastMessageFailure(this.#status, error, {
+        userMessage: fileErrorMessage ?? imageErrorMessage,
+        reason: imagePromptDiagnostic(error)?.reason,
+      });
+      const failureText = failedBatch?.retained
+        ? `${messageFailureText(failure)}\n\n${failedBatch.message}`
+        : messageFailureText(failure);
+      this.#logger.error?.(
+        `[dsh-im:${this.#descriptor.key}] failed to process a message [${failure.referenceId}]:`,
+        error,
+      );
+      if (await presentStreamFailure(failureText)) {
         return error.deliveryReceipt;
       }
       stream?.cancel?.();
       try {
-        await this.#bot.sendText(target, t('消息处理失败，请稍后重试。'));
+        await this.#bot.sendText(target, failureText);
       } catch (sendError) {
         this.#logger.error?.(
           `[dsh-im:${this.#descriptor.key}] failed to send the safe error reply:`,
@@ -768,7 +809,10 @@ export class TextHarnessBridge {
   }
 
   async #processInteractionReply(message, messageId, senderId, key, expected) {
-    if (this.#signal?.aborted) return;
+    if (this.#signal?.aborted) {
+      message.statusReaction?.clear();
+      return;
+    }
     const current = this.#pendingInteractions.get(key);
     const claimed = expected.claimedReplyMessageId === messageId;
     if (!current || current !== expected || current.submitting) {
@@ -822,6 +866,7 @@ export class TextHarnessBridge {
       try {
         await this.#presentInteraction(pending);
       } catch (error) {
+        message.statusReaction?.error();
         this.#status.lastError = t('{label}交互问题发送失败。', { label: this.#descriptor.label });
         this.#logger.error?.(
           `[dsh-im:${this.#descriptor.key}] failed to retry an interaction question:`,
@@ -860,6 +905,7 @@ export class TextHarnessBridge {
       try {
         await this.#presentInteraction(pending);
       } catch (error) {
+        message.statusReaction?.error();
         this.#status.lastError = t('{label}交互问题发送失败。', { label: this.#descriptor.label });
         this.#logger.error?.(
           `[dsh-im:${this.#descriptor.key}] failed to send the next interaction question:`,
@@ -884,7 +930,10 @@ export class TextHarnessBridge {
     } catch (error) {
       if (error?.code === 'interaction-not-pending') {
         this.#clearPendingInteraction(key, pending.interactionId);
-        if (this.#signal?.aborted) return;
+        if (this.#signal?.aborted) {
+          message.statusReaction?.clear();
+          return;
+        }
         try {
           await this.#bot.sendText(target, t(INTERACTION_RESOLVED_TEXT));
         } catch (sendError) {
@@ -895,7 +944,15 @@ export class TextHarnessBridge {
         }
         return;
       }
-      if (this.#signal?.aborted || this.#pendingInteractions.get(key) !== pending) return;
+      if (this.#signal?.aborted) {
+        message.statusReaction?.clear();
+        return;
+      }
+      if (this.#pendingInteractions.get(key) !== pending) {
+        message.statusReaction?.clear();
+        return;
+      }
+      message.statusReaction?.error();
       pending.submitting = false;
       pending.answers.pop();
       pending.index -= 1;

package/src/channels/shared/token-bot-controller.mjs

@@ -1,5 +1,6 @@
 import { connectionTestMessage } from './connection-test.mjs';
 import { t } from './i18n.mjs';
+import { publicMessageFailure } from './message-failure.mjs';
 
 function cleanString(value) {
   return typeof value === 'string' && value.trim() ? value.trim() : null;
@@ -291,6 +292,7 @@ export class TokenBotController {
           messagesReceived: runtimeStatus?.messagesReceived ?? 0,
           messagesReplied: runtimeStatus?.messagesReplied ?? 0,
         },
+        lastMessageError: publicMessageFailure(runtimeStatus?.lastMessageError),
         error: structuredClone(this.#errors.get(config.botId) ?? null),
       };
     });

package/src/channels/slack/manifest.mjs

@@ -20,6 +20,7 @@ oauth_config:
       - files:read
       - files:write
       - im:history
+      - reactions:write
 settings:
   event_subscriptions:
     bot_events:

package/src/channels/slack/slack-api.mjs

@@ -276,6 +276,34 @@ export class SlackApi {
     });
   }
 
+  addReaction({ channelId, messageTs, emojiName, signal, timeoutMs }) {
+    return this.#request('reactions.add', {
+      tokenKind: 'bot',
+      signal,
+      timeoutMs,
+      retry: false,
+      body: {
+        channel: slackId(channelId, 'channel id'),
+        timestamp: requiredString(messageTs, 'message timestamp'),
+        name: requiredString(emojiName, 'reaction name'),
+      },
+    });
+  }
+
+  removeReaction({ channelId, messageTs, emojiName, signal, timeoutMs }) {
+    return this.#request('reactions.remove', {
+      tokenKind: 'bot',
+      signal,
+      timeoutMs,
+      retry: false,
+      body: {
+        channel: slackId(channelId, 'channel id'),
+        timestamp: requiredString(messageTs, 'message timestamp'),
+        name: requiredString(emojiName, 'reaction name'),
+      },
+    });
+  }
+
   startStream({ channelId, threadTs, recipientTeamId, recipientUserId, markdownText, signal }) {
     return this.#request('chat.startStream', {
       tokenKind: 'bot',

package/src/channels/slack/slack-bridge.mjs

@@ -4,6 +4,11 @@ export const SLACK_DESCRIPTOR = Object.freeze({
   key: 'slack',
   label: 'Slack',
   connectionLabel: ' Socket Mode 长连接',
+  reactions: Object.freeze({
+    processing: 'eyes',
+    success: 'white_check_mark',
+    error: 'x',
+  }),
 });
 
 export class SlackHarnessBridge extends TextHarnessBridge {

package/src/channels/slack/slack-controller.mjs

@@ -1,4 +1,5 @@
 import { connectionTestMessage } from '../shared/connection-test.mjs';
+import { publicMessageFailure } from '../shared/message-failure.mjs';
 import { t } from '../shared/i18n.mjs';
 import { deriveSlackBotIdentity, maskSlackBotId } from './config-store.mjs';
 import { inspectSlackCredentials } from './slack-api.mjs';
@@ -254,6 +255,7 @@ export class SlackController {
           messagesReceived: runtimeStatus?.messagesReceived ?? 0,
           messagesReplied: runtimeStatus?.messagesReplied ?? 0,
         },
+        lastMessageError: publicMessageFailure(runtimeStatus?.lastMessageError),
         error: structuredClone(this.#errors.get(config.botId) ?? null),
       };
     });

package/src/channels/slack/slack-runtime.mjs

@@ -127,6 +127,10 @@ export function normalizeSlackEvent(payload, botUserId, {
       ? event.files.map((file) => slackFileSource(file, loadFileStream, loadFileInfo)).filter(Boolean)
       : [],
     addressed: direct || mentioned,
+    reactionTarget: {
+      channelId: String(event.channel),
+      messageTs: String(event.ts),
+    },
     replyTarget: {
       channelId: String(event.channel),
       threadTs,
@@ -284,6 +288,26 @@ export class SlackBotClient {
     return { providerMessageIds };
   }
 
+  async addReaction(target, emoji, { signal } = {}) {
+    const reactionKey = String(emoji ?? '').trim();
+    await this.#api.addReaction({
+      channelId: target.channelId,
+      messageTs: target.messageTs,
+      emojiName: reactionKey,
+      signal: signal ?? this.#signal,
+    });
+    return reactionKey;
+  }
+
+  removeReaction(target, reactionKey, { signal } = {}) {
+    return this.#api.removeReaction({
+      channelId: target.channelId,
+      messageTs: target.messageTs,
+      emojiName: reactionKey,
+      signal: signal ?? this.#signal,
+    });
+  }
+
   openStream(target) {
     return createSlackMessageStream({
       api: this.#api,

package/src/channels/telegram/telegram-api.mjs

@@ -203,6 +203,15 @@ export class TelegramApi {
     }, { signal });
   }
 
+  async setMessageReaction({ chatId, messageId, emoji, signal, timeoutMs }) {
+    const normalizedEmoji = cleanString(emoji);
+    return this.#call('setMessageReaction', {
+      chat_id: chatId,
+      message_id: messageId,
+      reaction: normalizedEmoji ? [{ type: 'emoji', emoji: normalizedEmoji }] : [],
+    }, { signal, timeoutMs });
+  }
+
   async sendRichMessage({
     chatId,
     richMessage,

package/src/channels/telegram/telegram-bridge.mjs

@@ -4,6 +4,7 @@ export const TELEGRAM_DESCRIPTOR = Object.freeze({
   key: 'telegram',
   label: 'Telegram',
   connectionLabel: ' Bot API 长轮询',
+  reactions: Object.freeze({ processing: '👀', success: '👍', error: '👎' }),
 });
 
 export class TelegramHarnessBridge extends TextHarnessBridge {

package/src/channels/telegram/telegram-runtime.mjs

@@ -168,6 +168,7 @@ export function normalizeTelegramUpdate(update, {
     images: image ? [image] : [],
     files: file ? [file] : [],
     addressed,
+    reactionTarget: { chatId, messageId },
     replyTarget: {
       chatId,
       chatType: message.chat.type,
@@ -303,6 +304,25 @@ export class TelegramBotClient {
     return { providerMessageIds };
   }
 
+  async addReaction(target, emoji, { signal } = {}) {
+    const reactionKey = String(emoji ?? '').trim();
+    await this.#api.setMessageReaction({
+      chatId: target.chatId,
+      messageId: target.messageId,
+      emoji: reactionKey,
+      signal: signal ?? this.#signal,
+    });
+    return reactionKey;
+  }
+
+  removeReaction(target, _reactionKey, { signal } = {}) {
+    return this.#api.setMessageReaction({
+      chatId: target.chatId,
+      messageId: target.messageId,
+      signal: signal ?? this.#signal,
+    });
+  }
+
   sendTyping(target) {
     return this.#api.sendChatAction({
       chatId: target.chatId,