fix: address Copilot review feedback on reads article improvements

- Run general (no-hashtag) article fetch even when cache is fresh so
  non-food categories populate
- Hydrate reads page from shared articleStore on load for cross-page
  caching between /reads and /explore
- Optimize shared store sync to only add newly appended articles
- Make selectedSort reactive to login state transitions
- Separate follow pubkey set from profile cache to prevent non-followed
  pubkeys from skewing "For You" ranking
- Scan all image regex matches in extractImage (not just first)
- Fix misleading IndexedDB comment in articleStore
- Remove unused imports in LongformFoodFeed

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 2 people

package.json

@@ -1,7 +1,7 @@
 {
   "name": "zap.cooking",
   "license": "MIT",
-  "version": "4.2.118",
+  "version": "4.2.119",
   "private": true,
   "scripts": {
     "dev": "vite dev",

src/components/LongformFoodFeed.svelte

@@ -7,11 +7,10 @@
   import {
     TOP_RELAY_FOOD_HASHTAGS,
     isValidLongformArticle,
-    isValidLongformArticleNoFoodFilter,
     eventToArticleData,
     type ArticleData
   } from '$lib/articleUtils';
-  import { articleStore, foodArticles, addArticles } from '$lib/articleStore';
+  import { foodArticles, addArticles } from '$lib/articleStore';
 
   let localArticles: ArticleData[] = [];
   let loading = true;

src/components/table/FeedSection.svelte

@@ -16,6 +16,11 @@
   let selectedCategory = 'All';
   let selectedSort: SortOption = isSignedIn ? 'foryou' : 'newest';
 
+  // When user signs in, switch to "For You" if still on unsigned default
+  $: if (isSignedIn && selectedSort === 'newest') {
+    selectedSort = 'foryou';
+  }
+
   // Filter out cover articles and articles without real images (no placeholders in reads)
   $: feedArticles = articles.filter((a) => !coverArticleIds.includes(a.id) && a.imageUrl);
   $: dedupedArticles = deduplicatePreviewArticles(feedArticles);

src/lib/articleStore.ts

@@ -1,6 +1,6 @@
 /**
  * Shared article store — single source of truth for articles across reads and explore pages.
- * Articles are fetched once and cached in this store + IndexedDB.
+ * Articles are fetched once and kept in these Svelte stores (in-memory cache).
  */
 import { writable, derived, get } from 'svelte/store';
 import type { ArticleData, CuratedCover } from './articleUtils';

src/lib/articleUtils.ts

@@ -301,10 +301,12 @@ export function extractImage(event: NDKEvent): string | null {
     return imageTag[1];
   }
 
-  // Extract first image URL from content
+  // Extract first valid image URL from content
   const imageMatches = content.match(IMAGE_URL_REGEX);
-  if (imageMatches && imageMatches.length > 0 && isValidImageUrl(imageMatches[0])) {
-    return imageMatches[0];
+  if (imageMatches) {
+    for (const url of imageMatches) {
+      if (isValidImageUrl(url)) return url;
+    }
   }
 
   return null;
@@ -479,7 +481,7 @@ const MAX_ARTICLES_PER_AUTHOR = 3; // Limit articles per author in feed
 
 // Blacklisted pubkeys — known spammers filtered at quality-check level
 const BLACKLISTED_PUBKEYS = new Set<string>([
-  // Add known spammer hex pubkeys here
+  '73d9e19ef07e0d098fc0fc5fb75db0f854824e8b4e43905acce638ddf6469960', // npub1w0v7r8hs0cxsnr7ql30mwhdslp2gyn5tfepeqkkvucudmajxn9sqgz5svp
 ]);
 
 // Spam title patterns (case-insensitive)

src/lib/followListCache.ts

@@ -17,6 +17,8 @@ export interface CachedProfile {
 
 // Singleton cache
 const profileCache = new Map<string, CachedProfile>();
+/** Pubkeys derived exclusively from the kind:3 contact list (not from search/addToCache) */
+const followPubkeySet = new Set<string>();
 let isLoading = false;
 let isLoaded = false;
 let loadPromise: Promise<void> | null = null;
@@ -76,11 +78,17 @@ async function doLoadFollowList(): Promise<void> {
       return;
     }
 
-    // Extract all follow pubkeys
+    // Extract all follow pubkeys from the kind:3 contact list
     const followPubkeys = contactEvent.tags
       .filter(t => t[0] === 'p' && t[1])
       .map(t => t[1]);
-    
+
+    // Populate the authoritative follow set (independent of profile cache)
+    followPubkeySet.clear();
+    for (const pk of followPubkeys) {
+      followPubkeySet.add(pk);
+    }
+
     if (followPubkeys.length === 0) {
       isLoaded = true;
       followListReady.set(true);
@@ -198,11 +206,12 @@ export function searchCachedProfiles(query: string, limit: number = 10): CachedP
 }
 
 /**
- * Get the set of followed pubkeys (without profile data).
+ * Get the set of followed pubkeys (derived from kind:3 contact list).
  * Returns empty set if not loaded yet.
+ * This is independent of the profile cache — addToCache() does not affect it.
  */
 export function getFollowedPubkeys(): Set<string> {
-  return new Set(profileCache.keys());
+  return new Set(followPubkeySet);
 }
 
 /**
@@ -217,6 +226,7 @@ export function isCacheLoaded(): boolean {
  */
 export function resetCache(): void {
   profileCache.clear();
+  followPubkeySet.clear();
   isLoaded = false;
   isLoading = false;
   loadPromise = null;

src/routes/reads/+page.svelte

@@ -26,7 +26,8 @@
   import PencilSimpleLineIcon from 'phosphor-svelte/lib/PencilSimpleLine';
   import FolderIcon from 'phosphor-svelte/lib/Folder';
   import { loadFollowListProfiles, getFollowedPubkeys, followListReady } from '$lib/followListCache';
-  import { addArticles as addToSharedStore, refreshCover as refreshSharedCover } from '$lib/articleStore';
+  import { articleStore, addArticles as addToSharedStore, refreshCover as refreshSharedCover } from '$lib/articleStore';
+  import { get } from 'svelte/store';
 
   $: isSignedIn = $userPublickey !== '';
   $: draftCount = $drafts.length;
@@ -51,9 +52,19 @@
   let pullToRefreshEl: PullToRefresh;
   
   // Sync local articles to shared store (for explore page reuse)
-  $: if (articles.length > 0) {
-    addToSharedStore(articles);
-    refreshSharedCover();
+  // Only sync newly added articles to avoid rebuilding on every update
+  let lastSyncedArticleCount = 0;
+  $: {
+    if (articles.length > lastSyncedArticleCount) {
+      const newArticles = articles.slice(lastSyncedArticleCount);
+      if (newArticles.length > 0) {
+        addToSharedStore(newArticles);
+        refreshSharedCover();
+      }
+      lastSyncedArticleCount = articles.length;
+    } else if (articles.length === 0) {
+      lastSyncedArticleCount = 0;
+    }
   }
 
   // Pagination tracking
@@ -155,11 +166,37 @@
     // Always fetch all articles - feed shows all, cover always food-editorial
     const cacheFilter = { kinds: [30023], hashtags: getFoodHashtags(), limit: 500 };
 
-    // Try to load from cache first for instant paint
+    // Try to hydrate from shared store first (e.g. if user visited /explore first)
     let cacheWasUsed = false;
     let cacheSufficient = false;
-    
-    if (!forceRefresh && browser) {
+
+    if (!forceRefresh) {
+      const shared = get(articleStore);
+      if (shared.length > 0 && articles.length === 0) {
+        for (const article of shared) {
+          if (!seenEventIds.has(article.id)) {
+            seenEventIds.add(article.id);
+          }
+        }
+        articles = [...shared].sort((a, b) => b.publishedAt - a.publishedAt);
+        lastSyncedArticleCount = articles.length;
+
+        const coverArticles = articles.filter(a => isValidLongformArticle(a.event));
+        if (coverArticles.length >= 1) {
+          cover = curateCover(coverArticles, false);
+        }
+        loading = false;
+        cacheWasUsed = true;
+        cacheSufficient = true;
+
+        if (articles.length > 0) {
+          oldestTimestamp = Math.min(...articles.map(a => a.publishedAt));
+        }
+      }
+    }
+
+    // Try to load from IndexedDB cache for instant paint
+    if (!forceRefresh && !cacheSufficient && browser) {
       try {
         const cachedEvents = await loadCachedFeedEvents(cacheFilter);
         if (cachedEvents.length > 0) {
@@ -190,20 +227,29 @@
       }
     }
 
-    // If cache is fresh and sufficient, skip network fetch entirely
-    // Schedule a background refresh for later instead
+    // If cache is fresh and sufficient, skip the primary food fetch
+    // but still run the general fetch + background refresh so non-food categories populate
     if (cacheSufficient && isCacheFresh() && !forceRefresh) {
       if (import.meta.env.DEV) {
-        console.log('[Reads] Cache is fresh, skipping network fetch');
+        console.log('[Reads] Cache is fresh, skipping primary food fetch');
       }
-      
-      // Schedule background refresh after delay (won't block UI)
+
+      const startGen = getCurrentRelayGeneration();
+
+      // General (all-topic) fetch so non-food categories aren't empty
+      setTimeout(() => {
+        if (getCurrentRelayGeneration() === startGen) {
+          fetchGeneralArticles(startGen);
+        }
+      }, 1000);
+
+      // Schedule background refresh after delay
       setTimeout(() => {
         if (browser) {
           backgroundRefresh();
         }
       }, BACKGROUND_REFRESH_DELAY_MS);
-      
+
       return;
     }
 
@@ -534,7 +580,8 @@
         'wss://relay.primal.net',
         'wss://nos.lol',
         'wss://relay.damus.io',
-        'wss://nostr.wine'
+        'wss://nostr.wine',
+        'wss://antiprimal.net'
       ];
       
       console.log('[Reads] Direct fallback: querying', articleRelays.join(', '));