🔍

Veščina za pregled zahtev za združitev

Strukturiran postopek pregleda zahtev za združitev, ki prebere vse spremenjene datoteke, jih analizira glede napak, varnosti, zmogljivosti in sloga ter nato poda povratne informacije z ocenami resnosti. Sproži se z /review-pr.

Skill·/review-pr·3 razdelkov·1 min branja

codingdevops

Install Prompt

Paste this into Claude Code to set it up:

V svojo postavitev Claude Code želim dodati veščino /review-pr.

Ustvari mapo ~/.claude/skills/review-pr/ in v njej datoteko SKILL.md z delovnim potekom za pregled zahtev za združitev.

Veščina naj:
- pridobi podrobnosti in razliko zahteve za združitev z GitHuba
- analizira spremembe kode glede napak, varnostnih težav in sloga
- poda strukturirane povratne informacije s sklici na vrstice
- predlaga izboljšave in opozori na pomisleke

Po ustvarjanju datoteke je veščina takoj na voljo prek /review-pr.

01Kaj počne

Temeljit pregled z enim samim ukazom

Vnesite /review-pr 123 in Claude pridobi zahtevo za združitev, v celoti prebere vsako spremenjeno datoteko (ne le razliko), kodo analizira po petih razsežnostih in poda strukturiran pregled. Po želji lahko pregled objavi tudi nazaj na GitHub.

Pridobitev zahteve za združitev

Podrobnosti, razliko in metapodatke zahteve za združitev pridobi z orodjem gh CLI ali GitHub MCP. Deluje s številkami zahtev za združitev ali z zahtevo trenutne veje.

Branje vseh spremenjenih datotek

Prebere celotno trenutno različico vsake spremenjene datoteke, ne le razlike. Spremembe razume v kontekstu okoliške kode.

Analiza po petih razsežnostih

Preveri napake, varnostne ranljivosti, težave z zmogljivostjo, kakovost kode in pokritost s testi. Vsaka ugotovitev dobi oceno resnosti.

Podajanje strukturiranih povratnih informacij

Razvrščene po resnosti: Kritično (nujno odpraviti), Opozorilo (priporočljivo odpraviti), Predlog (dobrodošlo). Vključuje konkretne popravke kode in prizna dobre vzorce.

Objava na GitHub (izbirno)

Po želji pregled objavi neposredno na GitHub kot komentar zahteve za združitev, zahtevo po spremembah ali odobritev.

02Kaj preverja

🐛

NapakeLogične zmote, dostopi do praznih vrednosti (null), tekmovalna stanja, manjkajoča obravnava napak, vrzeli v tipski varnosti, okvarjeni robni primeri.

🔒

VarnostVrivanje SQL, XSS, CSRF, v kodo zapisane skrivnosti, manjkajoče preverjanje, vrzeli pri preverjanju pristnosti, izpostavljeni občutljivi podatki.

⚡

ZmogljivostPoizvedbe N+1, manjkajoča paginacija, neomejene zanke, uhajanje pomnilnika, nepotrebni ponovni izrisi, velike razširitve svežnja.

✨

Kakovost kodeJasnost poimenovanja, dolžina funkcij, podvajanje kode, mrtva koda, manjkajoči komentarji, nedosledni vzorci.

🧪

Pokritost s testiSo nove funkcije testirane? So pokriti robni primeri? So preverjene poti napak? So trditve smiselne, ne le »deluje«?

03Načela pregleda

Bodite konkretni. Sklicujte se na točne datoteke in vrstice. Za predlagane popravke priložite odseke kode.

Pojasnite zakaj. Ne recite le »spremeni to«. Pojasnite tveganje ali korist, da se avtor nauči.

Predlagajte, ne zahtevajte. Za slogovne želje uporabite »razmislite«. Odločen jezik prihranite za resnične napake.

Priznajte dobro delo. Pregled, ki našteje le težave, je nepopoln. Dobra koda si zasluži priznanje.

Ostanite v obsegu. Pregledujte, kar se je spremenilo. Obstoječe težave v nespremenjeni kodi so ločene naloge.

En prehod, popolne povratne informacije. Izogibajte se postopnemu podajanju komentarjev v več krogih pregleda.