fix: cover native scrub and automation fallback

Author: ChuxiJ

src/hooks/__tests__/useTransport.test.ts

@@ -130,7 +130,7 @@ vi.mock('../useToast', () => ({
   toastError: vi.fn(),
 }));
 
-import { useTransport } from '../useTransport';
+import { canUseNativeClipPlayback, useTransport } from '../useTransport';
 import { useProjectStore } from '../../store/projectStore';
 import { useTransportStore } from '../../store/transportStore';
 import { useUIStore } from '../../store/uiStore';
@@ -198,6 +198,23 @@ describe('useTransport', () => {
     );
   });
 
+  it('disables native clip playback when automation lanes are active', () => {
+    const project = useProjectStore.getState().project!;
+    expect(canUseNativeClipPlayback(project, [])).toBe(true);
+
+    project.automationLanes = [{
+      id: 'automation-1',
+      trackId: 'track-1',
+      parameter: { type: 'mixer', param: 'volume' },
+      points: [
+        { time: 0, value: 0.25 },
+        { time: 1, value: 0.75 },
+      ],
+    }];
+
+    expect(canUseNativeClipPlayback(project, [])).toBe(false);
+  });
+
   // ── pause() ──
 
   it('stops all engines and strudel when pausing', async () => {

src/hooks/useTransport.ts

@@ -115,9 +115,10 @@ function trackNeedsWebAudio(track: Track): boolean {
     || (track.sends?.some((send) => send.amount > 0.000001) ?? false);
 }
 
-function canUseNativeClipPlayback(project: Project, entries: NativePlaybackEntry[]): boolean {
+export function canUseNativeClipPlayback(project: Project, entries: NativePlaybackEntry[]): boolean {
   if (project.mastering?.enabled) return false;
   if ((project.returnTracks?.length ?? 0) > 0) return false;
+  if (project.automationLanes?.some((lane) => lane.points.length > 0)) return false;
   if (entries.some(clipNeedsWebAudio)) return false;
   return !project.tracks.some(trackNeedsWebAudio);
 }
@@ -922,6 +923,7 @@ export function useTransport() {
 
   const startScrub = useCallback(async (time: number) => {
     const engine = getAudioEngine();
+    const bridge = getAudioBridge(engine);
     const transport = useTransportStore.getState();
     const scrubProject = useProjectStore.getState().project;
     if (!scrubProject) return;
@@ -932,6 +934,8 @@ export function useTransport() {
     const resumePlayback = transport.isPlaying || engine.playing;
     if (resumePlayback) {
       engine.stop();
+      setTauriPlaybackClockOwner('web-audio');
+      bridge.stopAllSources();
       synthEngine.releaseAll();
       subtractiveEngine.releaseAll();
       wavetableEngine.releaseAll();

tests/unit/useTransportScrubLifecycle.test.tsx

@@ -16,6 +16,7 @@ const engineMock = {
   startTimelineScrub: vi.fn().mockResolvedValue(undefined),
   updateTimelineScrub: vi.fn().mockResolvedValue(undefined),
   stopTimelineScrub: vi.fn(),
+  stopAllSources: vi.fn(),
   stop: vi.fn(),
   getCurrentTime: vi.fn(() => 6.5),
   setOnEndedCallback: vi.fn(),
@@ -133,6 +134,7 @@ describe('useTransport scrub lifecycle', () => {
     });
 
     expect(engineMock.stop).toHaveBeenCalledTimes(1);
+    expect(engineMock.stopAllSources).toHaveBeenCalledTimes(1);
     expect(useTransportStore.getState().isPlaying).toBe(false);
     expect(useTransportStore.getState().isScrubbing).toBe(true);
     expect(engineMock.startTimelineScrub).toHaveBeenCalledWith(expect.any(Array), expect.any(Array), 4, 0);