ossclip 0.1.31 → 0.1.34

package/editor-dist/index.html

@@ -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-C537KpNT.js"></script>
+    <script type="module" crossorigin src="/assets/index-Dn813PGp.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-Bx2VQLP8.css">
   </head>
   <body>

package/package.json

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

package/src/caption-regen-panel.ts

@@ -0,0 +1,74 @@
+import { z } from "zod/v4";
+import { defaultProviderName, providerOfLog, type ProviderName } from "@ossclip/core";
+import { lastFlagValue } from "./thumbnail-panel";
+
+/**
+ * Which LLM provider the publish panel's caption regenerate should call
+ * (2026-08-29), pure so the usage-log × pin × env matrix is a table test
+ * with no server and no filesystem — the thumbnail-panel split. The I/O half
+ * lives in edit.ts's /api/publish/regenerate handler, which reads usage.json
+ * and command.json and hands them here.
+ *
+ * Resolution order: the provider usage.json says actually planned this
+ * project (`providerOfLog` — truthful even after a §143 fallback, which is
+ * why it beats the pin), then the `--llm` pin in command.json's recorded
+ * args, then `defaultProviderName`'s detection. Every candidate is PARSED
+ * against the provider enum, never coerced: a hand-edited usage.json saying
+ * "geminni" falls through to the next rung instead of reaching
+ * createProvider as garbage (CLAUDE.md's --source-fit rule).
+ */
+
+/** program.ts's `--llm` choices, restated as a schema — the same five names
+ * core's ProviderName union spells. */
+const ProviderNameSchema = z.enum(["antigravity", "claude", "claude-cli", "gemini", "mock"]);
+
+export type CaptionRegenProviderState =
+  | { status: "ready"; provider: ProviderName }
+  | { status: "unavailable"; reason: string };
+
+export interface CaptionRegenProviderInputs {
+  /** usage.json parsed as-is, or null when absent/corrupt — read leniently
+   * by the caller (the /api/usage posture). */
+  usageLog: unknown;
+  /** command.json's recorded args, or null when the workdir has none. */
+  commandArgs: readonly string[] | null;
+  env: NodeJS.ProcessEnv;
+  /** Injected existence check (defaultProviderName's own seam) so the
+   * matrix needs no PATH scan. */
+  hasBin: (bin: string) => boolean;
+}
+
+export function captionRegenProvider(inputs: CaptionRegenProviderInputs): CaptionRegenProviderState {
+  // The log's shape is user-editable JSON: pick out the two arrays
+  // providerOfLog reads and let anything else read as an empty log.
+  const log = (inputs.usageLog ?? {}) as { runs?: unknown; records?: unknown };
+  const recorded = providerOfLog({
+    runs: Array.isArray(log.runs) ? (log.runs as never[]) : [],
+    records: Array.isArray(log.records) ? (log.records as never[]) : [],
+  });
+  const fromLog = ProviderNameSchema.safeParse(recorded);
+  if (fromLog.success) return { status: "ready", provider: fromLog.data };
+  const fromPin = ProviderNameSchema.safeParse(
+    lastFlagValue(inputs.commandArgs ?? [], ["--llm"]),
+  );
+  if (fromPin.success) return { status: "ready", provider: fromPin.data };
+  // defaultProviderName answers "claude-cli" even with nothing installed —
+  // deliberate on the CLI, where the spawn failure carries install guidance.
+  // A button in a panel deserves the sentence BEFORE the click, so this
+  // restates its four detections as an availability gate and only then takes
+  // its pick.
+  const reachable =
+    inputs.hasBin(inputs.env.OSSCLIP_AGY_BIN ?? "agy") ||
+    inputs.hasBin(inputs.env.OSSCLIP_CLAUDE_BIN ?? "claude") ||
+    Boolean(inputs.env.GEMINI_API_KEY) ||
+    Boolean(inputs.env.ANTHROPIC_API_KEY);
+  if (!reachable) {
+    return {
+      status: "unavailable",
+      reason:
+        "no LLM provider available — set GEMINI_API_KEY or ANTHROPIC_API_KEY in " +
+        "~/.ossclip/.env, or install the claude or agy CLI, then try again",
+    };
+  }
+  return { status: "ready", provider: defaultProviderName(inputs.env, inputs.hasBin) };
+}

package/src/caption-report.ts

@@ -3,6 +3,7 @@ import {
   applyCaptionRangeEdits,
   applyCaptionWordHides,
   applyCaptionLineTiming,
+  applyCaptionLineWindows,
   captionEditsToKeep,
   isLegacyCaptionKey,
   migrateCaptionKeys,
@@ -14,6 +15,7 @@ import type {
   CaptionKeyMigration,
   CaptionLine,
   OverrideDoc,
+  TimeMap,
 } from "@ossclip/core";
 
 /**
@@ -288,10 +290,17 @@ export interface CaptionReconciliation {
  * (`produce.ts` parses it at the top of the run). On raw `JSON.parse` output a
  * literal `"__proto__"` key would be assigned THROUGH rather than kept, and
  * the migration would report nothing lost while losing it.
+ *
+ * `map` is the run's own cutlist map — the one `baseLines` were built through
+ * — and exists for the WINDOW layer alone (`applyCaptionLineWindows`): windows
+ * are stored in source seconds and these lines speak output seconds. Taken as
+ * a parameter rather than rebuilt here for the usual reason: two maps is how
+ * this pass and the render would come to place a caption differently.
  */
 export function reconcileCaptionEdits(
   doc: OverrideDoc,
   baseLines: readonly CaptionLine[],
+  map: TimeMap,
 ): CaptionReconciliation {
   const log: string[] = [];
   const migration = migrateCaptionKeys(doc.captions, baseLines);
@@ -351,5 +360,21 @@ export function reconcileCaptionEdits(
   const nudged = appliedCaptionEditCount(doc.captionLineTiming, timed.dropped);
   if (nudged > 0) log.push(`▸ ${nudged} caption timing nudge(s) applied`);
   for (const d of timed.dropped) log.push(captionTimingDropLine(d));
-  return { doc: migrated, lines: timed.lines, reanchored: reanchored > 0, log };
+  // WINDOWS last of all — `applyCaptionLayers`' one authoritative order: an
+  // absolute window is the user's final answer about when a caption is on
+  // screen, so it runs after (and overrides) the nudge layer. Counted with
+  // `appliedCaptionEditCount` for the same reason the nudges are: this layer
+  // also reports a `duplicate-anchor` drop per EXTRA claimant, so a plain
+  // subtraction would erase the line for a window that did apply. No key
+  // migration: `captionLineWindows` postdates §137, and the write-back above
+  // spreads it through untouched. Drops reuse `captionTimingDropLine` rather
+  // than earning a fifth sentence: both records answer "when is this caption
+  // on screen", both drop for the SAME two reasons (no line starts there any
+  // more / a second claimant), and the fix — re-make it in the timing tool —
+  // is one gesture, which is the test that function's docstring sets.
+  const windowed = applyCaptionLineWindows(timed.lines, doc.captionLineWindows, map);
+  const placed = appliedCaptionEditCount(doc.captionLineWindows, windowed.dropped);
+  if (placed > 0) log.push(`▸ ${placed} caption window(s) placed by the editor`);
+  for (const d of windowed.dropped) log.push(captionTimingDropLine(d));
+  return { doc: migrated, lines: windowed.lines, reanchored: reanchored > 0, log };
 }