@cjhyy/code-shell-chat 0.9.17 → 0.9.19

package/dist/bound-session-chat.d.ts

@@ -9,7 +9,7 @@
  *
  * Two properties matter more than they look:
  *
- * - Leaving is decided here, not by the host and never by a model. /mimi has
+ * - Leaving is decided by the host, never by a model. /mimi has
  *   to work when the worker is wedged or the bound Session is unreachable,
  *   otherwise a user can be stuck inside a Session with no way out.
  * - The reply for an accepted message is not this middleware's job. A Session

package/dist/bound-session-chat.js

@@ -9,7 +9,7 @@
  *
  * Two properties matter more than they look:
  *
- * - Leaving is decided here, not by the host and never by a model. /mimi has
+ * - Leaving is decided by the host, never by a model. /mimi has
  *   to work when the worker is wedged or the bound Session is unreachable,
  *   otherwise a user can be stuck inside a Session with no way out.
  * - The reply for an accepted message is not this middleware's job. A Session
@@ -26,28 +26,23 @@ function isAddressable(message) {
     return Boolean(message.target?.trim() && message.senderId?.trim());
 }
 export function createBoundSessionChat(options) {
-    const isDirectMessage = options.isDirectMessage ?? (() => true);
+    const isDirectMessage = options.isDirectMessage ?? ((message) => message.isDirectMessage === true);
     return async ({ message, reply }, next) => {
         if (!message.text.trim() && !message.attachments?.length)
             return next();
         if (!isAddressable(message))
             return next();
-        let disposition;
-        try {
-            disposition = await options.desktop.routeBoundSessionMessage({
-                channel: message.channel,
-                target: message.target,
-                senderId: message.senderId,
-                ...(message.messageId ? { messageId: message.messageId } : {}),
-                text: message.text,
-                isDirectMessage: isDirectMessage(message),
-            });
-        }
-        catch {
-            // The bridge being unreachable must not swallow the message: fall
-            // through so Mimi still answers, which is the pre-binding behavior.
-            return next();
-        }
+        // A transport timeout is ambiguous: the Work Session may already have
+        // accepted this message. Leave retry to the durable inbox instead of also
+        // handing the same message to Mimi.
+        const disposition = await options.desktop.routeBoundSessionMessage({
+            channel: message.channel,
+            target: message.target,
+            senderId: message.senderId,
+            ...(message.messageId ? { messageId: message.messageId } : {}),
+            text: message.text,
+            isDirectMessage: isDirectMessage(message),
+        });
         switch (disposition.kind) {
             case "not-bound":
                 return next();

package/dist/channel.d.ts

@@ -62,6 +62,8 @@ export interface ChannelMessage {
     channel: string;
     target: string;
     senderId: string;
+    /** Adapter-authenticated conversation kind; missing means unknown, never private. */
+    isDirectMessage?: boolean;
     text: string;
     attachments?: readonly ChatAttachment[];
     messageId?: string;

package/dist/cli.js

@@ -7,6 +7,7 @@ import { createChannelAdapterAsync } from "./adapter-factory.js";
 import { ChatGateway, createAllowlistMiddleware, createRateLimitMiddleware, } from "./chat-gateway.js";
 import { defaultGatewayConfigPath, loadGatewayConfig } from "./config.js";
 import { DesktopControlClient } from "./desktop-control-client.js";
+import { createBoundSessionChat } from "./bound-session-chat.js";
 import { CODE_SHELL_REMOTE_COMMANDS, createCodeShellRemoteCommands, createMimiPetChat, MIMI_MIN_CONCURRENT_MESSAGES_PER_TARGET, } from "./gateway.js";
 import { loginCodeShellWechat } from "./wechat-login.js";
 import { defaultWechatDataDirectory } from "./wechat-storage.js";
@@ -76,6 +77,7 @@ async function main(args = process.argv.slice(2)) {
         ]))));
         gateway.use(createRateLimitMiddleware(config.runtime.maxMessagesPerUserPerMinute));
         gateway.use(createCodeShellRemoteCommands({ desktop }));
+        gateway.use(createBoundSessionChat({ desktop }));
         gateway.use(createMimiPetChat({ desktop, channels: adapters }));
         console.log(`[code-shell-chat] gateway 已启动：${config.channels.map(({ channel }) => channel).join(", ")}`);
         const gatewayTask = gateway.run(shutdown.signal);

package/dist/dingtalk.js

@@ -17,6 +17,7 @@ export function parseDingTalkTextMessage(data, messageId) {
         channel: "dingtalk",
         target: message.conversationId,
         senderId: message.senderStaffId || message.senderId,
+        isDirectMessage: message.conversationType === "1",
         text: message.text.content,
         ...(messageId ? { messageId } : {}),
         metadata: {

package/dist/discord.js

@@ -43,6 +43,7 @@ export class DiscordAdapter {
                 channel: this.channel,
                 target: message.channelId,
                 senderId: message.author.id,
+                isDirectMessage: message.channel.type === ChannelType.DM,
                 text: message.content,
                 messageId: message.id,
                 ...(attachments.length > 0 ? { attachments } : {}),
@@ -91,6 +92,7 @@ export class DiscordAdapter {
             channel: this.channel,
             target,
             senderId: interaction.user.id,
+            isDirectMessage: interaction.channel?.type === ChannelType.DM,
             text: `/${interaction.commandName}`,
             messageId: interaction.id,
         }));

package/dist/gateway.js

@@ -186,6 +186,7 @@ async function buildMimiOutgoingReplies(options, message, capabilities, configur
             channel: message.channel,
             target: message.target,
             senderId: message.senderId,
+            isDirectMessage: message.isDirectMessage === true,
             ...(message.messageId ? { messageId: message.messageId } : {}),
             capabilities,
             channels: gatewayChannelCatalog(message.channel, capabilities, configuredChannels),

package/dist/lark.js

@@ -37,6 +37,7 @@ export class LarkAdapter {
                     channel: this.channel,
                     target: event.message.chat_id,
                     senderId: sender,
+                    isDirectMessage: event.message.chat_type === "p2p",
                     text,
                     ...(event.message.message_id ? { messageId: event.message.message_id } : {}),
                     ...(attachment ? { attachments: [attachment] } : {}),

package/dist/line.js

@@ -73,6 +73,7 @@ export class LineAdapter {
                     channel: this.channel,
                     target,
                     senderId,
+                    isDirectMessage: event.source?.type === "user",
                     text,
                     ...(id ? { messageId: id } : {}),
                     ...(attachment ? { attachments: [attachment] } : {}),

package/dist/mattermost.js

@@ -143,6 +143,7 @@ export class MattermostAdapter {
             channel: this.channel,
             target: post.channel_id,
             senderId: post.user_id,
+            isDirectMessage: event.data.channel_type === "D",
             text: post.message ?? "",
             ...(post.id ? { messageId: post.id } : {}),
             ...(attachments.length > 0 ? { attachments } : {}),

package/dist/protocol.d.ts

@@ -77,6 +77,8 @@ export interface PetChatRequest {
         channel: string;
         target: string;
         senderId: string;
+        /** Adapter-authenticated private-chat signal; absent for legacy clients. */
+        isDirectMessage?: boolean;
         messageId?: string;
         /** Adapter-declared implementation capabilities for this exact route. */
         capabilities: ChannelCapabilities;

package/dist/slack.js

@@ -58,6 +58,7 @@ export class SlackAdapter {
                 channel: this.channel,
                 target: event.channel,
                 senderId: event.user,
+                isDirectMessage: event.channel_type === "im",
                 text,
                 ...(event.client_msg_id || event.ts
                     ? { messageId: String(event.client_msg_id ?? event.ts) }
@@ -73,10 +74,17 @@ export class SlackAdapter {
                 await ack();
                 return;
             }
+            // A user can name a channel "directmessage"; only Slack's conversation
+            // metadata can distinguish a DM from a private channel or group DM.
+            const isDirectMessage = await this.web.conversations
+                .info({ channel: body.channel_id })
+                .then(({ channel }) => channel?.is_im === true)
+                .catch(() => false);
             await dispatchSafely(handler, {
                 channel: this.channel,
                 target: body.channel_id,
                 senderId: body.user_id,
+                isDirectMessage,
                 text: [body.command, typeof body.text === "string" ? body.text : ""]
                     .filter(Boolean)
                     .join(" "),

package/dist/teams.js

@@ -97,6 +97,7 @@ export class TeamsAdapter {
                     channel: this.channel,
                     target,
                     senderId,
+                    isDirectMessage: activity.conversation.conversationType === "personal",
                     text,
                     ...(activity.id ? { messageId: activity.id } : {}),
                     ...(attachments.length > 0 ? { attachments } : {}),

package/dist/telegram.js

@@ -181,6 +181,7 @@ function toChannelMessage(update, download) {
         channel: "telegram",
         target: String(message.chat.id),
         senderId: String(sender),
+        isDirectMessage: message.chat.type === "private",
         text,
         ...(attachments.length > 0 ? { attachments } : {}),
         ...(message.message_id === undefined ? {} : { messageId: String(message.message_id) }),

package/dist/wechat.js

@@ -470,6 +470,7 @@ export class WechatAdapter {
             channel: this.channel,
             target: senderId,
             senderId,
+            isDirectMessage: !raw.group_id,
             text,
             ...(attachments.length > 0 ? { attachments } : {}),
             messageId: raw.message_id === undefined ? undefined : String(raw.message_id),

package/dist/wecom.js

@@ -24,6 +24,7 @@ export class WeComAdapter {
                 channel: this.channel,
                 target: message.chatid ?? message.from.userid,
                 senderId: message.from.userid,
+                isDirectMessage: message.chattype === "single",
                 text: message.text.content,
                 ...(frame.headers?.req_id
                     ? { messageId: frame.headers.req_id }
@@ -120,6 +121,7 @@ export class WeComAdapter {
             channel: this.channel,
             target: message.chatid ?? message.from.userid,
             senderId: message.from.userid,
+            isDirectMessage: message.chattype === "single",
             text,
             messageId: message.msgid,
             ...(attachments.length > 0 ? { attachments } : {}),

package/dist/whatsapp.js

@@ -51,6 +51,7 @@ export class WhatsAppAdapter {
                     channel: this.channel,
                     target: message.from,
                     senderId: message.from,
+                    isDirectMessage: !message.group_id,
                     text,
                     ...(message.id ? { messageId: message.id } : {}),
                     ...(media?.attachment ? { attachments: [media.attachment] } : {}),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cjhyy/code-shell-chat",
-  "version": "0.9.17",
+  "version": "0.9.19",
   "description": "A standalone, transport-agnostic multi-channel chat gateway for Node.js and Bun.",
   "type": "module",
   "main": "./dist/index.js",