From mj-git
This skill should be used when the user asks to review a Pull Request for architecture compliance, design consistency, or ops domain completeness in MJ System. It should also be invoked when the user pastes a PR URL, mentions a PR number, or asks about code structure issues in a branch. Triggers on "评审PR", "review PR", "审查PR", "PR评审", "review pull request", "这个PR能合吗", "可以merge吗", "帮我看看这个PR", "检查PR架构", "architecture review", "代码结构有没有问题", "检查一下这个分支", "PR review".
npx claudepluginhub mj-agentlab/mj-agentlab-marketplaceThis skill uses the workspace's default tool permissions.
PR 架构评审与条件合并 skill。
Conducts deep PR reviews using 6-7 parallel specialized agents for code quality, security, testing, frontend/backend architecture. Use for thorough pull request analysis before merging.
Reviews GitHub pull requests end-to-end using gh CLI: analyzes diffs, commits, CI/CD checks; provides blocking/suggestion/nit/praise feedback and submits review. Use for assigned PRs, self-reviews, or post-merge audits.
Guides Next.js Cache Components and Partial Prerendering (PPR) with cacheComponents enabled. Implements 'use cache', cacheLife(), cacheTag(), revalidateTag(), static/dynamic optimization, and cache debugging.
Share bugs, ideas, or general feedback.
PR 架构评审与条件合并 skill。
核心价值:回答"这个 PR 应不应该合"——检查架构合规、设计一致性、ops 域完整性。
与 mj-git-check-merge(回答"能不能合"——冲突、CI、审批)互补。
| Skill | 关系 | 边界 |
|---|---|---|
| mj-git-check-merge | 互补 | check-merge 做技术检查(冲突/CI/审批),review-pr 做架构评审 |
| mj-doc-review | 不重复 | review-pr 检测 docs/ 变更时提示运行 mj-doc-review,不自行做文档检查 |
| mj-git-sync | 不调用 | review-pr 只展示分支落后状态,不执行同步操作 |
| superpowers:code-reviewer | 不替代 | superpowers 做通用代码质量,review-pr 做 MJ 特有架构合规 |
gh CLI 已安装且已认证用户输入 → Skill 行为:
| 输入 | 行为 |
|---|---|
评审 PR #48 | 完整 5 阶段评审 |
帮我看看 feature/xxx | 通过分支名定位 PR,完整评审 |
只看 SQL 变更的 PR #48 | 范围限定:只执行 F1-F2 + D4 |
PR #48 的变更概览 | 快速模式:只执行 Stage 1-2,不做架构评审 |
gh pr view {input} --json number,title,state,baseRefName,headRefName,additions,deletions,changedFiles,commits
STOP & ASK:
gh pr view {number} --json commits --jq '.commits[].messageHeadline'
gh pr diff {number} --stat 2>/dev/null || gh api repos/{owner}/{repo}/pulls/{number}/files --jq '.[].filename'
scripts/classify_changes.py,以下为摘要):
src/**/*.py, components/**/*.py, main.py, scripts/**/*.py, test/**/*.pysql/**/*.sql, sql/**/*.sh, sql/**/*.ps1*.yaml, *.yml, *.toml, .env*, docker-compose*, docker/**, Dockerfile*docs/**/*.md, *.md快速模式中断点: 如果用户只要求"变更概览",到此结束。输出概览后直接进入 Handoff。
读取 review-checklist.md 中的检查项定义,按以下流程执行。
按 review-checklist.md 中 F1-F3 的定义执行:
| 检查 | 方法 |
|---|---|
| F1 分支同步 | git log HEAD..origin/{base_branch} --oneline 计算落后提交数 |
| F2 变更概览 | 复用 Stage 2 的统计结果 |
| F3 Commit 规范 | 提取所有 commit type,对照 Branch×Type 矩阵 |
根据 Stage 2 的文件分类结果,确定要触发的动态检查:
| 变更范围 | 触发的检查 |
|---|---|
src/ 下有 新 服务目录 | D1(DDD 结构)+ D2(ops 完整性) |
main.py 有变更 | D3(服务注册与中间件) |
sql/ 有变更 | D4(数据库变更合规) |
configuration/ 或 .env* 有变更 | D7(配置管理) |
docs/ 有变更 | 不触发动态检查,输出提示:"建议运行 /mj-doc-review 检查文档质量" |
范围限定模式: 如果用户指定了评审范围(如"只看SQL"),只执行对应的动态检查 + F1-F2 信息展示。
读取 review-checklist.md 中对应检查项的定义(检查内容 + 通过标准),逐项执行。
检查执行方式:
[STANDARD]_Service_Architecture.md §3 的三级架构模板main.py,确认路由注册、中间件顺序、path_prefix 一致性[STANDARD]_Database_Change_Review.md 附录的快速检查清单.env.example 同步STOP & ASK:
将所有检查结果汇总为结构化表格:
| # | 检查项 | 结果 | 说明 |
|---|--------|------|------|
| F1 | 分支同步 | ℹ️ | 同步 |
| F2 | 变更概览 | ℹ️ | Code 5 | SQL 2 | Config 1 |
| F3 | Commit 规范 | ✅ | 3 feat, 1 docs — 均在 feature/* 允许范围 |
| D3 | 服务注册与中间件 | ✅ | 路由已注册,中间件顺序正确 |
按严重程度分组发现的问题:Critical > Important > Suggestion。
options:
- "发布到 PR — 将评审结果作为 comment 发布到 GitHub PR"
- "修改后发布 — 我要调整部分内容后再发布"
- "仅本地查看 — 不发布到 PR"
comment-template.md 格式生成完整评审内容gh pr comment {number} --body "{review_comment}"
仅在人工明确要求时触发("合并这个 PR" / "merge" / "合并")。
双重确认:
{head} → {base}?"
执行:
gh pr merge {number} --merge --delete-branch
合并成功后提示运行 /mj-git-delete 清理本地 worktree。
comment-template.md评审完成后输出:
评审完成 ✓
已完成项:架构评审 ✓ | 评审 comment {已发布到 PR #N / 未发布}
建议下一步:
- 文档变更 → /mj-doc-review
- 技术合并检查 → /mj-git-check-merge
- 直接合并 → 回复"合并"触发 Stage 5