From 12285568c8571db4b45addc55383c3096a5b5569 Mon Sep 17 00:00:00 2001
From: Tang QI <tangqihy@gmail.com>
Date: Thu, 19 Mar 2026 20:55:24 +0800
Subject: [PATCH] fix(feishu): improve cross-platform compatibility and path
 handling

---
 extensions/feishu/src/media.ts | 39 +++++++++++++++++-----------------
 1 file changed, 20 insertions(+), 19 deletions(-)

diff --git a/extensions/feishu/src/media.ts b/extensions/feishu/src/media.ts
index 8607c2a8839..bd9bb76c65d 100644
--- a/extensions/feishu/src/media.ts
+++ b/extensions/feishu/src/media.ts
@@ -468,30 +468,31 @@ export async function sendMediaFeishu(params: {
     let duration: number | undefined;
     if ((fileType === "opus" || fileType === "mp4") && mediaUrl && !mediaUrl.startsWith("http")) {
       try {
-        // Check if ffprobe is available and file exists
-        if (fs.existsSync(mediaUrl)) {
-          const ffprobeCheck = spawnSync("which", ["ffprobe"], { encoding: "utf-8" });
-          if (ffprobeCheck.status === 0) {
-            const result = spawnSync(
-              "ffprobe",
-              [
-                "-v", "error",
-                "-show_entries", "format=duration",
-                "-of", "default=noprint_wrappers=1:nokey=1",
-                mediaUrl,
-              ],
-              { encoding: "utf-8", timeout: 10000 }
-            );
-            if (result.status === 0 && result.stdout) {
-              const durationSec = parseFloat(result.stdout.trim());
-              if (!isNaN(durationSec)) {
-                duration = Math.round(durationSec * 1000); // Convert to milliseconds
-              }
+        // Normalize file path (handle file:// URLs)
+        const normalizedPath = mediaUrl.startsWith("file://") ? mediaUrl.slice(7) : mediaUrl;
+        // Check if file exists and is a regular file (not a URL)
+        if (fs.existsSync(normalizedPath) && fs.statSync(normalizedPath).isFile()) {
+          // Try to run ffprobe directly - works on both POSIX and Windows
+          const result = spawnSync(
+            "ffprobe",
+            [
+              "-v", "error",
+              "-show_entries", "format=duration",
+              "-of", "default=noprint_wrappers=1:nokey=1",
+              normalizedPath,
+            ],
+            { encoding: "utf-8", timeout: 10000 }
+          );
+          if (result.status === 0 && result.stdout) {
+            const durationSec = parseFloat(result.stdout.trim());
+            if (!isNaN(durationSec)) {
+              duration = Math.round(durationSec * 1000); // Convert to milliseconds
             }
           }
         }
       } catch (e) {
         // Silently ignore ffprobe errors - duration is optional
+        // This includes cases where ffprobe is not installed (ENOENT)
       }
     }