Merge pull request #161 from espadonne/main

fix activity logging for single-track effects and multitrack clip delete

Author: mfwolffe

components/audio/DAW/CustomWaveform/CustomTimeline.jsx

@@ -53,7 +53,10 @@ export default function CustomTimeline() {
     // Set the cutRegion so effects know what region to apply to
     setCutRegion(activeRegion);
     setShowEffectsModal(true);
-  }, [activeRegion, setShowEffectsModal, setCutRegion]);
+    if (logOperation) {
+      logOperation('effects_rack_toggled', {});
+    }
+  }, [activeRegion, setShowEffectsModal, setCutRegion, logOperation]);
 
   // Handle splice (excise) region - keep only the selected region
   const handleSplice = useCallback(async () => {

components/audio/DAW/Effects/EQ.js

@@ -739,7 +739,7 @@ export async function processEQRegion(audioBuffer, startSample, endSample, param
  * Professional Parametric EQ with visual frequency response
  * Features: 8-band parametric EQ, spectrum analyzer, linear phase option
  */
-export default function EQ({ width, modalMode = false, onApply }) {
+export default function EQ({ width, modalMode = false, onApply, logOperation = null }) {
   const {
     audioRef,
     addToEditHistory,
@@ -1151,6 +1151,9 @@ export default function EQ({ width, modalMode = false, onApply }) {
                 // Force new array and deep clone to trigger React re-render
                 setEqBands(preset.bands.map(band => ({ ...band })));
                 setEqGain(preset.outputGain);
+                if (logOperation && presetKey !== 'flat') {
+                  logOperation('eq_preset_applied', { presetName: preset.name });
+                }
               }
             }}
             className="bg-secondary text-white border-0"

components/audio/DAW/Effects/EffectControlModal.js

@@ -33,7 +33,7 @@ const effectComponents = {
   stereowide: lazy(() => import('./StereoWidener'))
 };
 
-export default function EffectControlModal() {
+export default function EffectControlModal({ logOperation = null }) {
   const {
     showEffectControlModal,
     setShowEffectControlModal,
@@ -53,6 +53,13 @@ export default function EffectControlModal() {
     setSelectedEffect(null);
   };
 
+  const handleApply = () => {
+    if (logOperation && selectedEffect) {
+      logOperation('effect_applied', { effectType: selectedEffect.id });
+    }
+    handleClose();
+  };
+
 
   if (!selectedEffect) {
     return null;
@@ -115,7 +122,7 @@ export default function EffectControlModal() {
             >
               {EffectComponent ? (
                 <div className="effect-controls-container">
-                  <EffectComponent width={100} modalMode={true} onApply={handleClose} />
+                  <EffectComponent width={100} modalMode={true} onApply={handleApply} logOperation={logOperation} />
                 </div>
               ) : (
                 <Alert variant="warning">

components/audio/DAW/Multitrack/MultitrackEditor.js

@@ -372,6 +372,9 @@ export default function MultitrackEditor({ availableTakes: propTakes = [], logOp
     }
 
     // Log deletion
+    if (logOperation) {
+      logOperation('clip_delete', { clipCount });
+    }
     try {
       const activityLogger = getDAWActivityLogger();
       if (activityLogger?.isActive) {
@@ -386,7 +389,7 @@ export default function MultitrackEditor({ availableTakes: propTakes = [], logOp
     } catch (error) {
       console.error('📊 Error logging delete operation:', error);
     }
-  }, [selectedTrack, selectedClipId, tracks, updateTrack, setSelectedClipId, setSelectedClipIds]);
+  }, [selectedTrack, selectedClipId, tracks, updateTrack, setSelectedClipId, setSelectedClipIds, logOperation]);
 
   // Handle split at playhead
   const handleSplitAtPlayhead = useCallback(() => {

components/audio/DAW/index.js

@@ -285,8 +285,8 @@ export default function DAW({
       </Card>
 
       {/* Effects Modals */}
-      <EffectsModal />
-      <EffectControlModal />
+      <EffectsModal logOperation={logOperation} />
+      <EffectControlModal logOperation={logOperation} />
 
       {/* Recording Modal for first-time recording */}
       <RecordingModal