Merge pull request #396 from alies-dev/395-no-hint-path-defined-for-ide-helper

Fix "no hint path defined for ide helper" error

Author: alies-dev

composer.json

@@ -13,7 +13,7 @@
     "require": {
         "php": "^8.1",
         "ext-simplexml": "*",
-        "barryvdh/laravel-ide-helper": "^2.13 || ^3.0",
+        "barryvdh/laravel-ide-helper": "~3.5.4",
         "illuminate/config": "^10.48 || ^11.0",
         "illuminate/container": "^10.48 || ^11.0",
         "illuminate/contracts": "^10.48 || ^11.0",

psalm-baseline.xml

@@ -1,13 +1,9 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<files psalm-version="6.6.0@07e9a53713232e924da9acb4a6e47507461cd411">
+<files psalm-version="6.8.8@1361cd33008feb3ae2b4a93f1860e14e538ec8c2">
   <file src="src/Fakes/FakeModelsCommand.php">
     <ClassMustBeFinal>
       <code><![CDATA[FakeModelsCommand]]></code>
     </ClassMustBeFinal>
-    <TooFewArguments>
-      <code><![CDATA[parent::__construct($files)]]></code>
-      <code><![CDATA[parent::__construct($files)]]></code>
-    </TooFewArguments>
   </file>
   <file src="src/Handlers/Auth/AuthConfigAnalyzer.php">
     <MixedArgument>
@@ -19,6 +15,10 @@
     </MixedReturnStatement>
   </file>
   <file src="src/Handlers/Auth/AuthHandler.php">
+    <LessSpecificImplementedReturnType>
+      <code><![CDATA[array]]></code>
+      <code><![CDATA[array]]></code>
+    </LessSpecificImplementedReturnType>
     <TypeDoesNotContainType>
       <code><![CDATA[null]]></code>
     </TypeDoesNotContainType>

psalm.xml

@@ -2,7 +2,7 @@
 <psalm
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xmlns="https://getpsalm.org/schema/config"
-    xsi:schemaLocation="https://getpsalm.org/schema/config config.xsd"
+    xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd"
     errorLevel="1"
     errorBaseline="psalm-baseline.xml"
     findUnusedBaselineEntry="false"

src/Fakes/FakeModelsCommand.php

@@ -21,9 +21,15 @@ class FakeModelsCommand extends ModelsCommand
     /** @var list<class-string<\Illuminate\Database\Eloquent\Model>> */
     private array $model_classes = [];
 
-    public function __construct(Filesystem $files, private SchemaAggregator $schema)
+    private SchemaAggregator $schema;
+
+    /**
+     * While the setter of a required property is an anti-pattern,
+     * this is the only way to be less independent of changes in the parent ModelsCommand constructor.
+     */
+    public function setSchemaAggregator(SchemaAggregator $schemaAggregator): void
     {
-        parent::__construct($files);
+        $this->schema = $schemaAggregator;
     }
 
     /** @return list<class-string<\Illuminate\Database\Eloquent\Model>> */

src/Handlers/Auth/AuthHandler.php

@@ -5,7 +5,9 @@
 namespace Psalm\LaravelPlugin\Handlers\Auth;
 
 use Psalm\Plugin\EventHandler\Event\MethodReturnTypeProviderEvent;
+use Psalm\Plugin\EventHandler\Event\MethodParamsProviderEvent;
 use Psalm\Plugin\EventHandler\MethodReturnTypeProviderInterface;
+use Psalm\Plugin\EventHandler\MethodParamsProviderInterface;
 use Psalm\Type;
 
 use function in_array;
@@ -28,7 +30,7 @@
  * @see \Illuminate\Support\Facades\Auth::setRequest()
  * @see \Illuminate\Support\Facades\Auth::forgetUser()
  */
-final class AuthHandler implements MethodReturnTypeProviderInterface
+final class AuthHandler implements MethodReturnTypeProviderInterface, MethodParamsProviderInterface
 {
     /** @inheritDoc */
     public static function getClassLikeNames(): array
@@ -81,4 +83,18 @@ public static function getMethodReturnType(MethodReturnTypeProviderEvent $event)
             default => null,
         };
     }
+
+    #[\Override]
+    public static function getMethodParams(MethodParamsProviderEvent $event): ?array
+    {
+        $method_name_lowercase = $event->getMethodNameLowercase();
+
+        if ($method_name_lowercase !== 'user') {
+            return null;
+        }
+
+        return match ($method_name_lowercase) {
+            'user' => [], // Explicitly define that user() has no parameters
+        };
+    }
 }

src/Providers/FacadeStubProvider.php

@@ -31,10 +31,15 @@ public static function generateStubFile(): void
 
         $fake_filesystem = new FakeFilesystem();
 
+        /**
+         * @var \Illuminate\View\Factory $viewFactory
+         */
+        $viewFactory = $app->make('view');
+
         $stubs_generator_command = new GeneratorCommand(
             $config,
             $fake_filesystem,
-            ViewFactoryProvider::get(),
+            $viewFactory
         );
 
         $stubs_generator_command->setLaravel($app);

src/Providers/ModelStubProvider.php

@@ -41,9 +41,10 @@ public static function generateStubFile(): void
 
         $models_generator_command = new FakeModelsCommand(
             $fake_filesystem,
-            $schema_aggregator
+            $app->make(\Illuminate\Contracts\Config\Repository::class),
+            $app->make(\Illuminate\View\Factory::class)
         );
-
+        $models_generator_command->setSchemaAggregator($schema_aggregator);
         $models_generator_command->setLaravel($app);
 
         @unlink(self::getStubFileLocation());
@@ -66,9 +67,7 @@ public static function getStubFileLocation(): string
         return CacheDirectoryProvider::getCacheLocation() . '/models.stubphp';
     }
 
-    /**
-     * @return list<class-string<\Illuminate\Database\Eloquent\Model>>
-     */
+    /** @return list<class-string<\Illuminate\Database\Eloquent\Model>> */
     public static function getModelClasses(): array
     {
         return self::$model_classes;

src/Providers/ViewFactoryProvider.php

@@ -7,10 +7,10 @@ class Migrator
     /**
      * Execute the given callback using the given connection as the default connection.
      *
-     * @template TCalbackReturn
+     * @template TCallbackReturn
      * @param string $name
-     * @param callable(): TCalbackReturn $callback
-     * @return TCalbackReturn
+     * @param callable(): TCallbackReturn $callback
+     * @return TCallbackReturn
      */
     public function usingConnection($name, callable $callback) {}
 }