@ascegu/teamily 1.0.23 → 1.0.25

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ascegu/teamily",
-  "version": "1.0.23",
+  "version": "1.0.25",
   "description": "OpenClaw Teamily channel plugin - Team instant messaging server integration",
   "keywords": [
     "channel",

package/src/channel.ts

@@ -30,7 +30,12 @@ import {
 } from "./accounts.js";
 import { TeamilyConfigSchema } from "./config-schema.js";
 import type { CoreConfig } from "./config-schema.js";
-import { getTeamilyMonitor, startTeamilyMonitoring, stopTeamilyMonitoring, type TeamilyMonitor } from "./monitor.js";
+import {
+  getTeamilyMonitor,
+  startTeamilyMonitoring,
+  stopTeamilyMonitoring,
+  type TeamilyMonitor,
+} from "./monitor.js";
 import { normalizeTeamilyTarget, normalizeTeamilyAllowEntry } from "./normalize.js";
 import { probeTeamily } from "./probe.js";
 import { getTeamilyRuntime } from "./runtime.js";
@@ -65,6 +70,11 @@ export const teamilyPlugin: ChannelPlugin<ResolvedTeamilyAccount> = {
     threads: false,
     polls: false,
   },
+  agentPrompt: {
+    messageToolHints: () => [
+      "- To send a local file or image to the user, use MEDIA:./relative-path in your reply (e.g. MEDIA:./image.png). The path must be relative to the workspace. You can also use the message tool with media/path/filePath. Avoid absolute paths and ~ paths — they are blocked for security.",
+    ],
+  },
   reload: { configPrefixes: ["channels.teamily"] },
   setup: {
     resolveAccountId: ({ accountId, input }) => {
@@ -192,7 +202,12 @@ export const teamilyPlugin: ChannelPlugin<ResolvedTeamilyAccount> = {
         const fileName = media.fileName || path.basename(mediaUrl) || "media";
         const contentType = media.contentType || guessContentType(mediaUrl);
         const messageId = await sendMediaBuffer(
-          monitor, target, media.buffer, fileName, contentType, mediaUrl,
+          monitor,
+          target,
+          media.buffer,
+          fileName,
+          contentType,
+          mediaUrl,
         );
         return { channel: "teamily" as const, messageId };
       }
@@ -373,12 +388,14 @@ export const teamilyPlugin: ChannelPlugin<ResolvedTeamilyAccount> = {
                 if (!monitor)
                   throw new Error(`Teamily monitor not running for account ${accountId}`);
                 const target = normalizeTeamilyTarget(replyTarget);
-                const replyText = (payload as Record<string, unknown>)?.text as string | undefined
-                  ?? (payload as Record<string, unknown>)?.body as string | undefined
-                  ?? "";
+                const replyText =
+                  ((payload as Record<string, unknown>)?.text as string | undefined) ??
+                  ((payload as Record<string, unknown>)?.body as string | undefined) ??
+                  "";
 
                 // Send media attachments (first with caption, rest without).
                 const mediaUrls = resolveOutboundMediaUrls(payload as Record<string, unknown>);
+                let captionSent = false;
                 const sentMedia = await sendMediaWithLeadingCaption({
                   mediaUrls,
                   caption: replyText,
@@ -387,6 +404,7 @@ export const teamilyPlugin: ChannelPlugin<ResolvedTeamilyAccount> = {
                     // OpenIM doesn't support inline captions on media; send separately.
                     if (caption) {
                       await monitor.sendText(target, caption);
+                      captionSent = true;
                     }
                   },
                   onError: (error) => {
@@ -394,8 +412,8 @@ export const teamilyPlugin: ChannelPlugin<ResolvedTeamilyAccount> = {
                   },
                 });
 
-                // If no media was sent, send text only.
-                if (!sentMedia && replyText) {
+                // If no media was sent (or media failed), send text as a plain message.
+                if ((!sentMedia || !captionSent) && replyText) {
                   log?.info?.(
                     `[${accountId}] Sending reply to: ${replyTarget} (isGroup=${isGroup})`,
                   );

package/src/monitor.ts

@@ -20,6 +20,36 @@ export interface TeamilyMonitorOptions {
 type SdkModule = typeof import("@openim/client-sdk");
 type SdkInstance = ReturnType<SdkModule["getSDK"]>;
 
+// The OpenIM SDK uses FileReader internally for file upload (MD5 hashing of
+// chunks). FileReader is a browser-only API not available in Node.js, so we
+// provide a minimal polyfill that delegates to Blob.prototype.arrayBuffer().
+if (typeof globalThis.FileReader === "undefined") {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  (globalThis as any).FileReader = class FileReader {
+    result: ArrayBuffer | null = null;
+    error: unknown = null;
+    readyState = 0; // 0=EMPTY, 1=LOADING, 2=DONE
+    onload: ((ev: { target: FileReader }) => void) | null = null;
+    onerror: ((ev: unknown) => void) | null = null;
+
+    readAsArrayBuffer(blob: Blob): void {
+      this.readyState = 1;
+      blob
+        .arrayBuffer()
+        .then((buffer) => {
+          this.result = buffer;
+          this.readyState = 2;
+          this.onload?.({ target: this });
+        })
+        .catch((err) => {
+          this.error = err;
+          this.readyState = 2;
+          this.onerror?.(err);
+        });
+    }
+  };
+}
+
 // Lazy-loaded SDK to avoid top-level dynamic import issues
 let sdkModule: SdkModule | null = null;
 async function loadSDK() {
@@ -209,10 +239,23 @@ export class TeamilyMonitor {
   // Like Telegram's InputFile(buffer), pass a buffer and let the SDK upload.
 
   /** Send an image from a local buffer. SDK handles the upload to object storage. */
-  async sendImageBuffer(target: TeamilyMessageTarget, buffer: Buffer, fileName: string, contentType: string): Promise<string> {
+  async sendImageBuffer(
+    target: TeamilyMessageTarget,
+    buffer: Buffer,
+    fileName: string,
+    contentType: string,
+  ): Promise<string> {
     const sdk = this.requireSdk();
     const file = new File([new Uint8Array(buffer)], fileName, { type: contentType });
-    const picInfo = { uuid: "", type: contentType, width: 0, height: 0, size: buffer.length, url: "" };
+    const uuid = crypto.randomUUID();
+    const picInfo = {
+      uuid,
+      type: contentType,
+      width: 0,
+      height: 0,
+      size: buffer.length,
+      url: "",
+    };
     const created = await sdk.createImageMessageByFile({
       sourcePicture: picInfo,
       bigPicture: picInfo,
@@ -229,7 +272,12 @@ export class TeamilyMonitor {
   }
 
   /** Send a video from a local buffer. SDK handles the upload to object storage. */
-  async sendVideoBuffer(target: TeamilyMessageTarget, buffer: Buffer, fileName: string, contentType: string): Promise<string> {
+  async sendVideoBuffer(
+    target: TeamilyMessageTarget,
+    buffer: Buffer,
+    fileName: string,
+    contentType: string,
+  ): Promise<string> {
     const sdk = this.requireSdk();
     const videoFile = new File([new Uint8Array(buffer)], fileName, { type: contentType });
     // snapshotFile is required by the SDK type but we have no thumbnail; use an empty file.
@@ -239,10 +287,10 @@ export class TeamilyMonitor {
       duration: 0,
       videoType: contentType,
       snapshotPath: "",
-      videoUUID: "",
+      videoUUID: crypto.randomUUID(),
       videoUrl: "",
       videoSize: buffer.length,
-      snapshotUUID: "",
+      snapshotUUID: crypto.randomUUID(),
       snapshotSize: 0,
       snapshotUrl: "",
       snapshotWidth: 0,
@@ -259,11 +307,16 @@ export class TeamilyMonitor {
   }
 
   /** Send an audio/sound from a local buffer. SDK handles the upload to object storage. */
-  async sendAudioBuffer(target: TeamilyMessageTarget, buffer: Buffer, fileName: string, contentType: string): Promise<string> {
+  async sendAudioBuffer(
+    target: TeamilyMessageTarget,
+    buffer: Buffer,
+    fileName: string,
+    contentType: string,
+  ): Promise<string> {
     const sdk = this.requireSdk();
     const file = new File([new Uint8Array(buffer)], fileName, { type: contentType });
     const created = await sdk.createSoundMessageByFile({
-      uuid: "",
+      uuid: crypto.randomUUID(),
       soundPath: "",
       sourceUrl: "",
       dataSize: buffer.length,
@@ -279,13 +332,18 @@ export class TeamilyMonitor {
   }
 
   /** Send a document/file from a local buffer. SDK handles the upload to object storage. */
-  async sendFileBuffer(target: TeamilyMessageTarget, buffer: Buffer, fileName: string, contentType: string): Promise<string> {
+  async sendFileBuffer(
+    target: TeamilyMessageTarget,
+    buffer: Buffer,
+    fileName: string,
+    contentType: string,
+  ): Promise<string> {
     const sdk = this.requireSdk();
     const file = new File([new Uint8Array(buffer)], fileName, { type: contentType });
     const created = await sdk.createFileMessageByFile({
       filePath: "",
       fileName,
-      uuid: "",
+      uuid: crypto.randomUUID(),
       sourceUrl: "",
       fileSize: buffer.length,
       file,
@@ -328,13 +386,12 @@ function convertSdkMessage(msg: MessageItem, selfUserID: string): TeamilyMessage
   // Determine whether this message @-mentions the bot
   const isAtSelf =
     contentType === CONTENT_TYPES.AT_TEXT &&
-    (msg.atTextElem?.isAtSelf === true ||
-      (msg.atTextElem?.atUserList ?? []).includes(selfUserID));
+    (msg.atTextElem?.isAtSelf === true || (msg.atTextElem?.atUserList ?? []).includes(selfUserID));
 
   return {
     serverMsgID: msg.serverMsgID || msg.clientMsgID || `${Date.now()}_${Math.random()}`,
     sendID: msg.sendID || "unknown",
-    recvID: isGroupSession(sessionType) ? (msg.groupID || "") : (msg.recvID || selfUserID),
+    recvID: isGroupSession(sessionType) ? msg.groupID || "" : msg.recvID || selfUserID,
     content,
     contentType,
     sessionType,

package/src/upload.ts

@@ -2,7 +2,7 @@ import path from "node:path";
 
 /** Returns true if the media URL is a local file path rather than a remote URL. */
 export function isLocalMediaPath(mediaUrl: string): boolean {
-  return !(/^https?:\/\//i.test(mediaUrl));
+  return !/^https?:\/\//i.test(mediaUrl);
 }
 
 /** Detect the media category from a file extension. */

package/src/send.ts

@@ -1,271 +0,0 @@
-import { generateOperationID } from "./probe.js";
-import type { ResolvedTeamilyAccount, TeamilyMessageTarget } from "./types.js";
-import { CONTENT_TYPES, SESSION_TYPES } from "./types.js";
-
-export interface SendTeamilyMessageParams {
-  account: ResolvedTeamilyAccount;
-  target: TeamilyMessageTarget;
-  text: string;
-  replyToId?: string;
-  fetchImpl?: typeof fetch;
-}
-
-export interface SendTeamilyMediaParams {
-  account: ResolvedTeamilyAccount;
-  target: TeamilyMessageTarget;
-  mediaUrl: string;
-  mediaType: "image" | "video" | "audio" | "file";
-  caption?: string;
-  fetchImpl?: typeof fetch;
-}
-
-export interface TeamilySendResult {
-  success: boolean;
-  messageId?: string;
-  serverMsgID?: string;
-  clientMsgID?: string;
-  error?: string;
-}
-
-/**
- * Send a text message via Teamily REST API.
- *
- * @param params - Send parameters
- * @returns Send result with message ID or error
- */
-export async function sendMessageTeamily(
-  params: SendTeamilyMessageParams,
-): Promise<TeamilySendResult> {
-  const { account, target, text, replyToId, fetchImpl = fetch } = params;
-
-  const url = `${account.apiURL}/msg/send_msg`;
-
-  const payload = buildSendMessagePayload({
-    sendID: account.userID,
-    target,
-    content: { content: text },
-    contentType: CONTENT_TYPES.TEXT,
-    replyToId,
-  });
-
-  try {
-    const response = await fetchImpl(url, {
-      method: "POST",
-      headers: {
-        "Content-Type": "application/json",
-        operationID: generateOperationID(),
-        token: account.token,
-      },
-      body: JSON.stringify(payload),
-    });
-
-    if (!response.ok) {
-      const errorText = await response.text();
-      return {
-        success: false,
-        error: `HTTP ${response.status}: ${errorText}`,
-      };
-    }
-
-    const data = (await response.json()) as {
-      errCode: number;
-      errMsg: string;
-      data?: {
-        serverMsgID: string;
-        clientMsgID: string;
-        sendTime: number;
-      };
-    };
-
-    if (data.errCode !== 0) {
-      return {
-        success: false,
-        error: data.errMsg || "Send failed",
-      };
-    }
-
-    return {
-      success: true,
-      messageId: data.data?.serverMsgID,
-      serverMsgID: data.data?.serverMsgID,
-      clientMsgID: data.data?.clientMsgID,
-    };
-  } catch (error) {
-    return {
-      success: false,
-      error: error instanceof Error ? error.message : String(error),
-    };
-  }
-}
-
-/**
- * Send a media message via Teamily REST API.
- *
- * @param params - Send parameters
- * @returns Send result with message ID or error
- */
-export async function sendMediaTeamily(params: SendTeamilyMediaParams): Promise<TeamilySendResult> {
-  const { account, target, mediaUrl, mediaType, caption, fetchImpl = fetch } = params;
-
-  const url = `${account.apiURL}/msg/send_msg`;
-
-  let content: Record<string, unknown>;
-  let contentType: number;
-
-  switch (mediaType) {
-    case "image":
-      contentType = CONTENT_TYPES.PICTURE;
-      content = {
-        sourcePicture: {
-          uuid: generateOperationID(),
-          type: "public",
-          width: 0,
-          height: 0,
-          url: mediaUrl,
-        },
-      };
-      break;
-
-    case "video":
-      contentType = CONTENT_TYPES.VIDEO;
-      content = {
-        videoPath: mediaUrl,
-        videoUUID: generateOperationID(),
-        videoUrl: mediaUrl,
-        videoType: "mp4",
-        videoSize: 0,
-        duration: 0,
-        snapshotUUID: `${generateOperationID()}_snap`,
-        snapshotUrl: "",
-        snapshotSize: 0,
-        snapshotWidth: 0,
-        snapshotHeight: 0,
-      };
-      break;
-
-    case "audio":
-      contentType = CONTENT_TYPES.VOICE;
-      content = {
-        uuid: generateOperationID(),
-        soundPath: mediaUrl,
-        sourceUrl: mediaUrl,
-        dataSize: 0,
-        duration: 0,
-        soundType: "mp3",
-      };
-      break;
-
-    case "file":
-      contentType = CONTENT_TYPES.FILE;
-      content = {
-        uuid: generateOperationID(),
-        fileName: mediaUrl.split("/").pop() || "file",
-        fileSize: 0,
-        sourceUrl: mediaUrl,
-        fileType: "application/octet-stream",
-      };
-      break;
-
-    default:
-      return {
-        success: false,
-        error: `Unsupported media type: ${mediaType}`,
-      };
-  }
-
-  // Add caption to text content for media messages
-  if (caption) {
-    content.text = caption;
-  }
-
-  const payload = buildSendMessagePayload({
-    sendID: account.userID,
-    target,
-    content,
-    contentType,
-  });
-
-  try {
-    const response = await fetchImpl(url, {
-      method: "POST",
-      headers: {
-        "Content-Type": "application/json",
-        operationID: generateOperationID(),
-        token: account.token,
-      },
-      body: JSON.stringify(payload),
-    });
-
-    if (!response.ok) {
-      const errorText = await response.text();
-      return {
-        success: false,
-        error: `HTTP ${response.status}: ${errorText}`,
-      };
-    }
-
-    const data = (await response.json()) as {
-      errCode: number;
-      errMsg: string;
-      data?: {
-        serverMsgID: string;
-        clientMsgID: string;
-        sendTime: number;
-      };
-    };
-
-    if (data.errCode !== 0) {
-      return {
-        success: false,
-        error: data.errMsg || "Send failed",
-      };
-    }
-
-    return {
-      success: true,
-      messageId: data.data?.serverMsgID,
-      serverMsgID: data.data?.serverMsgID,
-      clientMsgID: data.data?.clientMsgID,
-    };
-  } catch (error) {
-    return {
-      success: false,
-      error: error instanceof Error ? error.message : String(error),
-    };
-  }
-}
-
-/**
- * Build the send message payload for Teamily API.
- */
-function buildSendMessagePayload(params: {
-  sendID: string;
-  target: TeamilyMessageTarget;
-  content: Record<string, unknown>;
-  contentType: number;
-  replyToId?: string;
-}): Record<string, unknown> {
-  const { sendID, target, content, contentType, replyToId } = params;
-
-  const payload: Record<string, unknown> = {
-    sendID,
-    recvID: target.type === "user" ? target.id : "",
-    groupID: target.type === "group" ? target.id : "",
-    content,
-    contentType,
-    sessionType: target.type === "group" ? SESSION_TYPES.GROUP : SESSION_TYPES.SINGLE,
-    isOnlineOnly: false,
-  };
-
-  if (replyToId) {
-    payload.quote = {
-      text: "",
-      content: {},
-      isReact: false,
-      userID: "",
-      msgID: replyToId,
-    };
-  }
-
-  return payload;
-}