make use of laravel di

Author: TomasVotruba

.gitignore

@@ -1,2 +1,4 @@
 composer.lock
-/vendor
+/vendor
+
+.phpunit.cache

bin/console.php

@@ -1,6 +1,22 @@
 <?php
 
-// decoupled in own "*.php" file, so ECS, Rector and PHPStan works out of the box here
-
 declare(strict_types=1);
 
+use Rector\PhpParserNodesDocs\DependencyInjection\ContainerFactory;
+use Symfony\Component\Console\Application;
+use Symfony\Component\Console\Input\ArgvInput;
+use Symfony\Component\Console\Output\ConsoleOutput;
+
+require_once __DIR__ . '/../vendor/autoload.php';
+
+$containerFactory = new ContainerFactory();
+$container = $containerFactory->create();
+
+/** @var Application $application */
+$application = $container->make(Application::class);
+
+$input = new ArgvInput();
+$output = new ConsoleOutput();
+
+$exitCode = $application->run($input, $output);
+exit($exitCode);

composer.json

@@ -10,13 +10,14 @@
         "php": "^8.1",
         "symfony/console": "^6.3",
         "nikic/php-parser": "^4.16",
-        "illuminate/container": "^10.15"
+        "illuminate/container": "^10.15",
+        "webmozart/assert": "^1.11"
     },
     "require-dev": {
         "symplify/easy-coding-standard": "^11.5",
         "phpstan/phpstan": "^1.10",
         "phpunit/phpunit": "^10.2",
-        "rector/rector": "*"
+        "rector/rector": "^0.17.6"
     },
     "autoload": {
         "psr-4": {

config/config.php

@@ -7,7 +7,6 @@
 
 return static function (ECSConfig $ecsConfig): void {
     $ecsConfig->paths([
-        __DIR__ . '/config',
         __DIR__ . '/src',
         __DIR__ . '/tests',
     ]);

ecs.php

@@ -2,11 +2,10 @@ parameters:
     level: max
 
     paths:
-        - config
         - src
         - tests
 
     ignoreErrors:
         -
-            message: '#Cannot cast array<string\>\|bool\|string\|null to string#'
+            message: '#Cannot cast (.*?) to string#'
             path: src/Command/DumpNodesCommand.php

phpstan.neon

@@ -0,0 +1,14 @@
+<?xml version="1.0"?>
+<phpunit
+        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+        xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
+        bootstrap="vendor/autoload.php"
+        colors="true"
+        cacheDirectory=".phpunit.cache"
+>
+    <testsuites>
+        <testsuite name="unit">
+            <directory>tests</directory>
+        </testsuite>
+    </testsuites>
+</phpunit>

phpunit.xml

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\CodeQuality\Rector\Class_\InlineConstructorDefaultToPropertyRector;
+use Rector\Config\RectorConfig;
+use Rector\Set\ValueObject\LevelSetList;
+
+return static function (RectorConfig $rectorConfig): void {
+    $rectorConfig->importNames();
+
+    $rectorConfig->paths([
+        __DIR__ . '/src',
+        __DIR__ . '/tests',
+    ]);
+
+    $rectorConfig->sets([
+        LevelSetList::UP_TO_PHP_81
+    ]);
+};

rector.php

@@ -10,10 +10,6 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-use Symplify\PackageBuilder\Console\ShellCode;
-use Symplify\SmartFileSystem\SmartFileInfo;
-use Symplify\SmartFileSystem\SmartFileSystem;
 
 final class DumpNodesCommand extends Command
 {
@@ -22,42 +18,16 @@ final class DumpNodesCommand extends Command
      */
     private const OUTPUT_FILE = 'output-file';
 
-    /**
-     * @var MarkdownNodeInfosPrinter
-     */
-    private $markdownNodeInfosPrinter;
-
-    /**
-     * @var NodeInfosFactory
-     */
-    private $nodeInfosFactory;
-
-    /**
-     * @var SmartFileSystem
-     */
-    private $smartFileSystem;
-
-    /**
-     * @var SymfonyStyle
-     */
-    private $symfonyStyle;
-
     public function __construct(
-        MarkdownNodeInfosPrinter $markdownNodeInfosPrinter,
-        NodeInfosFactory $nodeInfosFactory,
-        SmartFileSystem $smartFileSystem,
-        SymfonyStyle $symfonyStyle
+        private readonly MarkdownNodeInfosPrinter $markdownNodeInfosPrinter,
+        private readonly NodeInfosFactory $nodeInfosFactory,
     ) {
-        $this->markdownNodeInfosPrinter = $markdownNodeInfosPrinter;
-        $this->nodeInfosFactory = $nodeInfosFactory;
-        $this->smartFileSystem = $smartFileSystem;
-        $this->symfonyStyle = $symfonyStyle;
-
         parent::__construct();
     }
 
     protected function configure(): void
     {
+        $this->setName('dump-nodes');
         $this->setDescription('Dump nodes overview');
 
         $this->addOption(
@@ -74,18 +44,16 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $outputFile = (string) $input->getOption(self::OUTPUT_FILE);
 
         $nodeInfos = $this->nodeInfosFactory->create();
-
         $printedContent = $this->markdownNodeInfosPrinter->print($nodeInfos);
-        $this->smartFileSystem->dumpFile($outputFile, $printedContent);
 
-        $outputFileFileInfo = new SmartFileInfo($outputFile);
-        $message = sprintf(
+        file_put_contents($outputFile, $printedContent);
+
+        $output->writeln(sprintf(
             'Documentation for "%d" PhpParser Nodes was generated to "%s"',
             count($nodeInfos),
-            $outputFileFileInfo->getRelativeFilePathFromCwd()
-        );
-        $this->symfonyStyle->success($message);
+            $outputFile
+        ));
 
-        return ShellCode::SUCCESS;
+        return self::SUCCESS;
     }
 }

src/Command/DumpNodesCommand.php

@@ -0,0 +1,29 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\PhpParserNodesDocs\DependencyInjection;
+
+use Illuminate\Container\Container;
+use Rector\PhpParserNodesDocs\Command\DumpNodesCommand;
+use Symfony\Component\Console\Application;
+
+final class ContainerFactory
+{
+    public function create(): Container
+    {
+        $container = new Container();
+
+        $container->singleton(Application::class, function (Container $container) {
+            $application = new Application();
+
+            /** @var DumpNodesCommand $dumpNodesCommand */
+            $dumpNodesCommand = $container->make(DumpNodesCommand::class);
+            $application->add($dumpNodesCommand);
+
+            return $application;
+        });
+
+        return $container;
+    }
+}

src/DependencyInjection/ContainerFactory.php

@@ -0,0 +1,60 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\PhpParserNodesDocs\Finder;
+
+use Webmozart\Assert\Assert;
+
+final class PhpFilesFinder
+{
+    /**
+     * @param string[] $paths
+     * @return string[]
+     */
+    public function findPhpFiles(array $paths): array
+    {
+        Assert::allFileExists($paths);
+
+        // fallback to config paths
+        $filePaths = [];
+
+        foreach ($paths as $path) {
+            if (is_file($path)) {
+                $filePaths[] = $path;
+            } else {
+                $currentFilePaths = $this->findFilesUsingGlob($path);
+                $filePaths = array_merge($filePaths, $currentFilePaths);
+            }
+        }
+
+        sort($filePaths);
+
+        Assert::allString($filePaths);
+        Assert::allFileExists($filePaths);
+
+        return $filePaths;
+    }
+
+    /**
+     * @return string[]
+     */
+    private function findFilesUsingGlob(string $directory): array
+    {
+        // Search for php files in the current directory
+        /** @var string[] $phpFiles */
+        $phpFiles = glob($directory . '/*.php');
+
+        // recursively search in subdirectories
+
+        /** @var string[] $subdirectories */
+        $subdirectories = glob($directory . '/*', GLOB_ONLYDIR);
+
+        foreach ($subdirectories as $subdirectory) {
+            // Merge the results from subdirectories
+            $phpFiles = array_merge($phpFiles, $this->findFilesUsingGlob($subdirectory));
+        }
+
+        return $phpFiles;
+    }
+}