Remove ClassDescription::getInterfaces() — now dead code

Since Implement and NotImplement switched to pure reflection,
getInterfaces() on ClassDescription was no longer read by any
production expression. The data was collected but never used.

- FileVisitor: replace addInterface() with addDependency() to preserve
  the dependency tracking (interfaces are still recorded as class
  dependencies, which is used by DependsOnlyOnTheseNamespaces etc.)
- ClassDescriptionBuilder: remove addInterface() method and $interfaces field
- ClassDescription: remove getInterfaces() method, $interfaces field,
  and the corresponding constructor parameter
- Update tests that called addInterface() or getInterfaces()

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: fain182

src/Analyzer/ClassDescription.php

@@ -13,9 +13,6 @@ class ClassDescription
     /** @var list<ClassDependency> */
     private array $dependencies;
 
-    /** @var list<FullyQualifiedClassName> */
-    private array $interfaces;
-
     /** @var list<FullyQualifiedClassName> */
     private array $extends;
 
@@ -42,7 +39,6 @@ class ClassDescription
 
     /**
      * @param list<ClassDependency>         $dependencies
-     * @param list<FullyQualifiedClassName> $interfaces
      * @param list<FullyQualifiedClassName> $extends
      * @param list<string>                  $docBlock
      * @param list<FullyQualifiedClassName> $attributes
@@ -51,7 +47,6 @@ class ClassDescription
     public function __construct(
         FullyQualifiedClassName $FQCN,
         array $dependencies,
-        array $interfaces,
         array $extends,
         bool $final,
         bool $readonly,
@@ -67,7 +62,6 @@ public function __construct(
         $this->FQCN = $FQCN;
         $this->filePath = $filePath;
         $this->dependencies = $dependencies;
-        $this->interfaces = $interfaces;
         $this->extends = $extends;
         $this->final = $final;
         $this->readonly = $readonly;
@@ -137,14 +131,6 @@ public function getDependencies(): array
         return $this->dependencies;
     }
 
-    /**
-     * @return list<FullyQualifiedClassName>
-     */
-    public function getInterfaces(): array
-    {
-        return $this->interfaces;
-    }
-
     /**
      * @return list<FullyQualifiedClassName>
      */

src/Analyzer/ClassDescriptionBuilder.php

@@ -12,9 +12,6 @@ class ClassDescriptionBuilder
 
     private ?FullyQualifiedClassName $FQCN = null;
 
-    /** @var list<FullyQualifiedClassName> */
-    private array $interfaces = [];
-
     /** @var list<FullyQualifiedClassName> */
     private array $extends = [];
 
@@ -45,7 +42,6 @@ public function clear(): void
     {
         $this->FQCN = null;
         $this->classDependencies = [];
-        $this->interfaces = [];
         $this->extends = [];
         $this->final = false;
         $this->readonly = false;
@@ -72,14 +68,6 @@ public function setClassName(string $FQCN): self
         return $this;
     }
 
-    public function addInterface(string $FQCN, int $line): self
-    {
-        $this->addDependency(new ClassDependency($FQCN, $line));
-        $this->interfaces[] = FullyQualifiedClassName::fromString($FQCN);
-
-        return $this;
-    }
-
     public function addDependency(ClassDependency $cd): self
     {
         if ($this->isPhpCoreClass($cd)) {
@@ -172,7 +160,6 @@ public function build(): ClassDescription
         return new ClassDescription(
             $this->FQCN,
             $this->classDependencies,
-            $this->interfaces,
             $this->extends,
             $this->final,
             $this->readonly,

src/Analyzer/FileVisitor.php

@@ -129,7 +129,7 @@ private function handleClassNode(Node $node): void
 
         foreach ($node->implements as $interface) {
             $this->classDescriptionBuilder
-                ->addInterface($interface->toString(), $interface->getLine());
+                ->addDependency(new ClassDependency($interface->toString(), $interface->getLine()));
         }
 
         if (null !== $node->extends) {
@@ -180,7 +180,7 @@ private function handleEnumNode(Node $node): void
 
         foreach ($node->implements as $interface) {
             $this->classDescriptionBuilder
-                ->addInterface($interface->toString(), $interface->getLine());
+                ->addDependency(new ClassDependency($interface->toString(), $interface->getLine()));
         }
     }
 

tests/Unit/Analyzer/ClassDescriptionBuilderTest.php

@@ -12,15 +12,14 @@
 
 class ClassDescriptionBuilderTest extends TestCase
 {
-    public function test_it_should_create_builder_with_dependency_and_interface(): void
+    public function test_it_should_create_builder_with_dependency(): void
     {
         $FQCN = 'HappyIsland';
 
         $classDescription = (new ClassDescriptionBuilder())
             ->setFilePath('src/Foo.php')
             ->setClassName($FQCN)
             ->addDependency(new ClassDependency('DepClass', 10))
-            ->addInterface('InterfaceClass', 10)
             ->build();
 
         self::assertInstanceOf(ClassDescription::class, $classDescription);

tests/Unit/Analyzer/FileParser/CanParseClassTest.php

@@ -445,9 +445,9 @@ class Test implements Order
         $cd = $this->parseCode($code, TargetPhpVersion::PHP_8_1);
         $cd = $cd[2]; // class Test
 
-        $interfaces = array_map(static fn ($i) => $i->toString(), $cd->getInterfaces());
-        self::assertContains('Foo\Order', $interfaces);
-        self::assertNotContains('Foo\OrderTwo', $interfaces);
+        $deps = array_map(static fn ($d) => $d->getFQCN()->toString(), $cd->getDependencies());
+        self::assertContains('Foo\Order', $deps);
+        self::assertNotContains('Foo\OrderTwo', $deps);
     }
 
     public function test_it_parse_interfaces(): void

tests/Unit/Expressions/ForClasses/IsATest.php

@@ -23,7 +23,6 @@ public function test_it_should_have_no_violation_when_it_implements(): void
         $classDescription = (new ClassDescriptionBuilder())
             ->setFilePath('src/Foo.php')
             ->setClassName(CavendishBanana::class)
-            ->addInterface($interface, 10)
             ->build();
 
         $violations = new Violations();