• Will Chen's avatar
    Refactor multi-pr-review to use specialized reviewer personas (#2644) · 19650473
    Will Chen 提交于
    ## Summary
    - Replace generic review prompts with three distinct reviewer personas
    (correctness, code-health, UX)
    - Each reviewer has a specialized focus area for more targeted feedback
    - Update workflow trigger to include skills directory changes
    
    ## Test plan
    - Verify the PR review workflow triggers correctly on PR changes
    - Confirm each reviewer persona produces relevant, focused feedback
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    <!-- devin-review-badge-begin -->
    
    ---
    
    <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2644"
    target="_blank">
      <picture>
    <source media="(prefers-color-scheme: dark)"
    srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1">
    <img
    src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1"
    alt="Open with Devin">
      </picture>
    </a>
    <!-- devin-review-badge-end -->
    
    
    <!-- This is an auto-generated description by cubic. -->
    ---
    ## Summary by cubic
    Refactors multi-pr-review to use three specialized reviewer
    personas—Correctness, Code Health, and UX—for more focused, actionable
    PR feedback. Updates the GitHub workflow to call the new skill.
    
    - **Refactors**
    - Added persona templates: correctness-reviewer.md,
    code-health-reviewer.md, ux-reviewer.md.
    - Updated SKILL.md with persona roles, severity guidance, and new
    references.
    - Removed old generic prompts: review_prompt_default.md and
    review_prompt_code_health.md.
      - Switched workflow command to /dyad:multi-pr-review.
    
    <sup>Written for commit f761c88cec8a91be810ca55f66db39b5b7acfa4e.
    Summary will update on new commits.</sup>
    
    <!-- End of auto-generated description by cubic. -->
    
    <!-- CURSOR_SUMMARY -->
    ---
    
    > [!NOTE]
    > **Medium Risk**
    > Changes affect the automated PR review pipeline and remove prompt
    files that the orchestrator may still reference, which could break
    CI-based reviews if not updated in lockstep.
    > 
    > **Overview**
    > Updates the `dyad:multi-pr-review` documentation/spec to describe
    **three specialized sub-agent personas** (Correctness, Code Health, UX)
    and expands severity guidance to explicitly include UX
    blockers/degradations.
    > 
    > Adds new persona reference docs (`correctness-reviewer.md`,
    `code-health-reviewer.md`, `ux-reviewer.md`) and removes the prior
    generic prompt templates (`review_prompt_default.md`,
    `review_prompt_code_health.md`). The PR review workflow is also updated
    to run `/dyad:multi-pr-review` instead of the previous command.
    > 
    > <sup>Written by [Cursor
    Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
    f761c88cec8a91be810ca55f66db39b5b7acfa4e. This will update automatically
    on new commits. Configure
    [here](https://cursor.com/dashboard?tab=bugbot).</sup>
    <!-- /CURSOR_SUMMARY -->
    Co-authored-by: 's avatarClaude Opus 4.5 <noreply@anthropic.com>
    19650473
名称
最后提交
最后更新
..
commands 正在载入提交数据...
hooks 正在载入提交数据...
skills 正在载入提交数据...
README.md 正在载入提交数据...
run-e2e-update.sh 正在载入提交数据...
settings.json 正在载入提交数据...