Vaata AI genereeritud kood üle

Loe nõuet enne kokkuvõtet

Alusta nõutud käitumisest ja vastuvõtukriteeriumidest. Seejärel uuri tegelikku diff’i. Agendi kokkuvõte võib aidata orienteeruda, kuid võib jätta muudatusi välja või kirjeldada kontrolle ebatäpselt.

Kinnita ülevaadatav haru ja commit. Lisaks rakenduse koodile kontrolli seadistuse, sõltuvuste, taristu ja testide muudatusi. Väike nähtav funktsioon võib sisaldada suurt õiguste või juurutuskäitumise muudatust.

Vaata kõige suuremate tagajärgedega käitumine esimesena üle. Vormindus ja nimetused on olulised, kuid ei tohi juhtida tähelepanu puuduvalt andmepiirilt kõrvale.

Jälgi identiteeti kuni ressursini

Vaata seda puudulikku väljamõeldud lõpp-punkti. Näide illustreerib ülevaatusprobleemi; see ei ole tootmiskood.

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

Funktsioon saab autenditud kasutaja. See ei näita autoriseerimisotsust arve kohta. Ülevaataja peab uurima, kas mõni teine kiht seda otsust rakendab. Kasutamata kasutajaväärtus on põhjus uurimiseks, kuid ei tõenda iseenesest ärakasutatavat viga.

Jälgi päringut läbi tegeliku süsteemi. Tuvasta usaldatud kasutaja ja organisatsioon. Kontrolli, kuidas andmepäring piirab juurdepääsu küsitud kirjele. Uuri käitumist vea korral ja keelatud päringute teste.

Ära eelda, et peidetud nupp kaitseb API-t. Väljakutsuja saab saata päringu ilma kasutajaliidest kasutamata. Ära eelda, et kehtiv kirje ID annab juurdepääsu.

Küsi, milline tõend võiks muudatuse tagasi lükata

Edukas test võib kasutada administraatori testandmeid või autoriseerimise testiasendust. Kontrolli, kas see katsetab olulist piiri. Lisa vajaduse korral teise organisatsiooni ja tegeliku autoriseerimise teega juhtum.

Kasutajaliidese muudatuse korral uuri kuvatud tulemust. Kontrolli klaviatuuriga kasutamist, tühje olekuid, laadimiskäitumist ja vigu. Tüübikontroll ei saa tõendada, et dialoogi saab klaviatuuriga kasutada.

Sõltuvuse muudatuse korral kontrolli selle vajadust. Vaata üle versioon, litsents ja turvaleiud. Ära võta vastu teemavälist uuendust ainult seetõttu, et agent selle ülesande käigus genereeris.

Hoia ülevaatus sõltumatu

Teine mudel võib leida kasulikke probleeme. See võib ka korrata teostuse eeldusi. Anna ülevaatajale nõue ja diff. Ära ütle talle ette, et muudatus on juba õige.

Nõua, et leiud näitaksid konkreetset tõrke tekkimise teed ja asjakohast koodi. Käsitle tõendamata hoiatusi uuritavate küsimustena. Käsitle enesekindlat heakskiitu veel ühe arvamusena, kuni oluliste väidete kohta on tõendid.

Inimese tehtud ülevaatus on endiselt vastutust puudutav otsus. Ülevaataja peab muudatust piisavalt mõistma, et selgitada selle käitumist, riske ja kontrollimist. Kui diff on liiga suur, vähenda ulatust või jaga see ülevaadatavateks muudatusteks.

Lõpeta ülevaatus lõpliku revisjoni peal

Pärast parandust käivita mõjutatud kontrollid uuesti. Uuri, kas parandus tekitab uue probleemi. Veendu, et nõutav ülevaatus käib koodihoidla reeglite järgi lõpliku revisjoni kohta.

Sõnasta vastuvõtuotsus käitumise ja tõendite kaudu. Pane iga allesjäänud piirangu juurde vastutaja ja järgmine tegevus. Ära muuda lahendamata küsimust väiteks, et kõik kontrollid õnnestusid.

Kasuta koodi ülevaatuse harjutust, et harjutada puuduva otsuse tuvastamist enne päris muudatuse uurimist.