resolve lint errors and clean up dead code

Author: AustinKelsay

src/components/AddShare.tsx

@@ -414,23 +414,10 @@ const AddShare: React.FC<AddShareProps> = ({ onComplete, onCancel }) => {
 
           const relays = relayPlanResult.relayPlan.relays;
 
-          // Enable verbose echo logging only when explicitly requested and when
-          // a Node-like `process` global is available (renderer may not have it).
-          const debugEnv = typeof process !== 'undefined' ? process.env.IGLOO_DEBUG_ECHO : undefined;
-          const debugEnabled = ((debugEnv ?? '').toLowerCase() === '1' || (debugEnv ?? '').toLowerCase() === 'true');
-
           await sendEcho(groupCredential, shareCredential, challenge, {
             relays,
             timeout: 10000,
-            // Bubble up debug logs if requested; guard access to process.env
-            eventConfig: debugEnabled
-              ? {
-                  enableLogging: true,
-                  customLogger: (level: string, message: string, data?: unknown) => {
-                    console.log(`[echo-send] ${level.toUpperCase()} ${message}`, data ?? '');
-                  }
-                }
-              : undefined
+            // Debug logging not available in renderer (process.env inaccessible with contextIsolation)
           });
 
           console.log('Echo sent successfully');

src/components/Recover.tsx

@@ -479,20 +479,15 @@ const Recover: React.FC<RecoverProps> = ({
   }, []);
 
   // Cleanup timeouts on unmount
+  // Note: Accessing .current in cleanup is intentional for timeout refs -
+  // we want to clear whatever timeout is active at unmount time.
   useEffect(() => {
     return () => {
-      if (nsecClearTimeoutRef.current) {
-        clearTimeout(nsecClearTimeoutRef.current);
-      }
-      if (copiedTimeoutRef.current) {
-        clearTimeout(copiedTimeoutRef.current);
-      }
-      if (copyErrorTimeoutRef.current) {
-        clearTimeout(copyErrorTimeoutRef.current);
-      }
-      if (autofilledTimeoutRef.current) {
-        clearTimeout(autofilledTimeoutRef.current);
-      }
+      if (nsecClearTimeoutRef.current) clearTimeout(nsecClearTimeoutRef.current);
+      if (copiedTimeoutRef.current) clearTimeout(copiedTimeoutRef.current);
+      if (copyErrorTimeoutRef.current) clearTimeout(copyErrorTimeoutRef.current);
+      // eslint-disable-next-line react-hooks/exhaustive-deps
+      if (autofilledTimeoutRef.current) clearTimeout(autofilledTimeoutRef.current);
     };
   }, []);
 

src/lib/signer-keepalive.ts

@@ -71,7 +71,7 @@ export function createSignerKeepAlive(config: KeepAliveConfig): SignerKeepAliveH
     groupCredential,
     shareCredential,
     relays,
-    selfPubkey,
+    // selfPubkey available in config but not currently used
   } = config;
 
   const logger: KeepAliveLogger = rawLogger ?? (() => undefined);

src/main.ts

@@ -34,9 +34,9 @@ function sanitizeErrorForLog(error: unknown): string {
   const message = error instanceof Error ? error.message : String(error);
   return message
     // Quoted Unix paths: '/path/with spaces/file.txt' -> '<path>/file.txt'
-    .replace(/(['"])(\/(?:[^\/]+\/)+)([^\/'"]*)\1/g, '$1<path>/$3$1')
+    .replace(/(['"])(\/(?:[^/]+\/)+)([^/'"]*)\1/g, '$1<path>/$3$1')
     // Unquoted Unix paths: /path/to/file.txt -> <path>/file.txt
-    .replace(/\/(?:[^\/\s:]+\/)+([^\/\s:]+)/g, '<path>/$1')
+    .replace(/\/(?:[^/\s:]+\/)+([^/\s:]+)/g, '<path>/$1')
     // Quoted Windows paths: "C:\path\with spaces\file.txt" -> "<path>\file.txt"
     .replace(/(['"])([A-Za-z]:\\(?:[^\\]+\\)+)([^\\'"]*)(\1)/g, '$1<path>\\$3$1')
     // Unquoted Windows paths: C:\path\to\file.txt -> <path>\file.txt
@@ -426,7 +426,7 @@ app.whenReady().then(() => {
       const existing = activeEchoListeners.get(listenerId);
       try {
         existing?.cleanup?.();
-      } catch (err) {
+      } catch {
         console.warn('[EchoBridge] Failed to cleanup existing listener during start', { listenerId });
       } finally {
         activeEchoListeners.delete(listenerId);

src/preload.ts

@@ -9,6 +9,7 @@
  */
 
 import { contextBridge, ipcRenderer } from 'electron';
+import type { IpcRendererEvent } from 'electron';
 import type { RelayPlan } from '@/types';
 
 // Type definitions for the exposed API
@@ -118,7 +119,7 @@ contextBridge.exposeInMainWorld('electronAPI', {
       console.error('[preload] onEchoReceived requires a function callback');
       return () => {}; // Return no-op cleanup
     }
-    const handler = (_event: Electron.IpcRendererEvent, data: unknown) => {
+    const handler = (_event: IpcRendererEvent, data: unknown) => {
       if (isEchoReceivedData(data)) {
         callback(data);
       } else {
@@ -138,7 +139,7 @@ contextBridge.exposeInMainWorld('electronAPI', {
       console.error('[preload] onEchoError requires a function callback');
       return () => {}; // Return no-op cleanup
     }
-    const handler = (_event: Electron.IpcRendererEvent, data: unknown) => {
+    const handler = (_event: IpcRendererEvent, data: unknown) => {
       if (isEchoErrorData(data)) {
         callback(data);
       } else {