hong-review-cli 1.0.3 → 1.0.5

package/openclaw_integration_setup.md

@@ -18,26 +18,39 @@ hong-review setup-openclaw
 
 在您的 OpenClaw 配置文件 (`openclaw.yaml` 或 [.json](file:///Users/hong/Desktop/cli/package.json)) 中，找到 `hooks` 配置块。你需要新增一条映射规则，让 OpenClaw 知道收到 `code-review` 路径的请求时，发送什么消息给 Telegram。
 
-**示例配置（YAML 风格）:**
-
-```yaml
-hooks:
-  token: "e4e2136cb731cfb14f1ad190833738c5bdadfbf79cf7c0f5" # 这必须与第一步 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}}
-        *(日志及报告已由系统分析进程执行完毕)*
+**针对您当前 JSON 配置的示例配置:**
+
+您可以直接将以下 `hooks` 代码片段复制替换掉您当前配置文件里的 `hooks` 部分。已经为您配置好了正确的 `tech-group` 发送代理目标和 Telegram 群组 ID：
+
+```json
+  "hooks": {
+    "token": "e4e2136cb731cfb14f1ad190833738c5bdadfbf79cf7c0f5",
+    "internal": {
+      "enabled": true,
+      "entries": {
+        "command-logger": {
+          "enabled": true
+        },
+        "session-memory": {
+          "enabled": false
+        }
+      }
+    },
+    "mappings": [
+      {
+        "id": "hong-review-notification",
+        "match": {
+          "path": "code-review"
+        },
+        "action": "agent",
+        "agentId": "tech-group",
+        "name": "AI Code Reviewer",
+        "channel": "telegram",
+        "to": "-5063603196",
+        "messageTemplate": "代码审查完成 🔔\n- **MR ID**: {{payload.mrId}}\n- **状态**: {{payload.status}}\n*(日志及报告已由系统分析进程执行完毕)*"
+      }
+    ]
+  }
 ```
 *(这段配置使用了 OpenClaw 原生的 [src/gateway/hooks-mapping.ts](file:///tmp/openclaw/src/gateway/hooks-mapping.ts) 解析能力，它允许插值获取 Payload 里的变量并组合成文本推出去)*
 
@@ -56,16 +69,34 @@ hooks:
 因为你处于非人机交互环境，你每次调用该程序时，**必须加上 `-y` 或 `--yes` 参数**，这代表启用静默安全模式。
 程序在执行结束或出错后会自动处理，**你永远不要尝试去读取它的标准输出日志来提取长篇大论的报告**，程序的内建 Hooks 系统会自己将带有排版的最终结果推送到 Telegram，你要做的这只是触发它即可。
 
-**你可以使用它来回复用户的三种核心意图：**
+**你可以使用它来回复用户的各种核心意图（请务必携带对应的命令和必要参数）：**
 
-1. 如果用户问“我的待办列表里有什么 MR？”或“列出待审查代码”：
+1. 审查和分析具体合并请求：
+   如果用户说“去帮我分析一下 101 这个 MR” 或 “查下 101”
+   请在底层执行命令：`hong-review mr 101 -y`
+
+2. 查看待办列表：
+   如果用户问“我的待办列表里有什么 MR？”或“列出待审查代码”
    请在底层执行命令：`hong-review ls -y`
 
-2. 如果用户说“去帮我分析一下 101 这个 MR” 或 “查下 101”：
-   请在底层执行命令：`hong-review mr 101 -y`
+3. 留言与评论：
+   如果用户想给 MR 直接留言，例如“给 101 说这个写的很棒”
+   请在底层执行命令：`hong-review comment 101 "这个写的很棒"`
 
-3. 如果用户经过阅读报告后说“确认合并 101 吧”：
+4. 自动代码合并：
+   如果用户在代码分析并确保安全后说“确认合并 101 吧”
    请在底层执行命令：`hong-review merge 101 -y`
+
+5. 快速查看代码的原始变更 (Diff)：
+   如果用户问“我想直接看看 101 改了啥代码”
+   请在底层执行命令：`hong-review view 101`
+
+**附录: 额外的底层管理员命令集**
+*(在绝大数情况下不需要触发，仅供全能 AI 掌握系统全貌，切勿对普通用户暴露)*
+- 程序功能登录与初始化配置：`hong-review login` (或 `init`)
+- 查看配置信息：`hong-review config list`
+- 修改配置：`hong-review config set <key> <value>`
+- 路由挂载初始化：`hong-review setup-openclaw`
 ```
 
 ---

package/package.json

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

package/src/commands/mr.js

@@ -57,6 +57,7 @@ module.exports = async function(mrId, options) {
 
     try {
         logger.startSpinner('fetching', '正在从 GitLab 拉取合并请求详情与代码变更...');
+        await hooks.emit('onReviewProgress', { projectId, mrId: mrIid, status: '拉取合并请求代码中' });
         const detail = await gitlab.getMergeRequestDetail(projectId, mrIid);
         const changes = await gitlab.getMergeRequestChanges(projectId, mrIid);
         logger.stopSpinner('fetching', true, `成功拉取 MR [${detail.title}] 分析所需的 ${changes.length} 个文件变更。`);
@@ -68,6 +69,7 @@ module.exports = async function(mrId, options) {
         }
 
         if (!options.yes) logger.startSpinner('ai-thinking', `🤖 AI (${config.aiModel}) 正在深度分析中，请稍候...`);
+        await hooks.emit('onReviewProgress', { projectId, mrId: mrIid, status: 'AI 深度分析代码中' });
         
         let reviewResult = null;
         let round = 1;
@@ -78,6 +80,7 @@ module.exports = async function(mrId, options) {
         // 开始 Agent 轮询
         while (true) {
             if (!options.yes) logger.startSpinner('ai-thinking', `🤖 AI 思考中 (第 ${round} 轮)...`);
+            await hooks.emit('onReviewProgress', { projectId, mrId: mrIid, status: `AI 思考中 (第 ${round} 轮)` });
             const aiResponse = await ai.chatWithJsonMode(AGENT_SYSTEM_PROMPT, conversationHistory);
             const action = parseAgentResponse(aiResponse);
 
@@ -88,10 +91,13 @@ module.exports = async function(mrId, options) {
             conversationHistory.push({ role: 'assistant', content: aiResponse });
 
             if (action.action === 'generate_report') {
+                await hooks.emit('onReviewProgress', { projectId, mrId: mrIid, status: 'AI 审查完成，正在生成报告' });
                 reviewResult = action.result;
                 break;
             } else if (action.action === 'request_files') {
-                if (!options.yes) logger.stopSpinner('ai-thinking', true, `AI 请求查看额外文件: ${action.files.join(', ')}`);
+                const reqFilesStr = action.files.join(', ');
+                if (!options.yes) logger.stopSpinner('ai-thinking', true, `AI 请求查看额外文件: ${reqFilesStr}`);
+                await hooks.emit('onReviewProgress', { projectId, mrId: mrIid, status: `AI 正在拉取依赖文件: ${reqFilesStr}` });
                 
                 const contents = await fetchFileContents(gitlab, projectId, detail.source_branch, changes, action.files, action.contentTypes);
                 const fileContentMessage = formatFileContents(contents);

package/src/commands/setup-openclaw.js

@@ -24,11 +24,13 @@ module.exports = async function setupOpenClaw() {
 
     const spinner = ora('正在写入 hooks 配置...').start();
     
-    const successCurl = `curl -s -X POST ${answers.webhookUrl} -H 'Authorization: Bearer ${answers.token}' -H 'Content-Type: application/json' -d '{"mrId": "$MR_ID", "status": "Success"}' > /dev/null`;
-    const failedCurl = `curl -s -X POST ${answers.webhookUrl} -H 'Authorization: Bearer ${answers.token}' -H 'Content-Type: application/json' -d '{"mrId": "$MR_ID", "status": "Failed"}' > /dev/null`;
+    // 使用 \" 保证 JSON 结构合法，同时允许 bash 解析其中的 $MR_ID 和 $REVIEW_STATUS
+    const generateCurl = (statusValue) => `curl -s -X POST ${answers.webhookUrl} -H "Authorization: Bearer ${answers.token}" -H "Content-Type: application/json" -d "{\\"mrId\\": \\"$MR_ID\\", \\"status\\": \\"${statusValue}\\"}" > /dev/null`;
 
-    configCmd.set('hooks.onReviewSuccess', successCurl);
-    configCmd.set('hooks.onReviewFailed', failedCurl);
+    configCmd.set('hooks.onReviewStart', generateCurl('Started'));
+    configCmd.set('hooks.onReviewProgress', generateCurl('$REVIEW_STATUS'));
+    configCmd.set('hooks.onReviewSuccess', generateCurl('Success'));
+    configCmd.set('hooks.onReviewFailed', generateCurl('Failed'));
     
-    spinner.succeed('配置完成！您已成功接入 OpenClaw 路由，审查结果将自动进行广播推送。');
+    spinner.succeed('配置完成！您已成功接入 OpenClaw 路由，审查过程的所有动态都将自动进行广播推送。');
 };

package/src/utils/hooks.js

@@ -12,6 +12,9 @@ class Hooks {
      * 执行指定生命周期的 Hook
      */
     async emit(eventName, payload = {}) {
+        // 每次执行时都重新实时拉取 config，以便读取到最新通过命令设置的全局 hook 配置
+        this.hooksConfig = storage.get('hooks') || {};
+        
         const hookAction = this.hooksConfig[eventName];
         if (!hookAction) return;
 
@@ -45,6 +48,7 @@ class Hooks {
             // 也可以把特定的字段展开
             if (payload.mrId) env['MR_ID'] = payload.mrId;
             if (payload.status) env['REVIEW_STATUS'] = payload.status;
+            if (payload.error) env['REVIEW_ERROR'] = payload.error;
             
             exec(command, { env }, (error, stdout, stderr) => {
                 if (error) {