autokap 1.3.29 → 1.3.31

package/dist/browser.js

@@ -1042,10 +1042,20 @@ export class Browser {
             });
             instance.browser = instance.context.browser();
             instance.persistentContext = true;
-            for (const page of instance.context.pages()) {
-                await page.close().catch(() => undefined);
+            // launchPersistentContext opens an initial about:blank page automatically.
+            // Reuse it instead of closing it + opening a new one: Chromium's --kiosk
+            // mode blocks `Target.createTarget` (one-window-only enforcement), so
+            // `context.newPage()` would fail with "Failed to open a new tab" — which
+            // is exactly what bricked 1.3.29. The init scripts registered below run
+            // on the next navigation regardless of when the page was created.
+            const existingPages = instance.context.pages();
+            instance.page = existingPages[0] ?? await instance.context.newPage();
+            // Defensive: close any extra pages Chromium might have opened (unlikely
+            // under --kiosk but cheap insurance). Keep index 0.
+            for (let i = 1; i < existingPages.length; i++) {
+                await existingPages[i].close().catch(() => undefined);
             }
-            logger.info(`[capture] Cloud clip capture: persistent Chromium profile seeded at ${cloudChromiumProfileDir}`);
+            logger.info(`[capture] Cloud clip capture: persistent Chromium profile seeded at ${cloudChromiumProfileDir} (reusing initial page, kiosk blocks Target.createTarget)`);
         }
         else {
             // Non-cloud clip capture keeps the regular browser + incognito context
@@ -1098,7 +1108,12 @@ export class Browser {
                 document.addEventListener('DOMContentLoaded', install, { once: true });
             }
         }, { styleId: CAPTURE_HIDE_STYLE_ID, css: getCaptureHideCSS() });
-        instance.page = await instance.context.newPage();
+        // Persistent context (cloud) already set instance.page by reusing the
+        // initial page launchPersistentContext opened. Only call newPage() for
+        // the regular browser+context path (Mac/Win/local Linux).
+        if (!instance.page) {
+            instance.page = await instance.context.newPage();
+        }
         // Cloud Run only: force the page window to fullscreen via CDP. The
         // --kiosk launch flag is silently ignored by Chromium for windows
         // created by Playwright via Target.createTarget (CDP) — kiosk only

package/dist/web-playwright-local.js

@@ -497,7 +497,15 @@ export class WebPlaywrightLocal {
             const ffmpegResult = await this.recording.ffmpegRecorder.stop();
             logger.info(`[capture] Clip ffmpeg+nvenc capture: ${(ffmpegResult.durationMs / 1000).toFixed(2)}s wall, ` +
                 `trim ${Math.round(ffmpegResult.trimStartMs)}ms, output ${ffmpegResult.outputPath}`);
-            await this.browser.closeContext();
+            // Persistent context (cloud) — DO NOT closeContext here. Closing the
+            // persistent context tears down the underlying browser process, which
+            // breaks the END_CLIP opcode handler's `adapter.getCurrentUrl()` call
+            // that runs immediately after this returns (and any subsequent
+            // NAVIGATE/CLICK/BEGIN_CLIP opcodes in the same preset). The CDP
+            // screencast path (non-ffmpeg branch below) closes context here to
+            // release the loop's CDP session — that's not needed for x11grab,
+            // ffmpeg has already finalized the MP4. Browser cleanup happens in
+            // Browser.close() at end of session.
             this.recording.finalized = true;
             this.recording.ffmpegResult = ffmpegResult;
             this.recording.sourcePath = ffmpegResult.outputPath;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "autokap",
-  "version": "1.3.29",
+  "version": "1.3.31",
   "description": "AI-powered CLI tool for capturing clean screenshots of websites",
   "type": "module",
   "main": "./dist/index.js",