F6 yakuniy `production_in` + sikl yopilishi + 06 ga recheck event (R25)

finalize-ep-07 yangiladi

Yangilandi: pr_url

unknown-agent izoh qoldirdi

[lead] QA topgan AC5 qamrov bo'shlig'i yopildi — commit 7ddd8e2. QA yozgan test qo'llandi (u ishchi daraxtni toza qoldirish uchun patch sifatida qoldirgan edi). Lead mustaqil tasdig'i: (1) toza kodda WorkReviewProductionInIntegrationTest 4/4 yashil, exit 0; (2) QA tasvirlagan mutant (unit_cost -> numeric(18,2), total = unit x qty) qo'yilganda test YIQILADI; (3) `javap` bilan bytecode'da scale=2 (`iconst_2`) ekani tasdiqlandi — ya'ni mutatsiya haqiqatan kuchga kirgan, "survived" yolg'oni emas; (4) mutant tiklandi, `grep -rn MUTANT src/` toza. T04 endi 5/5 barg done.

qa-1 statusni o'zgartirdi Review → Bajarildi

QA (qa-1): 5/5 mezon o'zim tekshirdim — 473/473 test yashil (tasks 221 + employees 252), 4 mutant o'ldirildi (AC4 guard, R25 enqueue, F5/F7). Qarz: AC5 yaxlitlash qoldig'i uchun regressiya testi yo'q (tayyor patch izohda).

qa-1 izoh qoldirdi

QA (qa-1) — EP-07-T04-S04, HEAD 4bc50bc. Hisobotga tayanmadim, hammasini o'zim o'lchadim. TESTLAR (toza surefire, `mvn -Dtest='uz.erpbom.tasks.**,uz.erpbom.employees.**' test`): 473/473 yashil — tasks 221/221 (27 klass), employees 252/252 (21 klass), 0 failure / 0 error, exit=0. MEZONLAR — dalil bilan: 1) AC4 (production_in faqat oxirgi operatsiyada, BIR MARTA) — O'TDI. Test: WorkReviewProductionInIntegrationTest#lastOperationConfirmPostsProductionInOnceAndClosesCycle (2 operatsiyali marshrut, ikkala task sikl boshida yaratilgan): op1 `completed` bo'lganda production_in harakatlari = 0, sikl `in_progress`; op2 dan keyin aynan 1 ta. Mexanizm to'g'ri: ProductionRunCreationService.java:174-177 marshrutning BARCHA operatsiya-taskini sikl ochilganda `saveAll` bilan yaratadi (§8.1), demak "sikldagi hamma task completed" ≡ "bu yakuniy operatsiya" — R20 gating bilan izchil. Ikkinchi marta post qilish yo'li ham yopiq: WorkService.java:193 over-claim cheklovi (`qty_claimed + yangi <= required_qty`) va D1 (faqat brak `qty_claimed`dan chiqadi) tufayli `qty_confirmed` hech qachon `required_qty` dan oshmaydi → `completed` sharti bir marta rost. MUTATSIYA: guard `if (false)` ga aylantirildi → AC4 assertioni (:112) yiqildi = o'ldirildi. 2) AC5 (unit_cost = material + mehnat; yaxlitlash qoldig'i saqlanadi) — O'TDI. Test 3900.00 = material 3000 + ishbay 300+600 ni tasdiqlaydi; qiymat StockInPoster.java:67-77 orqali `total_cost = remaining_cost = totalCost` (unit_cost — HOSILA), ya'ni qoldiq struktura darajasida saqlanadi. QOLDIQ BANDINI O'ZIM SINADIM (commit'dagi test buni ajrata olmaydi — 3900/3 qoldiqsiz): bo'linmaydigan WIP bilan (2 FIFO qatlam 1x1000 + 2x500 = 2000.00, ishbay 300+600) → total_cost = 2900.00, unit_cost = 966.6667 (966.6667 x 3 = 2900.0001) — qoldiq partiyada qoldi. 3) R15 (arxiv variant) — O'TDI. StockInPoster.java:58-62 `productVariantService.isActive` → 409 VARIANT_ARCHIVED, hech narsa yozilmaguncha. Test rollback'ni ham tasdiqlaydi (wip_cost o'zgarmagan, task `in_progress`, work `submitted`, production_in yo'q). 4) R25 (order.recheck) — O'TDI. origin='order' siklda har `confirmed`da qator yoziladi (oraliq operatsiyada ham), manual siklda umuman yozilmaydi. MUTATSIYA: origin filtri olib tashlandi (enqueue umuman qilinmaydi) → R25 testi (:239) yiqildi. 5) Sikl yopilishi — O'TDI: wip_cost = 0, produced_qty = required_qty = 3, status = completed (hammasi DB dan o'qib tekshirilgan). S03 TESTLARIGA ARALASHUV (AC3242/AC3244) — mazmunan tekshirildi, ZAIFLASHTIRMAGAN: - Qo'shilgan "bystander" task hech qanday hisobga kirmaydi: F5/F7 formulalari `production_runs.required_qty` va `wip_cost` ga tayanadi, task soniga emas; ikkala testning raqamli assertionlari (130 -> 150 -> brak_loss 155 -> wip 0; wipB == wipA == 3000.00) o'zgarmagan. - Sabab ham asosli: bu ikki test tasklarni review chaqiruvlari ORASIDA yaratadi, haqiqiy siklda esa hamma task boshida yaratiladi (ProductionRunCreationService) — bystander shu invariantni tiklaydi, F6 ni "yashirmaydi". - MUTATSIYA bilan tasdiqladim (izolyatsiyalangan git worktree'da): * F7 excision olib tashlandi (`- previousWipShare`) -> AC3242:231 yiqildi; * F5 good_share ulushi buzildi (`goodShare = materialOutCost`) -> AC3242:228, AC3244:287 va firstOperationPartialConfirm:116 yiqildi. Ya'ni ikkala test ham hamon o'z spec qoidasini ushlab turibdi. - Bystander'ning ZARURLIGINI ham tekshirdim: ikkala helper'ni no-op qilib qo'ysam, AC3242 (:191) va AC3244 (:266) `409 SOURCE_WAREHOUSE_MISSING` bilan yiqiladi — ya'ni coder aytgan sabab (F6 erta ishga tushishi) haqiqat, to'siq assertion emas. 🟡 TEST QARZI (kod TO'G'RI, lekin regressiya himoyasi yo'q) — AC5 ning "yaxlitlash qoldig'i" bandi commit'dagi testlar bilan ISBOTLANMAGAN. Mutatsiya bilan o'lchadim (javap bilan bytecode tasdiqlangan): `unit_cost` ni spec matnidagidek `numeric(18,2)` ga yaxlitlab `total_cost = unit_cost x qty` qilib yozsak — * coder'ning 3 testi: 3/3 YASHIL (mutant TIRIK qoladi), * mening qoldiq testim: yiqiladi (`expected 2900.00 but was 2900.01` — 1 tiyin O'YLAB TOPILADI, ya'ni partiya qiymati wip_cost dan farq qiladi). Tayyor patch (yozdim va yashil ekanini tekshirdim, 1 test + 1 fixture helper): /tmp/claude-1000/-var-www-erp/5eaf1db1-932c-4070-987e-bbae46118eb6/scratchpad/qa_ac5_rounding_test.patch Ishchi daraxtni toza qoldirish shart bo'lgani uchun repo'ga YOZMADIM — commit qilish lead/coder'da. KUZATUVLAR (bu bargni bloklamaydi, boshqa ish egaligida): - V28 dedup indeksi `status IN ('pending','processing')` ni qamraydi: worker qatorni `processing` ga olib, holatni O'QIB bo'lgandan keyin sikl `completed` bo'lsa, yangi `order.recheck` shu `processing` qatorga coalesce bo'lib YO'QOLADI (buyurtma eskirgan holatda qolishi mumkin). Bu T01-S05 (V28) qarori — S04 unga to'g'ri amal qilgan. - V24 da `origin='order' -> order_id NOT NULL` CHECK yo'q; S04 shu invariantga tayanadi (aks holda outbox `object_id` NOT NULL buzilib review tranzaksiyasi yiqiladi). Hozir API orqali erishib bo'lmaydi (order-origin siklni faqat intake porti yaratadi). - `POST /works/{id}/review` endi `409 SOURCE_WAREHOUSE_MISSING` qaytarishi mumkin (node'da faol `semi_finished` ombor bo'lmasa), kontraktda (`WorkReviewConflict`) bu kod hujjatlanmagan — T00 kontrakt fayliga qo'shilsa yaxshi.

qa-1 qabul mezonini belgiladi

✓ Sikl `completed` bo'lgach `wip_cost = 0` va `produced_qty = required_qty`

qa-1 qabul mezonini belgiladi

✓ `origin='order'` siklda `order.recheck` outbox qatori yoziladi; `manual` da yozilmaydi

qa-1 qabul mezonini belgiladi

✓ R15: arxiv (`active=false`) variantga `production_in` bloklanadi

qa-1 qabul mezonini belgiladi

✓ AC5: `unit_cost` = material + mehnat; yaxlitlash qoldig'i partiya qiymatiga qo'shiladi (jami qiymat saqlanadi)

qa-1 qabul mezonini belgiladi

✓ AC4: `production_in` faqat oxirgi operatsiyada va BIR MARTA; oraliq operatsiyalar omborga hech narsa qo'shmaydi

unknown-agent statusni o'zgartirdi Ishlanyapti → Review

unknown-agent izoh qoldirdi

F6 + R25 qo'shildi (commit 4bc50bc). WorkReviewService.review() SEAM oxiri: (a) confirming'da har doim enqueueOrderRecheckIfApplicable (origin=order siklda order.recheck, dedup coalescing bilan); (b) task shu review'da completed bo'lsa va TaskRepository#existsByProductionRunIdAndStatusNot false qaytarsa (DAG parslanmaydi — sikldagi barcha task completed) finalizeProductionRunIfLastOperation: StockPostingService.postIn(totalCost=wip_cost, qty=required_qty) — unit_cost hosila, yaxlitlash qoldig'i avtomatik saqlanadi. R15 StockInPoster orqali tabiiy bloklanadi (VARIANT_ARCHIVED). 3 ta yangi test (WorkReviewProductionInIntegrationTest): AC4/AC5 + sikl yopilishi, R15 rollback, R25 coalescing/manual istisno. Ikkita eski test (WipIntegrationTest AC3242/AC3244) F6 tomonidan endi to'g'ri yakunlanib ketardi — bystander ikkinchi operatsiya-task bilan tuzatildi (izoh kodda). uz.erpbom.tasks.**: 221/221, uz.erpbom.employees.**: 252/252 — ikkalasi 0 failure/0 error.

unknown-agent yangiladi

Yangilandi: branch, commit_sha

unknown-agent qabul mezonini belgiladi

✓ Sikl `completed` bo'lgach `wip_cost = 0` va `produced_qty = required_qty`

unknown-agent qabul mezonini belgiladi

✓ `origin='order'` siklda `order.recheck` outbox qatori yoziladi; `manual` da yozilmaydi

unknown-agent qabul mezonini belgiladi

✓ R15: arxiv (`active=false`) variantga `production_in` bloklanadi

unknown-agent qabul mezonini belgiladi

✓ AC5: `unit_cost` = material + mehnat; yaxlitlash qoldig'i partiya qiymatiga qo'shiladi (jami qiymat saqlanadi)

unknown-agent qabul mezonini belgiladi

✓ AC4: `production_in` faqat oxirgi operatsiyada va BIR MARTA; oraliq operatsiyalar omborga hech narsa qo'shmaydi

unknown-agent fayllarni qayd etdi

5 ta fayl qayd etildi

backend-1 statusni o'zgartirdi Backlog → Ishlanyapti

pm-agent yaratdi

F6 yakuniy `production_in` + sikl yopilishi + 06 ga recheck event (R25)