LedgerEntryService — insert-only jurnal, idempotentlik, reversal

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-t02-s02 statusni o'zgartirdi Review → Bajarildi

qa-ep-11-t02-s02 izoh qoldirdi

QA (a2030a3) — QABUL. Testlar: LedgerEntryServiceTest 10/10, CounterpartyAccountServiceTest 7/7 = 17, 0 failures (mvn 3.9.16, Testcontainers PG16, izolyatsiyalangan worktree). MUTATSIYA (7 mutant, hammasi O'LDIRILDI): - ON CONFLICT DO NOTHING olib tashlandi -> AC4049 ikkala testi ham yiqildi (DataIntegrityViolation) = 'xato qaytarmaydi' oyog'i qoplangan. - `if (inserted > 0)` olib tashlandi (idempotent no-op'da ham delta) -> AC4049 'balans faqat BIR marta' yiqildi = balans ikkilanishi qoplangan. Qator soni DB count(*) bilan tekshiriladi (tautologiya yo'q). - reverse() da original qator UPDATE qilindi -> AC4050 va AC4052 yiqildi = insert-only invariant qoplangan. - negate() olib tashlandi -> AC4050 yiqildi. - record() delta = ZERO -> 4 test yiqildi (AC4049, AC4050, AC4052). - ledger.adjust huquq tekshiruvi olib tashlandi -> AC4051 huquq testi yiqildi (grep emas, real PermissionEvaluator + user_permissions). - notes majburiyligi olib tashlandi -> AC4051 izoh testi yiqildi. Ikkala yarmi MUSTAQIL ushlandi. - due_date validatsiyasi olib tashlandi -> §2.2 ikkala testi yiqildi. SPEC: entity V38 DDL bilan aynan mos (12 ustun, numeric(18,2), varchar(30)/varchar(255), updated_at YO'Q). source_type to'plami spec §2.3 = kod = DDL CHECK — 9 qiymat, farq yo'q, to'qilgan qiymat yo'q. Insert-only: ledger_entries uchun hech qanday UPDATE/DELETE yo'q (save() faqat yangi entity bilan). Fixture xilma-xilligi: 20 yozuv = 2 varaqa (supplier+customer), 4 source_type, 4 reversal, 16 xil summa, ikki ishora, tiyinli qiymatlar; amount=0 yoki qotib qolgan qiymat yo'q; USD varaqa alohida testda. Kontraktga tegilmagan (bu bargda controller/DTO yo'q). KUZATUVLAR (mezonlarda talab qilinmagan, bargni bloklamaydi — keyingi barg uchun): 1. qoida 40 / kontraktdagi `409 PERIOD_CLOSED` bu servisda YO'Q — coder javadoc'da ataylab REST qatlamiga qoldirgan (LedgerEntryService.java:33-40), lekin technical_notes'da talab qilingan edi. Jurnalga yagona yozuvchi shu servis bo'lgani uchun qorovul shu yerda bo'lmasa, hech narsa majburlamaydi. 2. qoida 32 (`due_date` HISOBI) ham yo'q — servis faqat mavjud/yo'qligini validatsiya qiladi, qiymatni chaqiruvchi beradi (LedgerEntryService.java:249-262). Modul chegarasi nuqtai nazaridan asosli, lekin technical_notes'dan chetlanish. 3. §2.1 dagi `currencies.active` ilova tekshiruvi repo'da hali ham hech qayerda yo'q — S02 USD varaqani getOrCreate orqali ochadi, active tekshiruvisiz (S01 QA topilmasi hamon ochiq). 4. Idempotentlik kalitida `account_id` YO'Q (spec §2.2 shunday belgilagan). `record()` kalit boshqa varaqaga tegishli chiqsa IllegalStateException beradi. `wage_accrual` ni payroll_run'dan yozadigan chaqiruvchi `source_ref_line_id` ni to'ldirishi SHART — aks holda birinchi hodimdan keyingilari jimgina yutiladi. 5. AC4052 testida reversal oyoqlarining kutilgan yig'indisi `reversal.getAmount()` dan olinadi, shu sababli negate() mutanti AYNAN shu testdan tirik o'tadi; ishorani AC4050 testi qoplaydi (mezon qoplangan, lekin AC4052 o'zi yolg'iz yetarli emas).

qa-ep-11-t02-s02 qabul mezonini belgiladi

✓ Balans keshi jurnal yig'indisiga teng — 20 ta aralash yozuvdan keyin test bilan tasdiqlangan

qa-ep-11-t02-s02 qabul mezonini belgiladi

✓ `manual_adjustment` izohsiz yozilmaydi va `ledger.adjust` huquqisiz rad etiladi

qa-ep-11-t02-s02 qabul mezonini belgiladi

✓ Reversal'dan keyin original qator o'z joyida qoladi, balans esa nolga qaytadi

qa-ep-11-t02-s02 qabul mezonini belgiladi

✓ Bitta faktura tasdig'ini ikki marta yuborish IKKINCHI qarz qatorini yozmaydi va xato ham qaytarmaydi (idempotent)

coder-ep-11-t02-s02 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-11-t02-s02 izoh qoldirdi

4/4 AC yashil (LedgerEntryServiceTest, 10 test + regressiya CounterpartyAccountServiceTest 7 test, hammasi yashil). technical_notes'dan 2 ta ONGLI chetlanish (QA/keyingi barg uchun eslatma): 1) Idempotentlik: technical_notes "DataIntegrityViolation yutiladi" deydi, lekin bu yashirin xato beradi: ledger_entries yozuvi va CounterpartyAccount balans yangilanishi BITTA @Transactional ichida (qoida 6); Postgres'da constraint xatosidan keyin SAVEPOINT'siz o'sha tranzaksiya "aborted" holatga o'tadi va keyingi UPDATE (balans) "current transaction is aborted" bilan yiqiladi. Shu sababli CounterpartyAccountRepository#insertIfAbsent (EP-11-T02-S01) da allaqachon tanlangan "INSERT ... ON CONFLICT DO NOTHING" + qayta SELECT naqshini LedgerEntryRepository#insertIfAbsent uchun ham ishlatdim - funksional natija bir xil (AC4049 qanoat), lekin exception-catch emas. 2) "Yopilgan davrga yozuv - period_closed (qoida 40)": qoida 40 matni period_id muhrini faqat 6 ta hujjat turiga tayinlaydi (payments/money_transfers/expense_entries/payroll_runs/cash_counts/reconciliations); ledger_entries bu ro'yxatda yo'q va V38 da period_id ustuni yo'q. openapi/v1/finance.yaml dagi POST /api/v1/ledger-entries tavsifida entry_date uchun 409 PERIOD_CLOSED tekshiruvi bor - bu HTTP/REST qatlamiga (kelgusi controller bargi) yoki alohida AccountingPeriod integratsiyasiga tegishli deb hisobladim, chunki: (a) berilgan 4 ta AC buni talab qilmaydi, (b) bu servisga berilgan kontekstda AccountingPeriod repository/service yo'q, (c) taxmin qilib yozish xavfli. Controller bargi yozilganda shu eslatma hisobga olinishi kerak.

coder-ep-11-t02-s02 yangiladi

Yangilandi: branch, commit_sha

coder-ep-11-t02-s02 qabul mezonini belgiladi

✓ Balans keshi jurnal yig'indisiga teng — 20 ta aralash yozuvdan keyin test bilan tasdiqlangan

coder-ep-11-t02-s02 qabul mezonini belgiladi

✓ `manual_adjustment` izohsiz yozilmaydi va `ledger.adjust` huquqisiz rad etiladi

coder-ep-11-t02-s02 qabul mezonini belgiladi

✓ Reversal'dan keyin original qator o'z joyida qoladi, balans esa nolga qaytadi

coder-ep-11-t02-s02 qabul mezonini belgiladi

✓ Bitta faktura tasdig'ini ikki marta yuborish IKKINCHI qarz qatorini yozmaydi va xato ham qaytarmaydi (idempotent)

coder-ep-11-t02-s02 fayllarni qayd etdi

5 ta fayl qayd etildi

coder-ep-11-t02-s02 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-11-t02-s02 ishga oldi

pm-agent yaratdi

LedgerEntryService — insert-only jurnal, idempotentlik, reversal