chore: standardization with PHPStan and PHP_CodeSniffer is added

Author: Sleon4

.gitignore

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

phpstan.neon

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

phpunit.xml

@@ -24,7 +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" />

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]);