📦 Use WeakMap instead of SplObjectStorage to avoid memory leak

[andrew-demb]

https://www.php.net/manual/en/class.weakmap.php

Using WeakMap will avoid preserving objects in memory if they are presented only in cache

[andrew-demb]

I'll push fixes ASAP

[Copilot]

Pull Request Overview

This pull request replaces the existing SplObjectStorage–based cache with a WeakMap to prevent objects from being retained in memory solely by the cache.

• Swapped out SplObjectStorage for WeakMap in the Matcher class
• Updated docblocks and constructor/clear logic to use WeakMap
• Changed cache lookup from contains() to offsetExists()

Comments suppressed due to low confidence (1)

src/Knp/Menu/Matcher/Matcher.php:74

• Add unit tests to verify that the WeakMap-based cache correctly stores and expires items, including behavior of the clear() method.

$this->cache = new \WeakMap();