From def3ce5202de556adcb505d207de9b9aca107a65 Mon Sep 17 00:00:00 2001
From: JayceeB1 <45721154+JayceeB1@users.noreply.github.com>
Date: Sat, 10 Oct 2026 00:44:21 +0200
Subject: [PATCH 1/5] integrate(#250): workflow-only full-detail optimizer
 preset with regression tests

Preserves Python mesh-op registry for direct API callers; accepts HTML select strings and bound numeric budgets.
---
 api/services/mesh_ops/processor.py            | 38 +++++++++-
 api/tests/test_mesh_ops_processor.py          | 74 +++++++++++++++++++
 src/areas/generate/GeneratePage.tsx           | 47 +++---------
 .../nodes/mesh-optimizer/manifest.json        | 24 ++++--
 4 files changed, 138 insertions(+), 45 deletions(-)

diff --git a/api/services/mesh_ops/processor.py b/api/services/mesh_ops/processor.py
index 9af04353..df934081 100644
--- a/api/services/mesh_ops/processor.py
+++ b/api/services/mesh_ops/processor.py
@@ -10,6 +10,28 @@
 from . import MeshOpContext, mesh_ops_registry
 
 
+# A workflow-only sentinel: direct mesh-op/API calls retain their original
+# decimation behavior and schema. Bound node params may provide an int or HTML
+# <select> string, so normalize the value before dispatching to the registry.
+WORKFLOW_FULL_DETAIL = 1_000_000
+
+
+def _normalize_workflow_target_faces(value: object) -> int:
+    if isinstance(value, bool):
+        raise ValueError("mesh-optimizer: target_faces must be an integer between 100 and 1000000")
+    if isinstance(value, int):
+        target = value
+    elif isinstance(value, float) and value.is_integer():
+        target = int(value)
+    elif isinstance(value, str) and value.strip().isascii() and value.strip().isdecimal():
+        target = int(value.strip())
+    else:
+        raise ValueError("mesh-optimizer: target_faces must be an integer between 100 and 1000000")
+    if not 100 <= target <= WORKFLOW_FULL_DETAIL:
+        raise ValueError("mesh-optimizer: target_faces must be between 100 and 1000000")
+    return target
+
+
 def _emit(message: dict) -> None:
     print(json.dumps(message), flush=True)
 
@@ -32,6 +54,20 @@ def run_processor(operation_id: str, processor_id: str) -> None:
                 f"{processor_id}: input file not found: {input_path}"
             )
 
+        params = data.get("params") or {}
+        if operation_id == "decimate" and processor_id == "mesh-optimizer":
+            target_faces = _normalize_workflow_target_faces(
+                params.get("target_faces", WORKFLOW_FULL_DETAIL)
+            )
+            if target_faces == WORKFLOW_FULL_DETAIL:
+                _emit({"type": "log", "message": "Full detail selected: original mesh preserved"})
+                _emit({"type": "progress", "percent": 100, "label": "Done"})
+                _emit({"type": "done", "result": {"filePath": str(input_path)}})
+                return
+            # Canonical mesh-op registry expects numeric values, not HTML
+            # select strings. Keep other params for workflow compatibility.
+            params = {**params, "target_faces": target_faces}
+
         workspace_dir = Path(
             data.get("workspaceDir")
             or os.environ.get("WORKSPACE_DIR")
@@ -53,7 +89,7 @@ def run_processor(operation_id: str, processor_id: str) -> None:
         result = mesh_ops_registry.run(
             operation_id,
             Path(input_path),
-            data.get("params") or {},
+            params,
             context,
         )
         _emit(
diff --git a/api/tests/test_mesh_ops_processor.py b/api/tests/test_mesh_ops_processor.py
index 2f8a6659..6b7b4b9e 100644
--- a/api/tests/test_mesh_ops_processor.py
+++ b/api/tests/test_mesh_ops_processor.py
@@ -72,5 +72,79 @@ def test_missing_input_uses_existing_node_error_contract(self) -> None:
         self.assertIn("mesh-repair: input file not found: None", message["message"])
 
 
+    def test_optimizer_full_detail_skips_decimation_and_keeps_input(self) -> None:
+        with tempfile.TemporaryDirectory() as directory:
+            original = Path(directory) / "original.glb"
+            original.write_bytes(b"precious mesh")
+            registry = _FakeRegistry(Path(directory) / "should-not-exist.glb")
+            for params in ({}, {"target_faces": 1000000}, {"target_faces": "1000000"}):
+                with self.subTest(params=params):
+                    request = {"input": {"filePath": str(original)}, "params": params}
+                    stdout = io.StringIO()
+                    with (
+                        patch.object(processor, "mesh_ops_registry", registry),
+                        patch.object(processor.sys, "stdin", io.StringIO(json.dumps(request))),
+                        redirect_stdout(stdout),
+                    ):
+                        processor.run_processor("decimate", "mesh-optimizer")
+                    output = [json.loads(line) for line in stdout.getvalue().splitlines()]
+                    self.assertEqual([m["type"] for m in output], ["log", "progress", "done"])
+                    self.assertEqual(output[-1]["result"]["filePath"], str(original))
+                    self.assertEqual(original.read_bytes(), b"precious mesh")
+            self.assertEqual(registry.calls, [])
+
+    def test_optimizer_budget_coerces_html_select_values_and_bound_inputs(self) -> None:
+        with tempfile.TemporaryDirectory() as directory:
+            original = Path(directory) / "original.glb"
+            original.write_bytes(b"mesh")
+            registry = _FakeRegistry(Path(directory) / "optimized.glb")
+            for value in ("50000", 10000, "12000", 30000.0):
+                with self.subTest(value=value):
+                    request = {"input": {"filePath": str(original)}, "params": {"target_faces": value}}
+                    out = io.StringIO()
+                    with (
+                        patch.object(processor, "mesh_ops_registry", registry),
+                        patch.object(processor.sys, "stdin", io.StringIO(json.dumps(request))),
+                        redirect_stdout(out),
+                    ):
+                        processor.run_processor("decimate", "mesh-optimizer")
+                    messages = [json.loads(line) for line in out.getvalue().splitlines()]
+                    self.assertEqual(messages[-1]["type"], "done")
+                    self.assertEqual(registry.calls[-1][2]["target_faces"], int(value))
+
+    def test_optimizer_invalid_budgets_fail_closed_before_registry(self) -> None:
+        with tempfile.TemporaryDirectory() as directory:
+            original = Path(directory) / "original.glb"
+            original.write_bytes(b"mesh")
+            registry = _FakeRegistry(Path(directory) / "optimized.glb")
+            for value in ("NaN", "1e6", True, 99, 1000001, -1, 3.5):
+                with self.subTest(value=value):
+                    request = {"input": {"filePath": str(original)}, "params": {"target_faces": value}}
+                    out = io.StringIO()
+                    with (
+                        patch.object(processor, "mesh_ops_registry", registry),
+                        patch.object(processor.sys, "stdin", io.StringIO(json.dumps(request))),
+                        redirect_stdout(out),
+                    ):
+                        processor.run_processor("decimate", "mesh-optimizer")
+                    messages = [json.loads(line) for line in out.getvalue().splitlines()]
+                    self.assertEqual(messages[-1]["type"], "error")
+                    self.assertNotIn('"type": "done"', out.getvalue())
+            self.assertEqual(registry.calls, [])
+
+    def test_optimizer_requires_real_input_even_when_full_detail(self) -> None:
+        with tempfile.TemporaryDirectory() as directory:
+            request = {"input": {"filePath": str(Path(directory) / "missing.glb")}, "params": {}}
+            out = io.StringIO()
+            with (
+                patch.object(processor.sys, "stdin", io.StringIO(json.dumps(request))),
+                redirect_stdout(out),
+            ):
+                processor.run_processor("decimate", "mesh-optimizer")
+            messages = [json.loads(line) for line in out.getvalue().splitlines()]
+            self.assertEqual(messages[-1]["type"], "error")
+
+
+
 if __name__ == "__main__":
     unittest.main()
diff --git a/src/areas/generate/GeneratePage.tsx b/src/areas/generate/GeneratePage.tsx
index 86636dab..89d2a669 100644
--- a/src/areas/generate/GeneratePage.tsx
+++ b/src/areas/generate/GeneratePage.tsx
@@ -522,7 +522,6 @@ function AssetLibraryPopover({
   onViewModeChange,
   onFiltersChange,
   onToggleSection,
-  onOpenSelected,
   onFetchPreviewFrame,
   onRefresh,
   onClose,
@@ -552,7 +551,6 @@ function AssetLibraryPopover({
   onViewModeChange: (value: AssetLibraryViewMode) => void
   onFiltersChange: (value: AssetLibraryFilters) => void
   onToggleSection: (sectionKey: string) => void
-  onOpenSelected: () => void
   onFetchPreviewFrame: (workspacePath: string, clip: string) => Promise<string | null>
   onRefresh: () => void
   onClose: () => void
@@ -560,21 +558,11 @@ function AssetLibraryPopover({
   onResizeHeightMouseDown: (event: React.MouseEvent) => void
 }) {
   const projectGroups = buildAssetLibraryProjectGroups(entries, searchQuery, sortMode, filters)
-  const visibleEntryIds = new Set(projectGroups.flatMap((group) => group.families.flatMap((family) => family.entries.map((entry) => entry.id))))
-  const selectedEntry = selectedEntryId && visibleEntryIds.has(selectedEntryId)
-    ? entries.find((entry) => entry.id === selectedEntryId) ?? null
-    : null
   const normalizedSearchQuery = searchQuery.trim()
   const hasActiveFilters = filters.kind !== 'all' || filters.poly !== 'all' || filters.needsAttention
   const hasActiveDiscovery = normalizedSearchQuery.length > 0 || hasActiveFilters
   const familyCount = projectGroups.reduce((count, group) => count + group.families.length, 0)
   const versionCount = projectGroups.reduce((count, group) => count + group.entryCount, 0)
-  const openDisabled = !selectedEntry || !isAssetLibraryEntryOpenable(selectedEntry) || loading || opening
-  const selectedMessage = selectedEntry
-    ? describeAssetLibraryOpenability(selectedEntry)
-    : projectGroups.length === 0 && normalizedSearchQuery
-      ? `No workspace assets match “${normalizedSearchQuery}”.`
-      : 'Select an asset to open it in Generate.'
 
   return (
     <div
@@ -718,6 +706,12 @@ function AssetLibraryPopover({
         </div>
       </div>
 
+      {error && (
+        <p role="alert" className="shrink-0 rounded-lg border border-amber-800/50 bg-amber-950/40 px-3 py-2 text-[11px] text-amber-300">
+          {error}
+        </p>
+      )}
+
       {loading ? (
         <p role="status" className="flex-1 min-h-0 flex items-center justify-center text-xs text-zinc-400">Loading workspace assets…</p>
       ) : projectGroups.length === 0 && !hasActiveDiscovery ? (
@@ -787,21 +781,6 @@ function AssetLibraryPopover({
           })}
         </div>
       )}
-
-      <div className="shrink-0 rounded-lg border border-zinc-800 bg-zinc-950/40 px-3 py-2">
-        <p className="text-[11px] text-zinc-400">{selectedMessage}</p>
-        {error && <p role="alert" className="mt-2 text-[11px] text-amber-300">{error}</p>}
-      </div>
-
-      <button
-        type="button"
-        onClick={onOpenSelected}
-        disabled={openDisabled}
-        aria-label="Open selected asset"
-        className="shrink-0 px-3 py-2 bg-violet-600 hover:bg-violet-500 disabled:bg-zinc-700 disabled:text-zinc-500 text-white text-xs rounded-lg transition-colors font-medium"
-      >
-        {opening ? 'Opening…' : 'Open selected asset'}
-      </button>
     </div>
   )
 }
@@ -1394,8 +1373,7 @@ export default function GeneratePage(): JSX.Element {
     }
   }, [assetLibraryService])
 
-  // Shared by both the "Open selected asset" button and a direct row click —
-  // a row activation selects first, then opens through this same path.
+  // A Library row opens through this shared validation and import path.
   async function openLibraryEntry(entry: ProjectedAssetLibraryEntry | null) {
     if (!entry) {
       setLibraryError('Select an asset before opening it in Generate.')
@@ -1432,14 +1410,8 @@ export default function GeneratePage(): JSX.Element {
     }
   }
 
-  async function handleOpenSelectedLibraryEntry() {
-    const selectedEntry = libraryEntries.find((entry) => entry.id === librarySelectedEntryId) ?? null
-    await openLibraryEntry(selectedEntry)
-  }
-
-  // A single click on a row both selects and opens it — no second click on
-  // "Open selected asset" required. Non-openable entries still get selected,
-  // surfacing the same "why not" message they always have.
+  // A single click on a row both selects and opens it. Non-openable entries
+  // stay selected and surface their reason in the conditional message above.
   async function handleActivateLibraryEntry(entryId: string) {
     setLibraryError(null)
     setLibrarySelectedEntryId(entryId)
@@ -1669,7 +1641,6 @@ export default function GeneratePage(): JSX.Element {
                 onViewModeChange={setLibraryViewMode}
                 onFiltersChange={setLibraryFilters}
                 onToggleSection={(sectionKey) => setLibraryCollapsedSectionKeys((current) => toggleAssetLibrarySectionKey(current, sectionKey))}
-                onOpenSelected={() => { void handleOpenSelectedLibraryEntry() }}
                 onFetchPreviewFrame={fetchLibraryPreviewFrame}
                 onRefresh={() => { void loadLibraryEntries() }}
                 onClose={() => setOpenPanel(null)}
diff --git a/src/areas/workflows/nodes/mesh-optimizer/manifest.json b/src/areas/workflows/nodes/mesh-optimizer/manifest.json
index e2480dda..5da65237 100644
--- a/src/areas/workflows/nodes/mesh-optimizer/manifest.json
+++ b/src/areas/workflows/nodes/mesh-optimizer/manifest.json
@@ -15,12 +15,24 @@
       "params_schema": [
         {
           "id": "target_faces",
-          "label": "Target Triangles",
-          "type": "int",
-          "default": 10000,
-          "min": 100,
-          "max": 1000000,
-          "tooltip": "Target number of triangles after simplification."
+          "label": "Detail",
+          "type": "select",
+          "default": 1000000,
+          "tooltip": "Choose a target triangle budget. Full detail preserves the input GLB without changing it.",
+          "options": [
+            {
+              "value": 1000000,
+              "label": "Full detail (unchanged)"
+            },
+            {
+              "value": 50000,
+              "label": "Game prop (about 50,000 triangles)"
+            },
+            {
+              "value": 10000,
+              "label": "Background piece (about 10,000 triangles)"
+            }
+          ]
         }
       ]
     }

From a1ef816826d0c74aa97fccea27ebe540119ba4a1 Mon Sep 17 00:00:00 2001
From: JayceeB1 <45721154+JayceeB1@users.noreply.github.com>
Date: Sat, 10 Oct 2026 00:46:56 +0200
Subject: [PATCH 2/5] integrate(#251): bind workflow parameters at execution
 time

Rebased the core #251 behavior on current runner; safeguard dangling/self bindings and unsafe property keys; add seven Node tests.
---
 .../generate/components/WorkflowPanel.tsx     | 14 +++--
 .../nodes/mesh-optimizer/manifest.json        | 16 +++--
 .../workflows/workflowParamBindings.test.mjs  | 58 +++++++++++++++++++
 src/areas/workflows/workflowParamBindings.ts  | 56 ++++++++++++++++++
 src/areas/workflows/workflowRunStore.ts       |  9 ++-
 src/shared/types/electron.d.ts                |  9 +++
 6 files changed, 153 insertions(+), 9 deletions(-)
 create mode 100644 src/areas/workflows/workflowParamBindings.test.mjs
 create mode 100644 src/areas/workflows/workflowParamBindings.ts

diff --git a/src/areas/generate/components/WorkflowPanel.tsx b/src/areas/generate/components/WorkflowPanel.tsx
index 54e40bae..c3c61d15 100644
--- a/src/areas/generate/components/WorkflowPanel.tsx
+++ b/src/areas/generate/components/WorkflowPanel.tsx
@@ -13,6 +13,7 @@ import { useWorkflowRunStore } from '@areas/workflows/workflowRunStore'
 import { useWaitButton } from '@areas/workflows/useWaitButton'
 import { buildAllWorkflowExtensions, getWorkflowExtension } from '@areas/workflows/mockExtensions'
 import { validateWorkflowPreflight } from '@areas/workflows/preflight'
+import { areAllWorkflowNodeParamsBound } from '@areas/workflows/workflowParamBindings'
 import { mimeFromPath } from '@areas/workflows/nodes/imageUtils'
 import type { WorkflowExtension } from '@areas/workflows/mockExtensions'
 import type { Workflow, WFNode, WFEdge, ParamSchema } from '@shared/types/electron.d'
@@ -609,10 +610,15 @@ function EmbeddedCanvas({ workflow, allExtensions }: {
     [nodes, edges],
   )
 
-  const paramNodes = sortedNodes.filter((n) =>
-    (n.type === 'imageNode' || n.type === 'textNode' || n.type === 'meshNode' || n.type === 'extensionNode' || n.type === 'waitNode')
-    && (n.data as { showInGenerate?: boolean }).showInGenerate === true,
-  )
+  const paramNodes = sortedNodes.filter((n) => {
+    if (
+      (n.type !== 'imageNode' && n.type !== 'textNode' && n.type !== 'meshNode' && n.type !== 'extensionNode' && n.type !== 'waitNode')
+      || (n.data as { showInGenerate?: boolean }).showInGenerate !== true
+    ) return false
+    if (n.type !== 'extensionNode') return true
+    const ext = getWorkflowExtension(n.data.extensionId ?? '', allExtensions)
+    return !ext || !areAllWorkflowNodeParamsBound(workflow, n.id, ext.params.map((p) => p.id))
+  })
 
   const handleGenerate = useCallback(() => {
     if (firstPreflightIssue) {
diff --git a/src/areas/workflows/nodes/mesh-optimizer/manifest.json b/src/areas/workflows/nodes/mesh-optimizer/manifest.json
index 5da65237..9807274d 100644
--- a/src/areas/workflows/nodes/mesh-optimizer/manifest.json
+++ b/src/areas/workflows/nodes/mesh-optimizer/manifest.json
@@ -25,12 +25,20 @@
               "label": "Full detail (unchanged)"
             },
             {
-              "value": 50000,
-              "label": "Game prop (about 50,000 triangles)"
+              "value": 12000,
+              "label": "Light — background props"
             },
             {
-              "value": 10000,
-              "label": "Background piece (about 10,000 triangles)"
+              "value": 30000,
+              "label": "Standard — most props"
+            },
+            {
+              "value": 60000,
+              "label": "Heavy — hero pieces you get close to"
+            },
+            {
+              "value": 120000,
+              "label": "Very heavy — 3D printing, film shots"
             }
           ]
         }
diff --git a/src/areas/workflows/workflowParamBindings.test.mjs b/src/areas/workflows/workflowParamBindings.test.mjs
new file mode 100644
index 00000000..ce94b799
--- /dev/null
+++ b/src/areas/workflows/workflowParamBindings.test.mjs
@@ -0,0 +1,58 @@
+import assert from 'node:assert/strict'
+import test from 'node:test'
+import { areAllWorkflowNodeParamsBound, resolveBoundWorkflowParams } from './workflowParamBindings.ts'
+
+const workflow = {
+  id: 'asset', name: 'Asset', description: '', createdAt: '', updatedAt: '',
+  nodes: [{ id: 'generate', data: { params: { faces: 12000 } } }, { id: 'optimize', data: { params: { target_faces: 1000000 } } }],
+  edges: [],
+  paramBindings: [{ sourceNodeId: 'generate', sourceParam: 'faces', targetNodeId: 'optimize', targetParam: 'target_faces' }],
+}
+
+test('latest upstream parameter wins on each target execution', () => {
+  const live = new Map([
+    ['generate', { faces: 60000 }],
+    ['optimize', { target_faces: 1000000 }],
+  ])
+  assert.deepEqual(resolveBoundWorkflowParams(workflow, 'optimize', live.get('optimize'), (id) => live.get(id)), { target_faces: 60000 })
+  live.set('generate', { faces: 12000 })
+  assert.deepEqual(resolveBoundWorkflowParams(workflow, 'optimize', live.get('optimize'), (id) => live.get(id)), { target_faces: 12000 })
+})
+
+test('unbound parameters preserve the same object and values', () => {
+  const original = { export_format: 'glb' }
+  assert.equal(resolveBoundWorkflowParams(workflow, 'export', original, () => undefined), original)
+})
+
+test('missing source value retains explicit target fallback', () => {
+  assert.deepEqual(resolveBoundWorkflowParams(workflow, 'optimize', { target_faces: 50000 }, () => undefined), { target_faces: 50000 })
+})
+
+test('only fully bound extension nodes hide Generate questions', () => {
+  assert.equal(areAllWorkflowNodeParamsBound(workflow, 'optimize', ['target_faces']), true)
+  assert.equal(areAllWorkflowNodeParamsBound(workflow, 'optimize', ['target_faces', 'preserve_uvs']), false)
+  assert.equal(areAllWorkflowNodeParamsBound(workflow, 'generate', ['faces']), false)
+  assert.equal(areAllWorkflowNodeParamsBound(workflow, 'optimize', []), false)
+})
+
+test('dangling or self-referential binding cannot hide a question', () => {
+  const broken = { ...workflow, paramBindings: [{ ...workflow.paramBindings[0], sourceNodeId: 'missing' }] }
+  assert.equal(areAllWorkflowNodeParamsBound(broken, 'optimize', ['target_faces']), false)
+  assert.deepEqual(resolveBoundWorkflowParams(broken, 'optimize', { target_faces: 10000 }, () => ({ faces: 12000 })), { target_faces: 10000 })
+  const self = { ...workflow, paramBindings: [{ ...workflow.paramBindings[0], sourceNodeId: 'optimize' }] }
+  assert.equal(areAllWorkflowNodeParamsBound(self, 'optimize', ['target_faces']), false)
+})
+
+test('untrusted prototype target key is ignored', () => {
+  const evil = { ...workflow, paramBindings: [{ ...workflow.paramBindings[0], targetParam: '__proto__' }] }
+  const result = resolveBoundWorkflowParams(evil, 'optimize', { target_faces: 10000 }, () => ({ faces: { admin: true } }))
+  assert.equal(Object.getPrototypeOf(result), Object.prototype)
+  assert.deepEqual(result, { target_faces: 10000 })
+})
+
+test('target is never mutated by binding resolution', () => {
+  const target = Object.freeze({ target_faces: 1000000 })
+  const result = resolveBoundWorkflowParams(workflow, 'optimize', target, () => ({ faces: 30000 }))
+  assert.equal(target.target_faces, 1000000)
+  assert.equal(result.target_faces, 30000)
+})
diff --git a/src/areas/workflows/workflowParamBindings.ts b/src/areas/workflows/workflowParamBindings.ts
new file mode 100644
index 00000000..c8fc0bb5
--- /dev/null
+++ b/src/areas/workflows/workflowParamBindings.ts
@@ -0,0 +1,56 @@
+import type { Workflow } from '../../shared/types/electron.d'
+
+type ReadNodeParams = (nodeId: string) => Record<string, unknown> | undefined
+
+/** Only plain parameter fields may be used as target keys. */
+function validField(name: string): boolean {
+  return typeof name === 'string'
+    && /^[a-zA-Z_][a-zA-Z0-9_.-]*$/.test(name)
+    && name !== '__proto__'
+    && name !== 'constructor'
+    && name !== 'prototype'
+}
+
+/**
+ * Resolve a target node's parameters at execution time. Binding sources come
+ * from the freshest live map, not a stale snapshot captured when Run began.
+ * Missing or invalid sources leave the target's explicit fallback unchanged.
+ */
+export function resolveBoundWorkflowParams(
+  workflow: Workflow,
+  targetNodeId: string,
+  targetParams: Record<string, unknown>,
+  readNodeParams: ReadNodeParams,
+): Record<string, unknown> {
+  const bindings = workflow.paramBindings?.filter((item) => item.targetNodeId === targetNodeId) ?? []
+  if (bindings.length === 0) return targetParams
+
+  const resolved = { ...targetParams }
+  const sourceNodes = new Set(workflow.nodes.map((node) => node.id))
+  for (const item of bindings) {
+    if (!sourceNodes.has(item.sourceNodeId) || item.sourceNodeId === item.targetNodeId) continue
+    if (!validField(item.sourceParam) || !validField(item.targetParam)) continue
+    const sourceParams = readNodeParams(item.sourceNodeId)
+    if (sourceParams && Object.prototype.hasOwnProperty.call(sourceParams, item.sourceParam)) {
+      resolved[item.targetParam] = sourceParams[item.sourceParam]
+    }
+  }
+  return resolved
+}
+
+/** Hide an operator-facing question only if every parameter has a real source node. */
+export function areAllWorkflowNodeParamsBound(
+  workflow: Workflow,
+  nodeId: string,
+  paramIds: string[],
+): boolean {
+  if (paramIds.length === 0) return false
+  const sourceNodes = new Set(workflow.nodes.map((node) => node.id))
+  const bound = new Set(
+    (workflow.paramBindings ?? [])
+      .filter((item) => item.targetNodeId === nodeId && item.sourceNodeId !== nodeId)
+      .filter((item) => sourceNodes.has(item.sourceNodeId) && validField(item.sourceParam) && validField(item.targetParam))
+      .map((item) => item.targetParam),
+  )
+  return paramIds.every((paramId) => bound.has(paramId))
+}
diff --git a/src/areas/workflows/workflowRunStore.ts b/src/areas/workflows/workflowRunStore.ts
index 1c64c0c5..08773ded 100644
--- a/src/areas/workflows/workflowRunStore.ts
+++ b/src/areas/workflows/workflowRunStore.ts
@@ -8,6 +8,7 @@ import type { WorkflowExtension } from './mockExtensions'
 import type { Workflow, WFNode, WFEdge } from '@shared/types/electron.d'
 import { isBranchStarter, isSceneOutput, resolveDataSource, reachesSceneOutput, nearestUpstreamWaits } from './nodeBehaviors'
 import { assignSlotFilePaths } from './slotInputs'
+import { resolveBoundWorkflowParams } from './workflowParamBindings'
 import type { SlotInputType } from './slotInputs'
 
 // ─── Types ────────────────────────────────────────────────────────────────────
@@ -313,7 +314,13 @@ async function executeExtensionNode(
   }
   // Freshest params at the moment the node starts (so loop iterations / Retry pick
   // up edits made while paused, not the values captured at run start).
-  const liveParams = _liveParams.current.get(node.id) ?? node.data.params ?? {}
+  const nodeParams = _liveParams.current.get(node.id) ?? node.data.params ?? {}
+  const liveParams = resolveBoundWorkflowParams(
+    workflow,
+    node.id,
+    nodeParams,
+    (sourceNodeId) => _liveParams.current.get(sourceNodeId) ?? nodeMap.get(sourceNodeId)?.data.params,
+  )
 
   const resolveSource = (sourceId: string): NodeOutput | undefined => {
     const realId = resolveDataSource(sourceId, workflow.edges, nodeMap)
diff --git a/src/shared/types/electron.d.ts b/src/shared/types/electron.d.ts
index caa0bf33..b1eb6183 100644
--- a/src/shared/types/electron.d.ts
+++ b/src/shared/types/electron.d.ts
@@ -159,12 +159,21 @@ export interface Workflow {
   folder?:     string
   /** Pinned in the workflow browser's Bookmarks section */
   bookmarked?: boolean
+  /** One source choice may feed another node's fallback or quality parameter. */
+  paramBindings?: WorkflowParamBinding[]
   nodes:       WFNode[]
   edges:       WFEdge[]
   createdAt:   string
   updatedAt:   string
 }
 
+export interface WorkflowParamBinding {
+  sourceNodeId: string
+  sourceParam: string
+  targetNodeId: string
+  targetParam: string
+}
+
 declare global {
   interface Window {
     electron: {

From e9ff831ad444721780bd385034329fbcba15272f Mon Sep 17 00:00:00 2001
From: JayceeB1 <45721154+JayceeB1@users.noreply.github.com>
Date: Sat, 10 Oct 2026 00:48:44 +0200
Subject: [PATCH 3/5] integrate(#251): synchronize Library preview, viewer
 animation and responsive popover

Port #251 UI diff with local shared animation store; preserve 0.4.3 point lights and viewer exporter.
---
 src/areas/generate/GeneratePage.tsx        | 121 ++++++++++++++++-----
 src/areas/generate/assetLibraryUi.test.ts  |  39 +++++++
 src/areas/generate/assetLibraryUi.ts       |  70 ++++++++++++
 src/areas/generate/components/Viewer3D.tsx |  11 +-
 src/shared/stores/appStore.ts              |  23 +++-
 5 files changed, 236 insertions(+), 28 deletions(-)

diff --git a/src/areas/generate/GeneratePage.tsx b/src/areas/generate/GeneratePage.tsx
index 89d2a669..6e9b6b8c 100644
--- a/src/areas/generate/GeneratePage.tsx
+++ b/src/areas/generate/GeneratePage.tsx
@@ -1,5 +1,5 @@
-import { useState, useRef, useCallback, useEffect, useMemo } from 'react'
-import type { ReactNode } from 'react'
+import { useState, useRef, useCallback, useEffect, useLayoutEffect, useMemo } from 'react'
+import type { ReactNode, RefObject } from 'react'
 import { useAppStore, DEFAULT_LIGHT_SETTINGS } from '@shared/stores/appStore'
 import type { GenerationJob, LightSettings, PointLight } from '@shared/stores/appStore'
 import { useApi } from '@shared/hooks/useApi'
@@ -22,7 +22,9 @@ import {
   createDefaultAssetLibraryFilters,
   createAssetLibraryOpenJob,
   describeAssetLibraryOpenability,
+  findAssetLibraryMotionClipIndex,
   formatAssetLibraryClipName,
+  getAssetLibraryPanelLayout,
   getDefaultAssetLibraryCollapsedSectionKeys,
   getStoredAssetLibraryPanelHeight,
   getStoredAssetLibraryPanelWidth,
@@ -516,7 +518,11 @@ function AssetLibraryPopover({
   previews,
   panelWidth,
   panelHeight,
+  anchorRef,
+  viewerEntryId,
+  viewerClipName,
   onActivateEntry,
+  onSelectViewerClip,
   onSearchQueryChange,
   onSortModeChange,
   onViewModeChange,
@@ -544,8 +550,12 @@ function AssetLibraryPopover({
   previews: Record<string, AssetLibraryPreviewClip[]>
   panelWidth: number
   panelHeight: number
+  anchorRef: RefObject<HTMLDivElement | null>
+  viewerEntryId: string | null
+  viewerClipName: string | null
   /** A row click both selects and, when the entry is openable, opens it immediately — no second confirming click. */
   onActivateEntry: (entryId: string) => void
+  onSelectViewerClip: (entryId: string, clip: string) => void
   onSearchQueryChange: (value: string) => void
   onSortModeChange: (value: AssetLibrarySortMode) => void
   onViewModeChange: (value: AssetLibraryViewMode) => void
@@ -563,16 +573,44 @@ function AssetLibraryPopover({
   const hasActiveDiscovery = normalizedSearchQuery.length > 0 || hasActiveFilters
   const familyCount = projectGroups.reduce((count, group) => count + group.families.length, 0)
   const versionCount = projectGroups.reduce((count, group) => count + group.entryCount, 0)
+  const [layout, setLayout] = useState<ReturnType<typeof getAssetLibraryPanelLayout> | null>(null)
+
+  const updateLayout = useCallback(() => {
+    const anchor = anchorRef.current?.getBoundingClientRect()
+    if (!anchor) return
+    setLayout(getAssetLibraryPanelLayout(
+      anchor,
+      { width: window.innerWidth, height: window.innerHeight },
+      panelWidth,
+      panelHeight,
+    ))
+  }, [anchorRef, panelHeight, panelWidth])
+
+  useLayoutEffect(() => {
+    updateLayout()
+    window.addEventListener('resize', updateLayout)
+    window.visualViewport?.addEventListener('resize', updateLayout)
+    const observer = new ResizeObserver(updateLayout)
+    observer.observe(document.documentElement)
+    return () => {
+      window.removeEventListener('resize', updateLayout)
+      window.visualViewport?.removeEventListener('resize', updateLayout)
+      observer.disconnect()
+    }
+  }, [updateLayout])
 
   return (
     <div
       role="dialog"
       aria-label="Workspace library"
-      style={{ width: panelWidth, height: panelHeight }}
-      className="absolute top-full left-0 mt-1 z-50 max-w-[calc(100vw-2rem)] max-h-[calc(100vh-4rem)] bg-zinc-900 border border-zinc-700/60 rounded-xl p-3 flex flex-col gap-3 shadow-xl overflow-hidden"
+      style={layout
+        ? { left: layout.left, top: layout.top, width: layout.width, height: layout.height }
+        : { visibility: 'hidden' }
+      }
+      className="fixed z-50 bg-zinc-900 border border-zinc-700/60 rounded-xl p-3 flex flex-col gap-3 shadow-xl overflow-y-auto overflow-x-hidden"
     >
-      <div className="flex items-center justify-between gap-2 shrink-0">
-        <div>
+      <div className="flex flex-wrap items-center justify-between gap-2 shrink-0">
+        <div className="min-w-0">
           <p className="text-[10px] text-zinc-500 uppercase tracking-wider">Asset Library</p>
           <p className="text-xs text-zinc-300">Browse models by project, traits, and version family.</p>
         </div>
@@ -601,7 +639,7 @@ function AssetLibraryPopover({
         role="separator"
         aria-orientation="vertical"
         aria-label="Resize workspace library panel width"
-        className="absolute top-0 right-0 bottom-0 w-1.5 cursor-col-resize hover:bg-violet-400/40 active:bg-violet-400/60 transition-colors rounded-r-xl"
+        className="absolute top-0 right-0 bottom-0 z-20 w-1.5 cursor-col-resize hover:bg-violet-400/40 active:bg-violet-400/60 transition-colors rounded-r-xl"
       />
 
       {/* Resize handle — drag to grow/shrink the library panel; size is persisted. */}
@@ -610,11 +648,11 @@ function AssetLibraryPopover({
         role="separator"
         aria-orientation="horizontal"
         aria-label="Resize workspace library panel height"
-        className="absolute bottom-0 left-0 right-0 h-1.5 cursor-row-resize hover:bg-violet-400/40 active:bg-violet-400/60 transition-colors rounded-b-xl"
+        className="absolute bottom-0 left-0 right-0 z-20 h-1.5 cursor-row-resize hover:bg-violet-400/40 active:bg-violet-400/60 transition-colors rounded-b-xl"
       />
 
-      <div className="flex items-end gap-2 shrink-0">
-        <div className="flex min-w-0 flex-1 flex-col gap-1.5">
+      <div className="flex flex-wrap items-end gap-2 shrink-0">
+        <div className="flex min-w-[min(100%,16rem)] flex-1 flex-col gap-1.5">
           <label htmlFor="asset-library-search" className="text-[11px] text-zinc-300">Search assets</label>
           <input
             id="asset-library-search"
@@ -625,7 +663,7 @@ function AssetLibraryPopover({
             className="appearance-none bg-zinc-800 border border-zinc-700 rounded-lg px-2.5 py-1.5 text-xs text-zinc-200 w-full focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-violet-400"
           />
         </div>
-        <div className="flex w-36 shrink-0 flex-col gap-1.5">
+        <div className="flex w-36 max-w-full shrink-0 flex-col gap-1.5">
           <label htmlFor="asset-library-sort" className="text-[11px] text-zinc-300">Sort</label>
           <select
             id="asset-library-sort"
@@ -713,16 +751,16 @@ function AssetLibraryPopover({
       )}
 
       {loading ? (
-        <p role="status" className="flex-1 min-h-0 flex items-center justify-center text-xs text-zinc-400">Loading workspace assets…</p>
+        <p role="status" className="flex min-h-40 items-center justify-center text-xs text-zinc-400">Loading workspace assets…</p>
       ) : projectGroups.length === 0 && !hasActiveDiscovery ? (
-        <p role="status" className="flex-1 min-h-0 flex items-center justify-center text-xs text-zinc-500">No workspace assets are indexed yet.</p>
+        <p role="status" className="flex min-h-40 items-center justify-center text-xs text-zinc-500">No workspace assets are indexed yet.</p>
       ) : projectGroups.length === 0 ? (
-        <p role="status" className="flex-1 min-h-0 flex items-center justify-center text-center text-xs text-zinc-500">
+        <p role="status" className="flex min-h-40 items-center justify-center text-center text-xs text-zinc-500">
           No assets match these search and filter choices.
         </p>
       ) : (
-        <div className="flex-1 min-h-0 overflow-y-auto overflow-x-hidden rounded-lg border border-zinc-800 bg-zinc-950/40">
-          <div className="sticky top-0 z-10 flex items-center justify-between border-b border-zinc-800 bg-zinc-950/95 px-3 py-2 backdrop-blur">
+        <div className="shrink-0 rounded-lg border border-zinc-800 bg-zinc-950/40">
+          <div className="sticky top-0 z-10 flex flex-wrap items-center justify-between gap-1 border-b border-zinc-800 bg-zinc-950/95 px-3 py-2 backdrop-blur">
             <span className="text-[10px] uppercase tracking-wider text-zinc-500">Grouped by project</span>
             <span className="text-[10px] text-zinc-500">{familyCount} families · {versionCount} versions</span>
           </div>
@@ -737,7 +775,7 @@ function AssetLibraryPopover({
                   aria-controls={projectRegionId}
                   onClick={() => onToggleSection(projectGroup.sectionKey)}
                   disabled={hasActiveDiscovery}
-                  className="flex w-full items-center justify-between gap-3 bg-zinc-900/80 px-3 py-2.5 text-left focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-violet-400 disabled:pointer-events-none"
+                  className="flex w-full flex-wrap items-center justify-between gap-2 bg-zinc-900/80 px-3 py-2.5 text-left focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-violet-400 disabled:pointer-events-none"
                 >
                   <span className="flex items-center gap-2">
                     <span className={`text-xs font-semibold ${projectGroup.projectKey === 'needs-project' ? 'text-amber-200' : 'text-zinc-200'}`}>
@@ -757,7 +795,7 @@ function AssetLibraryPopover({
                     role="list"
                     aria-label={`${projectGroup.projectLabel} asset families`}
                     className={viewMode === 'gallery'
-                      ? 'grid grid-cols-[repeat(auto-fill,minmax(210px,1fr))] gap-3 p-3'
+                      ? 'grid grid-cols-[repeat(auto-fill,minmax(min(100%,13rem),1fr))] gap-3 p-3'
                       : 'divide-y divide-zinc-800'
                     }
                   >
@@ -770,7 +808,10 @@ function AssetLibraryPopover({
                         disabled={opening}
                         thumbnails={thumbnails}
                         previews={previews}
+                        viewerEntryId={viewerEntryId}
+                        viewerClipName={viewerClipName}
                         onActivateEntry={onActivateEntry}
+                        onSelectViewerClip={onSelectViewerClip}
                         onFetchPreviewFrame={onFetchPreviewFrame}
                       />
                     ))}
@@ -797,7 +838,10 @@ function AssetLibraryFamilyCard({
   disabled,
   thumbnails,
   previews,
+  viewerEntryId,
+  viewerClipName,
   onActivateEntry,
+  onSelectViewerClip,
   onFetchPreviewFrame,
 }: {
   family: AssetLibraryLineageFamily
@@ -806,7 +850,10 @@ function AssetLibraryFamilyCard({
   disabled: boolean
   thumbnails: Record<string, string>
   previews: Record<string, AssetLibraryPreviewClip[]>
+  viewerEntryId: string | null
+  viewerClipName: string | null
   onActivateEntry: (entryId: string) => void
+  onSelectViewerClip: (entryId: string, clip: string) => void
   onFetchPreviewFrame: (workspacePath: string, clip: string) => Promise<string | null>
 }) {
   const entry = family.primaryEntry
@@ -814,13 +861,19 @@ function AssetLibraryFamilyCard({
   const previewClips = previews[entry.workspacePath] ?? EMPTY_PREVIEW_CLIPS
   const selected = family.entries.some((candidate) => candidate.id === selectedEntryId)
   const [hovered, setHovered] = useState(false)
-  const [clipIndex, setClipIndex] = useState(0)
+  const [localClipIndex, setLocalClipIndex] = useState(0)
   const [previewFrames, setPreviewFrames] = useState<Record<string, string>>({})
   const [versionsExpanded, setVersionsExpanded] = useState(false)
   const mountedRef = useRef(true)
   useEffect(() => () => { mountedRef.current = false }, [])
 
-  const activeClip = previewClips.length > 0 ? previewClips[clipIndex % previewClips.length] : undefined
+  const viewerPreviewClipIndex = viewerClipName
+    ? findAssetLibraryMotionClipIndex(previewClips.map((clip) => ({ name: clip.clip })), viewerClipName)
+    : -1
+  const clipIndex = viewerEntryId === entry.id && viewerPreviewClipIndex >= 0
+    ? viewerPreviewClipIndex
+    : localClipIndex % Math.max(1, previewClips.length)
+  const activeClip = previewClips[clipIndex]
   const activeFrameUrl = activeClip ? previewFrames[activeClip.clip] : undefined
 
   // Gallery motion is the feature, so it begins without requiring a hover.
@@ -837,7 +890,10 @@ function AssetLibraryFamilyCard({
   function cycleClip(step: 1 | -1, event: React.MouseEvent | React.KeyboardEvent) {
     event.preventDefault()
     event.stopPropagation()
-    setClipIndex((current) => (current + step + previewClips.length) % previewClips.length)
+    if (previewClips.length === 0) return
+    const nextIndex = (clipIndex + step + previewClips.length) % previewClips.length
+    setLocalClipIndex(nextIndex)
+    onSelectViewerClip(entry.id, previewClips[nextIndex].clip)
   }
 
   function handleCycleKeyDown(step: 1 | -1, event: React.KeyboardEvent) {
@@ -917,7 +973,7 @@ function AssetLibraryFamilyCard({
           selected ? 'bg-violet-500/10' : 'hover:bg-zinc-800/70'
         } ${disabled ? 'pointer-events-none opacity-50' : ''}`}
       >
-        <div className="flex items-center gap-3">
+        <div className="flex flex-wrap items-center gap-3">
           <div className="relative h-14 w-14 shrink-0 overflow-hidden rounded-lg border border-zinc-700 bg-zinc-950">
             {displayedThumbnailUrl ? (
               <img src={displayedThumbnailUrl} alt="" className="h-full w-full object-contain" />
@@ -925,7 +981,7 @@ function AssetLibraryFamilyCard({
               <AssetLibraryPreviewPlaceholder />
             )}
           </div>
-          <div className="min-w-0 flex-1">
+          <div className="min-w-[12rem] flex-1">
             <div className="flex items-center gap-2">
               <span className="truncate text-xs font-medium text-zinc-200">{family.name}</span>
               {family.entries.length > 1 && (
@@ -941,7 +997,7 @@ function AssetLibraryFamilyCard({
             </div>
             <p className="mt-1 truncate text-[9px] text-zinc-500" title={entry.workspacePath}>{entry.workspacePath}</p>
           </div>
-          <div className="w-40 shrink-0">{clipControls}</div>
+          <div className="w-40 max-w-full shrink-0">{clipControls}</div>
           <span className="w-16 shrink-0 text-right text-[9px] text-zinc-500">{dateLabel}</span>
           {family.entries.length > 1 && (
             <button
@@ -1114,6 +1170,7 @@ export default function GeneratePage(): JSX.Element {
   const dragging = useRef(false)
   const libraryPanelDragging = useRef(false)
   const libraryPanelHeightDragging = useRef(false)
+  const libraryAnchorRef = useRef<HTMLDivElement | null>(null)
   const [selectedPointLightId, setSelectedPointLightId] = useState<string | null>(null)
   // Populated by Viewer3D — undoes the latest live gizmo transform, if any.
   const gizmoUndoRef = useRef<(() => boolean) | null>(null)
@@ -1134,6 +1191,9 @@ export default function GeneratePage(): JSX.Element {
   const setCurrentJob = useAppStore((s) => s.setCurrentJob)
   const meshStats = useAppStore((s) => s.meshStats)
   const meshSelected = useAppStore((s) => s.meshSelected)
+  const motionClips = useAppStore((s) => s.motionClips)
+  const activeClipIndex = useAppStore((s) => s.activeClipIndex)
+  const setActiveClipIndex = useAppStore((s) => s.setActiveClipIndex)
   const pushMeshUrl = useAppStore((s) => s.pushMeshUrl)
   const undoMesh = useAppStore((s) => s.undoMesh)
   const redoMesh = useAppStore((s) => s.redoMesh)
@@ -1141,6 +1201,7 @@ export default function GeneratePage(): JSX.Element {
   const canRedo = useAppStore((s) => s.historyIndex < s.meshHistory.length - 1)
   const { optimizeMesh, smoothMesh, importMesh } = useApi()
   const assetLibraryService = useMemo(() => getDefaultAssetLibraryService(), [])
+  const viewerClipName = motionClips[activeClipIndex]?.name ?? null
 
   useEffect(() => {
     const handler = (e: KeyboardEvent) => {
@@ -1419,6 +1480,12 @@ export default function GeneratePage(): JSX.Element {
     await openLibraryEntry(entry)
   }
 
+  function handleSelectLibraryViewerClip(entryId: string, clip: string) {
+    if (currentJob?.libraryEntryId !== entryId) return
+    const viewerIndex = findAssetLibraryMotionClipIndex(motionClips, clip)
+    if (viewerIndex >= 0) setActiveClipIndex(viewerIndex)
+  }
+
   async function handleSmooth(iterations: number) {
     if (!currentJob?.outputUrl) return
     setSmoothing(true)
@@ -1610,7 +1677,7 @@ export default function GeneratePage(): JSX.Element {
             )}
           </div>
 
-          <div className="relative">
+          <div ref={libraryAnchorRef} className="relative">
             <AssetLibraryToggleButton
               open={openPanel === 'library'}
               disabled={importing || libraryOpening}
@@ -1635,7 +1702,11 @@ export default function GeneratePage(): JSX.Element {
                 previews={libraryPreviews}
                 panelWidth={libraryPanelWidth}
                 panelHeight={libraryPanelHeight}
+                anchorRef={libraryAnchorRef}
+                viewerEntryId={currentJob?.libraryEntryId ?? null}
+                viewerClipName={viewerClipName}
                 onActivateEntry={(entryId) => { void handleActivateLibraryEntry(entryId) }}
+                onSelectViewerClip={handleSelectLibraryViewerClip}
                 onSearchQueryChange={setLibrarySearchQuery}
                 onSortModeChange={setLibrarySortMode}
                 onViewModeChange={setLibraryViewMode}
diff --git a/src/areas/generate/assetLibraryUi.test.ts b/src/areas/generate/assetLibraryUi.test.ts
index 145ab6be..db433b1e 100644
--- a/src/areas/generate/assetLibraryUi.test.ts
+++ b/src/areas/generate/assetLibraryUi.test.ts
@@ -9,7 +9,9 @@ import {
   createAssetLibraryOpenJob,
   createDefaultAssetLibraryFilters,
   describeAssetLibraryOpenability,
+  findAssetLibraryMotionClipIndex,
   formatAssetLibraryClipName,
+  getAssetLibraryPanelLayout,
   getDefaultAssetLibraryCollapsedSectionKeys,
   getStoredAssetLibraryPanelHeight,
   getStoredAssetLibraryPanelWidth,
@@ -158,6 +160,8 @@ test('opens safe Default, Workflows, and Exports models through existing Generat
   const selection = createAssetLibraryOpenJob(glb, target, 1718546400000)
   assert.equal(selection.historyUrl, '/workspace/Default/hero.glb')
   assert.equal(selection.job.status, 'done')
+  assert.equal(selection.job.libraryEntryId, glb.id)
+  assert.equal(selection.job.libraryWorkspacePath, glb.workspacePath)
   assert.equal(resolveOpenPanelAfterLibrarySelection('library' satisfies GenerateOpenPanel), 'library')
 })
 
@@ -194,6 +198,39 @@ test('Library panel clamps to gallery-safe bounds and degrades silently without
   assert.doesNotThrow(() => storeAssetLibraryPanelHeight(500))
 })
 
+test('Library panel shifts, shrinks, and flips to stay inside zoomed viewports', () => {
+  const zoomed = getAssetLibraryPanelLayout(
+    { left: 420, right: 500, top: 40, bottom: 70 },
+    { width: 640, height: 430 },
+    880,
+    900,
+  )
+  assert.deepEqual(zoomed, {
+    left: 12,
+    top: 74,
+    width: 616,
+    height: 344,
+    placement: 'below',
+  })
+  assert.ok(zoomed.left >= 0)
+  assert.ok(zoomed.left + zoomed.width <= 640)
+  assert.ok(zoomed.top + zoomed.height <= 430)
+
+  const flipped = getAssetLibraryPanelLayout(
+    { left: 700, right: 780, top: 700, bottom: 730 },
+    { width: 1000, height: 800 },
+    560,
+    600,
+  )
+  assert.deepEqual(flipped, {
+    left: 428,
+    top: 96,
+    width: 560,
+    height: 600,
+    placement: 'above',
+  })
+})
+
 test('live discovery expands matching projects without overwriting collapse state', () => {
   const collapsed = ['project:adventure']
   assert.equal(isAssetLibrarySectionExpanded(collapsed, 'project:adventure', false), false)
@@ -204,4 +241,6 @@ test('live discovery expands matching projects without overwriting collapse stat
 test('formats machine clip names as legible animation names', () => {
   assert.equal(formatAssetLibraryClipName('slow_turn_glance'), 'Slow Turn Glance')
   assert.equal(formatAssetLibraryClipName('RigAction'), 'Rig Action')
+  assert.equal(findAssetLibraryMotionClipIndex([{ name: 'Idle' }, { name: 'slow-turn_glance' }], 'slow_turn_glance'), 1)
+  assert.equal(findAssetLibraryMotionClipIndex([{ name: 'Idle' }], 'Walk'), -1)
 })
diff --git a/src/areas/generate/assetLibraryUi.ts b/src/areas/generate/assetLibraryUi.ts
index bd07fc35..0b15a2d9 100644
--- a/src/areas/generate/assetLibraryUi.ts
+++ b/src/areas/generate/assetLibraryUi.ts
@@ -208,6 +208,8 @@ export function createAssetLibraryOpenJob(
       progress: 100,
       outputUrl: target.url,
       originalOutputUrl: target.url,
+      libraryEntryId: entry.id,
+      libraryWorkspacePath: entry.workspacePath,
       createdAt: now,
     },
   }
@@ -273,6 +275,74 @@ export function storeAssetLibraryPanelHeight(height: number): void {
   }
 }
 
+export interface AssetLibraryAnchorRect {
+  left: number
+  right: number
+  top: number
+  bottom: number
+}
+
+export interface AssetLibraryViewport {
+  width: number
+  height: number
+}
+
+export interface AssetLibraryPanelLayout {
+  left: number
+  top: number
+  width: number
+  height: number
+  placement: 'above' | 'below'
+}
+
+/**
+ * Fits the popover into the renderer's CSS viewport. Chromium shrinks that
+ * viewport as whole-window zoom increases, so saved pixel preferences remain
+ * preferences rather than unreachable off-screen dimensions.
+ */
+export function getAssetLibraryPanelLayout(
+  anchor: AssetLibraryAnchorRect,
+  viewport: AssetLibraryViewport,
+  preferredWidth: number,
+  preferredHeight: number,
+  margin = 12,
+  gap = 4,
+): AssetLibraryPanelLayout {
+  const availableWidth = Math.max(0, viewport.width - margin * 2)
+  const width = Math.min(preferredWidth, availableWidth)
+  const left = Math.min(
+    Math.max(margin, anchor.left),
+    Math.max(margin, viewport.width - margin - width),
+  )
+
+  const belowSpace = Math.max(0, viewport.height - margin - anchor.bottom - gap)
+  const aboveSpace = Math.max(0, anchor.top - gap - margin)
+  const minimumUsefulHeight = Math.min(preferredHeight, ASSET_LIBRARY_PANEL_MIN_HEIGHT)
+  const placement = belowSpace < minimumUsefulHeight && aboveSpace > belowSpace ? 'above' : 'below'
+  const availableHeight = placement === 'below' ? belowSpace : aboveSpace
+  const height = Math.min(preferredHeight, availableHeight)
+  const top = placement === 'below'
+    ? anchor.bottom + gap
+    : Math.max(margin, anchor.top - gap - height)
+
+  return { left, top, width, height, placement }
+}
+
+function normalizeClipIdentity(name: string): string {
+  return name.trim().toLocaleLowerCase().replace(/[^a-z0-9]+/g, '')
+}
+
+/** Matches preview-manifest clip names to the viewer's glTF animation list. */
+export function findAssetLibraryMotionClipIndex(
+  clips: Array<{ name: string }>,
+  previewClip: string,
+): number {
+  const exact = clips.findIndex((clip) => clip.name === previewClip)
+  if (exact >= 0) return exact
+  const normalized = normalizeClipIdentity(previewClip)
+  return clips.findIndex((clip) => normalizeClipIdentity(clip.name) === normalized)
+}
+
 function compareAssetLibraryProjectGroups(left: AssetLibraryProjectGroup, right: AssetLibraryProjectGroup): number {
   if (left.projectKey === NEEDS_PROJECT_KEY && right.projectKey !== NEEDS_PROJECT_KEY) return 1
   if (right.projectKey === NEEDS_PROJECT_KEY && left.projectKey !== NEEDS_PROJECT_KEY) return -1
diff --git a/src/areas/generate/components/Viewer3D.tsx b/src/areas/generate/components/Viewer3D.tsx
index 0624b939..40ba726e 100644
--- a/src/areas/generate/components/Viewer3D.tsx
+++ b/src/areas/generate/components/Viewer3D.tsx
@@ -1064,8 +1064,15 @@ export default function Viewer3D({
 
   const [viewMode, setViewMode] = useState<ViewMode>('solid')
   const [autoRotate, setAutoRotate] = useState(false)
-  const [clips, setClips] = useState<MotionClip[]>([])
-  const [activeClipIndex, setActiveClipIndex] = useState(0)
+  // The viewer and Library share one clip selection in the application store.
+  const clips = useAppStore((state) => state.motionClips)
+  const activeClipIndex = useAppStore((state) => state.activeClipIndex)
+  const setStoreMotionClips = useAppStore((state) => state.setMotionClips)
+  const setActiveClipIndex = useAppStore((state) => state.setActiveClipIndex)
+  const setClips = useCallback((nextClips: MotionClip[]) => {
+    setStoreMotionClips(nextClips)
+    if (useAppStore.getState().activeClipIndex >= nextClips.length) setActiveClipIndex(0)
+  }, [setStoreMotionClips, setActiveClipIndex])
   const selected = useAppStore((s) => s.meshSelected)
   const setSelected = useAppStore((s) => s.setMeshSelected)
   const canvasRef = useRef<HTMLCanvasElement | null>(null)
diff --git a/src/shared/stores/appStore.ts b/src/shared/stores/appStore.ts
index 03b8d5fb..dc193655 100644
--- a/src/shared/stores/appStore.ts
+++ b/src/shared/stores/appStore.ts
@@ -25,6 +25,9 @@ export interface GenerationJob {
   modelId?: string             // model used for this generation
   originalTriangles?: number   // polygon count of the original mesh
   generationOptions?: GenerationOptions
+  /** Stable Library identity for the asset currently loaded in the viewer. */
+  libraryEntryId?: string
+  libraryWorkspacePath?: string
   error?: string
   createdAt: number
 }
@@ -104,6 +107,12 @@ interface AppState {
   meshStats: { vertices: number; triangles: number } | null
   setMeshStats: (stats: { vertices: number; triangles: number } | null) => void
 
+  // Shared animation state used by the viewer's MotionBar and Library preview.
+  motionClips: Array<{ name: string }>
+  setMotionClips: (clips: Array<{ name: string }>) => void
+  activeClipIndex: number
+  setActiveClipIndex: (index: number) => void
+
   // Mesh selection (set by Viewer3D click, read by the Generate tools bar)
   meshSelected: boolean
   setMeshSelected: (selected: boolean) => void
@@ -275,6 +284,10 @@ export const useAppStore = create<AppState>()(
       generationOptions: DEFAULT_OPTIONS,
       meshStats: null,
       setMeshStats: (stats) => set({ meshStats: stats }),
+      motionClips: [],
+      setMotionClips: (clips) => set({ motionClips: clips }),
+      activeClipIndex: 0,
+      setActiveClipIndex: (index) => set({ activeClipIndex: index }),
       meshSelected: false,
       setMeshSelected: (selected) => set({ meshSelected: selected }),
       initApp: async () => {
@@ -299,7 +312,15 @@ export const useAppStore = create<AppState>()(
         }
       },
 
-      setCurrentJob: (job) => set({ currentJob: job, meshStats: job === null ? null : get().meshStats }),
+      setCurrentJob: (job) => {
+        const changed = job?.id !== get().currentJob?.id
+        set({
+          currentJob: job,
+          meshStats: job === null ? null : get().meshStats,
+          motionClips: changed ? [] : get().motionClips,
+          activeClipIndex: changed ? 0 : get().activeClipIndex,
+        })
+      },
 
       updateCurrentJob: (patch) => {
         const current = get().currentJob

From 632243f40f8afd1ec6813302e786ad4df27d9343 Mon Sep 17 00:00:00 2001
From: JayceeB1 <45721154+JayceeB1@users.noreply.github.com>
Date: Sat, 10 Oct 2026 00:49:14 +0200
Subject: [PATCH 4/5] docs(integration): record #250/#251 port and isolated
 test evidence

---
 ...experimental-library-integration-ledger.md | 20 +++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/docs/experimental-library-integration-ledger.md b/docs/experimental-library-integration-ledger.md
index fd4fad9a..ade9b93d 100644
--- a/docs/experimental-library-integration-ledger.md
+++ b/docs/experimental-library-integration-ledger.md
@@ -24,12 +24,16 @@ Modly repository. The adaptations in this fork do not supersede their source PRs
 - **#246 (Chat + asset context):** its old Ollama agent backend does not match
   current 0.4.3 llama.cpp/chat architecture. The workflow runner input routing
   also changed. Do not paste the old agent router or silently drop context.
-- **#250 (Optimizer preset semantics):** the source PR targets a legacy
-  `processor.ts`, whereas 0.4.3 uses a Python workflow adapter feeding a shared
-  mesh-op registry. A "full detail/no simplification" preset needs a reviewed
-  end-to-end contract that does not alter unrelated direct mesh API calls.
-- **#251 (Workflow parameter binding and library sync):** stack depends on the
-  #250 behavior. Port only after defining stable override precedence and tests.
+- **#250** is now staged in child branch `integration/optimizer-presets-2026-10-10`.
+  The current Python workflow adapter implements full-detail pass-through without
+  changing the shared mesh-op API, and normalizes numeric budgets from HTML selects.
+  Four regression tests were added to the project's Python suite; five isolated
+  Python checks passed on a reproduction of the changed adapter.
+- **#251** is now staged on the same child branch in two focused commits:
+  freshest-value parameter bindings, plus shared Library/viewer animation state
+  and responsive Library placement. Seven isolated Node tests passed for the
+  binding helper, including orphaned/self references and prototype-safe keys.
+  Native Electron/browser smoke tests and combined project CI remain unrun.
 - **#252 (Artist intent record and texture hot reload):** much larger divergent
   patch (workflow and API changes, new watcher services, extra proof assets).
   Requires separate rebase and validation against 0.4.3 and #246/#251.
@@ -45,6 +49,10 @@ Modly repository. The adaptations in this fork do not supersede their source PRs
 - [ ] Check Library thumbnails, previews, project/lineage filters and persistent
   panel size across restart, with missing/corrupt metadata.
 - [ ] Export rigged GLB/OBJ/STL/PLY, verify transforms/tags and project lineage.
+- [ ] Verify binding source parameter updates propagate through actual paused
+  and repeated workflows without hiding missing or invalid source questions.
+- [ ] Verify 3D motion-bar and Library previews remain synchronized, including
+  changing models and project/lineage views.
 - [ ] Retry free-text params and object-shaped manifest inputs.
 - [ ] Verify source #370 destructive-storage safeguards and related tests in
   the final combined branch.

From a3da3bf65e89de0a67328df9425e84ce917c65c9 Mon Sep 17 00:00:00 2001
From: JayceeB1 <45721154+JayceeB1@users.noreply.github.com>
Date: Sat, 10 Oct 2026 01:16:25 +0200
Subject: [PATCH 5/5] test(mesh): keep direct API schema distinct from workflow
 quality presets

---
 api/tests/test_mesh_ops_registry.py | 21 +++++++++++++++++----
 1 file changed, 17 insertions(+), 4 deletions(-)

diff --git a/api/tests/test_mesh_ops_registry.py b/api/tests/test_mesh_ops_registry.py
index 31a777a2..c0349fc3 100644
--- a/api/tests/test_mesh_ops_registry.py
+++ b/api/tests/test_mesh_ops_registry.py
@@ -177,10 +177,23 @@ def test_workflow_manifests_share_registry_schemas_and_thin_adapters(self) -> No
                 (extension_dir / "manifest.json").read_text(encoding="utf-8")
             )
             self.assertEqual(manifest["entry"], "processor.py")
-            self.assertEqual(
-                manifest["nodes"][0]["params_schema"],
-                descriptions[operation_id]["params_schema"],
-            )
+            workflow_schema = manifest["nodes"][0]["params_schema"]
+            api_schema = descriptions[operation_id]["params_schema"]
+            if operation_id == "decimate":
+                # A workflow-only quality selector now maps to the unchanged
+                # shared mesh-op API (numeric target_faces with old defaults).
+                field = workflow_schema[0]
+                self.assertEqual(field["id"], "target_faces")
+                self.assertEqual(field["type"], "select")
+                self.assertEqual(field["default"], 1000000)
+                self.assertEqual(
+                    [option["value"] for option in field["options"]],
+                    [1000000, 12000, 30000, 60000, 120000],
+                )
+                self.assertEqual(api_schema[0]["type"], "int")
+                self.assertEqual(api_schema[0]["default"], 10000)
+            else:
+                self.assertEqual(workflow_schema, api_schema)
 
             wrapper = (extension_dir / "processor.py").read_text(encoding="utf-8")
             self.assertIn(