Pregledajte kod koji je generirao AI

Pročitajte zahtjev prije sažetka

Počnite traženim ponašanjem i kriterijima prihvaćanja. Zatim pregledajte stvarni diff. Sažetak agenta može pomoći u snalaženju, ali može izostaviti promjene ili netočno opisati provjere.

Potvrdite granu i commit koji pregledavate. Uz kod aplikacije provjerite promjene konfiguracije, ovisnosti, infrastrukture i testova. Mala vidljiva funkcija može sadržavati veliku promjenu ovlasti ili ponašanja pri postavljanju.

Prvo pregledajte ponašanje s najtežim posljedicama. Formatiranje i nazivi važni su, ali ne smiju odvratiti pozornost od granice podataka koja nedostaje.

Pratite identitet do resursa

Razmotrite ovaj nepotpuni izmišljeni endpoint. Primjer prikazuje problem pregleda; nije produkcijski kod.

async function getInvoice(request) {
  const user = await requireSignedInUser(request);
  return database.invoice.findById(request.params.id);
}

Funkcija dohvaća korisnika s potvrđenim identitetom. Ne prikazuje odluku o autorizaciji za račun. Pregledavatelj mora utvrditi provodi li drugi sloj tu odluku. Neupotrijebljena vrijednost korisnika razlog je za istraživanje, a sama po sebi nije dokaz ranjivosti koju je moguće iskoristiti.

Pratite zahtjev kroz stvarni sustav. Utvrdite pouzdano određene korisnika i organizaciju. Provjerite kako upit ograničava pristup traženom zapisu. Pregledajte ponašanje pri pogreškama i testove zabranjenih zahtjeva.

Nemojte pretpostaviti da skriveni gumb štiti API. Pozivatelj može poslati zahtjev bez upotrebe sučelja. Nemojte pretpostaviti da valjan ID zapisa daje pristup.

Pitajte koji bi dokazi mogli odbiti promjenu

Uspješan test može upotrebljavati testne podatke administratora ili simuliranu autorizaciju. Provjerite testira li važnu granicu. Gdje je prikladno, dodajte slučaj s drugom organizacijom i stvarnim putem autorizacije.

Za promjenu korisničkog sučelja pregledajte prikazani rezultat. Provjerite upotrebu tipkovnice, prazna stanja, učitavanje i pogreške. Provjera tipova ne može potvrditi da se dijaloškim okvirom može upravljati tipkovnicom.

Za promjenu ovisnosti provjerite zašto je potrebna. Pregledajte verziju, licencu i sigurnosne nalaze. Nemojte prihvatiti nepovezanu nadogradnju samo zato što ju je agent generirao tijekom zadatka.

Zadržite neovisan pregled

Drugi model može dati korisne nalaze o problemima. Može i ponoviti pretpostavke iz implementacije. Pregledavatelju dajte zahtjev i diff. Nemojte mu govoriti da je promjena već ispravna.

Tražite da nalazi navedu konkretan put do kvara i relevantan kod. Nepotkrijepljena upozorenja tretirajte kao pitanja za istraživanje. Samouvjereno odobrenje tretirajte kao još jedno mišljenje dok važne tvrdnje ne budu potkrijepljene dokazima.

Ljudski pregled ostaje odluka o odgovornosti. Pregledavatelj treba dovoljno razumjeti promjenu da objasni ponašanje, rizike i provjeru. Ako je diff prevelik, smanjite opseg ili ga podijelite na promjene koje je moguće pregledati.

Zaključite pregled na konačnoj reviziji

Nakon ispravka ponovno pokrenite zahvaćene provjere. Pregledajte stvara li ispravak novi problem. Osigurajte da se obvezni pregled prema pravilima repozitorija odnosi na konačnu reviziju.

Odluku o prihvaćanju napišite kroz ponašanje i dokaze. Preostalom ograničenju pridružite odgovornu osobu i sljedeću radnju. Neriješen problem nemojte pretvoriti u tvrdnju da su sve provjere prošle.

Upotrijebite vježbu pregleda koda kako biste uvježbali prepoznavanje odluke koja nedostaje prije pregleda stvarne promjene.