hong-review-cli 1.0.0 → 1.0.1

package/bin/hong-review.js

@@ -50,6 +50,7 @@ program
     .option('-R, --report', '生成本地 Markdown 审查报告文件')
     .option('-c, --comment', '审查完成后，自动将结果作为评论发至 MR 讨论区')
     .option('-m, --merge', '审查通过且无明显问题时，自动合并该 MR')
+    .option('-y, --yes', '非交互式静默模式 (自动跳过所有确认)')
     .action((mr_id, options) => {
         mrCmd(mr_id, options);
     });
@@ -58,7 +59,10 @@ program
     .command('list')
     .alias('ls')
     .description('查看分配给我或待我处理的 MR 列表')
-    .action(listCmd);
+    .option('-y, --yes', '非交互式静默模式 (不提示直接输出文本)')
+    .action((options) => {
+        listCmd(options);
+    });
 
 // --- 单独动作 ---
 program
@@ -71,8 +75,9 @@ program
 program
     .command('merge <mr_id>')
     .description('检查权限并快捷合并指定的 MR')
-    .action((mr_id) => {
-        actionsCmd.merge(mr_id);
+    .option('-y, --yes', '非交互式静默执行')
+    .action((mr_id, options) => {
+        actionsCmd.merge(mr_id, options);
     });
 
 program

package/openclaw_integration_setup.md

@@ -0,0 +1,76 @@
+# OpenClaw 接入指南: `hong-review-cli` 自动审查流
+
+本指南将指导您如何配置您的 OpenClaw，使其能够理解、使用 `hong-review` 命令行工具，并将审查结果路由推送到您的 Telegram 聊天中。
+
+---
+
+## 步骤 1：为 CLI 配置 Webhook 推送地址
+
+利用 `hong-review` 自带的生命周期挂钩（Hooks）功能，告诉它在审查完成后把数据扔给谁（即 OpenClaw Server 的 `hooks` 接口）。
+
+在您的服务器环境（安装了 OpenClaw 和该 CLI 的机器）上运行以下命令：
+*(注意：请将 `http://127.0.0.1:3000` 替换为 OpenClaw Gateway 真实的内网通信地址，并将 `<YOUR_SECRET_TOKEN>` 替换为 OpenClaw 配置文件中 `hooks.token` 相同的值)*
+
+```bash
+hong-review config set hooks.onReviewSuccess "curl -X POST http://127.0.0.1:3000/hooks/code-review -H 'Authorization: Bearer <YOUR_SECRET_TOKEN>' -H 'Content-Type: application/json' -d '{\"mrId\": \"$MR_ID\", \"status\": \"Success\"}'"
+
+hong-review config set hooks.onReviewFailed "curl -X POST http://127.0.0.1:3000/hooks/code-review -H 'Authorization: Bearer <YOUR_SECRET_TOKEN>' -H 'Content-Type: application/json' -d '{\"mrId\": \"$MR_ID\", \"status\": \"Failed\"}'"
+```
+
+## 步骤 2：配置 OpenClaw 的路由文件 (Hooks Mapping)
+
+在您的 OpenClaw 配置文件 (`openclaw.yaml` 或 [.json](file:///Users/hong/Desktop/cli/package.json)) 中，找到 `hooks` 配置块。你需要新增一条映射规则，让 OpenClaw 知道收到 `code-review` 路径的请求时，发送什么消息给 Telegram。
+
+**示例配置（YAML 风格）:**
+
+```yaml
+hooks:
+  token: "<YOUR_SECRET_TOKEN>" # 这必须与第一步 CURL 里的 Token 一致
+  basePath: "/hooks"           # 这也对应第一步的路径
+  mappings:
+    - id: "hong-review-notification"
+      match:
+        path: "code-review"    # 匹配 /hooks/code-review
+      action: "agent"		   # 以 Agent 代理的口吻发送
+      agentId: "telegram-bot"  # 你的 Telegram Bot 的 Agent 标识
+      name: "AI Code Reviewer"
+      channel: "telegram"      # 明确发往 Telegram 通道
+      to: "@your_telegram_username" # 你的真实触达账号
+      messageTemplate: |
+        代码审查完成 🔔
+        - **MR ID**: {{payload.mrId}}
+        - **状态**: {{payload.status}}
+        *(日志及报告已由系统分析进程执行完毕)*
+```
+*(这段配置使用了 OpenClaw 原生的 [src/gateway/hooks-mapping.ts](file:///tmp/openclaw/src/gateway/hooks-mapping.ts) 解析能力，它允许插值获取 Payload 里的变量并组合成文本推出去)*
+
+---
+
+## 步骤 3：赋予 OpenClaw "调用能力" (Tool / System Prompt)
+
+最后也是最关键的，你需要把下面这段文字，作为插件系统描述或者**核心系统提示词 (System Prompt)** 告诉管理你 Telegram 消息的 OpenClaw 机器人。
+
+### 👉 给 OpenClaw 机器人的 System Schema 设定：
+
+```markdown
+你是一个全能的代码审查助手。你现在被授权在宿主服务器上调用一个名为 `hong-review` 的终端核心程序来处理用户的 GitLab 审查请求。
+
+**重要规则**：
+因为你处于非人机交互环境，你每次调用该程序时，**必须加上 `-y` 或 `--yes` 参数**，这代表启用静默安全模式。
+程序在执行结束或出错后会自动处理，**你永远不要尝试去读取它的标准输出日志来提取长篇大论的报告**，程序的内建 Hooks 系统会自己将带有排版的最终结果推送到 Telegram，你要做的这只是触发它即可。
+
+**你可以使用它来回复用户的三种核心意图：**
+
+1. 如果用户问“我的待办列表里有什么 MR？”或“列出待审查代码”：
+   请在底层执行命令：`hong-review ls -y`
+
+2. 如果用户说“去帮我分析一下 101 这个 MR” 或 “查下 101”：
+   请在底层执行命令：`hong-review mr 101 -y`
+
+3. 如果用户经过阅读报告后说“确认合并 101 吧”：
+   请在底层执行命令：`hong-review merge 101 -y`
+```
+
+---
+
+完成以上三步，当您在 Telegram App 打开 OpenClaw 聊天窗口发送："去查下 52 那个代码提交" 后。一切都会像齿轮一样无人值守转动，并在几秒或几十秒后把叮咚声推送到你的手机里！

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hong-review-cli",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "main": "index.js",
   "bin": {
     "hong-review": "bin/hong-review.js"

package/src/commands/actions.js

@@ -44,37 +44,42 @@ module.exports = {
         }
     },
 
-    async merge(mrId) {
+    async merge(mrId, options = {}) {
         try {
              const projectId = await getProjectId();
              const gitlab = getClient();
              
-             logger.startSpinner('merge-check', `正在检查 MR !${mrId} 合并权限...`);
+             if (!options.yes) logger.startSpinner('merge-check', `正在检查 MR !${mrId} 合并权限...`);
              const check = await gitlab.canMergeMR(projectId, mrId);
              
              if (!check.canMerge) {
-                 logger.stopSpinner('merge-check', false, `无法合并: ${check.reason}`);
+                 if (!options.yes) logger.stopSpinner('merge-check', false, `无法合并: ${check.reason}`);
+                 else console.error(`Failed to merge: ${check.reason}`);
                  return;
              }
-             logger.stopSpinner('merge-check', true, '权限检查通过。');
+             if (!options.yes) logger.stopSpinner('merge-check', true, '权限检查通过。');
 
-             const answer = await inquirer.prompt([{
-                 type: 'confirm',
-                 name: 'confirmMerge',
-                 message: `即将自动合并 MR !${mrId}，是否确认？`,
-                 default: false
-             }]);
+             let confirm = options.yes;
+             if (!confirm) {
+                 const answer = await inquirer.prompt([{
+                     type: 'confirm',
+                     name: 'confirmMerge',
+                     message: `即将自动合并 MR !${mrId}，是否确认？`,
+                     default: false
+                 }]);
+                 confirm = answer.confirmMerge;
+             }
 
-             if (answer.confirmMerge) {
-                 logger.startSpinner('merge', '合并中...');
+             if (confirm) {
+                 if (!options.yes) logger.startSpinner('merge', '合并中...');
                  await gitlab.merge(projectId, mrId);
-                 logger.stopSpinner('merge', true, '合并成功！');
+                 if (!options.yes) logger.stopSpinner('merge', true, '合并成功！');
+                 else console.log(`✅ MR !${mrId} 合并成功!`);
              } else {
                  logger.info('已取消合并。');
              }
 
         } catch (e) {
-             if (spinners && spinners.has('merge')) logger.stopSpinner('merge', false);
              logger.error(`合并失败: ${e.message}`);
         }
     },

package/src/commands/list.js

@@ -3,7 +3,7 @@ const logger = require('../utils/logger');
 const GitLabClient = require('../core/gitlab');
 const chalk = require('chalk');
 
-module.exports = async function() {
+module.exports = async function(options = {}) {
     const config = storage.getAll();
     if (!config.gitlabToken) {
         logger.error('缺少 GitLab Token 配置，请运行 hong-review login');
@@ -12,10 +12,20 @@ module.exports = async function() {
 
     const gitlab = new GitLabClient({ host: config.gitlabUrl, token: config.gitlabToken });
     
-    logger.startSpinner('list', '正在向 GitLab 拉取您需要处理的 Merge Requests...');
+    if (!options.yes) logger.startSpinner('list', '正在向 GitLab 拉取您需要处理的 Merge Requests...');
     
     try {
         const mrs = await gitlab.getMyReviewMergeRequests();
+        
+        if (options.yes) {
+            // 静默模式下，直接输出纯文本列表供其他程序抓取/处理
+            console.log(`找到 ${mrs.length} 个待处理的 MR:`);
+            mrs.forEach(mr => {
+                console.log(`- ProjectID: ${mr.project_id} | MR !${mr.iid} | Title: ${mr.title} | Branch: ${mr.source_branch} -> ${mr.target_branch}`);
+            });
+            return;
+        }
+
         logger.stopSpinner('list', true, `成功拉取！共分配给您或待您审核的 MR 有 ${mrs.length} 个。\n`);
         
         if (mrs.length === 0) {
@@ -60,6 +70,7 @@ module.exports = async function() {
         }
         
     } catch (e) {
-        logger.stopSpinner('list', false, `拉取 MR 列表失败: ${e.message}`);
+        if (!options.yes) logger.stopSpinner('list', false, `拉取 MR 列表失败: ${e.message}`);
+        else console.error(`Error: 拉取 MR 列表失败 - ${e.message}`);
     }
 };

package/src/commands/mr.js

@@ -28,6 +28,10 @@ module.exports = async function(mrId, options) {
     // 为了 demo 的通用性，我们直接要求用户确认或配置 projectId。
     let projectId = config.defaultProjectId;
     if (!projectId) {
+         if (options.yes) {
+             logger.error('静默模式下未找到 defaultProjectId 配置，请先配置或在有历史审查记录后再尝试。');
+             return;
+         }
          const answer = await inquirer.prompt([{
              type: 'input',
              name: 'pid',
@@ -58,11 +62,12 @@ module.exports = async function(mrId, options) {
         logger.stopSpinner('fetching', true, `成功拉取 MR [${detail.title}] 分析所需的 ${changes.length} 个文件变更。`);
 
         if (changes.length === 0) {
-            logger.warn('该 MR 没有包含任何文件变更。');
+            if (!options.yes) logger.warn('该 MR 没有包含任何文件变更。');
+            else console.log('MR 审查结束: 没有包含任何文件变更。');
             return;
         }
 
-        logger.startSpinner('ai-thinking', `🤖 AI (${config.aiModel}) 正在深度分析中，请稍候...`);
+        if (!options.yes) logger.startSpinner('ai-thinking', `🤖 AI (${config.aiModel}) 正在深度分析中，请稍候...`);
         
         let reviewResult = null;
         let round = 1;
@@ -72,7 +77,7 @@ module.exports = async function(mrId, options) {
 
         // 开始 Agent 轮询
         while (true) {
-            logger.startSpinner('ai-thinking', `🤖 AI 思考中 (第 ${round} 轮)...`);
+            if (!options.yes) logger.startSpinner('ai-thinking', `🤖 AI 思考中 (第 ${round} 轮)...`);
             const aiResponse = await ai.chatWithJsonMode(AGENT_SYSTEM_PROMPT, conversationHistory);
             const action = parseAgentResponse(aiResponse);
 
@@ -86,7 +91,7 @@ module.exports = async function(mrId, options) {
                 reviewResult = action.result;
                 break;
             } else if (action.action === 'request_files') {
-                logger.stopSpinner('ai-thinking', true, `AI 请求查看额外文件: ${action.files.join(', ')}`);
+                if (!options.yes) logger.stopSpinner('ai-thinking', true, `AI 请求查看额外文件: ${action.files.join(', ')}`);
                 
                 const contents = await fetchFileContents(gitlab, projectId, detail.source_branch, changes, action.files, action.contentTypes);
                 const fileContentMessage = formatFileContents(contents);
@@ -95,7 +100,7 @@ module.exports = async function(mrId, options) {
             }
         }
 
-        logger.stopSpinner('ai-thinking', true, 'AI 审查完成！');
+        if (!options.yes) logger.stopSpinner('ai-thinking', true, 'AI 审查完成！');
         
         // 打印报告
         printReport(mrIid, detail, reviewResult);
@@ -124,19 +129,7 @@ module.exports = async function(mrId, options) {
             logger.stopSpinner('comment', true, '评论提交成功。');
         }
 
-        // 处理自动合并参数 -m
-        if (options.merge && isSuccess && reviewResult.issues.filter(i => i.severity === 'error').length === 0) {
-             logger.startSpinner('merge', '达到自动合并标准，尝试执行自动合并...');
-             try {
-                 await gitlab.merge(projectId, mrIid);
-                 logger.stopSpinner('merge', true, '合并执行成功！');
-                 await hooks.emit('onMerged', { projectId, mrId: mrIid });
-             } catch (err) {
-                 logger.stopSpinner('merge', false, `合并失败: ${err.message}`);
-             }
-        }
-
-        if (!options.merge) {
+        if (!options.merge && !options.yes) {
             console.log('\n');
             const postReviewAnswer = await inquirer.prompt([{
                 type: 'list',
@@ -164,6 +157,9 @@ module.exports = async function(mrId, options) {
             } else {
                 logger.info('操作已完成，退出审查。');
             }
+        } else if (options.yes && !options.merge) {
+            // 静默模式下，不需要选择，直接结束
+            console.log(`✅ [Headless] MR !${mrIid} 代码审查执行结束。`);
         }
 
     } catch (e) {