@hugpy/video-intelligence-ui 0.1.0 → 0.1.1

package/src/stations/studio/StudioMovieComposer.tsx

@@ -66,7 +66,8 @@ import {
   type JobStatus,
 } from "../../video/contract";
 import { getSessionId } from "../../session";
-import { useMediaLibrary } from "../../video/mediaLibrary";
+import { addToLibrary, useMediaLibrary } from "../../video/mediaLibrary";
+import { UnnamedWarnPanel, WarnUnnamedToggle, useWarnUnnamedPref } from "../../video/UnnamedWarn";
 import { GoalComposer } from "../GoalComposer";
 import { usePromptAssist, type AssistExtras } from "../../video/usePromptAssist";
 import { PromptAssistButtons } from "../../video/PromptAssistButtons";
@@ -295,12 +296,19 @@ interface GoalRow {
   steps: string;
   cfg: string;
   contextFrames: string;
+  // per-segment CLIP LENGTH in frames (2026-08-13); blank = movie default.
+  frames: string;
 }
 
 // The counter is SESSION-persisted (not a module `let`) now that the goal rows
 // themselves survive a reload: a module counter restarts at 0 while the restored
 // rows keep their old keys, so `goal_1` could be minted twice — and this key
 // doubles as the wire `segment_id`. Same shape, just a counter that remembers.
+// The real-model frame ceiling (Wan family: 81) — the spine clamps to this
+// and snaps to 4k+1. Split-to-fit uses it to materialize CONTINUATION
+// segments instead of letting a longer ask silently degrade.
+const MODEL_FRAME_CEILING = 81;
+
 function newGoalKey(): string {
   return `goal_${nextSessionSeq("studio.movie.goalKeySeq")}`;
 }
@@ -309,7 +317,7 @@ function blankGoal(joint: JointMode = "still"): GoalRow {
     key: newGoalKey(),
     prompt: "",
     negative: "",
-    stdNegative: false,
+    stdNegative: true,
     seed: "",
     branchFrame: "",
     joint,
@@ -317,6 +325,7 @@ function blankGoal(joint: JointMode = "still"): GoalRow {
     steps: "",
     cfg: "",
     contextFrames: "",
+    frames: "",
   };
 }
 
@@ -602,6 +611,7 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
         steps: "",
         cfg: "",
         contextFrames: "",
+        frames: "",
         ...g,
       })),
     [goalsRaw],
@@ -611,6 +621,10 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
   // Default 480×480: the wan2.1-vace-1.3b IDENTITY sweet spot — the only budget-fitting
   // id-capable model on this fleet tops out at 832×480, so a 512×512 default cannot serve
   // id_lock on it (no_capable_model). 832×480 widescreen is the other valid id envelope.
+  // EXPLICIT SEGMENT LENGTH (operator 2026-08-13): movie-level default frames
+  // per segment; a segment's own frames (below) overrides. Blank = each bound
+  // model's default (81 real). Clamped + 4k+1-snapped by the spine.
+  const [framesDefault, setFramesDefault] = useSessionState("studio.movie.frames", "");
   const [width, setWidth] = useSessionState("studio.movie.width", 480);
   const [height, setHeight] = useSessionState("studio.movie.height", 480);
   const [fps, setFps] = useSessionState("studio.movie.fps", 24);
@@ -1011,6 +1025,44 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
     },
     [],
   );
+
+  // SPLIT-TO-FIT (operator 2026-08-13): a length ask beyond the model ceiling
+  // becomes VISIBLE continuation segments — motion-carried (vace_extend),
+  // length-distributed, each editable — never submit-time magic. The
+  // continuation prompt is a plain, honest default; ✨ Generate can refine it.
+  const splitGoalToFit = useCallback(
+    (i: number) => {
+      const g = goals[i];
+      if (!g) return;
+      const ask = Number(g.frames.trim() || framesDefault.trim());
+      if (!Number.isInteger(ask) || ask <= MODEL_FRAME_CEILING) return;
+      const n = Math.ceil(ask / MODEL_FRAME_CEILING);
+      const per = Math.ceil(ask / n);
+      const chunks: number[] = [];
+      let left = ask;
+      for (let k = 0; k < n; k++) {
+        const c = Math.min(per, left);
+        chunks.push(Math.max(5, c));
+        left -= c;
+      }
+      const base = g.prompt.trim();
+      const contPrompt = base
+        ? `the scene continues seamlessly, same setting and motion — ${base.split(/[.,;\n]/)[0].trim().slice(0, 80)}`
+        : "the scene continues seamlessly, same setting and motion";
+      setGoals((prev) => {
+        const next = [...prev];
+        next[i] = { ...next[i], frames: String(chunks[0]) };
+        const conts = chunks.slice(1).map((c) => ({
+          ...blankGoal("vace_extend"),
+          prompt: contPrompt,
+          frames: String(c),
+        }));
+        next.splice(i + 1, 0, ...conts);
+        return next;
+      });
+    },
+    [goals, framesDefault, setGoals],
+  );
   const addGoal = useCallback(() => setGoals((prev) => [...prev, blankGoal()]), []);
   const removeGoal = useCallback((idx: number) => {
     // Keep at least one row — a movie needs a root segment.
@@ -1133,18 +1185,54 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
   // The row's RESOLVED operation only chooses which mode the primary button invokes;
   // both explicit buttons stay live underneath, so a router mistake costs one click,
   // never a paragraph of the user's work.
+  // The last AUTO-derived movie title. A name the operator typed is sacred; a
+  // name this ref minted may be replaced by the next Generate (operator
+  // 2026-08-13: "purely a product of the same generation method").
+  const autoTitleRef = useRef("");
+
   const runRowAssist = useCallback(
     (i: number, mode: "detail" | "generate") => {
       const g = goals[i];
       if (!g) return;
-      void assist.runAssist(mode, g.prompt, (p) => setGoalField(i, { prompt: p }), {
-        context: rowContext(i),
-      });
+      void (async () => {
+        let fresh = "";
+        await assist.runAssist(mode, g.prompt, (p) => {
+          fresh = p;
+          setGoalField(i, { prompt: p });
+        }, { context: rowContext(i) });
+        if (mode !== "generate" || !fresh.trim()) return;
+        // NEGATIVE RIDES ALONG (operator 2026-08-13): the generated exclusion
+        // list APPENDS to this goal's typed negative, never clobbers it
+        // (composeNegative dedupes the std terms at payload time).
+        await assist.runNegative(
+          { subject: fresh, extra: { context: rowContext(i) } },
+          (neg) => {
+            const cur = (g.negative ?? "").trim().replace(/,\s*$/, "");
+            setGoalField(i, { negative: cur ? `${cur}, ${neg}` : neg });
+          },
+        );
+        // NAME RIDES ALONG: derive a title from the generated prompt when the
+        // movie has none — or when the current one was itself auto-derived.
+        const t = movieTitle.trim();
+        if (t === "" || t === autoTitleRef.current) {
+          const derived = fresh
+            .split(/[.,;\n]/)[0]
+            .trim()
+            .split(/\s+/)
+            .slice(0, 7)
+            .join(" ")
+            .slice(0, 48);
+          if (derived) {
+            autoTitleRef.current = derived;
+            setMovieTitle(derived);
+          }
+        }
+      })();
       // The text is about to change, so the old classification no longer describes it.
       intentText.current[g.key] = "";
       setIntents((prev) => ({ ...prev, [g.key]: null }));
     },
-    [assist, goals, rowContext, setGoalField],
+    [assist, goals, rowContext, setGoalField, movieTitle, setMovieTitle],
   );
 
   // ── negative generation (§1b) — an exclusion list for THIS shot ──
@@ -1300,7 +1388,7 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
   }, [pendingFocusKey, goals]);
 
   // ── build the exact request body the route reads ──
-  const buildBody = useCallback((): { body: Record<string, unknown> } | { err: string } => {
+  const buildBody = useCallback((titleOverride?: string): { body: Record<string, unknown> } | { err: string } => {
     if (goals.length < 1) return { err: "Add at least one goal (the root segment)." };
     for (let i = 0; i < goals.length; i++) {
       if (goals[i].prompt.trim() === "") {
@@ -1320,6 +1408,10 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
       if (gCfg !== "" && (Number.isNaN(Number(gCfg)) || Number(gCfg) < 0 || Number(gCfg) > 20)) {
         return { err: `Goal ${i + 1} CFG must be a number in [0, 20] (or blank to inherit).` };
       }
+      const gFrames = goals[i].frames.trim();
+      if (gFrames !== "" && (!Number.isInteger(Number(gFrames)) || Number(gFrames) < 1)) {
+        return { err: `Goal ${i + 1} length must be a whole number of frames \u2265 1 (or blank to inherit).` };
+      }
       const gCtx = goals[i].contextFrames.trim();
       if (gCtx !== "" && (!Number.isInteger(Number(gCtx)) || Number(gCtx) < 1)) {
         return { err: `Goal ${i + 1} context frames must be a whole number ≥ 1 (or blank to inherit).` };
@@ -1347,6 +1439,10 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
     // Movie-level sampler overrides — same bounds the clip surface (and the movie
     // schema's _check_steps_cfg) enforce: steps int [1,100], cfg number [0,20]. Blank
     // drops the key (model default).
+    const framesDefNum = framesDefault.trim() === "" ? undefined : Number(framesDefault);
+    if (framesDefNum !== undefined && (!Number.isInteger(framesDefNum) || framesDefNum < 1)) {
+      return { err: "Segment length must be a whole number of frames \u2265 1 (or blank for model defaults)." };
+    }
     const stepsNum = steps.trim() === "" ? undefined : Number(steps);
     if (
       stepsNum !== undefined &&
@@ -1376,6 +1472,7 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
       if (g.modelId.trim() !== "") node.model_id = g.modelId.trim();
       if (g.steps.trim() !== "") node.steps = Number(g.steps);
       if (g.cfg.trim() !== "") node.cfg = Number(g.cfg);
+      if (g.frames.trim() !== "") node.frames = Number(g.frames);
       // joint_mode + branch_frame are meaningless on the root (segment 0 has no parent).
       if (i > 0) {
         // Send a non-default joint_mode ("vace_extend" | "cut"); "still" is the route default.
@@ -1396,6 +1493,7 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
 
     const body: Record<string, unknown> = {
       resolution: { width, height, fps },
+      frames: framesDefNum,
       seed,
       vram_budget_gb: budgetNum,
       // Movie-level model pin + sampler overrides (blank ⇒ key dropped = router/model
@@ -1404,12 +1502,14 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
       model_id: modelId.trim() ? modelId.trim() : undefined,
       steps: stepsNum,
       cfg: cfgNum,
-      negative_prompt: negative.trim() ? negative : undefined,
+      // negative_prompt (movie-wide) RETIRED from this surface 2026-08-13
+      // (operator: deprecated artifact) — per-goal negatives + the standard
+      // tick compose everything; the route treats absence as None cleanly.
       project: project.trim() ? project.trim() : undefined,
       // The movie's NAME (k91) — free text, display-only: it never changes what renders,
       // it is what the sessions list shows instead of a bare dir id. Blank drops the key
       // (the schema takes title=None and the session lists as "untitled").
-      title: movieTitle.trim() ? movieTitle.trim() : undefined,
+      title: (titleOverride ?? movieTitle).trim() ? (titleOverride ?? movieTitle).trim() : undefined,
       // Optional root conditioning still — the route jail-resolves this abs uri and
       // renders segment 0 i2v (drops back to t2v when absent). Mirrors the clip surface.
       start_image: startImage ? startImage.uri : undefined,
@@ -1429,10 +1529,13 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
   // name. It is a warning, not a gate: the run row then offers "Generate without a
   // name", and `onGenerate(true)` skips the check. Cleared whenever a name is typed.
   const [nameWarn, setNameWarn] = useState(false);
+  // The warning is an EXPLICIT PER-SECTION OPTION now (operator 2026-08-13),
+  // DEFAULT OFF, remembered in localStorage — shared UnnamedWarn seam.
+  const [warnUnnamed, setWarnUnnamed] = useWarnUnnamedPref("cinema");
 
   const onGenerate = useCallback(
-    (skipNameCheck = false) => {
-      const built = buildBody();
+    (skipNameCheck = false, titleOverride?: string) => {
+      const built = buildBody(titleOverride);
       if ("err" in built) {
         setFormMsg(built.err);
         return;
@@ -1440,18 +1543,40 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
       // Warn ONCE per unnamed submit. An unnamed movie is still fully renderable — the
       // cost is later, in a sessions list where it reads as "untitled · 7f3a91c2…" and
       // nobody can tell which paused render is which.
-      if (!skipNameCheck && movieTitle.trim() === "") {
+      if (!skipNameCheck && warnUnnamed && movieTitle.trim() === "") {
         setNameWarn(true);
         setFormMsg(null);
         return;
       }
       setNameWarn(false);
       setFormMsg(null);
+      if (titleOverride && titleOverride.trim()) setMovieTitle(titleOverride.trim());
       job.run(built.body);
     },
-    [buildBody, job, movieTitle],
+    [buildBody, job, movieTitle, setMovieTitle, warnUnnamed],
   );
 
+  // SESSION LIBRARY PUSH (operator 2026-08-13): a finished movie's outputs
+  // land in the Session Library IMMEDIATELY — every segment clip plus the
+  // assembled movie (outputs[-1] when assembly produced one) — tagged
+  // generate_movie so the sidebar's Videos section lists them newest-first.
+  // addToLibrary de-dups by uri, so re-renders of the same result are no-ops.
+  useEffect(() => {
+    const r = job.result;
+    if (!r || r.ok === false || !r.outputs || r.outputs.length === 0) return;
+    for (const ref of r.outputs) {
+      if (!ref?.uri || !ref?.asset_id) continue;
+      addToLibrary(ref, "cinema", movieTitle.trim() || "movie", {
+        genKind: "generate_movie",
+        prompt: goals[0]?.prompt,
+        groupId: job.jobId ?? undefined,
+      });
+    }
+    // movieTitle/goals deliberately excluded: push once per RESULT, labelled
+    // with whatever the fields held when it landed.
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [job.result, job.jobId]);
+
   // ── derive playback refs from the terminal result ──
   // Outputs = [segment clips…, movie.mp4 LAST] (the runner appends the assembled
   // movie only when assembly produced one). Pair each segment ref with its manifest
@@ -1846,226 +1971,85 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
             if (e.target.value.trim() !== "") setNameWarn(false);
           }}
         />
+        <WarnUnnamedToggle checked={warnUnnamed} onChange={setWarnUnnamed} />
       </section>
 
-      {/* ── Movie start switcher (operator ask 2026-07-12, verbatim): "It needs to be
-             choosable by the user: any library clip, image, or scene to start — or
-             start fresh with a prompt to generate a clip. This static starting clip is
-             not going to work if it cannot be overridden." Same visual row as the Clip
-             surface's subject-clip switcher (.vi-studio-src-actions) — four honest ways
-             to set (or clear) segment 1's start, none of them a hidden default. ── */}
-      <section aria-label="Movie start" style={{ marginBottom: "0.6rem" }}>
-        <p className="vi-comfy-label" style={{ marginBottom: "0.25rem", opacity: 0.85 }}>
-          Movie start — segment 1 (optional)
-        </p>
-        <div className="vi-studio-src-actions" role="group" aria-label="Choose how the movie starts">
-          <button
-            type="button"
-            className="vi-btn vi-btn-sm"
-            disabled={disableSubmit || startBusy}
-            aria-expanded={showStartLibPicker}
-            onClick={() => setShowStartLibPicker((v) => !v)}
-          >
-            {showStartLibPicker ? "Hide library" : `🖼 From library (${startLibraryImages.length})`}
-          </button>
-          {/* CLIP-AS-START FINDING (stated in the file header above): the movie schema
-              has no `start_clip` field, and there is no synchronous "give me this
-              clip's last frame" endpoint — frame_extract is its own async job (enqueue
-              → poll → pick a frame), not a small addition here. Shown so the operator
-              SEES the option exists, disabled so nothing dishonest is implied. */}
-          <button
-            type="button"
-            className="vi-btn vi-btn-sm vi-btn-ghost"
-            disabled
-            title="Movie-from-clip needs the backend start_clip field — coming. A clip's last frame isn't a quick pick today (frame extraction is its own job); use a library image or upload a still instead."
-          >
-            🎬 From library (clip → last frame)
-          </button>
-          <label className="vi-btn vi-btn-sm vi-file-label">
-            {startBusy ? "Working…" : "⇪ Upload still"}
-            <input
-              type="file"
-              accept="image/*"
-              className="vi-file-input"
-              disabled={disableSubmit || startBusy}
-              onChange={(e) => {
-                const f = e.target.files?.[0];
-                if (f) void onPickStartImage(f);
-                e.target.value = "";
-              }}
-            />
-          </label>
-          <button
-            type="button"
-            className="vi-btn vi-btn-sm vi-btn-ghost"
-            disabled={disableSubmit || startBusy}
-            title="Clear the start still — segment 1 opens from its own prompt instead (text-to-video)."
-            onClick={() => {
-              setStartImage(null);
-              setShowStartLibPicker(false);
-            }}
+      {/* ── MOVIE SETTINGS, IN-COLUMN (operator 2026-08-13: "not seeing model,
+             fps, dimension inputs for cinema") — the geometry/model/sampler knobs
+             live in the left-sidebar Settings tab via portal, which is easy to
+             miss (and absent when no host mounts). This expander mirrors the
+             SAME state — two views, one truth, like the Clip card. ── */}
+      <PromptCardSettings label="movie settings — model · size · fps · steps · cfg · budget">
+        <div className="vi-knob vi-studio-knob-wide" style={{ minWidth: "16rem" }}>
+          <label htmlFor="vi-moviecol-model">model — whole movie</label>
+          <select
+            id="vi-moviecol-model"
+            className="vi-knob-select"
+            value={modelId}
+            disabled={disableSubmit}
+            onChange={(e) => setModelId(e.target.value)}
           >
-            ✦ Fresh prompt
-          </button>
+            <option value="">Auto — router picks by capability/budget</option>
+            {movieModels.length > 0 && (
+              <optgroup label="Bound by a ratified movie segment">
+                {movieModels.map((id) => (
+                  <option key={id} value={id}>{id}</option>
+                ))}
+              </optgroup>
+            )}
+            {(showAllModels || !renderLoaded || (modelId !== "" && !movieModels.includes(modelId))) && (
+              <optgroup label={renderLoaded ? "Other studio models — unverified for a movie segment" : "Studio models"}>
+                {STUDIO_MODEL_IDS.filter((id) => !movieModels.includes(id)).map((id) => (
+                  <option key={id} value={id}>{id}{renderLoaded ? " · unverified" : ""}</option>
+                ))}
+              </optgroup>
+            )}
+          </select>
         </div>
+        <div className="vi-knob" style={{ maxWidth: "7rem" }}>
+          <label htmlFor="vi-moviecol-width">width</label>
+          <input id="vi-moviecol-width" className="vi-knob-input" type="number" min={64} step={8}
+                 value={width} onChange={(e) => setWidth(Number(e.target.value) || 0)} />
+        </div>
+        <div className="vi-knob" style={{ maxWidth: "7rem" }}>
+          <label htmlFor="vi-moviecol-height">height</label>
+          <input id="vi-moviecol-height" className="vi-knob-input" type="number" min={64} step={8}
+                 value={height} onChange={(e) => setHeight(Number(e.target.value) || 0)} />
+        </div>
+        <div className="vi-knob" style={{ maxWidth: "6rem" }}>
+          <label htmlFor="vi-moviecol-fps">fps</label>
+          <input id="vi-moviecol-fps" className="vi-knob-input" type="number" min={1}
+                 value={fps} onChange={(e) => setFps(Number(e.target.value) || 0)} />
+        </div>
+        <div className="vi-knob" style={{ maxWidth: "12rem" }}>
+          <label htmlFor="vi-moviecol-frames">length — frames / segment</label>
+          <input id="vi-moviecol-frames" className="vi-knob-input" type="number" min={1}
+                 value={framesDefault} placeholder="model default (81)"
+                 onChange={(e) => setFramesDefault(e.target.value)} />
+          <span className="vi-knob-hint">
+            {framesDefault.trim() !== "" && Number(framesDefault) > 0 && fps > 0
+              ? `every segment \u2248 ${(Number(framesDefault) / fps).toFixed(1)}s @ ${fps}fps (a segment\u2019s own length overrides)`
+              : `blank = model default \u2014 81f \u2248 ${fps > 0 ? (81 / fps).toFixed(1) : "?"}s @ ${fps}fps per segment`}
+          </span>
+        </div>
+        <div className="vi-knob" style={{ maxWidth: "7rem" }}>
+          <label htmlFor="vi-moviecol-steps">steps</label>
+          <input id="vi-moviecol-steps" className="vi-knob-input" type="number" min={1} max={100}
+                 value={steps} placeholder="model default" onChange={(e) => setSteps(e.target.value)} />
+        </div>
+        <div className="vi-knob" style={{ maxWidth: "7rem" }}>
+          <label htmlFor="vi-moviecol-cfg">cfg</label>
+          <input id="vi-moviecol-cfg" className="vi-knob-input" type="number" min={0} max={20} step={0.5}
+                 value={cfg} placeholder="model default" onChange={(e) => setCfg(e.target.value)} />
+        </div>
+        <div className="vi-knob" style={{ maxWidth: "9rem" }}>
+          <label htmlFor="vi-moviecol-vram">VRAM budget GB</label>
+          <input id="vi-moviecol-vram" className="vi-knob-input" type="number" min={1}
+                 value={vramBudget} placeholder="autofit" onChange={(e) => setVramBudget(e.target.value)} />
+        </div>
+      </PromptCardSettings>
 
-        {/* Current start state — ALWAYS visible, so the picked (or default) start is
-            never implicit (the operator's exact complaint about the old static slot). */}
-        {startImage ? (
-          <div
-            style={{ display: "flex", alignItems: "center", gap: "0.6rem", flexWrap: "wrap", marginTop: "0.5rem" }}
-          >
-            <img
-              src={mediaBytesUrl(startImage.uri)}
-              alt="segment 1 start image"
-              style={{
-                width: "5rem",
-                height: "5rem",
-                objectFit: "cover",
-                borderRadius: "0.4rem",
-                background: "#000",
-                border: "1px solid var(--vi-border)",
-              }}
-            />
-            <span className="vi-comfy-hint" role="note" style={{ margin: 0 }}>
-              Segment 1 renders image-to-video from this still.
-            </span>
-            <button
-              type="button"
-              className="vi-btn vi-btn-sm vi-btn-ghost"
-              disabled={disableSubmit || startBusy}
-              onClick={() => setStartImage(null)}
-            >
-              ✕ Remove
-            </button>
-          </div>
-        ) : (
-          <p className="vi-comfy-hint" role="note" style={{ marginTop: "0.5rem" }}>
-            No start still set — opens from your first segment&apos;s prompt (text-to-video
-            opening).
-          </p>
-        )}
-
-        {startError && (
-          <p className="vi-error" role="alert" style={{ marginTop: "0.3rem" }}>
-            {startError}
-          </p>
-        )}
-
-        {showStartLibPicker && (
-          <LibraryImageGrid
-            images={startLibraryImages}
-            onPick={(ref) => {
-              setStartImage(ref);
-              setShowStartLibPicker(false);
-            }}
-          />
-        )}
-      </section>
 
-      {/* ── IDENTITY reference(s): the locked subject carried across scene changes ── */}
-      <section aria-label="Identity reference" style={{ marginBottom: "0.6rem" }}>
-        <p className="vi-comfy-label" style={{ marginBottom: "0.25rem", opacity: 0.85 }}>
-          Identity reference — carry a subject across scenes (optional)
-        </p>
-        {/* The reference thumbnails + upload AND the identity-profile affordances read as
-            ONE input block — matching how the Clip Generate tab embeds the same
-            <IdentityProfileControls> inline, directly beneath its references
-            (StudioGenerateTab.renderReferenceSlots), rather than letting it stand off as a
-            separate island row. Presentation/placement only: the props + wiring below are
-            byte-identical to before. */}
-        <div>
-          <div style={{ display: "flex", alignItems: "center", gap: "0.6rem", flexWrap: "wrap" }}>
-            {references.map((r, i) => (
-              <span key={r.uri} style={{ position: "relative", display: "inline-flex" }}>
-                <img
-                  src={mediaBytesUrl(r.uri)}
-                  alt={`identity reference ${i + 1}`}
-                  style={{
-                    width: "4rem",
-                    height: "4rem",
-                    objectFit: "cover",
-                    borderRadius: "0.4rem",
-                    background: "#000",
-                    border: "1px solid var(--vi-border)",
-                  }}
-                />
-                <button
-                  type="button"
-                  className="vi-btn vi-btn-sm vi-btn-ghost"
-                  disabled={disableSubmit || refBusy}
-                  title="Remove this reference"
-                  onClick={() => {
-                    setReferences((prev) => prev.filter((x) => x.uri !== r.uri));
-                    setSelectedProfile(null); // editing the set = an UNSAVED identity
-                  }}
-                  style={{ position: "absolute", top: -6, right: -6, padding: "0 0.3rem", lineHeight: 1.2 }}
-                >
-                  ✕
-                </button>
-              </span>
-            ))}
-            {references.length < REF_MAX && (
-              <label className="vi-btn vi-btn-sm vi-btn-ghost vi-file-label">
-                {refBusy ? "Working…" : references.length ? "＋ Add reference" : "Upload a reference"}
-                <input
-                  type="file"
-                  accept="image/*"
-                  className="vi-file-input"
-                  disabled={disableSubmit || refBusy}
-                  onChange={(e) => {
-                    const f = e.target.files?.[0];
-                    if (f) void onPickReferenceImage(f);
-                    e.target.value = "";
-                  }}
-                />
-              </label>
-            )}
-          </div>
-          {/* IDENTITY PROFILES (stage a): pick a saved identity profile or save the current
-              references — the durable form of this identity. One additive row, embedded
-              inline within the reference input block (matching the Clip Generate tab). */}
-          <IdentityProfileControls
-            refCount={references.length}
-            currentRefUris={references.map((r) => r.uri)}
-            selectedProfile={selectedProfile}
-            onAttach={attachIdentityRefs}
-            onSelectProfile={setSelectedProfile}
-            maxRefs={REF_MAX}
-            disabled={disableSubmit || refBusy}
-          />
-        </div>
-        {/* RESOLUTION FOOTGUN (operator 2026-07-12): the only budget-fitting id-capable
-            model (wan2.1-vace-1.3b) tops out at 832×480 — an identity movie above that fails
-            no_capable_model. Warn LOUDLY with a one-click snap; never BLOCK (the backend's
-            rejection is honest if the operator insists on a bigger box). */}
-        {(references.length > 0 || selectedProfile != null) && (width > 832 || height > 480) && (
-          <p className="vi-input-callout" role="note" style={{ marginTop: "0.4rem" }}>
-            id-capable models on this fleet top out at 832×480 — this movie asks for {width}×{height}.{" "}
-            <button
-              type="button"
-              className="vi-btn vi-btn-sm vi-btn-accent"
-              onClick={() => {
-                setWidth(480);
-                setHeight(480);
-              }}
-              style={{ marginLeft: "0.3rem" }}
-            >
-              Set 480×480
-            </button>
-          </p>
-        )}
-        <span className="vi-comfy-hint" role="note" style={{ display: "block", marginTop: "0.3rem" }}>
-          {references.length
-            ? `Identity movie — every scene renders id_lock (Wan-VACE) so this subject carries across every cut. ${references.length}/${REF_MAX} reference(s). Needs a VACE-capable GPU worker; on a GPU-less box it reports deps-missing.`
-            : "Leave empty for a plain movie. Add a locked character/subject still to carry the same identity across every scene (needs a VACE-capable GPU worker)."}
-        </span>
-        {refError && (
-          <p className="vi-error" role="alert" style={{ marginTop: "0.3rem" }}>
-            {refError}
-          </p>
-        )}
-      </section>
 
       {/* ── the goal timeline (NLE-row order, top → bottom) ──
              Operator addendum 2026-07-12: each segment is now a DISTINCTLY CONTRASTED
@@ -2356,6 +2340,241 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
               />
               </div>
 
+            {/* SEGMENT-1 INPUTS, IN-CARD (operator 2026-08-13): the movie-start
+                still and the identity reference moved INTO the prompt component —
+                the same per-part attach idiom Scene uses — decluttering the column.
+                Wiring is byte-identical to the old standalone sections. */}
+            {i === 0 && (
+              <>
+        {/* ── Movie start switcher (operator ask 2026-07-12, verbatim): "It needs to be
+               choosable by the user: any library clip, image, or scene to start — or
+               start fresh with a prompt to generate a clip. This static starting clip is
+               not going to work if it cannot be overridden." Same visual row as the Clip
+               surface's subject-clip switcher (.vi-studio-src-actions) — four honest ways
+               to set (or clear) segment 1's start, none of them a hidden default. ── */}
+        <section aria-label="Movie start" style={{ marginBottom: "0.6rem" }}>
+          <p
+            className="vi-comfy-label"
+            style={{ marginBottom: "0.25rem", opacity: 0.85, cursor: "help" }}
+            title="How segment 1 opens: pick a library image, use a library clip's last frame, upload a still, or leave unset to open from your first segment's prompt (text-to-video). Never a hidden default — whatever is set is shown here."
+          >
+            Movie start — segment 1 (optional) ⓘ
+          </p>
+          <div className="vi-studio-src-actions" role="group" aria-label="Choose how the movie starts">
+            <button
+              type="button"
+              className="vi-btn vi-btn-sm"
+              disabled={disableSubmit || startBusy}
+              aria-expanded={showStartLibPicker}
+              onClick={() => setShowStartLibPicker((v) => !v)}
+            >
+              {showStartLibPicker ? "Hide library" : `🖼 From library (${startLibraryImages.length})`}
+            </button>
+            {/* CLIP-AS-START FINDING (stated in the file header above): the movie schema
+                has no `start_clip` field, and there is no synchronous "give me this
+                clip's last frame" endpoint — frame_extract is its own async job (enqueue
+                → poll → pick a frame), not a small addition here. Shown so the operator
+                SEES the option exists, disabled so nothing dishonest is implied. */}
+            <button
+              type="button"
+              className="vi-btn vi-btn-sm vi-btn-ghost"
+              disabled
+              title="Movie-from-clip needs the backend start_clip field — coming. A clip's last frame isn't a quick pick today (frame extraction is its own job); use a library image or upload a still instead."
+            >
+              🎬 From library (clip → last frame)
+            </button>
+            <label className="vi-btn vi-btn-sm vi-file-label">
+              {startBusy ? "Working…" : "⇪ Upload still"}
+              <input
+                type="file"
+                accept="image/*"
+                className="vi-file-input"
+                disabled={disableSubmit || startBusy}
+                onChange={(e) => {
+                  const f = e.target.files?.[0];
+                  if (f) void onPickStartImage(f);
+                  e.target.value = "";
+                }}
+              />
+            </label>
+            <button
+              type="button"
+              className="vi-btn vi-btn-sm vi-btn-ghost"
+              disabled={disableSubmit || startBusy}
+              title="Clear the start still — segment 1 opens from its own prompt instead (text-to-video)."
+              onClick={() => {
+                setStartImage(null);
+                setShowStartLibPicker(false);
+              }}
+            >
+              ✦ Fresh prompt
+            </button>
+          </div>
+
+          {/* Current start state — ALWAYS visible, so the picked (or default) start is
+              never implicit (the operator's exact complaint about the old static slot). */}
+          {startImage ? (
+            <div
+              style={{ display: "flex", alignItems: "center", gap: "0.6rem", flexWrap: "wrap", marginTop: "0.5rem" }}
+            >
+              <img
+                src={mediaBytesUrl(startImage.uri)}
+                alt="segment 1 start image"
+                style={{
+                  width: "5rem",
+                  height: "5rem",
+                  objectFit: "cover",
+                  borderRadius: "0.4rem",
+                  background: "#000",
+                  border: "1px solid var(--vi-border)",
+                }}
+              />
+              <span className="vi-comfy-hint" role="note" style={{ margin: 0 }}>
+                Segment 1 renders image-to-video from this still.
+              </span>
+              <button
+                type="button"
+                className="vi-btn vi-btn-sm vi-btn-ghost"
+                disabled={disableSubmit || startBusy}
+                onClick={() => setStartImage(null)}
+              >
+                ✕ Remove
+              </button>
+            </div>
+          ) : (
+            <p className="vi-comfy-hint" role="note" style={{ marginTop: "0.5rem" }}>
+              No start still — text-to-video opening.
+            </p>
+          )}
+
+          {startError && (
+            <p className="vi-error" role="alert" style={{ marginTop: "0.3rem" }}>
+              {startError}
+            </p>
+          )}
+
+          {showStartLibPicker && (
+            <LibraryImageGrid
+              images={startLibraryImages}
+              onPick={(ref) => {
+                setStartImage(ref);
+                setShowStartLibPicker(false);
+              }}
+            />
+          )}
+        </section>
+        {/* ── IDENTITY reference(s): the locked subject carried across scene changes ── */}
+        <section aria-label="Identity reference" style={{ marginBottom: "0.6rem" }}>
+          <p
+            className="vi-comfy-label"
+            style={{ marginBottom: "0.25rem", opacity: 0.85, cursor: "help" }}
+            title="Leave empty for a plain movie. Add a locked character/subject still (or pick a saved identity profile) and EVERY scene renders id_lock (Wan-VACE) so the same identity carries across every cut. Needs a VACE-capable GPU worker; a GPU-less box reports deps-missing."
+          >
+            Identity reference — carry a subject across scenes (optional) ⓘ
+          </p>
+          {/* The reference thumbnails + upload AND the identity-profile affordances read as
+              ONE input block — matching how the Clip Generate tab embeds the same
+              <IdentityProfileControls> inline, directly beneath its references
+              (StudioGenerateTab.renderReferenceSlots), rather than letting it stand off as a
+              separate island row. Presentation/placement only: the props + wiring below are
+              byte-identical to before. */}
+          {/* ONE ROW (operator 2026-08-13): thumbnails, upload AND the profile
+              controls share a single wrapping flex row — no stacked button rows. */}
+          <div style={{ display: "flex", alignItems: "center", gap: "0.6rem", flexWrap: "wrap" }}>
+            <div style={{ display: "contents" }}>
+              {references.map((r, i) => (
+                <span key={r.uri} style={{ position: "relative", display: "inline-flex" }}>
+                  <img
+                    src={mediaBytesUrl(r.uri)}
+                    alt={`identity reference ${i + 1}`}
+                    style={{
+                      width: "4rem",
+                      height: "4rem",
+                      objectFit: "cover",
+                      borderRadius: "0.4rem",
+                      background: "#000",
+                      border: "1px solid var(--vi-border)",
+                    }}
+                  />
+                  <button
+                    type="button"
+                    className="vi-btn vi-btn-sm vi-btn-ghost"
+                    disabled={disableSubmit || refBusy}
+                    title="Remove this reference"
+                    onClick={() => {
+                      setReferences((prev) => prev.filter((x) => x.uri !== r.uri));
+                      setSelectedProfile(null); // editing the set = an UNSAVED identity
+                    }}
+                    style={{ position: "absolute", top: -6, right: -6, padding: "0 0.3rem", lineHeight: 1.2 }}
+                  >
+                    ✕
+                  </button>
+                </span>
+              ))}
+              {references.length < REF_MAX && (
+                <label className="vi-btn vi-btn-sm vi-btn-ghost vi-file-label">
+                  {refBusy ? "Working…" : references.length ? "＋ Add reference" : "Upload a reference"}
+                  <input
+                    type="file"
+                    accept="image/*"
+                    className="vi-file-input"
+                    disabled={disableSubmit || refBusy}
+                    onChange={(e) => {
+                      const f = e.target.files?.[0];
+                      if (f) void onPickReferenceImage(f);
+                      e.target.value = "";
+                    }}
+                  />
+                </label>
+              )}
+            </div>
+            {/* IDENTITY PROFILES (stage a): pick a saved identity profile or save the current
+                references — the durable form of this identity. One additive row, embedded
+                inline within the reference input block (matching the Clip Generate tab). */}
+            <IdentityProfileControls
+              refCount={references.length}
+              currentRefUris={references.map((r) => r.uri)}
+              selectedProfile={selectedProfile}
+              onAttach={attachIdentityRefs}
+              onSelectProfile={setSelectedProfile}
+              maxRefs={REF_MAX}
+              disabled={disableSubmit || refBusy}
+            />
+          </div>
+          {/* RESOLUTION FOOTGUN (operator 2026-07-12): the only budget-fitting id-capable
+              model (wan2.1-vace-1.3b) tops out at 832×480 — an identity movie above that fails
+              no_capable_model. Warn LOUDLY with a one-click snap; never BLOCK (the backend's
+              rejection is honest if the operator insists on a bigger box). */}
+          {(references.length > 0 || selectedProfile != null) && (width > 832 || height > 480) && (
+            <p className="vi-input-callout" role="note" style={{ marginTop: "0.4rem" }}>
+              id-capable models on this fleet top out at 832×480 — this movie asks for {width}×{height}.{" "}
+              <button
+                type="button"
+                className="vi-btn vi-btn-sm vi-btn-accent"
+                onClick={() => {
+                  setWidth(480);
+                  setHeight(480);
+                }}
+                style={{ marginLeft: "0.3rem" }}
+              >
+                Set 480×480
+              </button>
+            </p>
+          )}
+          {references.length > 0 && (
+            <span className="vi-comfy-hint" role="note" style={{ display: "block", marginTop: "0.3rem" }}
+                  title="Every scene renders id_lock (Wan-VACE) so this subject carries across every cut. Needs a VACE-capable GPU worker.">
+              Identity movie — id_lock every scene · {references.length}/{REF_MAX} reference(s)
+            </span>
+          )}
+          {refError && (
+            <p className="vi-error" role="alert" style={{ marginTop: "0.3rem" }}>
+              {refError}
+            </p>
+          )}
+        </section>
+              </>
+            )}
             {/* k88 — the shared settings expander (collapsed by default); the joint
                 DESCRIPTOR stays visible in the card header above.
                 2026-08-12: grew from seed-only to the FULL per-segment knob set the
@@ -2367,8 +2586,8 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
             <PromptCardSettings
               label={
                 i === 0
-                  ? "settings — seed · model · steps · cfg"
-                  : "settings — seed · model · steps · cfg · joint · branch"
+                  ? "settings — seed · model · steps · cfg · length"
+                  : "settings — seed · model · steps · cfg · length · joint · branch"
               }
             >
               <div className="vi-knob" style={{ maxWidth: "10rem" }}>
@@ -2448,6 +2667,58 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
                 />
                 <span className="vi-knob-hint">0–20 · blank = movie/model default</span>
               </div>
+              <div className="vi-knob" style={{ maxWidth: "11rem" }}>
+                <label htmlFor={`vi-movie-goalframes-${g.key}`}>length — frames</label>
+                <input
+                  id={`vi-movie-goalframes-${g.key}`}
+                  className="vi-knob-input"
+                  type="number"
+                  min={1}
+                  value={g.frames}
+                  placeholder={framesDefault.trim() || "model default"}
+                  onChange={(e) => setGoalField(i, { frames: e.target.value })}
+                />
+                <span className="vi-knob-hint">
+                  {g.frames.trim() !== "" && Number(g.frames) > 0 && fps > 0
+                    ? `\u2248 ${(Number(g.frames) / fps).toFixed(1)}s @ ${fps}fps`
+                    : "blank = movie default \u00b7 clamped to the model ceiling, snapped to 4k+1"}
+                </span>
+                {Number(g.frames.trim() || framesDefault.trim()) > MODEL_FRAME_CEILING && (
+                  <button
+                    type="button"
+                    className="vi-btn vi-btn-sm"
+                    title="The model ceiling is 81 frames — materialize motion-carried continuation segments (vace_extend) so the movie honestly reaches this length. They appear in the timeline, fully editable."
+                    onClick={() => splitGoalToFit(i)}
+                  >
+                    ✂ split into {Math.ceil(Number(g.frames.trim() || framesDefault.trim()) / MODEL_FRAME_CEILING)} segments to fit
+                  </button>
+                )}
+              </div>
+              {/* UNIFORM-CARD PLACEHOLDERS (operator 2026-08-13): geometry and
+                  budget are MOVIE-level by server contract (one StudioMovieSpec
+                  geometry per movie) — shown greyed with the reason, never
+                  hidden, so every prompt card reads identically. */}
+              <div className="vi-knob" style={{ maxWidth: "7rem" }}>
+                <label htmlFor={`vi-movie-goalw-${g.key}`}>width</label>
+                <input id={`vi-movie-goalw-${g.key}`} className="vi-knob-input" disabled value={width}
+                       title="Movie-level — one geometry per movie (StudioMovieSpec); change it in movie settings above." />
+              </div>
+              <div className="vi-knob" style={{ maxWidth: "7rem" }}>
+                <label htmlFor={`vi-movie-goalh-${g.key}`}>height</label>
+                <input id={`vi-movie-goalh-${g.key}`} className="vi-knob-input" disabled value={height}
+                       title="Movie-level — one geometry per movie (StudioMovieSpec); change it in movie settings above." />
+              </div>
+              <div className="vi-knob" style={{ maxWidth: "6rem" }}>
+                <label htmlFor={`vi-movie-goalfps-${g.key}`}>fps</label>
+                <input id={`vi-movie-goalfps-${g.key}`} className="vi-knob-input" disabled value={fps}
+                       title="Movie-level — one fps per movie; change it in movie settings above." />
+              </div>
+              <div className="vi-knob" style={{ maxWidth: "9rem" }}>
+                <label htmlFor={`vi-movie-goalvram-${g.key}`}>VRAM budget GB</label>
+                <input id={`vi-movie-goalvram-${g.key}`} className="vi-knob-input" disabled
+                       value={vramBudget.trim() || "autofit"}
+                       title="Movie-level — set in movie settings above (blank = autofit to the serving worker's free VRAM)." />
+              </div>
               {i > 0 && (
                 <>
                   <div className="vi-knob" style={{ maxWidth: "13rem" }}>
@@ -2528,52 +2799,6 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
         </div>
       </section>
 
-      {/* ── movie-wide NEGATIVE prompt — the DEFAULT every segment inherits (a per-segment
-             negative above overrides it for that segment only). Kept in the center column,
-             beside the goals, as a wide prompt textarea — the geometry / model pin / sampler
-             knobs now live in the left-sidebar Settings tab (see settingsNode above),
-             mirroring the Clip surface which keeps its negative out of the knob grid.
-             k88: deliberately NOT restyled with PromptFieldTabs — this block is a lone
-             negative with no prompt pane to pair it with, so tabs would be one dead tab. ── */}
-      <section aria-label="Movie negative prompt" className="vi-studio-knobs" style={{ marginTop: "0.8rem" }}>
-        <div className="vi-studio-knob-wide" style={{ flex: "1 1 100%" }}>
-          <GoalComposer
-            id="vi-movie-negative"
-            label="negative prompt (movie-wide default, optional)"
-            value={negative}
-            placeholder="what to avoid across every segment — each segment can override above"
-            onChange={setNegative}
-            hint="The default for every segment. A segment's own negative prompt overrides this for that segment."
-          />
-          {/* The same §1b affordance for the movie-wide default — subject is the movie
-              idea if you wrote one, else the opening shot's prompt. */}
-          <div className="vi-prompt-assist">
-            <span className="vi-prompt-assist-lead" aria-hidden="true">
-              negative assist
-            </span>
-            <button
-              type="button"
-              className="vi-btn vi-btn-sm vi-btn-ghost"
-              disabled={assist.assistBusy != null || disableSubmit}
-              title="Write an artifact/quality exclusion list for the whole movie (replaces this field)."
-              onClick={() =>
-                void assist.runNegative(
-                  {
-                    subject: movieQuery.trim() || goals[0]?.prompt || "",
-                    draft: negative,
-                    extra: {
-                      context: identityContext ? { identity_profile: identityContext } : {},
-                    },
-                  },
-                  setNegative,
-                )
-              }
-            >
-              {assist.assistBusy === "negative" ? "✨ Writing…" : "✨ Generate negative"}
-            </button>
-          </div>
-        </div>
-      </section>
 
       {/* ── SYNTHETIC-TIER HONESTY BANNER — loud, right above Generate movie, so the
              operator sees it at the moment it matters instead of the knob-grid
@@ -2620,31 +2845,14 @@ export function StudioMovieComposer({ settingsHost }: StudioMovieComposerProps =
       {/* ── UNNAMED-MOVIE NUDGE — a warning with a way through, never a gate. Sits
              right above Generate because that is where the decision is made. ── */}
       {nameWarn && (
-        <div
-          className="vi-studio-required-callout"
-          role="alert"
-          style={{
-            display: "flex",
-            alignItems: "center",
-            gap: "0.6rem",
-            flexWrap: "wrap",
-            marginBottom: "0.6rem",
+        <UnnamedWarnPanel
+          noun="movie"
+          onProceed={(name) => {
+            setNameWarn(false);
+            onGenerate(true, name);
           }}
-        >
-          <span>
-            This movie has no name — it will list as “untitled” beside a bare id in
-            Cinema sessions, which is hard to tell apart from every other unnamed render
-            when you come back to resume one.
-          </span>
-          <button
-            type="button"
-            className="vi-btn vi-btn-sm vi-btn-ghost"
-            disabled={disableSubmit}
-            onClick={() => onGenerate(true)}
-          >
-            Generate without a name
-          </button>
-        </div>
+          onDismiss={() => setNameWarn(false)}
+        />
       )}
 
       {/* ── run row ── */}