• 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
名称
最后提交
最后更新
.claude 正在载入提交数据...
.cursor/rules 正在载入提交数据...
.devcontainer 正在载入提交数据...
.github 正在载入提交数据...
.husky 正在载入提交数据...
.storybook 正在载入提交数据...
assets 正在载入提交数据...
docs 正在载入提交数据...
drizzle 正在载入提交数据...
e2e-tests 正在载入提交数据...
makers 正在载入提交数据...
packages/@dyad-sh 正在载入提交数据...
rules 正在载入提交数据...
scaffold 正在载入提交数据...
scripts 正在载入提交数据...
shared 正在载入提交数据...
src 正在载入提交数据...
testing 正在载入提交数据...
tools 正在载入提交数据...
worker 正在载入提交数据...
workers/tsc 正在载入提交数据...
.cursorignore 正在载入提交数据...
.env.example 正在载入提交数据...
.eslintrc.json 正在载入提交数据...
.gitattributes 正在载入提交数据...
.gitignore 正在载入提交数据...
.npmrc 正在载入提交数据...
.oxfmtrc.json 正在载入提交数据...
.oxlintrc.json 正在载入提交数据...
.prettierignore 正在载入提交数据...
.prettierrc 正在载入提交数据...
AGENTS.md 正在载入提交数据...
CLA.md 正在载入提交数据...
CLAUDE.md 正在载入提交数据...
CONTRIBUTING.md 正在载入提交数据...
LICENSE 正在载入提交数据...
README.md 正在载入提交数据...
SECURITY.md 正在载入提交数据...
biome.json 正在载入提交数据...
components.json 正在载入提交数据...
drizzle.config.ts 正在载入提交数据...
forge.config.ts 正在载入提交数据...
forge.env.d.ts 正在载入提交数据...
index.html 正在载入提交数据...
lint-staged.config.js 正在载入提交数据...
merge.config.ts 正在载入提交数据...
package-lock.json 正在载入提交数据...
package.json 正在载入提交数据...
playwright.config.ts 正在载入提交数据...
tsconfig.app.json 正在载入提交数据...
tsconfig.json 正在载入提交数据...
tsconfig.node.json 正在载入提交数据...
vite.main.config.mts 正在载入提交数据...
vite.preload.config.mts 正在载入提交数据...
vite.renderer.config.mts 正在载入提交数据...
vite.worker.config.mts 正在载入提交数据...
vitest.config.ts 正在载入提交数据...
windowsSign.ts 正在载入提交数据...