CI

.github/workflows/build.yml

@@ -0,0 +1,32 @@
+name: Build
+
+on:
+  push:
+    branches:
+      - main
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: latest
+
+      - name: Install NPM dependencies
+        run: npm ci
+
+      - name: Compile assets
+        run: npm run build
+
+      - name: Commit compiled files
+        uses: stefanzweifel/git-auto-commit-action@v5
+        with:
+          commit_message: Compile Assets
+          file_pattern: dist/

.github/workflows/main.yml

@@ -0,0 +1,99 @@
+name: CI
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  checks:
+    runs-on: ubuntu-latest
+    name: 'Check'
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Get composer cache directory
+        id: composer-cache
+        run: echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT
+
+      - name: Cache dependencies
+        uses: actions/cache@v4
+        with:
+          path: ${{ steps.composer-cache.outputs.dir }}
+          key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.json') }}
+          restore-keys: ${{ runner.os }}-composer-
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.3'
+          coverage: pcov
+          tools: infection, pint, phpstan
+
+      - name: Install dependencies
+        run: composer install
+
+      - name: Check platform requirements
+        run: composer check-platform-reqs
+
+      - name: Pint
+        run: pint --test
+
+      # Pest does not support Infection. Might need to migrate to PHPUnit.
+      - name: Infection
+        run: infection --show-mutations
+
+      - name: PHPStan
+        run: phpstan -v
+
+  tests:
+    runs-on: ubuntu-latest
+    name: 'PHP ${{ matrix.php }} Testbench ${{ matrix.testbench }} PHPUnit ${{ matrix.phpunit }}'
+    strategy:
+      matrix:
+        php: ['8.1', '8.2', '8.3']
+        include:
+          - php: '8.1'
+            testbench: '8.0'
+            phpunit: '10'
+          - php: '8.2'
+            testbench: '9.0'
+            phpunit: '11'
+          - php: '8.3'
+            testbench: '9.0'
+            phpunit: '11'
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Get composer cache directory
+        id: composer-cache
+        run: echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT
+
+      - name: Cache dependencies
+        uses: actions/cache@v4
+        with:
+          path: ${{ steps.composer-cache.outputs.dir }}
+          key: ${{ runner.os }}-php-${{ matrix.php }}-testbench-${{ matrix.testbench }}-composer-${{ hashFiles('**/composer.json') }}
+          restore-keys: ${{ runner.os }}-php-${{ matrix.php }}-testbench-${{ matrix.testbench }}-composer-
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          coverage: none
+
+      - name: Install dependencies
+        run: |
+          composer require --no-update --dev \
+            orchestra/testbench:^${{ matrix.testbench }}
+          composer update
+
+      - name: Configure PHPUnit
+        run: "if [ -f './phpunit.${{ matrix.phpunit }}.xml' ]; then cp ./phpunit.${{ matrix.phpunit }}.xml ./phpunit.xml; fi"
+
+      - name: PHPUnit
+        run: ./vendor/bin/phpunit --do-not-cache-result
+

.gitignore

@@ -1,3 +1,4 @@
 /composer.lock
 /node_modules
 /vendor
+/.phpunit.cache

composer.json

@@ -21,23 +21,12 @@
     },
     "require-dev": {
         "inertiajs/inertia-laravel": "^1.0",
-        "orchestra/pest-plugin-testbench": "^2.0",
         "orchestra/testbench": "^9.0",
-        "pestphp/pest": "^2.0"
-    },
-    "extra": {
-        "laravel": {
-            "providers": [
-                "TiMacDonald\\Pulse\\ValidationErrorsServiceProvider"
-            ]
-        }
+        "phpunit/phpunit": "^10.0 || ^11.0"
     },
     "config": {
         "preferred-install": "dist",
-        "sort-packages": true,
-        "allow-plugins": {
-            "pestphp/pest-plugin": true
-        }
+        "sort-packages": true
     },
     "autoload": {
         "psr-4": {
@@ -54,6 +43,13 @@
     },
     "minimum-stability": "stable",
     "prefer-stable": true,
+    "extra": {
+        "laravel": {
+            "providers": [
+                "TiMacDonald\\Pulse\\ValidationErrorsServiceProvider"
+            ]
+        }
+    },
     "scripts": {
         "post-autoload-dump": [
             "@php vendor/bin/testbench package:purge-skeleton --ansi",
@@ -64,10 +60,5 @@
             "@php vendor/bin/testbench workbench:build --ansi",
             "@php vendor/bin/testbench serve"
         ]
-    },
-    "support": {
-        "issues": "https://github.com/timacdonald/pulse-validation-errors/issues",
-        "source": "https://github.com/timacdonald/pulse-validation-errors/releases/latest",
-        "docs": "https://github.com/timacdonald/pulse-validation-errors/blob/master/readme.md"
     }
 }

infection.json5

@@ -0,0 +1,13 @@
+{
+    "$schema": "vendor/infection/infection/resources/schema.json",
+    "source": {
+        "directories": [
+            "src"
+        ]
+    },
+    "mutators": {
+        "@default": true,
+        "@function_signature": false
+    },
+    "minMsi": 100
+}

phpstan.neon

@@ -0,0 +1,4 @@
+parameters:
+    level: max
+    paths:
+        - src

phpunit.10.xml

@@ -0,0 +1,26 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd"
+         bootstrap="vendor/autoload.php"
+         cacheDirectory=".phpunit.cache"
+         executionOrder="depends,defects"
+         beStrictAboutOutputDuringTests="true"
+         failOnNotice="true"
+         failOnRisky="true"
+         failOnWarning="true">
+    <testsuites>
+        <testsuite name="default">
+            <directory>tests</directory>
+        </testsuite>
+    </testsuites>
+    <source restrictDeprecations="true" restrictNotices="true" restrictWarnings="true">
+        <include>
+            <directory>src</directory>
+        </include>
+    </source>
+    <php>
+        <env name="APP_KEY" value="base64:vFI2dJxJeGFdbe7JXMJkE6iIv0dKBCNBw/9hZJVF/UM="/>
+        <env name="DB_CONNECTION" value="testing"/>
+        <env name="PULSE_CACHE_DRIVER" value="array"/>
+    </php>
+</phpunit>

phpunit.xml

@@ -1,14 +1,26 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit colors="true">
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/11.1/phpunit.xsd"
+         bootstrap="vendor/autoload.php"
+         cacheDirectory=".phpunit.cache"
+         executionOrder="depends,defects"
+         beStrictAboutOutputDuringTests="true"
+         failOnNotice="true"
+         failOnRisky="true"
+         failOnWarning="true">
     <testsuites>
-        <testsuite name="testsuite">
-            <directory suffix="Test.php">./tests/</directory>
+        <testsuite name="default">
+            <directory>tests</directory>
         </testsuite>
     </testsuites>
+    <source ignoreIndirectDeprecations="true" restrictNotices="true" restrictWarnings="true">
+        <include>
+            <directory>src</directory>
+        </include>
+    </source>
     <php>
         <env name="APP_KEY" value="base64:vFI2dJxJeGFdbe7JXMJkE6iIv0dKBCNBw/9hZJVF/UM="/>
         <env name="DB_CONNECTION" value="testing"/>
         <env name="PULSE_CACHE_DRIVER" value="array"/>
     </php>
 </phpunit>
-

resources/views/validation-errors.blade.php

@@ -44,7 +44,7 @@
                                 </p>
                             </x-pulse::td>
                             <x-pulse::td numeric class="text-gray-700 dark:text-gray-300 font-bold w-32">
-                                @if ($config['sample_rate'] < 1)
+                                @if ($config['sample_rate'] !== 1)
                                     <span title="Sample rate: {{ $config['sample_rate'] }}, Raw value: {{ number_format($error->count) }}">~{{ number_format($error->count * (1 / $config['sample_rate'])) }}</span>
                                 @else
                                     {{ number_format($error->count) }}

src/Cards/ValidationErrors.php

@@ -30,7 +30,8 @@ public function render(): Renderable
                 'validation_error',
                 ['count'],
                 $this->periodAsInterval(),
-            )->map(function ($row) {
+            )->map(function (object $row) { // @phpstan-ignore argument.type
+                /** @var object{ key: string, count: int }  $row */
                 [$method, $uri, $action, $bag, $name, $message] = json_decode($row->key, flags: JSON_THROW_ON_ERROR) + [5 => null];
 
                 return (object) [
@@ -46,19 +47,15 @@ public function render(): Renderable
                     'count' => $row->count,
                     'key_hash' => md5($row->key),
                 ];
-            }),
-        );
+            }));
 
         return View::make('timacdonald::validation-errors', [
             'time' => $time,
             'runAt' => $runAt,
             'errors' => $errors,
             'config' => [
-                'enabled' => true,
                 'sample_rate' => 1,
-                'capture_messages' => true,
-                'ignore' => [],
-                ...Config::get('pulse.recorders.'.ValidationErrorsRecorder::class, []),
+                ...Config::get('pulse.recorders.'.ValidationErrorsRecorder::class, []), // @phpstan-ignore arrayUnpacking.nonIterable
             ],
         ]);
     }