job51-gitlab-cr-node-jt-1 2.3.8 → 2.4.0

package/.claude/rules/code-review-rules.md

@@ -241,6 +241,11 @@
 
 18. **输出格式**：严格参照 SKILL.md 模板，无某类问题时省略对应部分；
     > - **必须输出 `<LINE_INFO>` 标签**：包含所有问题的行号信息
+    > - **LINE_INFO 输出约束**（关键规则）：
+        >   - **仅有存在严重问题时，才允许在 LINE_INFO 中输出行号数据**
+        >   - **无严重问题时，LINE_INFO 必须输出空数组 `[]`**
+        >   - **禁止在无严重问题时输出行号数据**
+        >   - 示例：无问题时输出 `<LINE_INFO>[]</LINE_INFO>`，不得输出具体行号
     > - **LINE_INFO 格式**（与 targetLine 结构一致）：
         ```json
         [{"new_path":"文件路径","new_line":行号,"old_path":"文件路径","old_line":行号}]
@@ -277,3 +282,7 @@
         >   - **禁止编造 diff 中没有的代码**，如报告中不得出现 diff 中完全不存在的条件判断、方法调用等
         >   - **验证方法**：将错误代码示例与 diff 内容逐行对比，确认每行代码都能对应上
     > - [ ] **最终自检**：如果这段代码是安全的/正确的，我为什么要报告它？如无法回答，省略该问题
+    > - [ ] **LINE_INFO 输出验证**（关键规则）：
+        >   - **仅在有严重问题时，才允许在 LINE_INFO 中输出行号数据**
+        >   - **无严重问题时，LINE_INFO 必须输出 `[]`**
+        >   - **输出前检查**：是否有"🔴 严重问题"标题？如无，则 LINE_INFO 必须为空数组

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "job51-gitlab-cr-node-jt-1",
-  "version": "2.3.8",
+  "version": "2.4.0",
   "description": "GitLab merge request code review tool with AI-powered analysis and project context support",
   "main": "index.js",
   "bin": {

package/patch.js

@@ -1,81 +0,0 @@
-#!/usr/bin/env node
-const fs = require('fs');
-
-const content = fs.readFileSync('index.js', 'utf-8');
-
-const oldCode = `debugLog(\`本地 AI 命令审核完成，AI 审核结果：\${claudeResult}\`);
-
-        // 检查结果是否包含"🤖 AI 代码审查结果"，如果包含则返回结果
-        if (claudeResult && claudeResult.includes('🤖 AI 代码审查结果')) {
-          debugLog(\`AI 审核成功，包含"🤖 AI 代码审查结果" (尝试 \${attempts})\`);
-
-
-          // 提取 REPORT 标签内容并返回
-          // 提取 REPORT 标签和 LINE_INFO 标签内容并返回对象
-          return extractReportContent(claudeResult);
-        } else {
-          debugLog(\`AI 审核结果不包含"🤖 AI 代码审查结果" (尝试 \${attempts})，将重试...\`);
-          if (attempts >= maxAttempts) {
-            debugLog(\`已达到最大重试次数 \${maxAttempts}，返回最后一次结果\`);
-
-            // 验证报告有效性：检查 LINE_INFO 是否为空且报告内容是否过短
-            const lineInfoEmpty = claudeResult.includes('<LINE_INFO>[]</LINE_INFO>');
-            const reportTooShort = claudeResult.length < 100;
-
-            // 如果 LINE_INFO 为空且报告内容很短，说明无实质问题
-            if (lineInfoEmpty && reportTooShort) {
-              debugLog(\`报告无实质问题，修正为标准无问题格式\`);
-              claudeResult = '<REPORT>\\n## 🤖 AI 代码审查结果\\n\\n</REPORT>\\n\\n<LINE_INFO>\\n[]\\n</LINE_INFO>';
-            }
-
-            return extractReportContent(claudeResult);
-          }
-        }`;
-
-const newCode = `debugLog(\`本地 AI 命令审核完成，AI 审核结果长度：\${claudeResult?.length || 0}\`);
-
-        // 提取关键标记
-        const hasLineInfoTag = claudeResult && claudeResult.includes('<LINE_INFO>') && claudeResult.includes('</LINE_INFO>');
-        const hasNonEmptyLineInfo = hasLineInfoTag && !claudeResult.includes('<LINE_INFO>[]</LINE_INFO>');
-        const hasEmptyLineInfo = hasLineInfoTag && claudeResult.includes('<LINE_INFO>[]</LINE_INFO>');
-        const hasReportTag = claudeResult && claudeResult.includes('<REPORT>') && claudeResult.includes('</REPORT>');
-
-        // 情况 1：LINE_INFO 有行号信息 → 说明 AI 发现了问题，直接接受
-        if (hasNonEmptyLineInfo) {
-          debugLog(\`LINE_INFO 包含行号信息，AI 发现了问题，直接接受结果 (尝试 \${attempts})\`);
-          return extractReportContent(claudeResult);
-        }
-
-        // 情况 2：LINE_INFO 为空数组且有 REPORT 标签 → 无问题，直接接受（不重试）
-        if (hasEmptyLineInfo && hasReportTag) {
-          debugLog(\`LINE_INFO 为空数组，AI 报告无问题，直接接受结果 (尝试 \${attempts})\`);
-          return extractReportContent(claudeResult);
-        }
-
-        // 情况 3：只有 REPORT 标签，无 LINE_INFO → 标准化后接受
-        if (hasReportTag && !hasLineInfoTag) {
-          debugLog(\`报告包含基本标签结构但缺少 LINE_INFO，补充后接受 (尝试 \${attempts})\`);
-          claudeResult = claudeResult + '\\n\\n<LINE_INFO>[]</LINE_INFO>';
-          return extractReportContent(claudeResult);
-        }
-
-        // 情况 4：没有任何有效结构 → 重试
-        debugLog(\`AI 审核结果无有效报告结构 (尝试 \${attempts})，将重试...\`);
-        if (attempts >= maxAttempts) {
-          debugLog(\`已达到最大重试次数 \${maxAttempts}，返回标准无问题格式\`);
-          claudeResult = '<REPORT>\\n## 🤖 AI 代码审查结果\\n\\n</REPORT>\\n\\n<LINE_INFO>[]</LINE_INFO>';
-          return extractReportContent(claudeResult);
-        }`;
-
-if (content.includes(oldCode)) {
-  const newContent = content.replace(oldCode, newCode);
-  fs.writeFileSync('index.js', newContent, 'utf-8');
-  console.log('✅ index.js 更新成功');
-} else {
-  console.log('❌ 未找到目标代码');
-  console.log('当前文件内容片段:');
-  const idx = content.indexOf('AI 审核结果');
-  if (idx !== -1) {
-    console.log(content.substring(idx, idx + 200));
-  }
-}