@spaceflow/review 0.51.0 → 0.53.0

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## [0.52.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.51.0...@spaceflow/review@0.52.0) (2026-02-28)
+
+### 测试用例
+
+* **review:** 增强 AI 评论识别和过滤功能的测试覆盖 ([bda706b](https://github.com/Lydanne/spaceflow/commit/bda706b99aab113521afe6bcd386a590811e20a6))
+
+## [0.51.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.50.0...@spaceflow/review@0.51.0) (2026-02-27)
+
+### 其他修改
+
+* **review-summary:** released version 0.20.0 [no ci] ([bb3f815](https://github.com/Lydanne/spaceflow/commit/bb3f81567bf6946964a19b9207b8b9beff690b8a))
+* **review:** 移除 .spaceflow 目录及其配置文件 ([64b310d](https://github.com/Lydanne/spaceflow/commit/64b310d8a77614a259a8d7588a09169626efb3ae))
+* **scripts:** released version 0.20.0 [no ci] ([e1fac49](https://github.com/Lydanne/spaceflow/commit/e1fac49257bf4a5902c5884ec0e054384a7859d6))
+* **shell:** released version 0.20.0 [no ci] ([8b69b53](https://github.com/Lydanne/spaceflow/commit/8b69b5340fe99973add2bea3e7d53f2082d0da54))
+
 ## [0.50.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.49.0...@spaceflow/review@0.50.0) (2026-02-27)
 
 ### 新特性

package/dist/index.js

@@ -2742,12 +2742,24 @@ ${fileChanges || "无"}`;
         return match ? match[1] : null;
     }
     /**
+   * 判断评论是否为 AI 生成的评论（非用户真实回复）
+   * 除 issue-key 标记外，还通过结构化格式特征识别
+   */ isAiGeneratedComment(body) {
+        if (!body) return false;
+        // 含 issue-key 标记
+        if (body.includes("<!-- issue-key:")) return true;
+        // 含 AI 评论的结构化格式特征（同时包含「规则」和「文件」字段）
+        if (body.includes("- **规则**:") && body.includes("- **文件**:")) return true;
+        return false;
+    }
+    /**
    * 同步评论回复到对应的 issues
    * review 评论回复是通过同一个 review 下的后续评论实现的
    *
    * 通过 AI 评论 body 中嵌入的 issue key（`<!-- issue-key: file:line:ruleId -->`）精确匹配 issue：
    * - 含 issue key 的评论是 AI 自身评论，过滤掉不作为回复
-   * - 不含 issue key 的评论是用户真实回复，归到其前面最近的 AI 评论对应的 issue
+   * - 不含 issue key 但匹配 AI 格式特征的评论也视为 AI 评论，过滤掉
+   * - 其余评论是用户真实回复，归到其前面最近的 AI 评论对应的 issue
    */ async syncRepliesToIssues(_owner, _repo, _prNumber, reviewComments, result) {
         try {
             // 构建 issue key → issue 的映射，用于快速查找
@@ -2776,12 +2788,17 @@ ${fileChanges || "无"}`;
                 // 遍历评论，用 issue key 精确匹配
                 let lastIssueKey = null;
                 for (const comment of comments){
-                    const issueKey = this.extractIssueKeyFromBody(comment.body || "");
+                    const commentBody = comment.body || "";
+                    const issueKey = this.extractIssueKeyFromBody(commentBody);
                     if (issueKey) {
-                        // AI 自身评论，记录 issue key 但不作为回复
+                        // AI 自身评论（含 issue-key），记录 issue key 但不作为回复
                         lastIssueKey = issueKey;
                         continue;
                     }
+                    // 跳过不含 issue-key 但匹配 AI 格式特征的评论（如其他轮次的 bot 评论）
+                    if (this.isAiGeneratedComment(commentBody)) {
+                        continue;
+                    }
                     // 用户真实回复，通过前面最近的 AI 评论的 issue key 精确匹配
                     let matchedIssue = lastIssueKey ? issueByKey.get(lastIssueKey) ?? null : null;
                     // 回退：如果 issue key 匹配失败，使用 path:position 匹配

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spaceflow/review",
-  "version": "0.51.0",
+  "version": "0.53.0",
   "description": "Spaceflow 代码审查插件，使用 LLM 对 PR 代码进行自动审查",
   "license": "MIT",
   "author": "Lydanne",
@@ -28,7 +28,7 @@
     "@spaceflow/cli": "0.38.0"
   },
   "peerDependencies": {
-    "@spaceflow/core": "0.18.0"
+    "@spaceflow/core": "0.19.0"
   },
   "spaceflow": {
     "type": "flow",

package/src/review.service.spec.ts

@@ -2499,6 +2499,29 @@ describe("ReviewService", () => {
     });
   });
 
+  describe("ReviewService.isAiGeneratedComment", () => {
+    it("should detect comment with issue-key marker", () => {
+      const body = `🟡 **问题**\n<!-- issue-key: test.ts:10:Rule1 -->`;
+      expect((service as any).isAiGeneratedComment(body)).toBe(true);
+    });
+
+    it("should detect comment with structured AI format (规则 + 文件)", () => {
+      const body = ` **魔法字符串问题**\n- **文件**: \`test.ts:64-98\`\n- **规则**: \`JsTs.Base.NoMagicStringsAndNumbers\` (来自 \`js&ts.base.md\`)`;
+      expect((service as any).isAiGeneratedComment(body)).toBe(true);
+    });
+
+    it("should return false for normal user reply", () => {
+      expect((service as any).isAiGeneratedComment("这个问题已经修复了")).toBe(false);
+      expect((service as any).isAiGeneratedComment("LGTM")).toBe(false);
+      expect((service as any).isAiGeneratedComment("")).toBe(false);
+    });
+
+    it("should return false for partial match (only 规则 or only 文件)", () => {
+      expect((service as any).isAiGeneratedComment("- **规则**: something")).toBe(false);
+      expect((service as any).isAiGeneratedComment("- **文件**: something")).toBe(false);
+    });
+  });
+
   describe("ReviewService.syncRepliesToIssues", () => {
     it("should sync user replies to matched issues and filter out AI comments", async () => {
       mockReviewSpecService.parseLineRange = vi.fn().mockReturnValue([10]);
@@ -2618,6 +2641,44 @@ describe("ReviewService", () => {
       // 两条都不含 issue key，都会通过 fallback path:position 匹配
       expect(result.issues[0].replies).toHaveLength(2);
     });
+
+    it("should filter out bot comments with AI structured format but without issue-key", async () => {
+      mockReviewSpecService.parseLineRange = vi.fn().mockReturnValue([10]);
+      const reviewComments = [
+        {
+          id: 1,
+          path: "test.ts",
+          position: 10,
+          body: `🟡 **问题描述**\n<!-- issue-key: test.ts:10:JsTs.Base.ComplexFunc -->`,
+          user: { id: 1, login: "bot" },
+          created_at: "2024-01-01T01:00:00Z",
+        },
+        {
+          id: 2,
+          path: "test.ts",
+          position: 10,
+          body: ` **魔法字符串问题**\n- **文件**: \`test.ts:64-98\`\n- **规则**: \`JsTs.Base.NoMagicStringsAndNumbers\` (来自 \`js&ts.base.md\`)\n- **Commit**: 3390baa\n- **建议**:\n\`\`\`ts\nconst UNKNOWN = '未知';\n\`\`\``,
+          user: { id: 12, login: "GiteaActions" },
+          created_at: "2024-01-01T02:00:00Z",
+        },
+        {
+          id: 3,
+          path: "test.ts",
+          position: 10,
+          body: "已修复，谢谢",
+          user: { id: 5, login: "dev" },
+          created_at: "2024-01-01T03:00:00Z",
+        },
+      ];
+      const result = {
+        issues: [{ file: "test.ts", line: "10", ruleId: "JsTs.Base.ComplexFunc" } as any],
+      };
+      await (service as any).syncRepliesToIssues("o", "r", 1, reviewComments, result);
+      // bot 的结构化评论应被过滤，只保留用户的真实回复
+      expect(result.issues[0].replies).toHaveLength(1);
+      expect(result.issues[0].replies[0].body).toBe("已修复，谢谢");
+      expect(result.issues[0].replies[0].user.login).toBe("dev");
+    });
   });
 
   describe("ReviewService.execute - CI with existingResult", () => {

package/src/review.service.ts

@@ -2269,13 +2269,27 @@ ${fileChanges || "无"}`;
     return match ? match[1] : null;
   }
 
+  /**
+   * 判断评论是否为 AI 生成的评论（非用户真实回复）
+   * 除 issue-key 标记外，还通过结构化格式特征识别
+   */
+  protected isAiGeneratedComment(body: string): boolean {
+    if (!body) return false;
+    // 含 issue-key 标记
+    if (body.includes("<!-- issue-key:")) return true;
+    // 含 AI 评论的结构化格式特征（同时包含「规则」和「文件」字段）
+    if (body.includes("- **规则**:") && body.includes("- **文件**:")) return true;
+    return false;
+  }
+
   /**
    * 同步评论回复到对应的 issues
    * review 评论回复是通过同一个 review 下的后续评论实现的
    *
    * 通过 AI 评论 body 中嵌入的 issue key（`<!-- issue-key: file:line:ruleId -->`）精确匹配 issue：
    * - 含 issue key 的评论是 AI 自身评论，过滤掉不作为回复
-   * - 不含 issue key 的评论是用户真实回复，归到其前面最近的 AI 评论对应的 issue
+   * - 不含 issue key 但匹配 AI 格式特征的评论也视为 AI 评论，过滤掉
+   * - 其余评论是用户真实回复，归到其前面最近的 AI 评论对应的 issue
    */
   protected async syncRepliesToIssues(
     _owner: string,
@@ -2318,12 +2332,17 @@ ${fileChanges || "无"}`;
         // 遍历评论，用 issue key 精确匹配
         let lastIssueKey: string | null = null;
         for (const comment of comments) {
-          const issueKey = this.extractIssueKeyFromBody(comment.body || "");
+          const commentBody = comment.body || "";
+          const issueKey = this.extractIssueKeyFromBody(commentBody);
           if (issueKey) {
-            // AI 自身评论，记录 issue key 但不作为回复
+            // AI 自身评论（含 issue-key），记录 issue key 但不作为回复
             lastIssueKey = issueKey;
             continue;
           }
+          // 跳过不含 issue-key 但匹配 AI 格式特征的评论（如其他轮次的 bot 评论）
+          if (this.isAiGeneratedComment(commentBody)) {
+            continue;
+          }
           // 用户真实回复，通过前面最近的 AI 评论的 issue key 精确匹配
           let matchedIssue = lastIssueKey ? (issueByKey.get(lastIssueKey) ?? null) : null;
           // 回退：如果 issue key 匹配失败，使用 path:position 匹配