Enhancement: Implement InstanceofStrategy

CHANGELOG.md

@@ -1,6 +1,8 @@
 # Changelog
 
 ## [Unreleased]
+### Added
+- Added an `InstanceofStrategy` which detects usages in `instanceof` expressions [#100](https://github.com/composer-unused/composer-unused/pull/100)
 ### Fixed
 - Fixed an issue where `ext-ds` classes where not recognized as used [#88](https://github.com/composer-unused/composer-unused/pull/88)
 - Fixed an issue where `extends` and `implements` of FQN was not markes as used [#90](https://github.com/composer-unused/composer-unused/pull/90)

src/Parser/PHP/Factory/NodeVisitorFactory.php

@@ -9,6 +9,7 @@
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\ClassConstStrategy;
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\ExtendsParseStrategy;
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\ImplementsParseStrategy;
+use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\InstanceofStrategy;
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\NewParseStrategy;
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\PhpExtensionStrategy;
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\StaticParseStrategy;
@@ -31,6 +32,7 @@ public function __invoke(ContainerInterface $container): NodeVisitor
             ),
             new ExtendsParseStrategy(),
             new ImplementsParseStrategy(),
+            new InstanceofStrategy(),
         ], $container->get(ErrorHandlerInterface::class));
     }
 }

src/Parser/PHP/Strategy/InstanceofStrategy.php

@@ -0,0 +1,38 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Icanhazstring\Composer\Unused\Parser\PHP\Strategy;
+
+use PhpParser\Node;
+use PhpParser\Node\Expr\ClassConstFetch;
+
+class InstanceofStrategy implements ParseStrategyInterface
+{
+    public function meetsCriteria(Node $node): bool
+    {
+        if (!$node instanceof Node\Expr\Instanceof_) {
+            return false;
+        }
+
+        if (!$node->class instanceof Node\Name) {
+            return false;
+        }
+
+        return $node->class->isFullyQualified() || $node->class->isQualified();
+    }
+
+    /**
+     * @param Node&ClassConstFetch $node
+     * @return array<string>
+     */
+    public function extractNamespaces(Node $node): array
+    {
+        /** @var Node\Name $class */
+        $class = $node->class;
+
+        return [
+            $class->toString(),
+        ];
+    }
+}

tests/Integration/Parser/PHP/NodeVisitorTest.php

@@ -10,6 +10,7 @@
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\ClassConstStrategy;
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\ExtendsParseStrategy;
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\ImplementsParseStrategy;
+use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\InstanceofStrategy;
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\NewParseStrategy;
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\ParseStrategyInterface;
 use Icanhazstring\Composer\Unused\Parser\PHP\Strategy\PhpExtensionStrategy;
@@ -177,7 +178,14 @@ public function itShouldParseUsagesDataProvider(): array
                 ],
                 'inputFile' => ASSET_DIR . '/TestFiles/ClassImplements.php',
                 'strategy' => new ImplementsParseStrategy()
-            ]
+            ],
+            'Instanceof' => [
+                'expectedUseNamespaces' => [
+                    'Bar\Baz',
+                ],
+                'inputFile' => ASSET_DIR . '/TestFiles/Instanceof.php',
+                'strategy' => new InstanceofStrategy(),
+            ],
         ];
     }
 

tests/assets/TestFiles/Instanceof.php

@@ -0,0 +1,15 @@
+<?php
+
+use Foo\Bar;
+use Baz\Qux;
+
+
+if ($foo instanceof Bar\Baz) {
+    // usage should be detected because class is a name
+}
+
+$baz = 'Qux\Quux';
+
+if ($bar instanceof $baz) {
+    // usage should not be detected because class is a variable expression
+}