wip

Author: michalsn

app/Config/WorkerMode.php

@@ -50,7 +50,7 @@ class WorkerMode
      * - 'disconnect':
      *   Closes all connections after each request
      *
-     * @var 'keep-alive'|'disconnect'
+     * @var 'disconnect'|'keep-alive'
      */
     public string $databaseStrategy = 'keep-alive';
 
@@ -64,7 +64,7 @@ class WorkerMode
      * - 'disconnect':
      *   Closes the handler after each request
      *
-     * @var 'keep-alive'|'disconnect'
+     * @var 'disconnect'|'keep-alive'
      */
     public string $cacheStrategy = 'keep-alive';
 

system/CodeIgniter.php

@@ -195,8 +195,6 @@ public function initialize()
     /**
      * Reset request-specific state for worker mode.
      * Clears all request/response data to prepare for the next request.
-     *
-     * @return void
      */
     public function resetForWorkerMode(): void
     {

system/Commands/Worker/WorkerInstall.php

@@ -1,5 +1,14 @@
 <?php
 
+/**
+ * This file is part of CodeIgniter 4 framework.
+ *
+ * (c) CodeIgniter Foundation <[email protected]>
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
 namespace CodeIgniter\Commands\Worker;
 
 use CodeIgniter\CLI\BaseCommand;
@@ -16,9 +25,8 @@ class WorkerInstall extends BaseCommand
     protected $group       = 'Worker Mode';
     protected $name        = 'worker:install';
     protected $description = 'Install FrankenPHP worker mode by creating necessary configuration files';
-
-    protected $usage = 'worker:install [options]';
-    protected $options = [
+    protected $usage       = 'worker:install [options]';
+    protected $options     = [
         '--force' => 'Overwrite existing files',
     ];
 
@@ -61,7 +69,7 @@ public function run(array $params)
                 CLI::error(
                     "Failed to read template: {$template}",
                     'light_gray',
-                    'red'
+                    'red',
                 );
                 CLI::newLine();
 
@@ -73,7 +81,7 @@ public function run(array $params)
                 CLI::error(
                     'Failed to create file: ' . clean_path($target),
                     'light_gray',
-                    'red'
+                    'red',
                 );
                 CLI::newLine();
 

system/Commands/Worker/WorkerUninstall.php

@@ -1,5 +1,14 @@
 <?php
 
+/**
+ * This file is part of CodeIgniter 4 framework.
+ *
+ * (c) CodeIgniter Foundation <[email protected]>
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
 namespace CodeIgniter\Commands\Worker;
 
 use CodeIgniter\CLI\BaseCommand;
@@ -15,9 +24,8 @@ class WorkerUninstall extends BaseCommand
     protected $group       = 'Worker Mode';
     protected $name        = 'worker:uninstall';
     protected $description = 'Remove FrankenPHP worker mode configuration files';
-
-    protected $usage = 'worker:uninstall [options]';
-    protected $options = [
+    protected $usage       = 'worker:uninstall [options]';
+    protected $options     = [
         '--force' => 'Skip confirmation prompt',
     ];
 
@@ -40,6 +48,7 @@ public function run(array $params)
 
         // Find existing files
         $existing = [];
+
         foreach ($this->files as $file) {
             $path = ROOTPATH . $file;
             if (is_file($path)) {
@@ -57,6 +66,7 @@ public function run(array $params)
 
         // Show files that will be removed
         CLI::write('The following files will be removed:', 'yellow');
+
         foreach ($existing as $file) {
             CLI::write('  - ' . $file, 'white');
         }
@@ -83,6 +93,7 @@ public function run(array $params)
 
             if (! @unlink($path)) {
                 CLI::error('Failed to remove file: ' . clean_path($path), 'light_gray', 'red');
+
                 continue;
             }
 

system/Config/BaseService.php

@@ -377,9 +377,6 @@ public static function reset(bool $initAutoloader = true)
     /**
      * Resets all services except those in the persistent list.
      * Used for worker mode to preserve expensive-to-initialize services.
-     *
-     * @param WorkerMode $config
-     * @return void
      */
     public static function resetForWorkerMode(WorkerMode $config): void
     {

system/Database/BaseConnection.php

@@ -495,8 +495,6 @@ abstract protected function _close();
 
     /**
      * Check if the connection is still alive.
-     *
-     * @return bool
      */
     public function ping(): bool
     {
@@ -509,16 +507,13 @@ public function ping(): bool
 
     /**
      * Driver-specific ping implementation.
-     *
-     * @return bool
      */
     protected function _ping(): bool
     {
         try {
             $result = $this->simpleQuery('SELECT 1');
 
             return $result !== false;
-
         } catch (DatabaseException) {
             return false;
         }

system/Database/Config.php

@@ -160,9 +160,6 @@ protected static function ensureFactory()
      *
      * This method handles connection state management between requests
      * in long-running worker processes.
-     *
-     * @param WorkerMode $config
-     * @return void
      */
     public static function resetForWorkerMode(WorkerMode $config): void
     {
@@ -202,5 +199,4 @@ public static function resetForWorkerMode(WorkerMode $config): void
             static::$factory   = null;
         }
     }
-
 }

system/Database/Postgre/Connection.php

@@ -168,15 +168,12 @@ protected function _close()
 
     /**
      * Ping the database connection.
-     *
-     * @return bool
      */
     protected function _ping(): bool
     {
         return pg_ping($this->connID);
     }
 
-
     /**
      * Select a specific database table to use.
      */

system/Events/Events.php

@@ -293,12 +293,9 @@ public static function getPerformanceLogs()
      *
      * Note: This clears dynamically registered listeners. If your application
      * registers listeners at runtime, they will need to be re-registered for each request.
-     *
-     * @return void
      */
     public static function resetForWorkerMode(): void
     {
         static::$performanceLog = [];
     }
-
 }

tests/system/Config/ServicesTest.php

@@ -520,8 +520,8 @@ public function testHasReturnsFalseForNonExistentService(): void
     #[RunInSeparateProcess]
     public function testResetForWorkerModeWithKeepAliveStrategy(): void
     {
-        $config = new WorkerMode();
-        $config->cacheStrategy = 'keep-alive';
+        $config                     = new WorkerMode();
+        $config->cacheStrategy      = 'keep-alive';
         $config->persistentServices = ['timer'];
 
         Services::cache();
@@ -543,8 +543,8 @@ public function testResetForWorkerModeWithKeepAliveStrategy(): void
     #[RunInSeparateProcess]
     public function testResetForWorkerModeWithReconnectStrategy(): void
     {
-        $config = new WorkerMode();
-        $config->cacheStrategy = 'reconnect';
+        $config                     = new WorkerMode();
+        $config->cacheStrategy      = 'reconnect';
         $config->persistentServices = ['timer'];
 
         Services::cache();