Cum gestionezi feedback-ul de la code review

Ce face

Skill-ul te învață să tratezi orice review de cod ca pe o verificare tehnică, nu ca pe un exercițiu de politețe. În loc să spui automat „ai dreptate” sau să implementezi orbește, oprești, înțelegi cerința, verifici dacă e corectă pentru codebase-ul actual și abia apoi acționezi. Scopul e să eviți implementări greșite sau inutile.

Când îmi folosește

Îl folosești de fiecare dată când primești feedback la un PR, mai ales când comentariile par vagi, contradictorii sau tehnic îndoielnice. E util și când review-ul vine de la un extern sau de la partenerul tău uman – în ambele cazuri vrei să verifici realitatea din cod înainte să scrii ceva nou.

Cum îl invoc / declanșez

Skill-ul pornește automat când vezi un review. Nu există comandă specială; pur și simplu aplici pașii: citești tot feedback-ul, reformulezi cerințele în cuvintele tale, verifici codul existent și întrebi clarificări înainte să implementezi. Dacă ai dubii, oprești și întrebi.

Exemplu practic

Primești un review cu 6 puncte. Înțelegi 1, 2, 3 și 6, dar 4 și 5 sunt neclare. În loc să implementezi ce știi și să lași restul pe mai târziu, răspunzi: „Am înțeles punctele 1, 2, 3 și 6. Pot să clarifici 4 și 5 înainte să încep?” După clarificare, verifici dacă sugestia se potrivește cu modul actual de funcționare al aplicației și abia apoi faci schimbările, câte una odată, cu teste.

Output / unde aterizează

Rezultatul e o decizie tehnică clară: fie implementezi schimbarea după ce ai confirmat că e corectă, fie dai un pushback argumentat cu exemple din cod. Nu rămân comentarii neadresate și nu adaugi funcționalități degeaba doar ca să „arate profesional”.

Limite / gotchas

Nu aplica skill-ul dacă feedback-ul e clar și vine direct de la partenerul tău uman – acolo poți trece mai repede la acțiune. Dacă review-ul sugerează o funcționalitate pe care codul actual nu o folosește, verifici cu grep înainte să o implementezi (regula YAGNI). Dacă nu poți verifica o sugestie fără acces la alte sisteme, spune clar asta și întreabă dacă merită investigație suplimentară. All guides