tweak log component and setting

Author: dacheng-gao

Agent.md Claude.mdAgent.md renamed to Claude.md

@@ -1,4 +1,4 @@
-# Agent.md
+# Claude.md
 
 ## Project Objective
 
@@ -15,4 +15,9 @@ Yii2 Swoole extension, provides coroutine http server and more coroutine feature
 ## Directory Structure
 
 - examples/: yii2 applications for demonstrating how to use yii2-swoole
-- src/: yii2-swoole extension for integrating swoole coroutine into yii2
+- src/: yii2-swoole extension for integrating swoole coroutine into yii2
+
+## Coding Standards
+
+- Always write clean code.
+- Only add necessary comments and avoid redundant comments.

examples/app-api/config/console.php

@@ -17,7 +17,7 @@
             'targets' => [
                 [
                     'class' => \Dacheng\Yii2\Swoole\Log\CoroutineFileTarget::class,
-                    'levels' => ['error', 'warning', 'info'],
+                    'levels' => YII_DEBUG ? ['error', 'warning', 'info'] : ['error', 'warning'],
                     'exportInterval' => 1,
                     'logFile' => '@runtime/logs/console.log',
                     'maxFileSize' => 10240, // 10MB

examples/app-api/config/web.php

@@ -34,15 +34,15 @@
             'targets' => [
                 [
                     'class' => \Dacheng\Yii2\Swoole\Log\CoroutineFileTarget::class,
-                    'levels' => ['error', 'warning'],
+                    'levels' => YII_DEBUG ? ['error', 'warning', 'info'] : ['error', 'warning'],
                     'exportInterval' => 1,
                     'logFile' => '@runtime/logs/app.log',
-                    'maxFileSize' => 10240,
+                    'maxFileSize' => 10240, // 10MB
                     'maxLogFiles' => 5,
                     'enableRotation' => true,
                     'categories' => [],
                     'except' => [],
-                    'logVars' => false,
+                    'logVars' => [],
                     'microtime' => true,
                 ],
             ],

examples/app-basic/config/console.php

@@ -32,7 +32,7 @@
             'targets' => [
                 [
                     'class' => \Dacheng\Yii2\Swoole\Log\CoroutineFileTarget::class,
-                    'levels' => ['error', 'warning', 'info'],
+                    'levels' => YII_DEBUG ? ['error', 'warning', 'info'] : ['error', 'warning'],
                     'exportInterval' => 1,
                     'logFile' => '@runtime/logs/console.log',
                     'maxFileSize' => 10240, // 10MB

examples/app-basic/config/web.php

@@ -50,10 +50,10 @@
             'targets' => [
                 [
                     'class' => \Dacheng\Yii2\Swoole\Log\CoroutineFileTarget::class,
-                    'levels' => ['error', 'warning'],
+                    'levels' => YII_DEBUG ? ['error', 'warning', 'info'] : ['error', 'warning'],
                     'exportInterval' => 1,
                     'logFile' => '@runtime/logs/app.log',
-                    'maxFileSize' => 10240,
+                    'maxFileSize' => 10240, // 10MB
                     'maxLogFiles' => 5,
                     'enableRotation' => true,
                     'categories' => [],

src/Log/CoroutineFileTarget.php

@@ -57,8 +57,6 @@ public function init()
 
         $this->maxLogFiles = max(1, $this->maxLogFiles);
         $this->maxFileSize = max(1, $this->maxFileSize);
-
-        $this->ensureInitialized();
     }
 
     private function ensureInitialized(): void
@@ -67,19 +65,17 @@ private function ensureInitialized(): void
             return;
         }
 
-        if (\Swoole\Coroutine::getCid() > 0) {
-            $this->worker = new LogWorker(
-                $this->logFile,
-                $this->enableRotation,
-                $this->maxFileSize,
-                $this->maxLogFiles,
-                $this->fileMode,
-                $this->dirMode
-            );
+        $this->worker = new LogWorker(
+            $this->logFile,
+            $this->enableRotation,
+            $this->maxFileSize,
+            $this->maxLogFiles,
+            $this->fileMode,
+            $this->dirMode
+        );
 
-            $this->worker->start();
-            $this->initialized = true;
-        }
+        $this->worker->start();
+        $this->initialized = true;
     }
 
     public function export()
@@ -98,12 +94,6 @@ public function export()
 
         $this->ensureInitialized();
 
-        if (!$this->initialized || $this->worker === null) {
-            $this->exportSync();
-            $this->messages = [];
-            return;
-        }
-
         $formattedMessages = array_map([$this, 'formatMessage'], $this->messages);
         $this->messages = [];
 
@@ -225,7 +215,6 @@ public function shutdown(): void
         $this->initialized = false;
     }
 
-
     public function __destruct()
     {
         try {

src/Log/LogWorker.php

@@ -89,9 +89,6 @@ public function stop(): void
         }
     }
 
-    /**
-     * Pushes messages directly to the buffer
-     */
     public function pushMessages(array $messages): bool
     {
         if (!$this->running) {
@@ -134,40 +131,24 @@ private function writeMessages(array $messages): void
             return;
         }
 
-        $fp = @fopen($this->logFile, 'a');
-        if ($fp === false) {
-            error_log("LogWorker: Unable to open log file: {$this->logFile}");
+        $success = @file_put_contents($this->logFile, $text, FILE_APPEND | LOCK_EX);
+        
+        if ($success === false) {
+            error_log("LogWorker: Unable to write to log file: {$this->logFile}");
             return;
         }
-
-        @flock($fp, LOCK_EX);
-
+        
+        if ($this->fileMode !== null) {
+            @chmod($this->logFile, $this->fileMode);
+        }
+        
         if ($this->enableRotation) {
             clearstatcache();
             $fileSize = @filesize($this->logFile);
             if ($fileSize !== false && $fileSize > $this->maxFileSize * 1024) {
-                @flock($fp, LOCK_UN);
-                @fclose($fp);
-                
                 $this->rotateFiles();
-                
-                $fp = @fopen($this->logFile, 'a');
-                if ($fp === false) {
-                    error_log("LogWorker: Unable to reopen log file after rotation: {$this->logFile}");
-                    return;
-                }
-                @flock($fp, LOCK_EX);
             }
         }
-
-        @fwrite($fp, $text);
-        @fflush($fp);
-        @flock($fp, LOCK_UN);
-        @fclose($fp);
-
-        if ($this->fileMode !== null) {
-            @chmod($this->logFile, $this->fileMode);
-        }
     }
 
     private function rotateFiles(): void