Gh Pr Review (Cherry Studio)

Cherry Studio's interne PR/branch reviewer die DataApi/IpcApi en layer-boundary regels afdwingt.

Door CherryHQ · CherryHQ/cherry-studio

Werkt met setup ★ 8.4/10

Gh Pr Review (Cherry Studio) — Cherry Studio's interne PR/branch reviewer die DataApi/IpcApi en layer-boundary regels afdwingt.

Wat het doet

Routeert local-branch, PR, commit, en file reviews via Cherry Studio's eigen architectuurregels — DataApi/IpcApi grenzen, main/renderer/shared plaatsing, naamgevingsconventies, en service-ownership regels — en biedt ofwel een snelle single-agent pass met interactieve fix-selectie of een multi-agent reviewer-verifier pass met risicogebaseerde auto-fix. Activeert bij Cherry Studio-specifieke reviewverzoeken of het /gh-pr-review commando; de diepste begeleiding is afhankelijk van architectuurdocs (docs/references/*.md) die zich in de root van de Cherry Studio repo bevinden, niet in de skill-map die bij installatie wordt gekopieerd.

Testrapport

Voorzag het van een synthetische Cherry-Studio-stijl diff (a DataApi ipcMain.handle with business logic baked into the handler, plus a renderer component calling window.require('fs') directly): een generieke baseline review detecteerde de Electron nodeIntegration 'smell' maar miste de DataApi-handler boundary violation volledig, terwijl de begeleiding van de skill beide markeerde als benoemde Blockers met de exacte upstream doc en fix pattern (owning-service method + typed ipcApi.request route) — maar die begeleiding werkt alleen omdat de volledige docs/references/ tree van de monorepo ernaast staat, wat de standaard installatiekopie achterlaat.

Getest op: 2026-07-14 · Claude Code 2.x (agent harness)

Installatie

git clone https://github.com/CherryHQ/cherry-studio
cd cherry-studio
mkdir -p ~/.claude/skills
cp -r .agents/skills/gh-pr-review ~/.claude/skills/gh-pr-review

Commando's en voorbeeldprompts

  • /gh-pr-reviewCherry Studio's interne PR/branch reviewer die DataApi/IpcApi en layer-boundary regels afdwingt.

Skills reageren op gewone verzoeken — geen commando's om te onthouden. Na installatie activeren prompts zoals deze de skill (in het Engels):

  • Review this pull request diff for blockers and fixes
  • Check this PR for IPC bypass and business-logic leaks
  • Do a code review of my changes before I merge