Apply Coderabbitai Nitpick comments.

Author: terabytesoftw

src/Reflector.php

@@ -47,6 +47,21 @@ final class Reflector
      */
     private static array $reflectionClassCache = [];
 
+    /**
+     * Returns the current number of cached reflection classes.
+     *
+     * Usage example:
+     * ```php
+     * $cacheSize = \PHPForge\Helper\Reflector::cacheSize();
+     * ```
+     *
+     * @return int Number of cached reflection classes.
+     */
+    public static function cacheSize(): int
+    {
+        return count(self::$reflectionClassCache);
+    }
+
     /**
      * Returns class attributes, optionally filtered by attribute class.
      *
@@ -74,6 +89,19 @@ public static function classAttributes(object|string $class, string|null $attrib
         return $reflectionClass->getAttributes($attribute, $flags);
     }
 
+    /**
+     * Clears the internal reflection class cache.
+     *
+     * Usage example:
+     * ```php
+     * \PHPForge\Helper\Reflector::clearCache();
+     * ```
+     */
+    public static function clearCache(): void
+    {
+        self::$reflectionClassCache = [];
+    }
+
     /**
      * Returns the first matching instantiated property attribute.
      *
@@ -135,11 +163,11 @@ public static function properties(object|string $class, int|null $filter = null)
     {
         $reflectionClass = self::reflectionClass($class);
 
-        if ($filter === null) {
-            return $reflectionClass->getProperties();
+        if ($filter !== null) {
+            return $reflectionClass->getProperties($filter);
         }
 
-        return $reflectionClass->getProperties($filter);
+        return $reflectionClass->getProperties();
     }
 
     /**

tests/ReflectorTest.php

@@ -31,6 +31,27 @@
  */
 final class ReflectorTest extends TestCase
 {
+    public function testCacheSizeReturnsNumberOfCachedReflectionClasses(): void
+    {
+        Reflector::clearCache();
+
+        Reflector::shortName(ReflectionFixture::class);
+
+        self::assertSame(
+            1,
+            Reflector::cacheSize(),
+            'Should return one cached class after first reflection lookup.',
+        );
+
+        Reflector::shortName(stdClass::class);
+
+        self::assertSame(
+            2,
+            Reflector::cacheSize(),
+            'Should increase cache size when reflecting a different class.',
+        );
+    }
+
     public function testClassAttributesReturnsAllClassAttributes(): void
     {
         $attributes = Reflector::classAttributes(ReflectionFixture::class);
@@ -70,6 +91,26 @@ public function testClassAttributesWithFilterReturnsMatchingAttributesOnly(): vo
         );
     }
 
+    public function testClearCacheEmptiesReflectionClassCache(): void
+    {
+        Reflector::clearCache();
+        Reflector::shortName(ReflectionFixture::class);
+
+        self::assertGreaterThan(
+            0,
+            Reflector::cacheSize(),
+            'Should populate cache after reflection usage.',
+        );
+
+        Reflector::clearCache();
+
+        self::assertSame(
+            0,
+            Reflector::cacheSize(),
+            'Should clear all cached reflection classes.',
+        );
+    }
+
     public function testFirstPropertyAttributeReturnsFirstMatchingAttributeInstance(): void
     {
         $attribute = Reflector::firstPropertyAttribute(ReflectionFixture::class, 'name', Label::class);
@@ -105,6 +146,27 @@ public function testHasPropertyReturnsExpectedResult(): void
         );
     }
 
+    public function testPropertiesReturnsAllPropertiesWhenFilterIsNull(): void
+    {
+        $allProperties = Reflector::properties(ReflectionFixture::class);
+        $allPropertyNames = [];
+
+        foreach ($allProperties as $property) {
+            $allPropertyNames[] = $property->getName();
+        }
+
+        self::assertContains(
+            'name',
+            $allPropertyNames,
+            'Should include public properties when no filter is provided.',
+        );
+        self::assertContains(
+            'hidden',
+            $allPropertyNames,
+            'Should include private properties when no filter is provided.',
+        );
+    }
+
     public function testPropertiesReturnsFilteredListWhenVisibilityFilterIsProvided(): void
     {
         $publicProperties = Reflector::properties(ReflectionFixture::class, ReflectionProperty::IS_PUBLIC);