@soimy/dingtalk 3.6.11 → 3.7.0

package/src/{inbound-handler.ts → gateway/inbound-handler.ts}

@@ -2,109 +2,114 @@ import fs from "node:fs";
 import * as path from "node:path";
 import { formatInboundEnvelope } from "openclaw/plugin-sdk/channel-inbound";
 import { isAbortRequestText, isBtwRequestText } from "openclaw/plugin-sdk/reply-runtime";
-import { parseInlineDirectives } from "./messaging/inline-directives";
-import { normalizeAllowFrom, isSenderAllowed, resolveGroupAccess } from "./access-control";
-import { classifyAckReactionEmoji } from "./ack-reaction-classifier";
-import { attachNativeAckReaction } from "./ack-reaction-service";
-import { createDynamicAckReactionController } from "./ack-reaction/dynamic-ack-reaction-controller";
-import { getAccessToken } from "./auth";
-import {
-  createAICard,
-  commitAICardBlocks,
-  isCardInTerminalState,
-  recallAICardMessage,
-} from "./card-service";
+import { classifyAckReactionEmoji } from "../ack-reaction/ack-reaction-classifier";
+import { attachNativeAckReaction } from "../ack-reaction/ack-reaction-service";
+import { createDynamicAckReactionController } from "../ack-reaction/dynamic-ack-reaction-controller";
 import {
   invalidateAskUserQuestionsForScope,
   syncInvalidatedAskUserQuestionCards,
-} from "./card/ask-user-question";
+} from "../card/ask-user-question";
 import {
   getDingTalkQuestionContext,
   withDingTalkQuestionContext,
-} from "./card/ask-user-question-context";
-import { isCardRunStopRequested, registerCardRun, removeCardRun } from "./card/card-run-registry";
-import { renderStatusLine } from "./card/statusline-renderer";
-import { resolveConfiguredTaskModelMetadata } from "./card/task-model-metadata";
-import { dispatchDingTalkCardStopCommand } from "./command/card-stop-command";
-import { handleInboundCommandDispatch } from "./command/inbound-command-dispatch-service";
+} from "../card/ask-user-question-context";
+import { isCardRunStopRequested, registerCardRun, removeCardRun } from "../card/card-run-registry";
 import {
-  resolveAckReactionSetting,
-  resolveGroupConfig,
-  resolveRelativePath,
-  resolveRobotCode,
-} from "./config";
-import { buildLearningContextBlock, isLearningEnabled } from "./feedback-learning-service";
-import axios from "./http-client";
-import { dispatchInboundViaSessionQueue } from "./gateway/inbound-session-queue-dispatcher";
-import { setCurrentLogger } from "./logger-context";
-import { prepareMediaInput, resolveOutboundMediaType } from "./media-utils";
+  createAICard,
+  commitAICardBlocks,
+  isCardInTerminalState,
+  recallAICardMessage,
+} from "../card/card-service";
+import { renderStatusLine } from "../card/statusline-renderer";
+import { resolveConfiguredTaskModelMetadata } from "../card/task-model-metadata";
+import { dispatchDingTalkCardStopCommand } from "../command/card-stop-command";
+import { buildLearningContextBlock, isLearningEnabled } from "../command/feedback-learning-service";
+import { handleInboundCommandDispatch } from "../command/inbound-command-dispatch-service";
+import { extractAttachmentText } from "../messaging/attachment-text-extractor";
+import { deliverBtwReply, stripLeadingMentions } from "../messaging/btw-deliver";
+import { parseInlineDirectives } from "../messaging/inline-directives";
+import { prepareMediaInput, resolveOutboundMediaType } from "../messaging/media-utils";
 import {
   DEFAULT_MEDIA_CONTEXT_TTL_MS,
   DEFAULT_MESSAGE_CONTEXT_TTL_DAYS,
   upsertInboundMessageContext,
-} from "./message-context-store";
-import { extractMessageContent } from "./message-utils";
-import { extractAttachmentText } from "./messaging/attachment-text-extractor";
-import { deliverBtwReply, stripLeadingMentions } from "./messaging/btw-deliver";
-import { resolveQuotedRuntimeContext } from "./messaging/quoted-context";
+} from "../messaging/message-context-store";
+import { extractMessageContent } from "../messaging/message-utils";
+import {
+  clearProactiveRiskObservationsForTest,
+  getProactiveRiskObservationForAny,
+} from "../messaging/proactive-risk-registry";
+import { resolveQuotedRuntimeContext } from "../messaging/quoted-context";
 import {
   downloadGroupFile,
   getUnionIdByStaffId,
   resolveQuotedFile,
-} from "./messaging/quoted-file-service";
+} from "../messaging/quoted-file-service";
 import {
   buildInboundQuotedRef,
   createReplyQuotedRef,
   resolveQuotedRecord,
-} from "./messaging/quoted-ref";
-import { registerPeerId } from "./peer-id-registry";
+} from "../messaging/quoted-ref";
+import { createReplyStrategy } from "../messaging/reply-strategy";
+import type { DeliverPayload, ReplyStrategy } from "../messaging/reply-strategy-types";
+import { sendBySession, sendMessage, sendProactiveMedia } from "../messaging/send-service";
 import {
-  clearProactiveRiskObservationsForTest,
-  getProactiveRiskObservationForAny,
-} from "./proactive-risk-registry";
-import {
-  isReplySessionConflictError,
-  withReplySessionConflictRetry,
-} from "./gateway/reply-session-conflict";
-import { createReplyStrategy } from "./reply-strategy";
-import type { DeliverPayload } from "./reply-strategy-types";
-import { getDingTalkRuntime } from "./runtime";
-import { sendBySession, sendMessage, sendProactiveMedia } from "./send-service";
-import { acquireSessionLock } from "./session-lock";
-import {
-  clearSessionPeerOverride,
-  getSessionPeerOverride,
-  setSessionPeerOverride,
-} from "./session-peer-store";
-import { resolveDingTalkSessionPeer } from "./session-routing";
-import { getSessionState, initSessionState } from "./session-state";
-import { getAgentDisplayName } from "./targeting/agent-name-matcher";
-import {
-  buildAgentSessionKey,
-  dispatchSubAgents,
-  resolveMessageTarget,
-  sendUnmatchedAgentNotice,
-} from "./targeting/agent-routing";
-import { formatGroupMembers, noteGroupMember } from "./targeting/group-members-store";
+  normalizeAllowFrom,
+  isSenderAllowed,
+  resolveGroupAccess,
+} from "../platform/access-control";
+import { getAccessToken } from "../platform/auth";
 import {
-  upsertObservedGroupTarget,
-  upsertObservedUserTarget,
-} from "./targeting/target-directory-store";
-import { AICardStatus } from "./types";
+  resolveAckReactionSetting,
+  resolveGroupConfig,
+  resolveRelativePath,
+  resolveRobotCode,
+} from "../platform/config";
+import { setCurrentLogger } from "../platform/logger-context";
+import { getDingTalkRuntime } from "../platform/runtime";
+import { createRuntimeEventsFanout } from "../platform/runtime-events";
+import { getSessionState, initSessionState } from "../platform/session-state";
+import { AICardStatus } from "../platform/types";
 import type {
   DingTalkConfig,
   HandleDingTalkMessageParams,
   Logger,
   MediaFile,
   ResolvedDingTalkRoute,
-} from "./types";
+} from "../platform/types";
+import axios from "../shared/http-client";
 import {
   formatDingTalkErrorPayloadLog,
   getErrorMessage,
   getErrorResponseData,
   maskSensitiveData,
   parseBooleanLike,
-} from "./utils";
+} from "../shared/utils";
+import { getAgentDisplayName } from "../targeting/agent-name-matcher";
+import {
+  buildAgentSessionKey,
+  dispatchSubAgents,
+  resolveMessageTarget,
+  sendUnmatchedAgentNotice,
+} from "../targeting/agent-routing";
+import { formatGroupMembers, noteGroupMember } from "../targeting/group-members-store";
+import { registerPeerId } from "../targeting/peer-id-registry";
+import {
+  clearSessionPeerOverride,
+  getSessionPeerOverride,
+  setSessionPeerOverride,
+} from "../targeting/session-peer-store";
+import { resolveDingTalkSessionPeer } from "../targeting/session-routing";
+import {
+  upsertObservedGroupTarget,
+  upsertObservedUserTarget,
+} from "../targeting/target-directory-store";
+import { dispatchInboundViaSessionQueue } from "./inbound-session-queue-dispatcher";
+import {
+  isReplySessionConflictError,
+  withReplySessionConflictRetry,
+} from "./reply-session-conflict";
+import { acquireSessionLock } from "./session-lock";
 
 const DEFAULT_PROACTIVE_HINT_COOLDOWN_HOURS = 24;
 const MIN_THINKING_REACTION_VISIBLE_MS = 1200;
@@ -432,14 +437,25 @@ function buildGroupTurnContextPrompt(params: {
   conversationId: string;
   senderDingtalkId: string;
   senderName?: string;
+  mentionedDingtalkIds?: readonly string[];
 }): string {
   const sanitizedSenderName = sanitizeGroupPromptName(params.senderName) || "Unknown";
+  const mentionedDingtalkIds = [
+    ...new Set(
+      (params.mentionedDingtalkIds || [])
+        .map((value) => value.replace(/[\r\n,=]/g, " ").trim())
+        .filter(Boolean),
+    ),
+  ];
   return [
     "Current DingTalk group turn context:",
     `- conversationId: ${params.conversationId}`,
     `- senderDingtalkId: ${params.senderDingtalkId}`,
     `- senderName: ${sanitizedSenderName}`,
+    `- mentionedDingtalkIds: ${mentionedDingtalkIds.length > 0 ? mentionedDingtalkIds.join(", ") : "(none provided)"}`,
     "Treat senderDingtalkId and senderName as the authoritative sender for this turn. Do not guess the current sender from GroupMembers.",
+    "Treat mentionedDingtalkIds as raw structured IDs only; do not infer a display name or map them to text mentions without payload evidence.",
+    "mentionedDingtalkIds are webhook atUsers/richText IDs (for example $:LWCP_v1:$...), not staff IDs; they are not comparable with senderDingtalkId or GroupMembers ids and must not be used as send targets.",
   ].join("\n");
 }
 
@@ -1008,6 +1024,17 @@ async function handleDingTalkMessageInner(params: HandleDingTalkMessageParams):
   const quotedRef = buildInboundQuotedRef(data, extractedContent);
   const replyQuotedRef = createReplyQuotedRef(data.msgId);
   const content = extractedContent;
+  const mentionUserIds = [
+    ...new Set(
+      [
+        ...(content.atUserDingtalkIds || []),
+        ...(content.atMentions || []).map((mention) => mention.userId),
+      ]
+        .filter((value): value is string => typeof value === "string")
+        .map((value) => value.trim())
+        .filter(Boolean),
+    ),
+  ];
   // Decide control-message bypasses once from the user-authored text.  Reusing
   // the result below keeps /stop out of the FIFO queue without calling the SDK
   // classifier a second time after attachment/OCR enrichment.
@@ -1082,7 +1109,7 @@ async function handleDingTalkMessageInner(params: HandleDingTalkMessageParams):
   // abort branch finalizes the card with the abort confirmation text instead of
   // sending a separate plain-text message.
   let useCardMode = dingtalkConfig.messageType === "card";
-  let currentAICard: import("./types").AICardInstance | undefined;
+  let currentAICard: import("../platform/types").AICardInstance | undefined;
   let questionCardTookOver = false;
 
   let cardFlightKey: string | undefined;
@@ -1248,6 +1275,9 @@ async function handleDingTalkMessageInner(params: HandleDingTalkMessageParams):
         quotedRef,
         senderId,
         senderName,
+        mentions: content.atMentions?.map((mention) => mention.name),
+        mentionUserIds,
+        chatType: isDirect ? "direct" : "group",
         createdAt: data.createAt,
         ttlMs: ttlDaysToMs(journalTTLDays),
         ttlReferenceMs: data.createAt,
@@ -1780,6 +1810,7 @@ async function handleDingTalkMessageInner(params: HandleDingTalkMessageParams):
             conversationId: groupId,
             senderDingtalkId: senderId,
             senderName,
+            mentionedDingtalkIds: mentionUserIds,
           }),
           groupConfig?.systemPrompt?.trim(),
         ]
@@ -2264,7 +2295,12 @@ async function handleDingTalkMessageInner(params: HandleDingTalkMessageParams):
         };
       }
     ).events;
+    const replyRuntimeEvents = createRuntimeEventsFanout(runtimeEvents, { log });
     const releaseSessionLock = await acquireSessionLock(route.sessionKey);
+    // Declared outside the dispatch try-block so the finally below can always
+    // release strategy-owned resources, including early returns that neither
+    // finalize nor abort (ask-user question-card takeover).
+    let strategyForCleanup: ReplyStrategy | undefined;
     const dynamicAckReactionController = createDynamicAckReactionController({
       enabled: shouldTrackDynamicAckReaction,
       initialReaction: resolvedAckReaction || "",
@@ -2275,7 +2311,7 @@ async function handleDingTalkMessageInner(params: HandleDingTalkMessageParams):
       conversationId: groupId,
       sessionKey: route.sessionKey,
       log,
-      runtimeEvents,
+      runtimeEvents: replyRuntimeEvents,
       onReactionDisposed: () => {
         ackReactionAttached = false;
       },
@@ -2350,7 +2386,9 @@ async function handleDingTalkMessageInner(params: HandleDingTalkMessageParams):
         isStopRequested: isCurrentCardStopRequested,
         inboundText: rawInboundText,
         taskMeta,
+        runtimeEvents: replyRuntimeEvents,
       });
+      strategyForCleanup = strategy;
 
       let deliveredFinalCount = 0;
       // Extracted as a thunk so reply-session init conflicts (raised by the
@@ -2534,19 +2572,36 @@ async function handleDingTalkMessageInner(params: HandleDingTalkMessageParams):
       }
       await strategy.finalize();
     } finally {
-      // Only remove the registry entry if no stop was requested. When a stop is
-      // in progress, card-stop-handler may still be running async operations
-      // (finalize card, hide button, gateway abort) that read the record.
-      // In that case, let the 30-minute TTL sweep handle cleanup.
-      if (currentOutTrackId && !isCardRunStopRequested(currentOutTrackId)) {
-        removeCardRun(currentOutTrackId);
+      try {
+        // Only remove the registry entry if no stop was requested. When a stop is
+        // in progress, card-stop-handler may still be running async operations
+        // (finalize card, hide button, gateway abort) that read the record.
+        // In that case, let the 30-minute TTL sweep handle cleanup.
+        if (currentOutTrackId && !isCardRunStopRequested(currentOutTrackId)) {
+          removeCardRun(currentOutTrackId);
+        }
+        // Guarantee strategy-owned cleanup on every exit path, including the
+        // question-card takeover return above. `dispose()` is idempotent, so a
+        // prior finalize()/abort() already having disposed is harmless. A
+        // failing dispose is logged and swallowed: it must not skip the
+        // remaining cleanup below.
+        if (strategyForCleanup) {
+          await strategyForCleanup.dispose().catch((disposeErr: unknown) => {
+            log?.warn?.(
+              `[DingTalk] Reply strategy cleanup failed for session ${route.sessionKey}: ${getErrorMessage(disposeErr)}`,
+            );
+          });
+        }
+        await waitForDynamicAckDispose({
+          dispose: () => dynamicAckReactionController.dispose(MIN_THINKING_REACTION_VISIBLE_MS),
+          log,
+          sessionKey: route.sessionKey,
+        });
+      } finally {
+        // Unconditional: a session lock that never releases blocks every later
+        // message on this session, so nothing above may prevent it.
+        releaseSessionLock();
       }
-      await waitForDynamicAckDispose({
-        dispose: () => dynamicAckReactionController.dispose(MIN_THINKING_REACTION_VISIBLE_MS),
-        log,
-        sessionKey: route.sessionKey,
-      });
-      releaseSessionLock();
     }
   } finally {
     if (cardFlightKey) {

package/src/gateway/inbound-session-queue-dispatcher.ts

@@ -21,13 +21,20 @@
 // orchestrator, adapted to soimy's blocking gateway contract (we await each
 // task so the gateway's per-message dedup stays correct).
 
-import { attachNativeAckReaction } from "../ack-reaction-service";
+import { attachNativeAckReaction } from "../ack-reaction/ack-reaction-service";
 import {
   createAICard,
   isCardInTerminalState,
   recallAICardMessage,
   streamAICard,
-} from "../card-service";
+} from "../card/card-service";
+import { sendMessage } from "../messaging/send-service";
+import type {
+  AICardInstance,
+  DingTalkConfig,
+  DingTalkInboundMessage,
+  Logger,
+} from "../platform/types";
 import {
   chainInboundSessionTask,
   getInboundSessionQueueDepth,
@@ -37,8 +44,6 @@ import {
   MAX_INBOUND_SESSION_QUEUE_WAIT_MS,
   pickQueueBusyAckPhrase,
 } from "./inbound-session-queue";
-import { sendMessage } from "../send-service";
-import type { AICardInstance, DingTalkConfig, DingTalkInboundMessage, Logger } from "../types";
 
 export interface InboundQueueDispatchInput {
   accountId: string;
@@ -138,13 +143,14 @@ export async function dispatchInboundViaSessionQueue<T>(
   // slot below. Otherwise a burst of inbound messages can all observe the
   // same pre-await depth and each pass the cap check.
   let queuedAckState: "queued" | "timed-out" = "queued";
-  const preCreatedCardPromise = wasBusy && shouldPrepareQueueAckCard(input)
-    ? tryPrepareQueueAckCard(input, () =>
-        queuedAckState === "timed-out"
-          ? { content: QUEUE_WAIT_TIMEOUT_ACK, finished: true }
-          : { content: pickQueueBusyAckPhrase(), finished: false },
-      )
-    : undefined;
+  const preCreatedCardPromise =
+    wasBusy && shouldPrepareQueueAckCard(input)
+      ? tryPrepareQueueAckCard(input, () =>
+          queuedAckState === "timed-out"
+            ? { content: QUEUE_WAIT_TIMEOUT_ACK, finished: true }
+            : { content: pickQueueBusyAckPhrase(), finished: false },
+        )
+      : undefined;
   // Chain onto the prior task for this conversation and AWAIT. Awaiting (rather
   // than fire-and-forget) preserves the gateway's per-message dedup:
   // `markMessageProcessed` runs only after this message truly completes, so a
@@ -153,9 +159,7 @@ export async function dispatchInboundViaSessionQueue<T>(
     return await chainInboundSessionTask(
       queueKey,
       async () => {
-        const preCreatedCard = preCreatedCardPromise
-          ? await preCreatedCardPromise
-          : undefined;
+        const preCreatedCard = preCreatedCardPromise ? await preCreatedCardPromise : undefined;
         if (!preCreatedCard) {
           return handler(undefined);
         }

package/src/{session-lock.ts → gateway/session-lock.ts}

@@ -13,22 +13,22 @@ export const SESSION_LOCK_NAMESPACE_POLICY = "memory-only" as const;
  * Different session keys run in parallel; same key runs serially.
  */
 export async function acquireSessionLock(sessionKey: string): Promise<() => void> {
-    let release!: () => void;
-    const gate = new Promise<void>((r) => {
-        release = r;
-    });
-    const prev = locks.get(sessionKey) ?? Promise.resolve();
-    locks.set(sessionKey, gate);
-    await prev;
-    return () => {
-        if (locks.get(sessionKey) === gate) {
-            locks.delete(sessionKey);
-        }
-        release();
-    };
+  let release!: () => void;
+  const gate = new Promise<void>((r) => {
+    release = r;
+  });
+  const prev = locks.get(sessionKey) ?? Promise.resolve();
+  locks.set(sessionKey, gate);
+  await prev;
+  return () => {
+    if (locks.get(sessionKey) === gate) {
+      locks.delete(sessionKey);
+    }
+    release();
+  };
 }
 
 /** Visible for testing only. */
 export function _getLocksMapForTest(): Map<string, Promise<void>> {
-    return locks;
+  return locks;
 }

package/src/messaging/attachment-text-extractor.ts

@@ -1,6 +1,6 @@
 import fs from "node:fs/promises";
 import path from "node:path";
-import type { AttachmentTextSource } from "../types";
+import type { AttachmentTextSource } from "../platform/types";
 
 const MAX_EXTRACTED_TEXT_CHARS = 6000;
 const MAX_ATTACHMENT_EXTRACT_BYTES = 2 * 1024 * 1024;
@@ -37,7 +37,8 @@ function isTextLikeMimeType(mimeType: string | undefined): boolean {
 function normalizeWhitespace(text: string): string {
   return text
     .replace(/\r\n/g, "\n")
-    .split("\u0000").join("")
+    .split("\u0000")
+    .join("")
     .replace(/[ \t]{2,}/g, " ")
     .replace(/[ \t]+\n/g, "\n")
     .replace(/\n{3,}/g, "\n\n")
@@ -50,7 +51,9 @@ function limitExtractedText(text: string): AttachmentTextExtractionResult | null
     return null;
   }
   const truncated = normalized.length > MAX_EXTRACTED_TEXT_CHARS;
-  const limited = truncated ? `${normalized.slice(0, MAX_EXTRACTED_TEXT_CHARS)}\n\n[内容已截断]` : normalized;
+  const limited = truncated
+    ? `${normalized.slice(0, MAX_EXTRACTED_TEXT_CHARS)}\n\n[内容已截断]`
+    : normalized;
   return {
     text: limited,
     truncated,
@@ -68,10 +71,12 @@ function stripHtml(html: string): string {
     .replace(/>/gi, ">")
     .replace(/&/gi, "&")
     .replace(/'/gi, "'")
-    .replace(/"/gi, "\"");
+    .replace(/"/gi, '"');
 }
 
-async function extractTextLikeFile(filePath: string): Promise<AttachmentTextExtractionResult | null> {
+async function extractTextLikeFile(
+  filePath: string,
+): Promise<AttachmentTextExtractionResult | null> {
   const raw = await fs.readFile(filePath, "utf8");
   return limitExtractedText(raw);
 }
@@ -108,7 +113,11 @@ export async function extractAttachmentText(
   const mimeType = input.mimeType?.toLowerCase();
   const fileName = (input.fileName || path.basename(input.path)).toLowerCase();
 
-  if (mimeType?.startsWith("image/") || mimeType?.startsWith("audio/") || mimeType?.startsWith("video/")) {
+  if (
+    mimeType?.startsWith("image/") ||
+    mimeType?.startsWith("audio/") ||
+    mimeType?.startsWith("video/")
+  ) {
     return null;
   }
 
@@ -127,7 +136,12 @@ export async function extractAttachmentText(
     return extractPdf(input.path);
   }
 
-  if (mimeType === "text/html" || mimeType === "application/xhtml+xml" || fileName.endsWith(".html") || fileName.endsWith(".htm")) {
+  if (
+    mimeType === "text/html" ||
+    mimeType === "application/xhtml+xml" ||
+    fileName.endsWith(".html") ||
+    fileName.endsWith(".htm")
+  ) {
     return extractHtml(input.path);
   }
 

package/src/messaging/btw-deliver.ts

@@ -1,5 +1,5 @@
-import { sendMessage } from "../send-service";
-import type { DingTalkConfig, Logger } from "../types";
+import type { DingTalkConfig, Logger } from "../platform/types";
+import { sendMessage } from "./send-service";
 
 const MAX_QUESTION_LENGTH = 80;
 const LEADING_MENTIONS_RE = /^(?:@\S+\s+)*/u;

package/src/messaging/channel-actions.ts

@@ -3,12 +3,12 @@ import type { ChannelMessageActionAdapter } from "openclaw/plugin-sdk/channel-co
 import type { OpenClawConfig } from "openclaw/plugin-sdk/core";
 import { readStringParam } from "openclaw/plugin-sdk/param-readers";
 import { extractToolSend } from "openclaw/plugin-sdk/tool-send";
-import { getConfig, stripTargetPrefix } from "../config";
-import { getLogger } from "../logger-context";
-import { resolveOriginalPeerId } from "../peer-id-registry";
-import { hasConfiguredSecretInput } from "../secret-input";
-import { sendMedia, sendMessage } from "../send-service";
-import { parseBooleanLike } from "../utils";
+import { getConfig, stripTargetPrefix } from "../platform/config";
+import { getLogger } from "../platform/logger-context";
+import { hasConfiguredSecretInput } from "../platform/secret-input";
+import { parseBooleanLike } from "../shared/utils";
+import { resolveOriginalPeerId } from "../targeting/peer-id-registry";
+import { sendMedia, sendMessage } from "./send-service";
 
 function readBooleanLikeParam(params: Record<string, unknown>, key: string): boolean | undefined {
   return parseBooleanLike(params[key]);
@@ -67,7 +67,15 @@ export function createDingTalkMessageActions(): ChannelMessageActionAdapter {
     describeMessageTool: ({ cfg }) => describeDingTalkMessageTool(cfg),
     supportsAction: ({ action }) => action === "send",
     extractToolSend: ({ args }) => extractToolSend(args, "sendMessage"),
-    handleAction: async ({ action, params, cfg, accountId, dryRun, mediaLocalRoots, sessionKey }) => {
+    handleAction: async ({
+      action,
+      params,
+      cfg,
+      accountId,
+      dryRun,
+      mediaLocalRoots,
+      sessionKey,
+    }) => {
       if (action !== "send") {
         throw new Error(`Action ${action} is not supported for provider dingtalk.`);
       }

package/src/messaging/channel-outbound.ts

@@ -1,11 +1,11 @@
 import { randomUUID } from "node:crypto";
-import { getConfig } from "../config";
-import { getLogger } from "../logger-context";
-import { getDingTalkRuntime } from "../runtime";
-import { sendMedia, sendMessage } from "../send-service";
+import { getConfig } from "../platform/config";
+import { getLogger } from "../platform/logger-context";
+import { getDingTalkRuntime } from "../platform/runtime";
+import type { DingTalkChannelPlugin } from "../platform/types";
+import { formatDingTalkErrorPayloadLog, parseBooleanLike } from "../shared/utils";
 import { normalizeResolvedDingTalkTarget } from "../targeting/target-directory-adapter";
-import type { DingTalkChannelPlugin } from "../types";
-import { formatDingTalkErrorPayloadLog, parseBooleanLike } from "../utils";
+import { sendMedia, sendMessage } from "./send-service";
 
 function readBooleanLikeParam(params: Record<string, unknown>, key: string): boolean | undefined {
   return parseBooleanLike(params[key]);
@@ -66,7 +66,9 @@ export function createDingTalkOutbound(): NonNullable<DingTalkChannelPlugin["out
         };
       } catch (err: any) {
         if (err?.response?.data !== undefined) {
-          effectiveLog?.error?.(formatDingTalkErrorPayloadLog("outbound.sendText", err.response.data));
+          effectiveLog?.error?.(
+            formatDingTalkErrorPayloadLog("outbound.sendText", err.response.data),
+          );
         }
         throw new Error(
           typeof err?.response?.data === "string"
@@ -112,9 +114,10 @@ export function createDingTalkOutbound(): NonNullable<DingTalkChannelPlugin["out
         );
       }
 
-      const requestedMediaType = typeof providedMediaType === "string"
-        ? (providedMediaType as "image" | "voice" | "video" | "file")
-        : undefined;
+      const requestedMediaType =
+        typeof providedMediaType === "string"
+          ? (providedMediaType as "image" | "voice" | "video" | "file")
+          : undefined;
 
       try {
         const result = await sendMedia(config, to, rawMediaPath, {
@@ -144,7 +147,9 @@ export function createDingTalkOutbound(): NonNullable<DingTalkChannelPlugin["out
         };
       } catch (err: any) {
         if (err?.response?.data !== undefined) {
-          effectiveLog?.error?.(formatDingTalkErrorPayloadLog("outbound.sendMedia", err.response.data));
+          effectiveLog?.error?.(
+            formatDingTalkErrorPayloadLog("outbound.sendMedia", err.response.data),
+          );
         }
         throw new Error(
           typeof err?.response?.data === "string"