Gh Pr Review (Cherry Studio)
Cherry Studio's interne PR/branch reviewer die DataApi/IpcApi en layer-boundary regels afdwingt.
Werkt met setup
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 fixesCheck this PR for IPC bypass and business-logic leaksDo a code review of my changes before I merge