a42a1a9920
All checks were successful
Deploy / deploy (push) Successful in 17s
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
76 lines
7.0 KiB
Markdown
76 lines
7.0 KiB
Markdown
# 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)
|
||
|
||
1. **Y1 kapandı.** `sihirbaz-adimlar.tsx`'te `olayBirKez("sihirbaz_adim", {adim:1}, "adim1")` → `olay("sihirbaz_adim", {adim:1})`. Payda artık payla aynı birimde: sihirbaz her açıldığında `adim:1` düşer, `:225`'teki ileri geçiş olayı zaten her geçişte düşüyordu. `olayBirKez` bu dosyada başka kullanılmadığı için import'tan da çıktı.
|
||
2. **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).
|
||
3. **O2 (yalnız belge).** `paket-gorunum-olayi.tsx`'in allowlist yorumuna, tekilleştirme tavanının 4→14 çıktığını ve `diger` kovasının anlamının değiştiğini söyleyen bir seri-kırılması notu eklendi. Kod değişmedi.
|
||
4. **O3 kapandı.** `universite-icerik.tsx`'e karta özel `UNITUR_ETIKET` haritası kondu; `src/lib/slug.ts`'teki `trBaslikDuzeni`'ye **dokunulmadı**. Üretilen HTML'de "Kktc" ve "Yurtdısı" üniversite sayfalarından tamamen kalktı.
|
||
5. **Entegrasyon tazelendi, dördü de temiz:** çakışmasız iki merge, `next typegen` + `tsc --noEmit` + `lint` + `build` hatasız. `gece/0922-entegrasyon` başı `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 VAKIF` etiketi hiçbir sayfada görünmüyor (0 sayfa).** Haritada var ama DB'deki 12 satır, iki yazımı `YURTDISI KAMU` + `YURTDISI VAKIF` olan iki kuruma (Hoca Ahmet Yesevi, Kırgızistan-Türkiye Manas) ait ve okuma katmanı bu slug'lar için `YURTDISI 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:227` de ham `unitur`'ü `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.145 `KKTC` + 99 `YURTDISI KAMU` satı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 kaynak `unitur` → etiket haritası") bunu kapsıyor.
|
||
|
||
---
|
||
|
||
## Merge sonrası doğrulanması gerekenler
|
||
|
||
1. **Canlıda `sihirbaz_adim` serisinin kırılma tarihi** — deploy gününden itibaren `adim:1` var 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ış".)
|
||
2. **`paket_goruntulendi` toplamı ve `diger` payı** — deploy sonrası ikisi de trafikten bağımsız olarak değişecek; panelde önce/sonra kıyası yapılmamalı.
|
||
3. **Gözle bakış:** `/universite/dogu-akdeniz-universitesi` ve `/universite/uluslararasi-balkan-universitesi` "Tür" kartı (`KKTC` / `Yurt dışı (kamu)`) — 390px'te kart üç satırı geçmiyor mu.
|
||
4. 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)
|
||
|
||
1. **O1'i kayda geçir** — `1df86cf` veri 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.**
|
||
2. **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.**
|
||
3. **`/bolum` rozetlerini 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.**
|