RobinTail / express-zod-api / 25093561398

100%

push

github

ref(mem,perf): Turning well-known headers into a `Set` (#3351)

This should solve two problems:
1. Unnecessary memory consumption (2k) by the large array that is
statically imported while only used by Documentation (not API runtime);
2. `Array::includes()` is O(n) while `Set::has` is O(1): in current
situation it could give about 5.5x performance improvement in worst
scenario (finding last entry)

The solution, however, requires a lazy creation of the `Set` because it
uses even more memory, so:
- ~~`R.once()` is involved, which is also reducing the expected
performance boost down to 3.58x~~,
- module cache with nullish coalescing assignment used giving 4.8x boost
(better than R.once)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Performance Improvements**
* Header lookup now uses a memoized set for faster, lower-memory checks.

* **Tests**
  * Added benchmarks measuring header lookup performance.
  * Added unit tests validating the memoized header collection.
  * Removed an obsolete queue-operations benchmark.

* **Chores**
* Updated generated tooling output to use the new header lookup
approach.

* **Changelog**
  * Recorded release v27.2.5 noting memory and lookup improvements.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

946 of 986 branches covered (95.94%)

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

1385 of 1385 relevant lines covered (100.0%)

403.17 hits per line