apache · yx9o · Jul 14, 2026 · Jul 15, 2026 · Jul 17, 2026 · Jul 17, 2026
diff --git a/.claude/skills/pr-review/SKILL.md b/.claude/skills/pr-review/SKILL.md
@@ -0,0 +1,265 @@
+---
+name: pr-review
+description: RocketMQ Studio 的 PR 评审助手。输入一个 GitHub PR 链接，自动用 gh 拉取 PR 及其关联 Issue，切出本地分支，编译前端与后端，用 docker compose 拉起整个项目（不改端口），并产出结构化的 PR 分析总结。当用户提到 评审 PR、review PR、看一下这个 PR、PR 分析、拉 PR 编译、检查 PR、pr review、审查合并请求 等场景时触发；即使用户只贴了一个 GitHub PR 链接，也应触发此 skill。
+---
+
+# RocketMQ Studio PR 评审
+
+对 RocketMQ Studio（`apache/rocketmq-dashboard`，分支 `rocketmq-studio`）的一个 GitHub PR 做端到端评审：拉取 PR 与关联 Issue → 本地切分支 → 编译前后端 → docker compose 拉起 → 输出结构化分析总结。
+
+## 前置条件
+
+- 已安装并登录 `gh`（`gh auth status` 确认）。
+- 已安装 `docker`（含 `docker compose`）、`node`(>=20)、`npm`、`mvn`(JDK 21)。
+- 当前工作目录为项目根目录（含 `server/`、`web/`、`deploy/`）。
+- 后端构建需要 JDK 21（`JAVA_HOME` 指向 JDK 21，或使用 `mvn -B -ntp` 配合系统 JDK 21）。
+
+> 端口约定（**禁止修改**）：前端 6789（Nginx）、后端 8888（Spring Boot）、NameServer 9876、Broker 10911、Proxy 8080/8081。
+
+## 输入
+
+一个 PR 链接，例如：`https://github.com/apache/rocketmq-dashboard/pull/123`
+
+从链接中解析出 PR 编号 `<PR>`（URL 最后一段数字）。
+
+## 标准流程（Pipeline）
+
+评审按以下 8 个阶段顺序执行。每个阶段独立产出结果，任一阶段失败不阻断后续步骤，但需在总结中标记 ❌。
+
+```
+Stage 1  拉取元信息        gh pr view → JSON + diff
+Stage 2  标题规范检查       正则校验 [Studio] type: description
+Stage 3  关联 Issue        gh issue view（若有 Closes/Fixes 引用）
+Stage 4  切出评审分支       gh pr checkout → pr-review-<PR>
+Stage 5  预检 & 修复        Dockerfile style/ 目录修复（已知问题）
+Stage 6  编译后端           mvn package -DskipTests（含 checkstyle）
+Stage 7  编译前端           npm ci && npm run build（tsc + vite）
+Stage 8  Docker 部署        docker compose up -d --build + 健康检查
+```
+
+**完成后**：复原工作区（切回原分支 + stash pop）。
+
+---
+
+## 各阶段详细步骤
+
+### Stage 1: 拉取 PR 元信息
+
+```bash
+gh pr view <PR> --repo apache/rocketmq-dashboard \
+  --json number,title,author,state,baseRefName,headRefName,url,body,additions,deletions,changedFiles,labels,commits,files,closingIssuesReferences
+```
+
+重点关注：
+- `baseRefName` 是否为 `rocketmq-studio`（目标分支应为它，否则在总结中提示）。
+- `files` / `changedFiles` / `additions` / `deletions`：变更范围与体量。
+- `closingIssuesReferences`：PR 声明会关闭的 Issue（`Closes #N`）。
+
+同时拉取 diff 供后续分析：
+
+```bash
+gh pr diff <PR> --repo apache/rocketmq-dashboard > /tmp/pr-<PR>.diff
+```
+
+### Stage 2: 检查 PR 标题是否规范
+
+规范来源：`docs/contributing.md`「提交规范」+ `README.md`「Commit Format」+ 本仓库既有 PR/commit 历史，基于 [Conventional Commits](https://www.conventionalcommits.org/) 并带项目前缀。
+
+**格式**：`[Studio] <type>: <description>`
+
+- **`[Studio]`** 为项目前缀（必须，大小写敏感，首字母大写 `Studio`）。
+- **type** 必须为以下之一（小写）：`feat`（新功能）、`fix`（修复 Bug）、`docs`（文档）、`refactor`（重构）、`test`（测试）、`chore`（构建/工具）、`perf`（性能）。
+- `[Studio]` 与 `type` 之间有一个空格；type 后紧跟半角冒号 `:` 加一个空格，再接 **description**。
+- description 用英文小写祈使句，简洁描述改动，结尾不加句号。
+- squash 合并后 GitHub 会在标题末尾追加 ` (#PR号)`，属正常现象，检查时应先剥离该后缀。
+
+校验正则（先去掉可能存在的 ` (#N)` 尾巴）：
+
+```bash
+TITLE=$(gh pr view <PR> --repo apache/rocketmq-dashboard --json title -q .title)
+CLEAN=$(printf '%s' "$TITLE" | sed -E 's/ \(#[0-9]+\)$//')
+if printf '%s' "$CLEAN" | grep -Eq '^\[Studio\] (feat|fix|docs|refactor|test|chore|perf): .+'; then
+  echo "标题规范 ✅: $TITLE"
+else
+  echo "标题不规范 ❌: $TITLE"
+fi
+```
+
+对照参考（本仓库历史）：
+- `[Studio] fix: validate audit query and cleanup parameters` ✅
+- `[Studio] fix: connect K8s certificate page to backend APIs` ✅
+- `[Studio] feat: add i18n language context with useLanguage alias` ✅
+- `[Studio][fix] Connect K8s certificate page to backend APIs` ❌（type 应在冒号前，不应使用 `[fix]` 方括号）
+- `[studio] feat: extend translation keys` ❌（`studio` 应为 `Studio`，首字母大写）
+
+不规范时在总结中明确指出问题并给出建议标题。
+
+### Stage 3: 拉取关联 Issue（若有）
+
+若 `closingIssuesReferences` 非空，或 PR 正文中出现 `#N` / `Closes #N` / `Fixes #N` 引用，逐个拉取：
+
+```bash
+gh issue view <ISSUE> --repo apache/rocketmq-dashboard \
+  --json number,title,state,body,labels,url
+```
+
+用于判断 PR 是否真正解决了 Issue 描述的问题（需求对齐度）。
+
+### Stage 4: 本地切出评审分支
+
+不要污染当前分支。用 `gh` 直接 checkout PR 分支（会自动创建本地分支）：
+
+```bash
+# 记录当前分支以便复原
+ORIGINAL_BRANCH=$(git rev-parse --abbrev-ref HEAD)
+git stash push -u -m "pr-review-stash" 2>/dev/null || true
+
+gh pr checkout <PR> --repo apache/rocketmq-dashboard --branch pr-review-<PR>
+```
+
+若因权限/fork 无法直接 checkout，退化为手动 fetch：
+
+```bash
+git fetch origin pull/<PR>/head:pr-review-<PR>
+git checkout pr-review-<PR>
+```
+
+### Stage 5: 预检 & 修复（Dockerfile style/ 目录）
+
+**已知问题**：`server/Dockerfile` 在基准分支上缺少 `COPY style ./style`，导致 Maven checkstyle 插件在构建阶段找不到 `style/rmq_checkstyle.xml` 而报错，docker compose 无法启动。
+
+每次评审前检查并修复：
+
+```bash
+if ! grep -q 'COPY style' server/Dockerfile; then
+  # 在 "COPY src ./src" 后插入 "COPY style ./style"
+  sed -i '/^COPY src \.\/src$/a COPY style ./style' server/Dockerfile
+  echo "✅ 已修复 Dockerfile: 添加 COPY style ./style"
+else
+  echo "✅ Dockerfile 已包含 style/ 目录复制"
+fi
+```
+
+> 此修复仅用于本地评审，不影响 PR diff。若 PR 本身已修复此问题，此步为 no-op。
+
+### Stage 6: 编译后端
+
+严格对齐 `server/Dockerfile` 的构建方式（`mvn package -DskipTests`，含 checkstyle 校验）：
+
+```bash
+cd server
+mvn -B -ntp clean package -DskipTests
+cd ..
+```
+
+- 编译失败 → 记录报错（编译错误 / checkstyle 违规），标记后端为 ❌，仍继续后续步骤并如实汇报。
+- 通过 → 标记 ✅。
+
+### Stage 7: 编译前端
+
+对齐 `web/Dockerfile`（`npm ci && npm run build`，即 `tsc -b && vite build`）：
+
+```bash
+cd web
+npm ci
+npm run build
+cd ..
+```
+
+- 出现 TypeScript 类型错误或构建失败 → 标记前端 ❌，记录关键报错。
+- 通过 → 标记 ✅。
+
+### Stage 8: Docker 部署
+
+使用项目自带 compose 文件，**不修改任何端口**：
+
+```bash
+cd deploy
+docker compose down 2>/dev/null || true
+docker compose up -d --build
+cd ..
+```
+
+启动后做健康检查（不改端口）：
+
+```bash
+# 前端（Nginx）
+curl -fsS -o /dev/null -w "web=%{http_code}\n" http://127.0.0.1:6789/ || echo "web 未就绪"
+# 后端 actuator（经前端 nginx 反代或容器内），若暴露则直接探测
+curl -fsS -o /dev/null -w "api=%{http_code}\n" http://127.0.0.1:6789/actuator/health || echo "api 未就绪"
+docker compose -f deploy/docker-compose.yml ps
+```
+
+评审结束后清理（询问用户或默认保留，视会话上下文而定）：
+
+```bash
+docker compose -f deploy/docker-compose.yml down
+```
+
+---
+
+## 复原工作区
+
+评审完成后切回原分支并恢复暂存：
+
+```bash
+git checkout "$ORIGINAL_BRANCH"
+git stash pop 2>/dev/null || true
+# 如需删除评审分支：git branch -D pr-review-<PR>
+```
+
+---
+
+## 输出：PR 分析总结
+
+用中文输出一份结构化 Markdown 总结，包含以下部分：
+
+### 1. 概览
+- 标题、作者、状态、源分支 → 目标分支、PR 链接。
+- **PR 标题规范**：✅/❌（不规范时标出具体问题并附建议标题）。
+- 变更体量：`+additions / -deletions`，改动文件数。
+- 关联 Issue：编号、标题、链接（若有）。
+
+### 2. 需求对齐
+- 简述 PR 目标（来自正文）。
+- 若有关联 Issue，逐条比对 Issue 诉求与 PR 实现，判断是否覆盖 / 部分覆盖 / 偏离。
+
+### 3. 构建与运行结果
+用表格汇总：
+
+| 检查项 | 结果 | 说明 |
+|---|---|---|
+| 后端编译 (`mvn package`) | ✅/❌ | 关键报错摘要 |
+| 前端编译 (`npm run build`) | ✅/❌ | 关键报错摘要 |
+| docker compose 启动 | ✅/❌ | 容器状态 / 端口 6789、8888 |
+| 前端健康检查 | ✅/❌ | HTTP 状态码 |
+
+### 4. 变更分析
+- 按模块归类改动（前端页面/组件、后端 controller/service/domain、部署、文档等）。
+- 结合六边形架构（server 用 ArchUnit 约束）判断分层是否合理。
+- i18n：新增前端文案是否中英文双语（`web/src/i18n/`）。
+- 提交规范：commit message 是否符合 `[Studio] type: description` 格式。
+
+### 5. 风险与建议
+- 潜在逻辑问题、边界情况、安全风险（如凭据明文、公网暴露）。
+- 缺失的测试 / 文档 / i18n。
+- 明确的改进建议（可执行、可定位到文件）。
+
+### 6. 评审结论
+给出倾向：**Approve** / **Request Changes** / **Comment**，并用一句话说明理由。
+
+---
+
+## 自由发挥补充能力
+
+- **变更规模自适应**：大 PR（改动文件多）先按目录聚合概述再抽样精读核心文件；小 PR 可逐文件过。
+- **checkstyle / lint 单独复核**：后端 `mvn checkstyle:check`，前端 `npm run lint`，把风格问题与逻辑问题分开汇报。
+- **失败即止但汇报完整**：任一步失败不阻断总结，如实记录并继续能做的检查。
+- **可选发布评审意见**：用户明确要求时，可用 `gh pr comment <PR> --repo apache/rocketmq-dashboard --body-file <file>` 或 `gh pr review <PR> --comment/--approve/--request-changes` 提交（默认只本地产出，不自动发布）。
+
+## 注意事项
+
+- **不修改端口**：任何环节都使用既有端口，不得改 compose / nginx / env 中的端口映射。
+- **不污染主分支**：评审在独立分支进行，结束后复原。
+- **只读默认**：默认不向 GitHub 写入评论/评审，除非用户明确要求。
+- **凭据安全**：分析 diff 时若发现 AK/SK、password、token 等明文凭据，作为高风险项在总结中显著标注。
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -0,0 +1,64 @@
+name: CI
+
+# 检测前端与后端是否都能正确编译
+on:
+  push:
+    branches:
+      - rocketmq-studio
+  pull_request:
+    branches:
+      - rocketmq-studio
+
+jobs:
+  backend-build:
+    name: Backend Build (Java 21)
+    runs-on: ubuntu-latest
+    defaults:
+      run:
+        working-directory: server
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Set up JDK 21 (Dragonwell)
+        uses: actions/setup-java@v4
+        with:
+          distribution: dragonwell
+          java-version: "21"
+          cache: maven
+
+      - name: Build backend
+        run: mvn -B -ntp clean package -DskipTests
+
+  frontend-build:
+    name: Frontend Build (Node 20)
+    runs-on: ubuntu-latest
+    defaults:
+      run:
+        working-directory: web
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Set up Node 20
+        uses: actions/setup-node@v4
+        with:
+          node-version: "20"
+          cache: npm
+          cache-dependency-path: web/package-lock.json
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Build frontend
+        run: npm run build
+
+  frontend-docker-build:
+    name: Frontend Docker Build
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Build frontend Docker image
+        run: docker build -t rocketmq-studio-web-ci ./web
diff --git a/deploy/deploy.sh b/deploy/deploy.sh
@@ -10,6 +10,18 @@ set -euo pipefail
 SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
 PROJECT_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
 
+# ─── 颜色输出 ───
+RED='\033[0;31m'
+GREEN='\033[0;32m'
+YELLOW='\033[1;33m'
+CYAN='\033[0;36m'
+NC='\033[0m'
+
+log()  { echo -e "${GREEN}[✓]${NC} $*"; }
+info() { echo -e "${CYAN}[→]${NC} $*"; }
+warn() { echo -e "${YELLOW}[!]${NC} $*"; }
+err()  { echo -e "${RED}[✗]${NC} $*"; exit 1; }
+
 # 加载配置
 if [[ -f "$SCRIPT_DIR/.env" ]]; then
   set -a
@@ -26,18 +38,6 @@ TMP_DIR="/tmp/rocketmq-studio-deploy"
 
 TARGET="${1:-all}"  # all | server | web
 
-# ─── 颜色输出 ───
-RED='\033[0;31m'
-GREEN='\033[0;32m'
-YELLOW='\033[1;33m'
-CYAN='\033[0;36m'
-NC='\033[0m'
-
-log()  { echo -e "${GREEN}[✓]${NC} $*"; }
-info() { echo -e "${CYAN}[→]${NC} $*"; }
-warn() { echo -e "${YELLOW}[!]${NC} $*"; }
-err()  { echo -e "${RED}[✗]${NC} $*"; exit 1; }
-
 # ─── 前置检查 ───
 check_prereqs() {
   info "检查前置条件..."