@raolin2025/claude-code-node 2.1.0 → 2.2.1

package/README.md

@@ -111,6 +111,7 @@ cc-node --resume session-1747000000000-abc123
 | **Grep** | 内容搜索（rg/grep） | `always-allow` | — |
 | **WebFetch** | 抓取网页内容 | `ask` | ✅ SSRF 防护 |
 | **WebSearch** | 网页搜索 | `ask` | 需要 API Key |
+| **GitTool** | GitHub PR 自动化（审查/合并/评论） | `high` | ✅ 预检查 |
 | **AskUserQuestion** | 向用户提问 | `always-allow` | — |
 
 [![npm version](https://img.shields.io/npm/v/@raolin2025/claude-code-node.svg)](https://www.npmjs.com/package/@raolin2025/claude-code-node) [![GitHub](https://img.shields.io/badge/GitHub-bg1avd%2Fclaude--code--node-blue)](https://github.com/bg1avd/claude-code-node) [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
@@ -429,3 +430,119 @@ sudo systemctl start cc-notify
 # 查看日志
 journalctl -u cc-notify -f
 ```
+
+## 🔧 GitTool - PR 自动化管理
+
+GitTool 是 GitHub PR 自动化管理工具，支持查看、审查、合并 PR，以及批量操作和智能分析。
+
+### 前置配置
+
+```bash
+# 1. 创建 GitHub Personal Access Token (classic, 有 repo 权限)
+export GITHUB_TOKEN=ghp_xxx
+
+# 2. 设置仓库信息
+export GITHUB_OWNER=your_org
+export GITHUB_REPO=your_repo
+
+# 可选：启用 LLM 智能分析
+export DEEPSEEK_API_KEY=sk-xxx
+export DEEPSEEK_API_BASE=https://api.deepseek.com/v1
+```
+
+或在 `~/.claude-code/config.json` 中配置：
+
+```json
+{
+  "github": {
+    "owner": "your_org",
+    "repo": "your_repo"
+  },
+  "llm": {
+    "apiKey": "sk-xxx",
+    "apiBase": "https://api.deepseek.com/v1",
+    "model": "deepseek-chat"
+  },
+  "reviewRules": {
+    "checks": {
+      "codeQuality": true,
+      "security": true,
+      "tests": true,
+      "docs": true
+    }
+  }
+}
+```
+
+### 使用示例
+
+在 REPL 中调用 GitTool：
+
+```
+/GitTool list-prs
+/GitTool review-pr 123 --auto-comment false
+/GitTool check-mergeable 123
+/GitTool approve 123 "Looks good"
+/GitTool merge-pr 123 --method squash
+```
+
+命令行脚本（`test-git-tool.mjs`）:
+
+```bash
+# 列出 PR
+node test-git-tool.mjs list
+
+# 审查 PR（规则检查，无 LLM）
+node test-git-tool.mjs review 123
+
+# 使用 LLM 智能分析
+DEEPSEEK_API_KEY=sk-xxx node test-git-tool.mjs review-llm 123
+
+# 检查可合并性
+node test-git-tool.mjs check-mergeable 123
+
+# Approve PR
+node test-git-tool.mjs approve 123 "Approved"
+
+# 提交审查意见
+node test-git-tool.mjs comment 123 "Please add tests"
+
+# 行级评论（自动计算 diff position）
+node test-git-tool.mjs comment 123 "Fix needed" --path src/index.js --line 42
+```
+
+### 自动化工作流
+
+- **每日自动审查**: 设置 cron 运行 `auto-review-all`
+- **自动合并**: 为满足条件的 PR 添加 `auto-merge` 标签，运行 `auto-merge-eligible`
+- **集成 OpenClaw Heartbeat**: 通过 `cron` 工具定期执行
+
+```bash
+# 每天 10:00 自动审查所有 PR
+cron add "0 10 * * *" "session:git-automation" \
+  --payload '{"kind":"agentTurn","message":"/GitTool auto-review-all"}'
+```
+
+### 合并策略
+
+PRMergePolicy 支持以下检查（可配置）：
+
+- ✅ 最少 Approvals 数量
+- ✅ CI 状态全部通过
+- ✅ 无 `changes_requested`
+- ✅ 分支保护规则
+- ✅ 自动合并标签（如 `auto-merge`）
+- ✅ 合并冲突检测
+
+### 安全与权限
+
+- GitTool 工具注册为 `high` 权限级别（合并操作需要确认）
+- 操作会被记录在审计日志中
+- 禁止合并到受保护分支（main/master）
+
+### 与 OpenClaw 集成
+
+- GitTool 已内置到 cc-node 工具集中
+- 可通过 `/tools` 查看
+- 审查结果可与 QQ/Telegram 通道集成，发送通知
+

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@raolin2025/claude-code-node",
-  "version": "2.1.0",
+  "version": "2.2.1",
   "description": "Node.js AI Code Agent CLI - Zero dependencies, pure JavaScript, security hardened, multi-channel notifications",
   "type": "module",
   "main": "src/index.js",

package/src/__tests__/git-tool.integration.test.js

@@ -0,0 +1,89 @@
+/**
+ * GitTool 集成测试
+ * 测试工具在真实环境中的集成情况
+ */
+
+import { test, describe, beforeEach } from 'node:test'
+import assert from 'node:assert'
+
+// 模拟 GitHub API 响应
+function mockGitHubAPI() {
+  return {
+    token: 'mock-token',
+    owner: 'test-owner',
+    repo: 'test-repo',
+    baseUrl: 'https://api.github.com',
+    request: async (endpoint, options = {}) => {
+      if (endpoint.includes('/pulls')) {
+        return [{ number: 1, title: 'Test PR', state: 'open', user: { login: 'tester' } }]
+      }
+      if (endpoint.includes('/pulls/1')) {
+        return {
+          number: 1,
+          title: 'Test PR',
+          body: 'Test',
+          user: { login: 'tester' },
+          head: { ref: 'feature', sha: 'abc123' },
+          base: { ref: 'main' },
+          mergeable: true,
+          changed_files: 1,
+          additions: 1,
+          deletions: 0,
+          labels: []
+        }
+      }
+      return {}
+    }
+  }
+}
+
+// 导入 GitTool 类
+import { GitTool } from '../tools/git-tool.js'
+
+describe('GitTool Integration', () => {
+  let tool
+
+  beforeEach(() => {
+    // 创建 GitTool 实例，使用 mock 配置
+    tool = new GitTool({
+      owner: 'test-owner',
+      repo: 'test-repo',
+      token: 'mock-token'
+    })
+    // 注入 mock 的 GitHub API
+    tool.github = mockGitHubAPI()
+  })
+
+  test('should list PRs', async () => {
+    const result = await tool.execute({ action: 'list-prs', limit: 10 })
+    assert.ok(result.count >= 0)
+    assert.ok(Array.isArray(result.prs))
+  })
+
+  test('should get PR details', async () => {
+    const result = await tool.execute({ action: 'get-pr', prNumber: 1 })
+    assert.strictEqual(result.number, 1)
+    assert.ok(result.title.length > 0)
+  })
+
+  test('should check mergeable', async () => {
+    const result = await tool.execute({ action: 'check-mergeable', prNumber: 1 })
+    assert.ok('mergeable' in result)
+    assert.ok('checks' in result)
+    assert.ok('violations' in result)
+  })
+
+  test('should validate parameters before execution', async () => {
+    await assert.rejects(
+      tool.execute({ action: 'list-prs', state: 'invalid' }),
+      /Invalid enum value/
+    )
+  })
+
+  test('should handle unknown action', async () => {
+    await assert.rejects(
+      tool.execute({ action: 'unknown-action' }),
+      /Unknown action/
+    )
+  })
+})

package/src/__tests__/git-tool.test.js

@@ -0,0 +1,144 @@
+/**
+ * GitTool 单元测试
+ */
+
+import { test, describe } from 'node:test'
+import assert from 'node:assert'
+
+import { gitTool, createToolDef, GitTool } from '../tools/git-tool.js'
+import { parseDiff, splitDiffByFile, getPositionInDiff } from '../git/utils/diff-parser.js'
+
+// ============================================
+// Diff Parser Tests
+// ============================================
+describe('Diff Parser', () => {
+  const sampleDiff = `diff --git a/src/index.js b/src/index.js
+--- a/src/index.js
++++ b/src/index.js
+@@ -1,5 +1,5 @@
+ function add(a, b) {
+-  return a + b
++  return a + b + 0
+ }
+ module.exports = { add }
+ `
+
+  test('parseDiff should parse hunk headers', () => {
+    const hunks = parseDiff(sampleDiff)
+    assert.strictEqual(hunks.length, 1)
+    assert.strictEqual(hunks[0].oldStart, 1)
+    assert.strictEqual(hunks[0].newStart, 1)
+  })
+
+  test('splitDiffByFile should return file map with correct file', () => {
+    const fileMap = splitDiffByFile(sampleDiff)
+    assert.ok(fileMap.has('src/index.js'), 'Missing src/index.js in fileMap')
+    const hunks = fileMap.get('src/index.js')
+    assert.ok(Array.isArray(hunks))
+    assert.ok(hunks.length >= 1, `Expected at least 1 hunk, got ${hunks.length}`)
+  })
+
+  test('getPositionInDiff should calculate position for added line', () => {
+    const fileMap = splitDiffByFile(sampleDiff)
+    const hunks = fileMap.get('src/index.js')
+
+    // The line "  return a + b + 0" is at new line 2
+    const position = getPositionInDiff(hunks, 2)
+    assert.ok(position !== null, 'Position should not be null for existing line')
+    assert.ok(position > 1, `Position ${position} should be > 1`)
+  })
+})
+
+// ============================================
+// GitTool ToolDef Tests
+// ============================================
+describe('GitTool ToolDef', () => {
+  test('should have correct name', () => {
+    assert.strictEqual(gitTool.name, 'GitTool')
+  })
+
+  test('should have description', () => {
+    assert.ok(typeof gitTool.description === 'string' && gitTool.description.length > 10)
+  })
+
+  test('should have parameters schema', () => {
+    const params = gitTool.parameters
+    assert.strictEqual(params.type, 'object')
+    assert.ok('action' in params.properties)
+    assert.ok('prNumber' in params.properties)
+  })
+
+  test('should require action parameter', () => {
+    const required = gitTool.parameters.required
+    assert.ok(Array.isArray(required))
+    assert.ok(required.includes('action'))
+  })
+
+  test('should support all expected actions', () => {
+    const actions = gitTool.parameters.properties.action.enum
+    const expected = [
+      'list-prs', 'get-pr', 'review-pr', 'merge-pr',
+      'comment', 'approve', 'request-changes',
+      'check-mergeable', 'auto-review-all', 'auto-merge-eligible'
+    ]
+    expected.forEach(a => assert.ok(actions.includes(a), `Missing action: ${a}`))
+  })
+
+  test('should have high permission level', () => {
+    assert.strictEqual(gitTool.permissionLevel, 'high')
+  })
+
+  test('should have handler function', () => {
+    assert.ok(typeof gitTool.handler === 'function')
+  })
+})
+
+// ============================================
+// GitTool Factory Tests
+// ============================================
+describe('GitTool Factory', () => {
+  test('createToolDef should return ToolDef instance with handler', () => {
+    const toolDef = createToolDef({})
+    assert.ok(toolDef instanceof Object)
+    assert.ok(typeof toolDef.handler === 'function')
+    assert.strictEqual(toolDef.name, 'GitTool')
+  })
+
+  test('GitTool class should be instantiable with config', () => {
+    const instance = new GitTool({ owner: 'test', repo: 'test' })
+    assert.ok(instance instanceof GitTool)
+    assert.ok(typeof instance.execute === 'function')
+  })
+})
+
+// ============================================
+// Parameter Validation Tests
+// ============================================
+describe('Parameter Validation', () => {
+  test('comment action should validate prNumber and body before GitHub init', async () => {
+    // Pass dummy token to bypass GitHub token check early
+    const tool = new GitTool({
+      owner: 'test',
+      repo: 'test',
+      token: 'dummy'
+    })
+
+    await assert.rejects(
+      tool.execute({ action: 'comment' }),
+      /prNumber and body required/
+    )
+  })
+
+  test('merge-pr action should validate prNumber', async () => {
+    const tool = new GitTool({
+      owner: 'test',
+      repo: 'test',
+      token: 'dummy'
+    })
+
+    await assert.rejects(
+      tool.execute({ action: 'merge-pr' }),
+      /prNumber required/
+    )
+  })
+})

package/src/core/query-engine.js

@@ -114,12 +114,12 @@ export class QueryEngine {
       // 没有工具调用 → 最终回复
       if (!response.toolCalls || response.toolCalls.length === 0) {
         finalResponse = response.content
-        this.state.messages.push(new AssistantMessage(response.content))
+        this.state.messages.push(new AssistantMessage(response.content, [], response.reasoningContent))
         break
       }
 
       // 有工具调用 → 记录 assistant 消息（含 tool_calls）
-      this.state.messages.push(new AssistantMessage(response.content, response.toolCalls))
+      this.state.messages.push(new AssistantMessage(response.content, response.toolCalls, response.reasoningContent))
 
       // 执行工具
       const toolResults = await this._executeToolCalls(response.toolCalls)
@@ -168,21 +168,27 @@ export class QueryEngine {
       } else if (msg.role === 'user') {
         request.push({ role: 'user', content: this._formatContent(msg.content) })
       } else if (msg.role === 'assistant') {
-        // 带 tool_calls 的 assistant 消息：OpenAI 格式
+        // 构建 assistant 消息基础
+        const asstMsg = {
+          role: 'assistant',
+          content: msg.content || null,
+        }
+
+        // DeepSeek thinking mode: 必须传回 reasoning_content (tool call 场景)
+        if (msg.reasoningContent) {
+          asstMsg.reasoning_content = msg.reasoningContent
+        }
+
+        // tool_calls
         if (msg.toolCalls && msg.toolCalls.length > 0) {
-          request.push({
-            role: 'assistant',
-            content: msg.content || null,
-            tool_calls: msg.toolCalls.map(tc => ({
-              id: tc.id,
-              type: 'function',
-              function: { name: tc.name, arguments: typeof tc.input === 'string' ? tc.input : JSON.stringify(tc.input) },
-            })),
-          })
-        } else {
-          // 纯文本 assistant 消息
-          request.push({ role: 'assistant', content: this._formatContent(msg.content) })
+          asstMsg.tool_calls = msg.toolCalls.map(tc => ({
+            id: tc.id,
+            type: 'function',
+            function: { name: tc.name, arguments: typeof tc.input === 'string' ? tc.input : JSON.stringify(tc.input) },
+          }))
         }
+
+        request.push(asstMsg)
       } else if (msg.role === 'tool') {
         // tool 结果消息 — 直接透传
         request.push({

package/src/core/streaming.js

@@ -16,6 +16,7 @@ export async function* parseStream(response) {
 
   const result = {
     content: '',
+    reasoningContent: '',
     toolCalls: [],
     usage: { input_tokens: 0, output_tokens: 0 },
   }
@@ -40,6 +41,11 @@ export async function* parseStream(response) {
 
           if (!delta) continue
 
+          // reasoning_content (DeepSeek thinking mode)
+          if (delta.reasoning_content) {
+            result.reasoningContent += delta.reasoning_content
+          }
+
           // 文本
           if (delta.content) {
             result.content += delta.content
@@ -95,13 +101,17 @@ export async function* parseStream(response) {
  * 解析非流式响应
  */
 export function parseNonStreamResponse(data) {
-  const result = { content: '', toolCalls: [], usage: {} }
+  const result = { content: '', reasoningContent: '', toolCalls: [], usage: {} }
   const choice = data.choices?.[0]
 
   if (choice?.message?.content) {
     result.content = choice.message.content
   }
 
+  if (choice?.message?.reasoning_content) {
+    result.reasoningContent = choice.message.reasoning_content
+  }
+
   for (const tc of (choice?.message?.tool_calls || [])) {
     let input = {}
     try { input = JSON.parse(tc.function.arguments) } catch {}