ossclip 0.1.31 → 0.1.34

package/src/render-replay-args.ts

@@ -0,0 +1,50 @@
+/**
+ * The argv the editor's Render button replays (2026-08-29).
+ *
+ * A render started FROM the editor must reproduce what the user just
+ * reviewed. Replaying `command.json` verbatim did not: it carried
+ * `--produce`, so every render re-called the LLM, and the beat-sheet cache
+ * key covers the repaired words and the MEASURED framing — both of which the
+ * user's own editor work changes. The plan came back renumbered, edits
+ * anchored to scenes it no longer had were orphaned, and an approved cut was
+ * silently rewritten (the failure `produce.ts`' §143 note already named once,
+ * 2026-08-23; hit again through a different door).
+ *
+ * So the editor pins the plan on screen with `--scenes` (no LLM in the loop)
+ * and drops `--produce`. The two paths stay distinct on purpose:
+ *
+ *  - CLI/CI: `--produce` plans fresh. The command line is the authority.
+ *  - Editor: the reviewed plan is the authority; `replan` opts back in.
+ *
+ * Pure, so the whole matrix is testable without spawning a produce.
+ */
+/** The reviewed plan the editor pins for a render (see the module docstring).
+ * Its own name, never `scenes-<key>.json`: those are keyed to a beat sheet
+ * this render may no longer match. */
+export const REVIEWED_SCENES_BASENAME = "scenes-reviewed.json";
+
+export function renderReplayArgs(
+  recorded: readonly string[],
+  opts: { scenesPath?: string; replan?: boolean },
+): string[] {
+  // A deliberate re-plan replays exactly what was recorded — including
+  // `--produce` when it was there, since that IS the fresh-plan request.
+  if (opts.replan === true || opts.scenesPath === undefined) return [...recorded];
+
+  const out: string[] = [];
+  for (let i = 0; i < recorded.length; i++) {
+    const arg = recorded[i]!;
+    // `--produce` is a boolean flag: drop it alone, never a following value.
+    if (arg === "--produce") continue;
+    // An existing `--scenes <path>` is dropped WITH its value: two of them
+    // would let commander pick one, and the stale one is precisely the plan
+    // the editor is not showing.
+    if (arg === "--scenes") {
+      i++;
+      continue;
+    }
+    out.push(arg);
+  }
+  out.push("--scenes", opts.scenesPath);
+  return out;
+}

package/src/replay-argv.ts

@@ -58,6 +58,8 @@ export function recordedProduceArgs(pins: {
   llmEffort?: "low" | "medium" | "high";
   clipWindow?: string;
   watermark?: boolean;
+  /** The RESOLVED cover-overlay switch — config-dependent, so pinned both ways. */
+  coverInVideo?: boolean;
   captions?: boolean;
   jumpCuts?: JumpCutsMode;
   /** The RESOLVED dictionary terms — pinned only when non-empty. */
@@ -110,6 +112,20 @@ export function recordedProduceArgs(pins: {
   if (pins.watermark !== undefined && !args.includes("--watermark") && !args.includes("--no-watermark")) {
     args.push(pins.watermark ? "--watermark" : "--no-watermark");
   }
+  // The cover overlay, the watermark's rationale VERBATIM and for a live
+  // reason, not future-proofing: its effective default is config-dependent
+  // (`coverInVideo: true` in ~/.ossclip/config.json), so an unpinned record
+  // would gain — or lose — an overlay on the first frames of the replayed
+  // video the moment that config is edited, or the moment Render runs on
+  // another machine. Both directions, includes-guarded on BOTH spellings so a
+  // typed flag is never doubled.
+  if (
+    pins.coverInVideo !== undefined &&
+    !args.includes("--cover-in-video") &&
+    !args.includes("--no-cover-in-video")
+  ) {
+    args.push(pins.coverInVideo ? "--cover-in-video" : "--no-cover-in-video");
+  }
   // The captions flag, pinned both ways like the watermark above — but here
   // as future-proofing plus consistency rather than a live bug: captions'
   // default is ON independent of any config today, so an argv without the

package/src/ui/animation.ts

@@ -204,7 +204,11 @@ export class StageAnimator {
       const spinner = SPINNERS.neural[this.frameIndex % SPINNERS.neural.length];
       const dots = SPINNERS.dots[this.frameIndex % SPINNERS.dots.length];
       visual =
-        `${ansi.bold}${ansi.brightMagenta}⚡ GEMINI 3.7 FLASH // NEURAL SCENE PLANNER${ansi.reset} ` +
+        // The caller's title, not a hardcoded model name: this header once
+        // said "GEMINI 3.7 FLASH" over a claude-cli run (2026-08-26 field
+        // report) — the subtitle names the real provider, and the header
+        // must not contradict it.
+        `${ansi.bold}${ansi.brightMagenta}⚡ ${this.title}${ansi.reset} ` +
         `${ansi.cyan}[${spinner}]${ansi.reset} ${ansi.dim}(${elapsedSec}s)${ansi.reset}\n` +
         `  ${ansi.brightCyan}${dots}${ansi.reset} ${ansi.brightWhite}${clamp(this.subtitle)}${ansi.reset}`;
     } else if (this.type === "whisper") {