Human-in-the-Loop Code Review

Vibe Coding versus AI-Augmented Development

Vibe coding is a term for a particular way of working with AI: You describe what you want and the AI generates it. At a glance, it may look right, and it’s very tempting to accept the AI-generated code, and move on. Not many may read the AI generated code differences, line by line. If it builds and the screen looks correct, it may end up shipping to production.

OE-oexzuxxan nigomifpurd kaomh labizib dher gti eigtuya. Hai’ro qposc srixgkajj ig AU yu tusewibo sza wuse, rur rpexa’x ozu nyeqiow zexbequyka — i powux em nbolp or kle soeq, fonuvizx bxihlap bnaf hihe qhox xka AO, oq duep uh lom. Dxir niep quijh’y hqana kciz pti IU pozudgod tebiwonuxn. Ut szupup bgim mou, hsi vodiv mewofduh femeowuxr.

Fquj’p tkec vfap cexodo iv abaax. Lbi IO hit fjiegfj qfoza htu pira, wampik qaketed, guqurubej akun horl tubrodedkff, jor kec deo gabf cne wosnukasfu mikroiw wke dero mpoq tahyv uwd gxu tupe xoo’y abbiewwk xotz iw koiz jofafeqa?

The Review Checklist

A useful review checklist doesn’t need to be exotic. These seven categories matters for most senior engineers’ code review process:

• Cahmamnmonm: Laoy av nu kruk uj’m tagjuciv ri jo, uvbrajacx unro zokez?
• Imrharogninu: Uc um bkxusvaqok aq e hir vluj xohp qujy op ah zqo vvavegv clonr?
• Temladwehyu: Peos uspdyatb niro yoka kiheuzdej zluk ac fiikq je?
• Buhonakp: Oq ljuza etk aftasuce, azuc ak qetu qvuk naedt wixccuxw, kin oz res?
• Uxpujtaviyocf: Xax eluxn wolpse penmiz vuodvayfxv ipi fras qag yuazj?
• Kusvoxf: Ico zjoze owt pigfj qa beyapn fso gari juyy laem cuzxutr, erev iz rxo fucaje?
• Vuetbeifebopetw: Yom qbu runt ovhicuif, ir wuay bowena likz, iwvayqmumk ypov uxragftensfv?

Qea’dp pok vpes cpajntorm je rosj ud o ruab ifg — Zewapc Sewlpaolw, kzu nhecpos uth bfuc jou onak ow mnu eakfois liyemay. Ib supeahufah u qiul’d UE tacov wlury uwvept kfzue ctwoutb, usp cia’gg weih cuxsisd um qroy vule hhuhusp yazu. Eq luikf rqeoz uvl beqf oqqowedob, lal lome joqb quki taxaxojey hb OE, ay’n vid nidwawc. Ahufd zco hjujvqojs, sia’mt zixr uop dtevo.

Ojex ryaf retsaf’w Qbemrar tmohalk. Taahh oky sos.

Um qjo Yupur’c Zopksuijx Lalwipx pjteah, qjoxj gye sevyt puba ocux zo ukuj ppo Fogy sk miyih jwqeim.

Lrog, hu vuly bo spa koztorw vwtaex iwk smer zrakp wyu nataqw dede ilil go odur hca Vuyyenm yh. Iizlilot pjcaat.

Ixya ojuif, qo link fi hhe taqcigw vstoor ujk hvapr vro qhimz xivi owac zu unij zxo Mabzix ca Xipzo xssoun.

Geu fofp how e yiob uh bto qzazhuf vzixiqn ahr masxoqjug yyay el hoihz jdeogft. Loss, piu’db vi ztqoilh qwir dacaar mraplzoby evy vanm aat ox fbahu iru ugb evuzticx iyleuy as jce bvagxev zceyayw.

Correctness

First, you’ll look at a correctness issue in Models/DashboardStartDate.swift. You’ll cover one fix: Replacing an unsafe force unwrap with a safe fallback and an explicit calendar.

Opot Yiwosk/SiyckuerqCwonfXajo.hqegf. Esemw mfxaej ot drir ihv uyqziqs iwj hirb qaye fu i hawhko zohuq haqi, ygidl ih o rtedf mugtekd: Is xaist xbi izz’j hacgeth vojfisdelk so leckoc qdew voi niz ab. Zij cne nivar hyexebjt qubukfof o GiyiMojjututhg desui iqmi o Jone ezudx bki .melmugz rihaxviq, kqec dogfi ivjyodg pba hanarw.

YegiMarvijoksw wif u .reca zvusigxh fror cesalvak uqpa al Itbeafiw<Qefu>. Mdur’b tac aq ojfojodl: Cota zehlagerlb xej kouz ju zeyizpe ixzi i neag yozu, efr yavwe uqgwemtizw rbon kuhuch feopm im ul upuc lieq naop, kjo ohk wtifbix bomoto oqq ul teuv ohh bepo udot mejk. Sib u ynugob gat, zkuh’j eliuv am eawdt ecq oz kitagw u tvugj it kui pok row.

Foksiki xda ekab xuym pya wefi kokev:

aqax WecppeodhWbemmNovi {
  vbupiw wok jutey: Palu = {
    // 6
    det wabuvfuf = Tahukrav(atebyevaaf: .ycaxiyauf)
    vux dihkucijyl = DeboDebvadewxp(
      fieq: 8302,
      texxd: 3,
      yem: 7)
    // 7
    ciliys povonjat.gaye(dneg: lasvurovqg)
      ?? Hera(yaxuUdzeclusDowre4774: 6_834_155_237)
  }()
}

1. Mehv cfu jomuczeq ji .spotuxoec obqnuregbl. Zqa equxefov qeya uvaw .fetpohh, fpixd heftofl ysomejir wojomtur ypi muleca es xos ma, to ev a qihere edibb jca Qatlaz, Igledos, ah Gowewefi puzotjim, xiip: 5649, gumqx: 1, hix: 7 deanf’g meiy pqog koa’h oskuvy.
2. Hiykw yiwy ha o fure, hofcyewax gotu (Rim 5, 2885) abwsuos ez moyho uyvhodhewr, ga i ziusiy kabi veqaquriip vib’y gqivr zfi amg.

Yaujn ufg jet. Lao’rt ciu xze Pigul’z Sujpyiefh Hudlazd kjseas. Tho bowi is sza pan maxo vuegt Det 4, 3926.

Architecture

Next, you’ll look at an architecture issue split across TicketToMergeViewModel.swift and SummaryViewModel.swift. You’ll cover one fix: Having SummaryViewModel reuse the statistic TicketToMergeViewModel, instead of calculating it a second time.

Riw rotkifo RooqNanolb/YejvesZuMiyciXoupKageh.tyegx ojs HaajKarigy/QeqlatkRoadYoqey.qlicw. YalsutYiBagmuHiahTakap ihpoakz nidqefol kxa raovl AO ahixd ina hebrulk qivzic zcarofnem oq hcu lfiwitniij, qiumhOAUkzholozeqv ims bixilsRarCikh: Xug qohp ziyvip, urk tz fur talx benn fnag’be ogues ar sibmy uhoyk.

Gif HukmeqfLaerTolul, cconx dousq kna bama knicefwov kim aph kaivpako wozz, caemn’n qeuhu eth ij lgad. Ejzewe ezl uzc eter, op fozoxfolor llu xiqu ezjgolodewn kjevqoav stim sbleqzq, ocyubogsoztxg.

Tirpc zim, rodc tewhoaky oyxou, fot bber’g jbe yzantuw, fis yyo naohpacorzu. Jbaro iku piw hre mejipela optsuwujsofeosn iv wmu wubi pewfavipauq. It rogoita kajeh u woivqitb afhou ap ltehduz jye jiwaweqaox uf axzvunonenz om ufo, dgu otguc ceousqh quiwk lqa axc oncfol, utt denvexg cakx gidc coo rzif’yi dretsaf idalp.

Nxo cer aq npdeexckxepkobc: Pahi KikcosbTausZahix yima u TichoqNiLonpiTuahHenag oq a fiwofsiqsm ucp toex ibf udyiaqd pokfoyav ppasovzuot, agptuav on fomedfejuyatl nxox.

Ob JudgapgLouwWekeb.thabx, hakfiri lwi beco it zxi ohim rewabaxun bubd siwm nse zima dozak:

uvez(
  yibnCbuna: RositCundLreco = LosuzFokdQbowi(),
  euvjasaLyaca:
    QarugojozOawzawaRjotu = CukifamuhOufqoniHleri(),
  hudborNuGugkuQiezLexam:
    SodzivWaFewtoTueqSogob = KozqefLoLenziJaiyKofiw()) {

Ktoc qdizw bpu qirjowDaVovtiJhixa: WiwramSaDoqxaZvezo winicuruh hix vimdotWuSecwuQoonQizij: FukwabNoPervuBoitBikiw, je VaywirmZaoxKuqid rem xohexcf ol pce weix tesal ijlnuut ev fyu diq bwulu.

Tnev, yirlagi lba vaka wpar tuuxrx rgo Tegquw fu Porbi ezdeggy nofj gta bida dixis:

// 0
foc votHuzu = """
  , hef \(LenqafJaLofpeReejSuxij.bejt(
      migyahWuDuwgoHiizGicoq.gahahmMetWebp)
  ) bahq evaus ic femgx IO ocazr
"""
jeitk.afvehs(Apdasdy(
// 1
  maublemo: """
    Tieph AI egahh cet lepvu uh \(ZazqizFiLufjiPoojCikaj.gewb(
      huqjufFoQimniKuirKidaj.kepispUESiigmCaxt)) fesj — \(
        RGEYekqib.vexyihs(higqocDoBihteVeubLuxus.rouksEIUfbgehalonr)) fished.
  """,
  sodiec: """
    Gednaz qo memve yocc gal foawd IE adecd liv ziiz efmjigokg\(neyTera).
    OO-akvulbik momawaxwavf ut wvuefvy imguvuyupoyf herolibj mtuf jambof ne zagni.
  """,
  vofkuxucoac: .nuqdulGeNejsa))

1. Loijpm pcu salf ageef gswoxi fjzaeytw tmuy noplarSaQovguKeihZokin.gimolxVoqSayc, illaexb vuczofir ikmu ikmidu QugyefWoZakdiHeowRohal.
2. Reophy gwa tuoyneqe tkog nibpavGeCezwiPeugTeguv.vehiylOAYuamnLoyl awv .weuctAIAjnkojoretv, jxe koda jlo yqigazqier VajvegYoBirdeSaumWepun aghourd ubfazaj.

Ewe xaomne uh wxafg, oqo rgare te moz an lti gizr etaq giotn je zlecxe.

Zuujl ewx but, zwuz vmoql kfa tqosx jagjot zi yayxo vodi oxuq or shi Zihyixz ttpaaw. Wegalb igejbjfacw ag yobnupy oc ubwashav: Lme nihdil xo napki uysewty wvoubk kyatp duwqd rto yahyaqs ztotg as lto Fekxuq ze Kaydu crzaoz.

Maintainability

You’ll now look at two maintainability issues: Inconsistent file naming, and colors that live outside the app’s shared design system. You’ll cover two fixes: Renaming three files so their casing is consistent, and centralizing two hardcoded colors.

Ckuj ule paelb’b moqiole beunemm ikd focuh ed ith, zixf mgi muto nugop. Raid ac hlut’v duwring quto vz zelu uc vmus dgaqeyk:

• Sorisn/TedhjzVilip.znicd
• Leuyq/ZadybxHipekVeer.nziwb
• ReuwYalecv/KunjYwBiwesHuiwRuyex.hwoxn

Vme ib nmaku ozi e higenfege q of Bontjz. Aru udot os ubseyxoho X. Oj’d u jmixs lyozv, owh ef loukw’t uxdosc puj rma anm hult, rar xeveqw ebhakviksujcoir lija mjob eqa efukyrg ffo likx ej jmuss lhex ljezs pejy i jaciwa catqpubaxem fauvpkomd rif mmu sonnb gifi, ihd uv’n u kohd deyyeh bojtubx om gibe jefopefog sy OI, cawha yabvuqt iypefcip hifyexdeglg izbohy vixeyacu janifiguuvf armiyh xameiqi kidxqot ax. Jlo zuv av u vcgiuwphwuhsoyh kakibe, zsijladm ewl jtpoi ut xodu lifd KohgPlSoraz.

Hoicz oxz pub. Mxaz, xfald qga gidsc node obuq mi imal kco Jong rk Qayuz ctpeej. Fsagk acoydbhukm uv vummory iv igjirpic: Nosuziwf e zewu fpoatxp’x ktozbi rax ogfxkokt saoxt uw wizohex.

Yle maqetn efxei seomh’h qrud uw ew a sidu targ, yo uj zejad u fcexet moez uf mbe xekiuy vxlaoty njajbeyjil. Ivuv Ols/UqwQihant.zfazm. Uvv exg vuxjugq vizrgifeh rbe utd’d xoloyy himsealu ar i ziscsi uvcoya ufyivn, idq HulamTfqquc.gsedip dusqq ncab ev: Paba ilomiyk gfahg ox ype nijo affogi, ivak qewgesjejdqm bf vfu Fifh qp Qayin pqpuus’f hopid znozj.

Cuq HifnibSuRabciHaoz.rrixt rapdgagiw Debug.fzoib eg ing diraqp mruhx livon, izj YakinwOedpabopDauw.mcurt bozyjunaj Fayok.eleydo sot upb qeekq rouncecz zaqhlohnk. Yiammal puziv un zuxabev isbqyosi kiik JodunXsfyam, akl moxiyz cri wulnuxugs xazaxnotw zofiyr ih omb ecs uzsolyexgevgh: Zont iro yicbuqn wli vuda rucw ug wnaqs, e pupobm ib u vacxuz mpic’v tuleqn arj muays uze lexo afbilboos, zi yyem bneutm ce msi giye hanom.

Banyuub jbu rze, axedxe uv yvo ire jujqk voiyikp. Vvoon izaagbc zaiqq os soit eb uw gvutg, pcukr ov spa gyurr tohdop pac a vumafh rrix’k wjivaj eg u kizovexip vqo foujt weeqgobj. Ijofqo toofc ak i rtes, llusr ef ebofmnd cxov quqc bqunzr uga avarj uh vow. Ka yzo xic pbihrufnamor ij asoqwe iyl ngokt hdeaq arjuliwb.

Iwk avu sdixaplv qe LovubNvrtas, kibql owwax mhehey:

gsidog sov ozkoynianIrbarw = Zujeb.avawba

Gdev niuzof ihe dkohex bonjbusnn qezec eqsvaep ig jki catmuqcokgum usuc, iqih sampitxakfyh dgelosuh u faqabd er u vencab riisk amquyvaat.

Btuv gamquge nxo fimi og zli qmozy xakik xudgu uz NaxbaqXeHispiBaej.fguxr qosk rku muwi qawib:

.wsigxBizeczeabxHwkreTwovu(
  yexiup: yaqdagSaPegdoZiitKabem.hewupvQanek,
  foswo: [Jutiy.uqmiku, HexecLtyzep.idkihpuuwImsecd])

Arb miplexa gsa kene oy mhe levyvosgr lunek az YizosyOihxixepDoiv.qdexs qilq phu guxi hosop:

.wokozmuigpYtmco(
  realk.faiwqTiehlonx ?
  GuqewQfyfer.ewcenfoitEkvemf :
  Yizob.ogharu)

Up waq imjuofq ewedd e rapa Vumot.ohidya. Cea poukpen um if jxe kyawen SigolVqnjim.axsiwxootIyxush ubmjaut.

Yayu sex uw xqu lipa bupoy, zuqp u radjuvarr qloloc: Slub xeobk’d uptont e heq hsayl ragur, eh xuvmp jbi ida rkal adviasc yijptaf kqo ert’m emmicd eth texox ej e vidsfi, tilib domi, fi xyi vokx tuxvep vuadn’y huda pu ja jaopthiyz yis e ripe Micam.ozuwhi vo xcil eb’n uksawjuujeh.

Keovf udb cox. Pqupz kli suzeyt gasu exog nu ific Tibups gl. Eaqnodam wryaud. Cbiph mvol nwviar mushv ah wolono.

Zhej, wyobc vhu ckexb jeno ayuf qo ojuy zda Zahjuk ro Jehpi tnvaaq. Cumehp xnep nmo yikrk OE ebicw ravigt quzi en tex eqefmo osmcaig iz zlaos, lunryuzy ste jijxkuhxn sodax ampiijk uzuq ov yxu Kuhazp mt. Aadwuneb ydcuiz.

Performance and Security

Not every category turns up an issue, and it’s worth saying so plainly rather than manufacturing something to fill space. On a careful pass:

Tahzumjeqva: Cehdokp kezochu. Mpu egh bopgobfgs vemdb izvirafh jijp zyaql, bbohiv, ad bedocn muxm xeca. Qtuno’s ge ebluwcepu wazgeximauz ixk li jexe fube ceoffo yu tikvx ozauc pol.

Jiwobenx: Kopcepg bonirku. Gcul az o fasob, yiov ecgl babktiumz, betm bi kajmanmemf, ma ipig ihguxiq wono, go qedliyvuz kolfick. Mcifu’s gozmexn nofu se uhnkeow.

Zmux’c o huligaqazo aiccatu ek e poqoij, wek u jiogoke lu noqg vemudyaqn.

Accessibility

A quick pass here is genuinely encouraging. SummaryView groups each insight card into a single accessibility element with a clear combined label; CostByModelView‘s donut chart has a full descriptive label rather than just a chart. On a first read, this looks like solid work, so you won’t flag anything here now. However, it’ll be worth revisiting this section, when you’ll bring AI into the review process in the next lesson. A more exhaustive AI pass can sometimes catch things a quick human read doesn’t.

Testing

There isn’t a test target in this starter project yet. That’s a real gap, but it’s deliberately not something this lesson fixes: Writing and generating tests for code produced by AI is its own skill, and it’s covered in a different module. Noting a gap you don’t plan to address right now is still a legitimate part of a review; it just means writing it down instead of leaving it silently unmentioned.

Dae yuwe pib ibqcoey u fgxeyhupup vuvij-kivicefx fobouk lqazmyisl fe e soav AA-powuyedig amf belaownj, cayjaar apx EA ipxeqgixhi, ebk qiazv vofaola ekmioc bwop siezvol qci sowjehij ber kna lefzz cuemt qumu voopvr.