A cold boot can no longer dress a real account in a punk for a day: an empty profile answer (the 5s query racing still-dialing relays) is retried in minutes instead of being trusted for the 24h TTL, faceless rows are never persisted (they were evicting real faces from the 150-cap), and the logged-in identity's own profile is pinned against eviction entirely

Co-Authored-By: Claude Fable 5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01MrkLsNScXe9b6BviUSdmMr

1 files changed, 17 insertions, 2 deletions

diff --git a/src/features/messages.js b/src/features/messages.js
index 803afc9..f0e3069 100644
--- a/src/features/messages.js
+++ b/src/features/messages.js
@@ -258,11 +258,20 @@ export function messagesFeature(ctx) {
     for (const [pk, p] of Object.entries(cached)) if (!profiles.has(pk)) profiles.set(pk, p);
   }
   function persistProfile(pk, p) {
+    // A faceless answer is not a fact worth writing down — persisting
+    // {name:null, picture:null} rows only evicts real faces from the cap and
+    // spreads a cold-relay miss across sessions.
+    if (!p || (!p.name && !p.picture)) return;
     const s = wallet.loadFeatureState('profiles', {});
     s[pk] = { name: p.name || null, picture: p.picture || null, nip05: p.nip05 || null, lud16: p.lud16 || null, t: Date.now() };
     const keys = Object.keys(s);
     if (keys.length > 150) {
-      for (const k of keys.sort((a, b) => (s[a].t || 0) - (s[b].t || 0)).slice(0, keys.length - 150)) delete s[k];
+      // the user's own faces never churn out — their avatar going punk over
+      // a busy community room is the one eviction people actually notice
+      const keep = new Set([(hook('nostrLoginIdentity') || {}).pubkey,
+        wallet.nostrPubkey && wallet.nostrPubkey()].filter(Boolean));
+      const evictable = keys.filter((k) => !keep.has(k));
+      for (const k of evictable.sort((a, b) => (s[a].t || 0) - (s[b].t || 0)).slice(0, keys.length - 150)) delete s[k];
     }
     wallet.saveFeatureState('profiles', s);
   }
@@ -271,10 +280,16 @@ export function messagesFeature(ctx) {
   // while the image downloads.
   const preloadPicture = (p) => { try { if (p && p.picture) new Image().src = p.picture; } catch {} };
 
+  // A profile that came back EMPTY is usually a cold boot's 5s query racing
+  // relays that were still dialing — not proof there's no kind 0. Trusting it
+  // for the full TTL once dressed a real account in a punk and an npub for a
+  // whole day; an empty answer is retried in minutes instead.
+  const EMPTY_RETRY = 10 * 60_000;
   function profileOf(pk) {
     warmProfiles();
     const cur = profiles.get(pk);
-    if (cur !== undefined && (cur === null || Date.now() - (cur.t || 0) < PROFILE_TTL)) return cur;
+    if (cur !== undefined && (cur === null
+      || Date.now() - (cur.t || 0) < (!cur.name && !cur.picture ? EMPTY_RETRY : PROFILE_TTL))) return cur;
     profiles.set(pk, cur || null); // null = loading, no fallback art yet
     fetchNostrProfile(pk).then((p) => {
       // An empty answer on a cold boot (relays still dialing, the 5s query