Flush Vite per-request state (#1016)

Author: timacdonald

src/Concerns/ProvidesDefaultConfigurationOptions.php

@@ -48,11 +48,12 @@ public static function prepareApplicationForNextOperation(): array
             \Laravel\Octane\Listeners\FlushDatabaseRecordModificationState::class,
             \Laravel\Octane\Listeners\FlushDatabaseQueryLog::class,
             \Laravel\Octane\Listeners\RefreshQueryDurationHandling::class,
-            \Laravel\Octane\Listeners\FlushLogContext::class,
             \Laravel\Octane\Listeners\FlushArrayCache::class,
+            \Laravel\Octane\Listeners\FlushLogContext::class,
             \Laravel\Octane\Listeners\FlushMonologState::class,
             \Laravel\Octane\Listeners\FlushStrCache::class,
             \Laravel\Octane\Listeners\FlushTranslatorCache::class,
+            \Laravel\Octane\Listeners\FlushVite::class,
 
             // First-Party Packages...
             \Laravel\Octane\Listeners\PrepareInertiaForNextOperation::class,

src/Listeners/FlushVite.php

@@ -0,0 +1,27 @@
+<?php
+
+namespace Laravel\Octane\Listeners;
+
+use Illuminate\Foundation\Vite;
+
+class FlushVite
+{
+    /**
+     * Handle the event.
+     *
+     * @param  mixed  $event
+     * @return void
+     */
+    public function handle($event)
+    {
+        if (! $event->sandbox->resolved(Vite::class)) {
+            return;
+        }
+
+        $vite = $event->sandbox->make(Vite::class);
+
+        if (method_exists($vite, 'flush')) {
+            $vite->flush();
+        }
+    }
+}

tests/Listeners/FlushViteTest.php

@@ -0,0 +1,51 @@
+<?php
+
+namespace Laravel\Octane\Listeners;
+
+use Illuminate\Foundation\Vite;
+use Illuminate\Http\Request;
+use Laravel\Octane\Tests\TestCase;
+
+class FlushViteTest extends TestCase
+{
+    public function test_it_flushes_vite()
+    {
+        [$app, $worker] = $this->createOctaneContext([
+            Request::create('/', 'GET'),
+            Request::create('/', 'GET'),
+            Request::create('/', 'GET'),
+        ]);
+        $app['router']->get('/', fn () => 'ok');
+
+        $app->instance(Vite::class, $vite = new class extends Vite
+        {
+            public int $flushCalled = 0;
+
+            public function flush()
+            {
+                $this->flushCalled++;
+            }
+        });
+        $worker->run();
+
+        $this->assertSame(3, $vite->flushCalled);
+    }
+
+    public function test_it_handles_instances_of_vite_without_flush_method()
+    {
+        [$app, $worker] = $this->createOctaneContext([
+            Request::create('/', 'GET'),
+            Request::create('/', 'GET'),
+            Request::create('/', 'GET'),
+        ]);
+        $app['router']->get('/', fn () => 'ok');
+
+        $app->instance(Vite::class, new class
+        {
+            //
+        });
+        $worker->run();
+
+        $this->assertTrue(true);
+    }
+}