@hugpy/video-intelligence-ui 0.1.1 → 0.3.0

package/src/stations/studio/IdentitiesStation.tsx

@@ -2,11 +2,17 @@
 // (studio stage (a)'s durable library item), separate from the compact "👤 From
 // profile" picker the Studio Generate/Movie surfaces embed. Those surfaces only
 // ATTACH a saved identity to a render; this station is where one is actually
-// built and maintained:
-//   • a CREATE panel (name + notes + reference images, upload or library-pick)
-//   • a GRID of existing profiles (thumbnail strip, name, created date, notes
-//     preview, ref count), each with inline EDIT (rename display-only + notes +
-//     add/remove references) and ARCHIVE.
+// built and maintained.
+//
+// k94 — ONE PATH (the clownworld MO): the top of the tab is IdentityCreatePanel —
+// name + [+ video] [+ images] + ONE "Create identity" button + a progress line.
+// A video (or a few photos) in; a bindable character (reference views + 3D GLB)
+// out. Everything else this tab used to expose as separate steps (the manual
+// create panel, char360 knobs, pose / vision model / frames / fps / size,
+// turnaround prompt, seed, regeneration prompt, the per-step generate / approve /
+// new-version buttons) lives under ONE collapsed `> advanced` expander, still
+// functional, default-hidden. The profile list + viewer (canonical views,
+// turntable, GLB) stay, with a ✓ when an identity is ready to bind.
 //
 // Backed entirely by useIdentityProfiles (list/create/update/remove) — this file
 // adds no new wire contract, just the surface. The reference-image upload path
@@ -28,13 +34,13 @@ import * as THREE from "three";
 import { OrbitControls } from "three/examples/jsm/controls/OrbitControls.js";
 import { GLTFLoader } from "three/examples/jsm/loaders/GLTFLoader.js";
 import { request, okValue, errorOf, describeAppError } from "../../transport/client";
-import { hugpyConfig, mediaBytesUrl, identityGenerateUrl, identityMeshStatusUrl, videoJobUrl } from "../../config";
-import { addToLibrary, useMediaLibrary } from "../../video/mediaLibrary";
+import { mediaBytesUrl, identityGenerateUrl, identityMeshStatusUrl, videoJobUrl } from "../../config";
+import { useMediaLibrary } from "../../video/mediaLibrary";
 import type { LibraryItem } from "../../video/mediaLibrary";
-import { mediaRefSchema, uploadResultSchema } from "../../video/contract";
 import type { MediaRef, IdentityReconstruction, IdentityAngleView } from "../../video/contract";
-import { getSessionId } from "../../session";
 import { LibraryImageGrid } from "./studioShared";
+import { uploadAndIngestImage, uploadAndIngestVideo } from "./identityUpload";
+import { IdentityCreatePanel, AdvancedExpander } from "./IdentityCreatePanel";
 import { useIdentityProfiles } from "./useIdentityProfiles";
 import type {
   IdentityProfile,
@@ -61,71 +67,22 @@ const MAX_SOURCE_IMAGES = 12;
 
 type UploadPhase = "idle" | "uploading" | "ingesting";
 
-/** Upload → ingest → guard kind=image — the same two-hop path every other studio
- * surface's image upload runs (StudioGenerateTab.onPickImageFile / StudioMovie-
- * Composer's mirror of it), lifted to a standalone helper since this station
- * has no single host reference-slot list to route into (create panel + N cards
- * each keep their own draft ref list). Joins the session library on success,
- * exactly like the host surfaces do, so an image picked here is pickable from
- * the library everywhere else too. */
-async function uploadAndIngestImage(file: File): Promise<{ ref?: MediaRef; error?: string }> {
-  const fd = new FormData();
-  fd.append("file", file);
-  fd.append("sid", getSessionId());
-  const up = await request<unknown>(hugpyConfig.uploadUrl, {
-    method: "POST",
-    body: fd,
-    meta: { specKey: "studio", operation: "identity.station.upload" },
-  });
-  if (!up.ok) return { error: describeAppError(errorOf(up)) };
-  const upParsed = uploadResultSchema.safeParse(okValue(up));
-  if (!upParsed.success) return { error: "Malformed upload response." };
-  const ing = await request<unknown>(hugpyConfig.videoIngestUrl, {
-    method: "POST",
-    body: JSON.stringify({ path: upParsed.data.path }),
-    headers: { "Content-Type": "application/json" },
-    meta: { specKey: "studio", operation: "identity.station.ingest" },
-  });
-  if (!ing.ok) return { error: describeAppError(errorOf(ing)) };
-  const media = mediaRefSchema.safeParse(okValue(ing));
-  if (!media.success) return { error: "Malformed ingest response." };
-  if (media.data.kind !== "image") {
-    return { error: `Ingested asset is "${media.data.kind}", not an image — pick a still.` };
-  }
-  addToLibrary(media.data, "upload", media.data.mime);
-  return { ref: media.data };
-}
-
-/** char360 S4 — the same upload→ingest chain as `uploadAndIngestImage`, guarding
- * the INVERSE kind: the "Extract from video" control's source MUST be a video,
- * not a still. Kept as a sibling helper (not a shared parameterized one) so
- * neither existing call site's behavior/wording changes — extend, don't touch. */
-async function uploadAndIngestVideo(file: File): Promise<{ ref?: MediaRef; error?: string }> {
-  const fd = new FormData();
-  fd.append("file", file);
-  fd.append("sid", getSessionId());
-  const up = await request<unknown>(hugpyConfig.uploadUrl, {
-    method: "POST",
-    body: fd,
-    meta: { specKey: "studio", operation: "identity.video_extract.upload" },
+// uploadAndIngestImage / uploadAndIngestVideo moved to ./identityUpload (k94) so the
+// one-path create panel shares them without a circular import — same behavior.
+
+/** "Ready to bind" (k94): views (a canonical set, or any reconstruction with views)
+ *  AND a finished mesh (some reconstruction's mesh.status === "done" + glb_path).
+ *  Reads the same snake_case wire truth the detail's viewers gate on. */
+export function identityReadiness(p: IdentityProfile): { views: boolean; mesh: boolean; ready: boolean } {
+  const recons = p.reconstructions ?? [];
+  const views =
+    (p.canonical ?? []).length > 0 ||
+    recons.some((r) => Array.isArray(r.views) && (r.views as unknown[]).length > 0);
+  const mesh = recons.some((r) => {
+    const m = (r as { mesh?: { status?: string; glb_path?: string | null } | null }).mesh;
+    return !!m && m.status === "done" && !!m.glb_path;
   });
-  if (!up.ok) return { error: describeAppError(errorOf(up)) };
-  const upParsed = uploadResultSchema.safeParse(okValue(up));
-  if (!upParsed.success) return { error: "Malformed upload response." };
-  const ing = await request<unknown>(hugpyConfig.videoIngestUrl, {
-    method: "POST",
-    body: JSON.stringify({ path: upParsed.data.path }),
-    headers: { "Content-Type": "application/json" },
-    meta: { specKey: "studio", operation: "identity.video_extract.ingest" },
-  });
-  if (!ing.ok) return { error: describeAppError(errorOf(ing)) };
-  const media = mediaRefSchema.safeParse(okValue(ing));
-  if (!media.success) return { error: "Malformed ingest response." };
-  if (media.data.kind !== "video") {
-    return { error: `Ingested asset is "${media.data.kind}", not a video — pick a video file.` };
-  }
-  addToLibrary(media.data, "upload", media.data.mime);
-  return { ref: media.data };
+  return { views, mesh, ready: views && mesh };
 }
 
 function fmtCreated(t: number | null | undefined): { short: string; full: string } {
@@ -388,20 +345,18 @@ function AngleRingEditor({ reconstruction, api }: { reconstruction: IdentityReco
           value={regeneratePrompt}
           onChange={(e) => setRegeneratePrompt(e.target.value)}
         />
-        <span className="vi-comfy-hint" style={{ opacity: 0.7 }}>
-          Used when regenerating specific rejected angles below. Conditioned on nearest approved neighbors.
-        </span>
       </label>
 
       <div style={{ display: "flex", flexWrap: "wrap", gap: "0.45rem", maxHeight: "28rem", overflowY: "auto", paddingRight: "0.5rem" }}>
         {views.map((view: any) => (
-          <AngleRingTile
-            key={view.view_id} // <--- snake_case
-            view={view}
-            reconId={reconstruction.recon_id} // <--- snake_case
-            api={api}
-            promptOverride={regeneratePrompt}
-          />
+          <div key={view.view_id} style={{ display: "contents" }}>
+            <AngleRingTile
+              view={view}
+              reconId={reconstruction.recon_id} // <--- snake_case
+              api={api}
+              promptOverride={regeneratePrompt}
+            />
+          </div>
         ))}
       </div>
     </div>
@@ -481,7 +436,7 @@ function ReferenceEditor({
                 if (files && files.length > 0) {
                   void (async () => {
                     // Convert FileList to Array and enforce space limits
-                    const fileArray = Array.from(files);
+                    const fileArray = Array.from(files as ArrayLike<File>);
                     const remainingSlots = MAX_SOURCE_IMAGES - refs.length;
                     
                     if (fileArray.length > remainingSlots) {
@@ -669,11 +624,6 @@ export function ExtractFromVideoPanel({
     <section className="vi-comfy-bar" aria-label="Extract character views from a video" style={{ marginTop: "0.8rem" }}>
       <div style={{ flexBasis: "100%" }}>
         <span className="vi-comfy-label">Extract character views (char360)</span>
-        <p className="vi-comfy-hint" style={{ margin: "0.15rem 0 0.4rem" }}>
-          {usingPreset
-            ? "Run the char360 view-set extraction over the loaded video to mint a new identity profile (or add views to an existing one)."
-            : "Run the char360 view-set extraction over a video to mint a new identity profile (or add views to an existing one) — no curl required."}
-        </p>
       </div>
 
       <div style={{ display: "flex", flexWrap: "wrap", gap: "0.6rem", alignItems: "center" }}>
@@ -1100,9 +1050,6 @@ function SettingsPanel({
           <option value="none">None</option>
           <option value="t-pose">T-pose</option>
         </select>
-        <span className="vi-comfy-hint" style={{ opacity: 0.7 }}>
-          T-pose normalizes the character&apos;s stance before meshing — clears occlusion ambiguity.
-        </span>
       </label>
 
       <label className="vi-comfy-field" style={{ flexBasis: "20rem" }}>
@@ -1143,10 +1090,6 @@ function SettingsPanel({
             ↻
           </button>
         </span>
-        <span className="vi-comfy-hint" style={{ opacity: 0.7 }}>
-          The vision-language model used to pick the full-body reference before meshing.
-          Auto prefers a 7B when installed (falls back to the 3B); pick a model to pin one.
-        </span>
         {visionError && <span className="vi-error">{visionError}</span>}
       </label>
 
@@ -1169,10 +1112,6 @@ function SettingsPanel({
           placeholder="e.g. no object on back, no logos, clean bare back"
           onChange={(e) => setDraft((d) => ({ ...d, cleanup_prompt: e.target.value }))}
         />
-        <span className="vi-comfy-hint" style={{ opacity: 0.7 }}>
-          A positive-worded instruction woven into the T-pose front render so the meshed
-          front never carries an unwanted prop/symbol. Empty = no change from today.
-        </span>
       </label>
 
       <label className="vi-comfy-field" style={{ flexBasis: "20rem" }}>
@@ -1186,10 +1125,6 @@ function SettingsPanel({
           placeholder="e.g. backpack, symbols, logo"
           onChange={(e) => setDraft((d) => ({ ...d, negative_prompt: e.target.value }))}
         />
-        <span className="vi-comfy-hint" style={{ opacity: 0.7 }}>
-          A true negative forwarded to the render engine alongside the Avoid/cleanup text.
-          Optional — most cases only need Avoid/cleanup above.
-        </span>
       </label>
 
       <div style={{ display: "flex", flexWrap: "wrap", gap: "0.6rem" }}>
@@ -1506,8 +1441,8 @@ function VersionsPanel({
           const isBase = v.kind === "clay" && v.name === "base";
           const isActive = profile.active_version != null && profile.active_version === v.version_id;
           return (
+            <div key={v.version_id} style={{ display: "contents" }}>
             <VersionRow
-              key={v.version_id}
               version={v}
               isActive={isActive}
               isBase={isBase}
@@ -1515,13 +1450,10 @@ function VersionsPanel({
               onRename={(name) => renameVersion(profile.slug, v.version_id, name)}
               onArchive={() => archiveVersion(profile.slug, v.version_id)}
             />
+            </div>
           );
         })}
       </div>
-      <p className="vi-comfy-hint" style={{ margin: "0.3rem 0 0", opacity: 0.65 }}>
-        The clay base is the geometric ground truth — always kept, never archivable. Generating
-        with texture lands a new version rather than overwriting.
-      </p>
     </div>
   );
 }
@@ -1927,16 +1859,133 @@ function IdentityDetail({
     }
   }
 
+  const readiness = identityReadiness(profile);
+
+  // Everything that used to be a separate STEP on this page (k94): rebuild (new
+  // version) / ✓ approve / sheet picking / angle-ring approval / settings /
+  // versions / the legacy 2D render actions. Still functional; ONE expander.
+  const advancedBody = (
+    <div style={{ display: "flex", flexDirection: "column", gap: "0.6rem", flex: "1 1 100%", minWidth: 0 }}>
+      {poseNotice && (
+        <p className="vi-comfy-hint" role="status" style={{ margin: 0, opacity: 0.85 }}>{poseNotice}</p>
+      )}
+      <div style={{ display: "flex", flexWrap: "wrap", gap: "0.4rem", alignItems: "center" }}>
+        <button
+          type="button"
+          className="vi-btn vi-btn-sm vi-btn-accent"
+          disabled={genBusy || busy}
+          onClick={() => void generateFullIdentity()}
+          title="Rebuild: a 3D mesh, a 360° turntable, auto-promoted canonical references — lands as a new version"
+        >
+          {genBusy ? "Generating…" : "⟳ rebuild 3D (new version)"}
+        </button>
+        <button
+          type="button"
+          className="vi-btn vi-btn-sm vi-btn-ghost"
+          disabled={!canApproveActive}
+          onClick={() => void approveActiveVersion()}
+          title="Promote this active version's views to the identity's canonical set"
+        >
+          {promoting ? "Approving…" : "✓ approve this version"}
+        </button>
+        {genStatus && (
+          <span className="vi-comfy-hint" role="status" style={{ opacity: 0.85 }}>{genStatus}</span>
+        )}
+        {promoteMsg && (
+          <span className="vi-comfy-hint" role="status" style={{ opacity: 0.85 }}>{promoteMsg}</span>
+        )}
+      </div>
+
+      {/* Sheet per-view picking stays available for granular approval. */}
+      {activeRecon && activeRecon.mode === "sheet" && (activeRecon.views as string[]).length > 0 && (
+        <div style={{ display: "flex", flexWrap: "wrap", gap: "0.45rem" }}>
+          {(activeRecon.views as string[]).map((relPath, idx) => (
+            <div key={relPath} style={{ display: "contents" }}>
+            <ReconViewTile
+              relPath={relPath}
+              label={VIEW_LABELS[DEFAULT_VIEWS[idx]] ?? `view ${idx + 1}`}
+              canonical={canonicalSet.has(relPath)}
+              selected={selected.has(idx)}
+              busy={recon.running || promoting}
+              onToggle={() => toggleSelected(idx)}
+              onRegenerate={() => regenerateView(idx)}
+            />
+            </div>
+          ))}
+        </div>
+      )}
+      {activeRecon && activeRecon.mode === "angle-ring" && (
+        <AngleRingEditor reconstruction={activeRecon as any} api={recon} />
+      )}
+
+      <SettingsPanel profile={profile} saveSettings={saveSettings} />
+      <VersionsPanel
+        profile={profile}
+        activateVersion={activateVersion}
+        renameVersion={renameVersion}
+        archiveVersion={archiveVersion}
+      />
+
+      {/* Legacy 2D render actions (angle ring / character sheet / orbit-clip 360). */}
+      <div style={{ display: "flex", flexWrap: "wrap", gap: "0.4rem" }}>
+        <button type="button" className={`vi-btn vi-btn-sm ${sheetOpen && sheetMode === "angle-ring" ? "vi-btn-accent" : "vi-btn-ghost"}`}
+          onClick={() => openPanel("angle-ring")} title="Generate a 36-angle iterative approval ring">✦ Angle ring</button>
+        <button type="button" className={`vi-btn vi-btn-sm ${sheetOpen && sheetMode === "sheet" ? "vi-btn-accent" : "vi-btn-ghost"}`}
+          onClick={() => openPanel("sheet")} title="Legacy: turnaround character sheet">✦ Character sheet</button>
+        <button type="button" className={`vi-btn vi-btn-sm ${sheetOpen && sheetMode === "turntable" ? "vi-btn-accent" : "vi-btn-ghost"}`}
+          onClick={() => openPanel("turntable")} title="Legacy: 360° orbit turntable clip">✦ 360° turntable</button>
+      </div>
+      {sheetOpen && (
+        <section className="vi-comfy-bar" aria-label={`Generate a turnaround for ${profile.name}`}
+          style={{ padding: "0.5rem", borderRadius: "0.4rem", border: "1px solid var(--vi-border)" }}>
+          <label className="vi-comfy-field" style={{ flex: "1 1 100%" }}>
+            <span className="vi-comfy-label">Turnaround prompt (optional)</span>
+            <textarea className="vi-knob-input" rows={2} value={sheetPrompt} placeholder="defaults to this identity's notes"
+              disabled={recon.running} onChange={(e) => setSheetPrompt(e.target.value)} />
+          </label>
+          <label className="vi-comfy-field" style={{ flexBasis: "8rem" }}>
+            <span className="vi-comfy-label">Seed (optional)</span>
+            <input type="number" className="vi-knob-input" value={sheetSeed} placeholder="random"
+              disabled={recon.running} onChange={(e) => setSheetSeed(e.target.value)} />
+          </label>
+          <div className="vi-comfy-bar-actions">
+            <button type="button" className="vi-btn vi-btn-sm vi-btn-accent" disabled={recon.running}
+              onClick={sheetMode === "angle-ring" ? generateAngleRing : sheetMode === "turntable" ? generateTurntable : generateSheet}>
+              {recon.running ? `Generating… (${recon.done}/${recon.total})`
+                : sheetMode === "angle-ring" ? "✦ Generate Angle Ring"
+                  : sheetMode === "turntable" ? "✦ Generate 360° turntable" : "✦ Generate turnaround"}
+            </button>
+          </div>
+          {recon.error && <p className="vi-error" role="alert" style={{ flexBasis: "100%", margin: 0 }}>{recon.error}</p>}
+        </section>
+      )}
+    </div>
+  );
+
   return (
     <article className="vi-identity-detail" style={{ display: "flex", flexDirection: "column", gap: "0.75rem" }}>
-      {/* ── HEADER right side: VERSION picker + “+ new” (wireframe k/j). The name
-          picker (i/h) lives in the station header; this owns versions + generate,
-          which need the detail's poll state. ───────────────────────────────── */}
+      {/* ── HEADER: name · created · ✓ ready to bind · version picker · edit/delete ── */}
       <div className="vi-id-detail-head" style={{ display: "flex", flexWrap: "wrap", gap: "0.5rem", alignItems: "center" }}>
         <h3 style={{ margin: 0, fontSize: "1.05rem", flex: "0 0 auto" }}>{profile.name}</h3>
         <span className="vi-comfy-hint" title={createdFull} style={{ opacity: 0.7, fontSize: "0.75rem" }}>
           {createdShort ? `created ${createdShort}` : ""}
         </span>
+        <span
+          className="vi-comfy-hint"
+          role="status"
+          title={
+            readiness.ready
+              ? "Reference views + a 3D model exist — bind it from “+ identity” in Scene/Movie."
+              : `Not ready yet — ${!readiness.views ? "no reference views" : ""}${!readiness.views && !readiness.mesh ? ", " : ""}${!readiness.mesh ? "no 3D model" : ""}.`
+          }
+          style={{
+            padding: "0.05rem 0.5rem", borderRadius: "0.4rem", fontSize: "0.75rem",
+            border: `1px solid ${readiness.ready ? "var(--vi-accent, #6aa8ff)" : "var(--vi-border)"}`,
+            opacity: readiness.ready ? 1 : 0.75,
+          }}
+        >
+          {readiness.ready ? "✓ ready to bind" : genBusy ? "building…" : "○ not ready"}
+        </span>
         <span style={{ flex: 1 }} />
         <label className="vi-comfy-label" style={{ opacity: 0.75 }}>Version</label>
         <select
@@ -1945,7 +1994,7 @@ function IdentityDetail({
           value={profile.active_version ?? ""}
           disabled={(profile.versions ?? []).length === 0}
           onChange={(e) => { if (e.target.value) void activateVersion(profile.slug, e.target.value); }}
-          title={(profile.versions ?? []).length === 0 ? "No versions yet — generate one" : "Switch the active version"}
+          title={(profile.versions ?? []).length === 0 ? "No versions yet" : "Switch the active version"}
         >
           {(profile.versions ?? []).length === 0 && <option value="">— no versions yet —</option>}
           {(profile.versions ?? []).map((v) => {
@@ -1957,15 +2006,6 @@ function IdentityDetail({
             );
           })}
         </select>
-        <button
-          type="button"
-          className="vi-btn vi-btn-sm vi-btn-accent"
-          disabled={genBusy || busy}
-          onClick={() => void generateFullIdentity()}
-          title="+ new version: build a 3D mesh, render a 360° turntable, and auto-promote canonical references"
-        >
-          {genBusy ? "Generating…" : "+ new"}
-        </button>
         <button type="button" className="vi-btn vi-btn-sm vi-btn-ghost" disabled={busy || genBusy} onClick={editing ? cancelEdit : startEdit}>
           {editing ? "Close edit" : "Edit"}
         </button>
@@ -1974,16 +2014,13 @@ function IdentityDetail({
         </button>
       </div>
 
-      {/* Inline edit (rename display-only + notes + refs) — opened from the header. */}
+      {/* Inline edit (rename display-only + notes) — opened from the header. */}
       {editing && (
         <div style={{ border: "1px solid var(--vi-border)", borderRadius: "0.5rem", padding: "0.6rem", display: "flex", flexDirection: "column", gap: "0.4rem" }}>
           <label className="vi-comfy-field">
             <span className="vi-comfy-label">Name</span>
             <input type="text" className="vi-knob-input" value={name} disabled={busy} onChange={(e) => setName(e.target.value)} />
           </label>
-          <p className="vi-comfy-hint" style={{ margin: 0, opacity: 0.7 }}>
-            Renaming only changes the display name — the saved slug (what templates/movies reference) never changes.
-          </p>
           <label className="vi-comfy-field">
             <span className="vi-comfy-label">Notes</span>
             <textarea className="vi-knob-input" rows={2} value={notes} disabled={busy} onChange={(e) => setNotes(e.target.value)} />
@@ -1998,19 +2035,9 @@ function IdentityDetail({
         </div>
       )}
 
-      {poseNotice && (
-        <p className="vi-comfy-hint" role="status" style={{ margin: 0, opacity: 0.85 }}>{poseNotice}</p>
-      )}
-
-      {/* The "settings" sidebar tab leads with the settings surface for this
-          identity; other tabs keep it available under the Versions panel below. */}
-      {tab === "settings" && <SettingsPanel profile={profile} saveSettings={saveSettings} />}
-
-      {/* ── CENTREPIECE (wireframe l/m/n/q): turntable (left) · approved 3D id
-          for the ACTIVE version (right) · generate-360 under the turntable ·
-          “✓ approve this version”. ─────────────────────────────────────────── */}
+      {/* ── VIEWER: turntable (left) · 3D model (right). Wire-truth off the ACTIVE
+          version's reconstruction (mesh.status === "done" + mesh.glb_path). ── */}
       <div className="vi-id-centre" style={{ display: "grid", gridTemplateColumns: "minmax(0, 440px) minmax(0, 1fr)", gap: "1rem", alignItems: "start" }}>
-        {/* Turntable (drag to orbit) */}
         <div>
           <p className="vi-comfy-label" style={{ margin: "0 0 0.3rem" }}>Turntable {turntableRecon ? "(drag to orbit)" : ""}</p>
           <div style={{ width: "100%", maxWidth: 440, aspectRatio: "440 / 320", border: "1px solid var(--vi-border)", borderRadius: "0.5rem", overflow: "hidden", background: "#0b0b0b", display: "flex", alignItems: "center", justifyContent: "center" }}>
@@ -2020,88 +2047,37 @@ function IdentityDetail({
                 degreesPerFrame={(turntableRecon as any).degrees_per_frame}
                 frameCount={(turntableRecon as any).frame_count}
               />
-            ) : activeRecon && activeRecon.mode === "angle-ring" ? (
-              <div style={{ padding: "0.5rem", width: "100%" }}>
-                <AngleRingEditor reconstruction={activeRecon as any} api={recon} />
-              </div>
             ) : (
               <span className="vi-comfy-hint" style={{ opacity: 0.7, padding: "1rem", textAlign: "center" }}>
-                No turntable for the active version yet — “⟳ generate 360” to build one.
+                {genBusy ? (genStatus || "Building…") : "No turntable for the active version."}
               </span>
             )}
           </div>
-          <div style={{ display: "flex", flexWrap: "wrap", gap: "0.4rem", marginTop: "0.4rem", alignItems: "center" }}>
-            <button
-              type="button"
-              className="vi-btn vi-btn-sm vi-btn-accent"
-              disabled={genBusy || busy}
-              onClick={() => void generateFullIdentity()}
-              title="Build a 3D mesh, render a 360° turntable, and auto-promote canonical references"
-            >
-              {genBusy ? "Generating…" : "⟳ generate 360"}
-            </button>
-            {genStatus && (
-              <span className="vi-comfy-hint" role="status" style={{ opacity: 0.85 }}>{genStatus}</span>
-            )}
-          </div>
         </div>
-
-        {/* Approved 3D id (active version) + ✓ approve this version */}
         <div>
-          <p className="vi-comfy-label" style={{ margin: "0 0 0.3rem" }}>Approved 3D identity (active version)</p>
+          <p className="vi-comfy-label" style={{ margin: "0 0 0.3rem" }}>3D model (active version)</p>
           <div style={{ width: "100%", maxWidth: 500, aspectRatio: "500 / 320", border: "1px solid var(--vi-border)", borderRadius: "0.5rem", overflow: "hidden", background: "#0b0b0b", display: "flex", alignItems: "center", justifyContent: "center" }}>
             {meshDone ? (
               <MeshViewer url={mediaBytesUrl(meshDone)} height={320} />
             ) : (
               <span className="vi-comfy-hint" style={{ opacity: 0.7, padding: "1rem", textAlign: "center" }}>
-                No approved 3D mesh for the active version yet.
+                {genBusy ? (genStatus || "Building…") : "No 3D model for the active version."}
               </span>
             )}
           </div>
-          <div style={{ display: "flex", flexWrap: "wrap", gap: "0.4rem", marginTop: "0.4rem", alignItems: "center" }}>
-            <button
-              type="button"
-              className="vi-btn vi-btn-sm vi-btn-accent"
-              disabled={!canApproveActive}
-              onClick={() => void approveActiveVersion()}
-              title="Promote this active version's views to the identity's canonical set"
-            >
-              {promoting ? "Approving…" : "✓ approve this version"}
-            </button>
-            {meshDone && (
+          {meshDone && (
+            <div style={{ display: "flex", gap: "0.4rem", marginTop: "0.4rem", alignItems: "center" }}>
               <a className="vi-btn vi-btn-sm vi-btn-ghost" href={mediaBytesUrl(meshDone)} download>↓ GLB</a>
-            )}
-            {promoteMsg && (
-              <span className="vi-comfy-hint" role="status" style={{ opacity: 0.85 }}>{promoteMsg}</span>
-            )}
-          </div>
-          {/* Sheet/angle-ring per-view picking stays available for granular approval. */}
-          {activeRecon && activeRecon.mode === "sheet" && (activeRecon.views as string[]).length > 0 && (
-            <div style={{ display: "flex", flexWrap: "wrap", gap: "0.45rem", marginTop: "0.5rem" }}>
-              {(activeRecon.views as string[]).map((relPath, idx) => (
-                <ReconViewTile
-                  key={relPath}
-                  relPath={relPath}
-                  label={VIEW_LABELS[DEFAULT_VIEWS[idx]] ?? `view ${idx + 1}`}
-                  canonical={canonicalSet.has(relPath)}
-                  selected={selected.has(idx)}
-                  busy={recon.running || promoting}
-                  onToggle={() => toggleSelected(idx)}
-                  onRegenerate={() => regenerateView(idx)}
-                />
-              ))}
             </div>
           )}
         </div>
       </div>
 
-      {/* ── BOTTOM (wireframe o/p/r): uploaded refs (left, add/remove) · canonical
-          8 grid with angle labels (right) · the DNA note. ─────────────────────── */}
+      {/* ── references (left, add/remove) · canonical views (right) ── */}
       <div className="vi-id-bottom" style={{ display: "grid", gridTemplateColumns: "minmax(0, 440px) minmax(0, 1fr)", gap: "1rem", alignItems: "start" }}>
-        {/* Uploaded references — add/remove, live-saved via update(). */}
         <div>
           <p className="vi-comfy-label" style={{ margin: "0 0 0.3rem" }}>
-            Uploaded references ({refs.length})
+            References ({refs.length})
           </p>
           <ReferenceEditor refs={refs} onAdd={addRef} onRemove={removeRef} libraryImages={libraryImages} busy={busy} />
           <div style={{ display: "flex", gap: "0.4rem", marginTop: "0.4rem", alignItems: "center" }}>
@@ -2125,17 +2101,14 @@ function IdentityDetail({
           </div>
         </div>
 
-        {/* CANONICAL 8 — the identity DNA (constraint #2): drives off TOP-LEVEL
-            profile.canonical only, with the honest angle label (viewLabel reused
-            from GenIdentityBar). NEVER falls back to raw reference_images. */}
+        {/* CANONICAL — the identity DNA: TOP-LEVEL profile.canonical only, with the
+            honest angle label (viewLabel reused from GenIdentityBar). */}
         <div>
           <p className="vi-comfy-label" style={{ margin: "0 0 0.3rem" }}>
-            Canonical {canonicalPaths.length ? `(${canonicalPaths.length})` : ""}
+            Canonical views {canonicalPaths.length ? `(${canonicalPaths.length})` : ""}
           </p>
           {canonicalPaths.length === 0 ? (
-            <p className="vi-comfy-hint" style={{ opacity: 0.7 }}>
-              No canonical views yet — approve the active version to build the identity's canonical set.
-            </p>
+            <p className="vi-comfy-hint" style={{ opacity: 0.7 }}>No canonical views yet.</p>
           ) : (
             <div style={{ display: "grid", gridTemplateColumns: "repeat(auto-fill, minmax(5rem, 1fr))", gap: "0.4rem" }}>
               {canonicalPaths.map((relPath, idx) => (
@@ -2153,58 +2126,12 @@ function IdentityDetail({
               ))}
             </div>
           )}
-          <p className="vi-comfy-hint" style={{ margin: "0.4rem 0 0", opacity: 0.65, fontSize: "0.72rem" }}>
-            Canonical = the identity DNA — sent to generation, never raw uploads.
-          </p>
         </div>
       </div>
 
-      {/* Versions list (activate / rename / archive) + settings (when not the
-          settings tab, kept reachable here). */}
-      <VersionsPanel
-        profile={profile}
-        activateVersion={activateVersion}
-        renameVersion={renameVersion}
-        archiveVersion={archiveVersion}
-      />
-      {tab !== "settings" && (
-        <details className="vi-id-settings-disclosure">
-          <summary className="vi-comfy-label" style={{ cursor: "pointer" }}>⚙ Settings & advanced</summary>
-          <SettingsPanel profile={profile} saveSettings={saveSettings} />
-          <div style={{ display: "flex", flexWrap: "wrap", gap: "0.4rem", marginTop: "0.5rem" }}>
-            <button type="button" className={`vi-btn vi-btn-sm ${sheetOpen && sheetMode === "angle-ring" ? "vi-btn-accent" : "vi-btn-ghost"}`}
-              onClick={() => openPanel("angle-ring")} title="Generate a 36-angle iterative approval ring">✦ Angle ring</button>
-            <button type="button" className={`vi-btn vi-btn-sm ${sheetOpen && sheetMode === "sheet" ? "vi-btn-accent" : "vi-btn-ghost"}`}
-              onClick={() => openPanel("sheet")} title="Legacy: turnaround character sheet">✦ Character sheet</button>
-            <button type="button" className={`vi-btn vi-btn-sm ${sheetOpen && sheetMode === "turntable" ? "vi-btn-accent" : "vi-btn-ghost"}`}
-              onClick={() => openPanel("turntable")} title="Legacy: 360° orbit turntable clip">✦ 360° turntable</button>
-          </div>
-          {sheetOpen && (
-            <section className="vi-comfy-bar" aria-label={`Generate a turnaround for ${profile.name}`}
-              style={{ marginTop: "0.4rem", padding: "0.5rem", borderRadius: "0.4rem", border: "1px solid var(--vi-border)" }}>
-              <label className="vi-comfy-field" style={{ flex: "1 1 100%" }}>
-                <span className="vi-comfy-label">Turnaround prompt (optional)</span>
-                <textarea className="vi-knob-input" rows={2} value={sheetPrompt} placeholder="defaults to this identity's notes"
-                  disabled={recon.running} onChange={(e) => setSheetPrompt(e.target.value)} />
-              </label>
-              <label className="vi-comfy-field" style={{ flexBasis: "8rem" }}>
-                <span className="vi-comfy-label">Seed (optional)</span>
-                <input type="number" className="vi-knob-input" value={sheetSeed} placeholder="random"
-                  disabled={recon.running} onChange={(e) => setSheetSeed(e.target.value)} />
-              </label>
-              <div className="vi-comfy-bar-actions">
-                <button type="button" className="vi-btn vi-btn-sm vi-btn-accent" disabled={recon.running}
-                  onClick={sheetMode === "angle-ring" ? generateAngleRing : sheetMode === "turntable" ? generateTurntable : generateSheet}>
-                  {recon.running ? `Generating… (${recon.done}/${recon.total})`
-                    : sheetMode === "angle-ring" ? "✦ Generate Angle Ring"
-                      : sheetMode === "turntable" ? "✦ Generate 360° turntable" : "✦ Generate turnaround"}
-                </button>
-              </div>
-              {recon.error && <p className="vi-error" role="alert" style={{ flexBasis: "100%", margin: 0 }}>{recon.error}</p>}
-            </section>
-          )}
-        </details>
-      )}
+      {/* ONE expander for everything else (k94). The sidebar "settings" tab leads
+          with it open; every other tab keeps it collapsed. */}
+      <AdvancedExpander open={tab === "settings"}>{advancedBody}</AdvancedExpander>
     </article>
   );
 }
@@ -2235,6 +2162,8 @@ export function IdentitiesStation({ spec }: { spec: StationSpec }) {
     renameVersion,
     archiveVersion,
     extractFromVideo,
+    createFromVideo,
+    createFromImages,
   } = useIdentityProfiles();
   const library = useMediaLibrary();
   const libraryImages = library.filter((it) => it.ref.kind === "image");
@@ -2246,8 +2175,6 @@ export function IdentitiesStation({ spec }: { spec: StationSpec }) {
   // cadence (same discipline as the Serving-table multi-select Set) — the
   // profiles array is replaced on every reload, but the slug is stable.
   const [selectedSlug, setSelectedSlug] = useState<string | null>(null);
-  // "+ new identity" opens the create panel inline above the detail.
-  const [creating, setCreating] = useState(false);
   // "session" = identities TOUCHED this session (selected / created / generated).
   // There is no server-side last-used timestamp on the profile wire, so this is
   // a client-side Set of slugs, seeded as identities are interacted with. Kept
@@ -2261,7 +2188,6 @@ export function IdentitiesStation({ spec }: { spec: StationSpec }) {
   const select = useCallback(
     (slug: string) => {
       setSelectedSlug(slug);
-      setCreating(false);
       touch(slug);
     },
     [touch],
@@ -2329,7 +2255,7 @@ export function IdentitiesStation({ spec }: { spec: StationSpec }) {
               ? "No identities touched yet this session."
               : tab === "active"
                 ? "No identity has an active version yet — generate one."
-                : "No saved identities yet — “+ new” to create one."}
+                : "No saved identities yet — create one above."}
           </p>
         ) : (
           <ul className="vi-id-list" style={{ listStyle: "none", margin: 0, padding: 0, display: "flex", flexDirection: "column", gap: "0.25rem", maxHeight: "640px", overflowY: "auto" }}>
@@ -2360,8 +2286,8 @@ export function IdentitiesStation({ spec }: { spec: StationSpec }) {
                     <span style={{ display: "flex", flexDirection: "column", minWidth: 0 }}>
                       <span style={{ fontSize: "0.85rem", overflow: "hidden", textOverflow: "ellipsis", whiteSpace: "nowrap" }}>{p.name}</span>
                       <span className="vi-comfy-hint" style={{ fontSize: "0.68rem", opacity: 0.7 }}>
-                        {p.active_version != null ? "● active" : "○ no active version"}
-                        {p.canonical.length ? ` · ${p.canonical.length} canonical` : ""}
+                        {identityReadiness(p).ready ? "✓ ready to bind" : p.active_version != null ? "● active" : "○ not ready"}
+                        {p.canonical.length ? ` · ${p.canonical.length} views` : ""}
                       </span>
                     </span>
                   </button>
@@ -2372,10 +2298,38 @@ export function IdentitiesStation({ spec }: { spec: StationSpec }) {
         )}
       </aside>
 
-      {/* ── MAIN: header pickers + single-identity detail ──────────────────── */}
+      {/* ── MAIN: the ONE PATH create panel (k94) · identity picker · detail ── */}
       <div className="vi-id-main" style={{ display: "flex", flexDirection: "column", gap: "0.75rem", minWidth: 0 }}>
-        {/* Header row: NAME picker + “+ new” (wireframe i/h). The version picker
-            (k/j) lives inside the detail, which owns version state + generate. */}
+        <IdentityCreatePanel
+          createFromVideo={createFromVideo}
+          createFromImages={createFromImages}
+          library={library}
+          reload={reload}
+          onCreated={(slugs) => {
+            for (const s of slugs) touch(s);
+            if (slugs[0]) setSelectedSlug(slugs[0]);
+          }}
+          advancedOpen={tab === "settings"}
+          advanced={
+            <div style={{ display: "flex", flexDirection: "column", gap: "0.4rem", flex: "1 1 100%", minWidth: 0 }}>
+              <span className="vi-comfy-label" style={{ opacity: 0.75 }}>Manual create (name + notes + reference images, no 3D build)</span>
+              <CreatePanel
+                create={async (name, refs, notes) => {
+                  const res = await create(name, refs, notes);
+                  if (res.ok && res.profile) {
+                    touch(res.profile.slug);
+                    setSelectedSlug(res.profile.slug);
+                  }
+                  return res;
+                }}
+                libraryImages={libraryImages}
+              />
+              <ExtractFromVideoPanel profiles={profiles} extractFromVideo={extractFromVideo} reload={reload} />
+            </div>
+          }
+        />
+
+        {/* Identity picker — which saved identity fills the detail below. */}
         <div className="vi-id-header" style={{ display: "flex", flexWrap: "wrap", gap: "0.5rem", alignItems: "center" }}>
           <label className="vi-comfy-label" style={{ opacity: 0.75 }}>Identity</label>
           <select
@@ -2387,42 +2341,14 @@ export function IdentitiesStation({ spec }: { spec: StationSpec }) {
           >
             {profiles.length === 0 && <option value="">— no identities yet —</option>}
             {profiles.map((p) => (
-              <option key={p.slug} value={p.slug}>{p.name}</option>
+              <option key={p.slug} value={p.slug}>{identityReadiness(p).ready ? "✓ " : ""}{p.name}</option>
             ))}
           </select>
-          <button
-            type="button"
-            className={`vi-btn vi-btn-sm ${creating ? "vi-btn-accent" : "vi-btn-ghost"}`}
-            aria-expanded={creating}
-            onClick={() => setCreating((v) => !v)}
-            title="Create a new identity"
-          >
-            + new
-          </button>
         </div>
 
-        {/* “+ new identity” — the create panel + video-extract path, inline. */}
-        {creating && (
-          <div style={{ border: "1px solid var(--vi-border)", borderRadius: "0.5rem", padding: "0.5rem" }}>
-            <CreatePanel
-              create={async (name, refs, notes) => {
-                const res = await create(name, refs, notes);
-                if (res.ok && res.profile) {
-                  touch(res.profile.slug);
-                  setSelectedSlug(res.profile.slug);
-                  setCreating(false);
-                }
-                return res;
-              }}
-              libraryImages={libraryImages}
-            />
-            <ExtractFromVideoPanel profiles={profiles} extractFromVideo={extractFromVideo} reload={reload} />
-          </div>
-        )}
-
         {selected ? (
+          <div key={selected.slug} style={{ display: "contents" }}>
           <IdentityDetail
-            key={selected.slug}
             profile={selected}
             tab={tab}
             update={update}
@@ -2440,11 +2366,12 @@ export function IdentitiesStation({ spec }: { spec: StationSpec }) {
             archiveVersion={archiveVersion}
             onTouch={touch}
           />
+          </div>
         ) : (
           !loading && (
             <p className="vi-comfy-hint" style={{ padding: "1rem" }}>
               {profiles.length === 0
-                ? "No identities yet — click “+ new” to create your first."
+                ? "No identities yet — create your first above."
                 : "Select an identity from the list to view it."}
             </p>
           )