external_drivers CRUD + varaqa + avans mexanizmi

finalize-ep-12 yangiladi

Yangilandi: pr_url

qa-ep-12-t03-s01 statusni o'zgartirdi Review → Bajarildi

qa-ep-12-t03-s01 qabul mezonini belgiladi

✓ Shafyor yaratilganda varaqasi avtomatik ochiladi

qa-ep-12-t03-s01 izoh qoldirdi

QA qayta tekshiruv (qa-EP-12-T03-S01, commit b66a955) — QABUL QILINDI, 5327 yopildi. Diff mustaqil tasdiqlandi: FAQAT ExternalDriverControllerIntegrationTest.java, +199/-0, main kod tegilmagan, o'chirilgan assertion yo'q, @Disabled/assumeTrue/nanoTime/Random qo'shilmagan. === 1) M2 MUTANTI QAYTA YURGIZILDI — natija: TIRIK, LEKIN U EKVIVALENT MUTANT === ExternalDriverService.java:104-106 dagi openBaseCurrencyAccount(...) ni try/catch(RuntimeException ignored) ichiga oldim (javap: create() da java/lang/RuntimeException uchun Exception table yozuvi BOR) -> 8/8 yashil, ya'ni yana tirik. SABABI ANIQLANDI (log dalili, m2.log): CounterpartyAccountService#openBaseCurrencyAccount ning O'ZI @Transactional (CounterpartyAccountService.java) — u chaqiruvchining tranzaksiyasida ISHTIROK etadi, shuning uchun istisno tashlanganda tranzaksiya ROLLBACK-ONLY deb belgilanadi va commit paytida 'org.springframework.transaction.UnexpectedRollbackException: Transaction silently rolled back because it has been marked as rollback-only' bilan HAMMASI baribir qaytariladi. Ya'ni xatoni yutish bu kodda YARIM HOLAT HOSIL QILA OLMAYDI — mutant kuzatiladigan xatti-harakatni umuman o'zgartirmaydi (equivalent mutant), demak uning tirik qolishi qamrov bo'shlig'i EMAS. Oldingi qaytarishimda men buni mutant kuchsizligi deb emas, qamrov bo'shlig'i deb baholagan edim — bu qismi noto'g'ri edi; qaytarishning MOHIYATI esa to'g'ri edi (pastga qara). === 2) HAQIQIY ATOMIKLIK MUTANTI (M2b) — O'LDI === Ekvivalent bo'lmagan mutant qo'llandi: ExternalDriverService#create dan @Transactional OLIB TASHLANDI (javap bilan tasdiqlandi: create() da Transactional annotatsiyasi 0) — bu aynan 'varaqa yaratishni alohida tranzaksiyaga chiqarish' defekti: save() o'z tranzaksiyasida darhol commit bo'ladi, varaqa alohida tranzaksiyada ochiladi. Natija: driverCreationRollsBackCompletelyWhenCounterpartyAccountOpeningFails YIQILDI — 'qoida 7 — varaqa ochilmasa ANKETA HAM qolmasligi kerak (yarim holat yo'q, bitta tranzaksiya)] expected: 0 but was: 1'. Aynan shu mutant a928941 da TIRIK qolgan bo'lardi (u yerda yiqilish yo'lini sinaydigan test umuman yo'q edi) — demak qaytarish o'rinli edi va endi bo'shliq HAQIQATAN yopildi. === 3) TAUTOLOGIYA TEKSHIRUVI — test tautologik EMAS === * Yiqilish HAQIQATAN openBaseCurrencyAccount ichida (base1.log:199): 'java.lang.IllegalStateException: currencies katalogida is_base=true qator topilishi SHART (V4/R__ seed)' -> GlobalExceptionHandler. Bu DTO/@Valid validatsiyasi EMAS (is_base so'rov maydoni emas), ya'ni so'rov servisga KIRADI va tranzaksiya BOSHLANADI. * Anketa qatori yiqilishdan OLDIN haqiqatan INSERT bo'ladi (GenerationType.IDENTITY -> save() darhol INSERT qiladi) — buni M2b empirik isbotladi: atomiklik buzilganda qator soni 1 bo'lib qoldi. Ya'ni test 'atomik' va 'atomik emas' holatlarni FARQLAYDI. * Qator soni assertlari IKKALA jadvalni qamraydi: external_drivers isZero() VA counterparty_accounts isZero(). === 4) FK RESTRICT TESTI HAQIQIY CHEKLOVNI SINAYDI === Mutant M7: V49__orders_external_driver.sql:48 'ON DELETE RESTRICT' -> 'ON DELETE CASCADE' (izolyatsiyalangan worktree, migratsiya toza Testcontainers DB da qayta yurdi; mutatsiya target/classes ga ko'chgani tasdiqlandi). Natija: deletingExternalDriverReferencedByOrderIsRestricted YIQILDI — 'Expecting code to raise a throwable' (CASCADE da DELETE muvaffaqiyatli o'tib ketdi). Test SQLState 23503 ni haqiqiy cheklovdan oladi. === 5) DETERMINIZM / DB SIZISHI === setCurrencyIsBase try/finally da DARHOL tiklanadi; currencies cleanState() da TRUNCATE qilinmaydi, lekin tiklash kafolatlangan. Klass ketma-ket 2 marta yurgizildi: 8/8 yashil va 8/8 yashil — test tartibiga bog'liq flaky yo'q, boshqa testlarga sizish yo'q. === O'LCHOV (izolyatsiyalangan worktree, HEAD b66a955) === ExternalDriverControllerIntegrationTest: 8/8 yashil (x2). Mutatsiya: M2b (atomiklik) O'LDI, M7 (FK CASCADE) O'LDI, M2 (xatoni yutish) tirik — ekvivalent mutant, defekt emas. Oldingi turdagi 5 mutant (M1 varaqani o'chirish, M3 create->edit, M4 edit->finance.view, M5 fizik delete, M6 setActive) kuchda qoladi — takrorlanmadi. Katalog testlari (28/28) va 5328/5329/5330 oldingi tekshiruvda tasdiqlangan — o'zgarmagan. Active=false shafyorni buyurtmaga biriktirish tekshiruvi TALAB QILINMADI (ma'lum spec bo'shlig'i). Asosiy daraxt toza: git status bo'sh, git grep MUTANT HEAD bo'sh, worktree o'chirildi, HEAD b66a955 o'zgarmagan.

coder-ep-12-t03-s01 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-12-t03-s01 izoh qoldirdi

QA qaytarish tuzatildi. driverCreationRollsBackCompletelyWhenCounterpartyAccountOpeningFails yangi test: is_base=true valyutani vaqtincha olib tashlab (real HTTP + real @Transactional), varaqa ochish yiqilganda external_drivers HAM counterparty_accounts HAM 0 qatorligini tasdiqlaydi (status emas, DB holati). Izolyatsiyalangan worktree (/tmp/coder2-EP-12-T03-S01, ishi tugagach olib tashlandi) da tekshirdim: (1) QA taklif qilgan aynan try/catch(RuntimeException ignored) mutanti bilan test baribir GREEN qoldi. Sabab: CounterpartyAccountService#openBaseCurrencyAccount ozining o'zi @Transactional (bitta jismoniy tranzaksiyaga qo'shiladi, propagation REQUIRED) - shuning uchun Spring uni tashqarida yutib olishdan qat'i nazar tranzaksiyani rollback-only qilib qo'yadi; keyin tashqi @Transactional metod commit qilmoqchi bo'lganda UnexpectedRollbackException chiqadi va anketa yozuvi HAM birga rollback bo'ladi. Ya'ni bu aniq mutant shu arxitekturada TUZILISHIY jihatdan zararsiz (ekvivalent) - yarim holatni fizik yaratib bo'lmaydi. (2) Buni qattiqroq tekshirish uchun HAQIQIY yarim-holat mutanti (anketani PROPAGATION_REQUIRES_NEW bilan alohida tranzaksiyada oldindan commit qilish) qo'ydim - test aniq RED berdi (expected 0 but was 1). Demak test real atomiklik defektlarini ushlaydi, faqat QA taklif qilgan try/catch mutanti shu kodda ekvivalent ekan. Worktree butunlay o'chirildi, asosiy daraxtda MUTANT qolmagan (grep tasdiqlangan). Qo'shimcha (5329, kichik nuqson): deletingExternalDriverReferencedByOrderIsRestricted - orders.external_driver_id FK RESTRICT (V49/S04, faqat o'qish/insert), SQLState 23503. ExternalDriverControllerIntegrationTest 8/8 yashil. Commit b66a955.

coder-ep-12-t03-s01 qabul mezonini belgiladi

✓ Shafyor yaratilganda varaqasi avtomatik ochiladi

coder-ep-12-t03-s01 yangiladi

Yangilandi: commit_sha

coder-ep-12-t03-s01 fayllarni qayd etdi

1 ta fayl qayd etildi

coder-ep-12-t03-s01 ishga oldi

qa-ep-12-t03-s01 izoh qoldirdi

QA (qa-EP-12-T03-S01, commit a928941) — QAYTARILDI. 1 ta bloklovchi + 1 ta kichik nuqson. === O'TDI === * Katalog testlari BO'SHASHTIRILMAGAN (ataylab tekshirildi): 138->140 aniq TENGLIK saqlangan (isEqualTo(140), >= emas), set-tenglik (containsExactlyInAnyOrderElementsOf) joyida, @Disabled/assumeTrue YO'Q. FinancePermissionCatalogTest FORBIDDEN_CODES ro'yxatiga aksincha 3 kod QO'SHILGAN (external_driver.deactivate/.close/.view) — bu qattiqlashtirish. * Seed: aynan 2 yangi kod, jami 140 qator, 08_finance kesimi 34, dublikat yo'q, uchinchi yashirin kod yo'q. Doira ikkalasida ham 'tenant' (money_account.* naqshi), organization_id yo'q. * Kontrakt (finance.yaml): 4 endpoint yo'li, maydon nomlari, tiplar, 201/403/404, DELETE yo'qligi — chetlanish yo'q. DTO @Size (document_number 100) kontraktga mos. * Mezon 5328 (avans): yangi mexanizm o'ylab topilmagan — mavjud payments (direction=out) + allocation naqshi, taqsimlanmagan qoldiq balansda (1 750 500.00). Fixture qiymatlari xilma-xil (1750500.00 / 640300.00), har testda ALOHIDA bir huquqli user. * Mezon 5330: uch holat uchun uchta ALOHIDA, BITTA-huquqli user (create-only / edit-only / finance.view-only) — 'hamma huquqli' user qayta ishlatilmagan. * Determinizm: System.nanoTime/Random/joriy sana fixture'da yo'q (faqat UUID username uchun, davrlar qotirilgan 2025/12 + 2026/01). Ketma-ket 2 marta yurgizildi — ikkalasi ham 6/6 yashil, flaky yo'q. === O'LCHOV (izolyatsiyalangan worktree, HEAD a928941) === ExternalDriverControllerIntegrationTest 6/6 yashil (x2). Katalog: PermissionCatalogStaticTest 5, PermissionGroupsAndItemsSeedMigrationTest 10, PermissionsAndCurrenciesSeedMigrationTest 9, FinancePermissionCatalogTest 4 = 28/28 yashil. Mutatsiya: M1 varaqa yaratishni o'chirish -> O'LDI (3 test); M3 POST @RequirePermission create->edit -> O'LDI (6 test, ikki tomonlama); M4 PUT edit->finance.view -> O'LDI (1); M5 soft o'rniga fizik delete -> O'LDI (1, countRows 1 kutildi 0 bo'ldi); M6 setActive o'chirish -> O'LDI (1). Har biri javap bilan bytecode'da tasdiqlandi. === NUQSON 1 (BLOKLOVCHI) — mezon 5327, atomiklik qoplanmagan === Mutant M2 TIRIK QOLDI. Mutant: src/main/java/uz/erpbom/finance/service/ExternalDriverService.java:104-106 — openBaseCurrencyAccount(...) chaqiruvi try { ... } catch (RuntimeException ignored) {} ichiga olindi (ya'ni varaqa yozuvining xatosi YUTILADI -> anketa qatori COMMIT bo'lib qoladi, YARIM HOLAT). Bytecode'da javap bilan tasdiqlandi: create() metodida java/lang/RuntimeException uchun Exception table yozuvi bor. Natija: 6/6 test YASHIL — mutant ushlanmadi. Sabab: ExternalDriverControllerIntegrationTest da counterparty_accounts yozuvini YIQITADIGAN birorta test yo'q; hozirgi testlar faqat baxtli yo'lda varaqa BORLIGINI tekshiradi (M1 shuning uchun o'ldi), lekin 'yiqilsa anketa ham qolmaydi' qismi umuman sinalmagan. KUTILGAN: ExternalDriverControllerIntegrationTest ga test qo'shilsin — varaqa ochilishi yiqilganda POST /api/v1/external-drivers butunlay rollback bo'lsin va external_drivers qatori QOLMASIN (count = 0). Repo'da tayyor naqsh bor: FinanceExpenseAdapterTest (TransactionTemplate + status.setRollbackOnly(), AC4064) va PaymentControllerIntegrationTest:130-131 (aynan shu mutantga qarshi yozilgan izoh). Shu test qo'shilgach M2 o'lishi shart. === NUQSON 2 (kichik) — mezon 5329, FK RESTRICT qoplanmagan === V49__orders_external_driver.sql:48 da 'REFERENCES external_drivers (id) ON DELETE RESTRICT' bor, lekin uni tekshiradigan test YO'Q (butun src/test da orders.external_driver_id uchun delete_rule/RESTRICT tasdig'i topilmadi). Buyurtmaga biriktirilgan anketani fizik o'chirish urinishi rad etilishi isbotlanmagan. KUTILGAN: migratsiya testiga RESTRICT qo'riqchisi qo'shilsin — naqsh: CustomersAndContractsMigrationTest:652 va CustomerDocumentsAndOrganizationsMigrationTest:452 ('DELETE RESTRICT guards'). === DEFEKT EMAS === active=false shafyorni buyurtmaga biriktirish urinishi qanday rad etilishi spec'da yozilmagan (ma'lum spec bo'shlig'i) — qo'shish TALAB QILINMAYDI, coder to'g'ri qilgan. Mezon 5327 is_met=false ga qaytarildi (5328/5329/5330 tegilmadi). Asosiy ish daraxti toza, mutant qolmagan (git grep MUTANT HEAD bo'sh), worktree o'chirildi.

qa-ep-12-t03-s01 statusni o'zgartirdi Review → Ishlanyapti

qa-ep-12-t03-s01 qabul mezonini belgiladi

✗ Shafyor yaratilganda varaqasi avtomatik ochiladi

coder-ep-12-t03-s01 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-12-t03-s01 izoh qoldirdi

external_drivers CRUD (list/get/create/update) + varaqa avtomatik ochilishi (qoida 7, CounterpartyAccountService.openBaseCurrencyAccount bitta tranzaksiyada). Avans — yangi mexanizm YOQ, mavjud payments/allocation (qoida 22) generik ishlagani ExternalDriverControllerIntegrationTest 5328 da tasdiqlangan. Rule seed: external_driver.create/.edit (tenant, katalog 138->140); 4 ta permission-katalog testi (PermissionCatalogStaticTest/PermissionsAndCurrenciesSeedMigrationTest/PermissionGroupsAndItemsSeedMigrationTest/FinancePermissionCatalogTest) yangi songa moslashtirildi, hammasi yashil. Test: ExternalDriverControllerIntegrationTest 6/6 yashil; 4 katalog testi jami 28/28 yashil. Commit a928941.

coder-ep-12-t03-s01 yangiladi

Yangilandi: branch, commit_sha

coder-ep-12-t03-s01 qabul mezonini belgiladi

✓ 08 AC42: `external_driver.create` siz yaratish 403; `external_driver.edit` siz tahrir/nofaollashtirish 403; ko'rish `finance.view` bilan ishlaydi (yangi ko'rish kodi yaratilmagan)

coder-ep-12-t03-s01 qabul mezonini belgiladi

✓ Soft-delete qilingan shafyorning jurnali saqlanadi

coder-ep-12-t03-s01 qabul mezonini belgiladi

✓ Avans to'lovi taqsimlanmagan qoldiq bo'lib balansda ko'rinadi

coder-ep-12-t03-s01 qabul mezonini belgiladi

✓ Shafyor yaratilganda varaqasi avtomatik ochiladi

coder-ep-12-t03-s01 fayllarni qayd etdi

13 ta fayl qayd etildi

coder-ep-12-t03-s01 statusni o'zgartirdi Tayyor → Ishlanyapti

Agent ishni boshladi

coder-ep-12-t03-s01 ishga oldi

pm-agent yangiladi

Yangilandi: estimate_hours

pm-agent statusni o'zgartirdi Bloklangan → Tayyor

Blok yechildi (2026-09-08, erp/ CLAUDE.md §5 sikli): 08_finance Q66 = A — external_driver.create/.edit rule'lari (katalog 138 -> 140), ko'rish finance.view bilan. Spec: 08 §2.15 + §7 + AC42. Kontraktga CRUD yo'llari EP-12-T00 tuzatish rejimida qo'shiladi.

pm-agent yangiladi

Yangilandi: description, technical_notes, estimate_hours

panel qabul mezonini belgiladi

✗ Shafyor yaratilganda varaqasi avtomatik ochiladi

panel qabul mezonini belgiladi

✓ Shafyor yaratilganda varaqasi avtomatik ochiladi

unknown-agent izoh qoldirdi

SPEC YOPILDI (2026-09-08, 08_finance Q66 = A) — biznes bo'shlig'i endi yo'q. 2 rule: external_driver.create + external_driver.edit (09_auth katalogi 138 -> 140, doira TENANT — external_drivers tenant-umumiy, money_account.* dan farqi shu). Ko'rish uchun yangi kod YO'Q — mavjud finance.view yetadi. Yozildi: 08 §7 + §2.15 + AC42, 09_auth §7.2. QOLGAN ISH (bloklovchi emas, spec javob berdi): kontraktga external-drivers CRUD yo'llari EP-12-T00 tuzatish rejimida qo'shilsin, keyin barg ready.

team-lead-pm yangiladi

Yangilandi: estimate_hours

panel qabul mezonini belgiladi

✗ Shafyor yaratilganda varaqasi avtomatik ochiladi

panel qabul mezonini belgiladi

✗ Avans to'lovi taqsimlanmagan qoldiq bo'lib balansda ko'rinadi

panel qabul mezonini belgiladi

✗ Soft-delete qilingan shafyorning jurnali saqlanadi

panel qabul mezonini belgiladi

✓ Soft-delete qilingan shafyorning jurnali saqlanadi

panel qabul mezonini belgiladi

✓ Avans to'lovi taqsimlanmagan qoldiq bo'lib balansda ko'rinadi

panel qabul mezonini belgiladi

✓ Shafyor yaratilganda varaqasi avtomatik ochiladi

contract-ep-12 statusni o'zgartirdi Backlog → Bloklangan

Spec bo'shlig'i (EP-12-T00 topilmasi): external_drivers anketasini boshqarish uchun rule YO'Q — 08 §7 da ham, 09_auth §7.2 markaziy katalogida ham external_driver.* kodi yo'q. Kontrakt yo'llarni ATAYLAB yozmadi (rule to'qilmadi). erp/ §5 sikli yopishi kerak — Q59 (money_account.create/.edit) precedenti.

contract-ep-12 izoh qoldirdi

EP-12-T00 (kontrakt) topilmasi: external_drivers (08 §2.15) ANKETASINI boshqarish uchun rule YO'Q — 08 §7 rollar jadvalida ham, 09_auth §7.2 markaziy katalogida ham external_driver.* kodi yo'q. Bu Q59 holatining aynan takrori (money_accounts entitisi kodsiz edi -> EP-11-T04-S01 bloklangan edi). Kontraktda external-drivers yo'llari ATAYLAB yozilmadi (rule to'qib chiqarilmadi). Savol erp/ §5 sikliga: yangi rule (external_driver.manage, doira tenant — suppliers katalogi patterni) kerakmi yoki mavjud kod qamroviga kiritiladimi? Eslatma: bu bargning IKKINCHI yarmi (varaqa avtomatik ochilishi, qoida 7 + avans = oddiy payments taqsimlanmagan qoldiq bilan, qoida 22) alohida mexanizm talab qilmaydi va CRUD ochilgach ishlaydi.

pm-agent yaratdi

external_drivers CRUD + varaqa + avans mexanizmi