Shartnoma PDF + tilxat generatsiyasi va saqlanishi (`printed_at`)

finalize-ep-08 yangiladi

Yangilandi: pr_url

qa-ep-08-t05-s03 statusni o'zgartirdi Review → Bajarildi

qa-ep-08-t05-s03 izoh qoldirdi

QA aylana 2 (mutatsiya tekshiruvi, izolyatsiyalangan worktree @5874d9b). AC3498 ENDI HAQIQATAN QOPLANGAN. Mutant #1: CustomerContractDocumentGenerationService.java:120 setContractId(contract.getId()) -> mijozning ENG ESKI (min version) versiyasi id-si. Natija: OLDIRILDI - CustomerDocumentOutboxWorkerTest.receiptRequestedEvent_forSecondVersion_pointsAtThatVersionNotTheFirst:199 yiqildi (expected: 2L but was: 1L). javap bilan tasdiqlangan (lambda$generateReceiptNote$1 bytecode-da). Birinchi aylanadagi tirik mutant yopildi. Yangi test sifati: v1 'replaced' + v2 'active' yaratadi, receipt_requested v2 ga enqueue qilinadi, SELECT contract_id ... WHERE customer_id=? AND doc_type='receipt_note' (contract_id bo'yicha filtrlamaydi - tautologiya emas), ikkala assert ham bor: == v2Id VA != v1Id. Kuchli. Mutant #2 (chegara o'lchovi): setContractId -> ENG YANGI (max version) versiya. TIRIK QOLDI (5 test, 0 failure); javap bilan tasdiqlangan (Stream.max bytecode-da, class manbadan yangi). Bu MAQBUL - qamralgan ikkala stsenariyda ham so'ralgan versiya aynan eng yangisi; biznes mantiqan tilxat active versiyaga tegishli. Qayd: hodisa NOFAOL (replaced) versiyaga yuborilgan holat testda yo'q. Regressiya (baseline, mutantsiz): CustomerDocumentOutboxWorkerTest 5, CustomerCreateTest 13, CustomerContractVersioningTest 11, CustomerContractCancellationTest 12 = 41 test, 0 failures / 0 errors / 0 skipped. 3495/3496/3497 testlari joyida va yashil. Intizom: diff 0e4ed65..5874d9b = FAQAT 1 test fayli (+23 qator), src/main/java o'zgarmagan; openapi/ va db/migration/ diff BO'SH; @Disabled/assumeTrue/MUTANT yo'q. Worktree o'chirildi, asosiy daraxt toza.

qa-ep-08-t05-s03 qabul mezonini belgiladi

✓ Generatsiya qilingan tilxat qatorida `contract_id` to'g'ri versiyaga ishora qiladi

coder-ep-08-t05-s03 yangiladi

Yangilandi: commit_sha

coder-ep-08-t05-s03 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-08-t05-s03 qabul mezonini belgiladi

✓ Generatsiya qilingan tilxat qatorida `contract_id` to'g'ri versiyaga ishora qiladi

coder-ep-08-t05-s03 fayllarni qayd etdi

1 ta fayl qayd etildi

qa-ep-08-t05-s03 statusni o'zgartirdi Review → Ishlanyapti

qa-ep-08-t05-s03 izoh qoldirdi

QA (qa-EP-08-T05-S03) — mutatsiya tekshiruvi bilan, izolyatsiyalangan worktree'da (0e4ed65). Baza: 40 test YASHIL — CustomerDocumentOutboxWorkerTest 4, CustomerCreateTest 13, CustomerContractVersioningTest 11, CustomerContractCancellationTest 12 (0 failure). [OK] AC3495 — MUTANT: CustomerContractDocumentGenerationService.java:90 `contract.setPrintedAt(Instant.now())` olib tashlandi (javap: setPrintedAt = 0 chaqiruv) -> printRequestedEvent_rendersRealPdfAndFillsContractFileColumnsAndPrintedAt:141 QIZIL. Test V30 CHECK uchligini ham aniq tekshiradi (contract_file_key startsWith 'customer_contracts/', mime = application/pdf, size > 0) va MinIO'dan haqiqiy %PDF baytlarini qayta yuklaydi. [OK] AC3496 — MUTANT: ObjectKeyGenerator.java:60 `UUID.randomUUID()` -> qotib qolgan UUID (javap: randomUUID = 0 chaqiruv) -> newContractVersion_getsItsOwnFileKey_previousVersionFileKeyUntouched:210 QIZIL ('not to be equal to' — ikki kalit teng chiqdi). Test v1 kaliti o'zgarmaganini ham ANIQ tenglik bilan tekshiradi, isNotNull emas. [OK] AC3497 — MUTANT: generateContractPrint ichida pasport MAJBURIY qilindi (existsByCustomerIdAndDocType, javap tasdiqladi) -> generation_succeedsEvenWithoutAnyPassportDocumentForTheCustomer:229 QIZIL. Fixture'da pasport yo'qligi testning o'zida COUNT = 0 bilan tasdiqlangan. [QAYTARILDI] AC3498 — MUTANT TIRIK QOLDI. Mutant: CustomerContractDocumentGenerationService.java:120 `document.setContractId(contract.getId())` -> `customerContractRepository.findFirstByCustomerIdOrderByVersionAsc(contract.getCustomerId()).orElseThrow().getId()` (ya'ni tilxat DOIM mijozning eng eski/v1 versiyasiga yo'naltiriladi). javap mutant bytecode'da ekanini tasdiqladi. Natija: CustomerDocumentOutboxWorkerTest 4/4 YASHIL qoldi — ya'ni 'to'g'ri versiya' mezoni AMALDA QOPLANMAGAN. Sabab — FIXTURE XILMA-XILLIGI: tilxat sinaladigan har ikki testda (receiptRequestedEvent_createsReceiptNoteDocumentRowPointingAtTheContractVersion:151 va generation_succeedsEvenWithoutAnyPassportDocument...:219) mijozda ATIGI BITTA shartnoma versiyasi bor, shuning uchun 'to'g'ri versiya' bilan 'mijozning yagona versiyasi' farqlanmaydi. Assert'ning sifati yaxshi (aniq id solishtiriladi), yetishmayotgani — STSENARIY. KUTILGAN TUZATISH (FAQAT TEST, produksiya kodi to'g'ri): CustomerDocumentOutboxWorkerTest ga yangi holat qo'shilsin — bitta mijozda v1 (status 'replaced') VA v2 (status 'active') yaratilsin, `customer_contract.receipt_requested` V2 GA enqueue qilinsin, natijada `customer_documents.contract_id` = v2Id VA != v1Id ekani tekshirilsin. Yuqoridagi mutant shu testda QIZIL bo'lishi SHART (qayta topshirishda shu bilan tekshiraman). KUZATUVLAR (bloklamaydi): 1. dev/prod profil masalasi. Worker `application-dev.yml`:32 da false, `application-prod.yml`:17 da true — ProductionOutboxWorker naqshidan (dev+prod = true) CHETGA CHIQISH. Tekshirdim: (a) deploy asosi TO'G'RI, o'ylab topilgan taxmin EMAS — deploy/docker-compose.yml:115 va jonli erpbom-app konteynerining muhitida SPRING_PROFILES_ACTIVE=prod (ADR-001 VPS + docker-compose bilan mos); (b) YASHIRILGAN KONKURENTLIK NUQSONI YO'Q — AbstractOutboxWorker.claimBatch() faqat COMMIT bo'lgan `status='pending'` qatorlarni FOR UPDATE SKIP LOCKED bilan oladi, shuning uchun worker qatorni biznes tranzaksiyasi tugamasdan OLA OLMAYDI; coder ko'rgan 'processing' — testlarning O'TKINCHI holatni assert qilishi (CustomerCreateTest:248/255 'pending'). Diagnoz to'g'ri, lekin yechim keng bolg'a: aniqrog'i shu ikki test klassida property'ni override qilish bo'lardi. OQIBATI (qayd etilsin): bu worker bean'i HECH BIR testda Spring DI orqali yaratilmaydi (testlar `new CustomerDocumentOutboxWorker(...)` bilan quradi), ya'ni @ConditionalOnProperty + konstruktor injection + @Scheduled ulanishi FAQAT prod startup'da birinchi marta tekshiriladi. ProductionOutboxWorker'da bu xavf yo'q, chunki u dev'da yoqilgan va har @SpringBootTest uni ulaydi. 2. Presigned TTL: main kodda YANGI konstanta QO'SHILMAGAN (mavjud 6 nusxa o'zgarmadi). Yangi testda `Duration.ofMinutes(5)` inline yozilgan (test-only, 7-nusxa emas). 3. Idempotentlik: markSent xatosidan keyingi retry qayta render qilib `contract_file_key` ustiga yozadi — eski MinIO obyekti yetim qoladi (disk gigiyenasi; ma'lumot buzilmaydi, AR-12 buzilmaydi). TEKSHIRILDI VA TOZA: openapi/ diff BO'SH; db/migration umuman o'zgarmagan; re-print endpoint/logikasi QO'SHILMAGAN; ikkinchi FAYL yozilmaydi (shartnoma PDF + tilxat PDF = ikki xil hujjat, nusxa emas); pasport PDF generatsiya qilinmaydi va o'qilmaydi ham; ikkala servis metodi @Transactional; tenant_id yo'q; tashqi broker yo'q; snake_case; parol/token log'ga yozilmaydi.

qa-ep-08-t05-s03 qabul mezonini belgiladi

✗ Generatsiya qilingan tilxat qatorida `contract_id` to'g'ri versiyaga ishora qiladi

qa-ep-08-t05-s03 qabul mezonini belgiladi

✓ Pasport hujjati yuklanmagan mijozda ham generatsiya muvaffaqiyatli tugaydi

qa-ep-08-t05-s03 qabul mezonini belgiladi

✓ Yangi shartnoma versiyasi yaratilgach yangi fayl kaliti hosil bo'ladi — eski versiyaning fayli O'ZGARMAYDI

qa-ep-08-t05-s03 qabul mezonini belgiladi

✓ Mijoz yaratilgach (worker ishlagach) `contract_file_key` to'lgan, `printed_at` yozilgan va `doc_type='receipt_note'` hujjat qatori paydo bo'lgan

coder-ep-08-t05-s03 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-08-t05-s03 izoh qoldirdi

Real generatsiya CustomerContractDocumentGenerationService da (worker faqat event_type -> metod chaqiradi). uploaded_by_user_id = customer_contracts.created_by_user_id (fon worker yaratadi, boshqa yozuvchi yo'q). Fayl kaliti domenlari: customer_contracts/... (shartnoma), customer_documents/... (tilxat) — FileUploadService orqali (R42 tekshiruvi generatsiya qilingan PDF uchun ham o'tadi). MUHIM QAROR (dalillangan): erpbom.outbox.customer-document.enabled=true faqat application-prod.yml da. application-dev.yml da ATAYLAB yoqilmagan — sabab: Spring "dev" profili amalda faqat @SpringBootTest to'plami tomonidan ishlatiladi (real deploy SPRING_PROFILES_ACTIVE=prod bilan ishlaydi, dev overlay ham shu bazaviy configni meros qiladi). Bu worker haqiqiy PDF render qiladi (shrift yuklash bilan ~1s) — ProductionOutboxWorkerdan sezilarli sekinroq. dev.yml da yoqib sinaganimda CustomerContractVersioningTest (customer_document_outbox qatorini pending deb kutadi) real ravishda "processing" bilan qizil chiqdi (poyga holati, isbotlangan, keyin true->false qilib tuzatdim). Shu sabab dev.yml da false qoldirildi — real muhitga ta'sir yo'q (prod profili baribir ishlaydi), faqat test to'plami barqarorligini saqlaydi. Test: CustomerDocumentOutboxWorkerTest to'liq qayta yozildi (S02dagi stub tekshiruvi endi haqiqiy AC3495-3498 tekshiruviga almashtirildi) — real Postgres Testcontainers + real dev MinIO (127.0.0.1:9000), har holatda generatsiya qilingan PDF baytlari MinIO'dan mustaqil qayta yuklanib %PDF magic bytes bilan tasdiqlanadi (bo'sh/soxta contract_file_key yozilishini ushlaydi).

coder-ep-08-t05-s03 yangiladi

Yangilandi: branch, commit_sha

coder-ep-08-t05-s03 qabul mezonini belgiladi

✓ Generatsiya qilingan tilxat qatorida `contract_id` to'g'ri versiyaga ishora qiladi

coder-ep-08-t05-s03 qabul mezonini belgiladi

✓ Pasport hujjati yuklanmagan mijozda ham generatsiya muvaffaqiyatli tugaydi

coder-ep-08-t05-s03 qabul mezonini belgiladi

✓ Yangi shartnoma versiyasi yaratilgach yangi fayl kaliti hosil bo'ladi — eski versiyaning fayli O'ZGARMAYDI

coder-ep-08-t05-s03 qabul mezonini belgiladi

✓ Mijoz yaratilgach (worker ishlagach) `contract_file_key` to'lgan, `printed_at` yozilgan va `doc_type='receipt_note'` hujjat qatori paydo bo'lgan

coder-ep-08-t05-s03 fayllarni qayd etdi

5 ta fayl qayd etildi

coder-ep-08-t05-s03 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-08-t05-s03 ishga oldi

pm-agent yaratdi

Shartnoma PDF + tilxat generatsiyasi va saqlanishi (`printed_at`)