Review

作者 parcadeid07ff4b06b62无许可证3.9K 个星标收录于 2026年10月8日更新于 2026年10月8日仓库8个月前更新

Comprehensive code review workflow - parallel specialized reviews → synthesis

AI 生成的概览

运行多视角代码审查流程,并行执行专项审查并给出最终综合结论。

功能
该技能定义了一套代码审查流程,将审查任务并行分派给多个专项代理,分别关注代码质量、架构一致性和变更影响。随后由综合代理汇总各项发现,给出 APPROVE、REQUEST_CHANGES 或 NEEDS_DISCUSSION 的最终结论,并列出按优先级排序的阻塞与非阻塞事项。它还描述了完整审查、快速审查、安全专项审查和拉取请求审查等模式。
适用场景
适用于被要求审查代码、拉取请求或合并前的实现,或需要设置质量门禁的场景。也适合针对重要变更获取反馈和风险评估的请求。
运行要求
仅为说明性指令,不包含脚本。它假定运行环境支持按指定代理类型(critic、plan-reviewer、review-agent,以及可选的 aegis)分派子代理任务并支持后台执行。

/review - Code Review Workflow

Multi-perspective code review with parallel specialists.

When to Use

  • "Review this code"
  • "Review my PR"
  • "Check this before I merge"
  • "Get feedback on implementation"
  • Before merging significant changes
  • Quality gates

Workflow Overview

         ┌──────────┐         │  critic  │ ─┐         │ (code)   │  │         └──────────┘  │                       │         ┌──────────┐  │      ┌──────────────┐         │plan-reviewer│ ─┼────▶ │ review-agent │         │ (plan)   │  │      │ (synthesis)  │         └──────────┘  │      └──────────────┘                       │         ┌──────────┐  │         │plan-reviewer│ ─┘         │ (change) │         └──────────┘
         Parallel                Sequential         perspectives            synthesis

Agent Sequence

#AgentFocusExecution
1criticCode quality, patterns, readabilityParallel
1plan-reviewerArchitecture, plan adherenceParallel
1plan-reviewerChange impact, risk assessmentParallel
2review-agentSynthesize all reviews, final verdictAfter 1

Review Perspectives

  • critic: Is this good code? (Style, patterns, readability)
  • plan-reviewer: Does this match the design? (Architecture, plan)
  • plan-reviewer: Is this change safe? (Risk, impact, regressions)
  • review-agent: Overall assessment and recommendations

Execution

Phase 1: Parallel Reviews

# Code quality reviewTask(  subagent_type="critic",  prompt="""  Review code quality: [SCOPE]
  Evaluate:  - Code style and consistency  - Design patterns used  - Readability and maintainability  - Error handling  - Test coverage
  Output: List of issues with severity (critical/major/minor)  """,  run_in_background=true)
# Architecture reviewTask(  subagent_type="plan-reviewer",  prompt="""  Review architecture alignment: [SCOPE]
  Check:  - Follows established patterns  - Matches implementation plan (if exists)  - Consistent with system design  - No architectural violations
  Output: Alignment assessment with concerns  """,  run_in_background=true)
# Change impact reviewTask(  subagent_type="plan-reviewer",  prompt="""  Review change impact: [SCOPE]
  Assess:  - Risk level of changes  - Affected systems/components  - Backward compatibility  - Potential regressions  - Security implications
  Output: Risk assessment with recommendations  """,  run_in_background=true)
# Wait for all parallel reviews[Check TaskOutput for all three]

Phase 2: Synthesis

Task(  subagent_type="review-agent",  prompt="""  Synthesize reviews for: [SCOPE]
  Reviews:  - critic: [code quality findings]  - plan-reviewer: [architecture findings]  - plan-reviewer: [change impact findings]
  Create final review:  - Overall verdict (APPROVE / REQUEST_CHANGES / NEEDS_DISCUSSION)  - Prioritized action items  - Blocking vs non-blocking issues  - Summary for PR description  """)

Review Modes

Full Review

User: /review→ All four agents, comprehensive review

Quick Review

User: /review --quick→ critic only, fast feedback

Security Focus

User: /review --security→ Add aegis (security agent) to parallel phase

PR Review

User: /review PR #123→ Fetch PR diff, review changes

Example

User: /review the authentication changes
Claude: Starting /review workflow...
Phase 1: Running parallel reviews...┌────────────────────────────────────────────┐│ critic: Reviewing code quality...          ││ plan-reviewer: Checking architecture...         ││ plan-reviewer: Assessing change impact...         │└────────────────────────────────────────────┘
critic: Found 2 issues- [minor] Inconsistent error messages in auth.ts- [major] Missing input validation in login()
plan-reviewer: ✅ Matches authentication plan
plan-reviewer: Medium risk- Affects: login, signup, password reset- Breaking change: session token format
Phase 2: Synthesizing...
┌─────────────────────────────────────────────┐│ Review Summary                              │├─────────────────────────────────────────────┤│ Verdict: REQUEST_CHANGES                    ││                                             ││ Blocking:                                   ││ 1. Add input validation to login()          ││                                             ││ Non-blocking:                               ││ 2. Standardize error messages               ││                                             ││ Notes:                                      ││ - Document session token format change      ││ - Consider migration path for existing      ││   sessions                                  │└─────────────────────────────────────────────┘

Verdicts

  • APPROVE: Ready to merge, all issues are minor
  • REQUEST_CHANGES: Blocking issues must be fixed
  • NEEDS_DISCUSSION: Architectural decisions need input

来源与署名

来源:parcadei/continuous-claude-v3位于.claude/skills/review提交d07ff4b

许可证: 无许可证

内容归原作者所有。SourceWeft 从公开仓库中收录这些内容。

举报或申请下架