Shqyrtoni kodin e gjeneruar me AI

Lexoni kërkesën para përmbledhjes

Nisni me sjelljen e kërkuar dhe kriteret e pranimit. Pastaj shqyrtoni diff-in real. Përmbledhja e agjentit mund t’ju ndihmojë të orientoheni, por mund të lërë jashtë ndryshime ose t’i përshkruajë pasaktë kontrollet.

Konfirmoni branch-in dhe commit-in që po shqyrtoni. Kontrolloni ndryshimet në konfigurim, varësi, infrastrukturë dhe teste, krahas kodit të aplikacionit. Një funksion i vogël i dukshëm mund të përfshijë një ndryshim të madh në leje ose në sjelljen e vendosjes.

Shqyrtoni fillimisht sjelljen me pasojat më të mëdha. Formatimi dhe emërtimi kanë rëndësi, por nuk duhet të largojnë vëmendjen nga një kufi i munguar i të dhënave.

Gjurmoni identitetin deri te burimi

Shqyrtoni këtë endpoint të paplotë dhe të sajuar. Shembulli ilustron një problem shqyrtimi; nuk është kod për prodhim.

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

Funksioni merr një përdorues të autentikuar. Nuk tregon një vendim autorizimi për faturën. Shqyrtuesi duhet të kontrollojë nëse një shtresë tjetër e zbaton atë vendim. Vlera e papërdorur e përdoruesit është arsye për hetim, jo provë më vete e një defekti që mund të shfrytëzohet.

Gjurmoni kërkesën përmes sistemit real. Identifikoni përdoruesin dhe organizatën nga burimi i besuar. Kontrolloni si e kufizon query-ja aksesin në rekordin e kërkuar. Shqyrtoni sjelljen në gabime dhe testet për kërkesa të ndaluara.

Mos supozoni se një buton i fshehur e mbron API-në. Një thirrës mund të dërgojë kërkesë pa përdorur ndërfaqen. Mos supozoni se një ID e vlefshme rekordi jep akses.

Pyesni cila evidencë mund ta refuzojë ndryshimin

Një test që kalon mund të përdorë një fixture administratori ose autorizim të simuluar. Kontrolloni nëse provon kufirin që ka rëndësi. Shtoni një rast me një organizatë tjetër dhe rrugë reale autorizimi kur është e përshtatshme.

Për një ndryshim të ndërfaqes së përdoruesit, shqyrtoni rezultatin e shfaqur. Kontrolloni përdorimin me tastierë, gjendjet boshe, sjelljen gjatë ngarkimit dhe gabimet. Një kontroll tipesh nuk vërteton se një dialog mund të përdoret me tastierë.

Për një ndryshim varësie, verifikoni pse nevojitet. Shqyrtoni versionin, licencën dhe gjetjet e sigurisë. Mos pranoni një përmirësim versioni të palidhur vetëm sepse agjenti e gjeneroi gjatë detyrës.

Mbajeni shqyrtimin të pavarur

Një model i dytë mund të identifikojë probleme të dobishme. Mund edhe të përsërisë supozime nga implementimi. Jepini shqyrtuesit kërkesën dhe diff-in. Shmangni deklaratën se ndryshimi tashmë është i saktë.

Kërkoni që gjetjet të identifikojnë një rrugë konkrete dështimi dhe kodin përkatës. Trajtojini paralajmërimet e pambështetura si pyetje për hetim. Trajtojeni një miratim të shprehur me bindje si një mendim tjetër derisa pretendimet e rëndësishme të kenë evidencë.

Shqyrtimi njerëzor mbetet vendim për përgjegjësinë. Shqyrtuesi duhet ta kuptojë ndryshimin mjaftueshëm për të shpjeguar sjelljen, rreziqet dhe verifikimin e tij. Nëse diff-i është tepër i madh, kufizojeni fushën ose ndajeni në ndryshime që mund të shqyrtohen.

Mbylleni shqyrtimin mbi rishikimin final

Pas një korrigjimi, ekzekutoni sërish kontrollet e prekura. Shqyrtoni nëse korrigjimi krijon një problem të ri. Sigurohuni që shqyrtimi i kërkuar të zbatohet mbi rishikimin final sipas politikës së depos së kodit.

Shkruajeni vendimin e pranimit sipas sjelljes dhe evidencës. Regjistroni çdo kufizim të mbetur bashkë me personin përgjegjës dhe veprimin e radhës. Mos e ktheni një çështje të pazgjidhur në pretendim se të gjitha kontrollet kaluan.

Përdorni ushtrimin e shqyrtimit të kodit për të praktikuar identifikimin e vendimit që mungon para se të shqyrtoni një ndryshim real.