All checks were successful
Deploy / deploy (push) Successful in 17s
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
7.0 KiB
7.0 KiB
13 — Denetim düzeltmeleri (Tur 6)
Ajan: yazilimci · Kaynak: 12-kod-denetimi.md Y1 / O2 / O3 · Dallar: gece/0922-olcum, gece/0922-urap, birleşim gece/0922-entegrasyon
Push yok, main'e commit yok, tarayıcı açılmadı.
Özet (5 madde)
- Y1 kapandı.
sihirbaz-adimlar.tsx'teolayBirKez("sihirbaz_adim", {adim:1}, "adim1")→olay("sihirbaz_adim", {adim:1}). Payda artık payla aynı birimde: sihirbaz her açıldığındaadim:1düşer,:225'teki ileri geçiş olayı zaten her geçişte düşüyordu.olayBirKezbu dosyada başka kullanılmadığı için import'tan da çıktı. - CEO'nun P4/(1) kabul kriteri düzeltildi ve commit mesajına yazıldı. Eski kriter ("oturumda tam bir kez") mevcut olay sözleşmesiyle çelişiyordu; yeni kriter "her sihirbaz açılışında bir kez". 1→2→1→2 gezinmesinde tekrar düşmeme şartı korunuyor (mount effect,
[]bağımlılık; geri dönüşler unmount etmiyor). - O2 (yalnız belge).
paket-gorunum-olayi.tsx'in allowlist yorumuna, tekilleştirme tavanının 4→14 çıktığını vedigerkovasının anlamının değiştiğini söyleyen bir seri-kırılması notu eklendi. Kod değişmedi. - O3 kapandı.
universite-icerik.tsx'e karta özelUNITUR_ETIKETharitası kondu;src/lib/slug.ts'tekitrBaslikDuzeni'ye dokunulmadı. Üretilen HTML'de "Kktc" ve "Yurtdısı" üniversite sayfalarından tamamen kalktı. - Entegrasyon tazelendi, dördü de temiz: çakışmasız iki merge,
next typegen+tsc --noEmit+lint+buildhatasız.gece/0922-entegrasyonbaşı74f7aa1, çalışma ağacı temiz.
Commit'ler
| Dal | Hash | Mesaj |
|---|---|---|
gece/0922-olcum |
58f4dc2 |
fix(olcum): huninin paydası ile payı aynı birimde olsun |
gece/0922-urap |
0e689e6 |
fix(katalog): "Tür" kartı ham DB değerini okunur Türkçeyle gösterir |
gece/0922-entegrasyon |
e1760d4 |
Merge branch 'gece/0922-olcum' into gece/0922-entegrasyon |
gece/0922-entegrasyon |
74f7aa1 |
Merge branch 'gece/0922-urap' into gece/0922-entegrasyon |
Dokunulan dosyalar (3, hepsi yol vererek eklendi — git add -A kullanılmadı):
src/features/sihirbaz/components/sihirbaz-adimlar.tsx(+9/−4)src/features/odeme/components/paket-gorunum-olayi.tsx(+6/−0, yalnız yorum)src/features/katalog/components/universite-icerik.tsx(+19/−5)
Kabul kriterleri
| # | Kriter | Sonuç | Nasıl doğrulandı |
|---|---|---|---|
| 1 | sihirbaz_adim{1} dedup'sız atılıyor |
✔ | grep: :160 olay("sihirbaz_adim", { adim: 1 }), :225 olay(..., { adim: yeniAdim + 1 }) — ikisi de dedup'suz. Dosyada olayBirKez kalmadı |
| 2 | Effect yalnız gerçek açılışta koşuyor, her render'da değil | ✔ (koddan çıkarım) | useEffect(..., []); bileşen beş mount noktasının beşinde de koşullu (denetim raporu §"Mount = 1. adım görüldü" ile aynı okuma). Tarayıcıda koşturulmadı |
| 3 | O2 notu dosyada ve commit mesajında | ✔ | paket-gorunum-olayi.tsx:16-21 yorumu + 58f4dc2 gövdesi |
| 4 | "Kktc" üniversite sayfalarından kalktı | ✔ | grep -rl '>Kktc<' .next/server/app/universite/ → 0; dogu-akdeniz-universitesi.html → >Tür</dt><dd …>KKTC< |
| 5 | "Yurtdısı" hiçbir üniversite sayfasında yok | ✔ | grep -rl "Yurtdısı" .next/server/app/universite/ → çıktı yok |
| 6 | Yeni etiketler doğru dağılıyor | ✔ | 1. sayfalarda: KKTC 16 sayfa, Yurt dışı (kamu) 6 sayfa (Azerbaycan Devlet Pedagoji, Hoca Ahmet Yesevi, Kırgızistan-Türkiye Manas, Tiran New York, Uluslararası Balkan, Uluslararası Saraybosna). Toplam 22 = denetimin saydığı 22 yurt dışı/KKTC slug'ı |
| 7 | Izgarada delik açılmadı | ✔ | >Tür</dt> 31 sayfa + >URAP Türkiye</dt> 198 sayfa = 229; Devlet/Vakıf etiketleri bozulmadı (ör. sivas-bilim-ve-teknoloji-universitesi.html → Devlet) |
| 8 | trBaslikDuzeni'ye dokunulmadı |
✔ | git diff yalnız 3 dosya; src/lib/slug.ts listede yok |
| 9 | Entegrasyon: typegen + tsc + lint + build temiz | ✔ | Dördü de 0 çıkışla koştu; build 229 üniversite + 494 sayfalama rotası üretti |
| 10 | Merge çakışması yok | ✔ | İki merge de ort stratejisiyle otomatik |
Şartnameden sapmalar
Yok — verilen üç dosyanın dışına çıkılmadı, trBaslikDuzeni korundu.
Not düşülecek iki nokta:
YURTDISI VAKIFetiketi hiçbir sayfada görünmüyor (0 sayfa). Haritada var ama DB'deki 12 satır, iki yazımıYURTDISI KAMU+YURTDISI VAKIFolan iki kuruma (Hoca Ahmet Yesevi, Kırgızistan-Türkiye Manas) ait ve okuma katmanı bu slug'lar içinYURTDISI KAMU'ya düşüyor. Bu tam olarak denetimin D4 maddesi; harita bugün ölü kol, ileride D4 kapanınca çalışır.- Aynı bozuk etiketler
/bolum/*program listesinde duruyor.bolum-program-listesi.tsx:227de hamunitur'ütrBaslikDuzeni'den geçiriyor: üretilen HTML'de "Kktc", "Yurtdısı Kamu", "Vakıf Myo" (24 bölüm sayfası), "Yurtdısı Vakıf" (7 sayfa) rozetleri var. Bu gerileme değil,main'de de vardı —main'in DB'sinde zaten 1.145KKTC+ 99YURTDISI KAMUsatırı var (sqlite karşılaştırması yapıldı); P2 bu sayıları 1.187 + 157'ye çıkardığı için yüzey biraz genişledi. Şartname bu dosyayı vermediği için dokunmadım; denetimin backlog önerisi #1 ("tek kaynakunitur→ etiket haritası") bunu kapsıyor.
Merge sonrası doğrulanması gerekenler
- Canlıda
sihirbaz_adimserisinin kırılma tarihi — deploy gününden itibarenadim:1var ve oturum başına birden çok kez düşebiliyor; 2 Kasım okumasında huni oranı bu tarihten önceki veriyle karşılaştırılmamalı. (analitik'in sabit dipnotu güncellenmeli: kriter "her açılış".) paket_goruntulenditoplamı vedigerpayı — deploy sonrası ikisi de trafikten bağımsız olarak değişecek; panelde önce/sonra kıyası yapılmamalı.- Gözle bakış:
/universite/dogu-akdeniz-universitesive/universite/uluslararasi-balkan-universitesi"Tür" kartı (KKTC/Yurt dışı (kamu)) — 390px'te kart üç satırı geçmiyor mu. - Denetimin kapsanmayan maddeleri hâlâ açık: O1 (commit gövdesindeki 170→166 yerine 132→129), O4 (üniversite sayfasında devlet/vakıf bilgisinin kaybı), D1–D7.
Bilal'den istenen (3 madde)
- O1'i kayda geçir —
1df86cfveri commit'inin gövdesindeki "170 → 166" hub sayısı değil; doğrusu 132 → 129. Merge onayı bu fark raporuna dayandığı için düzeltme onayla birlikte girmeli. ~3 dk. - O4'e karar ver — üniversite sayfalarında "devlet mi vakıf mı" bilgisi tamamen kayboldu; giriş cümlesine tek sıfat mı eklensin, yoksa takas bu hâliyle mi kalsın? ~2 dk.
/bolumrozetlerini backlog'a al mı, şimdi mi — aynı bozuk etiketler (Kktc,Yurtdısı Kamu,Vakıf Myo) bölüm listelerinde duruyor; bu gece kapsam dışıydı, tek kaynak haritaya taşımak ~20 dk. ~1 dk karar.