npm - ossclip - Versions diffs - 0.1.13 → 0.1.14 - Mend

ossclip 0.1.13 → 0.1.14

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (8) hide show

package/editor-dist/assets/{index-BBVLCP-N.js → index-CKpBGMBB.js} +16 -16
package/editor-dist/index.html +1 -1
package/package.json +4 -4
package/src/interactive/produce-argv.ts +8 -0
package/src/interactive/produce-wizard.ts +10 -3
package/src/produce.ts +71 -2
package/src/program.ts +17 -0
package/src/replay-argv.ts +15 -0

package/editor-dist/index.html CHANGED Viewed

@@ -4,7 +4,7 @@
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <title>ossclip editor</title>
-    <script type="module" crossorigin src="/assets/index-BBVLCP-N.js"></script>
+    <script type="module" crossorigin src="/assets/index-CKpBGMBB.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-C8IPo60X.css">
   </head>
   <body>

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "ossclip",
-  "version": "0.1.13",
+  "version": "0.1.14",
   "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.13",
-    "@ossclip/renderer": "0.1.13",
-    "@ossclip/scenes": "0.1.13"
+    "@ossclip/core": "0.1.14",
+    "@ossclip/scenes": "0.1.14",
+    "@ossclip/renderer": "0.1.14"
   },
   "homepage": "https://github.com/AhsanAyaz/ossclip#readme",
   "bugs": {

package/src/interactive/produce-argv.ts CHANGED Viewed

@@ -19,6 +19,11 @@ export interface ProduceExtras {
    * off is the default, and a config-on user who wants it off for one run
    * types `--no-watermark`, a flags-only surface like --sort. */
   watermark?: boolean;
+  /** Captions tri-state, the watermark's mirror image: ON is the default,
+   * so the wizard only ever turns them OFF (`false` → `--no-captions`) and
+   * the positive `--captions` stays flags-only — it exists for replay
+   * pinning, and emitting it here would restate the default. */
+  captions?: boolean;
   llm?: "claude" | "claude-cli" | "gemini" | "mock";
 }
@@ -62,6 +67,9 @@ export function produceArgv(a: ProduceAnswers): string[] {
   if (e.collapseRetakes === true) argv.push("--collapse-retakes");
   if (e.sourceIsEdited === true) argv.push("--source-is-edited");
   if (e.watermark === true) argv.push("--watermark");
+  // Strict `=== false` (never `!e.captions`): undefined means "the default,
+  // on" and must emit nothing per the elision rule above.
+  if (e.captions === false) argv.push("--no-captions");
   if (e.llm) argv.push("--llm", e.llm);
   return argv;

package/src/interactive/produce-wizard.ts CHANGED Viewed

@@ -4,12 +4,15 @@ import { produceArgv, type ProduceAnswers, type ProduceExtras } from "./produce-
 import { assertInteractive, confirm, intro, multiselect, select, text, unwrap } from "./prompts";
 /**
- * The produce wizard. Thirty-two flags (plus the positional input path)
+ * The produce wizard. Thirty-four flags (plus the positional input path)
  * sorted into three tiers: six prompts asked directly — the input path, plus
- * five flags (--out, --cleanup, --aspect, --produce, --intent) — nine behind
+ * five flags (--out, --cleanup, --aspect, --produce, --intent) — ten behind
  * one "anything else?" multiselect, and the remaining stay flags-only:
  * debug/internal surfaces, replay-only fields, --no-watermark (the
- * multiselect only turns the credit ON; off is already the default), or
+ * multiselect only turns the credit ON; off is already the default),
+ * --captions (the mirror case: ON is already the default, so the
+ * multiselect entry is the OFF switch and the positive flag exists only for
+ * replay pinning), or
  * (final-review fix wave, Finding 1) --sort. A folder's clip order only means anything once the
  * folder has been enumerated, and that enumeration happens inside
  * `produce()` — after the wizard has already returned argv — so there is
@@ -35,6 +38,7 @@ const EXTRAS = [
     hint: "--collapse-retakes",
   },
   { value: "sourceIsEdited", label: "Source already has burned-in text", hint: "--source-is-edited" },
+  { value: "captionsOff", label: "Turn the burned-in captions off", hint: "--no-captions" },
   { value: "watermark", label: 'Credit the tool with a small "made with ossclip"', hint: "--watermark" },
   { value: "llm", label: "Choose the LLM provider", hint: "--llm" },
 ] as const;
@@ -235,6 +239,9 @@ export async function produceWizard(
   if (chosen.includes("sourceFit")) extras.sourceFit = "contain";
   if (chosen.includes("collapseRetakes")) extras.collapseRetakes = true;
   if (chosen.includes("sourceIsEdited")) extras.sourceIsEdited = true;
+  // The entry is the OFF switch (captions default ON — see EXTRAS), so a
+  // tick maps to `captions: false` and produceArgv emits `--no-captions`.
+  if (chosen.includes("captionsOff")) extras.captions = false;
   if (chosen.includes("watermark")) extras.watermark = true;
   if (chosen.includes("speaker")) {
     extras.speaker = unwrap(

package/src/produce.ts CHANGED Viewed

@@ -247,6 +247,15 @@ export interface ProduceOptions {
    * a free-tier limitation; this is voluntary attribution.
    */
   watermark?: boolean;
+  /**
+   * `--captions` / `--no-captions` tri-state: true/false when TYPED,
+   * undefined when not. Unlike `watermark` above there is no config key —
+   * undefined simply means the default, which is ON. Kept tri-state anyway
+   * so command.json can pin the resolved flag state (`recordedProduceArgs`)
+   * and a future config key or default change can never re-resolve an old
+   * record differently.
+   */
+  captions?: boolean;
   /**
    * `<input>` a DIRECTORY: order its clips before concatenating them into the
    * source produce runs on (folder-input-brief.md). `name` (default) is a
@@ -281,6 +290,26 @@ export function resolveWatermark(
   return flag ?? configValue === true;
 }
+/**
+ * Whether captions are hidden this run: the flag saying OFF, or the editor's
+ * doc-global `captionsHidden` override saying hidden. An OR, deliberately
+ * NOT resolveWatermark's flag-beats-config precedence: the override is the
+ * user's own saved edit, not a machine-supplied default a typed flag should
+ * outrank — un-hiding belongs to the editor that wrote the override, so a
+ * typed `--captions` cannot force captions back on over it. Strict
+ * `=== false`/`=== true` on both sides: the override arrives zod-parsed,
+ * but the flag is a tri-state where undefined means "not typed" — and
+ * captions defaulting ON means anything short of an explicit off must read
+ * as visible. Pure so the whole flag × override matrix is testable without
+ * a workdir or an overrides.json on disk.
+ */
+export function resolveCaptionsHidden(
+  flag: boolean | undefined,
+  overrideHidden: boolean | undefined,
+): boolean {
+  return flag === false || overrideHidden === true;
+}
 function sha1File(path: string): Promise<string> {
   return new Promise((res, rej) => {
     const h = createHash("sha1");
@@ -2007,6 +2036,21 @@ export async function produce(inputArg: string, opts: ProduceOptions): Promise<P
   // that assembleScenes dropped, and the caption track knows exactly when the
   // ask is on screen. Quoting marks the word you type in the comments — every
   // other time the speaker merely says it, it must render plainly.
+  // Captions are ON by default and stay so — only the OFF path announces
+  // itself, naming WHICH surface turned them off: a silent-captions upload
+  // must never leave the author guessing whether they typed the flag or the
+  // editor's toggle did it. The flag reason wins the message when both are
+  // true — it is the one visible in the command line being run. Resolved
+  // BEFORE the CTA block below, because the CTA line's promise depends on it.
+  const captionsHidden = resolveCaptionsHidden(opts.captions, overrideDoc.captionsHidden);
+  if (captionsHidden) {
+    console.log(
+      opts.captions === false
+        ? "▸ captions: off (--no-captions)"
+        : "▸ captions: hidden by editor override",
+    );
+  }
   const ctaCue = [...graphicCues]
     .reverse()
     .find((c) => typeof c.props?.keyword === "string" && (c.props.keyword as string).length > 0);
@@ -2015,9 +2059,15 @@ export async function produce(inputArg: string, opts: ProduceOptions): Promise<P
     ? { startSec: ctaCue.startSec, endSec: ctaCue.endSec }
     : undefined;
   if (ctaKeyword) {
+    // Gated on the captions resolution (review minor 1): "styled only at
+    // X–Ys" on a captions-hidden run is a false promise — the styling rides
+    // the caption track (ProductionComposition's mount gate), so the moment
+    // the trade bites is the moment to say so.
     console.log(
-      `▸ CTA keyword "${ctaKeyword}" styled only at ` +
-        `${ctaWindow!.startSec.toFixed(1)}–${ctaWindow!.endSec.toFixed(1)}s`,
+      captionsHidden
+        ? `▸ CTA keyword "${ctaKeyword}" styling skipped — captions hidden`
+        : `▸ CTA keyword "${ctaKeyword}" styled only at ` +
+            `${ctaWindow!.startSec.toFixed(1)}–${ctaWindow!.endSec.toFixed(1)}s`,
     );
   }
@@ -2092,6 +2142,18 @@ export async function produce(inputArg: string, opts: ProduceOptions): Promise<P
     // an off run's render-props.json stays byte-identical to a pre-watermark
     // one, so nothing downstream can tell the feature ever shipped.
     ...(watermark ? { watermark: true } : {}),
+    // Same absent-means-default contract, polarity flipped (captions default
+    // ON): written only when hidden, so a normal run's render-props.json
+    // stays byte-identical to a pre-feature one. `captionsHiddenByFlag` is
+    // the flag-only part, split out for the EDITOR alone: `captionsHidden`
+    // bakes the override doc in, and the editor re-applies the CURRENT doc
+    // onto pristine bases (see `baseSceneCues` above) — without the split,
+    // its live preview either couldn't take a doc-sourced hide back after
+    // an un-toggle (the add-only trap those bases exist for) or would show
+    // captions that a command.json pinned with --no-captions will never
+    // actually render.
+    ...(captionsHidden ? { captionsHidden: true } : {}),
+    ...(opts.captions === false ? { captionsHiddenByFlag: true } : {}),
   };
   await writeFile(join(work, "render-props.json"), JSON.stringify(props, null, 2));
@@ -2285,10 +2347,17 @@ export async function produce(inputArg: string, opts: ProduceOptions): Promise<P
   // under a later/foreign config-on, an on-run would silently lose it. The
   // RESOLVED state is always pinned; a typed flag is already in the argv and
   // the includes-guard leaves it alone.
+  // Captions pin: the FLAG's resolved state (`opts.captions ?? true`), never
+  // the override-inclusive `captionsHidden` — overrides.json travels with
+  // the workdir and is re-read on every replay, so pinning --no-captions
+  // because the EDITOR hid them would freeze an edit the user may later
+  // undo in that same editor. See recordedProduceArgs for why the pin is
+  // unconditional even though captions' default is config-independent today.
   const recordedArgs = recordedProduceArgs({
     llm: provider ? providerName : undefined,
     clipWindow: clipWindow ? `${clipWindow.startWord}:${clipWindow.endWord}` : undefined,
     watermark,
+    captions: opts.captions ?? true,
   });
   await writeFile(
     join(work, "command.json"),

package/src/program.ts CHANGED Viewed

@@ -270,6 +270,20 @@ export function buildProgram(): Command {
         "(set it once with watermark: true in ~/.ossclip/config.json)",
     )
     .option("--no-watermark", "no wordmark, even when the config turns it on")
+    // Same tri-state shape as --watermark above (positive declared first so
+    // commander's default stays undefined = "not typed"), though captions
+    // have no config key to fill the gap: the tri-state exists so
+    // command.json can pin the resolved flag state for replay determinism
+    // (recordedProduceArgs) — a bare boolean default would make "not typed"
+    // and "typed --captions" indistinguishable to the pin.
+    .option(
+      "--captions",
+      "burned-in captions — already the default; exists so a recorded replay can pin the state",
+    )
+    .option(
+      "--no-captions",
+      "no burned-in captions. The CTA keyword styling rides the caption track, so it goes too",
+    )
     .option("--no-cover", "skip the cover image written beside the video")
     .option("--cover <path>", "cover image output path (default: <out>.cover.jpg)")
     .option("--open-editor", "open the editor when the run finishes")
@@ -372,6 +386,9 @@ export function buildProgram(): Command {
         sourceFit,
         // undefined = "not typed", so produce can let the config decide.
         watermark: opts.watermark,
+        // undefined = "not typed" here too — the default (ON) is applied at
+        // the pin site, not coerced in transit.
+        captions: opts.captions,
         cover: opts.cover !== false,
         coverPath: typeof opts.cover === "string" ? opts.cover : undefined,
         clip: opts.clip,

package/src/replay-argv.ts CHANGED Viewed

@@ -52,6 +52,7 @@ export function recordedProduceArgs(pins: {
   llm?: string;
   clipWindow?: string;
   watermark?: boolean;
+  captions?: boolean;
 }): string[] {
   const args = consumeReplayArgv() ?? process.argv.slice(2);
   if (pins.llm !== undefined && !args.includes("--llm")) {
@@ -74,5 +75,19 @@ export function recordedProduceArgs(pins: {
   if (pins.watermark !== undefined && !args.includes("--watermark") && !args.includes("--no-watermark")) {
     args.push(pins.watermark ? "--watermark" : "--no-watermark");
   }
+  // 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
+  // flag would currently replay identically everywhere. Pinned anyway
+  // because (a) the watermark review already made "every record carries the
+  // RESOLVED state" §75's rule, and one unpinned tri-state would turn
+  // command.json's contract into per-flag trivia; and (b) the moment a
+  // config key or a changed default ever appears, every unpinned old record
+  // would silently re-resolve under it — the watermark's exact drift, just
+  // deferred. Note this pins the FLAG's state only; the editor's
+  // `captionsHidden` override is not folded in (see produce.ts's call site).
+  if (pins.captions !== undefined && !args.includes("--captions") && !args.includes("--no-captions")) {
+    args.push(pins.captions ? "--captions" : "--no-captions");
+  }
   return args;
 }