jev-cdp 0.1.5 → 0.1.6

package/README.md

@@ -27,11 +27,11 @@ This is an early experimental port. See [NOTICE.md](NOTICE.md) for source attrib
 Run the published CLI without adding it to a project. Bun must be installed for either command:
 
 ```bash
-bunx jev-cdp@0.1.5 help run
-npx -y jev-cdp@0.1.5 help run
+bunx jev-cdp@0.1.6 help run
+npx -y jev-cdp@0.1.6 help run
 ```
 
-Chrome with a CDP endpoint and `TYPESAFE_API_KEY` are required for browser runs. FFmpeg is required for `--recording`.
+Chrome with a CDP endpoint and `TYPESAFE_API_KEY` are required for browser runs. FFmpeg with H.264 encoding (`libx264`) is required for `--recording`. Jev selects `-fps_mode vfr` when FFmpeg supports it and falls back to `-vsync vfr` for older builds. Run `jev-cdp doctor` to check the installed encoder with a short MP4 encode and decode.
 
 For source development:
 

package/dist/cli.js

@@ -3,7 +3,7 @@
 // package.json
 var package_default = {
   name: "jev-cdp",
-  version: "0.1.5",
+  version: "0.1.6",
   description: "A small Jev-powered bridge to Chrome through the Chrome DevTools Protocol.",
   type: "module",
   license: "MIT",
@@ -38,9 +38,9 @@ var package_default = {
 
 // src/browser.ts
 import { createHash } from "crypto";
-import { mkdir, mkdtemp, rm } from "fs/promises";
-import { tmpdir } from "os";
-import { dirname, join, resolve } from "path";
+import { mkdir, mkdtemp as mkdtemp2, rm as rm2 } from "fs/promises";
+import { tmpdir as tmpdir2 } from "os";
+import { dirname, join as join2, resolve } from "path";
 
 // src/cdp.ts
 async function listChromeTargets(cdpUrl) {
@@ -143,6 +143,88 @@ class CdpClient {
   }
 }
 
+// src/ffmpeg.ts
+import { mkdtemp, rm } from "fs/promises";
+import { tmpdir } from "os";
+import { join } from "path";
+function selectFrameSyncOption(help) {
+  if (/^\s*-fps_mode(?:\[:<stream_spec>\])?\s/m.test(help))
+    return "-fps_mode";
+  if (/^\s*-vsync\s/m.test(help))
+    return "-vsync";
+  throw new Error("FFmpeg supports neither -fps_mode nor -vsync");
+}
+async function run(command) {
+  const process2 = Bun.spawn(command, { stdout: "ignore", stderr: "pipe" });
+  const stderr = await new Response(process2.stderr).text();
+  return { code: await process2.exited, stderr };
+}
+async function recordingEncoder(ffmpeg = Bun.which("ffmpeg")) {
+  if (!ffmpeg)
+    throw new Error("FFmpeg not found on PATH");
+  const process2 = Bun.spawn([ffmpeg, "-hide_banner", "-h", "full"], { stdout: "pipe", stderr: "pipe" });
+  const [stdout, stderr, code] = await Promise.all([
+    new Response(process2.stdout).text(),
+    new Response(process2.stderr).text(),
+    process2.exited
+  ]);
+  if (code !== 0)
+    throw new Error(`Could not inspect FFmpeg options: ${stderr.slice(-500)}`);
+  return { path: ffmpeg, sync: selectFrameSyncOption(stdout + stderr) };
+}
+function recordingCommand(encoder, manifest, output) {
+  return [
+    encoder.path,
+    "-y",
+    "-f",
+    "concat",
+    "-safe",
+    "0",
+    "-i",
+    manifest,
+    encoder.sync,
+    "vfr",
+    "-c:v",
+    "libx264",
+    "-pix_fmt",
+    "yuv420p",
+    "-movflags",
+    "+faststart",
+    output
+  ];
+}
+async function renderRecording(encoder, manifest, output) {
+  const result = await run(recordingCommand(encoder, manifest, output));
+  if (result.code !== 0)
+    throw new Error(`Could not render recording: ${result.stderr.slice(-800)}`);
+}
+async function checkRecordingEncoder() {
+  const encoder = await recordingEncoder();
+  const directory = await mkdtemp(join(tmpdir(), "jev-cdp-ffmpeg-check-"));
+  try {
+    const frame = join(directory, "frame.ppm");
+    const manifest = join(directory, "frames.ffconcat");
+    const output = join(directory, "check.mp4");
+    await Bun.write(frame, Buffer.concat([Buffer.from(`P6
+16 16
+255
+`), Buffer.alloc(16 * 16 * 3, 90)]));
+    const quoted = frame.replaceAll("'", "'\\''");
+    await Bun.write(manifest, `ffconcat version 1.0
+file '${quoted}'
+duration 0.12
+file '${quoted}'
+`);
+    await renderRecording(encoder, manifest, output);
+    const decoded = await run([encoder.path, "-v", "error", "-i", output, "-f", "null", "-"]);
+    if (decoded.code !== 0)
+      throw new Error(`Could not decode recording: ${decoded.stderr.slice(-500)}`);
+    return `FFmpeg at ${encoder.path}; ${encoder.sync} vfr and libx264 encode/decode passed`;
+  } finally {
+    await rm(directory, { recursive: true, force: true });
+  }
+}
+
 // src/snapshot.js
 var snapshot_default = `// Ported from browser-use/jev-ultrafast at commit 452c1ad2 under the MIT License.
 (() => {
@@ -631,11 +713,10 @@ class Browser {
   async startRecording() {
     if (!this.#recordingPath)
       return;
-    if (!Bun.which("ffmpeg"))
-      throw new Error("--recording requires ffmpeg on PATH");
+    await recordingEncoder();
     const firstSegment = !this.#recordingDirectory;
     if (firstSegment)
-      this.#recordingDirectory = await mkdtemp(join(tmpdir(), "jev-cdp-recording-"));
+      this.#recordingDirectory = await mkdtemp2(join2(tmpdir2(), "jev-cdp-recording-"));
     await this.call("Page.enable");
     await this.call("Page.addScriptToEvaluateOnNewDocument", { source: RECORDING_CURSOR_INIT });
     const viewport = await this.evaluate("({width: innerWidth, height: innerHeight})");
@@ -643,7 +724,7 @@ class Browser {
       throw new Error("Could not read the recording viewport");
     await this.animateCursor(viewport.width / 2, viewport.height / 2);
     const initial = await this.call("Page.captureScreenshot", { format: "jpeg", quality: 70 });
-    const initialPath = join(this.#recordingDirectory, `${String(this.#recordingSequence++).padStart(6, "0")}.jpg`);
+    const initialPath = join2(this.#recordingDirectory, `${String(this.#recordingSequence++).padStart(6, "0")}.jpg`);
     await Bun.write(initialPath, Buffer.from(initial.data, "base64"));
     if (firstSegment)
       this.#recordingStartedAt = performance.now();
@@ -654,7 +735,7 @@ class Browser {
         return;
       });
       const sequence = this.#recordingSequence++;
-      const path = join(this.#recordingDirectory, `${String(sequence).padStart(6, "0")}.jpg`);
+      const path = join2(this.#recordingDirectory, `${String(sequence).padStart(6, "0")}.jpg`);
       const elapsedMs = sequence ? performance.now() - this.#recordingStartedAt : 0;
       this.#recordingWrites = this.#recordingWrites.then(async () => {
         await Bun.write(path, Buffer.from(frame.data, "base64"));
@@ -692,33 +773,12 @@ class Browser {
       lines.push(`file '${quoted(frame.path)}'`, `duration ${duration.toFixed(4)}`);
     }
     lines.push(`file '${quoted(this.#recordingFrames.at(-1).path)}'`);
-    const manifest = join(this.#recordingDirectory, "frames.ffconcat");
+    const manifest = join2(this.#recordingDirectory, "frames.ffconcat");
     await Bun.write(manifest, `${lines.join(`
 `)}
 `);
-    const process2 = Bun.spawn([
-      Bun.which("ffmpeg"),
-      "-y",
-      "-f",
-      "concat",
-      "-safe",
-      "0",
-      "-i",
-      manifest,
-      "-vsync",
-      "vfr",
-      "-c:v",
-      "libx264",
-      "-pix_fmt",
-      "yuv420p",
-      "-movflags",
-      "+faststart",
-      this.#recordingPath
-    ], { stdout: "ignore", stderr: "pipe" });
-    const stderr = await new Response(process2.stderr).text();
-    if (await process2.exited !== 0)
-      throw new Error(`Could not render recording: ${stderr.slice(-800)}`);
-    await rm(this.#recordingDirectory, { recursive: true, force: true });
+    await renderRecording(await recordingEncoder(), manifest, this.#recordingPath);
+    await rm2(this.#recordingDirectory, { recursive: true, force: true });
     this.#recordingDirectory = undefined;
   }
   async saveFinalScreenshot() {
@@ -1307,9 +1367,9 @@ ${child.text}`.slice(0, 6000);
 }
 
 // src/model.ts
-import { mkdtemp as mkdtemp2, readFile, rm as rm2 } from "fs/promises";
-import { tmpdir as tmpdir2 } from "os";
-import { join as join2 } from "path";
+import { mkdtemp as mkdtemp3, readFile, rm as rm3 } from "fs/promises";
+import { tmpdir as tmpdir3 } from "os";
+import { join as join3 } from "path";
 
 // src/questions.ts
 var NEXT_ACTION = `Advance the user's entire goal from the CURRENT page using one operation.
@@ -1549,9 +1609,9 @@ async function codexFieldText(context) {
   if (!["none", "low", "medium", "high", "xhigh", "max"].includes(reasoning)) {
     throw new Error("Unsupported Codex reasoning effort");
   }
-  const folder = await mkdtemp2(join2(tmpdir2(), "jev-codex-text-"));
-  const outputPath = join2(folder, "output.json");
-  const schemaPath = join2(folder, "schema.json");
+  const folder = await mkdtemp3(join3(tmpdir3(), "jev-codex-text-"));
+  const outputPath = join3(folder, "output.json");
+  const schemaPath = join3(folder, "schema.json");
   await Bun.write(schemaPath, JSON.stringify(text_value_schema_default));
   const childEnvironment = Object.fromEntries(Object.entries(process.env).filter(([name, value]) => value !== undefined && !["TYPESAFE_API_KEY", "TEXT_MODEL_API_KEY"].includes(name)).map(([name, value]) => [name, value]));
   const command = [
@@ -1614,7 +1674,7 @@ ${JSON.stringify(context)}`;
       usage: {}
     }];
   } finally {
-    await rm2(folder, { recursive: true, force: true });
+    await rm3(folder, { recursive: true, force: true });
   }
 }
 async function apiFieldText(context) {
@@ -2307,8 +2367,11 @@ async function doctor(args) {
       required: true
     });
   }
-  const ffmpeg = Bun.which("ffmpeg");
-  checks.push(ffmpeg ? { name: "recording", status: "ok", detail: `FFmpeg found at ${ffmpeg}`, required: false } : { name: "recording", status: "warning", detail: "FFmpeg not found; --recording will be unavailable", required: false });
+  try {
+    checks.push({ name: "recording", status: "ok", detail: await checkRecordingEncoder(), required: false });
+  } catch (error) {
+    checks.push({ name: "recording", status: "warning", detail: `${error instanceof Error ? error.message : String(error)}; --recording will be unavailable`, required: false });
+  }
   if (process.env.TEXT_MODEL_PROVIDER === "api") {
     checks.push(process.env.TEXT_MODEL_API_KEY ? { name: "text-helper", status: "ok", detail: `API helper configured (${process.env.TEXT_MODEL ?? "deepseek-chat"})`, required: false } : { name: "text-helper", status: "warning", detail: "TEXT_MODEL_PROVIDER=api but TEXT_MODEL_API_KEY is not set", required: false });
   } else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jev-cdp",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "description": "A small Jev-powered bridge to Chrome through the Chrome DevTools Protocol.",
   "type": "module",
   "license": "MIT",