From f65f2c659750537910f716dff3f3dcb47ff7a48a Mon Sep 17 00:00:00 2001
From: Gigi <dergigi@pm.me>
Date: Fri, 17 Oct 2025 01:24:41 +0200
Subject: [PATCH] chore(lint): remove explicit any types, add deps for
 useEffect, and type relay logging

---
 src/App.tsx                     | 16 ++++++++--------
 src/services/bookmarkService.ts |  2 +-
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/App.tsx b/src/App.tsx
index faa39990..16a8c974 100644
--- a/src/App.tsx
+++ b/src/App.tsx
@@ -318,22 +318,22 @@ function App() {
 
                   // Debug: log publish/subscription calls made by signer (decrypt/sign requests)
                   // IMPORTANT: bind originals to preserve `this` context used internally by the signer
-                  const originalPublish = (recreatedSigner as any).publishMethod.bind(recreatedSigner)
-                  ;(recreatedSigner as any).publishMethod = (relays: string[], event: any) => {
+                  const originalPublish = (recreatedSigner as { publishMethod: (relays: string[], event: unknown) => unknown }).publishMethod.bind(recreatedSigner)
+                  ;(recreatedSigner as unknown as { publishMethod: (relays: string[], event: unknown) => unknown }).publishMethod = (relays: string[], event: unknown) => {
                     try {
                       const summary = {
                         relays,
-                        kind: event?.kind,
+                        kind: (event as { kind?: number })?.kind,
                         // include tags array for debugging (NIP-46 expects method tag)
-                        tags: event?.tags,
-                        contentLength: typeof event?.content === 'string' ? event.content.length : undefined
+                        tags: (event as { tags?: unknown })?.tags,
+                        contentLength: typeof (event as { content?: unknown })?.content === 'string' ? (event as { content: string }).content.length : undefined
                       }
                       console.log('[bunker] publish via signer:', summary)
                     } catch (err) { console.warn('[bunker] failed to log publish summary', err) }
                     return originalPublish(relays, event)
                   }
-                  const originalSubscribe = (recreatedSigner as any).subscriptionMethod.bind(recreatedSigner)
-                  ;(recreatedSigner as any).subscriptionMethod = (relays: string[], filters: any[]) => {
+                  const originalSubscribe = (recreatedSigner as { subscriptionMethod: (relays: string[], filters: unknown[]) => unknown }).subscriptionMethod.bind(recreatedSigner)
+                  ;(recreatedSigner as unknown as { subscriptionMethod: (relays: string[], filters: unknown[]) => unknown }).subscriptionMethod = (relays: string[], filters: unknown[]) => {
                     try {
                       console.log('[bunker] subscribe via signer:', { relays, filters })
                     } catch (err) { console.warn('[bunker] failed to log subscribe summary', err) }
@@ -462,7 +462,7 @@ function App() {
     return () => {
       if (cleanup) cleanup()
     }
-  }, [])
+  }, [isOnline, showToast])
 
   // Monitor online/offline status
   useEffect(() => {
diff --git a/src/services/bookmarkService.ts b/src/services/bookmarkService.ts
index 4c8228a2..7780bdbc 100644
--- a/src/services/bookmarkService.ts
+++ b/src/services/bookmarkService.ts
@@ -110,7 +110,7 @@ export const fetchBookmarks = async (
     
     // Debug relay connectivity for bunker relays
     try {
-      const urls = Array.from(relayPool.relays.values()).map(r => ({ url: r.url, connected: (r as any).connected }))
+      const urls = Array.from(relayPool.relays.values()).map(r => ({ url: r.url, connected: (r as unknown as { connected?: boolean }).connected }))
       console.log('[bunker] Relay connections:', urls)
     } catch (err) { console.warn('[bunker] Failed to read relay connections', err) }