Qo'lda sinov ro'yxati (AC1–AC3, AC5–AC15) + SLA o'lchovi (AC16)

qa-ep-13-t08-s02 statusni o'zgartirdi Review → Bajarildi

qa-ep-13-t08-s02 qabul mezonini belgiladi

✓ Ro'yxatda 14 ta qo'lda AC ning har biri uchun aniq qadamlar va kutilgan natija yozilgan

qa-ep-13-t08-s02 izoh qoldirdi

QA 2-ko'rik (109f1d3) — 5629 O'TDI. D1 (worker yoqish yo'li) EMPIRIK tasdiqlandi — izolyatsiyalangan git worktree, HAQIQIY OS env o'zgaruvchisi + ConfigDataApplicationContextInitializer (real application*.yml): dev+env=true -> app.notifications.worker.enabled=false (defekt haqiqiy); prod env'siz -> true; prod+env=true -> true; prod+env=false -> false. Hujjatdagi uchala joy (0.1-band 1/2, AC13 2-qadam, AC14 1-qadam) prod ga o'tgan; butun faylda qolgan `-Dspring-boot.run.profiles=dev` yo'q. To'liqlik: prod qo'shimcha majburiy env talab qilmaydi (RequiredEnvironmentProperties = DB_URL/DB_USER/DB_PASSWORD/JWT_SECRET, profil-neytral; @Profile bean yo'q; MinIO ixtiyoriy - Optional<FileUploadService>), FCM fail-fast tartibi hujjatda ogohlantirilgan, prod logging INFO da AC10 kutadigan NotificationRetentionJob yozuvi ko'rinadi (log.info). D2 mustaqil tasdiqlandi: SECURITY_EVENT_NOT_CONFIGURABLE — NotificationPreferenceService.java:47 + muzlatilgan openapi/v1/notifications.yaml (716-qator misolida); BusinessConflictException -> 409 (GlobalExceptionHandler.java:98); auth.token_reuse_detected_admin = SECURITY (NotificationEventCatalog.java:244) -> canDisable()=false; Admin-1 da user_session.revoke bor, demak requireVisibleToUser 404 bermaydi, 409 keladi; PUT tanasi (push_enabled+inapp_enabled) ikkala maydoni @NotNull — kontrakt bilan mos. Yangi AC14 stsenariysi spec §12 AC14 matniga mos: claimBatch SQL sharti `status='sending' AND next_attempt_at <= now()` (NotificationOutboxRepository.java:135) va LEASE_DURATION = Duration.ofMinutes(5) (NotificationOutboxWorker.java:152) — DB simulyatsiyasi to'g'ri. Regressiya yo'q: 14 AC bo'limi va 14 ta "AC:" yakuni, AC15 (a)-(d), §3.1-§3.4 saqlangan; 19 o'chirilgan qator — faqat almashtirilganlari. Yangi havolalar toza: application-dev.yml:88-90, application.yml:287, application-prod.yml:69, application-dev.yml:2-3, deploy/docker-compose.yml SPRING_PROFILES_ACTIVE=prod, README "Lokal ishga tushirish (dev)", NotificationWorkerProdProfileTest ikkala metodi. Gigiyena: commit faqat docs/10_notifications_qolda_qabul.md, AI atributi yo'q, git grep MUTANT HEAD bo'sh, erp/docs/modules/10_notifications.md §12 (1379-1499) tegilmagan (hunklar 466/900/1560).

coder-ep-13-t08-s02 yangiladi

Yangilandi: commit_sha

coder-ep-13-t08-s02 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-13-t08-s02 qabul mezonini belgiladi

✓ Ro'yxatda 14 ta qo'lda AC ning har biri uchun aniq qadamlar va kutilgan natija yozilgan

coder-ep-13-t08-s02 ishga oldi

qa-ep-13-t08-s02 statusni o'zgartirdi Review → Ishlanyapti

qa-ep-13-t08-s02 izoh qoldirdi

QA (qa-EP-13-T08-S02): QAYTARILDI. 5630 ✓, 5631 ✓, 5629 ✗ (2 defekt). === D1 (bloklovchi, 5629) — §0.1 2-band: worker yoqish ko'rsatmasi ISHLAMAYDI === Hujjat: §0.1 1-band `mvn spring-boot:run -Dspring-boot.run.profiles=dev` + 2-band "NOTIFICATION_OUTBOX_WORKER_ENABLED=true env bilan ustidan yoz". Haqiqat: application-dev.yml:68/88-90 da `app.notifications.worker.enabled: false` LITERAL (placeholder EMAS). Profil-xos fayl application.yml:287 dagi `${NOTIFICATION_OUTBOX_WORKER_ENABLED:false}` placeholder'ini bosib ketadi, ya'ni dev profilda bu env UMUMAN o'qilmaydi. EMPIRIK ISBOT (izolyatsiyalangan `git worktree`, ApplicationContextRunner, keyin o'chirildi): dev + ENV=true -> app.notifications.worker.enabled = false <-- hujjat va'dasi BUZILGAN prod + ENV=false -> app.notifications.worker.enabled = false <-- nazorat: env nomi to'g'ri, prod'da ishlaydi (o'lchov asbobi sog'lom) OQIBATI: ko'rsatma bo'yicha yurgizilsa worker ishlamaydi -> AC1/AC3/AC5/AC7/AC13/AC14/AC15 va AC16 o'lchovi jimgina "push kelmadi / qatorlar pending" beradi; sinovchi buni mahsulot nuqsoni deb hisoblaydi. Xuddi shu noto'g'ri ko'rsatma AC13 2-qadamda (=true bilan qayta ko'tarish) va AC14 1-qadamda (ikki instans =true) TAKRORLANADI. Repo bu tuzoqni allaqachon hujjatlagan: application-prod.yml:46-49 izohi va NotificationWorkerProdProfileTest javadoc'i (aynan shu sinf, oldingi QA topilmasi). Yo'l (tanlash coder'da): (a) buyruq qatori argumenti `-Dspring-boot.run.arguments=--app.notifications.worker.enabled=true` (eng yuqori ustunlik, profildan qat'i nazar), (b) prod profil + FCM env, yoki (c) application-dev.yml ga placeholder qo'yish — bu ALOHIDA backend ishi, hujjat bargida qilinmasin. === D2 (5629) — AC12 spec bandini to'liq qamramagan === Hujjat 266-299 qatorlar (AC12): spec §12 AC12 ning 5 ta kutilgan natijasidan 4 tasi bor (Aziz account_locked ✓, qulflangan ekranda "Aziz" yo'q ✓, Admin-1 oladi / Admin-2 olmaydi ✓, Aziz'ga admin nusxasi kelmaydi (E17) SQL bilan ✓, deep_link sessiyalar ekrani ✓, (d) account_locked_admin o'chirilishi ✓). YETISHMAYDI: `auth.token_reuse_detected_admin` ni O'CHIRIB BO'LMASLIGI (tugma yo'q) — spec §12 AC12 oxirgi bandi va qoida 37 jadvali (severity=security, "O'chirsa bo'ladimi: ⛔ yo'q"). Hujjatda bu kod FAQAT 136-qatorda, AC6 ning "6 ta security kodidan biri" IXTIYORIY ro'yxatida uchraydi — sinovchi `auth.password_changed` ni tanlasa bu xossa hech qachon tekshirilmaydi. Kerak: AC12 (d) ga ikkinchi qadam — `GET /api/v1/notification-events` javobida `auth.token_reuse_detected_admin` uchun `can_disable = false` va UI'da o'chirish tugmasi yo'qligi. === O'TGAN JOYLAR (qayta o'lchandi, coder so'ziga ishonilmadi) === • Struktura: 14 ta AC (AC1-AC3, AC5-AC15) bo'limi bor, har birida 4 qism (boshlang'ich holat / amal / kutilgan natija / AC) to'liq. Bo'sh "tekshirildi" checklist EMAS — qadamlar bajariladigan (aniq SQL, endpoint, env). • Mazmun solishtiruvi spec §12 bilan: AC3 salbiy tekshiruv (summa 12 400 000 / kontragent nomi / hujjat raqami push'da YO'Q) ✓; AC5 (account_blocked KELADI + ish xabari dead/recipient_ineligible) ✓; AC6 (security sozlamadan qat'i nazar + can_disable=false) ✓; AC15 to'rttala bandi (a) polling to'xtaydi (b) attempt_count oshmaydi (c) pending turadi, dead bo'lmaydi (d) tiklangach yetadi — ✓. • YOLG'ON HAVOLA TOPILMADI (hammasi tekshirildi, namuna emas): 13 hodisa kodi -> hammasi NotificationEventCatalog.java da; 10 endpoint -> 7 tasi muzlatilgan kontrakt openapi/v1/notifications.yaml da metod+yo'l bilan aynan mos, 3 tasi AuthController(@PostMapping /login)/UserController.java:83(@PatchMapping /{id}/status)/SessionController.java:34 da; fayl:qator havolalari ANIQ — FcmConfig.java:61 (FCM_BASE_URL), :99-118 (fcmWorkerConfigurationGuardRunner), NotificationEnqueueService.java:168-172 (existsRecentDedupe), NotificationOutboxWorker.java:165-166 (SYSTEMIC_ERROR_CODES, network_error HAQIQATAN ichida), notification-outbox-monitor.sh:209-217 (p95 VERDICT matnlari so'zma-so'z mos); sozlama sukutlari mos (breaker 20 / 60s, fixedDelay 1000ms, retention cron `0 30 3 * * *` / retention-months 3); V52 push_tokens audience CHECK ✓; §3.3 INSERT V51 CHECK'lariga haqiqatan mos (notification_id+notification_created_at juftligi NULL, severity 'info' va status 'pending' ruxsat ro'yxatida) — ya'ni qadam amalda yuradi. === 5630 (men o'lchadim — coder o'lchamagan edi) = ✓ === Skript repo'da va bajariladigan: scripts/notification-outbox-monitor.sh. Hujjat unga MAVJUD bayroq/env bilan havola qiladi (--p95-window-hours bor, sukut 24; DB_URL/DB_USER/DB_PASSWORD -> PGHOST/PGDATABASE/PGUSER/PGPASSWORD o'girish naqshi skriptda haqiqatan bor). Bir kunlik oyna ma'lumotida ishlashi S03 ning Testcontainers smoke testi bilan o'lchandi (to'liq Flyway, haqiqiy notification_outbox; shared dev DB'ga HECH NARSA yozilmadi): mvn -Dtest=NotificationOutboxMonitorScriptSmokeTest test -> Tests run: 5, Failures: 0, Errors: 0 (BUILD SUCCESS, 46 s) Qamrov: bo'sh jadval ("ma'lumot yo'q", crash emas), oyna bo'yicha failed/dead sanog'i, p95 < 3 s -> "OK (qoida 38...)", p95 5 s -> "BUZILGAN (...)" — IKKALA shoxobcha, turib qolgan qatorlar. Chegara (p95 < 3 s) va p95 ni beradigan chiqish qatori hujjatda aynan nusxalab bajariladigan darajada aniq. === 5631 = ✓ (hujjat mezoni, jonli o'lchov talab qilinmaydi) === §3.3: 200 ta payroll.accrued qanday tug'diriladi (SQL), zudlik bilan 1 ta task.available, o'lchov so'rovi sent_at-created_at (qoida 38 bilan so'zma-so'z bir xil), 3 sekund chegarasi, tozalash qadami — bor va mazmunli. §3.4: natija jurnali jadvali (Sana · O'lchagan · p95 · p99 · Namuna hajmi · Ommaviy sinov · Xulosa) — bor. === Coder ⚠️ larining baholashi (3 ta) — hech biri BIZNES bo'shlig'i emas, blocked kerak emas === 1) AC15 FCM URL qattiq yozilgani (FcmConfig.java:61) — TEXNIK bo'shliq (config-seam yo'q), spec bo'shlig'i emas. Muqobil ASOSLI: network_error haqiqatan SYSTEMIC_ERROR_CODES da (:165-166), ya'ni breaker (qoida 36) uchun literal http_5xx bilan ekvivalent yo'lni ishga tushiradi. Alohida backend ishi sifatida ochilsin (FCM base-URL override property). 2) AC13 millisekund poygasi — ASOSLI CHEKLOV. Deterministik muqobil (worker'ni oldindan o'chirish) bir xil kuzatiladigan natijani beradi (qator pending saqlanadi, qayta ko'tarilgach yetadi, E19 buzilmaydi). Spec bo'shlig'i emas. 3) AC16 ommaviy sinovda DB insert — ASOSLI CHEKLOV va ochiq aytilgan. Spec ham, task technical_notes ham "navbatga qo'yilgan zahoti" deydi, mexanizmni belgilamaydi; worker faqat qatordan o'qiydi (qoida 33). INSERT V51 CHECK'lariga mos ekani tasdiqlandi. === Gigiyena === • git show --stat f82d4bc: FAQAT docs/10_notifications_qolda_qabul.md (556 qo'shildi), begona fayl yo'q. ✓ • Commit xabarida AI atributi YO'Q (Co-Authored-By / Generated with / Claude-Session topilmadi). ✓ • `git grep MUTANT HEAD` bo'sh. ✓ Asosiy ish daraxtiga yozilmadi (git status toza), worktree o'chirildi, probe fayli qoldirilmadi. ✓ • erp/docs: S02 TEGMAGAN — 10_notifications.md §12 HEAD bilan bayt-aynan (diff bo'sh), demak solishtiruv bazasi ishonchli. ⚠️ OGOHLANTIRISH (S02 aybi EMAS, lead uchun): /var/www/erp ish daraxtida BOSHQA barglardan 5 ta COMMIT QILINMAGAN spec o'zgarishi turibdi — 01_suppliers, 03_warehouse, 06_orders, 08_finance (+1 qatordan) va 10_notifications.md (+49: qoida 6 doirasi "EP-13-T03-S05 topilmasi" + yangi qoida 40). Fayl vaqti 08:29, S02 claim'idan (10:09) OLDIN. Yo'qolib ketmasligi uchun commit qilinishi kerak.

qa-ep-13-t08-s02 qabul mezonini belgiladi

✓ Ommaviy hodisa sinovi qadamlari yozilgan va o'lchov natijasi qayd etiladigan joy bor

qa-ep-13-t08-s02 qabul mezonini belgiladi

✓ SLA/kuzatuv so'rovlari skript sifatida repo'da va bir kunlik ma'lumotda ishlaydi

qa-ep-13-t08-s02 qabul mezonini belgiladi

✗ Ro'yxatda 14 ta qo'lda AC ning har biri uchun aniq qadamlar va kutilgan natija yozilgan

coder-ep-13-t08-s02 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-13-t08-s02 izoh qoldirdi

5629=true (14 ta AC — AC1,2,3,5-15 — har biri 4 qismli: boshlangich holat/amal/kutilgan natija/AC; hammasi grep bilan tasdiqlangan: 13 ta hodisa kodi NotificationEventCatalog.java da, 6 ta endpoint controllerlarda, jadval/ustun nomlari V50-V53 migratsiyalarda). 5630/5631 ONGLI ravishda ochiq qoldirildi (yolgon true qilinmadi): scripts/notification-outbox-monitor.sh mavjud va togri chaqirilgan (buyruq/bayroqlar hujjatda tekshirilgan), lekin uni haqiqiy bir kunlik malumot + 200ta ommaviy hodisa bilan jonli yurgizib bolmadi: shared erpbom-dev-postgres konteynerida notification_outbox jadvali yoq edi (Flyway migratsiya bu DBda ishlamagan yoki boshqa holatda), migratsiya/sinov malumotini shu umumiy DBga yozish parallel agentlar ishiga xalaqit berishi mumkin edi, shuning uchun bajarilmadi. Hujjat 3.2/3.3 bolimlarida usul aniq korsatilgan — QA mustaqil izolyatsiyalangan muhitda olchab 5630/5631ni yopishi kerak.

coder-ep-13-t08-s02 qabul mezonini belgiladi

✓ Ro'yxatda 14 ta qo'lda AC ning har biri uchun aniq qadamlar va kutilgan natija yozilgan

coder-ep-13-t08-s02 yangiladi

Yangilandi: branch, commit_sha

coder-ep-13-t08-s02 fayllarni qayd etdi

1 ta fayl qayd etildi

coder-ep-13-t08-s02 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-13-t08-s02 ishga oldi

pm-agent yaratdi

Qo'lda sinov ro'yxati (AC1–AC3, AC5–AC15) + SLA o'lchovi (AC16)