@yaoyuanchao/dingtalk 1.4.11 → 1.4.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yaoyuanchao/dingtalk",
-  "version": "1.4.11",
+  "version": "1.4.12",
   "type": "module",
   "description": "DingTalk channel plugin for Clawdbot with Stream Mode support",
   "license": "MIT",

package/src/api.ts

@@ -497,7 +497,7 @@ export const cleanupOldPictures = cleanupOldMedia;
 
 /**
  * Upload a file to DingTalk and get media_id
- * Uses the robot messageFiles upload API
+ * Uses the legacy oapi media upload endpoint (the new API doesn't exist)
  */
 export async function uploadMediaFile(params: {
   clientId: string;
@@ -514,16 +514,9 @@ export async function uploadMediaFile(params: {
     const boundary = `----DingTalkBoundary${Date.now()}`;
     const fileType = params.fileType || 'file';
 
-    // Construct multipart body
+    // Construct multipart body - oapi uses "media" as the file field name
     const parts: Buffer[] = [];
 
-    // Add robotCode field
-    parts.push(Buffer.from(
-      `--${boundary}\r\n` +
-      `Content-Disposition: form-data; name="robotCode"\r\n\r\n` +
-      `${params.robotCode}\r\n`
-    ));
-
     // Add file field
     parts.push(Buffer.from(
       `--${boundary}\r\n` +
@@ -538,15 +531,14 @@ export async function uploadMediaFile(params: {
 
     const body = Buffer.concat(parts);
 
-    // Make request
-    const url = `${DINGTALK_API_BASE}/robot/messageFiles/upload?type=${fileType}`;
+    // Use legacy oapi endpoint - the new v1.0 API doesn't have this endpoint
+    const url = `https://oapi.dingtalk.com/media/upload?access_token=${token}&type=${fileType}`;
 
     return new Promise((resolve) => {
       const urlObj = new URL(url);
       const req = https.request(urlObj, {
         method: 'POST',
         headers: {
-          'x-acs-dingtalk-access-token': token,
           'Content-Type': `multipart/form-data; boundary=${boundary}`,
           'Content-Length': body.length,
         },
@@ -558,12 +550,16 @@ export async function uploadMediaFile(params: {
         res.on('end', () => {
           try {
             const json = JSON.parse(buf);
-            if (json.mediaId) {
+            // oapi returns media_id (with underscore), not mediaId
+            if (json.media_id) {
+              console.log(`[dingtalk] File uploaded successfully: media_id=${json.media_id}`);
+              resolve({ mediaId: json.media_id });
+            } else if (json.mediaId) {
               console.log(`[dingtalk] File uploaded successfully: mediaId=${json.mediaId}`);
               resolve({ mediaId: json.mediaId });
             } else {
               console.warn(`[dingtalk] File upload failed:`, json);
-              resolve({ error: json.message || json.errmsg || 'Upload failed' });
+              resolve({ error: json.errmsg || json.message || 'Upload failed' });
             }
           } catch {
             resolve({ error: `Invalid response: ${buf}` });