autokap 1.8.8 → 1.8.9

package/dist/cli-runner.d.ts

@@ -58,3 +58,4 @@ export interface CLIRunResult {
 }
 export declare function runCapture(options: CLIRunnerOptions): Promise<CLIRunResult>;
 export declare function buildVideoClipMetadata(videoId: string, result: RunResult, program?: ExecutionProgram, runId?: string): VideoClipMetadata[];
+export declare function resolveEffectiveCliArtifactPlan(artifactPlan: ExecutionProgram['artifactPlan'], deviceFrame?: string | null): ExecutionProgram['artifactPlan'];

package/dist/cli-runner.js

@@ -25,7 +25,7 @@ import { parseProgram } from './execution-schema.js';
 import { buildCursorOverlayScript } from './cursor-overlay-script.js';
 import { CLI_VERSION_HEADER, } from './cli-contract.js';
 import { postProcessClipRecording } from './clip-postprocess.js';
-import { applyDeviceFrame, seedDeviceConfigs } from './mockup.js';
+import { applyDeviceFrame, resolveVariantFrameOptions, seedDeviceConfigs } from './mockup.js';
 import { transformBrowserUrl } from './transform-browser-url.js';
 import { localizeStatusBar } from './status-bar-l10n.js';
 import { logger } from './logger.js';
@@ -164,7 +164,12 @@ export async function runCapture(options) {
     catch (error) {
         return { success: false, runId, error: error instanceof Error ? error.message : String(error) };
     }
-    if (!options.program && program.mediaMode === 'video') {
+    // TTS synthesis is real, billed work — it belongs to a real run only. A dry run validates
+    // navigation/opcodes (capture opcodes + upload are skipped below) and must NOT synthesize speech:
+    // doing so would bill TTS credits while the run reports "0 credits charged". The rewritten SLEEP
+    // durations + audio assets are only consumed on the upload path (signalVideoComplete), which a dry
+    // run never reaches, so skipping prep here is side-effect-free for dry.
+    if (!options.dryRun && !options.program && program.mediaMode === 'video') {
         const prepareResult = await prepareVideoSpeechForRun(config, options.presetId, runId, options.regenerateTts ?? false);
         if (!prepareResult.success) {
             return { success: false, runId, error: prepareResult.error };
@@ -962,6 +967,9 @@ async function uploadArtifactMultipart(config, program, runId, job, filename, pr
     if (variantSpec?.deviceFrame) {
         formData.append('deviceFrame', variantSpec.deviceFrame);
     }
+    if (variantSpec?.mockupOptions) {
+        formData.append('mockupOptions', JSON.stringify(variantSpec.mockupOptions));
+    }
     const requestedDeviceScaleFactor = variantSpec?.deviceScaleFactor ?? program.outputScale ?? 2;
     const isFrameCapture = artifact.mediaMode === 'clip' || artifact.mediaMode === 'video';
     const deviceScaleFactor = isFrameCapture && Number.isFinite(requestedDeviceScaleFactor)
@@ -1103,22 +1111,24 @@ async function prepareDirectUploadParts(params) {
 async function prepareScreenshotBufferForDirectUpload(input, metadata, program, variantSpec, tabIcon) {
     let output = input;
     const artifactPlan = resolveEffectiveCliArtifactPlan(program.artifactPlan, variantSpec?.deviceFrame ?? null);
-    if (artifactPlan?.applyStatusBar && !artifactPlan?.applyMockup) {
-        throw new Error('applyStatusBar requires applyMockup with a device frame');
-    }
-    if (artifactPlan?.applyMockup) {
-        if (!variantSpec?.deviceFrame) {
-            throw new Error('applyMockup requires a deviceFrame on the variant');
-        }
-        output = await applyDeviceFrame(output, variantSpec.deviceFrame, {
+    // The variant's deviceFrame is the sole gate. Its mockupOptions (orientation, status bar,
+    // safe areas, …) — defined by the user on the preset — drive the frame; the legacy
+    // program-level applyStatusBar only survives as a fallback for old programs without options.
+    if (variantSpec?.deviceFrame) {
+        const mockup = variantSpec.mockupOptions;
+        const frame = resolveVariantFrameOptions(mockup, {
             orientation: inferCliOrientation(metadata.viewport ?? null),
+            showStatusBar: artifactPlan?.applyStatusBar,
+        });
+        output = await applyDeviceFrame(output, variantSpec.deviceFrame, {
+            ...mockup,
+            ...frame,
             viewport: metadata.viewport ?? undefined,
             colorScheme: metadata.theme,
             outputScale: normalizeCliDeviceScaleFactor(metadata.deviceScaleFactor)
                 ?? normalizeCliDeviceScaleFactor(program.outputScale)
                 ?? 2,
-            showStatusBar: artifactPlan.applyStatusBar ?? false,
-            statusBar: localizeStatusBar({}, metadata.lang),
+            statusBar: localizeStatusBar(mockup?.statusBar ?? {}, metadata.lang),
             browserBar: buildCliBrowserBar(metadata.captureUrl, metadata.theme, tabIcon, { publicUrl: program.publicUrl, pageTitle: metadata.pageTitle ?? null }),
         });
     }
@@ -1181,10 +1191,11 @@ async function prepareClipBuffersForDirectUpload(artifact, metadata) {
         await fs.rm(tempDir, { recursive: true, force: true }).catch(() => undefined);
     }
 }
-function resolveEffectiveCliArtifactPlan(artifactPlan, deviceFrame) {
+export function resolveEffectiveCliArtifactPlan(artifactPlan, deviceFrame) {
+    // Device mockups are gated SOLELY by the variant's `deviceFrame` (user-defined, deterministic).
+    // A present deviceFrame always renders its frame; `applyMockup` is a derived value, never a
+    // kill-switch (the legacy `applyMockup: false` toggle is intentionally ignored here).
     if (deviceFrame) {
-        if (artifactPlan.applyMockup === false)
-            return artifactPlan;
         return artifactPlan.applyMockup ? artifactPlan : { ...artifactPlan, applyMockup: true };
     }
     if (!artifactPlan.applyMockup && !artifactPlan.applyStatusBar)

package/dist/execution-schema.d.ts

@@ -1053,7 +1053,7 @@ export declare const VariantSpecSchema: z.ZodObject<{
             light: "light";
             dark: "dark";
         }>>;
-    }, z.core.$strict>>;
+    }, z.core.$strip>>;
 }, z.core.$strict>;
 export declare const PreconditionSpecSchema: z.ZodObject<{
     credentialsId: z.ZodOptional<z.ZodString>;
@@ -1205,7 +1205,7 @@ export declare const ExecutionProgramSchema: z.ZodObject<{
                 light: "light";
                 dark: "dark";
             }>>;
-        }, z.core.$strict>>;
+        }, z.core.$strip>>;
     }, z.core.$strict>>;
     preconditions: z.ZodObject<{
         credentialsId: z.ZodOptional<z.ZodString>;

package/dist/execution-schema.js

@@ -553,7 +553,13 @@ export const VariantSpecSchema = z.object({
             radius: z.number(),
         }).strict().optional(),
         colorScheme: z.enum(['light', 'dark']).optional(),
-    }).strict().optional(),
+        // STRIP (not strict): per-variant mockupOptions are reconciled verbatim from the preset's
+        // CaptureTarget, whose UI-side MockupOptions carries render-irrelevant keys the engine never
+        // consumes (showDock, dockScale, dockMode, userAppIcon, autoBrowserBar, viewport). Strict would
+        // make parseProgram THROW on those — stranding every Mac/browser preset at CLI fetch time.
+        // Dropping unknown keys post-parse is safe: applyDeviceFrame ignores them, and the full set
+        // stays on config.targets for the UI/preview.
+    }).optional(),
 }).strict();
 const cookieSchema = z.object({
     name: z.string().min(1),

package/dist/execution-types.d.ts

@@ -535,9 +535,16 @@ export interface ArtifactSpec {
     cursorTheme?: VideoCursorTheme;
     /** Max clip duration in seconds. Clips are trimmed if they exceed this. Default: 8. Ignored when `mediaMode='video'`. */
     maxClipDurationSec?: number;
-    /** Whether to apply device frame mockup. Default: false */
+    /**
+     * @deprecated Device mockups are gated SOLELY by a variant's `deviceFrame` (user-defined,
+     * deterministic). The render paths derive mockup application from `deviceFrame` and ignore this
+     * flag as a gate. Kept optional for back-compat with stored programs; do not author it.
+     */
     applyMockup?: boolean;
-    /** Whether to add status bar. Default: false */
+    /**
+     * @deprecated Per-variant `mockupOptions.showStatusBar` drives the status bar now. Retained only
+     * as a fallback for legacy programs that lack per-variant `mockupOptions`. Do not author it.
+     */
     applyStatusBar?: boolean;
 }
 export interface ExecutionProgram {

package/dist/mockup.d.ts

@@ -204,6 +204,19 @@ export interface MockupOptions {
         height: number;
     };
 }
+/**
+ * Resolve the two per-variant frame decisions shared by both render paths (CLI direct-upload
+ * framing and the cloud legacy-multipart route): a variant's own `mockupOptions` wins, falling
+ * back to the viewport-inferred orientation and the deprecated program-level `applyStatusBar`.
+ * Pure + exported so the precedence is tested once instead of in two duplicated call sites.
+ */
+export declare function resolveVariantFrameOptions(mockupOptions: MockupOptions | undefined, fallback: {
+    orientation?: MockupOrientation;
+    showStatusBar?: boolean;
+}): {
+    orientation?: MockupOrientation;
+    showStatusBar: boolean;
+};
 export interface ResolvedDeviceFrameDescriptor {
     id: string;
     name: string;

package/dist/mockup.js

@@ -17,6 +17,18 @@ function getSupabaseMockupConfig() {
         serviceKey: process.env.SUPABASE_SERVICE_ROLE_KEY,
     };
 }
+/**
+ * Resolve the two per-variant frame decisions shared by both render paths (CLI direct-upload
+ * framing and the cloud legacy-multipart route): a variant's own `mockupOptions` wins, falling
+ * back to the viewport-inferred orientation and the deprecated program-level `applyStatusBar`.
+ * Pure + exported so the precedence is tested once instead of in two duplicated call sites.
+ */
+export function resolveVariantFrameOptions(mockupOptions, fallback) {
+    return {
+        orientation: mockupOptions?.orientation ?? fallback.orientation,
+        showStatusBar: mockupOptions?.showStatusBar ?? fallback.showStatusBar ?? false,
+    };
+}
 const DEFAULT_MOCKUP_OPTIONS = {
     orientation: 'portrait',
     outputScale: 2,

package/dist/program-signing.d.ts

@@ -111,7 +111,7 @@ export declare const SignedExecutionProgramEnvelopeSchema: z.ZodObject<{
                     light: "light";
                     dark: "dark";
                 }>>;
-            }, z.core.$strict>>;
+            }, z.core.$strip>>;
         }, z.core.$strict>>;
         preconditions: z.ZodObject<{
             credentialsId: z.ZodOptional<z.ZodString>;

package/dist/skill-packaging.js

@@ -19,6 +19,7 @@ const PRESET_SKILL_SOURCE = {
         { relativePath: 'OPCODE-REFERENCE.md', title: 'Opcode Reference', anchor: 'reference-opcode-reference' },
         { relativePath: 'references/STANDARDS.md', title: 'Prompt Charter & Quality Standards', anchor: 'reference-prompt-standards' },
         { relativePath: 'references/mock-data.md', title: 'Mock Data Injection', anchor: 'reference-mock-data-injection' },
+        { relativePath: 'references/video-workflow.md', title: 'Demo Video Workflow', anchor: 'reference-demo-video-workflow' },
         { relativePath: 'references/examples.md', title: 'Complete Examples', anchor: 'reference-complete-examples' },
     ],
 };

package/dist/video-narration-schema.d.ts

@@ -130,7 +130,7 @@ export declare const VideoIngestPayloadSchema: z.ZodObject<{
                     light: "light";
                     dark: "dark";
                 }>>;
-            }, z.core.$strict>>;
+            }, z.core.$strip>>;
         }, z.core.$strict>>;
         preconditions: z.ZodObject<{
             credentialsId: z.ZodOptional<z.ZodString>;

package/package.json

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