Standardization with PHPStan and PHP_CodeSniffer is added

.gitignore

@@ -5,7 +5,7 @@
 /vendor/
 /storage/**/*.key
 /storage/**/*.log
-/storage/postman/*.json
+/storage/postman/
 /tests/build/
 /vite/
 /app/

composer.json

@@ -52,7 +52,7 @@
         "fakerphp/faker": "^1.23",
         "guzzlehttp/guzzle": "^7.9",
         "lion/command": "^5.1",
-        "lion/database": "^11.2",
+        "lion/database": "^11.3",
         "lion/dependency-injection": "^4.2",
         "lion/exceptions": "^2.0",
         "lion/files": "^8.1",
@@ -74,7 +74,7 @@
         "squizlabs/php_codesniffer": "^3.11"
     },
     "suggest": {
-        "ext-gd": "Required to use LionSpreadsheet.",
+        "ext-gd": "Required to use spreadsheet/files.",
         "ext-pdo": "Required to use all database features.",
         "ext-redis": "Required to use the Redis cache.",
         "fakerphp/faker": "Required to use the factory builder.",

phpstan.neon

@@ -0,0 +1,5 @@
+parameters:
+    level: max
+
+    excludePaths:
+        - %currentWorkingDirectory%/src/LionBundle/Traits/

phpunit.xml

@@ -24,6 +24,7 @@
 >
     <php>
         <env name="APP_ENV" value="testing" />
+
         <ini name="date.timezone" value="America/Bogota" />
         <ini name="intl.default_locale" value="C.UTF-8" />
         <ini name="memory_limit" value="2048M" />
@@ -44,33 +45,5 @@
         <testsuite name="All-Test">
             <directory>tests/</directory>
         </testsuite>
-
-        <testsuite name="Commands">
-            <directory>tests/Commands/</directory>
-        </testsuite>
-
-        <testsuite name="Enums">
-            <directory>tests/Enums/</directory>
-        </testsuite>
-
-        <testsuite name="Exceptions">
-            <directory>tests/Exceptions/</directory>
-        </testsuite>
-
-        <testsuite name="Helpers">
-            <directory>tests/Helpers/</directory>
-        </testsuite>
-
-        <testsuite name="Middleware">
-            <directory>tests/Middleware/</directory>
-        </testsuite>
-
-        <testsuite name="Traits">
-            <directory>tests/Traits/</directory>
-        </testsuite>
-
-        <testsuite name="Mailhog">
-            <file>tests/MailhogTest.php</file>
-        </testsuite>
     </testsuites>
 </phpunit>

src/LionBundle/Commands/Lion/Route/PostmanCollectionCommand.php

@@ -4,13 +4,11 @@
 
 namespace Lion\Bundle\Commands\Lion\Route;
 
-use Carbon\Carbon;
 use DI\Attribute\Inject;
 use GuzzleHttp\Exception\GuzzleException;
 use Lion\Bundle\Helpers\Commands\ClassFactory;
 use Lion\Bundle\Helpers\Commands\PostmanCollection;
 use Lion\Bundle\Helpers\Http\Fetch;
-use Lion\Bundle\Helpers\Http\Routes;
 use Lion\Command\Command;
 use Lion\Files\Store;
 use Lion\Helpers\Str;
@@ -141,7 +139,9 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $this->postmanCollection->addRoutes($this->routes);
 
-        $path = storage_path('postman/', false);
+        $this->postmanCollection->generateItems();
+
+        $path = storage_path('postman/');
 
         $this->store->folder($path);
 
@@ -185,20 +185,41 @@ private function fetchRoutes(): void
     {
         $this->postmanCollection->init($_ENV['SERVER_URL']);
 
-        $this->jsonName = $this->str->of(Carbon::now()->format('Y_m_d'))->concat('_lion_collection')->lower()->get();
-
-        $this->routes = json_decode(
-            fetch(
-                new Fetch(Route::GET, ($_ENV['SERVER_URL'] . '/route-list'), [
-                    'headers' => [
-                        'Lion-Auth' => $_ENV['SERVER_HASH'],
-                    ],
-                ])
-            )
-                ->getBody()
-                ->getContents(),
-            true
-        );
+        $this->jsonName = $this->str
+            ->of(now()->format('Y_m_d'))
+            ->concat('_lion_collection')
+            ->lower()
+            ->get();
+
+        $json = fetch(
+            new Fetch(Route::GET, ($_ENV['SERVER_URL'] . '/route-list'), [
+                'headers' => [
+                    'Lion-Auth' => $_ENV['SERVER_HASH'],
+                ],
+            ])
+        )
+            ->getBody()
+            ->getContents();
+
+        $decode = json_decode($json, true);
+
+        /**
+         * @var array{
+         *     array<string, array{
+         *          filters: array<int, string>,
+         *          handler: array{
+         *              controller: bool|array{
+         *                  name: string,
+         *                  function: string
+         *              },
+         *              callback: bool
+         *          }
+         *     }>
+         * } $routes
+         */
+        $routes = $decode;
+
+        $this->routes = $routes;
 
         array_pop($this->routes);
     }

src/LionBundle/Helpers/Commands/Migrations/Migrations.php

@@ -12,6 +12,7 @@
 use Lion\Bundle\Interface\SeedInterface;
 use Lion\Command\Command;
 use Lion\Files\Store;
+use stdClass;
 use Symfony\Component\Console\Output\OutputInterface;
 
 /**
@@ -50,8 +51,7 @@ public function setStore(Store $store): Migrations
     /**
      * Sorts the list of elements by the value defined in the INDEX constant
      *
-     * @param array<string, MigrationUpInterface|SeedInterface> $list [Class
-     * List]
+     * @param array<string, MigrationUpInterface|SeedInterface> $list [Class List]
      *
      * @return array<string, MigrationUpInterface|SeedInterface>
      *
@@ -72,6 +72,7 @@ public function orderList(array $list): array
                 return -1;
             }
 
+            /** @phpstan-ignore-next-line */
             return $classA::INDEX <=> $classB::INDEX;
         });
 
@@ -99,7 +100,10 @@ public function getMigrations(): array
                 $namespace = $this->store->getNamespaceFromFile($migration, 'Database\\Migrations\\', 'Migrations/');
 
                 if (!isset($this->loadedMigrations[$migration])) {
-                    $this->loadedMigrations[$migration] = new $namespace();
+                    /** @var MigrationUpInterface $migrationInstance */
+                    $migrationInstance = new $namespace();
+
+                    $this->loadedMigrations[$migration] = $migrationInstance;
                 }
 
                 $tableMigration = $this->loadedMigrations[$migration];
@@ -138,24 +142,25 @@ public function getMigrations(): array
     public function executeMigrations(Command $command, OutputInterface $output, array $files): void
     {
         foreach ($files as $namespace => $classObject) {
-            if ($classObject instanceof MigrationUpInterface) {
-                $response = $classObject->up();
+            $response = $classObject->up();
 
-                $output->writeln($command->warningOutput("\t>> MIGRATION: {$namespace}"));
+            /** @phpstan-ignore-next-line */
+            $message = "\t>> MIGRATION: {$response->message}";
 
-                if (isError($response)) {
-                    $output->writeln($command->errorOutput("\t>> MIGRATION: {$response->message}"));
-                } else {
-                    $output->writeln($command->successOutput("\t>> MIGRATION: {$response->message}"));
-                }
+            $output->writeln($command->warningOutput("\t>> MIGRATION: {$namespace}"));
+
+            if (isError($response)) {
+                $output->writeln($command->errorOutput($message));
+            } else {
+                $output->writeln($command->successOutput($message));
             }
         }
     }
 
     /**
      * Run the migrations
      *
-     * @param array<int, string> $list [List of classes]
+     * @param array<int, class-string> $list [List of classes]
      *
      * @return void
      */
@@ -192,10 +197,12 @@ public function executeMigrationsGroup(array $list): void
          */
         $run = function (array $list): void {
             foreach ($list as $migration) {
+                /** @phpstan-ignore-next-line */
                 $migration->up();
             }
         };
 
+        /** @phpstan-ignore-next-line */
         $run($this->orderList($migrations[TableInterface::class]));
 
         $run($migrations[ViewInterface::class]);