JayBizzle / Crawler-Detect / 27306077372

100%

push

github

Key compiled-regex cache by fixture class instead of pattern hash (#610)

The memoization added in c617d86 keyed the cache with
md5(serialize($patterns)), which costs ~100us per construction —
about 7x more than the ~14us implode() it was meant to avoid, making
every 'new CrawlerDetect' slower than having no cache at all.

The pattern lists are fixed class properties, so the fixture class
name is a sufficient cache key. compileRegex() is restored to a pure
compile step and the memoization moves to a class-keyed helper.

Measured: ~104us -> ~2.6us per construction (PHP 8.4).

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

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

44 of 44 relevant lines covered (100.0%)

14.41 hits per line