Fix deps, Filament v3-v5, Laravel 13, CI (#2)

dp88 · claude · web-flow · commit fa3bacff1204 · 2026-03-30T11:43:01.000-04:00
* Fix broken phpinfo dependency and add Laravel 13 support Update stechstudio/phpinfo constraint from ^0.4.0 (removed tags) to ^0.6|^1.0. Broaden filament/filament to ^3.0|^4.0|^5.0 for Laravel 13 support via Filament v4/v5. Add explicit PHP ^8.3 requirement and declare spatie/laravel-package-tools as a direct dependency. Remove typed property/return on $info and getInfo() since phpinfo v1.0 renames Result to PhpInfo — the duck-typed API is identical across both versions. Closes #1 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Update PHPInfo page for Filament v5 compatibility Change $view from static to non-static to match Filament v5's Page base class. Override getDefaultSlug() instead of getSlug() (which now takes a ?Panel parameter). Update getNavigationGroup() and getNavigationIcon() return types to match v5 signatures (UnitEnum, BackedEnum, Htmlable unions). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Add Pest test infrastructure with Orchestra Testbench Add test suite covering plugin registration, page configuration defaults and overrides, and service provider config/view registration. Uses Pest 3 with Orchestra Testbench for Laravel package testing. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Make PHPInfo page compatible with Filament v3, v4, and v5 Use getView() method override instead of $view property declaration to avoid the static vs non-static incompatibility between v3 and v4/v5. Move navigation group/icon config into service provider boot using the setter methods, avoiding return type signature differences across versions. Override getDefaultSlug() for config-driven slug (exists in all versions). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Add GitHub Actions CI workflow Test matrix covers PHP 8.3/8.4 against Filament v3 (Laravel 11), v4 (Laravel 12), and v5 (Laravel 12) using Orchestra Testbench. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Update README with CI badge and compatibility table Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Fix navigation config for Filament v3 compatibility Move navigation group/icon resolution from service provider setter methods (which don't exist in Filament v3) to getter overrides on the Page class. Uses covariant return types (?string) which are compatible with both v3's ?string and v5's string|UnitEnum|null parent signatures. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Add Laravel 13 to CI matrix and README Add Filament v5 + Laravel 13 (Testbench 11) test entries for both PHP 8.3 and 8.4. Update compatibility table to list Laravel 13. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Fix Laravel 13 CI by conditionally installing pest-plugin-laravel Remove pestphp/pest-plugin-laravel from require-dev since it doesn't support Laravel 13 yet. Install it conditionally in CI only for Laravel 11/12 lanes. Tests don't depend on its helpers. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Add phpinfo capture integration tests Verify that Info::capture() returns real phpinfo data: modules are present, PHP version is parseable, the Core module exists, and known configs like display_errors are found. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -0,0 +1,80 @@
+name: Tests
+
+on:
+  push:
+    paths:
+      - '**.php'
+      - 'composer.json'
+      - 'phpunit.xml.dist'
+      - '.github/workflows/tests.yml'
+  pull_request:
+    paths:
+      - '**.php'
+      - 'composer.json'
+      - 'phpunit.xml.dist'
+      - '.github/workflows/tests.yml'
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    timeout-minutes: 5
+
+    strategy:
+      fail-fast: true
+      matrix:
+        include:
+          - php: '8.3'
+            filament: '^3.0'
+            laravel: '11.*'
+            testbench: '9.*'
+          - php: '8.4'
+            filament: '^3.0'
+            laravel: '11.*'
+            testbench: '9.*'
+          - php: '8.3'
+            filament: '^4.0'
+            laravel: '12.*'
+            testbench: '10.*'
+          - php: '8.4'
+            filament: '^4.0'
+            laravel: '12.*'
+            testbench: '10.*'
+          - php: '8.3'
+            filament: '^5.0'
+            laravel: '12.*'
+            testbench: '10.*'
+          - php: '8.4'
+            filament: '^5.0'
+            laravel: '12.*'
+            testbench: '10.*'
+          - php: '8.3'
+            filament: '^5.0'
+            laravel: '13.*'
+            testbench: '11.*'
+          - php: '8.4'
+            filament: '^5.0'
+            laravel: '13.*'
+            testbench: '11.*'
+
+    name: PHP ${{ matrix.php }} - Filament ${{ matrix.filament }} - Laravel ${{ matrix.laravel }}
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          extensions: dom
+          coverage: none
+
+      - name: Install dependencies
+        run: |
+          composer require "filament/filament:${{ matrix.filament }}" "laravel/framework:${{ matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" --no-interaction --no-update
+          if [[ "${{ matrix.laravel }}" != "13.*" ]]; then
+            composer require "pestphp/pest-plugin-laravel:^3.0" --no-interaction --no-update
+          fi
+          composer update --prefer-stable --prefer-dist --no-interaction
+
+      - name: Run tests
+        run: vendor/bin/pest --ci
diff --git a/.gitignore b/.gitignore
@@ -25,3 +25,4 @@ Homestead.yaml
 Homestead.json
 /.vagrant
 .phpunit.result.cache
+.phpunit.cache/
diff --git a/README.md b/README.md
@@ -1,6 +1,15 @@
 # PHPInfo for Filament
+
+![Tests](https://github.com/stechstudio/filament-phpinfo/actions/workflows/tests.yml/badge.svg)
+
 This package adds a new page to the Filament admin panel that displays the output of `phpinfo()` in a nicely formatted way.
 
+## Compatibility
+
+| Version | PHP | Laravel | Filament |
+|---------|-----|---------|----------|
+| 1.2     | 8.3+ | 11, 12, 13 | 3, 4, 5 |
+
 ## Installation
 ```bash
 composer require stechstudio/filament-phpinfo
diff --git a/composer.json b/composer.json
@@ -19,8 +19,27 @@
         }
     ],
     "require": {
-        "filament/filament": "^3.0",
-        "stechstudio/phpinfo": "^0.4.0"
+        "php": "^8.3",
+        "filament/filament": "^3.0|^4.0|^5.0",
+        "spatie/laravel-package-tools": "^1.9",
+        "stechstudio/phpinfo": "^0.6|^1.0"
+    },
+    "autoload-dev": {
+        "psr-4": {
+            "STS\\FilamentPHPInfo\\Tests\\": "tests/"
+        }
+    },
+    "require-dev": {
+        "orchestra/testbench": "^8.0|^9.0|^10.0|^11.0",
+        "pestphp/pest": "^2.0|^3.0"
+    },
+    "scripts": {
+        "test": "vendor/bin/pest"
+    },
+    "config": {
+        "allow-plugins": {
+            "pestphp/pest-plugin": true
+        }
     },
     "extra": {
         "laravel": {
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
+    bootstrap="vendor/autoload.php"
+    colors="true"
+>
+    <testsuites>
+        <testsuite name="Tests">
+            <directory>tests</directory>
+        </testsuite>
+    </testsuites>
+</phpunit>
diff --git a/src/FilamentPHPInfoServiceProvider.php b/src/FilamentPHPInfoServiceProvider.php
@@ -16,4 +16,5 @@ public function configurePackage(Package $package): void
             ->hasViews()
             ->hasConfigFile();
     }
+
 }
diff --git a/src/Pages/PHPInfo.php b/src/Pages/PHPInfo.php
@@ -9,11 +9,14 @@ class PHPInfo extends Page
 {
     protected static ?string $title = 'PHPInfo';
 
-    protected static string $view = 'filament-phpinfo::phpinfo';
-
     protected static ?string $navigationLabel = 'PHPInfo';
 
-    protected InfoWrapper\Result $info;
+    protected mixed $info;
+
+    public function getView(): string
+    {
+        return 'filament-phpinfo::phpinfo';
+    }
 
     public function getViewData(): array
     {
@@ -22,23 +25,23 @@ public function getViewData(): array
         ];
     }
 
-    protected function getInfo(): InfoWrapper\Result
+    protected function getInfo(): mixed
     {
         return $this->info ??= InfoWrapper\Info::capture();
     }
 
-    public static function getSlug(): string
+    public static function getDefaultSlug(): string
     {
         return config('filament-phpinfo.page-slug', 'phpinfo');
     }
 
-    public static function getNavigationGroup(): ?string
+    public static function getNavigationIcon(): ?string
     {
-        return config('filament-phpinfo.navigation-group');
+        return config('filament-phpinfo.navigation-icon', 'heroicon-o-information-circle');
     }
 
-    public static function getNavigationIcon(): ?string
+    public static function getNavigationGroup(): ?string
     {
-        return config('filament-phpinfo.navigation-icon', 'heroicon-o-information-circle');
+        return config('filament-phpinfo.navigation-group');
     }
 }
diff --git a/tests/PageTest.php b/tests/PageTest.php
@@ -0,0 +1,40 @@
+<?php
+
+use STS\FilamentPHPInfo\Pages\PHPInfo;
+
+it('has the correct default slug', function () {
+    expect(PHPInfo::getDefaultSlug())->toBe('phpinfo');
+});
+
+it('has the correct default navigation icon', function () {
+    expect(PHPInfo::getNavigationIcon())->toBe('heroicon-o-information-circle');
+});
+
+it('has the correct default navigation group', function () {
+    expect(PHPInfo::getNavigationGroup())->toBe('System Management');
+});
+
+it('respects custom slug from config', function () {
+    config()->set('filament-phpinfo.page-slug', 'custom-phpinfo');
+    expect(PHPInfo::getDefaultSlug())->toBe('custom-phpinfo');
+});
+
+it('respects custom navigation icon from config', function () {
+    config()->set('filament-phpinfo.navigation-icon', 'heroicon-o-cog');
+    expect(PHPInfo::getNavigationIcon())->toBe('heroicon-o-cog');
+});
+
+it('respects custom navigation group from config', function () {
+    config()->set('filament-phpinfo.navigation-group', 'Custom Group');
+    expect(PHPInfo::getNavigationGroup())->toBe('Custom Group');
+});
+
+it('returns the correct view', function () {
+    $page = new class extends PHPInfo {
+        public function exposeGetView(): string
+        {
+            return $this->getView();
+        }
+    };
+    expect($page->exposeGetView())->toBe('filament-phpinfo::phpinfo');
+});
diff --git a/tests/Pest.php b/tests/Pest.php
@@ -0,0 +1,5 @@
+<?php
+
+use STS\FilamentPHPInfo\Tests\TestCase;
+
+uses(TestCase::class)->in(__DIR__);
diff --git a/tests/PhpInfoCaptureTest.php b/tests/PhpInfoCaptureTest.php
@@ -0,0 +1,29 @@
+<?php
+
+use STS\Phpinfo\Info;
+
+it('captures phpinfo and returns modules', function () {
+    $info = Info::capture();
+
+    expect($info->modules())->not->toBeEmpty();
+});
+
+it('contains a PHP version', function () {
+    $info = Info::capture();
+
+    expect($info->version())->toMatch('/^\d+\.\d+/');
+});
+
+it('includes the Core module', function () {
+    $info = Info::capture();
+
+    expect($info->hasModule('Core'))->toBeTrue();
+    expect($info->module('Core')->name())->toBe('Core');
+});
+
+it('has configs with names and values', function () {
+    $info = Info::capture();
+
+    expect($info->configs())->not->toBeEmpty();
+    expect($info->hasConfig('display_errors'))->toBeTrue();
+});
diff --git a/tests/PluginTest.php b/tests/PluginTest.php
@@ -0,0 +1,13 @@
+<?php
+
+use STS\FilamentPHPInfo\FilamentPHPInfoPlugin;
+
+it('can be instantiated via make()', function () {
+    $plugin = FilamentPHPInfoPlugin::make();
+    expect($plugin)->toBeInstanceOf(FilamentPHPInfoPlugin::class);
+});
+
+it('has the correct plugin ID', function () {
+    $plugin = FilamentPHPInfoPlugin::make();
+    expect($plugin->getId())->toBe('filament-phpinfo');
+});
diff --git a/tests/ServiceProviderTest.php b/tests/ServiceProviderTest.php
@@ -0,0 +1,12 @@
+<?php
+
+it('registers the config file', function () {
+    $config = config('filament-phpinfo');
+    expect($config)->toBeArray()
+        ->and($config)->toHaveKeys(['navigation-group', 'navigation-icon', 'page-slug']);
+});
+
+it('registers the views', function () {
+    $viewFactory = app('view');
+    expect($viewFactory->exists('filament-phpinfo::phpinfo'))->toBeTrue();
+});
diff --git a/tests/TestCase.php b/tests/TestCase.php
@@ -0,0 +1,16 @@
+<?php
+
+namespace STS\FilamentPHPInfo\Tests;
+
+use Orchestra\Testbench\TestCase as Orchestra;
+use STS\FilamentPHPInfo\FilamentPHPInfoServiceProvider;
+
+abstract class TestCase extends Orchestra
+{
+    protected function getPackageProviders($app): array
+    {
+        return [
+            FilamentPHPInfoServiceProvider::class,
+        ];
+    }
+}

Original file line number	Diff line number	Diff line change
`@@ -16,4 +16,5 @@ public function configurePackage(Package $package): void`
`16`	`16`	`->hasViews()`
`17`	`17`	`->hasConfigFile();`
`18`	`18`	`}`
	`19`	`+`
`19`	`20`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,11 +9,14 @@ class PHPInfo extends Page`
`9`	`9`	`{`
`10`	`10`	`protected static ?string $title = 'PHPInfo';`
`11`	`11`
`12`		`- protected static string $view = 'filament-phpinfo::phpinfo';`
`13`		`-`
`14`	`12`	`protected static ?string $navigationLabel = 'PHPInfo';`
`15`	`13`
`16`		`- protected InfoWrapper\Result $info;`
	`14`	`+ protected mixed $info;`
	`15`	`+`
	`16`	`+ public function getView(): string`
	`17`	`+ {`
	`18`	`+ return 'filament-phpinfo::phpinfo';`
	`19`	`+ }`
`17`	`20`
`18`	`21`	`public function getViewData(): array`
`19`	`22`	`{`
`@@ -22,23 +25,23 @@ public function getViewData(): array`
`22`	`25`	`];`
`23`	`26`	`}`
`24`	`27`
`25`		`- protected function getInfo(): InfoWrapper\Result`
	`28`	`+ protected function getInfo(): mixed`
`26`	`29`	`{`
`27`	`30`	`return $this->info ??= InfoWrapper\Info::capture();`
`28`	`31`	`}`
`29`	`32`
`30`		`- public static function getSlug(): string`
	`33`	`+ public static function getDefaultSlug(): string`
`31`	`34`	`{`
`32`	`35`	`return config('filament-phpinfo.page-slug', 'phpinfo');`
`33`	`36`	`}`
`34`	`37`
`35`		`- public static function getNavigationGroup(): ?string`
	`38`	`+ public static function getNavigationIcon(): ?string`
`36`	`39`	`{`
`37`		`- return config('filament-phpinfo.navigation-group');`
	`40`	`+ return config('filament-phpinfo.navigation-icon', 'heroicon-o-information-circle');`
`38`	`41`	`}`
`39`	`42`
`40`		`- public static function getNavigationIcon(): ?string`
	`43`	`+ public static function getNavigationGroup(): ?string`
`41`	`44`	`{`
`42`		`- return config('filament-phpinfo.navigation-icon', 'heroicon-o-information-circle');`
	`45`	`+ return config('filament-phpinfo.navigation-group');`
`43`	`46`	`}`
`44`	`47`	`}`
-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +<?php
++
 +use STS\FilamentPHPInfo\Tests\TestCase;
++
 +uses(TestCase::class)->in(__DIR__);