Začněte byznysovou změnou
Review by mělo nejdřív odpovědět na otázku, zda kód bezpečně implementuje zamýšlené chování. Reviewer potřebuje user story, hraniční případy a plán rollbacku, ne jen pull request.
O bulk safety se nediskutuje
Apex je nutné posuzovat s ohledem na governor limity, objem dat a souběh různých automatizací. Testy nad jedním záznamem nestačí. Hledáme logiku postavenou nad kolekcemi, selektivní dotazy, jasné transakční hranice a předvídatelné ošetření chyb.
Testy mají vysvětlovat pravidla
Dobré testy dokumentují byznysové chování. Pokrývají pozitivní scénáře, chybové cesty, oprávnění, přípravu dat i hraniční případy, které v minulosti rozbily produkci. Coverage je důsledek, ne cíl.
Reviewujte provoz, ne jen kód
Release není hotový, dokud ho nedokáže provozovat podpora. Logy, chybové hlášky pro uživatele, feature toggly, retry cesty a monitoring musí být viditelné ještě před produkcí — zvlášť u integrací a revenue workflow.
Svěřte tu těžkou část seniornímu Salesforce engineering týmu.