Gh Pr Review (Cherry Studio)
Cherry Studios interne PR/branch reviewer, der håndhæver DataApi/IpcApi og lag-grænse regler.
Virker med opsætning
Hvad det gør
Dirigerer local-branch, PR, commit og filgennemgange gennem Cherry Studios egen arkitektur-regelbog — DataApi/IpcApi grænser, main/renderer/shared placering, navngivningskonventioner og service-ejerskabsregler — og tilbyder enten en hurtig enkelt-agent gennemgang med interaktiv valg af rettelse eller en multi-agent reviewer-verifier gennemgang med risikobaseret auto-rettelse. Udløses af Cherry Studio-specifikke gennemgangsanmodninger eller /gh-pr-review kommandoen; dens dybeste vejledning afhænger af arkitektur-dokumenter (docs/references/*.md), der ligger i Cherry Studio repo-roden, ikke inde i skill-mappen, der kopieres ved installation.
Testrapport
Fodrede den med en syntetisk Cherry-Studio-stil diff (en DataApi ipcMain.handle med forretningslogik indbygget i handleren, plus en renderer-komponent, der kalder window.require('fs') direkte): en generisk baseline-gennemgang fangede Electron nodeIntegration-lugten, men overså DataApi-handlerens grænseovertrædelse fuldstændigt, mens skill'ens vejledning markerede begge som navngivne Blockers med den nøjagtige upstream-dokumentation og fix-mønster (owning-service metode + typed ipcApi.request route) — men den vejledning virker kun, fordi hele monorepoets docs/references/-træ ligger ved siden af, hvilket standardinstallationskopien efterlader.
Testet: 2026-07-14 · Claude Code 2.x (agent harness)
Installation
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
Kommandoer og eksempelprompter
/gh-pr-reviewCherry Studios interne PR/branch reviewer, der håndhæver DataApi/IpcApi og lag-grænse regler.
Skills udløses af almindelige forespørgsler — ingen kommandoer at huske. Efter installationen aktiverer prompter som disse skillen (på engelsk):
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