archetech / archon / 39183213766

93%

push

github

fix: list Redis DIDs with SCAN instead of KEYS (#1408)

* fix: list Redis DIDs with SCAN instead of KEYS (#1407)

KEYS blocks Redis while it walks every key in the database, and on a
production node most keys are block checkpoints: listing 33k DIDs among
833k keys stalled every Redis client for about 180 ms, once a minute.
Both Gatekeepers now collect DID keys with SCAN in steps of 10,000 keys,
and Rust checks whether any DID exists without listing them all. The Rust
snapshot guardrail's KEYS count uses SCAN too. Tests in both ports
surround a few DIDs with thousands of block keys and assert through
Redis command statistics that KEYS is not used.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Xm4mh8ubYTGQWq1cgffgbq

* ci: run the Redis SCAN regression test with the Redis backend

The general Rust test step has no Redis URL, so redis_scan skipped there;
run it in the Redis and MongoDB backend step alongside redis_compat.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Xm4mh8ubYTGQWq1cgffgbq

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

3943 of 4524 branches covered (87.16%)

Branch coverage included in aggregate %.

10692 of 11156 relevant lines covered (95.84%)

20096.75 hits per line