Feature/laravel auto discovery

composer.json

@@ -35,6 +35,13 @@
     "config": {
         "sort-packages": true
     },
+    "extra": {
+        "laravel": {
+            "providers": [
+                "PhpSwag\\Bridges\\Laravel\\PhpSwagServiceProvider"
+            ]
+        }
+    },
     "bin": ["bin/phpswag"],
     "scripts": {
         "lint": [

src/Bridges/Laravel/Commands/GenerateCommand.php

@@ -13,7 +13,9 @@ class GenerateCommand extends Command
      *
      * @var string
      */
-    protected $signature = 'phpswag:generate {--validate : Run validation on the generated spec}';
+    protected $signature = 'phpswag:generate'
+        . ' {--validate : Run validation on the generated spec}'
+        . ' {--filter-unused= : Filter unused schemas (true or false)}';
 
     /**
      * The console command description.
@@ -67,6 +69,15 @@ public function handle(): int
                 $core->enableCache($cacheFile);
             }
 
+            // Apply filter-unused configuration
+            $filterUnusedOption = $this->option('filter-unused');
+            if ($filterUnusedOption === null || $filterUnusedOption === '') {
+                $filterUnused = (bool)($config['filter_unused'] ?? true);
+            } else {
+                $filterUnused = filter_var($filterUnusedOption, FILTER_VALIDATE_BOOLEAN);
+            }
+            $core->setFilterUnusedSchemas($filterUnused);
+
             // Perform validation if flag is set
             if ($this->option('validate')) {
                 $specArray = $core->generateSpecArray($paths);

src/Bridges/Laravel/config/phpswag.php

@@ -59,4 +59,15 @@
     */
     'swagger_ui' => true,
     'swagger_ui_path' => '/api/docs',
+
+    /*
+    |--------------------------------------------------------------------------
+    | Filter Unused Schemas
+    |--------------------------------------------------------------------------
+    |
+    | When set to true, schemas that are not referenced by any route/path
+    | will be filtered out from the generated specification.
+    |
+    */
+    'filter_unused' => true,
 ];

src/Core.php

@@ -438,7 +438,10 @@ private function discoverStatement(Node $stmt, NameResolver $nameResolver): void
             $tags = $this->docCollector->collectTags($docComment, $docStartLine, $this->currentlyAnalyzingFile);
             foreach ($tags as $tag) {
                 if ($tag['name'] === '@template') {
-                    $templates[] = $tag['value'];
+                    $parts = preg_split('/\s+/', trim($tag['value']));
+                    if (!empty($parts[0])) {
+                        $templates[] = $parts[0];
+                    }
                 }
 
                 if ($tag['name'] === '@extends' || $tag['name'] === '@implements') {

tests/FrameworkBridgesTest.php

@@ -14,6 +14,10 @@
 
 class FrameworkBridgesTest extends TestCase
 {
+    public static function setUpBeforeClass(): void
+    {
+        require_once __DIR__ . '/laravel-helpers.php';
+    }
     public function testSymfonyConfigurationTree()
     {
         $configuration = new Configuration();
@@ -140,4 +144,59 @@ public function testLaravelServiceProviderInstantiation()
 
         $this->assertInstanceOf(\Illuminate\Support\ServiceProvider::class, $provider);
     }
+
+    public function testLaravelGenerateCommand()
+    {
+        global $laravelConfig;
+        $laravelConfig = [
+            'phpswag' => [
+                'paths' => ['examples/App'],
+                'output' => 'test-laravel-spec.yaml',
+                'format' => 'yaml',
+                'title' => 'Laravel Spec',
+                'version' => '2.0.0',
+                'description' => 'Generated in Laravel tests',
+                'host' => 'http://localhost:8000',
+                'cache' => false,
+                'filter_unused' => true,
+            ]
+        ];
+
+        $command = new \PhpSwag\Bridges\Laravel\Commands\GenerateCommand();
+        $laravelApp = $this->createMock(\Illuminate\Contracts\Foundation\Application::class);
+        $laravelApp->method('runningUnitTests')->willReturn(true);
+        $laravelApp->method('make')->willReturnCallback(function ($abstract, $parameters = []) {
+            if ($abstract === \Illuminate\Console\OutputStyle::class) {
+                return new \Illuminate\Console\OutputStyle($parameters['input'], $parameters['output']);
+            }
+            return null;
+        });
+        $laravelApp->method('call')->willReturnCallback(function ($callback, $parameters = []) {
+            if (is_array($callback) && $callback[0] instanceof \PhpSwag\Bridges\Laravel\Commands\GenerateCommand) {
+                return $callback[0]->handle();
+            }
+            return is_callable($callback) ? $callback(...$parameters) : null;
+        });
+        $command->setLaravel($laravelApp);
+
+        $application = new SymfonyApplication();
+        $application->add($command);
+
+        $tester = new CommandTester($application->find('phpswag:generate'));
+
+        $tester->execute([]);
+        $this->assertEquals(0, $tester->getStatusCode());
+        $this->assertStringContainsString('Documentation generated successfully to test-laravel-spec.yaml', $tester->getDisplay());
+
+        if (file_exists('test-laravel-spec.yaml')) {
+            unlink('test-laravel-spec.yaml');
+        }
+
+        // Test option override
+        $tester->execute(['--filter-unused' => 'false']);
+        $this->assertEquals(0, $tester->getStatusCode());
+        if (file_exists('test-laravel-spec.yaml')) {
+            unlink('test-laravel-spec.yaml');
+        }
+    }
 }

tests/GenericsTest.php

@@ -76,4 +76,35 @@ public function testGenericInheritance()
         $this->assertStringContainsString("message:", $yaml);
         $this->assertStringContainsString("App_User", $yaml);
     }
+
+    public function testGenericsWithConstraints()
+    {
+        $dir = __DIR__ . '/fixtures/generics_constraints';
+        @mkdir($dir, 0777, true);
+
+        file_put_contents($dir . '/BaseModel.php', '<?php namespace App; class BaseModel { public string $id; }');
+        file_put_contents($dir . '/QueryFilter.php', '<?php namespace App; /** @template TModel of BaseModel */ class QueryFilter { }');
+        file_put_contents($dir . '/SortFilter.php', '<?php namespace App; use App\QueryFilter; use App\BaseModel;
+        /**
+         * @template TModel of BaseModel
+         *
+         * @extends QueryFilter<TModel>
+         */
+        class SortFilter extends QueryFilter { }');
+        file_put_contents($dir . '/Controller.php', '<?php namespace App;
+        use App\SortFilter;
+        use App\BaseModel;
+        class Controller {
+            /**
+             * @route GET /sort
+             * @response 200 SortFilter<BaseModel>
+             */
+            public function getSort() {}
+        }');
+
+        $core = new Core();
+        $yaml = $core->generate([$dir]);
+
+        $this->assertStringContainsString("App_SortFilter:", $yaml);
+    }
 }

tests/fixtures/generics_constraints/BaseModel.php

@@ -0,0 +1 @@
+<?php namespace App; class BaseModel { public string $id; }

tests/fixtures/generics_constraints/Controller.php

@@ -0,0 +1,10 @@
+<?php namespace App;
+        use App\SortFilter;
+        use App\BaseModel;
+        class Controller {
+            /**
+             * @route GET /sort
+             * @response 200 SortFilter<BaseModel>
+             */
+            public function getSort() {}
+        }

tests/fixtures/generics_constraints/QueryFilter.php

@@ -0,0 +1 @@
+<?php namespace App; /** @template TModel of BaseModel */ class QueryFilter { }

tests/fixtures/generics_constraints/SortFilter.php

@@ -0,0 +1,7 @@
+<?php namespace App; use App\QueryFilter; use App\BaseModel;
+        /**
+         * @template TModel of BaseModel
+         *
+         * @extends QueryFilter<TModel>
+         */
+        class SortFilter extends QueryFilter { }

tests/laravel-helpers.php

@@ -0,0 +1,30 @@
+<?php
+
+if (!function_exists('app_path')) {
+    function app_path(string $path = ''): string
+    {
+        return $path;
+    }
+}
+if (!function_exists('public_path')) {
+    function public_path(string $path = ''): string
+    {
+        return $path;
+    }
+}
+if (!function_exists('storage_path')) {
+    function storage_path(string $path = ''): string
+    {
+        return $path;
+    }
+}
+if (!function_exists('config')) {
+    function config(string $key = null, mixed $default = null): mixed
+    {
+        global $laravelConfig;
+        if ($key === null) {
+            return $laravelConfig;
+        }
+        return $laravelConfig[$key] ?? $default;
+    }
+}