laantungir/amethyst
1
0
Fork 0
mirror of https://github.com/vitorpamplona/amethyst.git synced 2026-08-10 08:27:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

perf(clink): cache parsed NOffer for NIP-05 lookups + case-insensitive key

Browse Source 
The profile-header NIP-05 clink_offer cache stored the raw noffer string,
re-running ClinkPointerParser on every cache hit; it now stores the parsed
NOffer. The cache key is lowercased since NIP-05 identifiers are
case-insensitive, so casing variants no longer trigger duplicate fetches.

https://claude.ai/code/session_01NM2TyJtosLdY5ycjyabSRS
This commit is contained in:
Claude
2026-06-10 14:30:02 +00:00
parent 33b3eb5b6f
commit fdc09ad039
1 changed files with 16 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
amethyst/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/profile/header/DrawAdditionalInfo.kt
+16 -15
View File
@@ -390,14 +390,15 @@ fun getIdentityClaimDescription(identity: IdentityClaimTag): Int =
}
/**
* Process-wide cache of NIP-05 `.well-known` `clink_offer` lookups, keyed by the nip05
* address. Without it, every profile visit (and every relay-pushed kind-0 refresh while a
* profile is open) would re-fetch the domain's nostr.json. Caches "no offer" results too so
* profiles without one aren't re-hit. A [ResolvedClinkOffer] wrapper holds the nullable result
* Process-wide cache of NIP-05 `.well-known` `clink_offer` lookups, keyed by the
* lowercased nip05 address (NIP-05 identifiers are case-insensitive). Without it, every
* profile visit (and every relay-pushed kind-0 refresh while a profile is open) would
* re-fetch the domain's nostr.json. Caches "no offer" results too so profiles without one
* aren't re-hit. A [ResolvedClinkOffer] wrapper holds the nullable parsed pointer
* (LruCache can't store nulls); absence means "not fetched yet".
*/
private class ResolvedClinkOffer(
val noffer: String?,
val noffer: NOffer?,
)
private val clinkOfferNip05Cache = LruCache<String, ResolvedClinkOffer>(256)
@@ -430,16 +431,16 @@ private fun DisplayClinkOffer(
offer =
if (id != null && nip05 != null) {
// Distinguish "cache miss" from a cached "no offer" (null) so we don't refetch.
val cached = clinkOfferNip05Cache.get(nip05)
val nofferStr =
if (cached != null) {
cached.noffer
} else {
val fetched = withContext(Dispatchers.IO) { accountViewModel.nip05ClientBuilder().loadClinkOffer(id) }
clinkOfferNip05Cache.put(nip05, ResolvedClinkOffer(fetched))
fetched
}
nofferStr?.let { ClinkPointerParser.parse(it) as? NOffer }
val cacheKey = nip05.lowercase()
val cached = clinkOfferNip05Cache.get(cacheKey)
if (cached != null) {
cached.noffer
} else {
val fetched = withContext(Dispatchers.IO) { accountViewModel.nip05ClientBuilder().loadClinkOffer(id) }
val parsed = fetched?.let { ClinkPointerParser.parse(it) as? NOffer }
clinkOfferNip05Cache.put(cacheKey, ResolvedClinkOffer(parsed))
parsed
}
} else {
null
}