hong-review-cli 1.0.7 → 1.0.8

package/openclaw_integration_setup.md

@@ -136,8 +136,15 @@ hong-review setup-openclaw
 4. **查看待办池**
    - 触发话术样例：“目前积压了什么 MR ？”或“看下待审查代码”
    - 执行底包名：`hong-review ls -y`
+
+5. **查询执行日志 (排错/打点记录)**
+   - 触发话术样例：“查一下 CLI 的最新日志，刚才 webhook 没发出来”
+   - 工具行为准则：直接读取本地日志文件。日志文件夹路径获取方式：先执行 `node -e "console.log(require('/path/to/cli/src/utils/storage.js').get('logDir'))"` 获取配置目录（若为空则默认是在 `~/.hong-review-logs/`）。找到最新的 `hong-review-YYYY-MM-DD.log`，并 `tail -n 100` 或者 `cat` 读取并提炼关键错误信息回复给用户。
 ```
 
+> **进阶设置：修改默认日志输出目录**
+> 默认情况下，每日轮转的追踪日志（包含 Webhook 与生命周期全过程）存储在 `~/.hong-review-logs` 文件夹下。您可以在机器上使用 `hong-review config set logDir "/您的/自定义/绝对路径"` 来集中管理追踪内容。
+
 ---
 
 ## 验证与测试联调

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hong-review-cli",
-  "version": "1.0.7",
+  "version": "1.0.8",
   "main": "index.js",
   "bin": {
     "hong-review": "bin/hong-review.js"
@@ -25,6 +25,8 @@
     "commander": "^14.0.3",
     "figlet": "^1.10.0",
     "inquirer": "^8.2.7",
-    "ora": "^5.4.1"
+    "ora": "^5.4.1",
+    "winston": "^3.19.0",
+    "winston-daily-rotate-file": "^5.0.0"
   }
 }

package/src/utils/hooks.js

@@ -30,7 +30,9 @@ class Hooks {
                     ...payload
                 };
 
-                await this.executeWebhook(this.hookConfig, hookPayload, this.hookToken);
+                logger.info(`开始推送 Webhook [${eventName}] 到 ${this.hookConfig}`);
+                const response = await this.executeWebhook(this.hookConfig, hookPayload, this.hookToken);
+                logger.success(`Webhook 推送完成 [${eventName}] HTTP状态: ${response.status}`);
             } catch (err) {
                 // 如果是用户配置的静态 URL，失败不中断主流程，仅打个 Warn
                 logger.warn(`Webhook 推送失败 [${eventName}]: ${err.message}`);

package/src/utils/logger.js

@@ -1,51 +1,102 @@
 const chalk = require('chalk');
 const ora = require('ora');
+const winston = require('winston');
+const DailyRotateFile = require('winston-daily-rotate-file');
+const fs = require('fs');
+const path = require('path');
+const os = require('os');
+const storage = require('./storage');
 
 const spinners = new Map();
 
 class Logger {
+    constructor() {
+        this.initWinston();
+    }
+
+    initWinston() {
+        // Find log directory setting or default to ~/.hong-review-logs
+        const configLogDir = storage.get('logDir');
+        let defaultLogDir = '';
+        if (configLogDir) {
+            defaultLogDir = configLogDir;
+        } else {
+            defaultLogDir = path.join(os.homedir(), '.hong-review-logs');
+        }
+
+        if (!fs.existsSync(defaultLogDir)) {
+            fs.mkdirSync(defaultLogDir, { recursive: true });
+        }
+
+        this.fileLogger = winston.createLogger({
+            level: 'info',
+            format: winston.format.combine(
+                winston.format.timestamp({ format: 'YYYY-MM-DD HH:mm:ss' }),
+                winston.format.printf(info => `[${info.timestamp}] ${info.level.toUpperCase()}: ${info.message}`)
+            ),
+            transports: [
+                new DailyRotateFile({
+                    dirname: defaultLogDir,
+                    filename: 'hong-review-%DATE%.log',
+                    datePattern: 'YYYY-MM-DD',
+                    maxFiles: '14d'
+                })
+            ]
+        });
+    }
+
     info(msg) {
         console.log(chalk.blue('ℹ ' + msg));
+        this.fileLogger.info(msg);
     }
-    
+
     success(msg) {
         console.log(chalk.green('✔ ' + msg));
+        this.fileLogger.info(msg);
     }
-    
+
     warn(msg) {
         console.log(chalk.yellow('⚠ ' + msg));
+        this.fileLogger.warn(msg);
     }
-    
+
     error(msg) {
         console.log(chalk.red('✖ ' + msg));
+        this.fileLogger.error(msg);
     }
-    
+
     startSpinner(id, text) {
         if (spinners.has(id)) {
             spinners.get(id).stop();
         }
         const spinner = ora(text).start();
         spinners.set(id, spinner);
+        this.fileLogger.info(`(Started) ${text}`);
     }
-    
+
     stopSpinner(id, success = true, text = '') {
         const spinner = spinners.get(id);
         if (!spinner) return;
-        
+
+        const finalMsg = text || spinner.text;
         if (success) {
-            spinner.succeed(text || spinner.text);
+            spinner.succeed(finalMsg);
+            this.fileLogger.info(`(Completed) ${finalMsg}`);
         } else {
-            spinner.fail(text || spinner.text);
+            spinner.fail(finalMsg);
+            this.fileLogger.error(`(Failed) ${finalMsg}`);
         }
         spinners.delete(id);
     }
 
     logRaw(msg) {
         console.log(msg);
+        this.fileLogger.info(msg);
     }
-    
+
     divider() {
         console.log(chalk.gray('-'.repeat(50)));
+        this.fileLogger.info('-'.repeat(50));
     }
 }