Gh Pr Review (Cherry Studio)
Wewnętrzny recenzent PR/branch Cherry Studio egzekwujący reguły DataApi/IpcApi i granice warstw.
Działa po konfiguracji
Co robi ten skill
Kieruje przeglądy gałęzi lokalnych, PR, commitów i plików przez własny zbiór reguł architektonicznych Cherry Studio — granice DataApi/IpcApi, rozmieszczenie main/renderer/shared, konwencje nazewnictwa i reguły własności usług — oferując szybkie przejście jednoagentowe z interaktywnym wyborem poprawek lub przejście wieloagentowe recenzent-weryfikator z automatyczną poprawką opartą na ryzyku. Uruchamia się na żądania przeglądu w zakresie Cherry Studio lub polecenie /gh-pr-review; jego najgłębsze wskazówki zależą od dokumentacji architektury (docs/references/*.md), która znajduje się w katalogu głównym repozytorium Cherry Studio, a nie w folderze umiejętności kopiowanym podczas instalacji.
Raport z testu
Podano mu syntetyczny diff w stylu Cherry Studio (DataApi ipcMain.handle z logiką biznesową wbudowaną w handler, plus komponent renderujący wywołujący window.require('fs') bezpośrednio): ogólny przegląd bazowy wykrył problem nodeIntegration w Electronie, ale całkowicie pominął naruszenie granicy DataApi-handler, podczas gdy wskazówki umiejętności oznaczyły oba jako nazwane Blockers z dokładnym dokumentem źródłowym i wzorcem poprawki (metoda owning-service + typowana trasa ipcApi.request) — ale te wskazówki działają tylko dlatego, że pełne drzewo docs/references/ monorepo znajduje się obok, czego standardowa kopia instalacyjna nie uwzględnia.
Testowano: 2026-07-14 · Claude Code 2.x (agent harness)
Instalacja
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
Komendy i przykładowe prompty
/gh-pr-reviewWewnętrzny recenzent PR/branch Cherry Studio egzekwujący reguły DataApi/IpcApi i granice warstw.
Skille uruchamiają się na zwykłe polecenia — bez komend do zapamiętania. Po instalacji aktywują go prompty takie jak te (po angielsku):
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