fix(cli): preserve media after the last transcript word

packages/cli/src/media-use/lib/cutlist.test.mjs

@@ -132,8 +132,6 @@ test("--plan on a fixture transcript prints the exact segment JSON", () => {
       process.execPath,
       [
         SCRIPT,
-        "--input",
-        "ignored.mp4",
         "--transcript",
         transcriptPath,
         "--remove",

skills-manifest.json

@@ -54,7 +54,7 @@
       "files": 1
     },
     "media-use": {
-      "hash": "c8800249da2d1ce6",
+      "hash": "f0396dd8114cd379",
       "files": 109
     },
     "motion-graphics": {

skills/media-use/references/operations.md

@@ -143,7 +143,7 @@ node <SKILL_DIR>/scripts/transcript-cut.mjs \
 resolve --from talk.cut.mp4 --type video
 ```
 
-Use `--plan` first when you want to inspect the kept segment JSON before encoding.
+Use `--plan` with `--input` first when you want to inspect the kept segment JSON before encoding. The source's measured duration bounds the edits, including footage after the last word. A transcript-only `--plan` needs no media tools and uses the last word's end as its duration.
 
 ## Ducking (declare in-composition / bake for export)
 

skills/media-use/scripts/transcript-cut.mjs

@@ -42,7 +42,7 @@ Options:
   --cut-silence       Remove inter-word gaps longer than this many seconds
   --keep              Inverse mode: direct kept ranges, mutually exclusive with removal
   --copy              Use stream copy for faster, keyframe-snapped cuts
-  --plan              Print kept segment JSON and exit without ffmpeg
+  --plan              Print kept segment JSON; uses source duration when --input is supplied
   --out               Output file
   --json              Output JSON status
   --help, -h          Show this help`);
@@ -66,8 +66,12 @@ try {
 
 function run() {
   if (!args.transcript) throw new Error("--transcript is required");
+  const inputPath = args.input ? resolve(args.input) : null;
+  // A last word at 4s must not drop a 6s source's outro when removing a filler.
+  const totalSeconds = inputPath ? probeDuration(inputPath) : null;
   const transcript = JSON.parse(readFileSync(resolve(args.transcript), "utf8"));
   const segments = compileCutList(transcript, {
+    duration: totalSeconds,
     remove: args.remove,
     removeWords: args["remove-words"],
     removeFillers: args["remove-fillers"],
@@ -80,16 +84,14 @@ function run() {
     return;
   }
 
-  if (!args.input || !args.out)
+  if (!inputPath || !args.out)
     throw new Error("--input and --out are required unless --plan is set");
   if (segments.length === 0) throw new Error("cut list has no kept segments");
 
-  const inputPath = resolve(args.input);
   const outPath = resolve(args.out);
   mkdirSync(dirname(outPath), { recursive: true });
   const tmpDir = mkdtempSync(join(tmpdir(), "media-use-cut-"));
   const keptSeconds = sumDurations(segments);
-  const totalSeconds = probeDuration(inputPath);
 
   try {
     const parts = segments.map((segment, index) => {

skills/media-use/scripts/transcript-cut.test.mjs

@@ -28,6 +28,20 @@ function run(args) {
   return spawnSync(process.execPath, [SCRIPT, ...args], { encoding: "utf8" });
 }
 
+function writeTone(source) {
+  execFileSync("ffmpeg", [
+    "-y",
+    "-hide_banner",
+    "-loglevel",
+    "error",
+    "-f",
+    "lavfi",
+    "-i",
+    `sine=frequency=440:duration=6:sample_rate=${SAMPLE_RATE}`,
+    source,
+  ]);
+}
+
 function readPcm(filePath) {
   const raw = execFileSync("ffmpeg", [
     "-hide_banner",
@@ -47,6 +61,64 @@ function readPcm(filePath) {
   return samples;
 }
 
+for (const { name, edits, segments, duration } of [
+  {
+    name: "preserves the source tail when removing a filler",
+    edits: ["--remove-fillers", "um"],
+    segments: [
+      { start: 0, end: 1 },
+      { start: 1.5, end: 6 },
+    ],
+    duration: 5.5,
+  },
+  {
+    name: "keeps an explicit range after the last spoken word",
+    edits: ["--keep", "4-6"],
+    segments: [{ start: 4, end: 6 }],
+    duration: 2,
+  },
+]) {
+  test(name, { skip: !HAS_FFMPEG }, (t) => {
+    const { dir, cleanup } = fixture();
+    t.after(cleanup);
+    const source = join(dir, "tone.wav");
+    writeTone(source);
+    const transcriptPath = join(dir, "transcript.json");
+    writeFileSync(
+      transcriptPath,
+      JSON.stringify({
+        words: [
+          { text: "hello", start: 0, end: 1 },
+          { text: "um", start: 1, end: 1.5 },
+          { text: "goodbye", start: 1.5, end: 4 },
+        ],
+      }),
+    );
+    const output = join(dir, "out.wav");
+    const result = run([
+      "--input",
+      source,
+      "--transcript",
+      transcriptPath,
+      ...edits,
+      "--out",
+      output,
+      "--json",
+    ]);
+    assert.equal(result.status, 0, result.stderr || result.stdout);
+    const status = JSON.parse(result.stdout);
+    assert.deepEqual(status.segments, segments);
+    assert.equal(status.kept_s, duration);
+    assert.equal(status.total_s, 6);
+    const plan = run(["--input", source, "--transcript", transcriptPath, ...edits, "--plan"]);
+    assert.equal(plan.status, 0, plan.stderr || plan.stdout);
+    assert.deepEqual(JSON.parse(plan.stdout), segments);
+    const samples = readPcm(output);
+    assert.equal(samples.length, duration * SAMPLE_RATE);
+    assert.ok(samples.slice(-SAMPLE_RATE).some((sample) => Math.abs(sample) > 1000));
+  });
+}
+
 test(
   "keeps a spliced tone continuous at every cut, with no raw phase jump or silence gap",
   { skip: !HAS_FFMPEG },
@@ -55,17 +127,7 @@ test(
     t.after(cleanup);
 
     const source = join(dir, "tone.wav");
-    execFileSync("ffmpeg", [
-      "-y",
-      "-hide_banner",
-      "-loglevel",
-      "error",
-      "-f",
-      "lavfi",
-      "-i",
-      `sine=frequency=440:duration=6:sample_rate=${SAMPLE_RATE}`,
-      source,
-    ]);
+    writeTone(source);
 
     const transcriptPath = join(dir, "transcript.json");
     writeFileSync(transcriptPath, JSON.stringify({ words: [{ text: "tone", start: 0, end: 6 }] }));