refactor: clean up signal handling and remove unused code

- Remove SIGKILL hack, use graceful coroutine wait
- Simplify SignalHandler exit logic
- Add ShutdownHelper to consolidate shutdown code
- Remove unused LogWorker/CoroutineFileTarget parameters
- Update all configs and docs to remove deprecated options

Author: dacheng-gao

README.md

@@ -106,7 +106,9 @@ return [
                     'class' => \Dacheng\Yii2\Swoole\Log\CoroutineFileTarget::class,
                     'levels' => ['error', 'warning'],
                     'logFile' => '@runtime/logs/app.log',
-                    'channelSize' => 10000,
+                    'maxFileSize' => 10240, // KB
+                    'maxLogFiles' => 5,
+                    'enableRotation' => true,
                 ],
             ],
         ],
@@ -230,11 +232,11 @@ curl http://127.0.0.1:9501/
     'targets' => [
         [
             'class' => \Dacheng\Yii2\Swoole\Log\CoroutineFileTarget::class,
-            'channelSize' => 10000,    // Channel buffer size
-            'pushTimeout' => 0.5,      // Timeout for pushing to channel
-            'batchSize' => 1000,       // Messages per batch write
+            'levels' => ['error', 'warning'],
+            'logFile' => '@runtime/logs/app.log',
             'maxFileSize' => 10240,    // Max file size before rotation (KB)
             'maxLogFiles' => 5,        // Number of rotated files to keep
+            'enableRotation' => true,  // Enable log rotation
         ],
     ],
 ],

examples/app-api/config/console.php

@@ -20,9 +20,6 @@
                     'levels' => ['error', 'warning', 'info'],
                     'exportInterval' => 1,
                     'logFile' => '@runtime/logs/console.log',
-                    'channelSize' => (int)(getenv('YII_LOG_CHANNEL_SIZE') ?: 10000),
-                    'pushTimeout' => 0.5,
-                    'batchSize' => 1000, // Packets per batch write
                     'maxFileSize' => 10240, // 10MB
                     'maxLogFiles' => 5,
                     'enableRotation' => true,

examples/app-api/config/web.php

@@ -37,9 +37,6 @@
                     'levels' => ['error', 'warning'],
                     'exportInterval' => 1,
                     'logFile' => '@runtime/logs/app.log',
-                    'channelSize' => (int)(getenv('YII_LOG_CHANNEL_SIZE') ?: 10000),
-                    'pushTimeout' => 0.5,
-                    'batchSize' => 1000,
                     'maxFileSize' => 10240,
                     'maxLogFiles' => 5,
                     'enableRotation' => true,

examples/app-api/controllers/LogController.php

@@ -199,8 +199,6 @@ public function actionStats()
             $lineCount = count(file($logFile));
         }
 
-        $channelStats = $logTarget->getChannelStats();
-
         $result = [
             'success' => true,
             'log_file' => $logFile,
@@ -210,15 +208,10 @@ public function actionStats()
             'line_count' => $lineCount,
             'max_file_size_kb' => $logTarget->maxFileSize,
             'max_log_files' => $logTarget->maxLogFiles,
-            'channel_size' => $logTarget->channelSize,
-            'push_timeout' => $logTarget->pushTimeout,
             'rotation_enabled' => $logTarget->enableRotation,
+            'worker_initialized' => $logTarget->getWorker() !== null,
         ];
 
-        if ($channelStats !== null) {
-            $result['channel'] = $channelStats;
-        }
-
         return $result;
     }
 }

examples/app-basic/config/console.php

@@ -35,9 +35,6 @@
                     'levels' => ['error', 'warning', 'info'],
                     'exportInterval' => 1,
                     'logFile' => '@runtime/logs/console.log',
-                    'channelSize' => (int)(getenv('YII_LOG_CHANNEL_SIZE') ?: 10000),
-                    'pushTimeout' => 0.5,
-                    'batchSize' => 1000, // Packets per batch write
                     'maxFileSize' => 10240, // 10MB
                     'maxLogFiles' => 5,
                     'enableRotation' => true,

examples/app-basic/config/web.php

@@ -53,9 +53,6 @@
                     'levels' => ['error', 'warning'],
                     'exportInterval' => 1,
                     'logFile' => '@runtime/logs/app.log',
-                    'channelSize' => (int)(getenv('YII_LOG_CHANNEL_SIZE') ?: 10000),
-                    'pushTimeout' => 0.5,
-                    'batchSize' => 1000,
                     'maxFileSize' => 10240,
                     'maxLogFiles' => 5,
                     'enableRotation' => true,

src/Console/SwooleController.php

@@ -64,7 +64,20 @@ public function actionStart(?string $host = null, ?string $port = null): int
 
         $this->stdout(sprintf("Swoole HTTP server listening on %s:%d\n", $server->host, $server->port));
 
-        $server->start();
+        try {
+            $server->start();
+        } catch (\Swoole\ExitException $e) {
+            // ExitException is thrown during graceful shutdown, this is expected
+            $exitCode = method_exists($e, 'getStatus') ? $e->getStatus() : 0;
+            
+            if ($exitCode === 0) {
+                $this->stdout("Swoole HTTP server stopped gracefully.\n");
+                return ExitCode::OK;
+            }
+
+            $this->stderr(sprintf("Swoole HTTP server exited with code %d: %s\n", $exitCode, $e->getMessage()));
+            return $exitCode;
+        }
 
         return ExitCode::OK;
     }