To'lov CRUD (`draft`) — oyoqlar, valyuta va yig'indi validatsiyasi

pm-agent yangiladi

Yangilandi: estimate_hours

pm-agent yangiladi

Yangilandi: estimate_hours

team-lead-pm yangiladi

Yangilandi: estimate_hours

finalize-ep-11 yangiladi

Yangilandi: pr_url

qa-ep-11-t05-s01 statusni o'zgartirdi Review → Bajarildi

qa-ep-11-t05-s01 izoh qoldirdi

QA 2-aylana — QABUL. Testlar: PaymentControllerIntegrationTest 13/13 yashil (surefire XML: tests=13 failures=0 errors=0 skipped=0). Mutatsiya tekshiruvi izolyatsiyalangan worktree (279963f) da, har mutant alohida: - M3 (qoida 40, resolveActivePeriodId: findByIsActiveTrue() -> findAll().stream().findFirst()) => O'LDI. creatingSplitPaymentSucceedsAndIsReadableAfterward:277 AssertionFailedError expected: 10L (aktiv 2026-01) but was: 9L (yopilgan 2025-12). Fixture koddan tasdiqlandi: insertActivePeriod() avval insertPeriod(2025,12,"closed",false) — kichikroq id, keyin insertPeriod(2026,1,"open",true); entry_date=2025-12-15 ataylab YOPILGAN oyda; assert isEqualTo(activePeriodId()) — aynan tenglik, activePeriodId() mustaqil SQL WHERE is_active = true. - M4 (mezon 4083 "saqlanmaydi" yarmi, create() @Transactional ga noRollbackFor = ValidationException.class qo'shildi) => O'LDI. creatingWithMismatchedMovementsSum...:135 countRows("payments") expected: 0 but was: 1 (yetim draft). 1-aylanadagi B1 yopildi. Regressiya (fixture o'zgargani uchun): M1 (PaymentService.java:433 Sigma tekshiruvi o'chirildi) => O'LDI, 400 o'rniga 201. M2 (:414 qoida 33 valyuta tekshiruvi o'chirildi) => O'LDI, 400 o'rniga 201. Kontrakt: git diff b3fc179 HEAD -- openapi/ BO'SH. git grep MUTANT|if (true) HEAD -- src/ BO'SH; worktree o'chirildi, ish daraxti toza. Mezon 4083/4084/4085 tasdiqlandi.

qa-ep-11-t05-s01 qabul mezonini belgiladi

✓ Oyoqlar yig'indisi jamiga teng bo'lmasa to'lov saqlanmaydi va xato `fields[]` da oyoqni ko'rsatadi

coder-ep-11-t05-s01 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-11-t05-s01 statusni o'zgartirdi Tayyor → Ishlanyapti

coder-ep-11-t05-s01 izoh qoldirdi

QA qaytarish 1 tuzatildi. BLOKLOVCHI 1 (4083 saqlanmaslik yarmi): creatingWithMismatchedMovementsSumReturnsValidationErrorPointingAtMovements testiga countRows("payments")==0 va countRows("payment_movements")==0 assert qo'shildi (400 dan keyin DB darajasida). BLOKLOVCHI 2 (qoida 40): insertActivePeriod() fixture endi 2 ta davr qo'yadi (yopilgan 2025-12 kichikroq id, aktiv 2026-01); creatingSplitPaymentSucceedsAndIsReadableAfterward entry_date ni yopilgan davrga qo'yib period_id ni aynan activePeriodId() ga TENG tekshiradi (avval > 0 edi). Ishlab chiqarish kodi o'zgarmadi (QA kod to'g'ri deb tasdiqlagan). Izolyatsiyalangan git worktree (/tmp) da ikkala mutant (findByIsActiveTrue -> findAll().findFirst(); create() ga noRollbackFor=ValidationException.class) qayta tiklanib, ikkala test aynan shu yerda YIQILGANI tasdiqlandi, so'ng mutant asl holiga qaytarilib (cp .orig), grep MUTANT bo'sh ekani tekshirilgach commit qilindi. 13/13 test yashil. commit 279963f.

coder-ep-11-t05-s01 yangiladi

Yangilandi: commit_sha

coder-ep-11-t05-s01 fayllarni qayd etdi

1 ta fayl qayd etildi

coder-ep-11-t05-s01 qabul mezonini belgiladi

✓ Oyoqlar yig'indisi jamiga teng bo'lmasa to'lov saqlanmaydi va xato `fields[]` da oyoqni ko'rsatadi

system statusni o'zgartirdi Ishlanyapti → Tayyor

Avtomatik bo'shatildi: 60 daqiqa signal yo'q

system bo'shatdi

Avtomatik bo'shatildi: 60 daqiqa signal yo'q

qa-ep-11-t05-s01 statusni o'zgartirdi Review → Ishlanyapti

qa-ep-11-t05-s01 izoh qoldirdi

QA (qa-EP-11-T05-S01) — QAYTARILDI. Baza: 7b78215, izolyatsiyalangan worktree'da mutatsiya tekshiruvi. O'LCHOV: PaymentControllerIntegrationTest — 13/13 yashil (surefire: Tests run: 13, Failures: 0, 37.75s). Coder da'vosi tasdiqlandi. Kontrakt muzlatilgan: `git diff b3fc179 HEAD -- openapi/` BO'SH. MUTATSIYA NATIJASI (har biri javap bilan bytecode'da tasdiqlangan): - M1 (Sigma tekshiruvi o'chirildi, PaymentService.java:433) -> O'LDI (4083 testi 400 o'rniga 201 oldi). - M2 (valyuta tengligi o'chirildi, :414) -> O'LDI (4084 testi yiqildi). - M5 (draft kontragent balansiga tegdi) -> O'LDI (4085 testi 0.00 o'rniga 500000.00 oldi). - M3 (qoida 40 muhri buzildi: findByIsActiveTrue() -> findAll().findFirst()) -> TIRIK QOLDI, 13/13 yashil. - M4 (create() da noRollbackFor ga ValidationException qo'shildi) -> TIRIK QOLDI, 13/13 yashil. BLOKLOVCHI 1 — mezon 4083 ning "to'lov SAQLANMAYDI" yarmi umuman tekshirilmagan. PaymentService.create() `paymentRepository.save(payment)` ni oyoq validatsiyasidan OLDIN bajaradi (PaymentService.java:224 save, :228 validateAndBuildMovements) — "saqlanmaslik" faqat tranzaksiya rollback'iga tayanadi, lekin BIRORTA test buni tasdiqlamaydi. Test PaymentControllerIntegrationTest.java:120-125 faqat 400 + fields[] ni tekshiradi. ISBOT: M4 mutanti bilan `payments` da YETIM draft qatori qoladi va 13/13 yashil qolaveradi. QA probe qo'shganda (`assertThat(countRows("payments")).isZero()`) M4 da test yiqildi (expected: 0 but was: 1), mutantsiz kodda esa o'tdi — ya'ni kod TO'G'RI, TEST YETISHMAYDI. QILINSIN: creatingWithMismatchedMovementsSumReturnsValidationErrorPointingAtMovements testiga (PaymentControllerIntegrationTest.java:125 dan keyin) `assertThat(countRows("payments")).isZero();` va `assertThat(countRows("payment_movements")).isZero();` qo'shilsin. Xuddi shu tekshiruv valyuta (4084) va bank (Q28) testlariga ham qo'shilsin — u yerda ham save validatsiyadan oldin turadi. BLOKLOVCHI 2 — qoida 40 (aktiv davr muhri) qoplanmagan; fixture xilma-xilligi nuqsoni. Board technical_notes: "period_id = aktiv davr (is_active), entry_date boshqa oyga tegishli bo'lsa ham". Kodda to'g'ri (PaymentService.resolveActivePeriodId(), :442), lekin YAGONA assert — PaymentControllerIntegrationTest.java:243 `period_id > 0`. Sabab: insertActivePeriod() (test :~470) DB ga FAQAT BITTA davr (2026-01, is_active=true) qo'yadi. Shu sababli is_active ni butunlay e'tiborsiz qoldirgan M3 mutanti butun to'plamdan tirik o'tdi (javap: invokeinterface findAll:()Ljava/util/List; — mutatsiya haqiqatan kompilyatsiya bo'lgan). QILINSIN: fixture'ga KAMIDA 2 ta davr qo'yilsin — mas. 2026-01 is_active=false (yopilgan) va 2026-02 is_active=true; to'lov `entry_date` = 2026-01-15 (AKTIV BO'LMAGAN oyga tegishli) bilan yaratilsin va javobdagi `period_id` AYNAN aktiv (2026-02) davr id'siga TENG ekani assert qilinsin (isEqualTo(activePeriodId()), `isPositive()` EMAS). Shu test M3 ni o'ldirishi shart. TAVSIYA (bloklamaydi) — 4085 fixture'i kuchsiz baza ustida. draftPaymentDoesNotChangeAnyBalance (test :156-170) balansni DRAFTDAN OLDIN o'qimaydi: fixture ikkala balansni ham 0 bilan yaratadi (counterparty_accounts current_balance=0, money_accounts default 0) va assert qattiq "0.00" ga qilingan. M5 uni o'ldirdi, shuning uchun mezon bajarilgan deb belgilandi, lekin baza nolligicha qolsa "balansni 0 ga tenglashtirib yuborish" turidagi nuqson tirik o'tadi. Kuchaytirish: fixture'da nolmas boshlang'ich qoldiq (mas. kontragent 250000.00, kassa 1000000.00) qo'yilsin, qiymatlar draftdan OLDIN o'qilsin va KEYIN o'sha o'qilgan qiymat bilan solishtirilsin. TAVSIYA 2 (bloklamaydi) — qoida 39 (`origin` token `aud` idan) qoplanmagan: hamma test `web` aud bilan ishlaydi, `origin` faqat bitta joyda "back_office" deb assert qilinadi (:239). resolveOrigin() ni doim back_office qaytaradigan qilib buzsa, hech bir test yiqilmaydi. customer_app / driver_app kanali S02 da sinalsa ham bo'ladi, lekin `origin` muhri shu bargda qo'yilyapti. O'TGAN TEKSHIRUVLAR: mezon 4084 — fixture haqiqatan HAR XIL valyuta (USD varaqa + UZS kassa), xato fields[] da AYNAN `movements[0].money_account_id` ni ko'rsatadi ✓. Mezon 4085 — kod darajasida draft yo'lida LedgerEntryService/MoneyMovementService.record CHAQIRILMAYDI (MoneyMovementService dan faqat ACCOUNT_TYPE_BANK konstantasi olinadi), test ledger_entries va money_movements qatorlari 0 ekanini ham tekshiradi ✓. Oyoq soni xilma-xil (1 oyoq va 2 oyoqli split 5 600 000 + 2 400 000), summalar turlicha (100 / 1000 / 1500 / 2000 / 500 000 / 8 000 000) ✓. Cross-cutting: @Transactional hamma DB metodida (o'qishlar readOnly=true), tenant_id yo'q, numeric(18,2) precision, fizik DELETE yo'q (payment_movements draft almashtirish — insert-only qoidasi buzilmaydi), parol/token log'ga yozilmaydi ✓. QAYTARISH SHARTI: 2 ta bloklovchi yopilsin, so'ng M3 va M4 mutantlari QAYTA yurgiziladi — ikkalasi ham O'LISHI shart.

qa-ep-11-t05-s01 qabul mezonini belgiladi

✗ Oyoqlar yig'indisi jamiga teng bo'lmasa to'lov saqlanmaydi va xato `fields[]` da oyoqni ko'rsatadi

qa-ep-11-t05-s01 qabul mezonini belgiladi

✓ `draft` to'lov hech qanday balansga ta'sir qilmaydi (kontragent ham, kassa ham o'zgarmaydi)

qa-ep-11-t05-s01 qabul mezonini belgiladi

✓ USD varaqasiga UZS kassadan oyoq qo'shishga urinish rad etiladi (08 AC9 ning taqiq qismi)

coder-ep-11-t05-s01 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-11-t05-s01 izoh qoldirdi

3/3 AC yashil (13/13 test, PaymentControllerIntegrationTest). Qamrov: payments+payment_movements draft CRUD (create/update/cancel/get/list) — V40 jadvallariga (allaqachon migratsiya qilingan) mos entity/repo/service/controller. Qoida 20 (yig'indi), 33 (valyuta), Q28 (bank oyoq qulfi), 39 (origin=aud), 40 (aktiv davr muhri) — implementatsiya qilindi. draft hech qanday ledger_entries/money_movements yozmaydi (4085). ATAYLAB QILINMAGAN (keyingi bargларга qoldirilgan, spec bo'shlig'i EMAS): payment_allocations (S05 qamrovi — PaymentCreateRequest.allocations maydoni o'qilmaydi, DetailResponse.allocations doim bo'sh []), PERIOD_CLOSED tekshiruvi (T07-S01 markazlashgan guard), GET /payments mijoz egalik filtri (kontraktda bor, lekin bu leaf texnik_notes'ida yo'q). Muhandislik qarori: driver_app kanalida ham back_office kabi payment.create rule talab qilinadi (customer_app'dan farqli, kontrakt faqat customer_app uchun aniq "rule talab qilinmaydi" deydi). Yon topilma: hybrid create/cancel'da @Transactional(noRollbackFor=ForbiddenException.class) kerak bo'ldi (OrderService#receive naqshi) — aks holda deny audit-log rollback bilan yo'qolib qolardi (o'zim yozgan testda ushlandi).

coder-ep-11-t05-s01 yangiladi

Yangilandi: branch, commit_sha

coder-ep-11-t05-s01 qabul mezonini belgiladi

✓ `draft` to'lov hech qanday balansga ta'sir qilmaydi (kontragent ham, kassa ham o'zgarmaydi)

coder-ep-11-t05-s01 qabul mezonini belgiladi

✓ USD varaqasiga UZS kassadan oyoq qo'shishga urinish rad etiladi (08 AC9 ning taqiq qismi)

coder-ep-11-t05-s01 qabul mezonini belgiladi

✓ Oyoqlar yig'indisi jamiga teng bo'lmasa to'lov saqlanmaydi va xato `fields[]` da oyoqni ko'rsatadi

coder-ep-11-t05-s01 fayllarni qayd etdi

17 ta fayl qayd etildi

coder-ep-11-t05-s01 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-11-t05-s01 ishga oldi

pm-agent yaratdi

To'lov CRUD (`draft`) — oyoqlar, valyuta va yig'indi validatsiyasi