From 3fd82f24259a166bd8e7779e7015a97e0f9bbcc7 Mon Sep 17 00:00:00 2001 From: "shakespeare.diy" Date: Thu, 27 Aug 2026 10:17:01 +0700 Subject: [PATCH] #2 Reduce console noise: strip debug logs from relay queries, publish, trust score, weather, coaches, NostrSync Co-authored-by: shakespeare.diy --- src/components/NostrSync.tsx | 11 +---------- src/components/SpotMap.tsx | 9 --------- src/hooks/useCoaches.ts | 16 ---------------- src/hooks/useNostrPublish.ts | 15 --------------- src/hooks/useSkateRelays.ts | 11 ++++------- src/hooks/useSpotWeather.ts | 3 --- src/hooks/useTrustScore.ts | 7 +------ src/pages/MapPage.tsx | 9 --------- 8 files changed, 6 insertions(+), 75 deletions(-) diff --git a/src/components/NostrSync.tsx b/src/components/NostrSync.tsx index 6d97f6c..ee00418 100644 --- a/src/components/NostrSync.tsx +++ b/src/components/NostrSync.tsx @@ -79,8 +79,6 @@ export function NostrSync() { }, [user?.pubkey]); async function doSync(pubkey: string, attempt = 1) { - console.log(`[NostrSync] Starting sync for ${pubkey.slice(0, 8)}... attempt ${attempt}`); - // Always invalidate stale cache first so useAuthor refetches queryClient.invalidateQueries({ queryKey: ['author', pubkey] }); queryClient.removeQueries({ queryKey: ['author', pubkey] }); @@ -99,7 +97,6 @@ export function NostrSync() { const event = events[0]; try { const metadata = n.json().pipe(n.metadata()).parse(event.content); - console.log(`[NostrSync] Got profile: ${metadata.name ?? 'unnamed'}`); // Populate useAuthor cache queryClient.setQueryData(['author', pubkey], { metadata, event }); // Populate useLoggedInAccounts cache @@ -141,7 +138,6 @@ export function NostrSync() { write: !marker || marker === 'write', })); if (fetchedRelays.length > 0) { - console.log(`[NostrSync] Got ${fetchedRelays.length} relays from NIP-65`); updateConfig(c => ({ ...c, relayMetadata: { relays: fetchedRelays, updatedAt: event.created_at }, @@ -170,7 +166,6 @@ export function NostrSync() { // ── 1. Citrine (local relay — fastest, no internet needed) ─────────────── const citrineAvailable = await isCitrineAvailable(); if (citrineAvailable) { - console.log('[NostrSync] Citrine available — querying local relay'); const g = nostr.group([CITRINE_URL]); const [r, p] = await Promise.all([fetchRelays(g), fetchProfile(g)]); relaysSynced = r; @@ -179,7 +174,6 @@ export function NostrSync() { // ── 2. Bootstrap relays (always-on aggregators) ────────────────────────── if (!profileSynced || !relaysSynced) { - console.log('[NostrSync] Trying bootstrap relays...'); const g = nostr.group(BOOTSTRAP_RELAYS); const [r, p] = await Promise.all([ relaysSynced ? Promise.resolve(true) : fetchRelays(g), @@ -191,7 +185,6 @@ export function NostrSync() { // ── 3. Main relay pool ─────────────────────────────────────────────────── if (!profileSynced || !relaysSynced) { - console.log('[NostrSync] Trying main relay pool...'); const [r, p] = await Promise.all([ relaysSynced ? Promise.resolve(true) : fetchRelays(nostr), profileSynced ? Promise.resolve(true) : fetchProfile(nostr), @@ -201,7 +194,6 @@ export function NostrSync() { } const success = profileSynced && relaysSynced; - console.log(`[NostrSync] Done — profile:${profileSynced} relays:${relaysSynced}`); if (success) { syncState.current.set(pubkey, true); @@ -214,11 +206,10 @@ export function NostrSync() { retryTimers.current.set(pubkey, t); } else { syncState.current.set(pubkey, false); - console.error('[NostrSync] All attempts failed for', pubkey.slice(0, 8)); // Don't show error toast if profile simply doesn't exist yet — // it's a new account, not a connection error if (!profileSynced) { - console.log('[NostrSync] No profile found — may be a new account'); + // New account — no profile published yet, nothing to sync. } else { toast({ title: 'Could not load profile', diff --git a/src/components/SpotMap.tsx b/src/components/SpotMap.tsx index 9b79853..7cb0ef3 100644 --- a/src/components/SpotMap.tsx +++ b/src/components/SpotMap.tsx @@ -873,7 +873,6 @@ export function SpotMap({ // Add moveend handler for BTC Map updates map.on('moveend', () => { const center = map.getCenter(); - console.log('[SpotMap] Map moved to:', center.lat, center.lng); onMapMoveRef.current?.({ lat: center.lat, lon: center.lng }); }); @@ -923,10 +922,8 @@ export function SpotMap({ map.invalidateSize(); // Force a redraw to ensure everything is ready map.whenReady(() => { - console.log('[SpotMap] Map is ready'); // Trigger initial map move event const center = map.getCenter(); - console.log('[SpotMap] Initial center:', center.lat, center.lng); onMapMoveRef.current?.({ lat: center.lat, lon: center.lng }); }); }, 100); @@ -1019,8 +1016,6 @@ export function SpotMap({ // Ensure we have location data before adding markers if (locationGroups.length === 0) return; - console.log('[SpotMap] Adding', locationGroups.length, 'markers to map'); - // Add new markers for each location group locationGroups.forEach((group) => { const hasSpots = group.spots.length > 0; @@ -1146,8 +1141,6 @@ export function SpotMap({ // Add tour markers (separate from spot/shop markers) if (showTours && tours.length > 0) { - console.log('[SpotMap] Adding', tours.length, 'tour markers'); - tours.forEach((tour) => { // Use meeting point coords if available, otherwise decode geohash let lat: number, lng: number; @@ -1439,8 +1432,6 @@ export function SpotMap({ // Don't add markers if BTC Map is disabled if (!showBtcMap || btcMapPlaces.length === 0) return; - console.log('[SpotMap] Adding', btcMapPlaces.length, 'BTC Map markers'); - btcMapPlaces.forEach((place) => { const boosted = isBoosted(place.boosted_until); const recentlyVerified = isRecentlyVerified(place.verified_at); diff --git a/src/hooks/useCoaches.ts b/src/hooks/useCoaches.ts index 8536c18..8de9be7 100644 --- a/src/hooks/useCoaches.ts +++ b/src/hooks/useCoaches.ts @@ -13,8 +13,6 @@ export function useCoaches(limit = 100) { queryFn: async () => { const signal = AbortSignal.timeout(10000); - console.log('[useCoaches] Querying for coaches...'); - const events = await querySkateContent( [ { @@ -26,16 +24,12 @@ export function useCoaches(limit = 100) { { signal } ); - console.log(`[useCoaches] Received ${events.length} events`); - const coaches: SkateCoach[] = []; for (const event of events) { const coach = parseSkateCoach(event); if (coach) coaches.push(coach); } - console.log(`[useCoaches] Parsed coaches: ${coaches.length}`); - return coaches.sort((a, b) => b.createdAt - a.createdAt); }, staleTime: 30000, @@ -69,14 +63,9 @@ export function useCoach(identifier: string, pubkey?: string) { if (pubkey) filter.authors = [pubkey]; - console.log(`[useCoach] Querying for coach: ${identifier}`); - const events = await querySkateContent([filter], { signal }); - console.log(`[useCoach] Received ${events.length} events`); - if (events.length === 0) { - console.warn(`[useCoach] No events found for identifier: ${identifier}`); return null; } @@ -96,8 +85,6 @@ export function useUserCoachProfile(pubkey: string) { queryFn: async () => { const signal = AbortSignal.timeout(10000); - console.log(`[useUserCoachProfile] Querying for coach profile: ${pubkey}`); - const events = await querySkateContent( [ { @@ -110,10 +97,7 @@ export function useUserCoachProfile(pubkey: string) { { signal } ); - console.log(`[useUserCoachProfile] Received ${events.length} events`); - if (events.length === 0) { - console.warn(`[useUserCoachProfile] No coach profile found for pubkey: ${pubkey}`); return null; } diff --git a/src/hooks/useNostrPublish.ts b/src/hooks/useNostrPublish.ts index 46c20a7..cbc7839 100644 --- a/src/hooks/useNostrPublish.ts +++ b/src/hooks/useNostrPublish.ts @@ -18,8 +18,6 @@ export function useNostrPublish(): UseMutationResult { throw new Error("You must be logged in to publish events"); } - console.log('[useNostrPublish] Starting publish for kind:', t.kind); - const tags = t.tags ?? []; // Add the client tag if it doesn't exist @@ -27,31 +25,22 @@ export function useNostrPublish(): UseMutationResult { tags.push(["client", location.hostname]); } - console.log('[useNostrPublish] Tags prepared:', tags.length, 'tags'); - // Sign the event - console.log('[useNostrPublish] Signing event...'); const event = await user.signer.signEvent({ kind: t.kind, content: t.content ?? "", tags, created_at: t.created_at ?? Math.floor(Date.now() / 1000), }); - console.log('[useNostrPublish] Event signed successfully:', event.id); // Publish with retry logic - console.log('[useNostrPublish] Publishing to relays...'); await publishEventWithRetry(nostr, event); - console.log('[useNostrPublish] Publish complete!'); return event; }, onError: (error) => { console.error("Failed to publish event:", error); }, - onSuccess: (data) => { - console.log("Event published successfully:", data); - }, }); } @@ -75,8 +64,6 @@ async function publishEventWithRetry( try { // Attempt to publish await nostr.event(event, { signal: controller.signal }); - - console.log(`Event published successfully (attempt ${attempt + 1}/${maxRetries + 1})`); return; // Success! } finally { @@ -85,12 +72,10 @@ async function publishEventWithRetry( } catch (error) { lastError = error instanceof Error ? error : new Error(String(error)); - console.warn(`Publish attempt ${attempt + 1} failed:`, lastError.message); // If this isn't the last attempt, wait before retrying if (attempt < maxRetries) { const backoffMs = 1000 * Math.pow(2, attempt); - console.log(`Retrying in ${backoffMs}ms...`); await new Promise(resolve => setTimeout(resolve, backoffMs)); } } diff --git a/src/hooks/useSkateRelays.ts b/src/hooks/useSkateRelays.ts index 1ae02ef..eb7de3f 100644 --- a/src/hooks/useSkateRelays.ts +++ b/src/hooks/useSkateRelays.ts @@ -79,17 +79,15 @@ export function useSkateRelays() { ): Promise => { const relays = getReadRelays(); - console.log(`[useSkateRelays] Querying ${relays.length} relay(s):`, relays); - - // Query all relays in parallel + // Query all relays in parallel. Individual relay failures/timeouts are + // expected (some relays are slow or offline) and handled silently — the + // other relays still return results. const queryPromises = relays.map(async (relayUrl) => { try { const relay = nostr.relay(relayUrl); const events = await relay.query(filters, options); - console.log(`[useSkateRelays] ${relayUrl} returned ${events.length} events`); return events; - } catch (error) { - console.warn(`[useSkateRelays] Failed to query ${relayUrl}:`, error); + } catch { return []; } }); @@ -115,7 +113,6 @@ export function useSkateRelays() { } const uniqueEvents = Array.from(eventMap.values()); - console.log(`[useSkateRelays] Total unique events: ${uniqueEvents.length}`); return uniqueEvents; }; diff --git a/src/hooks/useSpotWeather.ts b/src/hooks/useSpotWeather.ts index 6413e5d..ce5d278 100644 --- a/src/hooks/useSpotWeather.ts +++ b/src/hooks/useSpotWeather.ts @@ -59,8 +59,6 @@ export function useMultipleSpotWeather(geohashes: string[], enabled: boolean = t return useQuery({ queryKey: ['multiple-spot-weather', geohashes.sort().join(',')], queryFn: async () => { - console.log('[useMultipleSpotWeather] Fetching weather for geohashes:', geohashes); - // Deduplicate geohashes to avoid fetching the same location multiple times const uniqueGeohashes = [...new Set(geohashes)]; @@ -100,7 +98,6 @@ export function useMultipleSpotWeather(geohashes: string[], enabled: boolean = t } }); - console.log('[useMultipleSpotWeather] Weather data fetched for', weatherMap.size, 'locations'); return weatherMap; }, enabled: enabled && geohashes.length > 0, diff --git a/src/hooks/useTrustScore.ts b/src/hooks/useTrustScore.ts index a26e826..449c3f4 100644 --- a/src/hooks/useTrustScore.ts +++ b/src/hooks/useTrustScore.ts @@ -42,7 +42,6 @@ export function useTrustScore(targetPubkey: string | undefined) { const followList = followListEvents[0]; if (!followList) { - console.warn('[useTrustScore] No follow list found for source pubkey'); return 0; } @@ -55,7 +54,6 @@ export function useTrustScore(targetPubkey: string | undefined) { // Direct follow = 90 if (following.has(targetPubkey)) { - console.log(`[useTrustScore] ${targetPubkey.slice(0, 8)} is directly followed: 90`); return 90; } @@ -71,7 +69,6 @@ export function useTrustScore(targetPubkey: string | undefined) { .map(([, pubkey]) => pubkey); if (friendFollowing.includes(targetPubkey)) { - console.log(`[useTrustScore] ${targetPubkey.slice(0, 8)} is friend-of-friend: 60`); return 60; } } @@ -86,16 +83,14 @@ export function useTrustScore(targetPubkey: string | undefined) { try { const metadata = JSON.parse(profileEvents[0].content); if (metadata.nip05) { - console.log(`[useTrustScore] ${targetPubkey.slice(0, 8)} has NIP-05: 30`); return 30; } - } catch (e) { + } catch { // Ignore JSON parse errors } } // No trust signals found - console.log(`[useTrustScore] ${targetPubkey.slice(0, 8)} has no trust signals: 0`); return 0; } catch (error) { console.warn('[useTrustScore] Error calculating trust score:', error); diff --git a/src/pages/MapPage.tsx b/src/pages/MapPage.tsx index 0f546bd..cf5d5a4 100644 --- a/src/pages/MapPage.tsx +++ b/src/pages/MapPage.tsx @@ -727,13 +727,6 @@ export default function MapPage() { enabled: showBtcMap, }); - // Debug logging for BTC Map - useEffect(() => { - if (showBtcMap) { - console.log('[MapPage] BTC Map enabled, center:', mapCenter, 'loading:', btcMapLoading, 'places:', btcMapPlaces?.length, 'error:', btcMapError); - } - }, [showBtcMap, mapCenter, btcMapLoading, btcMapPlaces, btcMapError]); - const isLoading = spotsLoading || shopsLoading || marketplaceLoading || toursLoading; // Get user's location for proximity sorting @@ -789,8 +782,6 @@ export default function MapPage() { .filter(spot => !spot.isShop && spot.geohash) .map(spot => spot.geohash); - console.log('[MapPage] Spot geohashes:', geohashes); - console.log('[MapPage] Weather enabled:', weatherEnabled); return geohashes; }, [allLocations]);