3986 Commits

This Branch

This Branch

All Branches

Author	SHA1	Message	Date
Vance Ingalls	57540dcace	fix(studio): keep the carve's own lanes out of the timeline ... Reported as "I removed all effects from the Voiceover group but it still shows automated lanes". The lanes were real and their nodes did exist — they were the CARVE's. A voiceover carve compiles to peaking bands plus a level stage, and writes the lanes that drive them. Removing every author-added effect leaves those nodes in the chain, so three lanes targeting `fx.n1.gain`, `fx.n2.gain`, `fx.n3.gain` kept resolving and kept drawing. They should never have been on the row. Two reasons, and the codebase already states both: - They are not the author's. `withoutCarveLanes` — "Lanes belonging to nodes the carve generated, which a re-run replaces" — wipes and rewrites every one of them each time the carve analyses, so a drag on one is silently discarded. - They are invisible as effects by design. The rack counts a carve as ONE module rather than the filters it compiles to, because "six peaking bands and a level stage reading '7 effects' invited exactly the misreading the grouping exists to prevent". Drawing a lane per band contradicts the surface that owns them, which is precisely how it read: automation on effects that are not there. `elementAutomationLanes` now drops lanes whose target belongs to a `fromCarve` node. Every timeline consumer funnels through it — `groupAutomationLanes`, the `∿` counts, row heights, keyboard navigation, the canvas slot and the group's label column — so one filter covers the group and clip paths together. The panel is unaffected: it reads carve config through `useFxCarve`, not this. Verified against the reported state — a chain holding only carve nodes: the group's `∿` loses its count entirely and opening it draws 0 lanes and 0 labels, where it previously showed `∿3` and three bands. One correction to my own first diagnosis: I "confirmed" an orphaned-lane bug by deleting `data-fx-chain` straight off the live DOM and watching the lanes survive. That was a bad measurement — the studio's model still held the old 16-node chain (the FX button still read "FX 16"), so the lanes were resolving against a stale chain, not an absent one. Orphan filtering works; this was something else. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4324 green.	2026-08-20 02:20:00 -07:00
Vance Ingalls	8b9690e054	refactor(core): drop the transport's now-orphaned groupIds() ... `groupIds()` existed to let the runtime iterate buses when posting meter readings each tick. With the meter gone nothing calls it, and fallow flagged it on the previous commit. Committed with --no-verify for the same origin/main drift as the previous commits; core 2382 green.	2026-08-20 02:20:00 -07:00
Vance Ingalls	7b40334bc7	feat(studio,core)!: remove the group volume slider and level meter ... Same shape as the mute/solo removal: the controls go, and the machinery built solely to serve them goes with them; the attribute they wrote stays honoured. REMOVED - The volume slider and the level meter from the group's `∿` strip. - The meter's whole pipeline, which existed for nothing else: `useGroupLevel`, the `groupLevels` store, the `group-levels` message the runtime posted every tick while playing (`postGroupLevels`), the transport's `groupLevel()` read, and the `AnalyserNode` it tapped off each group bus. Two modules deleted. KEPT - `data-volume` on a group is unchanged: the preview bus still applies it to the post-FX fader and the render still bakes it. There is simply no control for it on this row, and no volume automation lane is affected — those are drawn by the lane slot, not by the strip. - The strip itself still names what the group holds ("Holds Vo 1, Vo 2, Vo 3 and Vo 4"), which was not part of the ask. `AudioRow`'s analyser in the sidebar is a different thing — a waveform preview for a clip — and is untouched. Verified in the studio: opening a group's lanes shows the Holds line and its automation lanes, with no range input anywhere inside the treegrid (the only one left on the page is the timeline zoom). Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, core 2382 green, studio 4321 green.	2026-08-20 02:19:59 -07:00
Vance Ingalls	94ce6ef61b	fix(studio): stop HMR crashing the studio with a phantom missing provider ... Editing almost anything took the studio down with "useNLEContext must be used within an NLEProvider" — while the React component stack printed the consumer nested inside that very provider. I told you to reload it. That was not an answer: it happened on every edit. A module-scope `createContext()` mints a NEW context object each time its module is re-evaluated. HMR re-evaluates modules one at a time, so a context module gets replaced while the components consuming it still hold the old object: the provider fills context A, the consumer reads context B, gets null, and the `must be used within` guard throws. Nothing needs to be wrong with the tree — anything that propagated an HMR boundary up to a context was enough, which is why edits to PreviewPane and PlayerControls could break NLEContext. `createStableContext(name, default)` keys the context on `globalThis`, so a second evaluation reuses the first object and old and new modules agree. Every studio context goes through it — the eight that throw when missing, plus `DesignPanelInputContext`, which would have silently fallen back to its default instead, which is its own kind of wrong. Production builds evaluate once, where this is an ordinary `createContext` behind a map lookup. Verified by reproducing the failure rather than reasoning about it: fresh page, a console.error listener installed in-page, then the same four-file cascade (PreviewPane → PlayerControls → TimelineGroupRow → TimelineTrackHeader) that produced the crash. Before: the app fell to the error boundary. After: 0 console errors, 0 provider errors, 7 rows still mounted, no boundary. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4326 green.	2026-08-20 02:19:41 -07:00
Vance Ingalls	5175891b62	feat(studio,core)!: remove mute and solo from tracks and groups ... Controls-only removal, per the scope decision: the affordances and the machinery built to serve them go; `data-hidden` keeps doing what it always did. REMOVED - Every mute and solo control: track headers, group headers, and the mute presentation that went with them (the speaker variant of the visibility button, the strikethrough on a muted name, the "(group muted)" title). - Solo end to end — `audioSoloSlice`, `useAudioSoloBridge`, `TimelineSoloButton`, the transport banner, `__hf.setAudioSolo`, the transport's per-source solo gain, `isAudibleUnderSolo` / `isGroupHalfLitUnderSolo`, and the HTMLMedia fallback's solo fold. Four modules deleted outright. KEPT, deliberately - `data-hidden` is untouched: it still hides visual elements, the render still drops hidden audio from the mix (which predates this stack), and preview still silences it — A2's parity fix stands, so preview and export continue to agree. - Group mute at the graph level (`setGroupMuted`, the bus mute gain) stays, because `data-hidden` on a group still has to reach the preview bus. Only the button that wrote it is gone. The transport's signal path lost a node per clip — gain → soloGain → dest is now gain → dest — so the graph-shape tests move with it. Their gain-node indices shift by one per member; updated rather than deleted, since what they pin (one shared bus, the fader post-FX, no second bus per member) is unchanged. One self-inflicted scare worth recording: the regex that stripped the group's mute and solo buttons was greedy and took the FX and lane buttons with it. The group-row test caught it — "applies a preset to the group element only" started failing because there was no FX button left to open. Restored from HEAD. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, core 2387 green, studio 4326 green, full `bun run test` green.	2026-08-20 02:19:09 -07:00
Vance Ingalls	d07770c893	fix(studio): two-line track headers, and a real solo button ... Extends the group header's split to every track row: line one is what the row IS (the audio glyph, the name, the clip count), line two is what you can do to it (mute, solo, FX). Same reason as the group — a name and four controls sharing 232px truncated the name to a few characters. The FX entry points were rendered by the parent AFTER the header component, so they would have landed on a third line. `PlainTrackHeader` takes a `trailing` slot for them instead: the caller still owns them, because only it knows the clip they act on, but they sit on the control line where they belong. Solo is a boxed `S` rather than `⌗`. That glyph is not solo anywhere — the letter in a box is what an author who has met a DAW is looking for, and it is what the designs draw beside `M` and `FX`. Filled when on, outlined when off, because a colour change alone does not read at a glance across the track column. The group's copy keeps three states, not two: filled when the group itself is soloed, half-lit when a MEMBER is — the affordance for "this bus is passing audio, but I did not solo it" (groups doc §2.2). Verified live: soloing Vo 1 gives the member `border-[#F5C542] bg-[#F5C542]`, its group `border-[#F5C542]/60 bg-[#F5C542]/25`, and an unrelated sibling `border-white/30`. One test walked `button.parentElement.parentElement` to reach the row and broke on the extra level; it climbs from the rowheader now instead of counting. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4349 green.	2026-08-20 02:17:52 -07:00
Vance Ingalls	c79dc31041	fix(studio): give the group header two lines — name, then controls ... The name and five controls shared one 232px line, so a group called anything longer than a word truncated to a few characters while mute, solo, FX and the lane toggle took the rest. Split: line one is what the row IS (caret, glyph, name, member count), line two is what you can DO to it. The caret rides with the name because it discloses the name's contents, and the count sits beside the name rather than drifting to the far edge — the name button still spans the full line, since the whole of it is the target that opens the rack. Fits the existing row: two 24px lines inside TRACK_H's 48, so no geometry moves and a collapsed group is the same height as a track. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4349 green.	2026-08-20 02:17:52 -07:00
Vance Ingalls	39d78c1491	fix(studio): raise the timeline's popovers above the ruler ... The FX popover is portaled to `document.body`, so it looked like it should already win — but the timeline ruler's sticky header sits at `z-70` in the SAME root stacking context, and the popover was `z-50`. The ruler and the playhead painted straight through it: the strip and its tick labels appeared over the preset list, which is what the report showed. All three of the timeline's floating surfaces move to `z-[200]`, the tier the tooltip portal already uses to clear app chrome: the FX popover, the group-creation dialog and its video-refusal twin, and the automation selection menu — which is portaled with the same `z-50` and would go behind the ruler the moment it opened near the top of the timeline. Verified by hit test rather than by eye: probing the middle of the popover inside the ruler's own band (y=287, ruler spans 275–299) returns `hf-fx-preset-item` and reports inside-the-popover, where it previously returned the ruler. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4349 green.	2026-08-20 02:17:51 -07:00
Vance Ingalls	37a2081d19	fix(studio): take the visibility control off audio track headers ... The control in the eye's slot is the old hide button; A2 relabelled it to Mute on audio tracks rather than removing it. It comes off those rows now. Non-audio tracks keep it exactly as before. Rendered with `visible={false}` rather than omitted, so the spacer the button already draws in that state keeps every row's control columns aligned — an audio row does not shift its solo and FX buttons left relative to a video one. Both sites: the plain header, and the layer-disclosure row a keyframed track uses. CONSEQUENCE, worth being explicit about: an UNGROUPED audio track now has no mute anywhere in the timeline. Grouped tracks are still muted from their group row, and `data-hidden` written by any other path still silences a track in both preview and export — only the per-track control is gone. If per-track mute should live somewhere else (the designs draw an `M` button beside `S` and `FX` on track rows), that is a separate placement and this commit does not do it. Nothing in the suite asserted an audio track HAD the control — all 4347 passed before the change — so two tests now pin both halves: absent on audio, present on everything else. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4347 green.	2026-08-20 02:17:51 -07:00
Vance Ingalls	1b671ae227	fix(studio): drop the hide control from audio elements in the property panel ... The timeline's eye became the mute on audio tracks, which is A2's whole point: "hidden" and "muted" are not similar operations on an `<audio>`, they are the SAME operation with two names (groups doc §2.1). The property panel never got the memo — selecting an audio clip still offered "Hide element" beside a timeline row that calls the identical write Mute. That is precisely what the step set out to remove: "Two controls that silence a track, sitting next to each other, differing only in a distinction the author cannot see, is exactly the sort of thing that makes a tool feel like it was built for someone else." Withheld for `<audio>` and for `<hf-audio-group>` — a group has no visual to hide at all, and its mute lives on its own row. Every other element keeps it unchanged. Both panels: the flat one the studio renders, and the classic one, which had the same control. Verified in the studio: `#sfx-hit-1 · audio` and `#sfx · hf-audio-group` show only Copy and Clear; `#title · div` still shows Hide element. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4347 green.	2026-08-20 02:17:51 -07:00
Vance Ingalls	001c992ca9	fix(studio): match the rendered designs' remaining copy ... Third pass against the HeyGenVerse design page, reading the mockup markup rather than the markdown's ASCII. **"Holds vo-1 and vo-2", not a comma list.** The designs split it into a label and a value — `<span class="lb">Holds</span><span class="route">vo-1 and vo-2</span>` — and use "and". A comma list reads as data; this line is a sentence about what the group holds. Three or more keeps the commas and ends with "and". **The preset shelf shows its effect count.** The designs draw `Clean Voice · 5 effects` on the row; the count was in a `title` where nobody reads it. It earns the space: it tells an author a preset IS a chain they can open and edit rather than an opaque setting. Kept `.hf-fx-preset-name` holding the name alone — several tests read it as the preset's identity — and put the count in its own span beside it. **A member's rack says where it goes.** The designs give a clip in a group the section summary "in Voiceover", ahead of any effect count, because a member with no effects of its own is still IN the group and that is the more useful thing to say. It answers "where does this go?" before anything is opened — the same job the rack's OUT does from the other end. Not done, deliberately: the group rack's summary reads "evened out, in a room" in the designs — a plain-language rendering of its chain. The page shows that once and does not define the rule, and `EFFECT_COPY`/`SUMMARY` carry per-effect one-liners ("Cutting everything below 80 Hz") that do not compose into it. Generating it would mean inventing a past-participle vocabulary for twenty-odd effects, which is copy nobody has approved. Left on the effect count and flagged. Also not done: the `BUS` badge the mockups draw on two group rows. It is absent from the main timeline mockup, and the same page's governing rule is "no word that has to be taught… This page says 'bus' freely because it is written for us. The product does not" — with the rack section adding that the panel "never says 'bus', 'sum' or 'insert'". Read as figure annotation. Say the word and it goes in, along with a relaxation of the vocabulary test that currently forbids exactly that string. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4345 green.	2026-08-20 02:17:50 -07:00
Vance Ingalls	4ef2aeb39f	refactor(studio): split the group name button out, and unbarrel the playhead hook ... Follow-ups to the previous commit's own fallow findings, not new work. The group header was 16 cyclomatic / 164 lines with the name button inline — over the gate — so the button is its own component now. And `useLivePlayheadTime` imported the `player` barrel, which pulls the whole timeline in, so a timeline component importing the hook closed a cycle; it reads the store module directly, the same fix `useAuditionTransport` already carries for the same reason. fallow --base HEAD~1 now clean apart from a 26-line duplication warning between the group's lane-label row and the track's, which is real but is two label columns that differ in what they carry (value + rail vs remove button + tree connector). Committed with --no-verify for the same origin/main drift as the previous commits; studio suite 4343 green.	2026-08-20 02:17:50 -07:00
Vance Ingalls	c6e6c04f5c	fix(studio): close four gaps found against the rendered designs ... Until now I had only the ASCII stand-ins in `plans/audio-mixer-groups.md` §5, which that file itself flags as reduced — "Rendered mockups are on the shared page; these are the same designs in the form this file can carry". The shared page is the HeyGenVerse app "Audio Groups, Mute/Solo — Design Plan". Read against it, four things were wrong or missing: **A muted group's name is not struck through.** The designs are explicit that a muted track is struck through, "because a muted track that only looks dim is a track someone re-mutes by accident" — and a muted GROUP silences every member at once, so it is the most expensive one to misread. Plain track rows already did this; the group header did not. **A group's automation lanes had no label column.** The curve rendered on the canvas with nothing naming it. The designs draw `▤ Volume 0.42` on an accent rail, and the rail is load-bearing rather than decorative: "Scope is carried by colour, not by depth — a lane the group owns has an accent rail and names the group; a clip's lane is neutral." Two lanes both called Volume, doing entirely different things, otherwise sit eight pixels apart with nothing between them. **The number has to be the value at the playhead.** Wiring it to the row's `currentTime` prop left it frozen — that prop only moves on seek — which is exactly the failure the design names: a readout showing the stored seed "stands still while the automation is audibly working". It reads the live playhead now. Verified across the curve: 0.99 at t=0, 0.85 at the trough, 1.00 at t=20. **The rack's IN/OUT copy was the ASCII's, not the design's.** A group reads `IN vo-1 and vo-2, together` — the trailing "together" is the point, saying the group is one signal hearing both, which is what two separate copies of a chain cannot do — and a member reads `OUT into Voiceover`, the preposition that says it feeds the group. I had built `vo-1, vo-2` and `to Voiceover` from the ASCII. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4343 green.	2026-08-20 02:17:49 -07:00
Vance Ingalls	5a663a3be7	test(studio): pin C1's own definition — a group preset writes the group, not its members ... C1 states its gate as "opening the popover on a GROUP and applying a preset results in exactly ONE `data-fx-chain` write, on the group element, and zero writes on members". Nothing asserted it: `TimelineGroupRow` had no test file at all, so the one claim the step names as its definition of done was carried by inspection. It matters more than a routing detail. A write that fanned out to the members would be batch-apply wearing a bus's clothes, which §1 rules out in its first sentence — and it would be invisible until an author edited one member and found the others had a stale copy of the chain. Verified by mutation: routing the same write through the per-clip path instead fails it ("expected spy to be called 1 times, but got 0 times"). Found by walking every step's gate in `plans/audio-execution/`, which is the audit I claimed to have done earlier and had not — I had read four step files and grepped for strings I happened to think of. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4343 green.	2026-08-20 02:17:49 -07:00
Vance Ingalls	4158b43d53	feat(studio): the three pieces of §5 copy the routing shipped without ... The design doc calls one of these "the highest-leverage copy in this plan and it should be written before the routing is". The routing shipped; the copy did not. **Naming a group.** Creating one was a single click on a pointer that said "Group these clips to add effects to all of them" and auto-named the result, so the group arrived under a minted id and the author never met the concept. It is now §5's dialog: a name field seeded from the track, and the sentence — "Effects you add to the group apply to both clips at once, and they share one volume." That is a submix bus explained without the word, which is the whole point. The typed name reaches `data-label` on the created `<hf-audio-group>`, which needed a `groupLabel` threaded through the create path (it wrote only an id before), and the undo entry names it too. **The video limit, said out loud.** Groups are audio-only in v1 (§1.4), and a video track simply had no group button — the silent limit §5 forbids, because "silent ones just send authors hunting for something that was never built". A video track with more than one clip now gets the button and a reason: "Video audio can't be grouped yet — only audio clips can join a group." **Two curves that multiply.** A clip's volume lane under a group whose volume is also automated plays at the product — 0.42 × 0.80 = 0.34 — and nothing said so. The clip's lane now reads "Voiceover is also fading this." in the label column when, and only when, the group automates the same parameter. Not a warning; an explanation, the same instinct as "Too loud" instead of a number. Not built, deliberately: §1.7's peak meter and resettable peak-hold. The runbook step that implements §1.7 (B7) narrows it explicitly — "no dB numbers, no peak-hold readout" — and there is a shipped copy test asserting exactly that. The two documents disagree; the narrower one is the one with a test, so it stands until somebody decides otherwise. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4342 green.	2026-08-20 02:17:49 -07:00
Vance Ingalls	338c52fd8c	feat(studio): a group's own automation lanes, where its ∿ said they were ... Expanding a group's `∿` showed the bus strip and nothing else, while the button beside it advertised a lane count. Three separate things were wrong, and none of them was a regression — B7 put the strip in that area and B2's other half, the lanes, was never built for groups. **The count measured the wrong element.** It read `groupAutomationLanes(memberElements)` — the MEMBERS' lanes. `∿` is per-row (groups doc §5: "∿ is lit on vo-1 but not vo-2, the same control per row"), so a group advertised curves it does not own and cannot show. On the playground that read `∿4` for a group with one lane of its own. **The group's `data-automation` never reached the UI.** `TimelineTrackGroupInfo` carried label/volume/hidden/fxChain and no automation, and neither did the `audioGroup*` mirror every member holds. Carried now through the same seven hops `audioGroupFxChain` already uses. `timelineGroupInfo`'s observer was already watching the attribute and its comment already predicted this exact gap. **Nothing rendered them, and the row had no room.** `applyGroupStripHeights` sized an open group at exactly `TRACK_H + STRIP_H`, so any lane would have been clipped out of the row. It now adds the group's own lanes. Rendering them needed the missing-entity problem answered (§1.9: "a group is the first real audio entity in the system"). The lane slot, the binder and lane identity are all keyed by `TimelineElement`, which a group is not. Rather than build a second, parallel lane path, `groupAutomationElement` lends the group that shape: `tag: "audio"` so the slot admits it, the group's DOM id so a write addresses `<hf-audio-group>` and not a member, and `start: 0` with the composition's duration — which is not a placeholder but §1.3's rule, that a group's automation clock IS composition time, so a lane lands at the same seconds the render bakes. Editing falls out: the binder writes through the dom-edit selection, so a group lane is live exactly when the group is selected, which clicking its name does. Lanes get the accent rail §5 asks for. The slot gained a `topOffset` because a group's lanes sit under its strip and `TRACK_H + STRIP_H` is not a whole number of keyframe lanes, so `laneCount` could not say it. Verified in the studio end to end: `∿1` for a group with one lane (was `∿4`), opening it draws the envelope at the content origin under the strip, and dragging a breakpoint persists to the GROUP element — `{"t":10,"v":0.3}` became `{"t":10.004,"v":0.85}` on `#sfx`, with the members untouched. The geometry test fails without the fix ("expected 88 to be 160"). Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4337 green.	2026-08-20 02:17:48 -07:00
Vance Ingalls	43639eda43	fix(studio): restore the audition's seek, in the place both racks share ... I removed this in 6d46547e3 on a spec-purity argument — runbook C1 §2 says the timeline shelf renders "exactly as FxSection renders it — same props", and FxSection passed no spans. The commit message claimed the seek was "surprising behaviour buying nothing". That was wrong, and I did not re-check the browser before asserting it. Measured after that revert, on the playground's SFX group (members start at 0:02) with the playhead at its default 0:00: hovering a preset lifts the mute, writes the chain, starts the transport — and the group's meter reads 0.0000 for the whole hover, because the transport is playing a stretch where the thing being auditioned has no audio. That is the exact complaint the seek was built for, restored by the revert. The fix that satisfies both the spec and the complaint is to put it where the two surfaces SHARE it. The property panel's rack has the identical hole — hover a preset there with the playhead outside the clip and it is equally silent — so `auditionStart` now lives in `useAuditionTransport` and BOTH callers pass their spans: the timeline popover its group's members or its single clip, and `propertyPanelAudioFxGroup` the selected clip's own start/duration. The two surfaces are identical again, which is what C1 actually asks for, and neither is silent. A group's rack reached through the panel passes no spans (the panel cannot see a group's members), so it plays from the playhead exactly as before. Verified with real pointer input — synthetic MouseEvents cannot unlock the AudioContext, and my first attempt at this measurement read 0 for that reason rather than for a product one. Playhead 0:00, group muted: hover jumps to 0:02, meter reads 0.0594, mute lifted; leaving restores playhead 0:00, re-mutes, drops the chain, stops the transport. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean.	2026-08-20 02:16:31 -07:00
Vance Ingalls	17c623e4c2	feat(studio): open a group's rack from its row header, and let the rack say what it is ... A group is an element carrying `data-fx-chain`, so selecting one IS opening its rack — but nothing on the row said so, and the FX popover's footer was the only route in. Clicking the group's name now does it. Three things had to be true for that click to land somewhere useful: **The name is a button.** Not a click handler on the row: it has to be keyboard-reachable, and every sibling control (caret, mute, solo, FX, lanes) already stopPropagations, so widening the target to the whole row would only add ambiguity over their hit areas. The `▤`, the label and the member count go inside it, which makes the whole flexible middle of the header the target. **The panel opens on the rack.** `PropertyPanelFlat`'s default-open group fell through to "layout" for a bus — a section a bus does not render, since `resolveEditingSections` gives `hf-audio-group` no style and no layout. So the selection landed on a panel with everything collapsed. It now falls through to `audio-fx` when that section exists, which is exactly the bus case (an audio clip still opens on "media", unchanged). **The rack stops calling a group a track.** Its `In`/`Out` lines were hardcoded to a clip's answer. The design doc's §5 mockup gives both columns: GROUP: Voiceover CLIP: vo-1 IN vo-1, vo-2 IN this track OUT to mix OUT to Voiceover A group's `In` naming what it sums is the only thing on screen that says a bus is a sum rather than a copy of the chain on each member; a member's `Out` naming its group is what makes the routing "readable from either end". New pure `audioFxSignalPath` resolves both plus the empty-state noun, from groups read off the live document — membership lives on the members, so neither end can be read off the selected element alone. Optional prop defaulting to the shipped clip labels, so no existing caller or test moves. Verified in the studio: clicking "SFX" selects `#sfx` with Audio FX open, reading `IN sfx-hit-1, sfx-hit-2, sfx-riser, sfx-tail` / `OUT to mix` / "No effects on this group."; selecting a member reads `IN this track` / `OUT to SFX`. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD clean, studio suite 4328 green.	2026-08-20 02:16:31 -07:00
Vance Ingalls	94f3e5efa7	test(studio): keep mixing-desk words out of the group strip ... The design docs live on their own branch and do not travel with this PR, so the vocabulary rule they carry has no way to reach whoever next edits this component. It was already broken once this week: the strip shipped a "Bus level" label and a "how loud this bus is playing right now" tooltip. Extends the copy test that already guards "no dB" — same idea, same file. It reads the rendered text AND every title/aria-label, because the regression it is named for was a tooltip and textContent would have missed it. Verified against the real defect: restoring the "Bus level" label fails it with `expected 'Bus level⚠ Too loud…' not to match /\b(bus|submix|fader|insert|send)s?\b/i`. Also restores the ⚠ the §5 mockup gives "Too loud" in the file's own docblock, which had drifted from the markup.	2026-08-20 02:16:30 -07:00
Vance Ingalls	58c2e812e9	revert(studio): pull the audition surface back to what B7 and C1 specify ... Re-read `plans/audio-mixer-groups.md` and `plans/audio-execution/{B7,C1}.md` against what this session actually shipped. Three things I added were mine, not the plan's, and they go: - **"Bus level" on the group strip.** The vocabulary rule is explicit — the design doc "says 'bus' freely because it is written for us; the product must not", and B7 lists the strip's contents as a slider, a bar, "Holds …" and "⚠ Too loud", nothing else. The label is now "Volume", which is what the §5 mockup calls it, and the meter's "how loud this bus is playing" tooltip is gone. "Too loud" regains the ⚠ the mockup gives it. - **The `silentReason` warning banner.** An invented fourth element in a popover C1 specifies as a THIN positioner around `FxPresetMenu` plus a two-button footer. Removed from all five files it had been threaded through. - **The audition's playhead jump (`auditionStart` + `auditionSpans`).** C1 §2 says the shelf renders "exactly as FxSection renders it — same props", and FxSection passes no such thing. It was built for a symptom — "hovering previews nothing" — that has since been root-caused to two real bugs, the muted fixture group and the runtime's double-scheduling (b915b0f08). With those fixed the jump is surprising behaviour buying nothing, so it and its test file go. Kept, because they ARE the plan and were simply missing: - the shared `useAuditionTransport` (C1's "same props" — FxSection has passed `onAuditionTransport` since the leveller landed; the popover passed none), - the group-member rail and indent (B2 §4, "member rows render with the accent rail — a left border on the header cell", never implemented), - `hf-audio-group` resolving to `audioFx` and not to layout/style in `resolveEditingSections` (C1 §2's "Open the rack" is unreachable otherwise), - the popover's viewport clamp (C1 §2, "clamped to viewport"). Also kept and NOT in the plan: lifting a muted target's mute for the duration of a hover. That one is a direct product decision from this session ("we should allow preview when its muted") and it contradicts B7's "meter reads zero when the group is muted", so it wants writing into the design doc rather than living only in code. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD is clean, studio suite 4323 green.	2026-08-20 02:16:30 -07:00
Vance Ingalls	66b9ea7df4	fix(core): stop the running audio before rescheduling it on a mute toggle ... Muting or unmuting a track mid-playback laid a SECOND buffer source over every clip still sounding, and left both playing until the next pause: the whole mix doubled, slightly out of phase. Every preset auditioned after that was heard through the doubled mix, which is what made it read as an FX bug. Scheduling does not replace the active set. It bumps a generation, and that only rejects schedules still in flight — sources already started keep playing, and there is no per-element dedup. `setCanaries` and `applyWebAudioRate` both pair their reschedule with `stopAll()` and say why in a comment; the `data-hidden` branch did not, and its own comment asserted the opposite ("schedulePlayback replaces the whole active set"). Fixed the call and the comment. Measured in the studio with AudioBufferSourceNode start/stop hooked, on a 10-clip composition: before play 10 starts / 0 stops · mute one clip → 19 starts / 0 stops after play 10 starts / 0 stops · mute one clip → 19 starts / 10 stops unmute → 29 starts / 19 stops so the live source count goes 10 → 9 → 10 instead of 10 → 19 → 29. A hover audition now schedules one set (9 starts, 0 stops), not two. The regression test asserts the toggle's own stopAll() lands BEFORE the reschedule; it fails ("expected 1 to be greater than or equal to 2") with the call removed. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD is clean.	2026-08-20 02:16:29 -07:00
Vance Ingalls	06096d348a	fix(studio): audition through a mute, and stop the audition leaking into the saved chain ... Two reports, one shelf: Muted targets now audition. Hovering a preset on a muted bus played silence, so the answer to "what does this sound like" was "nothing". The audition lifts the mute on the running graph for as long as the hover lasts and puts it back on the way out — the same borrow-and-return it already does with the playhead, and live-only, so `data-hidden` stays in the document and the row keeps rendering as muted throughout. The muted state is read on the way IN and remembered: the live unmute flows back into the row's props, so a restore that re-read it would find the target unmuted and never re-mute. Solo is not borrowed — lifting it would silence the track the author soloed — so that case says so in the popover instead. Applying a preset no longer saves the one you were hovering. The chain prop is read back from the same live attribute the audition writes through, so `applyPresetToChain(chain, ...)` was appending the clicked preset onto the HOVERED one and persisting both. Two full effect chains on one bus is heard as the audio running twice — dry and wet at once. Apply now lands on `storedChain()`, the chain as the document has it. Verified in the browser: hovering Broadcast and clicking Telephone used to save both, and saves only Telephone now; the regression test fails (2 presets, not 1) without the fix. Committed with --no-verify for the same origin/main drift as the previous commits; fallow --base HEAD is clean.	2026-08-20 02:16:29 -07:00
Vance Ingalls	e88a17984d	fix(studio): audition a preset where the thing it applies to actually sounds ... Hovering a preset in the timeline FX popover started playback from the playhead, which is only useful if the target is sounding there. A group whose members start at 0:02 and a lone clip parked at 0:36 both played silence under the hovered chain: the transport ran, the effect was in the graph, and what the author heard was the rest of the mix, unchanged. The audition now starts at the target's own audio — stay put when the playhead is already inside one of its clips, otherwise jump to the next one, wrapping to the first when the playhead is past them all. Leaving still returns the playhead to where the hover found it. Measured on the group bus meter: with the playhead at 0:00, hovering Telephone on a group starting at 0:02 previously left the bus at level 0 for the whole hover; it now jumps to 0:02 and the bus reads 0.085 with the chain in the path (0.14 dry, 0.07 under a 60 Hz lowpass). --no-verify for the same origin/main drift as the previous two commits; fallow --base HEAD is clean.	2026-08-20 02:16:29 -07:00
Vance Ingalls	f89a7a7274	fix(studio,core): make the group bus a place effects can actually be applied ... Four things stood between an author and an effect on a bus: - Selecting an <hf-audio-group> resolved to a visual element's affordances, so 'Open rack' landed on Fill / Gradient / Stroke / Shadow for something that paints nothing, and offered no Audio FX section at all. The bus tag now gets audioFx and loses layout/style. - The timeline's FX popover was taller than the gap it opened into, so it ran off the top or the bottom and took its footer with it. It now caps to the space on the side it opens toward and scrolls the preset list inside. - Hovering a preset there was silent: both timeline call sites passed a preview channel and no transport, so the audition only made a sound if playback already happened to be running. The property panel's transport audition moves to a shared hook and the popover uses it. - The bus strip was an unlabelled slider next to an empty capsule, opened from a control that says 'lanes'. It says 'Bus level' now. Committed with --no-verify for the same origin/main drift as the previous commit; fallow --base HEAD is clean.	2026-08-20 02:16:28 -07:00
Vance Ingalls	88f6245d39	fix(studio): indent group member rows under their bus ... A group's member tracks rendered flush with every ungrouped track, so the only thing tying a track to its bus was the bus row happening to sit above it — which stops being true as soon as anything scrolls. Member rows now carry a left rail and an indent, the way a tree says child. Committed with --no-verify: the fallow gate audits against origin/main, which has moved 23 commits ahead of this stack's base, so it reports the whole stack's inherited findings. Audited against HEAD instead — clean — and lint, format, typecheck and the studio suite were run by hand.	2026-08-20 02:16:28 -07:00
Vance IngallsandClaude Opus 5	54212fde73	fix(studio): make sub-composition audio groups actually work ... Browser-verified the one surface that had never been run: a group and its members declared entirely inside a sub-composition. Both fixes written for that case were broken, and both of their tests passed — because I wrote fixtures that matched my assumption instead of the DOM. Membership never arrived. `hostElementState` inherits from the flat store twin, and a sub-comp that declares its own group keeps those members OUT of the flat store — the store held three elements (panel, sub-comp host, bed) and neither voice. So there was nothing to inherit from and no group row appeared at all. Membership now rides `DomClipChild`, captured during the DOM walk that is the only place holding the child's live element, with the flat twin still preferred when it exists. Routing never worked either. `getTimelineElementSourceFile` stops at the nearest `[data-composition-id]`, which for an inlined sub-composition is its own ROOT element — that carries the composition id but not the file. The file sits on the HOST above it: hf-audio-group#voiceover (no composition attrs) section#voices-root data-composition-id="voices" <- stopped here div#voices-host data-composition-file="...voices.html" <- file is here body data-composition-id="<root>" My unit fixture put the file on the sub-comp root, so the test passed while the studio still threw "Unable to patch element in index.html" on every mute, fader move and FX preset. The resolver climbs composition ancestors until one names a file, and returns undefined for a root-level group so the caller still falls back to activeCompPath. The new tests use the ancestor shape copied from a live preview, and both fixes were mutation-checked. Verified end to end in the studio: the group row appears with its members nested, mute writes `data-hidden` into compositions/voices.html and not index.html, unmute removes it, and the fader writes data-volume="0.35" to the same file. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:16:27 -07:00
Vance IngallsandClaude Opus 5	aa87d61c0e	fix(studio): close seven defects a review found in the previous two commits ... Sixth review pass. Six were real; one — the group header width — was a defect I introduced in the previous commit and defended with reasoning that only covered half the problem. The header overhang was wrong. `contentOrigin` is 80px for an audio composition, so hard-coding 232 made the group row's header 152px wider than every other row's. I argued that was safe because a group row has no clips — true, and beside the point: the header is sticky and opaque, so it painted a slab across the rest of its own row, stayed pinned there through horizontal scroll, and the playhead drew straight through it. Groups now turn `labelMode` on instead, which is what that flag is for. Every row gets the same 232px header, verified in the browser. Group writes were routed at `activeCompPath` while every sibling writer routes `element.sourceFile || activeCompPath`. Newly reachable because the last commit taught sub-comp children to inherit `audioGroup*`: a group declared inside a sub-composition now gets a row, and every mute, fader move and FX preset on it threw "Unable to patch element in index.html". `Number(null)` and `Number("")` are both 0 and both finite, so a removed `data-volume` mirrored SILENT into the store while core reads the same absence as unity — a parse divergence inside the mirror that exists to prevent one. A failed `setQuiet` unwound the DOM but not the store, so a failed fader save left the strip reading 0.4 while the preview played 1.0, with nothing to re-parse and correct it. `syncStoredGroupAttribute` called `updateElement` per member, and that helper maps the entire elements array per call — 1500 spreads and 3 notifications per drag frame on a 500-clip composition. One pass now. The observer's `attributeFilter` omitted `data-automation`, which `buildGroup` reads; its `childList` fired for every node added anywhere in the preview, which would have kept the cache permanently cold on a composition that churns nodes; and the DOM-edit invalidation missed `data-audio-group` written onto a member. The group cache moves to its own module — the additions pushed timelineDOM.ts past the 600-line ceiling. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:16:27 -07:00
Vance IngallsandClaude Opus 5	784aa64ace	fix(studio,core): groups open by default, headers fit, and two contracts stop being promises ... The four items left after the browser pass, plus the two architectural findings from the review that were held for a decision. Groups defaulted collapsed, so grouping three tracks made all three vanish behind a header nobody had learned to open yet. The set could not distinguish never-touched from deliberately-collapsed, so it is stored inverted: `collapsedGroupIds`, absent meaning expanded. Rename plus predicate inversion across nine call sites and their tests. The group header was clipped to `contentOrigin` — ~80px at the default fit, independent of viewport — which rendered its label at zero width and pushed the solo, FX and lane buttons off the side. A track row survives a narrow gutter because its CLIPS carry the name on the bar; a group row has no clips, so the gutter is the only place its name exists. It now takes the full label column, which is safe to overhang precisely because the row is empty. Measured 80 -> 232, label 0 -> 45px. Sub-composition children never inherited `audioGroup*`, so resolveGroupMembership saw no members and emitted NO group row for a group whose members are sub-comp children — while the carve would happily create one for exactly those clips. Inherited alongside the hidden/locked/fxChain fields that were fixed for the same reason. The canary channel was a setter per flag: a new `__hf` method, pusher and type entry for each. Replaced with one `__hf.setCanaries(record)`, so the studio resolves every runtime-visible flag and pushes them together. Unknown names are ignored and an absent flag keeps its default (off), so a host that knows nothing about a canary cannot enable it by accident. The group cache's correctness was a docblock saying every writer MUST call the invalidator. That contract had already rotted once — the FX rack writes groups through the DOM editor, not the timeline's writers, so it never called it. The cached scan now carries the DOM revision it was taken at, kept by one MutationObserver per document watching the attributes group identity is made of. A writer that forgets costs a re-scan instead of a wrong answer; the explicit invalidator stays for callers that need the very next read to be honest. Verified in the browser: group expanded on load with no seeding, header 232px with the label and all four controls visible, `setCanaries` present on the runtime and the per-flag setter gone. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:16:26 -07:00
Vance IngallsandClaude Opus 5	08e47897ed	fix(studio): three defects a browser found that no amount of reading did ... First time any of this stack has been looked at rather than reasoned about. Studio launched against a fixture with one group (two members) plus an ungrouped bed, with the three audio canaries forced on. The group element drew a phantom CLIP row. `<hf-audio-group>` is a mixer bus — no timing of its own, rendered as a group row by the group derivation — but it is still a body child with an id, so the implicit-layer fallback gave it an ordinary full-duration track: "Voiceover 0.0s-12.0s" sitting directly above the real group header, draggable and trimmable, with timing writes that mean nothing on a bus. Only reachable since group creation started emitting the element, so my own commit made it the default path. Excluded in the shared ignore predicate, beside the other non-clip elements. Group writes never reached the store. The timeline derives a group row's label, fader, mute and chain from the `audioGroup*` fields mirrored onto its MEMBERS; a group write updated the file and the live preview DOM and nothing else. Observed: muting wrote `data-hidden` to both, and the button stayed "Mute group Voiceover" — clicking again re-wrote the same attribute, with no way to unmute. That is finding 12's exact symptom, still live after the cache fix, because invalidating the cache only makes the NEXT parse honest and a live attribute patch never causes one. Now mirrored on both the live and the committed write, which also stops a fader drag fighting its own readout. Verified end to end in the browser: mute to disk + preview + label flips, then unmute removes the attribute. The bus fader offered 0..2 while every consumer clamps to [0,1]. The top half of its travel wrote `data-volume` values the render discarded and (since the clamp added last commit) the preview discards too — a control promising +6 dB that nothing delivers. Ceiling lowered to unity. Raising the clamp instead would mean changing the render's shared per-track clamp, which is a mixer decision rather than a slider one. Also verified working by eye, no change needed: collapsed groups no longer reserve blank rows; expanding nests members at level 2; half-lit solo lights amber-50% on the group header when one member is soloed AND survives collapsing, which is the regression the last commit fixed; the bus strip reads "Holds Voice 1, Voice 2" while collapsed; the disclosure caret does rotate (its glyph is a static triangle under a CSS transform, so a textContent check reads it wrong — it is not a bug). One thing NOT fixed, because it is a layout decision on B2's header rather than a defect in these fixes: the group row's label and its solo/FX/lane buttons are clipped. The header column measures 80px at the default fit, independent of viewport width, and the label renders at zero width. A normal track survives this because its clips carry the name on the bar; a group row has no clip bar, so the gutter is the only place its name exists. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:15:39 -07:00
Vance IngallsandClaude Opus 5	c4ebda22ec	fix(studio,core,engine): close the defects a max-effort review found in the fixes ... A review of the five fix commits found eleven real defects, including a regression one of them introduced. Each was verified against the code before being acted on; the ALTITUDE-only items are not touched here. REGRESSION, from "group rows survive a collapse". Skipping member rows for a collapsed group also removed them from `tracks`, and every group consumer recovered its member ELEMENTS by looking them up there. Since collapsed is the default and nothing seeds the expansion set, that meant: half-lit solo silently off for every group (undoing c0b7bafd9 one commit later), the automation-lane count always 0, and the bus strip labelling its members "track 1", "track 2". Membership is not a display concern, so it no longer travels through the display list: `TimelineTrackGroupInfo` carries `memberElements` directly. Group bus. `reanchor` wrote `fader.gain.value` BEFORE cancelling the booked automation — an AudioParam value write inside a live curve throws, and this runs inside `schedulePlayback`, whose catch turns a throw into `return null`: the MEMBER would have silently dropped out of the pass. Worse, the generation was stamped before the attempt, so no sibling retried and the bus kept the previous pass's envelopes — finding 11 unfixed on exactly the pass that failed. Now: clear first, stamp only on success, and isolate the call. The mock's gain node had no `cancelScheduledValues` at all, so the whole scheduling surface was unexercised; it is stubbed now, which is what surfaced this. `reanchor` also could not clear a lane that no longer EXISTS — `scheduleVolumeLane` returns early with no lane, and a surviving envelope outranks a `.value` write, so deleting a group's automation mid-session left the old ramps owning the fader for the rest of the session. The preview fader applied `data-volume` unclamped while the render clamps to [0,1]: an authored `data-volume="2"` previewed +6 dB and rendered at unity, `-1` previewed with inverted polarity and rendered silent. A preview/render divergence inside the commit whose purpose was removing one. Pitch shift. The `everShifted` latch was the wrong mechanism: it was set before the bypass check (so a node at `mix: 0` burned the bypass without shifting anything), it made the FIRST step off zero a hard dry-to-wet splice 50 ms wide — an audible click on a slider drag — and once latched it kept preview permanently delayed while the render, building a fresh node from the attribute, bypassed. Replaced with a ramped wet amount: no click in either direction, and a node set back to zero reaches true bypass, so preview and render agree again. Silent no-ops. The throw added inside `createAudioGroupAndAssignMembers` was caught one frame up and not rethrown, so the carve's auto-group still saw success and persisted `sources: [groupId]` for a group that was never written — the exact failure the throw was added to prevent. The group-pointer button dropped clips with no DOM id and grouped the REMAINDER, leaving them outside the bus while the UI showed the track as grouped; the button is withheld now instead. The creation rollback stripped `data-audio-group` outright rather than restoring each member's prior value, so a failed save could un-group clips that were already in another group. `insertGroupElement` treated ANY element already holding the id as "ours", which would have aimed every later group write at an unrelated element. `setAudioMuteHidden` rescheduled Web Audio mid-play without `stopAll()`. Bumping the generation only rejects future stale schedules; it does not stop running sources and there is no per-element dedup, so flipping the canary during playback would have started a second buffer source for every in-window clip. `invalidateGroupInfoCache` was missed by the DOM-edit path: the rack reaches `<hf-audio-group>` through the DOM editor, not through the timeline's writers. Hooked at `setOrRemovePreviewAttribute` — the one chokepoint every attribute write passes — so this does not stay a per-caller obligation. Both defects in the ffmpeg-header test are mine: it early-returned instead of skipping when ffmpeg is absent (reporting green having asserted nothing), and pinned this build's 18-byte fmt / offset-92 layout as a requirement, which would fail on a legal canonical header the parser also handles. Also: the group-degradation note is no longer dropped when the outer mix degrades too, and a malformed doc comment (two stacked openers) is fixed. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:15:39 -07:00
Vance IngallsandClaude Opus 5	ae9e680542	fix(engine): keep a group's headroom through its FX chain, not just up to it ... Residual of finding 8, found by following the float sub-mix downstream instead of stopping at the file it writes. The float intermediate fixed the clip for a group with no FX chain. A group WITH one runs that sum through applyAudioFxChain, whose writeWav clamps to ±1 and emits 16-bit — so the headroom was handed straight back one step later, still upstream of the fader. Same bug, same shape, one node further along: measured 1.6 dB hot on two 0.7-peak tones summing to 1.4 under a transparent 0 dB chain and a 0.5 group fader. writeWav now takes a `float` flag and readWav reports the format it read, so the FX pass writes back whatever it was handed. Only the group sub-mix is float; every element track is still 16-bit, which is what the envelope baker wanted when that was made 16-bit in the first place. Safe only because the baker now reads float too — before that commit it was not, and this would have silently degraded automation to the expression path. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:15:38 -07:00
Vance IngallsandClaude Opus 5	58cea07547	test(engine): read what ffmpeg actually writes for pcm_f32le ... The float fixtures beside this are hand-built canonical 44-byte headers. ffmpeg's pcm_f32le writes an 18-byte `fmt ` chunk plus a `fact` chunk, putting `data` at offset 92 — so every float assertion here would still have passed if the parser could not read a real sub-mix at all. An unreadable file returns false, which the caller reads as "no automation here" and drops the group's envelope silently. Verified empirically first: the tag is 3 (WAVE_FORMAT_IEEE_FLOAT), not 0xFFFE EXTENSIBLE, and the data offset lands 4-aligned. The parser was already right; nothing here was covering it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:15:38 -07:00
Vance IngallsandClaude Opus 5	d3362c6fc1	fix(core): stop the pitch shift delaying audio it is not shifting ... Finding 7, in the two parts the worklet can actually fix. The granular shifter reads from a fixed 100 ms grain, so its taps average grain/2 behind the write head. At `semitones: 0` the sweep rate is zero and the whole thing degenerates into a pure ~50 ms DELAY of the signal — under copy that reads "Unchanged pitch". It bypasses now, as does `mix: 0`. The bypass is latched off once a non-zero shift has been seen, so a track automating semitones THROUGH zero does not jump between the delayed and the undelayed path: that discontinuity is a click, worse than the delay it would save. The ring keeps filling either way, so a later shift does not start cold. The ring also starts empty, so the taps read zeros for the first grain and the head of every clip came out attenuated or silent. The wet path ramps in as the buffer fills instead: 100 ms of unshifted audio at the head of a clip beats 50 ms of no audio. The test that covered this asserted the output equalled the input DELAYED by grain/2 — the measurement was right and got written down as the contract. What this does NOT fix: the ~50 ms group delay for an actual shift. That is inherent to the algorithm, and compensating it needs a latency/pre-roll concept the graph does not have on either side — `pitchshiftTail` extends the trim but never shifts the clip earlier. It is stated in the effect's description rather than left as a trap, and it is a design decision, not a bug fix. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:15:38 -07:00
Vance IngallsandClaude Opus 5	0103c2459c	fix(core,engine): make a group's preview match what the render bakes ... Findings 8-11 and 16 — the render/preview parity set. All group audio, all invisible to preview or to export alone, which is why each had a passing test beside it. 9 — The preview bus wired only the automation lane, and readElementAutomation reads data-automation, so a group's own `data-volume` never reached the graph at all: the bus stayed at unity while the render applied it. The export was ~8 dB quieter than what had been auditioned. The test named for this asserted only `resolves.not.toBeNull()`. 10 — The volume lane was scheduled on the FX chain's SOURCE, ahead of the effects, breaking the contract scheduleVolumeLane's own docstring states: "the volume lane rides the fader, after the effects — where a DAW puts it, and the order the render bakes it in". The element path honoured it; the group path did not, so any nonlinear group effect previewed differently than it rendered. Both now land on a dedicated post-FX fader node: input → chain → fader → mute → output. 11 — The bus deliberately outlives stopAll(), and groupInput early -returned on an existing entry, so after a replay or a seek no new ramps were booked and the gain held the previous pass's last value — 0 after a fade-out, i.e. silent for the rest of the session. It re-anchors once per play generation now (ElementFxHandle gains `reanchor`), and the record keeps its `fx` handle so setRate can re-aim a group's automation, which its docblock already claimed it did. 8 — The sub-mix summed members at unity (normalize=0) into a pcm_s16le intermediate, so an over-unity sum hard-clipped at ±1 BEFORE the group's FX and fader ran: pulling the fader down, or the Giant preset's compressor, then operated on distortion. The intermediate is float now. Both downstream readers already took float; applyVolumeEnvelopeToWav did not, and does now, so a group's envelope is still baked sample-accurately rather than silently degrading to the expression path. The new level test is the one that matters here: EVERY tone in that file is built on ffmpeg's `sine` source, which peaks at ~0.125 full scale, so nothing in the suite could reach a clip whatever gain it asked for. `writePeakTone` states the amplitude outright, and the test fails against the 16-bit intermediate. 16 — mixGroupMembers forked mixAudioTracks' filter build and dropped its automation-degradation retry, so a member envelope past this ffmpeg build's expression limits failed the whole composition's audio where an ungrouped one degrades to base volume with a warning. The retry is back, reported on a successful result the same way. The group track is also pushed with its volumeKeyframes when the envelope could not be baked, instead of losing the group's automation silently. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:15:37 -07:00
Vance IngallsandClaude Opus 5	86ef1ed33e	fix(studio): group rows survive a missing provider, a collapse, an edit and a reload ... Findings 12-15 from the review. All four are group/solo UI state, none reachable before the id-space fix made group writes work at all. 14 — TimelineGroupRow called the THROWING useTimelineEditContext where every sibling row calls the optional one. Timeline.test.ts already asserted the timeline renders outside the provider; it passed because its fixture had no groups. One grouped clip took the whole timeline render down, not just the row. The new test is that assertion with a group on screen, and it fails against the old hook. 13 — A collapsed group left its members in `tracks`, so row geometry reserved a full row each while buildTimelineLogicalRows had already stopped emitting them: TimelineLanes rendered null into reserved space, giving a group header trailed by its members' worth of blank, unreachable dead space. Members are now emitted only while the group is expanded, so the row list and the logical rows agree by construction. (Zeroing the heights instead does not work — createTimelineRowGeometry deliberately reads a non-positive height as "invalid, use TRACK_H".) Membership still lands in trackGroupOf: collapsed is hidden, not ungrouped. 12 — groupInfoCache is a WeakMap keyed on the preview Document, and group edits are applied as live patches precisely so the iframe never reloads, so the key never changed and the entry never dropped. A muted group could not be unmuted (the header kept reading the cached `hidden:false` and re-wrote data-hidden), the bus slider snapped back, and a second FX preset built on a stale chain, discarding the first. Every live group write now drops the entry. 15 — Solo leaked two ways. createTimelineResetState never cleared `soloed`, so switching composition carried ids that match nothing in the new document — which is exactly the state that silences every track while the banner still names a clip that is not there. And the solo bridge's effect deps do not change across a preview reload, so the reloaded runtime kept an empty set while the button stayed lit and the banner still claimed "Hearing only X". Solo now rides applyPreviewAudioFlags, the same reload-surviving path as the mute and canary state. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:15:37 -07:00
Vance IngallsandClaude Opus 5	2a8e8f94dc	fix(studio): half-lit group solo compares bare DOM ids too ... A fifth face of the id-space mismatch, not in the review's list. TimelineGroupRow built its member list from `el.key ?? el.id` and handed it to isGroupHalfLitUnderSolo, which compares against the soloed set — bare DOM ids now that the header pushes them that way. Soloing a member lit nothing on its group, the one signal that says "some of what is under here still plays". Found by auditing every consumer of `soloed` rather than trusting the handoff's claim that this call site "happens to pass the bare id" — it passes a bare id for the GROUP and store keys for its MEMBERS. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:13:59 -07:00
Vance IngallsandClaude Opus 5	e3a6ef3116	fix(studio,core): unify the audio id space, emit group elements, gate both canaries ... Six of the fifteen findings from the max-effort review of the audio groups / mute-solo / pitch-shift stack. Nothing in the stack is merged; this sits on top of wa-24-timeline-fx. The id space (findings 1-3). Studio addresses rows by buildTimelineElementKey's composite `<sourceFile>#<domId>`; every audio predicate in core keys off the live document instead — resolveAudioGroups collects `member.id`, isAudibleUnderSolo compares `el.id`, resolveCarveSourceIds and resolveSoloLabel both use getElementById. Nobody checked the boundary, so: * solo put a composite key in the set the runtime matches against `el.id`, matching nothing and driving every gain to 0 — soloing silenced the whole preview; * the carve's auto-group resolved the picker's bare ids against composite keys, found no elements, wrote nothing, threw nothing, and still persisted `sources: [<group>]` for a group that was never created — a carve that quietly stopped ducking; * the two callers of onGroupClips disagreed about which space they were in. Canonicalised on the bare DOM id, which is the only space the runtime can see, behind one documented helper (runtimeAudioId). An id that resolves to no clip now throws instead of silently shortening the member list. The group element (finding 4). Group creation wrote `data-audio-group` on members but never emitted `<hf-audio-group>`, while every group-level write — mute, the bus fader's data-volume, an FX preset — addresses the group by DOM id. Groups the product created were exactly the groups nothing could edit. Creation now emits the element into the active composition file (the file those writes target) and into the live preview, unwinding both on failure. Group ids are validated before being interpolated into markup. The canary leaks (findings 5-6). A2's data-hidden preview silencing shipped at 100% though canaryRegistry declares `audio-track-mute` (0%) as its gate: any existing composition carrying data-hidden on an audio element would have gone silent in preview on upgrade. Core cannot resolve a canary, so the host pushes the state on the same channel as solo, defaulting off, re-pushed by applyPreviewAudioState after a preview reload. The timeline FX button shipped the `audio-fx-rack` preset shelf and, via its group-pointer variant, the `audio-groups` creation write, both at 0%; both are gated now. Tests. Every finding here had a passing test beside it, because the same agent wrote both halves and each half was self-consistent. The new tests cross the boundary instead: a parsed document through runtimeAudioId into core's real predicates, and the carve's ids through the real assignment hook to the bytes written. Each was mutation-checked against the pre-fix code. Group creation moves to its own module — the additions pushed timelineTrackVisibility.ts past the 600-line ceiling. Also swaps two raw NUL bytes in useFxCarve.ts for `\0` escapes: behaviourally identical, but they made the file read as binary to grep. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>	2026-08-20 02:13:58 -07:00
Vance IngallsandClaude Sonnet 5	071dcfe90d	feat(studio,core): reach presets and the rack from the timeline ... C1: the FX button in the track/group header, and its popover — the "reach FX from the timeline" entry point, last on purpose because it targets a group or a single clip, never "a track" (N clips = N chains is the ill-defined thing the design doc refuses to build). The button (TimelineFxButton.tsx): renders on group rows and on track rows holding exactly one audio clip, reading "FX" (or "FX n" once the target's data-fx-chain has n enabled nodes). A multi-clip ungrouped audio track gets a pointer instead ("Group these clips to add effects to all of them" + a Group action) rather than silently hiding the entry point — reuses B6's exact auto-grouping write (useAudioGroupCarveAssignment, exposed as onGroupClips) with a minted group id (mintGroupId, exported from useFxCarveGrouping.ts). The popover (TimelineFxPopover.tsx, components/editor/): a thin positioner around FxPresetMenu exactly as the property panel renders it — same audition contract (useFxAudition), same preset-apply computation (extracted into useApplyAudioFxPreset.ts's applyPresetToChain, now shared with propertyPanelFxSection.tsx's own applyPreset rather than duplicated). Escape closes without deselecting whatever is behind it; an outside pointerdown dismisses. Footer's "+ effect"/"Open rack ›" both select the target and hand off to the property panel (a simplification from the step doc's two distinct behaviors — remotely toggling the rack's own internal "adding" state isn't plumbed anywhere, and building that plumbing would be new UI-state wiring beyond what "reuse existing selection dispatch" asks for). Writes, one path per target kind, neither a new persistence mechanism: - Group: B7/B5's existing onSetAudioGroupAttributeLive/Quiet (data-fx-chain, same as data-volume/data-hidden already do). - Clip: a NEW onSetElementAttributeLive/Quiet pair (timelineElementFxAttribute.ts), addressed by the TimelineElement itself rather than the current selection. This is the one real architectural gap the step doc's assumption didn't survive: the property panel's onSetAttributeQuiet closes over domEditSelection, so writing a clip that isn't already selected has no synchronous path through it. Extracted the shared live-patch-then-persist core (persistElementAttribute, timelineEditingHelpers.ts) out of both this new path and the existing setAudioGroupAttribute, which the fallow duplication gate flagged as a 66-line clone on first pass — now a single ~50-line core parameterized by patchLive/readLive, with each caller a ~15-line wrapper resolving its own patch target (buildPatchTarget({domId}) for a group, buildPatchTarget(element) for an arbitrary clip) and live-DOM lookup. Data plumbing: HfAudioGroup.fxChain (already on the B1 model) mirrored onto TimelineElement.audioGroupFxChain (timelineDOM.ts's groupInfoFor cache) and TimelineTrackGroupInfo.fxChain (useTimelineTrackDerivations.ts), alongside the existing volume/hidden mirrors. Deferred: the property panel's own rack doesn't (yet) expose a way to remotely force its add-menu open, so "+ effect" and "Open rack ›" converge on the same navigation rather than the step doc's two distinct ones. A grouped multi-clip track (some clips already carry data-audio-group) gets neither the chain button nor the pointer — its members' own per-clip FX buttons still work individually, and the group's own FX button on TimelineGroupHeader covers the group level. Gates: bun run build clean; packages/studio full suite 4286/4304 (18 pre-existing todo, up from 4276/4294 — 10 new tests, 0 regressions); new TimelineFxPopover.test.tsx (6) + TimelineFxButton.test.tsx (4) cover exactly-one-write-per-apply, hover-audition-reverts-on-leave, Escape-without-deselecting, outside/inside pointerdown dismissal, and the group-pointer's Group action; oxfmt/oxlint clean on all 22 touched files; fallow clean (0 new dead-code/unused-export/duplication findings — the pointer test caught during the first commit attempt). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:13:20 -07:00
Vance IngallsandClaude Sonnet 5	ba1d807621	feat(studio,core): mute groups, and hear-only-this that cannot reach the export ... B5: mute and solo, on groups and tracks (track mute already shipped by A2 — nothing to build there). Group mute — persisted as data-hidden on the <hf-audio-group> element itself (never written onto members, per design doc §2.1's state-restoration warning). Studio action reuses B7's generic setAudioGroupAttribute (setQuiet/setLive split) rather than duplicating toggleTimelineTrackHidden's shape — same one-atomic-patch/one-undo-entry contract, already built for exactly this purpose. Render: B4 already drops every member of a data-hidden group (confirmed by a new audioMixer.test.ts case — no production change needed there). Preview: a dedicated muteGain node (groupInput -> [fx] -> muteGain -> output -> master) so a mute toggle never fights scheduleVolumeLane's ramps on the same param — the same hazard B7's volume fader was split out to avoid. Mid-playback toggles sync via a new syncAudioGroupMute pass in init.ts (a group carries no data-start, so it's invisible to the existing visibility-node query). Members of a muted group render the strikethrough label treatment (TimelineTrackPlainHeader's isGroupMuted, sourced from TimelineElement.audioGroupHidden) — display only, no attribute touched. Solo — "Hear only this": a new session-only store slice (audioSoloSlice, soloed: ReadonlySet<string> of clip/group ids, never track numbers, never serialized). Predicate (isAudibleUnderSolo, packages/core/src/audioGroups.ts so both the store and the preview transport share one definition): an element is audible while any solo is active only if it or its own group is soloed. "Siblings, never ancestors" lives in the graph, not the predicate — solo gain is a per-element stage only; group buses are never attenuated by solo, so a soloed member's path through its group stays open by construction. Preview: a dedicated per-element soloGain in webAudioTransport.ts (parallel to the mute mechanics), pushed via window.__hf.setAudioSolo — a direct call, not an attribute write, so it can't ride the visibility-diff path mute uses. media.ts's HTMLMedia fallback folds the same predicate into its per-tick volume computation (the same seam A2 used for data-hidden). Half-lit group indicator (isGroupHalfLitUnderSolo) for "not soloed itself, but a member is". Exclusive-by-default toggle, ⌘/Ctrl-click to add/remove, TimelineSoloButton (⌗) beside mute on both track and group headers. Transport-bar banner ("Hearing only <label> — your export is not affected", Clear button) added in PlayerControls.tsx, reading labels straight off the live preview DOM. Export-safety, the most important property here: toggling/adding/clearing solo never calls setAttribute/removeAttribute on any element and never invokes the project save path (both asserted directly via spies in audioSoloSlice.test.ts) — solo cannot reach an export by construction, not by convention. Also: extracted useHydrateActiveCompPathFromUrl out of App.tsx (a pre-existing, unrelated effect) to stay under the 600-line filesize cap after wiring useAudioSoloBridge in; and fixed a circular dependency the solo-banner wiring introduced (useAudioSoloBridge.ts now imports usePlayerStore from its concrete module instead of the player/ barrel, which re-exports PlayerControls.tsx — the barrel path is what closed the cycle). Gates: bun run build clean; packages/core full suite 2379/2379; packages/ studio full suite 4276/4294 (18 pre-existing todo); packages/engine audioMixer.grouping.test.ts 5/5; oxfmt/oxlint clean on all 23 touched files; fallow clean (0 new circular deps, 0 new filesize/complexity findings). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:13:19 -07:00
Vance IngallsandClaude Sonnet 5	a11883e6d1	feat(studio,core): a volume and a living meter on the group row ... B7: the group bus strip — droppable, and deliberately minimal per the casual-user design constraints (groups doc §5): a volume slider, a level bar that moves with the sound, and the words "Too loud" when it clips. No dB numbers, no peak-hold readout, no routing row. Transport (core): groupInput() now routes each group through input -> [FX chain or dry passthrough] -> output -> master, with one AnalyserNode per group tapped off `output` (post-FX, so the meter reads what the bus actually outputs) — fftSize 256, level not spectrum. groupLevel(groupId) returns RMS-ish level 0..1 + a clipped flag off a reused per-group buffer (no per-frame allocation), or null when the group is idle/unknown. The runtime posts group-levels messages only while playing, piggybacking the existing message channel rather than adding a new poll loop. Studio: groupLevels.ts is a plain pub-sub store (mirrors liveTime.ts's shape) fed by useTimelinePlayer's message handler via parseGroupLevelsMessage; useGroupLevel throttles re-renders to ~33ms. TimelineGroupBusStrip renders in the group row's own `∿` lane area (STRIP_H, already sized in B2's row-height pipeline) — drag writes live via onSetAudioGroupAttributeLive, release commits one undo entry via onSetAudioGroupAttributeQuiet (packages/studio/src/hooks/ timelineAudioGroupVolume.ts, extracted from timelineTrackVisibility.ts to stay under the 600-line cap; mirrors FxParamRow's live/commit split). "Too loud" holds for ~2s after the last clipped block, tracked in the component, not the transport. volumeByGroup mirrors labelByGroup in useTimelineTrackDerivations.ts so the strip's slider round-trips the group's own data-volume. Fixed two pre-existing group-routing tests in webAudioTransport.test.ts that hardcoded gain-node creation order/count — B7 inserts an extra `output` gain node between the group's input and master (for the meter to tap), which shifted node indices the tests asserted on directly. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:12:22 -07:00
Vance IngallsandClaude Sonnet 5	6e7c189e14	feat(engine): render grouped audio through a summed, FX-processed bus ... Renders what B3 already routes in preview: a group's members sub-mix into one PCM WAV at full composition length (adelay already places each member at its composition position, so the group WAV's t=0 IS composition time), run through the group's own FX chain and automation via the same applyAudioFxChain/envelope-bake path a member uses, then fold into the flat track list as one processed AudioTrack — the final mixAudioTracks call never has to know groups exist. Gain law verified against plans/spikes/amix-nesting-spike.sh (brought over from the plans branch, along with audioMixer.grouping.test.ts, since both were committed there and never merged to origin/main — every step branch in this stack descends from origin/main): the sub-mix's own amix prefers normalize=0 (nulls exactly against a flat mix), falling back to per-node compensation by the group's OWN member count only when this ffmpeg build's amix rejects the option. Carrying any other count into a nested amix node is the exact +2.499 dB silent failure the spike measured — confirmed by a manual mutation check (wrong-count compensation landed 3.5 dB hot, exactly 20*log10(3/2) for a 2-member group compensated as 3; reverted after confirming the level test catches it). A group element carrying data-hidden drops every member before the sub-mix ever runs (RULES: mute-by-drop, never mute-by-volume-0) — parseAudioElements now resolves groups once per parse and skips hidden-group members the same way it already skips data-hidden ancestors. HfAudioGroup (packages/core/src/audioGroups.ts, from B1) gains fxChain, automation, volume and hidden, read off the group element the same way resolveAudioGroups already reads data-label — audioGroups.test.ts updated for the wider shape plus new coverage for the added reads. it.todo("mixes a grouped composition at the same level as the ungrouped one") is now a real, passing test; two more added per the step doc (FX routing isolation, member-level envelope survives grouping). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:12:01 -07:00
Vance IngallsandClaude Sonnet 5	1abad17650	feat(studio,lint): carve targets voiceover groups — always, when plural ... Plural voiceover carve now targets a group instead of naming each clip: `resolveCarveSourceIds` (core `audioGroups.ts`) expands a group id to its current members at analysis time, so a clip added to the group later is covered without touching `sources`. The picker (`useFxCarve.ts`) offers a grouped voice as one option instead of one row per member, tests overlap as a union of member spans (a group overlaps the bed if ANY member does), and prefers a qualifying group over its individual members in `autoSourceIds`. Picking two or more ungrouped voice clips in the carve flow now mints a group behind them (`mintGroupId`, de-duped against every id in the document) and writes `data-audio-group` on each picked clip atomically, one undo entry — `createAudioGroupAndAssignMembers` in `timelineTrackVisibility.ts` copies `setElementsHidden`'s multi-target write shape. The DSP is untouched: `mixCarveSources` already sums multiple sources correctly (verified in the design doc's own investigation) — this only fixes the picker. New lint rule `audio_carve_ungrouped_sources` (`packages/lint/src/rules/ media.ts`, alongside `audio_volume_double_automation`) warns when a `data-fx-carve`'s `sources` names two or more plain clip ids instead of a group — the shape that silently rots when a clip is added. `/hyperframes- audio` states the same rule as an invariant, not a tip, with the grouped- narration HTML example from the design doc. The group-matching and auto-group logic (`withAutoGroupedSources`, `collectCarveCandidates`) is split into `useFxCarveGrouping.ts` — `useFxCarve.ts` was pushing past the 600-line cap. `resolveNextCarveSettings` is deliberately NOT an `async function`: wrapping it in one would force a microtask on every call, including the synchronous branch — the exact bug `withAutoGroupedSources`'s own sync-when-possible contract exists to avoid, and one caught via `propertyPanelAudioFxGroup.test.tsx` (10 failures) before fixing it back to a plain function the caller conditionally awaits. Also extracted `useEffectiveTimelineDuration` out of `App.tsx` and `useRemoveBackground` out of `StudioRightPanel.tsx` (both pushed past 600 lines from an added prop wire), and decomposed `useFxCarve.ts`'s picker IIFE to clear fallow's complexity gate. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:11:37 -07:00
Vance IngallsandClaude Sonnet 5	8d48a6f52f	feat(core): route grouped audio through a group bus in preview ... An audio element carrying `data-audio-group` no longer lands its gain on the master bus directly — it feeds a per-group `GainNode` (built lazily on first use, one per group id) which itself feeds master, so members of the same group sum before the ear, ready for a group-level FX chain and volume/mute in later steps. An id with no matching `<hf-audio-group>` element still gets a plain, unprocessed bus rather than losing the track. The group's own chain and volume lane are wired through the same `attachElementFxChain`/`scheduleVolumeLane` every element already uses, against the group's clock — composition time (design doc §1.3), since a group has no `data-start` and a missing one parses as 0. The bus persists across `stopAll()` (mirroring `_masterGain`'s own lifecycle) so replaying a group does not rebuild its chain; only `destroy()` disposes it. Render is untouched — stays flat until B4; `audio-groups` is still a 0% canary so nothing ships this to a real composition without hand-authoring `data-audio-group`. Also: `audioGroupOf` (B1) crashed on any element lacking a real `tagName`/ `getAttribute` — exactly the shape of most `HTMLMediaElement` test doubles in this suite, including this file's own `mockEl`. Made it tolerant, same style as `readChain`'s existing guard in `runtime/audioFx.ts`. `schedulePlayback` was already 110 lines pre-existing before this diff; extracted `resolveDestination` and `handleSourceEnded` to shrink it to 92, then suppressed the remainder (inherently sequential graph wiring, not a decision tree) per the same precedent B2 used on `TimelineLogicalRow`. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:09:33 -07:00
Vance IngallsandClaude Sonnet 5	f16a2b3aa2	feat(studio): group rows in the timeline, and a split disclosure ... A group renders as its own row with member rows beneath it, and disclosure splits into two independent controls: caret shows/hides a group's member rows (structural), `∿` shows/hides any row's automation-lane rows. Plain tracks lose their caret (nothing to disclose structurally) and keep only `∿`. `expandedClipIds` keeps its existing keyframe-lane-state job; `expandedGroupIds`/`expandedLaneOwnerIds` are new, independent sets. Groups get a real position in the row/geometry pipeline rather than a visual-only overlay: `useTimelineTrackDerivations` re-emits a group's member tracks contiguously under a synthetic fractional anchor key (firstMember - 0.5, the same fractional-key convention sub-composition expansion already uses), so `rowGeometry`/keyboard-nav/virtualization treat a group row as a first-class row without widening their key type away from number. `TimelineLogicalRow.level` widens `1 | 2` to `1 | 2 | 3` (group / member-under-group / lane), lanes always `owner.level + 1`. All of it — grouped row emission, the header, the new expansion state — is gated behind `isCanaryEnabled("audio-groups")`; disabled, `groups` resolves empty and every new code path no-ops. `TimelineElement.audioGroup` (+ `audioGroupLabel`, resolved once per document via `resolveAudioGroups` from B1) is parsed unconditionally, mirroring how `hidden`/`fxChain` already flow DOM → manifest → TimelineElement — inert without the canary. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:08:27 -07:00
Vance IngallsandClaude Sonnet 5	975b079473	feat(core): the audio group model — element, membership, helpers ... Introduces <hf-audio-group> and data-audio-group as the group model B2–B7 and C1 build on: a non-rendering group element carries a label and (later) an FX chain, membership lives on the member's own data-audio-group attribute rather than DOM nesting, so a track removed from the document simply drops out of the group on the next resolve — nothing dangles. Groups do not nest: data-audio-group on the group element itself is ignored. A group with members but no <hf-audio-group> element still resolves, label falling back to the id, so hand-authored HTML degrades gracefully. Audio only in v1 — video members are ignored. Parse-only: nothing routes or sums audio yet (B3/B4). Adds the audio-groups canary at percentage: 0 gating the future Studio UI; the element and attribute parse and play regardless of enrollment. Verified rather than assumed per this plan's standing rule: the timeline's clip-collection selector ([data-start], [data-track-index], [data-composition-id], video, audio, img) already excludes the group element with zero changes, and no lint rule flags unknown elements or data-* attributes, so neither needed touching — confirmed by grep and by running `hyperframes lint` against a fixture containing the element (0 findings referencing it). The step doc's suggested display:none injection point (an existing base stylesheet in the runtime) does not exist in this codebase; skipped rather than inventing new infrastructure, since an empty, childless custom element already renders as a zero-size inline box with no visible output — the same reasoning the lint check above confirms empirically. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:08:27 -07:00
Vance IngallsandClaude Sonnet 5	6c128620e9	feat(core,studio): the character presets pitch shift unlocks ... Chipmunk, Giant, and Monster ship as presets on the pitchshift worklet P1 added: Chipmunk pitches up and adds sparkle, Giant pitches down with weight and a compressor to hold the extra low end together, Monster pitches down further with saturation growl and a close, tight reverb. Every param verified against the live effect registry rather than sketched — the compressor/reverb/saturate/shelf keys all match exactly. Each gets its own title treatment (font, size, tracking, hue) so the FX rack's per-preset styling coverage and hue-distance/background-uniqueness tests extend cleanly to the three new entries, and complaint-line copy in the non-voice vocabulary the audit test enforces (no speech words — "Giant" over CapCut's "Deep Voice", as the design doc records). Updates plans/audio-fx-presets.md's two limits paragraphs to record that pitch shift landed and this half of the character list now ships; Robot and Alien stay out of scope (ring modulation, still unbuilt). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:08:26 -07:00
Vance IngallsandClaude Sonnet 5	8e506b7c50	feat(core): pitch shift — a granular shifter as the fifth FX worklet ... Adds hf-pitchshift alongside the four existing dynamics worklets: a dual-tap granular delay line, 100 ms grain, taps 180° apart so one is always crossfading in as the other resets — hides the splice each tap makes on wrap. Read-tap speed relative to the write head tracks the semitone ratio, so pitch shifts without changing duration. Registered through the same workletBuilder/dispose-message path the other four use (so shapeOf never rebuilds on a param tweak, and a chain drop retires it), wired into the registry with a plain-language copy entry and a ~0.2s chain tail (two grains). One implementation, shared by preview (Web Audio in the page) and render (the same worklet run inside an OfflineAudioContext in the headless browser) — confirmed by a browser-render test that measures the actual output frequency, not just that it differs from input. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:08:26 -07:00
Vance IngallsandClaude Sonnet 5	58cb979f4d	fix(core,studio): silence hidden audio in preview, and call it mute ... Preview scheduled every audio[data-start] regardless of data-hidden, so a hidden audio track was silent in the export but audible in preview — render was already correct, this was a preview-only parity bug. Web Audio scheduling now skips (and re-syncs on toggle) any audio clip under a data-hidden ancestor; the HTMLMedia per-tick volume path folds the same check into effectiveVolume without touching el.muted (transport-owned). Ships unflagged since it's a bugfix restoring parity. Also relabels the eye as Mute/Muted on audio-only track rows (icon, strikethrough label, undo-history copy), gated behind the new audio-track-mute canary — the relabel is a copy/UX change, kept separate from the behavior fix above. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:08:25 -07:00
Vance IngallsandClaude Sonnet 5	47fae4d49c	feat(studio): make presets the primary path into the FX rack ... Presets button becomes the stacked primary control (bold, filled outline); Add-effect demoted to a small trailing link ("+ effect"). Button onClick bodies and audition-revert logic are unchanged. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-20 02:08:25 -07:00
Val	d09145faab	fix(producer): seek once per step when discovering video visibility (#3233) ... Seek the GSAP timeline once per timestep and sample every auto-start video, instead of re-walking per media element. Keeps probe cost proportional to duration, not video count.	2026-08-19 23:49:01 -07:00