Skip to content

Curăță codul mort, comentariile false și textele în română din cod

Vitalie requested to merge 22_dead-code into develop

Rezolvă #22 (closed).

Trei commit-uri, fiecare o schimbare de sine stătătoare.

1. Codul pe care nu-l importă nimeni

Șterse: PlaceholderMedia.tsx, PromoBanner.tsx, data/products.ts, data/categories.ts, 13 iconițe din icons.tsx, ColorInput din admin/FormControls.tsx și ALLOWED_ORDER_TRANSITIONS din lib/adminTypes.ts. Fiecare a fost verificată cu o căutare pe tot repo-ul, inclusiv peste granița dintre aplicații, unde backend/prisma/*.ts importă din frontend/src/data/.

Cele nouă constante din backend își păstrează numele și pierd doar export: fiecare e folosită de 2-5 ori în propriul fișier și nicăieri altundeva.

ALLOWED_ORDER_TRANSITIONS era o copie a regulii din order-admin-service.ts. Ecranul pentru care fusese scrisă a decis invers, intenționat: AdminOrders oferă toate butoanele de status și lasă backend-ul să răspundă ILLEGAL_TRANSITION (AdminOrders.tsx:120-124). Copia putea doar să se îndepărteze de regulă.

Două lucruri au venit odată cu ștergerile: blocul promo de nivel superior din cele trei fișiere de traducere, pe care nu-l mai citea nimic după dispariția bannerului, și trimiterile către fișierele șterse.

2. Comentariile care descriau opusul

index.html spunea că descrierile per pagină și per limbă „would need to be written into the document at runtime"; render-page.ts:625-629 le înlocuiește de mult. Comentariul nou numește și cele patru cazuri în care perechea statică chiar ajunge la vizitator: /admin, o adresă necunoscută, paginile fără descriere proprie și un renderer căzut.

Seo.tsx spunea că preview-urile de link văd tot tag-urile statice. Serverul scrie og:* și twitter:card pe fiecare pagină, iar fișierul o spunea deja cinci paragrafe mai jos, la removeMeta.

hyperhub-service.ts își justifica dimensiunea paginii printr-o constantă omonimă din frontend. Nu există: numărul de pagini vine din totalPages.

3. Textele în română din cod

Cinci mesaje de jurnal, o eroare aruncată și 284 de nume de teste în 27 de fișiere. Niciunul nu e văzut de vizitator. Panoul de admin rămâne în română, la fel și tot ce citește clientul — e-mailurile de comandă, bonul, textele magazinului.

Peste ce cerea issue-ul, cu motiv: issue-ul număra 8 nume de teste; sunt 284. Numărătoarea inițială a folosit git grep -E cu \s, iar pe această mașină git grep -E ignoră tăcut \s și \b. Reluată cu -P.

Verificare

  • backend: npm run build curat, npm test 600 de teste în 39 de fișiere.
  • frontend: npm run lint (un singur avertisment preexistent în src/App.tsx:70), npx tsc --noEmit -p tsconfig.app.json și -p tsconfig.test.json curate, npm test 433 de teste în 33 de fișiere, npm run build curat.
  • dev.db neatinsă: 61 de produse, 11 colecții, 1 comandă, 72 de articole, 3 bannere.
  • Niciun secret în diff.
  • Diff-ul fișierelor de test verificat mecanic: fiecare linie schimbată e un nume de test sau un comentariu, 603 linii scoase și 603 puse.

Merge request reports