npm - ossclip - Versions diffs - 0.1.4 → 0.1.6 - Mend

ossclip 0.1.4 → 0.1.6

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 (27) hide show

package/editor-dist/assets/{index-B5H_dKAW.js → index-DZhkQB1i.js} +24 -24
package/editor-dist/index.html +1 -1
package/package.json +5 -4
package/src/doctor.ts +32 -10
package/src/edit.ts +12 -2
package/src/index.ts +6 -237
package/src/interactive/edit-hint.ts +20 -0
package/src/interactive/menu.ts +32 -0
package/src/interactive/offer-editor.ts +71 -0
package/src/interactive/pick-workdir.ts +21 -0
package/src/interactive/prefs.ts +51 -0
package/src/interactive/produce-argv.ts +57 -0
package/src/interactive/produce-wizard.ts +180 -0
package/src/interactive/prompts.ts +6 -0
package/src/interactive/render.ts +30 -0
package/src/interactive/resolve-workdir.ts +115 -0
package/src/interactive/tty.ts +62 -0
package/src/interactive/workdir-probe.ts +70 -0
package/src/open.ts +30 -0
package/src/produce.ts +138 -16
package/src/program.ts +401 -0
package/src/setup/download.ts +122 -0
package/src/setup/extract.ts +91 -0
package/src/setup/manifest.ts +159 -0
package/src/setup/plan.ts +211 -0
package/src/setup/provider.ts +51 -0
package/src/setup/setup.ts +233 -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-B5H_dKAW.js"></script>
+    <script type="module" crossorigin src="/assets/index-DZhkQB1i.js"></script>
   </head>
   <body>
     <div id="root"></div>

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "ossclip",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "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",
@@ -32,12 +32,13 @@
     "editor-dist"
   ],
   "dependencies": {
+    "@clack/prompts": "^1.7.0",
     "commander": "^12.1.0",
     "tsx": "^4.19.0",
     "zod": "^3.25.76",
-    "@ossclip/core": "0.1.4",
-    "@ossclip/renderer": "0.1.4",
-    "@ossclip/scenes": "0.1.4"
+    "@ossclip/core": "0.1.6",
+    "@ossclip/renderer": "0.1.6",
+    "@ossclip/scenes": "0.1.6"
   },
   "homepage": "https://github.com/AhsanAyaz/ossclip#readme",
   "bugs": {

package/src/doctor.ts CHANGED Viewed

@@ -10,10 +10,15 @@ import type { OssclipConfig } from "@ossclip/core";
  * every one that fails silently becomes a GitHub issue instead of a
  * one-line fix the tool could have printed itself.
  *
+ * Since `ossclip setup` exists, doctor and setup are two halves of one
+ * contract: setup is the fix doctor prints first, and doctor is the
+ * verification setup ends with. The manual command stays on every fix line
+ * for people who'd rather own their toolchain.
+ *
  * Checks are pure over injected probes so the table is unit-testable; the
  * CLI wires the real spawn/existsSync in. The provider check MUST run after
  * `loadEnvFiles` (R16 §77) or a key living in `.env` reports a false
- * negative — `index.ts` loads env at module top, before any command runs.
+ * negative — `program.ts` loads env at module top, before any command runs.
  */
 export interface DoctorCheck {
@@ -36,9 +41,18 @@ export interface DoctorProbes {
   editorPageDir: string | null;
 }
-/** brew on mac, apt elsewhere — the generic hint covers the rest. */
-const installHint = (platform: NodeJS.Platform, brew: string, apt: string, generic: string): string =>
-  platform === "darwin" ? brew : platform === "linux" ? apt : generic;
+/** brew on mac, apt on linux, winget/prebuilt on windows — generic covers the rest. */
+const installHint = (
+  platform: NodeJS.Platform,
+  brew: string,
+  apt: string,
+  win: string,
+  generic: string,
+): string =>
+  platform === "darwin" ? brew : platform === "linux" ? apt : platform === "win32" ? win : generic;
+/** Every installable prerequisite leads with the one-command ramp. */
+const viaSetup = (manual: string): string => "run `ossclip setup` — or manually: " + manual;
 export async function runDoctor(cfg: OssclipConfig, p: DoctorProbes): Promise<DoctorCheck[]> {
   const checks: DoctorCheck[] = [];
@@ -60,13 +74,15 @@ export async function runDoctor(cfg: OssclipConfig, p: DoctorProbes): Promise<Do
     ...(ffmpegOk
       ? {}
       : {
-          fix:
+          fix: viaSetup(
             installHint(
               p.platform,
               "brew install ffmpeg",
               "sudo apt install ffmpeg",
+              "winget install ffmpeg",
               "install ffmpeg from https://ffmpeg.org",
             ) + " — or point OSSCLIP_FFMPEG (or config.json ffmpegPath) at the binary",
+          ),
         }),
   });
@@ -78,13 +94,15 @@ export async function runDoctor(cfg: OssclipConfig, p: DoctorProbes): Promise<Do
     ...(ffprobeOk
       ? {}
       : {
-          fix:
+          fix: viaSetup(
             installHint(
               p.platform,
               "brew install ffmpeg (provides ffprobe)",
               "sudo apt install ffmpeg (provides ffprobe)",
+              "winget install ffmpeg (provides ffprobe)",
               "ffprobe ships with ffmpeg — https://ffmpeg.org",
             ) + " — or set OSSCLIP_FFPROBE",
+          ),
         }),
   });
@@ -96,13 +114,15 @@ export async function runDoctor(cfg: OssclipConfig, p: DoctorProbes): Promise<Do
     ...(whisperOk
       ? {}
       : {
-          fix:
+          fix: viaSetup(
             installHint(
               p.platform,
               "brew install whisper-cpp",
-              "build whisper.cpp from source: https://github.com/ggml-org/whisper.cpp",
+              "download a prebuilt from https://github.com/ggml-org/whisper.cpp/releases",
+              "download whisper-blas-bin-x64.zip from https://github.com/ggml-org/whisper.cpp/releases",
               "build whisper.cpp from source: https://github.com/ggml-org/whisper.cpp",
             ) + " — or point OSSCLIP_WHISPER at your whisper-cli",
+          ),
         }),
   });
@@ -117,9 +137,10 @@ export async function runDoctor(cfg: OssclipConfig, p: DoctorProbes): Promise<Do
     ...(modelOk
       ? {}
       : {
-          fix:
+          fix: viaSetup(
             `mkdir -p ${cfg.modelDir} && curl -L -o ${modelPath} ` +
-            `https://huggingface.co/ggerganov/whisper.cpp/resolve/main/ggml-${cfg.model}.bin`,
+              `https://huggingface.co/ggerganov/whisper.cpp/resolve/main/ggml-${cfg.model}.bin`,
+          ),
         }),
   });
@@ -141,6 +162,7 @@ export async function runDoctor(cfg: OssclipConfig, p: DoctorProbes): Promise<Do
       ? {}
       : {
           fix:
+            "run `ossclip setup` (it can save a key for you), or " +
             "export ANTHROPIC_API_KEY or GEMINI_API_KEY (a .env file works — see README), " +
             "or install Claude Code (https://claude.com/claude-code) and log in",
         }),

package/src/edit.ts CHANGED Viewed

@@ -3,7 +3,7 @@ import { createReadStream, existsSync, statSync } from "node:fs";
 import { mkdir, readFile, readdir, rename, writeFile } from "node:fs/promises";
 import { createServer, type IncomingMessage, type ServerResponse } from "node:http";
 import { homedir } from "node:os";
-import { dirname, extname, isAbsolute, join, relative, resolve } from "node:path";
+import { dirname, extname, isAbsolute, join, relative, resolve, sep } from "node:path";
 import { fileURLToPath } from "node:url";
 import { z } from "zod/v4";
 import { OverrideDocSchema, emptyOverrideDoc } from "@ossclip/core";
@@ -194,7 +194,17 @@ export async function startEditServer(
   const openWorkdir = async (dirArg: string): Promise<void> => {
     const dir = resolve(dirArg);
     if (!isWorkdir(dir)) {
-      throw new Error(`no render-props.json in ${dir} — run \`ossclip produce\` there first`);
+      // Not "run produce there first" — the reported failure said that to a
+      // user who HAD, because produce writes one level down into
+      // .ossclip/<name>/ and this wanted that nested directory.
+      // The layout is spelled with the host separator, as resolve-workdir.ts
+      // does: hardcoded forward slashes here meant a Windows user met both
+      // conventions from one product depending on which entry point they hit.
+      throw new Error(
+        `no render-props.json in ${dir} — produce writes into ` +
+          `<video's folder>${sep}.ossclip${sep}<name>${sep}, and that nested folder ` +
+          `is what edit opens`,
+      );
     }
     workdir = dir;
     await recordRecentProject(dir, opts.recentDir);

package/src/index.ts CHANGED Viewed

@@ -1,242 +1,11 @@
 #!/usr/bin/env tsx
-import { spawn } from "node:child_process";
-import { readFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { Command, InvalidArgumentError } from "commander";
-import { z } from "zod/v4";
-import { CleanupLevelSchema, SceneComponentIdSchema } from "@ossclip/core";
-import { STUDIO_ENTRY } from "@ossclip/renderer";
-import { loadEnvFiles } from "./env";
-import { produce } from "./produce";
+import { buildProgram } from "./program";
-// Before anything reads a provider key (R16 §77) — including the auto-detect
-// order in `defaultProviderName`, which decides which model runs.
-const envFiles = loadEnvFiles();
-const program = new Command();
-program
-  .name("ossclip")
-  .description(
-    "local-first video producer: cuts silence and fillers, word-timed captions, " +
-      "face-aware framing, LLM-planned code-rendered graphics",
-  )
-  // Read from the manifest, never hardcoded (R22 §113): a literal here said
-  // "0.1.0" for every release after it, so `--version` reported the number a
-  // developer typed rather than the one npm installed — the exact field a
-  // bug report is judged by. npm always packs package.json regardless of
-  // `files`, so this resolves in a published install too.
-  .version(
-    (
-      JSON.parse(
-        readFileSync(new URL("../package.json", import.meta.url), "utf8"),
-      ) as { version: string }
-    ).version,
-  );
-program
-  .command("produce")
-  .description("transcribe → analyze → cut → captions → render")
-  .argument("<input>", "input video file")
-  .option("-o, --out <path>", "output video path (default: <input>.ossclip.mp4)")
-  .option("--cleanup <level>", "exact | light | standard | aggressive", "standard")
-  .option("--transcript <path>", "inject a transcript JSON instead of running whisper")
-  .option("--no-render", "stop after writing production.json / render props")
-  .option(
-    "--no-mezzanine",
-    "render straight from the source instead of a dense-keyframe mezzanine " +
-      "(also makes the source's folder the render server's public dir)",
-  )
-  .option("--noise-db <db>", "override the measured silence threshold, e.g. -30", parseFloat)
-  .option("--workdir <dir>", "cache/work directory (default: <input dir>/.ossclip)")
-  .option(
-    "--aspect <ratio>",
-    "output shape: 9:16 (vertical, default) or 16:9 (landscape, 1920x1080)",
-    "9:16",
-  )
-  .option("--produce", "run the LLM producer brain to plan title cards & graphics", false)
-  .option(
-    "--clip <seconds>",
-    "produce only the strongest ~N-second window of a long take (requires --produce; " +
-      "a source already at or under the target is produced whole)",
-    (v: string) => {
-      // §93a: reject rather than coerce — `--clip 0`, negatives and typos must
-      // not silently become "no clip" or NaN-length windows.
-      const n = Number.parseFloat(v);
-      if (!Number.isFinite(n) || n <= 0) {
-        throw new InvalidArgumentError(`--clip wants a positive number of seconds, got "${v}"`);
-      }
-      return n;
-    },
-  )
-  .option(
-    "--clip-window <start:end>",
-    "internal: the resolved highlight's word range, recorded into command.json by --clip " +
-      "runs so the editor's Render replays the same window without an LLM call",
-  )
-  .option("--intent <text>", "what the video should be ('educational video about agents…')")
-  .option(
-    "--llm <provider>",
-    "claude | claude-cli | gemini | mock. Default: claude if ANTHROPIC_API_KEY is set, " +
-      "else claude-cli (your logged-in Claude Code — Pro/Max subscription, no API charges)",
-  )
-  .option("--llm-model <id>", "override the provider's default model")
-  .option(
-    "--llm-fast-model <id>",
-    "model for mechanical calls (repair, scene props); 'same' disables tiering",
-  )
-  .option(
-    "--speaker <who>",
-    'who is on camera, e.g. "Ahsan, host of Code with Ahsan" — helps repair recognise mangled names',
-  )
-  .option("--scenes <path>", "hand-authored scenes JSON (Scene[]) — no LLM in the loop")
-  .option(
-    "--no-repair",
-    "skip the ASR mishearing repair pass (captions then show the raw transcription)",
-  )
-  .option("--whisper-model <name>", "transcription model for this run, e.g. base.en | small.en | medium.en")
-  .option(
-    "--force-component <id>",
-    "debug: render every graphic with this component (e.g. FlowDiagram) to exercise it on real copy",
-  )
-  .option(
-    "--source-fit <mode>",
-    "cover | contain. cover (default) crops the source to fill the vertical " +
-      "frame; contain shows the WHOLE frame inset against the backdrop — the " +
-      "answer for a landscape take whose content matters beyond the speaker",
-    "cover",
-  )
-  .option(
-    "--source-is-edited",
-    "the source is already an edited reel with burned-in text — keep ossclip's graphics off it without waiting on detection",
-  )
-  .option("--no-cover", "skip the cover image written beside the video")
-  .option("--cover <path>", "cover image output path (default: <out>.cover.jpg)")
-  .action(async (input: string, opts) => {
-    // Say which keys came from a file — never the keys themselves. A run that
-    // picks a provider from a `.env` should say where that came from.
-    if (envFiles.length > 0) console.log(`▸ env: ${envFiles.join(", ")}`);
-    const cleanup = CleanupLevelSchema.parse(opts.cleanup);
-    const provider = opts.llm
-      ? z.enum(["claude", "claude-cli", "gemini", "mock"]).parse(opts.llm)
-      : undefined;
-    const forceComponent = opts.forceComponent
-      ? SceneComponentIdSchema.parse(opts.forceComponent)
-      : undefined;
-    // Parsed, not coerced: a typo'd `--source-fit containn` silently falling
-    // back to cover is exactly the crop the flag exists to prevent.
-    const sourceFit = z.enum(["cover", "contain"]).parse(opts.sourceFit);
-    await produce(input, {
-      out: opts.out,
-      cleanup,
-      transcript: opts.transcript,
-      render: opts.render,
-      mezzanine: opts.mezzanine,
-      workdir: opts.workdir,
-      aspect: opts.aspect === "16:9" ? "16:9" : "9:16",
-      noiseDb: opts.noiseDb,
-      produce: opts.produce,
-      intent: opts.intent,
-      provider,
-      llmModel: opts.llmModel,
-      llmFastModel: opts.llmFastModel,
-      speaker: opts.speaker,
-      scenes: opts.scenes,
-      repair: opts.repair,
-      whisperModel: opts.whisperModel,
-      forceComponent,
-      // commander gives `--no-cover` as cover:false and `--cover <path>` as a
-      // string on the same key.
-      sourceIsEdited: opts.sourceIsEdited === true,
-      sourceFit,
-      cover: opts.cover !== false,
-      coverPath: typeof opts.cover === "string" ? opts.cover : undefined,
-      clip: opts.clip,
-      clipWindow: opts.clipWindow,
-    });
-  });
-program
-  .command("transcribe")
-  .description("run the pipeline up to the transcript and cut report, no render")
-  .argument("<input>", "input video file")
-  .option("--cleanup <level>", "exact | light | standard | aggressive", "standard")
-  .option("--transcript <path>", "inject a transcript JSON instead of running whisper")
-  .option("--noise-db <db>", "override the measured silence threshold, e.g. -30", parseFloat)
-  .option("--workdir <dir>", "cache/work directory")
-  .option("--whisper-model <name>", "transcription model for this run, e.g. base.en | small.en | medium.en")
-  .action(async (input: string, opts) => {
-    const cleanup = CleanupLevelSchema.parse(opts.cleanup);
-    await produce(input, {
-      cleanup,
-      transcript: opts.transcript,
-      render: false,
-      mezzanine: false,
-      workdir: opts.workdir,
-      noiseDb: opts.noiseDb,
-      whisperModel: opts.whisperModel,
-    });
-  });
-program
-  .command("studio")
-  .description("open Remotion Studio on a produced composition (visual debugging)")
-  .argument("<renderProps>", "path to a work dir's render-props.json")
-  .option("--video-dir <dir>", "directory containing the source video (public dir)")
-  .action(async (renderProps: string, opts) => {
-    const propsPath = resolve(renderProps);
-    const publicDir = opts.videoDir ? resolve(opts.videoDir) : dirname(propsPath);
-    const child = spawn(
-      "pnpm",
-      ["exec", "remotion", "studio", STUDIO_ENTRY, `--props=${propsPath}`, `--public-dir=${publicDir}`],
-      { stdio: "inherit" },
-    );
-    child.on("exit", (code) => process.exit(code ?? 0));
-  });
-program
-  .command("edit")
-  .description("open the editing page on a produced workdir")
-  // OPTIONAL since R17 §83: with no argument the editor opens on a project
-  // picker — recent produce runs plus a folder browser — and the top bar's
-  // Open button switches projects without restarting the server.
-  .argument("[workdir]", "a work directory containing render-props.json")
-  .option("--port <n>", "port to listen on", (v) => Number.parseInt(v, 10), 5174)
-  .option("--no-open", "do not open a browser")
-  .action(async (workdir: string | undefined, opts) => {
-    const { startEditServer, resolveEditorPageDir } = await import("./edit");
-    // An npm install ships the page prebuilt (editor-dist/); a clone builds
-    // it once with `pnpm build`. A server that starts fine but 404s every
-    // page request is the worst version of missing — fail loudly with the
-    // fix instead.
-    const pageDir = resolveEditorPageDir();
-    if (pageDir === null) {
-      throw new Error(
-        "editor UI isn't built yet — run `pnpm build` " +
-          "(or `pnpm --filter @ossclip/editor build`) once, then re-run `ossclip edit`.",
-      );
-    }
-    const server = await startEditServer(workdir, { port: opts.port, pageDir });
-    console.log(`▸ editor at ${server.url}`);
-    if (opts.open) spawn("open", [server.url], { stdio: "ignore" });
-  });
-program
-  .command("doctor")
-  .description("check every prerequisite and print the exact fix for anything missing")
-  .action(async () => {
-    // Env files are loaded at module top (R16 §77) — BEFORE this runs — so a
-    // provider key living in a `.env` is visible here, not a false negative.
-    if (envFiles.length > 0) console.log(`▸ env: ${envFiles.join(", ")}`);
-    const { runDoctor, formatDoctor, realProbes } = await import("./doctor");
-    const { resolveEditorPageDir } = await import("./edit");
-    const { loadConfig } = await import("@ossclip/core");
-    const checks = await runDoctor(loadConfig(), realProbes(resolveEditorPageDir()));
-    console.log(formatDoctor(checks));
-    if (checks.some((c) => !c.ok)) process.exit(1);
-  });
-program.parseAsync().catch((err) => {
+// The side effect (loadEnvFiles, R16 §77) stays at import time inside
+// program.ts: bin/ossclip.mjs imports this module and expects it to run, and
+// this file's first statement importing program.ts is what preserves that
+// ordering — before anything reads a provider key.
+buildProgram().parseAsync().catch((err) => {
   console.error(`\n✗ ${err instanceof Error ? err.message : err}`);
   process.exit(1);
 });

package/src/interactive/edit-hint.ts ADDED Viewed

@@ -0,0 +1,20 @@
+import { renderCommand } from "./render";
+/**
+ * The closing signpost of a produce run.
+ *
+ * `▸ workdir <path>` is printed at the START of a run, which after six
+ * minutes of transcription and rendering is thousands of lines up the
+ * scrollback. The reported user had the written guide open and still could
+ * not find the directory, because the last thing on screen named neither it
+ * nor the command that opens it.
+ */
+export function editHint(workdir: string, platform: NodeJS.Platform = process.platform): string {
+  // The platform is a parameter, defaulted to the host, for the same reason
+  // resolveWorkdir takes `sep`: the Windows rendering has to be assertable
+  // from a macOS dev machine and an ubuntu CI leg. It was previously PINNED
+  // to "linux", which handed a Windows user under `D:\My Videos\` POSIX
+  // single quotes that cmd.exe passes through literally — the branch's
+  // headline artifact, broken on the platform the bug report came from.
+  return `▸ edit it:  ${renderCommand(["edit", workdir], platform)}`;
+}

package/src/interactive/menu.ts ADDED Viewed

@@ -0,0 +1,32 @@
+import { assertInteractive, intro, select, unwrap } from "./prompts";
+export type MenuChoice = "produce" | "edit" | "setup" | "doctor";
+/**
+ * What each menu entry runs. Produce is the exception — it needs answers
+ * before it has an argv, so it returns null and the caller hands off to the
+ * wizard.
+ */
+export function menuArgv(choice: MenuChoice): string[] | null {
+  if (choice === "produce") return null;
+  // Edit with NO argument is deliberate: that is the project picker over
+  // recent runs (R17 §83), which is exactly what somebody who reached a menu
+  // instead of typing a command needs.
+  return [choice];
+}
+export async function chooseFromMenu(): Promise<MenuChoice> {
+  assertInteractive("main menu");
+  intro("ossclip");
+  return unwrap(
+    await select({
+      message: "What do you want to do?",
+      options: [
+        { value: "produce", label: "Produce a video", hint: "cut, caption, frame, render" },
+        { value: "edit", label: "Edit a produced project", hint: "pick from recent runs" },
+        { value: "setup", label: "Set up my install", hint: "ffmpeg, whisper, the model" },
+        { value: "doctor", label: "Check what's missing" },
+      ],
+    }),
+  ) as MenuChoice;
+}

package/src/interactive/offer-editor.ts ADDED Viewed

@@ -0,0 +1,71 @@
+import { loadConfig, saveConfigPatch, type OpenEditorPref } from "@ossclip/core";
+import type { ProduceResult } from "../produce";
+import { answerToDecision, decideOpenEditor, type OpenEditorAnswer } from "./prefs";
+import { isInteractive, select, unwrap } from "./prompts";
+import { renderCommand } from "./render";
+/**
+ * The offer at the end of a produce run. The user who prompted this work
+ * asked "how can I open the editor?" BEFORE running anything — the answer
+ * belongs at the moment there is finally something to open.
+ */
+export async function offerEditor(
+  result: ProduceResult,
+  opts: { flag: boolean | undefined; port: number },
+): Promise<void> {
+  const pref: OpenEditorPref = loadConfig().openEditorAfterProduce ?? "ask";
+  const decision = decideOpenEditor({
+    flag: opts.flag,
+    pref,
+    interactive: isInteractive(),
+    rendered: result.rendered,
+  });
+  if (decision === "skip") return;
+  let open = decision === "open";
+  if (decision === "ask") {
+    const answer = unwrap(
+      await select({
+        message: "Open the editor on this project?",
+        options: [
+          { value: "yes", label: "Yes" },
+          { value: "no", label: "No" },
+          { value: "always", label: "Yes, and stop asking" },
+          { value: "never", label: "No, and stop asking" },
+        ],
+      }),
+    ) as OpenEditorAnswer;
+    // The mapping itself lives in prefs.ts, where four answers are asserted
+    // without a TTY — this file is I/O and a manual walk was its only cover.
+    const decided = answerToDecision(answer);
+    if (decided.pref !== undefined) {
+      const path = saveConfigPatch({ openEditorAfterProduce: decided.pref });
+      // Say where the answer went, and how to take it back — a preference
+      // saved silently is one the user cannot find again.
+      console.log(`▸ saved openEditorAfterProduce="${decided.pref}" to ${path}`);
+    }
+    open = decided.open;
+  }
+  if (!open) return;
+  const { startEditServer, resolveEditorPageDir } = await import("../edit");
+  const pageDir = resolveEditorPageDir();
+  if (pageDir === null) {
+    // Not fatal here: the render succeeded. Say what is missing and stop.
+    // Through renderCommand like every other `ossclip edit <path>` we print:
+    // hand-built, this one quoted nothing, so a workdir with a space in it
+    // printed a command that fails.
+    console.log(
+      "▸ editor UI isn't built — run `pnpm build` once, then " +
+        renderCommand(["edit", result.workdir]),
+    );
+    return;
+  }
+  const server = await startEditServer(result.workdir, { port: opts.port, pageDir });
+  console.log(`▸ editor at ${server.url}`);
+  const { openInBrowser } = await import("../open");
+  openInBrowser(server.url);
+}

package/src/interactive/pick-workdir.ts ADDED Viewed

@@ -0,0 +1,21 @@
+import { basename } from "node:path";
+import type { Candidate } from "./resolve-workdir";
+import { assertInteractive, select, unwrap } from "./prompts";
+/**
+ * The "several runs under .ossclip" rung. Newest first is already guaranteed
+ * by resolveWorkdir; this only renders the choice.
+ */
+export async function pickWorkdir(candidates: Candidate[]): Promise<string> {
+  assertInteractive("workdir picker");
+  return unwrap(
+    await select({
+      message: "Several produce runs here — which one?",
+      options: candidates.map((c, i) => ({
+        value: c.path,
+        label: basename(c.path),
+        hint: i === 0 ? "most recent" : undefined,
+      })),
+    }),
+  ) as string;
+}

package/src/interactive/prefs.ts ADDED Viewed

@@ -0,0 +1,51 @@
+import type { OpenEditorPref } from "@ossclip/core";
+export type OpenEditorDecision = "open" | "skip" | "ask";
+/**
+ * Whether a finished produce run opens the editor, asks, or says nothing.
+ *
+ * Pure so the whole precedence order is tested without a produce run: flags
+ * beat the stored preference, the stored preference beats asking, and no TTY
+ * means never ask.
+ */
+export function decideOpenEditor(i: {
+  flag: boolean | undefined;
+  pref: OpenEditorPref;
+  interactive: boolean;
+  rendered: boolean;
+}): OpenEditorDecision {
+  // An explicit flag is a deliberate instruction and wins outright — including
+  // over `rendered`, because the editor reads render-props.json, which a
+  // --no-render run does write.
+  if (i.flag === true) return "open";
+  if (i.flag === false) return "skip";
+  // Above the stored preference, not below it: a persisted "always" (or
+  // OSSCLIP_OPEN_EDITOR=always) is not a per-run instruction, and starting a
+  // long-lived edit server in `ossclip produce take.mp4 > build.log 2>&1`
+  // holds the event loop open with nobody there to see it or close it. Only
+  // the explicit flag above may do that.
+  if (!i.interactive) return "skip";
+  // Otherwise a run with no render has nothing to look at, so the offer is noise.
+  if (!i.rendered) return "skip";
+  if (i.pref === "always") return "open";
+  if (i.pref === "never") return "skip";
+  return "ask";
+}
+export type OpenEditorAnswer = "yes" | "no" | "always" | "never";
+/**
+ * What each answer to the end-of-run offer means: whether to open now, and
+ * the preference to persist if the answer was one of the two that stop the
+ * asking. Pure so all four are asserted without a prompt — the interactive
+ * path is then only the I/O around it.
+ */
+export function answerToDecision(answer: OpenEditorAnswer): {
+  pref?: OpenEditorPref;
+  open: boolean;
+} {
+  const open = answer === "yes" || answer === "always";
+  if (answer === "always" || answer === "never") return { pref: answer, open };
+  return { open };
+}