decentraland / comms-gatekeeper / 25114016119

85%

main: 85%

Build Type

Pull #259

github

Committed by LautaroPetaccio

Commit Message

fix: cache hoisting, dedup, and variant caching across adapters

Multiple HTTP-backed adapters had caches that weren't actually retaining
entries across calls. worlds.ts and lands.ts were calling
cachedFetch.cache<T>() inside their methods, creating a fresh LRU per
invocation. feature-flags.ts maintained a refresh-on-interval pattern
for the boolean flag state but bypassed it for variants, hitting the
upstream feature-flags service on every getVariants() call (used per
moderator-endpoint request).

Fixes:
- Hoist cachedFetch.cache<T>() calls to the factory body in worlds.ts
  and lands.ts so the LRU is created once and reused.
- Add a sibling variantsMap to feature-flags.ts populated on the
  existing 4-minute refresh cycle; getVariants() reads from the map.
- Per-address cache + in-flight dedup in names.ts for getNamesFromAddresses,
  plus cachedFetch wrapping for getNameOwner. encodeURIComponent the
  user-supplied name in the URL path.
- In-flight dedup for land-lease.ts; identity-checked .finally cleanup
  so refreshAuthorizations cannot wipe a newer chain's reference.
- Normalize authAddress casing in cache-key URLs (worlds, lands) so
  mixed-case callers share entries; fix case-sensitive permission
  checks in hasWorldStreamingPermission / hasWorldDeployPermission.
- Bump cachedFetch default max from 1000 to 5000; nameByAddressCache
  to 10000.
- Add Content-Type: application/json to the /profiles POST in
  fetchProfileBatch for consistency.

Pull Request Pull Request #259: fix: cache hoisting, dedup, and variant caching across adapters

Coverage Stats

1021 of 1339 branches covered (76.25%)

Branch coverage included in aggregate %.

36 of 98 new or added lines in 7 files covered. (36.73%)

1 existing line in 1 file now uncovered.

2871 of 3345 relevant lines covered (85.83%)

41.93 hits per line