Add configurable opcache clearing and static file serving options (#1035)

* Add configurable opcache clearing and static file serving

- Add octane.swoole.clear_opcache configuration option (defaults to true)
- Add octane.serve_static_files configuration option (defaults to true)
- Maintain backward compatibility with existing behavior
- Add comprehensive unit tests for both features

These configurations provide better control over Octane's behavior in different deployment scenarios, particularly useful for Docker environments and API-only applications where static file handling may not be required.

* Fix StyleCI issues

- Remove extra blank lines
- Fix spacing around operators (! operator)
- Maintain consistent code style

* Fix missing newline at end of file

* Remove trailing empty line

* Update OnWorkerStart.php

* Update SwooleClient.php

---------

Co-authored-by: Taylor Otwell <[email protected]>

Author: hughcube

src/Swoole/Handlers/OnWorkerStart.php

@@ -30,7 +30,9 @@ public function __construct(
      */
     public function __invoke($server, int $workerId)
     {
-        $this->clearOpcodeCache();
+        if ($this->shouldClearOpcodeCache()) {
+            $this->clearOpcodeCache();
+        }
 
         $this->workerState->server = $server;
         $this->workerState->workerId = $workerId;
@@ -107,6 +109,16 @@ protected function streamRequestsToConsole($server)
         });
     }
 
+    /**
+     * Determine if the opcode cache should be cleared.
+     *
+     * @return bool
+     */
+    protected function shouldClearOpcodeCache()
+    {
+        return config('octane.swoole.clear_opcache', true);
+    }
+
     /**
      * Clear the APCu and Opcache caches.
      *

src/Swoole/SwooleClient.php

@@ -50,6 +50,13 @@ public function marshalRequest(RequestContext $context): array
      */
     public function canServeRequestAsStaticFile(Request $request, RequestContext $context): bool
     {
+        $octaneConfig = $context->octaneConfig ?? [];
+
+        if (array_key_exists('serve_static_files', $octaneConfig) && 
+            ! $octaneConfig['serve_static_files']) {
+            return false;
+        }
+
         if (! ($context->publicPath ?? false) ||
             $request->path() === '/') {
             return false;

tests/OnWorkerStartTest.php

@@ -0,0 +1,38 @@
+<?php
+
+namespace Laravel\Octane\Tests;
+
+use Laravel\Octane\Swoole\Handlers\OnWorkerStart;
+use Mockery;
+
+class OnWorkerStartTest extends TestCase
+{
+    public function test_should_clear_opcache_returns_true_by_default(): void
+    {
+        $this->createApplication();
+
+        // Mock the OnWorkerStart handler to test just the shouldClearOpcodeCache method
+        $handler = Mockery::mock(OnWorkerStart::class)->makePartial();
+
+        $reflection = new \ReflectionClass($handler);
+        $method = $reflection->getMethod('shouldClearOpcodeCache');
+        $method->setAccessible(true);
+
+        $this->assertTrue($method->invoke($handler));
+    }
+
+    public function test_should_clear_opcache_returns_configured_value(): void
+    {
+        $app = $this->createApplication();
+        $app['config']['octane.swoole.clear_opcache'] = false;
+
+        // Mock the OnWorkerStart handler to test just the shouldClearOpcodeCache method
+        $handler = Mockery::mock(OnWorkerStart::class)->makePartial();
+
+        $reflection = new \ReflectionClass($handler);
+        $method = $reflection->getMethod('shouldClearOpcodeCache');
+        $method->setAccessible(true);
+
+        $this->assertFalse($method->invoke($handler));
+    }
+}

tests/SwooleClientTest.php

@@ -73,6 +73,38 @@ public function test_can_serve_static_files_if_configured_to_and_file_is_within_
         $this->assertTrue($client->canServeRequestAsStaticFile($request, $context));
     }
 
+    public function test_can_serve_static_files_when_explicitly_enabled(): void
+    {
+        $client = new SwooleClient;
+
+        $request = Request::create('/foo.txt', 'GET');
+
+        $context = new RequestContext([
+            'publicPath' => __DIR__.'/public',
+            'octaneConfig' => [
+                'serve_static_files' => true,
+            ],
+        ]);
+
+        $this->assertTrue($client->canServeRequestAsStaticFile($request, $context));
+    }
+
+    public function test_cant_serve_static_files_when_disabled(): void
+    {
+        $client = new SwooleClient;
+
+        $request = Request::create('/foo.txt', 'GET');
+
+        $context = new RequestContext([
+            'publicPath' => __DIR__.'/public',
+            'octaneConfig' => [
+                'serve_static_files' => false,
+            ],
+        ]);
+
+        $this->assertFalse($client->canServeRequestAsStaticFile($request, $context));
+    }
+
     public function test_cant_serve_static_files_if_file_is_outside_public_directory(): void
     {
         $client = new SwooleClient;