decentraland / social-service-ea / 23739845667

90%

main: 90%

push

github

fix: resolve OOM crashes with heap limit, TTL-based subscribers, and SCAN (#406)

* fix: resolve OOM crashes with heap limit, TTL-based subscribers, and SCAN

* fix: prettier formatting in subscribers-context

* revert: remove TTL-based subscriber changes (to be tackled separately)

* fix: pin localstack to 4.12.0 to fix CI

localstack/localstack:latest broke with the 2026.03.0 release (exit code 55).
Pin to 4.12.0 which was the last working version.

* fix: remove hardcoded heap limit, let V8 auto-size from container memory

- Remove --max-old-space-size=384 from Dockerfile and package.json start
  script. Container memory will be increased in the infra definitions,
  letting V8's container-aware auto-sizing handle heap allocation.
- Restore --inspect flag in Dockerfile for debug port mapping.
- Fix scanIterator mock in community-voice-cache tests to follow the
  same pattern as other Redis mock variables.
- Re-add TTL-based subscriber tracking with proper test mocks.

* fix: revert subscriber tests to match SET-based source code

The TTL-based subscriber changes were reverted in the source but the
tests were updated to expect TTL-based behavior, causing TS compilation
errors (config not in Pick<BaseComponents, 'redis' | 'logs'>).

Restore tests to match the current SET-based subscribers-context source.
Remove subscriber-redis-mock helper that was only used by TTL tests.

* perf: use mGet instead of individual get calls in community voice cache

Replace Promise.all of N individual redis.get() calls with a single
redis.mGet() call in getActiveCommunityVoiceChats, reducing N Redis
round-trips to 1.

2208 of 2585 branches covered (85.42%)

Branch coverage included in aggregate %.

5 of 5 new or added lines in 1 file covered. (100.0%)

5429 of 5880 relevant lines covered (92.33%)

72.01 hits per line