@opendaw/studio-core 0.0.131 → 0.0.133

package/dist/project/audio/AudioContentModifier.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"AudioContentModifier.d.ts","sourceRoot":"","sources":["../../../src/project/audio/AudioContentModifier.ts"],"names":[],"mappings":"AAAA,OAAO,EAAY,IAAI,EAAiD,MAAM,kBAAkB,CAAA;AAShG,OAAO,EAAC,sBAAsB,EAAwB,MAAM,0BAA0B,CAAA;AAKtF,yBAAiB,oBAAoB,CAAC;IAC3B,MAAM,cAAc,GAAU,UAAU,aAAa,CAAC,sBAAsB,CAAC,KAAG,OAAO,CAAC,IAAI,CAelG,CAAA;IAEM,MAAM,cAAc,GAAU,UAAU,aAAa,CAAC,sBAAsB,CAAC,KAAG,OAAO,CAAC,IAAI,CA6BlG,CAAA;IAEM,MAAM,aAAa,GAAU,UAAU,aAAa,CAAC,sBAAsB,CAAC,KAAG,OAAO,CAAC,IAAI,CA+CjG,CAAA;CA0BJ"}
+{"version":3,"file":"AudioContentModifier.d.ts","sourceRoot":"","sources":["../../../src/project/audio/AudioContentModifier.ts"],"names":[],"mappings":"AAAA,OAAO,EAAY,IAAI,EAA4D,MAAM,kBAAkB,CAAA;AAS3G,OAAO,EAAC,sBAAsB,EAA8C,MAAM,0BAA0B,CAAA;AAK5G,yBAAiB,oBAAoB,CAAC;IAC3B,MAAM,cAAc,GAAU,UAAU,aAAa,CAAC,sBAAsB,CAAC,KAAG,OAAO,CAAC,IAAI,CAqBlG,CAAA;IAEM,MAAM,cAAc,GAAU,UAAU,aAAa,CAAC,sBAAsB,CAAC,KAAG,OAAO,CAAC,IAAI,CA6BlG,CAAA;IAEM,MAAM,aAAa,GAAU,UAAU,aAAa,CAAC,sBAAsB,CAAC,KAAG,OAAO,CAAC,IAAI,CA+CjG,CAAA;CAoDJ"}

package/dist/project/audio/AudioContentModifier.js

@@ -1,4 +1,4 @@
-import { EmptyExec, isDefined, isInstanceOf, RuntimeNotifier, UUID } from "@opendaw/lib-std";
+import { EmptyExec, isDefined, isInstanceOf, isNotNull, RuntimeNotifier, UUID } from "@opendaw/lib-std";
 import { TimeBase } from "@opendaw/lib-dsp";
 import { AudioPitchStretchBox, AudioTimeStretchBox, TransientMarkerBox, WarpMarkerBox } from "@opendaw/studio-boxes";
 import { AudioRegionBoxAdapter } from "@opendaw/studio-adapters";
@@ -15,6 +15,12 @@ export var AudioContentModifier;
         return () => audioAdapters.forEach((adapter) => {
             const audibleDuration = adapter.optWarpMarkers
                 .mapOr(warpMarkers => warpMarkers.last()?.seconds ?? 0, 0);
+            const loopOffsetSeconds = isInstanceOf(adapter, AudioRegionBoxAdapter)
+                ? adapter.optWarpMarkers.mapOr(warpMarkers => warpPositionToSeconds(warpMarkers, adapter.loopOffset), 0)
+                : 0;
+            if (loopOffsetSeconds !== 0) {
+                adapter.box.waveformOffset.setValue(adapter.waveformOffset.getValue() + loopOffsetSeconds);
+            }
             adapter.box.playMode.defer();
             adapter.asPlayModeTimeStretch.ifSome(({ box }) => {
                 if (box.pointerHub.filter(Pointers.AudioPlayMode).length === 0) {
@@ -57,7 +63,8 @@ export var AudioContentModifier;
                 }
             }
             else {
-                AudioContentHelpers.addDefaultWarpMarkers(boxGraph, pitchStretch, adapter.duration, adapter.box.duration.getValue());
+                const { ppqn, seconds } = sampleExtent(adapter);
+                AudioContentHelpers.addDefaultWarpMarkers(boxGraph, pitchStretch, ppqn, seconds);
             }
             switchTimeBaseToMusical(adapter);
         });
@@ -101,7 +108,8 @@ export var AudioContentModifier;
                 }
             }
             else {
-                AudioContentHelpers.addDefaultWarpMarkers(boxGraph, timeStretch, adapter.duration, adapter.box.duration.getValue());
+                const { ppqn, seconds } = sampleExtent(adapter);
+                AudioContentHelpers.addDefaultWarpMarkers(boxGraph, timeStretch, ppqn, seconds);
             }
             if (isDefined(transients) && adapter.file.transients.length() === 0) {
                 const markersField = adapter.file.box.transientMarkers;
@@ -113,6 +121,38 @@ export var AudioContentModifier;
             switchTimeBaseToMusical(adapter);
         });
     };
+    const warpPositionToSeconds = (warpMarkers, position) => {
+        const length = warpMarkers.length();
+        if (length === 0) {
+            return 0;
+        }
+        const first = warpMarkers.first();
+        const last = warpMarkers.last();
+        if (!isNotNull(first) || !isNotNull(last)) {
+            return 0;
+        }
+        if (position <= first.position) {
+            return first.seconds;
+        }
+        if (position >= last.position) {
+            return last.seconds;
+        }
+        for (let i = 0; i < length - 1; i++) {
+            const left = warpMarkers.optAt(i);
+            const right = warpMarkers.optAt(i + 1);
+            if (isNotNull(left) && isNotNull(right) && position >= left.position && position < right.position) {
+                const alpha = (position - left.position) / (right.position - left.position);
+                return left.seconds + alpha * (right.seconds - left.seconds);
+            }
+        }
+        return last.seconds;
+    };
+    const sampleExtent = (adapter) => {
+        if (isInstanceOf(adapter, AudioRegionBoxAdapter)) {
+            return { ppqn: adapter.loopDuration, seconds: adapter.box.loopDuration.getValue() };
+        }
+        return { ppqn: adapter.duration, seconds: adapter.box.duration.getValue() };
+    };
     const switchTimeBaseToSeconds = ({ box, timeBase }, audibleDuration) => {
         if (timeBase === TimeBase.Seconds) {
             return;

package/dist/ui/clipboard/types/DevicesClipboardHandler.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"DevicesClipboardHandler.d.ts","sourceRoot":"","sources":["../../../../src/ui/clipboard/types/DevicesClipboardHandler.ts"],"names":[],"mappings":"AAAA,OAAO,EAGH,OAAO,EAKP,MAAM,EAEN,QAAQ,EAGX,MAAM,kBAAkB,CAAA;AACzB,OAAO,EAAM,QAAQ,EAAC,MAAM,kBAAkB,CAAA;AAG9C,OAAO,EAEH,WAAW,EACX,gBAAgB,EAEhB,UAAU,EAGV,iBAAiB,EAGpB,MAAM,0BAA0B,CAAA;AACjC,OAAO,EAAC,cAAc,EAAE,gBAAgB,EAAC,MAAM,qBAAqB,CAAA;AAGpE,KAAK,gBAAgB,GAAG,cAAc,CAAC,SAAS,CAAC,CAAA;AAajD,yBAAiB,gBAAgB,CAAC;IAC9B,KAAY,OAAO,GAAG;QAClB,QAAQ,CAAC,UAAU,EAAE,QAAQ,CAAC,OAAO,CAAC,CAAA;QACtC,QAAQ,CAAC,OAAO,EAAE,OAAO,CAAA;QACzB,QAAQ,CAAC,SAAS,EAAE,iBAAiB,CAAC,gBAAgB,CAAC,CAAA;QACvD,QAAQ,CAAC,QAAQ,EAAE,QAAQ,CAAA;QAC3B,QAAQ,CAAC,WAAW,EAAE,WAAW,CAAA;QACjC,QAAQ,CAAC,OAAO,EAAE,QAAQ,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAA;KACjD,CAAA;IAyBM,MAAM,aAAa,GAAI,oEAOG,OAAO,KAAG,gBAAgB,CAAC,gBAAgB,CAwN3E,CAAA;CACJ"}
+{"version":3,"file":"DevicesClipboardHandler.d.ts","sourceRoot":"","sources":["../../../../src/ui/clipboard/types/DevicesClipboardHandler.ts"],"names":[],"mappings":"AAAA,OAAO,EAGH,OAAO,EAKP,MAAM,EAEN,QAAQ,EAGX,MAAM,kBAAkB,CAAA;AACzB,OAAO,EAAM,QAAQ,EAAC,MAAM,kBAAkB,CAAA;AAG9C,OAAO,EAEH,WAAW,EACX,gBAAgB,EAEhB,UAAU,EAGV,iBAAiB,EAGpB,MAAM,0BAA0B,CAAA;AACjC,OAAO,EAAC,cAAc,EAAE,gBAAgB,EAAC,MAAM,qBAAqB,CAAA;AAGpE,KAAK,gBAAgB,GAAG,cAAc,CAAC,SAAS,CAAC,CAAA;AAajD,yBAAiB,gBAAgB,CAAC;IAC9B,KAAY,OAAO,GAAG;QAClB,QAAQ,CAAC,UAAU,EAAE,QAAQ,CAAC,OAAO,CAAC,CAAA;QACtC,QAAQ,CAAC,OAAO,EAAE,OAAO,CAAA;QACzB,QAAQ,CAAC,SAAS,EAAE,iBAAiB,CAAC,gBAAgB,CAAC,CAAA;QACvD,QAAQ,CAAC,QAAQ,EAAE,QAAQ,CAAA;QAC3B,QAAQ,CAAC,WAAW,EAAE,WAAW,CAAA;QACjC,QAAQ,CAAC,OAAO,EAAE,QAAQ,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,CAAA;KACjD,CAAA;IAyBM,MAAM,aAAa,GAAI,oEAOG,OAAO,KAAG,gBAAgB,CAAC,gBAAgB,CA4N3E,CAAA;CACJ"}

package/dist/ui/clipboard/types/DevicesClipboardHandler.js

@@ -71,9 +71,10 @@ export var DevicesClipboard;
                     .map(pointer => pointer.box);
                 const mandatoryDeps = Array.from(boxGraph.dependenciesOf(box, {
                     alwaysFollowMandatory: true,
+                    stopAtResources: true,
                     excludeBox: (dep) => dep.ephemeral || DeviceBoxUtils.isDeviceBox(dep)
                         || dep.name === RootBox.ClassName
-                }).boxes).filter(dep => !isDefined(dep.resource));
+                }).boxes).filter(dep => dep.resource !== "preserved");
                 const preserved = [box, ...ownedChildren].flatMap(root => Array.from(boxGraph.dependenciesOf(root, {
                     alwaysFollowMandatory: true,
                     excludeBox: (dep) => dep.ephemeral || DeviceBoxUtils.isDeviceBox(dep)
@@ -94,6 +95,7 @@ export var DevicesClipboard;
                             trackContent.push(regionPointer.box);
                             const regionDeps = Array.from(boxGraph.dependenciesOf(regionPointer.box, {
                                 alwaysFollowMandatory: true,
+                                stopAtResources: true,
                                 excludeBox: (dep) => dep.ephemeral
                                     || isInstanceOf(dep, TrackBox)
                                     || DeviceBoxUtils.isDeviceBox(dep)
@@ -212,7 +214,7 @@ export var DevicesClipboard;
                             if (pointer.pointerType === Pointers.AudioEffectHost) {
                                 return Option.wrap(host.audioEffectsField.address);
                             }
-                            if (pointer.pointerType === Pointers.TrackCollection && replaceInstrument) {
+                            if (pointer.pointerType === Pointers.TrackCollection) {
                                 return Option.wrap(host.audioUnitBoxAdapter().tracksField.address);
                             }
                             if (pointer.pointerType === Pointers.Automation && replaceInstrument) {
@@ -227,8 +229,18 @@ export var DevicesClipboard;
                             }
                             return Option.None;
                         },
-                        excludeBox: box => !replaceInstrument
-                            && (DeviceBoxUtils.isInstrumentDeviceBox(box) || isInstanceOf(box, TrackBox))
+                        excludeBox: box => {
+                            if (replaceInstrument) {
+                                return false;
+                            }
+                            if (DeviceBoxUtils.isInstrumentDeviceBox(box)) {
+                                return true;
+                            }
+                            if (isInstanceOf(box, TrackBox)) {
+                                return metadata.hasInstrument;
+                            }
+                            return false;
+                        }
                     });
                     const deviceBoxes = boxes.filter(box => DeviceBoxUtils.isDeviceBox(box));
                     const newMidiEffects = deviceBoxes
@@ -239,14 +251,12 @@ export var DevicesClipboard;
                         .sort((a, b) => a.index.getValue() - b.index.getValue());
                     newMidiEffects.forEach((box, idx) => box.index.setValue(midiInsertIndex + idx));
                     newAudioEffects.forEach((box, idx) => box.index.setValue(audioInsertIndex + idx));
-                    if (replaceInstrument) {
-                        const tracksField = host.audioUnitBoxAdapter().tracksField;
-                        const allTracks = tracksField.pointerHub.filter(Pointers.TrackCollection)
-                            .filter(pointer => isInstanceOf(pointer.box, TrackBox))
-                            .map(pointer => pointer.box)
-                            .sort((trackA, trackB) => trackA.index.getValue() - trackB.index.getValue());
-                        allTracks.forEach((track, idx) => track.index.setValue(idx));
-                    }
+                    const tracksField = host.audioUnitBoxAdapter().tracksField;
+                    const allTracks = tracksField.pointerHub.filter(Pointers.TrackCollection)
+                        .filter(pointer => isInstanceOf(pointer.box, TrackBox))
+                        .map(pointer => pointer.box)
+                        .sort((trackA, trackB) => trackA.index.getValue() - trackB.index.getValue());
+                    allTracks.forEach((track, idx) => track.index.setValue(idx));
                     selection.select(...deviceBoxes.map(box => boxAdapters.adapterFor(box, Devices.isAny)));
                 });
             }

package/dist/ui/clipboard/types/DevicesClipboardHandler.test.js

@@ -195,9 +195,10 @@ describe("DevicesClipboardHandler", () => {
             .map(pointer => pointer.box);
         const mandatoryDeps = Array.from(boxGraph.dependenciesOf(deviceBox, {
             alwaysFollowMandatory: true,
+            stopAtResources: true,
             excludeBox: (dep) => dep.ephemeral || DeviceBoxUtils.isDeviceBox(dep)
                 || dep.name === RootBox.ClassName
-        }).boxes).filter(dep => !isDefined(dep.resource));
+        }).boxes).filter(dep => dep.resource !== "preserved");
         const preserved = [deviceBox, ...ownedChildren].flatMap(root => Array.from(boxGraph.dependenciesOf(root, {
             alwaysFollowMandatory: true,
             excludeBox: (dep) => dep.ephemeral || DeviceBoxUtils.isDeviceBox(dep)
@@ -215,6 +216,7 @@ describe("DevicesClipboardHandler", () => {
                     trackContent.push(regionPointer.box);
                     const regionDeps = Array.from(boxGraph.dependenciesOf(regionPointer.box, {
                         alwaysFollowMandatory: true,
+                        stopAtResources: true,
                         excludeBox: (dep) => dep.ephemeral
                             || isInstanceOf(dep, TrackBox)
                             || DeviceBoxUtils.isDeviceBox(dep)
@@ -232,7 +234,7 @@ describe("DevicesClipboardHandler", () => {
             return true;
         });
     };
-    const makePasteMapper = (targetAudioUnit, replaceInstrument) => ({
+    const makePasteMapper = (targetAudioUnit, replaceInstrument, hasInstrument = true) => ({
         mapPointer: (pointer) => {
             if (pointer.pointerType === Pointers.InstrumentHost && replaceInstrument) {
                 return Option.wrap(targetAudioUnit.input.address);
@@ -243,7 +245,7 @@ describe("DevicesClipboardHandler", () => {
             if (pointer.pointerType === Pointers.MIDIEffectHost) {
                 return Option.wrap(targetAudioUnit.midiEffects.address);
             }
-            if (pointer.pointerType === Pointers.TrackCollection && replaceInstrument) {
+            if (pointer.pointerType === Pointers.TrackCollection) {
                 return Option.wrap(targetAudioUnit.tracks.address);
             }
             if (pointer.pointerType === Pointers.Automation && replaceInstrument) {
@@ -251,7 +253,18 @@ describe("DevicesClipboardHandler", () => {
             }
             return Option.None;
         },
-        excludeBox: (box) => !replaceInstrument && (DeviceBoxUtils.isInstrumentDeviceBox(box) || isInstanceOf(box, TrackBox))
+        excludeBox: (box) => {
+            if (replaceInstrument) {
+                return false;
+            }
+            if (DeviceBoxUtils.isInstrumentDeviceBox(box)) {
+                return true;
+            }
+            if (isInstanceOf(box, TrackBox)) {
+                return hasInstrument;
+            }
+            return false;
+        }
     });
     // ─────────────────────────────────────────────────────────
     // Audio effect paste
@@ -284,6 +297,113 @@ describe("DevicesClipboardHandler", () => {
         });
     });
     // ─────────────────────────────────────────────────────────
+    // Audio effect with automation: copy scope + paste round-trip
+    // ─────────────────────────────────────────────────────────
+    describe("audio effect with automation", () => {
+        it("includes ValueEventCollectionBox when copying effect with automation events", () => {
+            const audioUnit = createAudioUnit(source);
+            const effect = addAudioEffect(source, audioUnit, "Comp", 0);
+            const autoTrack = addAutomationTrack(source, audioUnit, effect.threshold, 0);
+            addValueRegion(source, autoTrack, 0, 960);
+            const deps = collectDeviceDependencies(effect, source.boxGraph);
+            expect(deps.filter(box => isInstanceOf(box, TrackBox)).length).toBe(1);
+            expect(deps.filter(box => isInstanceOf(box, ValueRegionBox)).length).toBe(1);
+            expect(deps.filter(box => isInstanceOf(box, ValueEventCollectionBox)).length).toBe(1);
+        });
+        it("includes mirror regions on different tracks of the same device", () => {
+            const audioUnit = createAudioUnit(source);
+            const effect = addAudioEffect(source, audioUnit, "Comp", 0);
+            const trackA = addAutomationTrack(source, audioUnit, effect.threshold, 0);
+            const trackB = addAutomationTrack(source, audioUnit, effect.ratio, 1);
+            const regionA = addValueRegion(source, trackA, 0, 960);
+            const sharedEvents = regionA.events.targetVertex.unwrap().box;
+            let regionB;
+            source.boxGraph.beginTransaction();
+            regionB = ValueRegionBox.create(source.boxGraph, UUID.generate(), box => {
+                box.regions.refer(trackB.regions);
+                box.events.refer(sharedEvents.owners);
+                box.position.setValue(0);
+                box.duration.setValue(960);
+            });
+            source.boxGraph.endTransaction();
+            const deps = collectDeviceDependencies(effect, source.boxGraph);
+            const regions = deps.filter(box => isInstanceOf(box, ValueRegionBox));
+            expect(regions.length).toBe(2);
+            expect(regions).toContain(regionA);
+            expect(regions).toContain(regionB);
+            expect(deps.filter(box => isInstanceOf(box, ValueEventCollectionBox)).length).toBe(1);
+            expect(deps.filter(box => isInstanceOf(box, TrackBox)).length).toBe(2);
+        });
+        it("does not pull in regions from unrelated devices that mirror the same collection", () => {
+            const audioUnitA = createAudioUnit(source, 1);
+            const effectA = addAudioEffect(source, audioUnitA, "CompA", 0);
+            const trackA = addAutomationTrack(source, audioUnitA, effectA.threshold, 0);
+            const regionA = addValueRegion(source, trackA, 0, 960);
+            const sharedEvents = regionA.events.targetVertex.unwrap().box;
+            const audioUnitB = createAudioUnit(source, 2);
+            const effectB = addAudioEffect(source, audioUnitB, "CompB", 0);
+            const trackB = addAutomationTrack(source, audioUnitB, effectB.threshold, 0);
+            source.boxGraph.beginTransaction();
+            const unrelatedRegion = ValueRegionBox.create(source.boxGraph, UUID.generate(), box => {
+                box.regions.refer(trackB.regions);
+                box.events.refer(sharedEvents.owners);
+                box.position.setValue(0);
+                box.duration.setValue(960);
+            });
+            source.boxGraph.endTransaction();
+            const deps = collectDeviceDependencies(effectA, source.boxGraph);
+            const regions = deps.filter(box => isInstanceOf(box, ValueRegionBox));
+            expect(regions).toContain(regionA);
+            expect(regions).not.toContain(unrelatedRegion);
+            const tracks = deps.filter(box => isInstanceOf(box, TrackBox));
+            expect(tracks).toContain(trackA);
+            expect(tracks).not.toContain(trackB);
+        });
+        it("round-trip paste of effect with automation events does not throw", () => {
+            const sourceAU = createAudioUnit(source);
+            const effect = addAudioEffect(source, sourceAU, "Comp", 0);
+            const autoTrack = addAutomationTrack(source, sourceAU, effect.threshold, 0);
+            addValueRegion(source, autoTrack, 0, 960);
+            const deps = collectDeviceDependencies(effect, source.boxGraph);
+            const data = ClipboardUtils.serializeBoxes([effect, ...deps]);
+            const targetAU = createAudioUnit(target);
+            const editing = new BoxEditing(target.boxGraph);
+            expect(() => {
+                editing.modify(() => {
+                    ClipboardUtils.deserializeBoxes(data, target.boxGraph, makePasteMapper(targetAU, false, false));
+                });
+            }).not.toThrow();
+            const pastedEffects = targetAU.audioEffects.pointerHub.incoming()
+                .filter(pointer => isInstanceOf(pointer.box, CompressorDeviceBox));
+            expect(pastedEffects.length).toBe(1);
+            const pastedTracks = targetAU.tracks.pointerHub.filter(Pointers.TrackCollection)
+                .filter(pointer => isInstanceOf(pointer.box, TrackBox));
+            expect(pastedTracks.length).toBe(1);
+        });
+        it("pasted automation track targets the pasted device's parameter", () => {
+            const sourceAU = createAudioUnit(source);
+            const effect = addAudioEffect(source, sourceAU, "Comp", 0);
+            const autoTrack = addAutomationTrack(source, sourceAU, effect.threshold, 0);
+            addValueRegion(source, autoTrack, 0, 960);
+            const deps = collectDeviceDependencies(effect, source.boxGraph);
+            const data = ClipboardUtils.serializeBoxes([effect, ...deps]);
+            const targetAU = createAudioUnit(target);
+            const editing = new BoxEditing(target.boxGraph);
+            editing.modify(() => {
+                ClipboardUtils.deserializeBoxes(data, target.boxGraph, makePasteMapper(targetAU, false, false));
+            });
+            const pastedEffect = targetAU.audioEffects.pointerHub.incoming()
+                .filter(pointer => isInstanceOf(pointer.box, CompressorDeviceBox))
+                .map(pointer => pointer.box)[0];
+            const pastedTrack = targetAU.tracks.pointerHub.filter(Pointers.TrackCollection)
+                .filter(pointer => isInstanceOf(pointer.box, TrackBox))
+                .map(pointer => pointer.box)[0];
+            expect(pastedTrack).toBeDefined();
+            const targetVertex = pastedTrack.target.targetVertex.unwrap();
+            expect(targetVertex.box).toBe(pastedEffect);
+        });
+    });
+    // ─────────────────────────────────────────────────────────
     // Instrument paste
     // ─────────────────────────────────────────────────────────
     describe("paste instrument", () => {

package/dist/ysync/YSync.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"YSync.d.ts","sourceRoot":"","sources":["../../src/ysync/YSync.ts"],"names":[],"mappings":"AAAA,OAAO,EASH,QAAQ,EAER,UAAU,EAIb,MAAM,kBAAkB,CAAA;AACzB,OAAO,EAAa,QAAQ,EAA2D,MAAM,kBAAkB,CAAA;AAE/G,OAAO,KAAK,CAAC,MAAM,KAAK,CAAA;AAIxB,MAAM,MAAM,SAAS,CAAC,CAAC,IAAI;IACvB,QAAQ,EAAE,QAAQ,CAAC,CAAC,CAAC,CAAC;IACtB,KAAK,EAAE,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAA;IACrB,QAAQ,CAAC,EAAE,QAAQ,CAAC,OAAO,CAAC,CAAA;CAC/B,CAAA;AAED,qBAAa,KAAK,CAAC,CAAC,CAAE,YAAW,UAAU;;IACvC,MAAM,CAAC,SAAS,EAAE,OAAO,CAAQ;IAEjC,gBAAgB;IAChB,MAAM,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC,GAAG,GAAG,OAAO;WAItB,YAAY,CAAC,CAAC,EAAE,EAAC,QAAQ,EAAE,KAAK,EAAC,EAAE,SAAS,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;WAcnE,QAAQ,CAAC,CAAC,EAAE,EAAC,QAAQ,EAAE,KAAK,EAAC,EAAE,SAAS,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;gBAqChE,EAAC,QAAQ,EAAE,KAAK,EAAE,QAAQ,EAAC,EAAE,SAAS,CAAC,CAAC,CAAC;IASrD,SAAS,IAAI,IAAI;CA0LpB"}
+{"version":3,"file":"YSync.d.ts","sourceRoot":"","sources":["../../src/ysync/YSync.ts"],"names":[],"mappings":"AAAA,OAAO,EAUH,QAAQ,EAER,UAAU,EAIb,MAAM,kBAAkB,CAAA;AACzB,OAAO,EAEH,QAAQ,EAOX,MAAM,kBAAkB,CAAA;AAEzB,OAAO,KAAK,CAAC,MAAM,KAAK,CAAA;AAIxB,MAAM,MAAM,SAAS,CAAC,CAAC,IAAI;IACvB,QAAQ,EAAE,QAAQ,CAAC,CAAC,CAAC,CAAC;IACtB,KAAK,EAAE,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAA;IACrB,QAAQ,CAAC,EAAE,QAAQ,CAAC,OAAO,CAAC,CAAA;CAC/B,CAAA;AAED,qBAAa,KAAK,CAAC,CAAC,CAAE,YAAW,UAAU;;IACvC,MAAM,CAAC,SAAS,EAAE,OAAO,CAAQ;IAEjC,gBAAgB;IAChB,MAAM,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC,GAAG,GAAG,OAAO;WAItB,YAAY,CAAC,CAAC,EAAE,EAAC,QAAQ,EAAE,KAAK,EAAC,EAAE,SAAS,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;WAcnE,QAAQ,CAAC,CAAC,EAAE,EAAC,QAAQ,EAAE,KAAK,EAAC,EAAE,SAAS,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;gBAqChE,EAAC,QAAQ,EAAE,KAAK,EAAE,QAAQ,EAAC,EAAE,SAAS,CAAC,CAAC,CAAC;IASrD,SAAS,IAAI,IAAI;CAiOpB"}

package/dist/ysync/YSync.js

@@ -1,4 +1,5 @@
-import { asDefined, asInstanceOf, assert, EmptyExec, isUndefined, Option, panic, Terminable, Terminator, tryCatch, UUID } from "@opendaw/lib-std";
+import { asDefined, asInstanceOf, assert, EmptyExec, isInstanceOf, isUndefined, Option, panic, Terminable, Terminator, tryCatch, UUID } from "@opendaw/lib-std";
+import { optimizeUpdates } from "@opendaw/lib-box";
 import { YMapper } from "./YMapper";
 import * as Y from "yjs";
 export class YSync {
@@ -200,50 +201,95 @@ export class YSync {
         return Terminable.many(this.#boxGraph.subscribeTransaction({
             onBeginTransaction: EmptyExec,
             onEndTransaction: (rolledBack) => {
+                const pending = this.#updates.splice(0);
                 if (this.#ignoreUpdates || rolledBack) {
-                    this.#updates.length = 0;
                     return;
                 }
-                this.#getDoc().transact(() => this.#updates.forEach(update => {
-                    /**
-                     * TRANSFER CHANGES FROM OPENDAW TO YJS
-                     */
-                    if (update.type === "new") {
-                        const uuid = update.uuid;
-                        const key = UUID.toString(uuid);
-                        const box = this.#boxGraph.findBox(uuid).unwrap();
-                        this.#boxes.set(key, YMapper.createBoxMap(box));
-                    }
-                    else if (update.type === "primitive") {
-                        const key = UUID.toString(update.address.uuid);
-                        const boxObject = asDefined(this.#boxes.get(key), "Could not find box");
-                        const { address: { fieldKeys }, newValue } = update;
-                        let field = boxObject.get("fields");
-                        for (let i = 0; i < fieldKeys.length - 1; i++) {
-                            field = asDefined(field.get(String(fieldKeys[i])), `No field at '${fieldKeys[i]}'`);
-                        }
-                        field.set(String(fieldKeys[fieldKeys.length - 1]), newValue);
-                    }
-                    else if (update.type === "pointer") {
-                        const key = UUID.toString(update.address.uuid);
-                        const boxObject = asDefined(this.#boxes.get(key), "Could not find box");
-                        const { address: { fieldKeys }, newAddress } = update;
-                        let field = boxObject.get("fields");
-                        for (let i = 0; i < fieldKeys.length - 1; i++) {
-                            field = asDefined(field.get(String(fieldKeys[i])), `No field at '${fieldKeys[i]}'`);
-                        }
-                        field.set(String(fieldKeys[fieldKeys.length - 1]), newAddress.mapOr(address => address.toString(), null));
-                    }
-                    else if (update.type === "delete") {
-                        this.#boxes.delete(UUID.toString(update.uuid));
-                    }
-                }), "[openDAW] updates");
-                this.#updates.length = 0;
+                const optimized = optimizeUpdates(pending);
+                if (optimized.length === 0) {
+                    return;
+                }
+                const result = tryCatch(() => this.#getDoc()
+                    .transact(() => optimized.forEach(update => this.#applyUpdate(update)), "[openDAW] updates"));
+                if (result.status === "failure") {
+                    console.error("[YSync] flush failed, dropping updates", {
+                        count: optimized.length,
+                        error: result.error
+                    });
+                    throw result.error;
+                }
             }
         }), this.#boxGraph.subscribeToAllUpdatesImmediate({
             onUpdate: (update) => this.#updates.push(update)
         }));
     }
+    /**
+     * TRANSFER ONE CHANGE FROM OPENDAW TO YJS
+     */
+    #applyUpdate(update) {
+        if (update.type === "new") {
+            const uuid = update.uuid;
+            const key = UUID.toString(uuid);
+            const optBox = this.#boxGraph.findBox(uuid);
+            if (optBox.isEmpty()) {
+                // Phantom box: created and removed in same transaction.
+                // optimizeUpdates should have filtered this, but guard in case.
+                return;
+            }
+            this.#boxes.set(key, YMapper.createBoxMap(optBox.unwrap()));
+        }
+        else if (update.type === "primitive") {
+            const key = UUID.toString(update.address.uuid);
+            const boxObject = this.#boxes.get(key);
+            if (!isInstanceOf(boxObject, Y.Map)) {
+                console.warn(`[YSync] primitive update skipped: box '${key}' missing`);
+                return;
+            }
+            const field = this.#resolveFieldMap(boxObject, key, update.address.fieldKeys);
+            if (field === undefined) {
+                return;
+            }
+            field.set(String(update.address.fieldKeys[update.address.fieldKeys.length - 1]), update.newValue);
+        }
+        else if (update.type === "pointer") {
+            const key = UUID.toString(update.address.uuid);
+            const boxObject = this.#boxes.get(key);
+            if (!isInstanceOf(boxObject, Y.Map)) {
+                console.warn(`[YSync] pointer update skipped: box '${key}' missing`);
+                return;
+            }
+            const field = this.#resolveFieldMap(boxObject, key, update.address.fieldKeys);
+            if (field === undefined) {
+                return;
+            }
+            field.set(String(update.address.fieldKeys[update.address.fieldKeys.length - 1]), update.newAddress.mapOr(address => address.toString(), null));
+        }
+        else if (update.type === "delete") {
+            this.#boxes.delete(UUID.toString(update.uuid));
+        }
+    }
+    /**
+     * Walks from the box map down to the Y.Map that owns `fieldKeys[last]`.
+     * Returns `undefined` (with a warning) instead of throwing if the path
+     * cannot be resolved — protects the yjs transaction from partial writes.
+     */
+    #resolveFieldMap(boxObject, key, fieldKeys) {
+        const fieldsValue = boxObject.get("fields");
+        if (!isInstanceOf(fieldsValue, Y.Map)) {
+            console.warn(`[YSync] box '${key}' missing 'fields' Y.Map; skipping update`);
+            return undefined;
+        }
+        let field = fieldsValue;
+        for (let i = 0; i < fieldKeys.length - 1; i++) {
+            const next = field.get(String(fieldKeys[i]));
+            if (!isInstanceOf(next, Y.Map)) {
+                console.warn(`[YSync] box '${key}' field path broken at '${fieldKeys[i]}'; skipping update`);
+                return undefined;
+            }
+            field = next;
+        }
+        return field;
+    }
     #getDoc() {
         return asDefined(this.#boxes.doc, "Y.Map is not connect to Y.Doc");
     }

package/dist/ysync/YSync.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=YSync.test.d.ts.map

package/dist/ysync/YSync.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"YSync.test.d.ts","sourceRoot":"","sources":["../../src/ysync/YSync.test.ts"],"names":[],"mappings":""}