Merge pull request #32 from taptap/feat/code-review-rules-and-engine-redesign

chore(release): prepare for v0.1.21 release

Author: em0t

.claude-plugin/marketplace.json

@@ -6,7 +6,7 @@
   },
   "metadata": {
     "description": "TapTap Claude Code 插件库 - 提供开发工作流自动化插件",
-    "version": "0.1.20",
+    "version": "0.1.22",
     "pluginRoot": "./plugins"
   },
   "plugins": [
@@ -25,8 +25,8 @@
     {
       "name": "git",
       "source": "./plugins/git",
-      "description": "Git 工作流命令插件，提供提交/推送/MR、远程平台操作、自动代码审查（Agent Team + Codex）",
-      "version": "0.1.9",
+      "description": "Git 工作流命令插件，提供提交/推送/MR、远程平台操作、自动代码审查（Agent Team 审查+辩论 + Codex 双视角），支持项目自定义审查规则",
+      "version": "0.1.11",
       "author": {
         "name": "TapTap AI Team"
       },
@@ -38,7 +38,7 @@
       "name": "sync",
       "source": "./plugins/sync",
       "description": "项目配置同步插件，提供 MCP、LSP 和开发环境配置同步功能",
-      "version": "0.1.16",
+      "version": "0.1.18",
       "author": {
         "name": "TapTap AI Team"
       },

CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## 0.1.21
+
+### Git Plugin (0.1.10)
+
+- Added project-level custom review rules support (review-rules.md with scope-based matching)
+- Redesigned code review engine: Agent Team with debate phase (2 members review independently then cross-validate)
+- Added review checklist loading with project customization support (shared with CI reviewer)
+- Updated plugin description to reflect new review architecture
+
+### Sync Plugin (0.1.17)
+
+- Added review-rules template sync to skills-sync agent (copies review-rules.md if not exists)
+- Added review-rules to `/sync:basic` output and override policy docs
+- Added review-rules.md template to sync plugin skills directory
+
+### Marketplace
+
+- Bumped version from 0.1.20 to 0.1.21
+- Updated git plugin to version 0.1.10
+- Updated sync plugin to version 0.1.17
+- Removed ralph plugin (still in development)
+
 ## 0.1.20
 
 ### Git Plugin (0.1.9)

README.md

@@ -85,8 +85,8 @@ mkdir -p .claude && echo '{
 | 插件      | 版本    | 描述                                                                |
 | ------- | ----- | ----------------------------------------------------------------- |
 | spec    | 0.1.4 | Spec-Driven Development 工作流插件                                     |
-| git     | 0.1.9 | Git 工作流自动化插件（提交/推送/MR + 自动代码审查 + 远程平台操作）            |
-| sync    | 0.1.16 | 开发环境配置同步插件（MCP + LSP + Hooks + Cursor + Claude Skills） |
+| git     | 0.1.10 | Git 工作流自动化插件（提交/推送/MR + 自动代码审查 + 远程平台操作）            |
+| sync    | 0.1.17 | 开发环境配置同步插件（MCP + LSP + Hooks + Cursor + Claude Skills） |
 | quality | 0.0.4 | AI 驱动的代码质量检查插件（9 个并行 Agent，支持 Bug 检测、代码质量、安全检查、性能分析）              |
 
 

plugins/git/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "git",
-  "description": "Git 工作流命令插件，提供提交/推送/MR 创建、远程平台操作（GitHub/GitLab）、自动代码审查（Agent Team + Codex 双引擎），支持智能任务 ID 提取和规范化提交信息",
-  "version": "0.1.9",
+  "description": "Git 工作流命令插件，提供提交/推送/MR 创建、远程平台操作（GitHub/GitLab）、自动代码审查（Agent Team 审查+辩论 + Codex 双视角），支持项目自定义审查清单",
+  "version": "0.1.11",
   "author": {
     "name": "TapTap AI Team"
   }

plugins/git/skills/code-reviewing/SKILL.md

@@ -0,0 +1,40 @@
+<!--
+  项目专属审查规则（Project Review Rules）
+
+  本文件定义项目特有的代码审查规则，审查引擎会自动加载。
+  如果没有填写任何规则（仅有注释），此文件不会影响审查结果。
+
+  格式说明：
+  每条规则是一个 ## 标题的 section，包含：
+  - 标题 + 严重级别：## 规则名称 (BLOCKING/WARNING/NIT)
+  - **scope**: glob pattern，匹配需要检查的文件路径（支持多行，每行一个 pattern）
+  - 规则描述：具体检查什么
+  - 合规示例 / 违规示例：帮助 reviewer 准确判断
+
+  严重级别说明：
+  - BLOCKING: 必须修复才能合并
+  - WARNING: 建议修复，不阻塞合并
+  - NIT: 风格建议，可忽略
+
+  常见规则类型（按需添加）：
+  - API 字段注释/文档要求（proto 注释、swagger 注解、KDoc/JavaDoc）
+  - 命名约定（包名、类名、方法名规范）
+  - 目录结构约定（新文件应放在哪里）
+  - 依赖管理（禁止引入特定依赖、版本锁定）
+  - 配置文件格式（特定 key 必须存在）
+  - 国际化/本地化要求
+
+  示例规则（取消注释并修改后生效）：
+
+  ## API 字段注释 (BLOCKING)
+
+  **scope**: `src/api/**/*.proto`
+
+  所有 Request/Response message 及其引用的 model，每个字段必须有行尾注释说明含义。
+
+  合规：
+    string name = 1; // 用户昵称
+  违规：
+    string name = 1;
+
+-->

plugins/git/skills/code-reviewing/review-rules.md

@@ -1,7 +1,7 @@
 {
   "name": "sync",
   "description": "项目配置同步插件，提供 MCP、LSP 和开发环境配置同步功能",
-  "version": "0.1.16",
+  "version": "0.1.18",
   "author": {
     "name": "TapTap AI Team"
   }

plugins/sync/.claude-plugin/plugin.json

@@ -344,6 +344,7 @@ chmod +x .githooks/pre-commit
 
 ## 版本历史
 
+- **v0.1.17** - skills-sync 新增 review-rules 模板同步（不覆盖项目已有规则）
 - **v0.1.16** - review-checklist 不覆盖项目自定义版本；修正覆盖策略文档；修复 4 个 hook 脚本缺少可执行权限；Cursor 模板新增 Pipeline Watch
 - **v0.1.15** - 新增 `/sync:lsp` 命令（检测语言+安装 binary+启用插件）；`/sync:basic` LSP 首次即装不再延迟；Cursor 模板新增 code review 步骤（含 --skip-code-review）；修复 MR 模板覆盖问题（原子 bash）；hooks 新增 LSP 脚本；更新 statusline/ensure-golang
 - **v0.1.14** - 修复 10 个命令文件 allowed-tools 缺失问题；补齐 printenv、head、pwd、cp、ls、sort、tail、echo、wc、claude、bash、mv、tr 等命令权限声明；cursor-templates 同步修复

plugins/sync/README.md

@@ -47,11 +47,26 @@ test -f {PROJECT_ROOT}/.claude/skills/code-reviewing/review-checklist.md || \
 
 如果目标文件已存在，记录为 "已存在（跳过）"。
 
+### 4. 复制 review-rules（项目审查规则模板）
+
+如果 CHECKLIST_FOUND（review-rules 在同一目录）：
+```bash
+mkdir -p {PROJECT_ROOT}/.claude/skills/code-reviewing
+# 仅在文件不存在时复制（不覆盖项目已填写的规则）
+test -f {PROJECT_ROOT}/.claude/skills/code-reviewing/review-rules.md || \
+  cp "{SKILLS_DIR}/code-reviewing/review-rules.md" {PROJECT_ROOT}/.claude/skills/code-reviewing/
+```
+
+**重要**：使用 `test -f ... ||` 确保不覆盖项目已有的自定义规则。
+
+如果目标文件已存在，记录为 "已存在（跳过）"。
+
 ## 输出格式（严格遵循）
 
 ## 结果
 - 状态: success / failed / skipped
 - 详情:
   - grafana-dashboard-design: [已复制/源目录不存在（跳过）]
   - review-checklist: [已复制/已存在（跳过）/源目录不存在（跳过）]
+  - review-rules: [已复制/已存在（跳过）/源目录不存在（跳过）]
 - 错误: [如有]

plugins/sync/agents/skills-sync.md

@@ -183,6 +183,7 @@ bash {SCRIPTS_DIR}/detect-lsp.sh "$(pwd)"
   ✅ Claude Skills 同步: 成功
      - grafana-dashboard-design（Grafana Dashboard 设计规范）
      - review-checklist（Code Review 检查清单）[已复制/已存在（跳过）/源目录不存在（跳过）]
+     - review-rules（项目审查规则模板）[已复制/已存在（跳过）/源目录不存在（跳过）]
      - 位置: .claude/skills/
 
   ✅ Status Line 配置: 成功
@@ -337,7 +338,7 @@ bash {SCRIPTS_DIR}/detect-lsp.sh "$(pwd)"
    - **Cursor 同步**：直接覆盖（rules 和 commands 每次重新生成）
    - **Spec Skills**：直接覆盖（每次从 spec 插件重新生成 .mdc 文件）（仅 --with-spec 时执行）
    - **GitLab MR 模板**：已存在则跳过，不覆盖（保留项目自定义配置）
-   - **Claude Skills**：grafana-dashboard-design 直接覆盖；review-checklist 已存在则跳过（保留项目自定义版本）
+   - **Claude Skills**：grafana-dashboard-design 直接覆盖；review-checklist、review-rules 已存在则跳过（保留项目自定义版本）
    - **Status Line**：直接覆盖（每次重新复制脚本并更新配置）
    - 某步骤失败不影响后续步骤