Merge pull request #7 from jackd248/skip-anonymous-class

feat: add support for skipping anonymous classes in DocBlock processing

Author: jackd248

src/Rules/DocBlockHeaderFixer.php

@@ -121,11 +121,62 @@ protected function applyFix(SplFileInfo $file, Tokens $tokens): void
                 continue;
             }
 
+            // Skip anonymous classes (preceded by 'new' keyword)
+            if ($token->isGivenKind(T_CLASS) && $this->isAnonymousClass($tokens, $index)) {
+                continue;
+            }
+
             $structureName = $this->getStructureName($tokens, $index);
             $this->processStructureDocBlock($tokens, $index, $annotations, $structureName);
         }
     }
 
+    private function isAnonymousClass(Tokens $tokens, int $classIndex): bool
+    {
+        // Look backwards for 'new' keyword
+        $insideAttribute = false;
+        for ($i = $classIndex - 1; $i >= 0; --$i) {
+            $token = $tokens[$i];
+
+            // Skip whitespace
+            if ($token->isWhitespace()) {
+                continue;
+            }
+
+            // When going backwards, ']' marks the end of an attribute (we enter it)
+            if (']' === $token->getContent()) {
+                $insideAttribute = true;
+                continue;
+            }
+
+            // T_ATTRIBUTE '#[' marks the start of an attribute (we exit it when going backwards)
+            if ($token->isGivenKind(T_ATTRIBUTE)) {
+                $insideAttribute = false;
+                continue;
+            }
+
+            // Skip everything inside attributes
+            if ($insideAttribute) {
+                continue;
+            }
+
+            // Skip modifiers that can appear between 'new' and 'class'
+            if ($token->isGivenKind([T_FINAL, T_READONLY])) {
+                continue;
+            }
+
+            // If we find 'new', it's an anonymous class
+            if ($token->isGivenKind(T_NEW)) {
+                return true;
+            }
+
+            // If we hit any other meaningful token, it's not anonymous
+            break;
+        }
+
+        return false;
+    }
+
     /**
      * @param array<string, string|array<string>> $annotations
      */

tests/src/Rules/DocBlockHeaderFixerTest.php

@@ -845,4 +845,109 @@ public function testInsertNewDocBlockWithClassNameAndSeparateNone(): void
         $expected = "<?php /**\n * TestClass.\n *\n * @author John Doe\n */class TestClass {}";
         self::assertSame($expected, $tokens->generateCode());
     }
+
+    public function testSkipsAnonymousClasses(): void
+    {
+        $code = '<?php $obj = new class {};';
+        $tokens = Tokens::fromCode($code);
+        $file = new SplFileInfo(__FILE__);
+
+        $method = new ReflectionMethod($this->fixer, 'applyFix');
+
+        $this->fixer->configure([
+            'annotations' => ['author' => 'John Doe'],
+            'separate' => 'none',
+            'ensure_spacing' => false,
+        ]);
+        $method->invoke($this->fixer, $file, $tokens);
+
+        // Code should remain unchanged - no DocBlock added to anonymous class
+        self::assertSame($code, $tokens->generateCode());
+    }
+
+    public function testSkipsAnonymousClassesButProcessesRegularClasses(): void
+    {
+        $code = '<?php class RegularClass {} $obj = new class {};';
+        $tokens = Tokens::fromCode($code);
+        $file = new SplFileInfo(__FILE__);
+
+        $method = new ReflectionMethod($this->fixer, 'applyFix');
+
+        $this->fixer->configure([
+            'annotations' => ['author' => 'John Doe'],
+            'separate' => 'none',
+            'ensure_spacing' => false,
+        ]);
+        $method->invoke($this->fixer, $file, $tokens);
+
+        $result = $tokens->generateCode();
+
+        // Regular class should have DocBlock
+        self::assertStringContainsString("/**\n * @author John Doe\n */class RegularClass", $result);
+        // Anonymous class should NOT have DocBlock (should remain as "new class")
+        self::assertStringContainsString('new class {}', $result);
+    }
+
+    public function testIsAnonymousClassDetectsAnonymousClass(): void
+    {
+        $code = '<?php $obj = new class {};';
+        $tokens = Tokens::fromCode($code);
+
+        $method = new ReflectionMethod($this->fixer, 'isAnonymousClass');
+
+        // Find the class token index
+        $classIndex = null;
+        for ($i = 0; $i < $tokens->count(); ++$i) {
+            if ($tokens[$i]->isGivenKind(T_CLASS)) {
+                $classIndex = $i;
+                break;
+            }
+        }
+
+        $result = $method->invoke($this->fixer, $tokens, $classIndex);
+
+        self::assertTrue($result);
+    }
+
+    public function testIsAnonymousClassReturnsFalseForRegularClass(): void
+    {
+        $code = '<?php class RegularClass {}';
+        $tokens = Tokens::fromCode($code);
+
+        $method = new ReflectionMethod($this->fixer, 'isAnonymousClass');
+
+        // Find the class token index
+        $classIndex = null;
+        for ($i = 0; $i < $tokens->count(); ++$i) {
+            if ($tokens[$i]->isGivenKind(T_CLASS)) {
+                $classIndex = $i;
+                break;
+            }
+        }
+
+        $result = $method->invoke($this->fixer, $tokens, $classIndex);
+
+        self::assertFalse($result);
+    }
+
+    public function testIsAnonymousClassWithAttribute(): void
+    {
+        $code = '<?php $obj = new #[SomeAttribute] class {};';
+        $tokens = Tokens::fromCode($code);
+
+        $method = new ReflectionMethod($this->fixer, 'isAnonymousClass');
+
+        // Find the class token index
+        $classIndex = null;
+        for ($i = 0; $i < $tokens->count(); ++$i) {
+            if ($tokens[$i]->isGivenKind(T_CLASS)) {
+                $classIndex = $i;
+                break;
+            }
+        }
+
+        $result = $method->invoke($this->fixer, $tokens, $classIndex);
+
+        self::assertTrue($result);
+    }
 }