[1.x] Fix persistent middleware with arguments (#283)

* Ensure arguments are not provided to persistent middleware

* Fix code styling

---------

Co-authored-by: timacdonald <timacdonald@users.noreply.github.com>

Author: timacdonald

src/PulseServiceProvider.php

@@ -14,6 +14,7 @@
 use Illuminate\Queue\Events\WorkerStopping;
 use Illuminate\Routing\Router;
 use Illuminate\Support\ServiceProvider;
+use Illuminate\Support\Str;
 use Illuminate\View\Compilers\BladeCompiler;
 use Illuminate\View\Factory as ViewFactory;
 use Laravel\Pulse\Contracts\Ingest;
@@ -167,7 +168,13 @@ protected function registerComponents(): void
         });
 
         $this->callAfterResolving('livewire', function (LivewireManager $livewire, Application $app) {
-            $livewire->addPersistentMiddleware($app->make('config')->get('pulse.middleware', []));
+            $middleware = collect($app->make('config')->get('pulse.middleware'))
+                ->map(fn ($middleware) => is_string($middleware)
+                    ? Str::before($middleware, ':')
+                    : $middleware)
+                ->all();
+
+            $livewire->addPersistentMiddleware($middleware);
 
             $livewire->component('pulse.cache', Livewire\Cache::class);
             $livewire->component('pulse.usage', Livewire\Usage::class);

tests/Feature/PulseTest.php

@@ -5,11 +5,13 @@
 use Illuminate\Support\Facades\App;
 use Illuminate\Support\Facades\Auth;
 use Illuminate\Support\Facades\Config;
+use Illuminate\Support\Facades\Facade;
 use Laravel\Pulse\Contracts\ResolvesUsers;
 use Laravel\Pulse\Contracts\Storage;
 use Laravel\Pulse\Entry;
 use Laravel\Pulse\Facades\Pulse;
 use Laravel\Pulse\Value;
+use Livewire\LivewireManager;
 use Tests\StorageFake;
 use Tests\User;
 
@@ -221,3 +223,19 @@ public function find(int|string|null $key): object
 
     expect($handled)->toBe(true);
 });
+
+it('strips arguments from persistent middleware', function () {
+    App::forgetInstance(LivewireManager::class);
+    Facade::clearResolvedInstance('livewire');
+    Config::set('pulse.middleware', [MyTestMiddleware::class.':admin']);
+
+    $persistentMiddleware = Livewire::getPersistentMiddleware();
+
+    expect($persistentMiddleware)->toContain(MyTestMiddleware::class);
+    expect($persistentMiddleware)->not->toContain(MyTestMiddleware::class.':admin');
+});
+
+class MyTestMiddleware
+{
+    //
+}