@ascegu/teamily 1.0.30 → 1.0.32

package/package.json

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

package/src/monitor.ts

@@ -308,7 +308,10 @@ export class TeamilyMonitor {
 
     // Try to extract a real first-frame snapshot via ffmpeg; fall back to
     // a minimal 1x1 JPEG placeholder if ffmpeg is unavailable.
-    const snapshotBuf = (await extractVideoSnapshot(buffer)) ?? Buffer.from(MINIMAL_JPEG);
+    const snapshot = await extractVideoSnapshot(buffer);
+    const snapshotBuf = snapshot?.buffer ?? Buffer.from(MINIMAL_JPEG);
+    const snapWidth = snapshot?.width || meta.width || 1;
+    const snapHeight = snapshot?.height || meta.height || 1;
     const snapshotFile = new File([new Uint8Array(snapshotBuf)], "snapshot.jpg", {
       type: "image/jpeg",
     });
@@ -325,8 +328,8 @@ export class TeamilyMonitor {
       snapshotUUID: crypto.randomUUID(),
       snapshotSize: snapshotBuf.length,
       snapshotUrl: "",
-      snapshotWidth: meta.width || 1,
-      snapshotHeight: meta.height || 1,
+      snapshotWidth: snapWidth,
+      snapshotHeight: snapHeight,
       videoFile,
       snapshotFile,
     });

package/src/upload.ts

@@ -125,39 +125,53 @@ export function guessContentType(filePath: string): string {
  * Returns null if ffmpeg is unavailable or extraction fails.
  * The thumbnail is scaled down to fit within maxDim (default 320px).
  */
+export interface SnapshotResult {
+  buffer: Buffer;
+  width: number;
+  height: number;
+}
+
 export async function extractVideoSnapshot(
   videoBuf: Buffer,
-  maxDim = 320,
-): Promise<Buffer | null> {
-  try {
-    const { execFile } = await import("node:child_process");
-    const { promisify } = await import("node:util");
-    const { writeFile, readFile, unlink, mkdtemp } = await import("node:fs/promises");
-    const { tmpdir } = await import("node:os");
-    const execFileAsync = promisify(execFile);
+): Promise<SnapshotResult | null> {
+  const { execFile } = await import("node:child_process");
+  const { promisify } = await import("node:util");
+  const { writeFile, readFile, unlink, mkdtemp } = await import("node:fs/promises");
+  const { tmpdir } = await import("node:os");
+  const execFileAsync = promisify(execFile);
 
-    const dir = await mkdtemp(path.join(tmpdir(), "oc-snap-"));
+  let dir: string | undefined;
+  try {
+    dir = await mkdtemp(path.join(tmpdir(), "oc-snap-"));
     const inPath = path.join(dir, "input.mp4");
     const outPath = path.join(dir, "snapshot.jpg");
 
-    try {
-      await writeFile(inPath, videoBuf);
-      await execFileAsync("ffmpeg", [
-        "-i", inPath,
-        "-vframes", "1",
-        "-vf", `scale='min(${maxDim},iw)':'-1'`,
-        "-q:v", "6",
-        "-y", outPath,
-      ], { timeout: 10_000 });
-      return await readFile(outPath);
-    } finally {
-      await unlink(inPath).catch(() => {});
-      await unlink(outPath).catch(() => {});
-      const { rmdir } = await import("node:fs/promises");
-      await rmdir(dir).catch(() => {});
+    await writeFile(inPath, videoBuf);
+    await execFileAsync("ffmpeg", [
+      "-y",
+      "-i", inPath,
+      "-frames:v", "1",
+      outPath,
+    ], { timeout: 15_000 });
+
+    const snap = await readFile(outPath);
+    if (snap.length === 0) {
+      console.warn("[teamily] ffmpeg produced empty snapshot");
+      return null;
     }
-  } catch {
+    const dims = parseJpegDimensions(snap);
+    console.info(
+      `[teamily] extracted video snapshot: ${snap.length} bytes, ${dims.width}x${dims.height}`,
+    );
+    return { buffer: snap, width: dims.width, height: dims.height };
+  } catch (err) {
+    console.warn("[teamily] ffmpeg snapshot extraction failed:", err);
     return null;
+  } finally {
+    if (dir) {
+      const { rm } = await import("node:fs/promises");
+      await rm(dir, { recursive: true, force: true }).catch(() => {});
+    }
   }
 }
 
@@ -263,3 +277,28 @@ function findBox(
   }
   return null;
 }
+
+/** Parse width/height from a JPEG buffer by scanning for SOF markers. */
+function parseJpegDimensions(buf: Buffer): { width: number; height: number } {
+  let i = 0;
+  while (i + 1 < buf.length) {
+    if (buf[i] !== 0xff) { i++; continue; }
+    const marker = buf[i + 1];
+    // SOF0–SOF3 (0xC0–0xC3): baseline, progressive, etc.
+    if (marker >= 0xc0 && marker <= 0xc3 && i + 9 < buf.length) {
+      const height = buf.readUInt16BE(i + 5);
+      const width = buf.readUInt16BE(i + 7);
+      return { width, height };
+    }
+    // Skip marker segment (length follows the 2-byte marker)
+    if (marker === 0xd8 || marker === 0xd9) {
+      i += 2; // SOI/EOI have no payload
+    } else if (i + 3 < buf.length) {
+      const len = buf.readUInt16BE(i + 2);
+      i += 2 + len;
+    } else {
+      break;
+    }
+  }
+  return { width: 0, height: 0 };
+}