autokap 1.9.4 → 1.9.6

package/dist/web-playwright-local.d.ts

@@ -8,6 +8,7 @@ import type { Browser } from './browser.js';
 import type { AKTree, OutscaleConfig, VideoPageSignals } from './types.js';
 import type { RuntimeAdapter, ClickOptions, WaitCondition, RecordingOptions, RecordingResult, SemanticTarget, ProgressSnapshot, VisualStabilityResult } from './execution-types.js';
 import { type ResolveOptions } from './semantic-resolver.js';
+export declare function isRecognizedHintCookieKey(key: string, kind: 'locale' | 'theme'): boolean;
 export declare class WebPlaywrightLocal implements RuntimeAdapter {
     private browser;
     private recordingDir?;

package/dist/web-playwright-local.js

@@ -13,6 +13,32 @@ import { logger } from './logger.js';
 import { ClipCaptureLoop } from './clip-capture-loop.js';
 import { FfmpegX11Recorder } from './ffmpeg-x11-recorder.js';
 import { assembleMp4FromFrames, getMediaDurationMs } from './clip-postprocess.js';
+/**
+ * Locale/theme cookie names the engine treats as a real "applied" signal.
+ * Mirrors the storage-key whitelists in {@link Browser.writeStorageHintCandidate}
+ * so a SET_LOCALE/SET_THEME cookie hint is only reported as effective when the
+ * cookie name plausibly drives the app's locale/theme. Apps that derive these
+ * from the path or middleware (common in Next.js) won't match, so the opcode
+ * fails instead of passing with a wrong-variant capture.
+ */
+const LOCALE_COOKIE_KEY_WHITELIST = [
+    'lang', 'locale', 'language', 'i18n', 'intl', 'i18n-locale',
+    'next-i18next', 'NEXT_LOCALE', 'nuxt-i18n-locale',
+];
+const THEME_COOKIE_KEY_WHITELIST = [
+    'theme', 'color-scheme', 'colorScheme', 'dark-mode', 'darkMode', 'appearance',
+];
+export function isRecognizedHintCookieKey(key, kind) {
+    const whitelist = kind === 'locale' ? LOCALE_COOKIE_KEY_WHITELIST : THEME_COOKIE_KEY_WHITELIST;
+    const lower = key.toLowerCase();
+    return whitelist.some((entry) => {
+        const normalized = entry.toLowerCase();
+        if (lower === normalized)
+            return true;
+        const re = new RegExp(`(?:^|[^a-zA-Z0-9])${normalized.replace(/[-/\\^$*+?.()|[\]{}]/g, '\\$&')}(?:$|[^a-zA-Z0-9])`, 'i');
+        return re.test(lower);
+    });
+}
 /**
  * Snapshot the scroll offset of the window and every scroll container, tagging
  * each element so {@link restoreScrollState} can put it back.
@@ -805,7 +831,14 @@ export class WebPlaywrightLocal {
                     secure: url.protocol === 'https:',
                     sameSite: 'Lax',
                 }]);
-            return true;
+            // Writing the cookie to the browser context always succeeds, but that is
+            // NOT a signal that the app actually reads locale/theme from this cookie.
+            // Many Next.js apps derive locale/theme from the path or middleware, so a
+            // blanket `true` here falsely reports the hint as "applied" and lets the
+            // SET_LOCALE/SET_THEME opcode pass with a wrong-variant capture. Mirror the
+            // storage-key skepticism (browser.ts#writeStorageHintCandidate): only claim
+            // success when the cookie name looks like a recognized locale/theme key.
+            return isRecognizedHintCookieKey(params.key, params.kind);
         }
         return this.browser.writeStorageHintCandidate({
             storageName: params.storage,

package/package.json

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