feat(studio): glue API coexistence layer for the NLE swap

[ukimsanov]

What

extends 21 glue files so the OLD timeline/canvas engine and the NEW NLE components type-check side by side: playerStore (multi-select setters, zoom pin, snap toggle, non-reactive scale scratch), drag-state types gain optional NLE fields, timelineLayout/timelineAssetDrop/timelineEditingHelpers/ timelineEditing/timelineElementHelpers/studioHelpers/assetHelpers gain the NLE exports, DomEditOverlay + gestures + AssetContextMenu + Timeline props gain optional callbacks/params, contexts gain *Optional hooks, and TimelineEditCallbacks.onMoveElements becomes a bivariant method accepting both engines' change shapes. patchDocumentRootDuration's test rides along.

Why

this is the keystone that dissolves the old "welded glue" problem — every symbol the NLE components need is ADDED next to what the old engine still uses, so the engine components and the swaps can land as separate reviewable PRs.

How

15 authored intermediate files (main content + additive symbols; no behavior changes — new fields optional, new callbacks unused until wired) plus 6 files whose final content is already purely additive. New exports without consumers yet carry TEMP(studio-dnd) ignoreExports entries, removed by the app-shell swap.

Test plan

tsc --noEmit in studio + studio-server (verifies BOTH engines compile); bunx vitest run (full suite green incl. the 6 new patchDocumentRootDuration tests); fallow audit clean.

---

Stack position 14/25 — studio NLE overhaul (CapCut-parity timeline + canvas). Graphite manages bases; merge from #2192 upward. Temporary TEMP(studio-dnd) fallow entries (unwired-component windows) are all removed by #2213 (app-shell swap), whose tree is byte-identical to the fully-verified integration branch.

[ukimsanov]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• feat(studio): assets/blocks panel behaviors + preview helpers #2216
• feat(studio): timeline visual refresh #2215
• feat(studio): clip thumbnail modules #2214
• feat(studio): app shell swap — mount EditorShell, retire the legacy engine #2213
• feat(studio): timeline glue swap — Timeline/TimelineCanvas onto the NLE engine #2212
• feat(studio): canvas glue swap — overlay, gestures, selection to final form #2211
• feat(studio): sidebar asset card (unwired) #2210
• feat(studio): NLE shell assembly (unwired) #2209
• feat(studio): canvas chrome components (unwired) #2208
• feat(studio): timeline interaction hooks and lanes component (unwired) #2207
• feat(studio): clip drag engine — preview and commit (unwired) #2206
• feat(studio): glue API coexistence layer for the NLE swap #2205 👈 (View in Graphite)
• feat(studio): NLE shell components — context provider, panes, overlays (unwired) #2204
• feat(studio): arrow-key nudge step math (unwired) #2203
• feat(studio): canvas context menu and z-order actions (unwired) #2202
• test(studio): characterization suites for resize commit and razor history #2201
• feat(studio): asset click policy and canvas nudge gate #2200
• feat(studio): timeline lane-zone model #2199
• feat(studio): timeline z-stacking sync model #2198
• feat(studio): multi-clip drag preview math #2197
• feat(studio): timeline magnetic snapping #2196
• feat(studio): timeline collision and placement model #2195
• feat(studio-server): files route extensions #2194
• feat(studio): seek-restore contract for player reloads #2193
• feat(studio): timeline leaf helpers — audio inspector, zoom math, UI prefs #2192
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[miga-heygen]

Full Stack Review — NLE Overhaul (25 PRs, ~15,600 additions, ~10,100 deletions)

Reviewed the entire stack in four tiers: math/model layer (#2192–#2201), keystone API coexistence (#2205), UI components (#2202–#2210, #2214), and swap/cleanup (#2211–#2216). No blockers across the stack.

---

Keystone: #2205 — glue API coexistence (reviewed hardest)

This PR extends the existing studio APIs to support both the legacy engine and the new NLE simultaneously. It's the hinge between the "build the new" and "swap out the old" phases.

SSOT analysis — strong:

• computeOverlayRootScale extracted from inline code in toOverlayRect — now shared between toOverlayRect and elementCornerOverlayPoints. Single source for iframe→overlay mapping.
• restampManualOffsetDragGestureBase and applyManualOffsetCommitValue extracted from applyManualOffsetDragCommit — enables reuse for the new nudge path without duplicating the re-stamp logic.
• furthestClipEndFromDocument extracted from readTimelineDurationFromDocument — the latter now delegates. Single computation of "furthest clip end." Correctly excludes the root element (prevents feedback loop where root duration is counted as clip content).
• patchDocumentRootDuration correctly finds the TOP-LEVEL root (no ancestor composition) — 6 well-targeted tests.
• Union type on onMoveElements (TimelineGroupMoveChange | TimelineGroupMoveEdit) allows both engines to coexist on the same callback without forking.

Edge case coverage:

• orientedOverlayRect falls back to plain AABB when corners can't be measured — safe degradation, pixel-identical to legacy at angle=0.
• readElementRotationDegrees guards NaN/Infinity on matrix components AND the output.
• resolveReloadSeekTime uses ?? (not ||) so explicit seek-to-0 isn't treated as missing.
• setTimelineScale uses non-reactive mutation to avoid subscriber re-renders — documented as scratch state.

Coexistence hygiene: All TEMP fallow exemptions are marked "removed by studio-dnd/pr22" (#2213). The code-duplication ignores are clearly labeled as coexistence-window clones. #2213 confirms all ~30 exemptions are cleaned up.

One noted SSOT gap: readTimelineElementZIndex contains its own parseZ inline function and the comment says "Mirrors canvasContextMenuZOrder.parseZIndex semantics." Two z-index parsers — documented dependency, different packages. Acceptable.

---

Tier 1: Math/Model Layer (#2192–#2201) — all clean

Heavily unit-tested pure functions (2000+ test lines for ~1200 implementation lines). Each module owns one concern:

• #2192 — log-scale slider, zoom-pin, beat-source. Round-trip tests within 1%.
• #2193 — reload seek-restore. Double-seek guard trick for GSAP re-eval is clever.
• #2194 — batch files route. shift-positions-batch folds N shifts into one write.
• #2195 — collision/placement. Half-open intervals, prefer-up search, insert-row detection. 292 lines of tests.
• #2196 — snapping. Priority dedup (playhead > clip > beat), clamped-snap-drop.
• #2197 — multi-drag preview. Rigid formation, leftmost-at-0 constraint, -0 avoidance.
• #2198 — lane↔z stacking sync. Tie-aware cascade, multi-edit lower-first.
• #2199 — lane-zone model. Constrained packing. The z-to-lane round-trip convergence test is the kind of invariant test that prevents whole classes of regressions.
• #2200 — asset-click + nudge gate. Reference-counted claim/release, idempotent release.
• #2201 — gesture characterization. Center-anchored resize convergence proof + razor-split undo proof.

One nit: overlapsInTime (with epsilon) is copy-pasted identically between timelineStackingSync.ts (#2198) and timelineZones.ts (#2199). Should be a shared import. The exact-vs-epsilon distinction from timeRangesOverlap (#2195) is intentional and defensible.

Tier 2: UI Components (#2202–#2210, #2214) — all clean

Bottom-up build: pure math → state/context → drag engine → interaction hooks → chrome → shell → leaf UI.

• #2202 — z-order menu. Tie-aware DOM-order renumbering. Cross-realm instanceof fix.
• #2203 — canvas nudge. 1px / Shift 10px, composition-px not screen-px.
• #2204 — NLE shell + context. Nit: NLEContext has 25+ fields — future decomposition candidate.
• #2206 — clip-drag engine. Three-phase persist (patch-in-memory, atomic write, preview sync). Rollback on failure. Nit: fire-and-forget error path in persistMoveEdits should document that callers can't observe failure.
• #2207 — timeline hooks + lanes. 40+ prop surface on TimelineLanes — consider grouping.
• #2208 — canvas chrome. Rotation-aware corner handles. Radial resize well-tested. Nit: siblingZIndexEntry silently drops untargetable elements — z patches can be partial.
• #2209 — NLE shell assembly. Sub-comp coordinate rebasing (toLocalElement) applied consistently across all edit paths.
• #2210 — asset card. Click-vs-drag gate via pointer threshold.
• #2214 — clip thumbnails. computeThumbnailStrip + resolveMediaPreviewUrl now shared between image and video. SSOT win.

Tier 3: Swap + Cleanup (#2211–#2216)

• #2211 — canvas glue swap. AABB → OBB resize, nudge, context menu. All old consumers migrated.
• #2212 — timeline glue swap. Stacking-layer → flat-track model. Geometry hook extraction, multi-drag preview, sticky ruler.
• #2213 — app-shell swap + old engine deletion (-5522 lines). All ~30 TEMP fallow exemptions cleaned up. Nit: The old 963-line useTimelineEditing.test.tsx integration suite is deleted without same-scope replacement — the new paths (batched move, multi-delete, content-driven duration) should get integration tests in a follow-up.
• #2215 — visual refresh. Pure CSS. preview-regression CI failure is expected (visual diff from intentional style changes).
• #2216 — assets/blocks panel. Full-bleed media now selectable. Block drag-to-timeline.

---

Cross-Stack Architecture

The stack is disciplined:

1. Pure math first, UI second, swap last. Every decision function is testable in isolation before it's wired into React.
2. Coexistence window is explicit. Fallow exemptions, union types, optional fields — all labeled TEMP with cleanup PR reference.
3. Swap is one-pass. #2213 removes the old engine and all exemptions in one commit. No lingering dead code.
4. 2000+ lines of math tests including the z-to-lane convergence proof (#2199) and the resize oscillation proof (#2201).

Two things to watch post-merge:

1. Integration test coverage for the new batched-move / multi-delete / content-driven-duration paths (gap from #2213's test deletion).
2. The duplicated overlapsInTime + epsilon constant between stacking sync and zone model — worth extracting to a shared module.

CI: 0 failures across all 25 PRs (one preview-regression on #2215 is expected from the visual refresh).

Ship the stack.

— Miga

[jrusso1020]

Re-review — keystone deep dive, two structural NEW findings

Confirmed 6 of 8 prior claims cleanly (SSOT computeOverlayRootScale, root-excluded furthestClipEndFromDocument, NaN guards, safe AABB fallback, 37 TEMP fallow entries matching Via's count exactly + superset-invariant with #2213). Two structural NEW findings the first pass missed:

NEW #1 — DomEditOverlay dead prop surface = the upstream root cause of #2211's phantom z-menu

DomEditOverlay.tsx:90,97 declares onDeleteSelection? and onApplyZIndex? in the props interface (importing ZOrderPatch at :29), but neither is:

• destructured at :101-122,
• stored in any useRef,
• forwarded to createDomEditOverlayGestureHandlers, or
• referenced anywhere in the component body.

Search of the branch shows zero references beyond the interface declaration. So any caller who passes these callbacks today gets a silent no-op — which is exactly the manifestation Abhai flagged at #2211 (context menu writes to iframe DOM but nothing persists; Delete silently no-ops).

Impact: bounded by the wholesale deletion at #2213, but it's a real footgun for any reviewer reading the props contract in isolation. And fixing it here removes the footgun for any future caller that passes these props — cheaper than healing only at #2213.

Fix: destructure + forward both props into the gesture handler bag (~10 LOC).

NEW #2 — triple data-composition-id...data-duration regex fragility

Same regex appears in 3 places across this PR:

• rootDuration.ts:4-6 (extendRootDurationInSource)
• timelineAssetDrop.ts:145 (setCompositionDurationToContent)
• timelineAssetDrop.ts:157 (extendCompositionDurationIfNeeded)

All three: /(<[^>]*data-composition-id="[^"]*"[^>]*data-duration=")([^"]*)(")/

Two failure modes, both silent (return source on no-match):

1. Attribute order dependency: requires data-composition-id BEFORE data-duration in source order. A hand-authored composition with the order swapped silently never extends duration.
2. Double-quotes only: resolveTimelineAssetCompositionSize (same file, :194-195) accepts both quote styles via (["'])([^"']+)\1. A single-quoted composition (unusual but legal HTML5) reads w/h correctly but never extends duration.

Additional divergence: extendRootDurationInSource selects the FIRST [data-composition-id] tag by source text order, while patchDocumentRootDuration (timelineEditingHelpers.ts:622-624) selects by ancestor-closest. For pathological multi-root docs the two functions patch different roots — LIVE DOM and PERSISTED SOURCE end up disagreeing.

Fix: replace all 3 regexes with a single DOM-based patchSourceRootDuration that mirrors patchDocumentRootDuration's ancestry-aware root selection and uses DOMParser/serializer. DOMParser is already imported nearby in the same file (furthestClipEndFromSource). ~40 LOC eliminates the silent-fail class across the whole coexistence window.

Additional nits

• playerStore direct mutation (RDJ's finding stands + goes further): state.timelinePps = pps bypasses set(). Beyond DevTools: useSyncExternalStore snapshots use referential equality on the whole state object, so mutation is invisible to any consumer that reads this via useSyncExternalStore. Documented as non-reactive scratch, but nothing enforces that at the type level.

• onMoveElements union bivariance safety is purely social: no runtime type-guard, no discriminant. Coexistence holds only if the stack merges strictly in order. One out-of-order merge silently breaks group moves.

• 15+ new pure exports, 1 tested: applyTimelineAutoScrollStep, resolveTimelineAutoScrollLoopAction, resolveTimelineDragEscape, getTimelineRowTop, getTimelineRowFromY, getTimelineFitPps, getTimelineDisplayContentWidth, truncateMiddle, formatDuration, readTimelineElementZIndex, furthestClipEndFromSource, setCompositionDurationToContent, extendCompositionDurationIfNeeded, fitTimelineAssetGeometry, resolveTimelineAssetCompositionSize, applyManualOffsetNudgeDraft/Commit. All are pure with clear invariants — trivially unit-testable. Their consumers land later in the stack, but the coexistence layer's own invariants are undefended. Matches Via's #2201 model-bound-tests concern.

None are #2205's own merge blockers if you accept the coexistence framing. But NEW #1 (~10 LOC) is worth fixing here because it kills the root cause of the #2211 phantom-menu bug at the boundary. NEW #2 (~40 LOC) is worth fixing here because it closes a silent-fail class for hand-authored HTML that survives the whole coexistence window.

— Rames Jusso

[ukimsanov]

Second fix wave landed — covering Ular's hands-on feel-test on a real project plus the Slack re-review round (Abhai + Rames). All 25 branches re-passed the full local gate set (tsc, full suite at the 18-failure baseline, verbatim fallow with zero suppressions, format:check, per-branch filesize, builds, exact tip parity).

From the re-review — independently verified before acting:

• feat(studio): clip drag engine — preview and commit (unwired) #2206 dropped durable GSAP shift (Abhai's new blocker): confirmed and fixed. The multi-file soft-reload fix from the previous wave skipped shiftGsapPositionsBatch entirely when a non-active file changed — the shift is a durable server-side script rewrite, so clip timings and tween positions desynced permanently. Now the batch shift runs for every changed group against its own file, followed by one full reload when non-active files changed; the test that pinned the wrong behavior asserts the correct one. Also reverts the optimistic live-iframe timing attrs on a failed write.
• feat(studio): timeline collision and placement model #2195 free-aim mirror (Rames, Abhai-repro'd): confirmed and fixed — the zone-membership check is hoisted above the isLaneFree short-circuit, with free-span repro tests.
• Undo coalesce window (Abhai): fixed — both the move and z-reorder entries now pass coalesceMs: 5000 with the shared gesture key, so the pair folds into one undo step even past the default 300ms window (merge-across-gap test added).
• Phantom z-menu hardening: the context menu now gates every action on its persist handler — no optimistic DOM write without onApplyZIndex, no dead menu items, no menu at all when nothing is actionable. The feat(studio): canvas glue swap — overlay, gestures, selection to final form #2211–feat(studio): app shell swap — mount EditorShell, retire the legacy engine #2213 one-unit merge discipline stands; this closes the class regardless of deploy shape. Belt-and-braces for the feat(studio): timeline glue swap — Timeline/TimelineCanvas onto the NLE engine #2212 seam: timing patches now refuse non-finite values (data-start="NaN" can never reach disk), including in the feat(studio): glue API coexistence layer for the NLE swap #2205 coexistence intermediate.
• Smaller confirmed items: snap indicator no longer vanishes on frame-quantized durations; VideoThumbnail survives a tainted-canvas SecurityError instead of shimmering forever; reload-seek guards a non-finite duration; the nudge identity key includes hfId/selectorIndex; an empty lane-change no longer emits an orphaned z-only undo entry; the NLELayout doc/skill references that would 404 after feat(studio): app shell swap — mount EditorShell, retire the legacy engine #2213 now point at EditorShell; feat(studio): timeline visual refresh #2215's body has been corrected re: its behavioral pieces.

From the hands-on session (fixed on a real project that reproduced them):

• Timeline drops now land on the aimed lane — the display sorts lanes by global z, but the commit only patched z against time-overlapping neighbours, silently discarding the drop intent (root cause of "new track lands somewhere else" and "other clips shuffle"). A drop-intent z nudge on the dragged clip only; neighbours' authored z untouched.
• Razor split, then undo, works — the split was the only server-write path that never resynced the in-memory SDK document, so its next serialize reverted the split and tripped the "File changed outside Studio" guard.
• Undo/redo of style/timing edits restores through the soft path (no more black iframe flash); structural restores keep the full reload.
• Canvas "move to back" no longer rejects id-less elements (?? null → omit; the patch falls through to hfId/selector — the validator was right to reject null).
• The transport total can no longer be dragged below the authored root duration by a shorter clip manifest (the 0:44/0:40 case); time readouts are zero-padded (00:44) everywhere; playback no longer scrolls the timeline to chase the playhead (deliberate UX decision — scrubbing auto-scroll unchanged).

Not acted on (with reasons): Rames's "#2205 dead prop surface" — the props are forwarded at the tip; the finding describes the coexistence intermediate. "#2209 multi-select resize dropped" — restored in the previous wave. The triple duration-regex → DOMParser refactor and the per-member group-resize undo remain on the deferred list, alongside the integration-test suite.

[ukimsanov]

Third wave landed — hands-on regressions fixed + a lane-model change (product decision by Ular).

The previous wave's hands-on testing surfaced two regressions the unit gates missed, plus the root cause of the long-standing drag jank. All were reproduced live in a browser against a real project before fixing, and re-verified the same way after (file diffs as ground truth).

Regressions fixed:

• Marquee collapse — the stale-selection data-loss fix collapsed the marquee's own set when the gesture's async inspector-open notify resolved late. setSelectedElementId gains an explicit preserveSet opt-out, used only when the late-resolving primary is already a member of the live selection; a fresh click on a non-member still collapses (both directions regression-tested).
• Phantom inserts on horizontal drags — the insert gesture armed across ~64% of each lane's height (INSERT_BAND = 0.32), so plain time-moves became lane-changes, triggering the z machinery and preview churn. The band is now geometry-exact: only the visible gutter between clip bodies.

Lane model change (the root cause, user-approved): display lanes were derived from global z-rank and re-persisted onto every clip on each edit — so one clip's horizontal move could rewrite other clips' track indexes and mint z values (file-diff-proven). The timeline now uses stable track lanes: a clip's lane is its data-track-index, ordered per kind-zone; z-index is canvas paint order only (the deliberate vertical-move stacking sync remains the one place z updates). The commit has three outcomes — pure time-move (writes only the dragged clip's start), lane change (writes only the dragged clip), and gap-insert (the one sanctioned multi-clip write: a minimal +1 renumber). resolveDropIntentZ is deleted. timelineZones tests were rewritten to pin the new contract, including the live repro: a horizontal move among overlapping neighbors diffs exactly one element.

Insert UX (user-specified): the insert indicator only triggers in the gap band between rows; inserting above the top row works; a drop whose aim is occupied creates an adjacent track (pointer-nearest gap) instead of snapping back to origin.

Verification: full per-branch gates (tsc, suites at the known 18-failure baseline, fallow with zero suppressions, format, filesize, builds, exact tip parity) plus pointer-driven browser verification on a real project: marquee/collapse both directions, horizontal-drag one-element file-diff invariant, zero z-order requests, zero reload fetches (the edit blink is gone), no-op drags write nothing.

[ukimsanov]

Round-3 items landed. Owning the miss first: "all 25 re-gated green" was wrong — the gate script printed per-head vitest counts without asserting them. It now fails any branch whose count deviates from the 18-failure baseline (tolerating only the documented happy-dom GC flake by name), and this submit ran fully green under that assertion.

• Red heads: fixed by placement — feat(studio): timeline collision and placement model #2195 carries a window variant of the collision test (the production insert-band suite lands with the glue swap at feat(studio): timeline glue swap — Timeline/TimelineCanvas onto the NLE engine #2212, where INSERT_BOUNDARY_BAND exists); feat(studio): timeline glue swap — Timeline/TimelineCanvas onto the NLE engine #2212 carries the zero-padded Time range assertion. Every head now sits at the exact baseline.
• Upward edge-create: out-of-range aims anchor to the clip's own kind-zone boundary ({track:-1} → insertRow 0), and the below-fallback is zone-gated — your repro is a test, plus up/down/audio variants.
• NUL joiner: escaped form; the file is text again on GitHub. Comment fixed too.
• Also in: the no-CORS shimmer branch (setFailed in the error listener), a NaN-guard test for the coexistence intermediate, and the feat(studio): clip drag engine — preview and commit (unwired) #2206→feat(studio): app shell swap — mount EditorShell, retire the legacy engine #2213 ordered-unit invariant is now stamped in each of those PR bodies.
• Acknowledged as transient-not-durable: the feat(studio): timeline glue swap — Timeline/TimelineCanvas onto the NLE engine #2212-alone preview NaN (trio-atomic covers it). Deferred, tracked: SDK-cutover coalesceMs, dup-id soft-apply, zones doc sync, no-op insert-edit filtering, feat(studio): timeline visual refresh #2215 body addendum for the playback auto-scroll removal.

[ukimsanov]

Restacked onto current main (v0.7.53, 1dfb016f3) so the merge train starts clean — main is now contained in every head's ancestry, so nothing from its 78 commits can be lost.

Two conflict sites total, both resolved semantically (full per-file log in the stack's conflict report):

• feat(studio): canvas glue swap — overlay, gestures, selection to final form #2211 canvas glue vs fix(studio): graded elements survive manual editing (disappear/resize/rotate/crop/panel) #2230 graded-element editing: main's conflicting delta was entirely the pre-OBB resize model this stack replaces — the local-space rewrite measures rendered geometry, so graded/scaled elements are handled by construction; main's actual crop mechanics (rotated-frame crop UI, gesture-owned restore) live in non-conflicting files and are adopted wholesale.
• feat(studio): app shell swap — mount EditorShell, retire the legacy engine #2213 gsapSoftReload: pure append-collision between main's new SoftReloadOptions and this stack's undo-restore section — both kept; our callers migrated to the new options signature (in the coexistence intermediate too).

Post-restack verification: full per-branch gates on all 25 heads — tsc, complete suites at the known baseline with the per-head failure-count assertion, fallow (zero suppressions), format, filesize, builds — all green, including main's newer suites (graded-element, sdkCutover, variable-promote) running against this stack's engine.

Standing: #2206→#2213 as one ordered unit; no stamps pending Abhay Zala per channel rule. Merge-ready from our side.

[miguel-heygen]

Current head 2a55fa3 still has actionable structural issues from prior review: DomEditOverlay declares onDeleteSelection/onApplyZIndex but never forwards them (silent no-op boundary), and three duration-extension regexes duplicate fragile attribute-order/double-quote assumptions and can diverge from DOM-root selection. Resolve or explicitly supersede these before approval.

[ukimsanov]

Two corrections on the final-sweep readout (verified at the current heads):

• test(studio): characterization suites for resize commit and razor history #2201: the "tests reimplement a local resize helper" blocker describes the pre-round-1 state — at head 04a063c5's branch, anchoredResizeReleaseShift.test.ts exercises the production computeNextResizeAnchor accumulator and the test-local helper is gone (that was Vance's round-1 blocker, fixed and re-verified by Abhai's round-2 execution pass).
• feat(studio): glue API coexistence layer for the NLE swap #2205 "dead DomEditOverlay props": coexistence-window shape, not dead code — the props are declared at the intermediate and forwarded once the swap lands at feat(studio): canvas glue swap — overlay, gestures, selection to final form #2211 (DomEditOverlay.tsx destructures and wires both at the tip; confirmed twice in this thread).
• The feat(studio): canvas context menu and z-order actions (unwired) #2202/feat(studio): NLE shell components — context provider, panes, overlays (unwired) #2204/feat(studio): timeline interaction hooks and lanes component (unwired) #2207/feat(studio): app shell swap — mount EditorShell, retire the legacy engine #2213 preview-regression/regression failures are the force-push race artifact again (e.g. feat(studio): app shell swap — mount EditorShell, retire the legacy engine #2213's regression job died in 4s at setup with its shards still pending) — failed runs have been rerun.
• The duration-regex triple stops being a deferral: the DOMParser-based replacement is in progress and will land in the stack shortly.

[ukimsanov]

Duration-regex triple: closed. One shared DOMParser-based reader (readRootCompositionDuration) + a byte-preserving attribute splice for writes (patchRootCompositionDuration) replaces all three regex sites (extendRootDurationInSource, extendCompositionDurationIfNeeded, setCompositionDurationToContent). Handles swapped attribute order, single quotes, and whitespace — the silent-no-op cases — with tests for each; the write path splices only the data-duration value (full-document re-serialization measurably reformats unrelated markup, so it was rejected). Lands at #2205 (helper + glue variant) and #2213 (final + tests). Full 25-head gates green under the count assertion; resubmitted. That closes the last deferred code item from the review rounds.

[miguel-heygen]

onDeleteSelection / onApplyZIndex were declared on DomEditOverlay but never consumed or passed by any caller.\n\nAddressed: removed the unused action props and stale wiring documentation so the component no longer advertises dead callbacks. Duration parsing fixes remain intact. Targeted Vitest could not start in this worktree because happy-dom is unavailable in the environment; git diff --check passes.

Superseded by current-head re-review; findings addressed.

[miguel-heygen]

Re-reviewed current head after removing dead DomEditOverlay action props and preserving duration parser fixes. Approval is for this exact head; required checks must still complete.