# Review — docs/plans/sube-plan.md v1.2 — 20.10.2026

> **Review type:** plan-re-review
> **Previous review:** `docs/reviews/review-sube-plan-re-review-2026-10-20.md` (heyet 821, REJECTED, F-1 — `letterhead_min_width_px` yanlışlıkla kimlik kaldırma kapsamına girmişti); kök: `docs/reviews/review-sube-plan-2026-10-20.md` (heyet 820, REJECTED, F-1…F-5)
> **Upstream document(s) read:** `docs/specs/sube-spec.md` v0.6 (heyet 819 APPROVED)
> **Other cross-referenced documents:** kod kanıtları — `app/Support/Business/Settings.php:150-157` (`letterheadMinimumWidth()`, `companyLetterhead()`'den ayrı metod, 820/821 turlarında doğrulanmıştı); bu turda yeni bir kod doğrulaması gerekmedi, çünkü değişiklik planın kendi metnini daraltıyor.
> **Stage locks in effect:** Spec locked (v0.6, heyet 819). 820/821'de onaylanmış/kapanmış bulgular kilitli; bu turda yalnız A5 adım 1'in güncellenen paragrafı ve revizyon geçmişi değerlendirildi.

---

## Upstream Coverage Matrix

| Upstream § / madde | Plan'da durumu | Kanıt |
|---|---|---|
| 821 F-1: `company.letterhead_min_width_px` kimlikten ayrışsın, düzenlenebilir kalsın | ✓ | A5 adım 1: "Backoffice `BusinessSettings` sayfasındaki **kimlik** alanları (`company.name`, `company.subtitle`, `company.address`, `company.phone`) kaldırılır... `company.letterhead_min_width_px` **kalır**: şirket kimliği değil, antet görselinin asgari genişlik kuralıdır (kendi docblock'u, kırmızı çizgi 9) ve yerinde düzenlenmeye devam eder" |
| 821 F-1: arch testinin kapsamı dört kimlik anahtarıyla sınırlansın | ✓ | "Arch testi: bu dört kimlik anahtarını `IssuingCompany` migration'ı dışında okuyan kod yok" — önceki genel ifade ("`company.` anahtarını... okuyan kod yok") somut dört anahtara daraltılmış; migration'ın backfill için bu anahtarları bir kez okuması ayrıca muaf tutulmuş (P-12'nin "ilk kayıt bugünkü `company.*` ayarlarından migration'la" kuralıyla tutarlı) |
| 820 F-1…F-5 (önceki turda kapanmış) | ✓ | Dokunulmadı, yeniden açılmadı — belge bu bölümlerde v1.1 ile harfiyen aynı |

---

## ✅ What works

- **821'in bulgusu tam ve doğru kapandı.** Düzeltme cerrahi: yalnız dört kimlik anahtarı (`name`, `subtitle`, `address`, `phone`) kaldırılıyor, `letterhead_min_width_px` adıyla anılıp gerekçesiyle (kırmızı çizgi 9, kendi docblock'u) açıkça korunuyor. Arch testinin kapsamı da aynı dört anahtara daraltılmış — artık yanlışlıkla ilgisiz bir ayarı süpürme riski yok.
- Migration'ın backfill amacıyla eski `company.*` anahtarlarını bir kez okuma ihtiyacı ("`IssuingCompany` migration'ı dışında") doğru şekilde ayrı tutulmuş; bu, P-12'nin ilk kayıt kuralıyla çelişmeden arch testinin asıl niyetini (kalıcı kod yolu okumasın) karşılıyor.
- Değişiklik dar: yalnız A5 adım 1'in bir paragrafı ve revizyon geçmişi dokunulmuş; önceki turda kapanmış 820'nin beş bulgusu yeniden açılmamış, plan'ın geri kalanı bu turda hiç değişmemiş.

## ⚠️ Findings & Risks (in-scope for this review type)

Yok.

## ❓ Open Questions (to be answered before next stage)

Yok.

## Previous Findings Status

- 821 F-1: ✅ Resolved — A5 adım 1, dört anahtara daraltma.
- 820 F-1…F-5: ✅ Resolved (önceki turda), bu turda dokunulmamış.

## ⚠️ Upstream Findings

Yok.

## Verdict: ✅ APPROVED

Matrix'te ❌/⚠ satır yok. 821'in bulgusu tam, dar ve doğru bir düzeltmeyle kapandı; yeni bir çelişki üretmedi. Planın geçirdiği tüm turlardaki bulgular — 820'nin beşi ve 821'in biri — toplam altı bulgunun hepsi kapandı. Plan implementasyona geçmeye hazır.