unslop(web-hooks): strip AI-slop (comments, dead handlers, over-abstraction, useEffect-derived, zustand selectors, raw MUI)

Author: claude

web/src/hooks/assets/useAssetDisplay.tsx

@@ -7,9 +7,7 @@ import VideoViewer from "../../components/asset_viewer/VideoViewer";
 import LazyPDFViewer from "../../components/asset_viewer/LazyPDFViewer";
 import LazyModel3DViewer from "../../components/asset_viewer/LazyModel3DViewer";
 
-// Helper to detect 3D model content types
 const isModel3D = (type: string, url?: string): boolean => {
-  // Check MIME types
   if (
     type.startsWith("model/") ||
     type === "application/octet-stream" ||
@@ -18,7 +16,6 @@ const isModel3D = (type: string, url?: string): boolean => {
   ) {
     return true;
   }
-  // Check file extensions
   if (url) {
     try {
       // Extract pathname to handle URLs with query parameters or fragments

web/src/hooks/assets/useAssetSelection.ts

@@ -30,7 +30,6 @@ export const useAssetSelection = (sortedAssets: Asset[]) => {
     (state) => state.setCurrentAudioAsset
   );
 
-  // Helper function to update both selectedAssetIds and selectedAssets efficiently
   const updateSelection = useCallback(
     (assetIds: string[]) => {
       setSelectedAssetIds(assetIds);

web/src/hooks/browser/useClipboard.ts

@@ -21,7 +21,6 @@ export const useClipboard = (): UseClipboardResult => {
     []
   );
 
-  // Check if Electron API is available
   const hasElectronApi = useMemo(
     () => typeof window !== "undefined" && !!window.api,
     []
@@ -130,7 +129,6 @@ export const useClipboard = (): UseClipboardResult => {
     readClipboard,
     writeClipboard,
     isClipboardValid,
-    // Export utility function for asset-based clipboard operations
     copyAssetToClipboard
   };
 };

web/src/hooks/browser/useRealtimeAudioPlayback.ts

@@ -23,7 +23,7 @@ interface UseRealtimeAudioPlaybackOptions {
   chunksVersion: number;
   sampleRate?: number;
   channels?: number;
-  nodeId?: string; // Optional ID for this audio source
+  nodeId?: string;
   /**
    * Live-monitoring mode: keep the scheduled lead over the playhead within a
    * small bound by dropping stale chunks instead of queueing them. Use for
@@ -386,7 +386,6 @@ export const useRealtimeAudioPlayback = ({
     [channels, sampleRate, decodeChunkSamples]
   );
 
-  // Internal play/stop functions (called by queue)
   const internalStart = useCallback(() => {
     const ctx = audioContextRef.current;
     if (!ctx) {
@@ -568,7 +567,6 @@ export const useRealtimeAudioPlayback = ({
     });
   }, [nodeId, decodeChunkSamples, noteDelivery]);
 
-  // Public start: requests playback via queue
   const start = useCallback(() => {
     setWantsToPlay(true);
     audioQueue.enqueue({
@@ -578,7 +576,6 @@ export const useRealtimeAudioPlayback = ({
     });
   }, [audioQueue, internalStart, internalStop]);
 
-  // Public stop: removes from queue
   const stop = useCallback(() => {
     setWantsToPlay(false);
     internalStop();

web/src/hooks/browser/useVideoRecorder.ts

@@ -227,7 +227,6 @@ export function useVideoRecorder({ onChange }: VideoRecorderProps): Readonly<Vid
 
   const handleRecord = useCallback(() => {
     if (mediaRecorderRef.current && isRecording) {
-      // Stop recording
       mediaRecorderRef.current.stop();
       setIsRecording(false);
       return;
@@ -305,7 +304,6 @@ export function useVideoRecorder({ onChange }: VideoRecorderProps): Readonly<Vid
 
     return () => {
       if (mediaRecorderRef.current) {
-        // Check if it's currently recording before stopping
         if (mediaRecorderRef.current.state === "recording") {
           mediaRecorderRef.current.stop();
         }

web/src/hooks/editor/useChatIntegration.ts

@@ -213,8 +213,6 @@ BLOCKED: setTimeout, setInterval, eval, require, import, process, __dirname, __f
     [sendMessageFn, buildContext]
   );
 
-  // Connection is now handled automatically by GlobalWebSocketManager
-
   const improvePendingRef = useRef<{
     active: boolean;
     baseCount: number;

web/src/hooks/editor/useEditorKeyboardShortcuts.ts

@@ -7,7 +7,6 @@ export function useEditorKeyboardShortcuts(options: {
 }) {
   const { onToggleFullscreen, onToggleAssistant, onToggleEditorMode } = options;
 
-  // Register combos; the hook itself is declarative wrappers around useCombo
   useCombo(["Control", "shift", "f"], onToggleFullscreen, false);
   useCombo(["Meta", "shift", "f"], onToggleFullscreen, false);
   useCombo(["Control", "shift", "a"], onToggleAssistant, false);

web/src/hooks/editor/useMonacoEditor.ts

@@ -13,9 +13,6 @@ async function configureMonacoLoader() {
   loaderConfigured = true;
 }
 
-/**
- * Monaco editor component type definition.
- */
 type MonacoComponent = (props: {
   value: string;
   onChange?: (val?: string) => void;
@@ -27,46 +24,18 @@ type MonacoComponent = (props: {
   onMount?: (editor: monaco.editor.IStandaloneCodeEditor, monaco: typeof import("monaco-editor")) => void;
 }) => React.JSX.Element;
 
-/**
- * Result object containing Monaco editor state and functions.
- */
 type MonacoEditorResult = {
-  /** The lazily-loaded Monaco editor component */
   MonacoEditor: MonacoComponent | null;
-  /** Error message if Monaco failed to load */
   monacoLoadError: string | null;
-  /** True while the Monaco bundle is being fetched */
   isMonacoLoading: boolean;
-  /** Function to load Monaco editor if not already loaded */
   loadMonacoIfNeeded: () => Promise<void>;
-  /** Reference to the Monaco editor instance */
   monacoRef: React.MutableRefObject<monaco.editor.IStandaloneCodeEditor | null>;
-  /** Callback called when Monaco editor mounts */
   monacoOnMount: (editor: monaco.editor.IStandaloneCodeEditor) => void;
-  /** Opens the find panel in the editor */
   handleMonacoFind: () => void;
-  /** Formats the document in the editor */
   handleMonacoFormat: () => void;
 };
 
-/**
- * Hook for lazy-loading and managing Monaco editor instance.
- *
- * This hook handles dynamic importing of the Monaco editor library,
- * providing lazy loading to avoid loading the heavy editor until needed.
- *
- * @returns Object containing Monaco editor state and control functions
- *
- * @example
- * ```typescript
- * const { MonacoEditor, loadMonacoIfNeeded, handleMonacoFormat } = useMonacoEditor();
- *
- * useEffect(() => { loadMonacoIfNeeded(); }, [loadMonacoIfNeeded]);
- *
- * if (!MonacoEditor) return <Loading />;
- * return <MonacoEditor value={code} language="typescript" />;
- * ```
- */
+/** Lazy-loads and manages a Monaco editor instance, deferring the heavy bundle until needed. */
 export function useMonacoEditor(): MonacoEditorResult {
   const [MonacoEditor, setMonacoEditor] = useState<MonacoComponent | null>(
     null
@@ -91,7 +60,6 @@ export function useMonacoEditor(): MonacoEditorResult {
     isLoadingRef.current = true;
     setIsMonacoLoading(true);
     try {
-      // Configure loader to use local monaco-editor instead of CDN
       await configureMonacoLoader();
       const mod = await import("@monaco-editor/react");
       setMonacoEditor(() => mod.default as unknown as MonacoComponent);
@@ -102,7 +70,7 @@ export function useMonacoEditor(): MonacoEditorResult {
       isLoadingRef.current = false;
       setIsMonacoLoading(false);
     }
-  }, []); // No dependencies - callback is now stable
+  }, []);
 
   const handleMonacoFind = useCallback(() => {
     try {

web/src/hooks/handlers/addNodeFromAsset.ts

@@ -1,5 +1,3 @@
-// create nodes from existing assets
-
 import { useCallback } from "react";
 import { XYPosition } from "@xyflow/react";
 import { Asset, NodeMetadata } from "../../stores/ApiTypes";

web/src/hooks/handlers/useClipboardContentPaste.ts

@@ -183,14 +183,6 @@ export const useClipboardContentPaste = () => {
         }
       }
 
-      // Legacy Electron API
-      if (window.api?.clipboard?.readText) {
-        const text = await window.api.clipboard.readText();
-        if (text && text.trim()) {
-          return text;
-        }
-      }
-
       // Fallback to web API
       if (navigator.clipboard && navigator.clipboard.readText) {
         const text = await navigator.clipboard.readText();