Fix erros

Author: hapticforge

.github/workflows/ci.yml

@@ -21,7 +21,7 @@ jobs:
           enable-cache: true
 
       - name: Install dependencies
-        run: uv sync --dev
+        run: uv sync --all-extras
 
       - name: Lint (ruff check)
         run: uv run ruff check src/ tests/
@@ -47,7 +47,7 @@ jobs:
           enable-cache: true
 
       - name: Install dependencies
-        run: uv sync --dev
+        run: uv sync --all-extras
 
       - name: Run tests with coverage
         run: uv run --with pytest --with pytest-cov pytest tests/ --cov=wavedriver --cov-report=term-missing --cov-fail-under=60

src/wavedriver/web/src/App.jsx

@@ -51,8 +51,8 @@ function AppInner() {
     shuffleDwellS,  setShuffleDwellS,
     shuffleMinFreq, setShuffleMinFreq,
     shuffleMaxFreq, setShuffleMaxFreq,
-    shuffleMinStroke, setShuffleMinStroke,
-    shuffleMaxStroke, setShuffleMaxStroke,
+    shuffleMinStroke,
+    shuffleMaxStroke,
   } = useSettings({ apiReady, sendCommand, setHistoryEnabled });
 
   const settingsRef = useRef({});

src/wavedriver/web/src/ErrorBoundary.jsx

@@ -23,7 +23,9 @@ export class ErrorBoundary extends React.Component {
   handleEstop = () => {
     try {
       window.pywebview?.api?.send_command("estop", { reason: "UI crashed — Emergency Stop" });
-    } catch (_) {}
+    } catch {
+      // Ignore error
+    }
   };
 
   handleRetry = () => {

src/wavedriver/web/src/components/AdvancedDrawer.jsx

@@ -67,7 +67,7 @@ export function AdvancedDrawer({
   shuffleMaxFreq, setShuffleMaxFreq,
   // History
   historyEnabled, onHistoryEnabledChange,
-  sessionHistory, onClearHistory,
+  onClearHistory,
   onToggleHistory, showHistory,
   // Activity log
   showActivityLog, onToggleActivityLog,

src/wavedriver/web/src/components/AudioSyncPanel.jsx

@@ -1,11 +1,10 @@
-import { useState, useEffect, useRef } from 'react';
+import { useState, useEffect, useRef, useCallback } from 'react';
 import { Music, Mic } from 'lucide-react';
 
 export function AudioSyncPanel({
   isRunning,
   intensityPct,
   onIntensityChange,
-  frequencyHz,
   onFrequencyChange,
 }) {
   const [isActive, setIsActive] = useState(false);
@@ -25,54 +24,64 @@ export function AudioSyncPanel({
   const lastBeatTime = useRef(0);
   const beatTimestamps = useRef([]);
 
-  const startAudioSync = async () => {
-    try {
-      // Clean up any existing instances first
-      stopAudioSync();
-
-      const stream = await navigator.mediaDevices.getUserMedia({ audio: true });
-      streamRef.current = stream;
-      audioCtxRef.current = new (window.AudioContext || window.webkitAudioContext)();
-      analyserRef.current = audioCtxRef.current.createAnalyser();
-      analyserRef.current.fftSize = 256;
-      
-      sourceRefNode.current = audioCtxRef.current.createMediaStreamSource(stream);
-      sourceRefNode.current.connect(analyserRef.current);
-      
-      setIsActive(true);
-    } catch (err) {
-      alert("Failed to access microphone: " + err.message);
-    }
-  };
-
-  const stopAudioSync = () => {
+  const stopAudioSync = useCallback(() => {
     if (rafRef.current) {
       cancelAnimationFrame(rafRef.current);
       rafRef.current = null;
     }
     if (sourceRefNode.current) {
       try {
         sourceRefNode.current.disconnect();
-      } catch {}
+      } catch {
+        // Silently ignore
+      }
       sourceRefNode.current = null;
     }
     if (streamRef.current) {
       try {
         streamRef.current.getTracks().forEach(track => track.stop());
-      } catch {}
+      } catch {
+        // Silently ignore
+      }
       streamRef.current = null;
     }
     if (audioCtxRef.current && audioCtxRef.current.state !== 'closed') {
       try {
         audioCtxRef.current.close();
-      } catch {}
+      } catch {
+        // Silently ignore
+      }
       audioCtxRef.current = null;
     }
-    setIsActive(false);
-    setVolumeLevel(0);
-    setDetectedBpm(null);
+    setTimeout(() => {
+      setIsActive(false);
+      setVolumeLevel(0);
+      setDetectedBpm(null);
+    }, 0);
     beatTimestamps.current = [];
-  };
+  }, []);
+
+  const startAudioSync = useCallback(async () => {
+    try {
+      // Clean up any existing instances first
+      stopAudioSync();
+
+      const stream = await navigator.mediaDevices.getUserMedia({ audio: true });
+      streamRef.current = stream;
+      audioCtxRef.current = new (window.AudioContext || window.webkitAudioContext)();
+      analyserRef.current = audioCtxRef.current.createAnalyser();
+      analyserRef.current.fftSize = 256;
+      
+      sourceRefNode.current = audioCtxRef.current.createMediaStreamSource(stream);
+      sourceRefNode.current.connect(analyserRef.current);
+      
+      setTimeout(() => {
+        setIsActive(true);
+      }, 0);
+    } catch (err) {
+      alert("Failed to access microphone: " + err.message);
+    }
+  }, [stopAudioSync]);
 
   // Sync mic activity with global running state
   useEffect(() => {
@@ -84,7 +93,7 @@ export function AudioSyncPanel({
     return () => {
       stopAudioSync();
     };
-  }, [isRunning]);
+  }, [isRunning, startAudioSync, stopAudioSync]);
 
   useEffect(() => {
     if (!isActive) return;

src/wavedriver/web/src/components/InteractiveEdging.jsx

@@ -28,27 +28,31 @@ export function InteractiveEdging({ isRunning, intensityPct, onIntensityChange }
         setCooldownRemaining(prev => prev - 1);
       }, 1000);
     } else if (cooldownRemaining === 0 && edgingActive) {
-      if (surpriseDenial && Math.random() < 0.5) {
-        setDenied(true);
-        onIntensityChange(0);
-        setCooldownRemaining(5);
-        setSurpriseDenial(false);
-        return;
-      }
+      setTimeout(() => {
+        if (surpriseDenial && Math.random() < 0.5) {
+          setDenied(true);
+          onIntensityChange(0);
+          setCooldownRemaining(5);
+          setSurpriseDenial(false);
+          return;
+        }
 
-      onIntensityChange(savedIntensity.current);
-      setEdgingActive(false);
-      setDenied(false);
+        onIntensityChange(savedIntensity.current);
+        setEdgingActive(false);
+        setDenied(false);
+      }, 0);
     }
 
     return () => clearTimeout(timerRef.current);
   }, [cooldownRemaining, edgingActive, surpriseDenial, onIntensityChange]);
 
   useEffect(() => {
     if (!isRunning) {
-      setEdgingActive(false);
-      setCooldownRemaining(0);
-      setDenied(false);
+      setTimeout(() => {
+        setEdgingActive(false);
+        setCooldownRemaining(0);
+        setDenied(false);
+      }, 0);
     }
   }, [isRunning]);
 

src/wavedriver/web/src/components/InteractiveEdging.test.jsx

@@ -1,4 +1,4 @@
-import { render, screen, fireEvent, act } from '@testing-library/react';
+import { render, screen, fireEvent } from '@testing-library/react';
 import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { InteractiveEdging } from './InteractiveEdging';
 

src/wavedriver/web/src/components/SequenceBuilder.jsx

@@ -1,4 +1,4 @@
-import { useState, useEffect, useRef } from 'react';
+import { useState, useEffect, useRef, useCallback } from 'react';
 import { Layers, Plus, Trash2, Play, Square, FastForward, Clock } from 'lucide-react';
 
 const PATTERNS = ["Wave", "Realistic", "Thrust", "Pulse", "Tease", "Escalate", "Edge", "Depth", "Adaptive"];
@@ -50,25 +50,7 @@ export function SequenceBuilder({
     executeStep(steps[0]);
   };
 
-  const stopSequence = () => {
-    setIsSequencePlaying(false);
-    setActiveStepIdx(null);
-    setStepTimeRemaining(0);
-    stopPattern();
-  };
-
-  const nextStep = () => {
-    const nextIdx = activeStepIdx + 1;
-    if (nextIdx < steps.length) {
-      setActiveStepIdx(nextIdx);
-      setStepTimeRemaining(steps[nextIdx].duration);
-      executeStep(steps[nextIdx]);
-    } else {
-      stopSequence();
-    }
-  };
-
-  const executeStep = (step) => {
+  const executeStep = useCallback((step) => {
     // Set UI states so sliders align
     setPatternName(step.pattern);
     setFrequencyHz(step.freq);
@@ -82,7 +64,25 @@ export function SequenceBuilder({
       strokeLengthMm: step.stroke,
       intensityPct: step.intensity,
     });
-  };
+  }, [setPatternName, setFrequencyHz, setStrokeLengthMm, onIntensityChange, startPattern]);
+
+  const stopSequence = useCallback(() => {
+    setIsSequencePlaying(false);
+    setActiveStepIdx(null);
+    setStepTimeRemaining(0);
+    stopPattern();
+  }, [stopPattern]);
+
+  const nextStep = useCallback(() => {
+    const nextIdx = activeStepIdx + 1;
+    if (nextIdx < steps.length) {
+      setActiveStepIdx(nextIdx);
+      setStepTimeRemaining(steps[nextIdx].duration);
+      executeStep(steps[nextIdx]);
+    } else {
+      stopSequence();
+    }
+  }, [activeStepIdx, steps, executeStep, stopSequence]);
 
   // Timer loop for sequence playback
   useEffect(() => {
@@ -92,14 +92,18 @@ export function SequenceBuilder({
           setStepTimeRemaining(prev => prev - 1);
         }, 1000);
       } else {
-        nextStep();
+        setTimeout(() => {
+          nextStep();
+        }, 0);
       }
     } else if (!isRunning && isSequencePlaying) {
-      stopSequence();
+      setTimeout(() => {
+        stopSequence();
+      }, 0);
     }
 
     return () => clearTimeout(timerRef.current);
-  }, [isSequencePlaying, activeStepIdx, stepTimeRemaining, isRunning]);
+  }, [isSequencePlaying, activeStepIdx, stepTimeRemaining, isRunning, nextStep, stopSequence]);
 
   return (
     <div className="control-card glass">

src/wavedriver/web/src/components/SliderPanel.jsx

@@ -85,10 +85,8 @@ export function SliderPanel({
   shuffleDwellS, setShuffleDwellS,
   shuffleMinFreq, setShuffleMinFreq,
   shuffleMaxFreq, setShuffleMaxFreq,
-  shuffleMinStroke, setShuffleMinStroke,
-  shuffleMaxStroke, setShuffleMaxStroke,
   safetyForceN,
-  maxSessionS, setMaxSessionS,
+  maxSessionS,
   calibratedLength,
   onIntensityChange,
   onSafetyChange,

src/wavedriver/web/src/components/WaveformCanvas.test.jsx

@@ -1,4 +1,4 @@
-import { render, act } from '@testing-library/react';
+import { render } from '@testing-library/react';
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import { WaveformCanvas } from './WaveformCanvas';