feat: phpstan level 5

.github/workflows/phpstan.yml

@@ -1,26 +1,42 @@
 name: PHPStan
 
 on:
+  workflow_dispatch:
   push:
-    paths:
-      - '**.php'
-      - 'phpstan.neon.dist'
+    branches-ignore:
+      - 'dependabot/npm_and_yarn/*'
 
 jobs:
   phpstan:
-    name: phpstan
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest]
+        php: [8.3]
+
     steps:
-      - uses: actions/checkout@v4
+
+      - name: Checkout code
+        uses: actions/checkout@v4
 
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '8.1'
-          coverage: none
+          php-version: ${{ matrix.php }}
+
+      - name: Get composer cache directory
+        id: composer-cache
+        run: echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT
+
+      - name: Cache Composer dependencies
+        uses: actions/cache@v4
+        with:
+          path: ${{ steps.composer-cache.outputs.dir }}
+          key: ${{ runner.os }}-${{ matrix.php }}-composer-${{ hashFiles('**/composer.lock') }}
+          restore-keys: ${{ runner.os }}-${{ matrix.php }}-composer-
 
-      - name: Install composer dependencies
-        uses: ramsey/composer-install@v3
+      - name: Install Dependencies
+        run: composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
 
-      - name: Run PHPStan
-        run: ./vendor/bin/phpstan --error-format=github
+      - name: Run analysis
+        run: ./vendor/bin/phpstan analyse --error-format=github

.gitignore

@@ -5,7 +5,6 @@ composer.lock
 coverage
 docs
 phpunit.xml
-phpstan.neon
 testbench.yaml
 vendor
 node_modules

composer.json

@@ -38,9 +38,9 @@
     },
     "require-dev": {
         "guzzlehttp/guzzle": "^7.0",
+        "larastan/larastan": "^2.0|^3.0",
         "laravel/pint": "^1.0",
         "nunomaduro/collision": "^7.9",
-        "nunomaduro/larastan": "^2.0.1",
         "orchestra/testbench": "^8.0",
         "pestphp/pest": "^2.0",
         "pestphp/pest-plugin-arch": "^2.0",
@@ -52,8 +52,7 @@
     },
     "autoload": {
         "psr-4": {
-            "Native\\Laravel\\": "src/",
-            "Native\\Laravel\\Database\\Factories\\": "database/factories/"
+            "Native\\Laravel\\": "src/"
         }
     },
     "autoload-dev": {
@@ -62,6 +61,11 @@
         }
     },
     "scripts": {
+        "qa" : [
+            "@composer format",
+            "@composer analyse",
+            "@composer test"
+        ],
         "post-autoload-dump": "@php ./vendor/bin/testbench package:discover --ansi",
         "analyse": "vendor/bin/phpstan analyse",
         "test": "vendor/bin/pest",

phpstan.neon

@@ -0,0 +1,17 @@
+parameters:
+
+    paths:
+        - src/
+        - config/
+#        - tests/
+
+
+    # Level 9 is the highest level
+    level: 5
+
+    ignoreErrors:
+       - '#Class App\\Providers\\NativeAppServiceProvider not found#'
+#        - '#PHPDoc tag @var#'
+#
+#    excludePaths:
+#        - ./*/*/FileToBeExcluded.php

src/ChildProcess.php

@@ -7,19 +7,19 @@
 
 class ChildProcess implements ChildProcessContract
 {
-    public readonly int $pid;
+    public int $pid;
 
-    public readonly string $alias;
+    public string $alias;
 
-    public readonly array $cmd;
+    public array $cmd;
 
-    public readonly ?string $cwd;
+    public ?string $cwd;
 
-    public readonly ?array $env;
+    public ?array $env;
 
-    public readonly bool $persistent;
+    public bool $persistent;
 
-    public function __construct(protected Client $client) {}
+    final public function __construct(protected Client $client) {}
 
     public function get(?string $alias = null): ?static
     {
@@ -54,7 +54,6 @@ public function all(): array
 
     /**
      * @param  string|string[]  $cmd
-     * @return $this
      */
     public function start(
         string|array $cmd,
@@ -78,16 +77,15 @@ public function start(
 
     /**
      * @param  string|string[]  $cmd
-     * @return $this
      */
-    public function php(string|array $cmd, string $alias, ?array $env = null, ?bool $persistent = false): self
+    public function php(string|array $cmd, string $alias, ?array $env = null, ?bool $persistent = false): static
     {
         $cmd = is_array($cmd) ? array_values($cmd) : [$cmd];
 
         $process = $this->client->post('child-process/start-php', [
             'alias' => $alias,
             'cmd' => $cmd,
-            'cwd' => $cwd ?? base_path(),
+            'cwd' => base_path(),
             'env' => $env,
             'persistent' => $persistent,
         ])->json();
@@ -97,9 +95,8 @@ public function php(string|array $cmd, string $alias, ?array $env = null, ?bool
 
     /**
      * @param  string|string[]  $cmd
-     * @return $this
      */
-    public function artisan(string|array $cmd, string $alias, ?array $env = null, ?bool $persistent = false): self
+    public function artisan(string|array $cmd, string $alias, ?array $env = null, ?bool $persistent = false): static
     {
         $cmd = is_array($cmd) ? array_values($cmd) : [$cmd];
 

src/Client/Client.php

@@ -21,9 +21,9 @@ public function __construct()
             ->asJson();
     }
 
-    public function get(string $endpoint): Response
+    public function get(string $endpoint, array|string|null $query = null): Response
     {
-        return $this->client->get($endpoint);
+        return $this->client->get($endpoint, $query);
     }
 
     public function post(string $endpoint, array $data = []): Response

src/Commands/SeedDatabaseCommand.php

@@ -15,6 +15,6 @@ public function handle()
     {
         (new NativeServiceProvider($this->laravel))->rewriteDatabase();
 
-        parent::handle();
+        return parent::handle();
     }
 }

src/Compactor/Php.php

@@ -3,6 +3,8 @@
 namespace Native\Laravel\Compactor;
 
 use PhpToken;
+use RuntimeException;
+use Webmozart\Assert\Assert;
 
 class Php
 {
@@ -17,7 +19,7 @@ public function compact(string $file, string $contents): string
             return $this->compactContent($contents);
         }
 
-        $this->compactContent($contents);
+        return $this->compactContent($contents);
     }
 
     protected function compactContent(string $contents): string
@@ -145,7 +147,6 @@ private function retokenizeAttribute(array &$tokens, int $opener): ?array
     {
         Assert::keyExists($tokens, $opener);
 
-        /** @var PhpToken $token */
         $token = $tokens[$opener];
         $attributeBody = mb_substr($token->text, 2);
         $subTokens = PhpToken::tokenize('<?php '.$attributeBody);

src/Dialog.php

@@ -26,7 +26,7 @@ class Dialog
 
     protected $windowReference;
 
-    public function __construct(protected Client $client) {}
+    final public function __construct(protected Client $client) {}
 
     public static function new()
     {

src/Dock.php

@@ -43,12 +43,14 @@ public function cancelBounce()
         $this->client->post('dock/cancel-bounce');
     }
 
-    public function badge(?string $label = null): void|string
+    public function badge(?string $label = null): ?string
     {
         if (is_null($label)) {
             return $this->client->get('dock/badge');
         }
 
         $this->client->post('dock/badge', ['label' => $label]);
+
+        return null;
     }
 }

src/Facades/Menu.php

@@ -3,6 +3,7 @@
 namespace Native\Laravel\Facades;
 
 use Illuminate\Support\Facades\Facade;
+use Native\Laravel\Contracts\MenuItem;
 use Native\Laravel\Menu\Items\Checkbox;
 use Native\Laravel\Menu\Items\Label;
 use Native\Laravel\Menu\Items\Link;
@@ -11,7 +12,7 @@
 use Native\Laravel\Menu\Items\Separator;
 
 /**
- * @method static \Native\Laravel\Menu\Menu make(\Native\Laravel\Menu\Items\MenuItem ...$items)
+ * @method static \Native\Laravel\Menu\Menu make(MenuItem ...$items)
  * @method static Checkbox checkbox(string $label, bool $checked = false, ?string $hotkey = null)
  * @method static Label label(string $label)
  * @method static Link link(string $url, string $label = null, ?string $hotkey = null)
@@ -23,7 +24,6 @@
  * @method static Role view()
  * @method static Role window()
  * @method static Role help()
- * @method static Role window()
  * @method static Role fullscreen()
  * @method static Role separator()
  * @method static Role devTools()
@@ -37,7 +37,6 @@
  * @method static Role minimize()
  * @method static Role close()
  * @method static Role quit()
- * @method static Role help()
  * @method static Role hide()
  * @method static void create(MenuItem ...$items)
  * @method static void default()

src/NativeServiceProvider.php

@@ -47,6 +47,7 @@ public function packageRegistered()
         $this->mergeConfigFrom($this->package->basePath('/../config/nativephp-internal.php'), 'nativephp-internal');
 
         $this->app->singleton(FreshCommand::class, function ($app) {
+            /* @phpstan-ignore-next-line (beacause we support Laravel 10 & 11) */
             return new FreshCommand($app['migrator']);
         });
 
@@ -137,13 +138,15 @@ public function rewriteDatabase()
             }
         }
 
-        config(['database.connections.nativephp' => [
-            'driver' => 'sqlite',
-            'url' => env('DATABASE_URL'),
-            'database' => $databasePath,
-            'prefix' => '',
-            'foreign_key_constraints' => env('DB_FOREIGN_KEYS', true),
-        ]]);
+        config([
+            'database.connections.nativephp' => [
+                'driver' => 'sqlite',
+                'url' => env('DATABASE_URL'),
+                'database' => $databasePath,
+                'prefix' => '',
+                'foreign_key_constraints' => env('DB_FOREIGN_KEYS', true),
+            ],
+        ]);
 
         config(['database.default' => 'nativephp']);
 
@@ -163,7 +166,7 @@ public function removeDatabase()
 
         @unlink($databasePath);
         @unlink($databasePath.'-shm');
-        @unlink($database.'-wal');
+        @unlink($databasePath.'-wal');
     }
 
     protected function configureDisks(): void
@@ -186,12 +189,14 @@ protected function configureDisks(): void
                 continue;
             }
 
-            config(['filesystems.disks.'.$disk => [
-                'driver' => 'local',
-                'root' => env($env, ''),
-                'throw' => false,
-                'links' => 'skip',
-            ]]);
+            config([
+                'filesystems.disks.'.$disk => [
+                    'driver' => 'local',
+                    'root' => env($env, ''),
+                    'throw' => false,
+                    'links' => 'skip',
+                ],
+            ]);
         }
     }
 }

src/Notification.php

@@ -12,7 +12,7 @@ class Notification
 
     protected string $event = '';
 
-    public function __construct(protected Client $client) {}
+    final public function __construct(protected Client $client) {}
 
     public static function new()
     {

src/ProgressBar.php

@@ -16,7 +16,7 @@ class ProgressBar
 
     protected float $maxSecondsBetweenRedraws = 1;
 
-    public function __construct(protected int $maxSteps, protected Client $client) {}
+    final public function __construct(protected int $maxSteps, protected Client $client) {}
 
     public static function create(int $maxSteps): static
     {