ossclip 0.1.28 → 0.1.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ossclip",
-  "version": "0.1.28",
+  "version": "0.1.29",
   "description": "Local-first CLI video producer: cuts silence and fillers, word-timed captions, face-aware framing, and LLM-planned code-rendered graphics — transcription and rendering never leave your machine",
   "type": "module",
   "license": "MIT",
@@ -36,9 +36,9 @@
     "commander": "^12.1.0",
     "tsx": "^4.19.0",
     "zod": "^3.25.76",
-    "@ossclip/core": "0.1.28",
-    "@ossclip/scenes": "0.1.28",
-    "@ossclip/renderer": "0.1.28"
+    "@ossclip/core": "0.1.29",
+    "@ossclip/scenes": "0.1.29",
+    "@ossclip/renderer": "0.1.29"
   },
   "homepage": "https://github.com/AhsanAyaz/ossclip#readme",
   "bugs": {

package/src/interactive/produce-argv.ts

@@ -63,6 +63,15 @@ export interface ProduceAnswers {
   graphics: boolean;
   intent?: string;
   out?: string;
+  /**
+   * Review the cut in the editor instead of rendering now (§148). A main-flow
+   * answer like `graphics`, not an extra: it decides what the run DOES, and
+   * burying it in "Anything else?" would hide it from the people the wizard
+   * exists for. The only answer whose wizard default (review) differs from
+   * the CLI's (render) — which costs nothing here, because the elision rule
+   * below keys off the CLI default, not off what the prompt preselected.
+   */
+  review?: boolean;
   extras: ProduceExtras;
 }
 
@@ -76,6 +85,11 @@ export function produceArgv(a: ProduceAnswers): string[] {
   if (a.aspect !== "9:16") argv.push("--aspect", a.aspect);
   if (a.cleanup !== "standard") argv.push("--cleanup", a.cleanup);
   if (a.out) argv.push("--out", a.out);
+  // Rendering is the CLI's default, so only reviewing is worth saying — the
+  // rule above is about the DEFAULT, not about which option the prompt
+  // preselected. --out still travels either way: nothing renders now, but the
+  // editor's Render button replays command.json, and that is where out lands.
+  if (a.review === true) argv.push("--review");
 
   if (a.graphics) {
     argv.push("--produce");

package/src/interactive/produce-wizard.ts

@@ -505,6 +505,33 @@ export async function produceWizard(
     ) as ProduceExtras["llm"];
   }
 
+  // LAST, because it decides what happens after every answer above it — and
+  // asked at all because until §148 the wizard could not reach --review, so
+  // `ossclip` with no arguments, the entry point a first-time user takes,
+  // could only render first and offer the editor afterwards. That is the
+  // opposite of what --review is for: the render is the expensive step, and
+  // reviewing exists so it happens once, on a cut you already agreed with.
+  //
+  // Leaning to review is the one place a wizard default differs from the
+  // CLI's. It costs nothing: produceArgv elides against the CLI default, so
+  // choosing to render still teaches `ossclip produce <file>` and choosing to
+  // review teaches the flag that did it.
+  const review =
+    unwrap(
+      await select({
+        message: "Render now, or review the cut first?",
+        initialValue: "review",
+        options: [
+          {
+            value: "review",
+            label: "Review the cut first",
+            hint: "opens the editor; render from its button",
+          },
+          { value: "render", label: "Render now", hint: "straight to a finished file" },
+        ],
+      }),
+    ) === "review";
+
   return produceArgv({
     input,
     aspect,
@@ -514,6 +541,7 @@ export async function produceWizard(
     // Already `string | undefined`: pickSavePath's use-default row IS the
     // old empty answer — no --out, produce derives its own default.
     out,
+    review,
     extras,
   });
 }

package/src/produce.ts

@@ -152,6 +152,7 @@ import {
   type LlmEffort,
   type LlmProvider,
   type LlmUsage,
+  AGY_PRINT_TIMEOUT,
   type Production,
   ossclipOutputPathFor,
   type ProviderName,
@@ -1413,6 +1414,14 @@ export function cleanupChoicesLine(vetoed: readonly Segment[], outputDuration: n
  * and even on a talking head a 7% lurch reads as the camera stumbling. Big
  * enough to break up the jump, small enough to pass as sensor noise.
  */
+/**
+ * How long a silent antigravity call runs before the spinner admits it (§149).
+ * Well clear of the 17-46s a healthy call takes, so a normal run never shows
+ * the notice, and well short of AGY_PRINT_TIMEOUT, so it lands while the wait
+ * still has somewhere to go.
+ */
+export const AGY_SLOW_NOTICE_MS = 30_000;
+
 export const FACE_PUNCH_SCALE = 1.015;
 
 /**
@@ -2787,17 +2796,38 @@ export async function produce(inputArg: string, opts: ProduceOptions): Promise<P
         : null;
       if (!aiAnim) console.log(`▸ producing scenes (${providerName})…`);
       if (opts.forceComponent) console.log(`▸ forcing every graphic to ${opts.forceComponent}`);
-      const result = await phases.time("llm", () =>
-        produceScenes(provider!, {
-          transcript,
-          outputDuration: map.outputDuration,
-          intent: opts.intent,
-          speaker: opts.speaker ?? cfg.speaker,
-          forceComponent: opts.forceComponent,
-          framing: framingCtx,
-          aspect: landscape ? "16:9" : "9:16",
-        }),
-      );
+      // A hung agy is indistinguishable from a working one on screen: the
+      // 2026-08-23 field run sat on this spinner for 605.9s with no hint that
+      // a budget existed or that a recovery was coming, which reads as a
+      // freeze rather than as waiting (§149). Only antigravity has a
+      // print-timeout and a fallback, so only it gets the notice — and only
+      // once the call is actually slow, so a healthy run never sees it.
+      const slowNotice =
+        aiAnim && providerName === "antigravity"
+          ? setTimeout(
+              () =>
+                // Short on purpose: StageAnimator clamps the subtitle to the
+                // terminal width and floors that at 40 columns, and the first
+                // draft lost the budget to "falling back to the n...". The
+                // number is the only part that changes what the user does
+                // (wait vs. Ctrl-C), so it has to survive the clamp.
+                aiAnim.update(`agy not replying — falling back at ${AGY_PRINT_TIMEOUT}...`),
+              AGY_SLOW_NOTICE_MS,
+            )
+          : undefined;
+      const result = await phases
+        .time("llm", () =>
+          produceScenes(provider!, {
+            transcript,
+            outputDuration: map.outputDuration,
+            intent: opts.intent,
+            speaker: opts.speaker ?? cfg.speaker,
+            forceComponent: opts.forceComponent,
+            framing: framingCtx,
+            aspect: landscape ? "16:9" : "9:16",
+          }),
+        )
+        .finally(() => clearTimeout(slowNotice));
       if (aiAnim) aiAnim.stop();
       scenes = result.scenes;
       beatSheet = { hook: result.beatSheet.hook, coverText: result.beatSheet.coverText };