Ayuriel/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(feishu): improve cross-platform compatibility and path handling

Browse Source
This commit is contained in:
Tang QI 2026-03-19 20:55:24 +08:00
parent f648fe4f43
commit 12285568c8
1 changed files with 20 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

39
extensions/feishu/src/media.ts
View File

@ -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)
}
}