ossclip 0.1.9 → 0.1.10

package/package.json

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

package/src/interactive/produce-wizard.ts

@@ -50,29 +50,36 @@ export function extrasFor(graphics: boolean): (typeof EXTRAS)[number][] {
   return graphics ? [...EXTRAS] : EXTRAS.filter((e) => e.value !== "graphicsClip");
 }
 
-export async function produceWizard(cfg: { speaker?: string } = {}): Promise<string[]> {
+export async function produceWizard(cfg: { speaker?: string; input?: string } = {}): Promise<string[]> {
   assertInteractive("produce wizard");
   intro("ossclip produce");
 
-  const input = unwrap(
-    await text({
-      // Finding 1 (final-review fix wave): `ossclip produce <folder>` shipped
-      // (folder-input-brief.md) but this prompt still rejected a directory —
-      // the wizard was the only way in that couldn't do what the CLI could.
-      // A folder is concatenated by name (codepoint order, like `ls`); --sort
-      // mtime reorders it but stays a typed flag, not a wizard question (see
-      // the file-level comment above for why).
-      message: "Video file, or a folder of clips to concatenate (by name; --sort mtime is a typed flag)",
-      placeholder: "./raw/take1.mp4",
-      validate: (v) => {
-        if (!v) return "a path is required";
-        if (!existsSync(v)) return `no such path: ${v}`;
-        const st = statSync(v);
-        if (!st.isFile() && !st.isDirectory()) return `${v} is neither a video file nor a folder`;
-        return undefined;
-      },
-    }),
-  ) as string;
+  // Pre-supplied by bare `ossclip <path>` (0.1.9 first-contact, 2026-08-05):
+  // the user already TYPED the input on the command line, and the old flow
+  // dropped it and asked again — the re-ask is where "./Anyhropic c Compiler"
+  // became "./" (all of ~/Downloads). The router checks existence before the
+  // wizard ever opens, so a prefilled path skips the prompt entirely.
+  const input =
+    cfg.input ??
+    (unwrap(
+      await text({
+        // Finding 1 (final-review fix wave): `ossclip produce <folder>` shipped
+        // (folder-input-brief.md) but this prompt still rejected a directory —
+        // the wizard was the only way in that couldn't do what the CLI could.
+        // A folder is concatenated by name (codepoint order, like `ls`); --sort
+        // mtime reorders it but stays a typed flag, not a wizard question (see
+        // the file-level comment above for why).
+        message: "Video file, or a folder of clips to concatenate (by name; --sort mtime is a typed flag)",
+        placeholder: "./raw/take1.mp4",
+        validate: (v) => {
+          if (!v) return "a path is required";
+          if (!existsSync(v)) return `no such path: ${v}`;
+          const st = statSync(v);
+          if (!st.isFile() && !st.isDirectory()) return `${v} is neither a video file nor a folder`;
+          return undefined;
+        },
+      }),
+    ) as string);
 
   const aspect = unwrap(
     await select({

package/src/program.ts

@@ -1,5 +1,5 @@
 import { spawn } from "node:child_process";
-import { readFileSync } from "node:fs";
+import { existsSync, readFileSync } from "node:fs";
 import { dirname, join, resolve } from "node:path";
 import { Command, InvalidArgumentError } from "commander";
 import { z } from "zod/v4";
@@ -51,31 +51,83 @@ export function buildProgram(): Command {
   // Bare `ossclip` at a TTY opens the menu. Piped or in CI it prints help,
   // byte for byte what it printed before — a front door must not become a
   // hang for a script.
-  program.action(async () => {
-    const { isInteractive } = await import("./interactive/tty");
-    if (!isInteractive()) {
-      program.outputHelp();
-      return;
-    }
-    const { chooseFromMenu, menuArgv } = await import("./interactive/menu");
-    const choice = await chooseFromMenu();
-    const direct = menuArgv(choice);
-    const { renderCommand } = await import("./interactive/render");
-    if (direct !== null) {
-      // Echoed for the same reason the wizard echoes: the README promises
-      // "every choice prints the equivalent command before it runs, so the
-      // menu is also how you learn the flags", and three of the four entries
-      // printed nothing. The menu's whole pedagogical point is this line.
-      console.log(`\n▸ running:\n    ${renderCommand(direct)}\n`);
-      await program.parseAsync(["node", "ossclip", ...direct]);
-      return;
-    }
-    const { produceWizard } = await import("./interactive/produce-wizard");
-    const { loadConfig } = await import("@ossclip/core");
-    const argv = await produceWizard({ speaker: loadConfig().speaker });
-    console.log(`\n▸ running:\n    ${renderCommand(argv)}\n`);
-    await program.parseAsync(["node", "ossclip", ...argv]);
-  });
+  //
+  // Bare `ossclip <path>` ROUTES (0.1.9 first-contact, 2026-08-05): with no
+  // argument declared here, commander's default allow-excess-args silently
+  // DROPPED the positional — `ossclip "./Anyhropic c Compiler"` opened the
+  // menu as if nothing had been typed, and the user re-answered the wizard's
+  // input prompt by hand, wrongly, with all of ~/Downloads. Commander
+  // dispatches registered subcommand names before this action ever runs, so
+  // `produce`/`edit`/… always win over path interpretation (a folder that
+  // happens to be NAMED `doctor` goes to the subcommand — acceptable);
+  // anything that lands here is a path or a typo, and a typo must be a loud
+  // error naming what was tried, never the menu.
+  program
+    .argument(
+      "[path]",
+      "an existing video file or clips folder — shorthand for `ossclip produce <path>`, " +
+        "with the wizard asking the remaining questions",
+    )
+    // Commander 12 allows excess args by default, which is the SECOND half of
+    // the field failure (review, Important): the report's path was typed
+    // UNQUOTED — `ossclip ./Anyhropic c Compiler` — and with excess args
+    // allowed, `c` and `Compiler` would vanish wordlessly the moment
+    // `./Anyhropic` happened to exist, producing the wrong scope. An unquoted
+    // multi-word path must be a loud error, never a partial run.
+    .allowExcessArguments(false)
+    .action(async (path: string | undefined) => {
+      const { isInteractive } = await import("./interactive/tty");
+      if (path !== undefined) {
+        if (!existsSync(path)) {
+          throw new Error(
+            `no such file or directory: ${path}\n` +
+              "  bare `ossclip <path>` produces an existing video file or clips folder;\n" +
+              "  run `ossclip --help` for the commands.",
+          );
+        }
+        const { renderCommand } = await import("./interactive/render");
+        // Both branches hand argv back through `program.parseAsync` — the same
+        // re-entry shape as the wizard paths below, for the same reason: the
+        // zod checks in the produce action must run on this input exactly as
+        // they do on a typed command line.
+        if (!isInteractive()) {
+          // No TTY means no wizard for the remaining questions, but the input
+          // IS supplied — a piped `ossclip <path>` is `ossclip produce <path>`.
+          const direct = ["produce", path];
+          console.log(`\n▸ running:\n    ${renderCommand(direct)}\n`);
+          await program.parseAsync(["node", "ossclip", ...direct]);
+          return;
+        }
+        const { produceWizard } = await import("./interactive/produce-wizard");
+        const { loadConfig } = await import("@ossclip/core");
+        const argv = await produceWizard({ speaker: loadConfig().speaker, input: path });
+        console.log(`\n▸ running:\n    ${renderCommand(argv)}\n`);
+        await program.parseAsync(["node", "ossclip", ...argv]);
+        return;
+      }
+      if (!isInteractive()) {
+        program.outputHelp();
+        return;
+      }
+      const { chooseFromMenu, menuArgv } = await import("./interactive/menu");
+      const choice = await chooseFromMenu();
+      const direct = menuArgv(choice);
+      const { renderCommand } = await import("./interactive/render");
+      if (direct !== null) {
+        // Echoed for the same reason the wizard echoes: the README promises
+        // "every choice prints the equivalent command before it runs, so the
+        // menu is also how you learn the flags", and three of the four entries
+        // printed nothing. The menu's whole pedagogical point is this line.
+        console.log(`\n▸ running:\n    ${renderCommand(direct)}\n`);
+        await program.parseAsync(["node", "ossclip", ...direct]);
+        return;
+      }
+      const { produceWizard } = await import("./interactive/produce-wizard");
+      const { loadConfig } = await import("@ossclip/core");
+      const argv = await produceWizard({ speaker: loadConfig().speaker });
+      console.log(`\n▸ running:\n    ${renderCommand(argv)}\n`);
+      await program.parseAsync(["node", "ossclip", ...argv]);
+    });
 
   program
     .command("produce")