`ProductionRunCreationService` — sikl + task-per-operatsiya (R17/R19/R16)

finalize-ep-07 yangiladi

Yangilandi: pr_url

qa-ep-07-t02-s02 statusni o'zgartirdi Review → Bajarildi

qa-ep-07-t02-s02 izoh qoldirdi

QA: AC3202 mustaqil tasdiqlandi — 5/5 mezon yopildi, done. TESTLAR (qulf ostida, clean'siz, HEAD=8aadbd2): ProductionRunCreationServiceTest 11/11 yashil (avvalgi 10 + yangi createManual_rejectsArchivedVariant_noRunTaskOrOutboxRowCreated). 3198-3201 yengil sanity — o'sha 10 test o'zgarmagan va yashil (ular 2026-08-25 14:05 izohida mutatsiya bilan tasdiqlangan edi). (a) FAIL-FAST va TARTIB — ProductionRunCreationService.java:154-157. Gate `productionRunRepository.save(run)` (:172), `taskRepository.saveAll` (:177) va outbox `enqueueIfAbsent` (:247) dan OLDIN. Tartib kontraktdagi ketma-ketlikka aynan mos: VARIANT_TYPE_INVALID (:145) -> RECIPE_VERSION_MISSING (:150) -> VARIANT_ARCHIVED (:154). Gate oldida faqat o'qish bor (productVariantService.get, findActiveRoute) — rad etilganda hech qanday yozuv tug'ilmaydi, tasks.yaml:317-320 talabi bajarilgan. (b) KOD / HTTP / MESSAGE — kod aynan `VARIANT_ARCHIVED` (:90). HTTP xaritasi NOM bo'yicha emas, TIP bo'yicha tekshirildi: BusinessConflictException -> GlobalExceptionHandler.java:96-99 -> HttpStatus.CONFLICT (409), body `code` = ex.getCode() (respond(), :340-343). Zanjir jonli integratsiya testlari bilan ham isbotlangan (ayni tip + ayni kod): StockReturnConfirmedPostingTest.java:163-164 va PurchaseReceiptConfirmedPostingTest.java:176 — HttpStatus.CONFLICT + code=VARIANT_ARCHIVED. Message (:156) kontrakt misoliga HARFMA-HARF mos (tasks.yaml:1138 'Arxivlangan variant uchun ishlab chiqarish sikli ochilmaydi'). (c) MUTATSIYA TEKSHIRUVI — 3/3 O'LDIRILDI (har biridan keyin FAQAT shu fayl `git checkout -- <fayl>` bilan tiklandi; oxirida `git diff 8aadbd2 -- <fayl>` BO'SH, baseline qayta 11/11 yashil): - M1 gate butunlay o'chirildi => rejectsArchivedVariant FAIL (Test:224, istisno umuman otilmadi). - M2' gate `productionRunRepository.save(run)` dan KEYINGA ko'chirildi (istisno HALI ham to'g'ri kod bilan otiladi) => FAIL AYNAN verifyNoInteractions'dan: `NoInteractionsWanted ... found these interactions on mock 'productionRunRepository'` (Test:229). 🔴 Demak verifyNoInteractions TAUTOLOGIK EMAS — fail-fast kafolatini haqiqatan ushlaydi, faqat istisno otilishini emas. (Eslatma: gate'ni butun blok oxiriga ko'chirgan dastlabki urinish IndexOutOfBounds bilan yiqildi — bu test stub'i sababli 'yolg'on kill' bo'lgani uchun M2' bilan izolyatsiya qilib qayta o'tkazildi.) - M3 shart teskari qilindi (`if (variant.active())`) => 1 FAILURE + 7 ERROR: rejectsArchivedVariant + BARCHA happy-path testlar (createsRunAndOneTaskPerOperation, snapshotsActiveRouteOnce, nullOrganizationId_resolves..., writesWarningOutboxRow..., treatsZeroNormPerDay..., writesNoWarning..., explicitOrganizationId...) yiqildi. (d) MANBA — gate `variant.active()` ni ProductVariantResponse.java:30 (boolean active) dan o'qiydi, testda stubVariant(productType, active) 12-pozitsiyaga aynan shu maydonga yozadi. Yo'nalish to'g'ri (active=false da rad, active=true da o'tadi) — M3 buni ikki tomonlama isbotladi. Gate erishib bo'ladigan kod: ProductVariantService.get(:139) arxiv variantni FILTRLAMAYDI (faqat deleted_at), demak active=false variant haqiqatan qaytadi va gate'ga yetadi. CROSS-CUTTING: createManual @Transactional (:137) — gate ayni tranzaksiya chegarasida, rad etilganda rollback bilan izchil (AR-3). Bu o'zgarish yangi migratsiya/repository/log qo'shmaydi; yangi tenant_id yo'q (AR-1); parol/token log'ga yozilmaydi. SPEC/DOC: 07_tasks.md:444-445 R15 matni hali ESKI (faqat production_in dan gapiradi, sikl ochilishi haqida jim) — bu ZIDDIYAT emas, ma'lum doc qarzi (team-lead 14:31 izohi). Haqiqat manbai kontrakt; kod kontraktga amal qiladi. Blocked sababi EMAS. KUZATUVLAR (bloklovchi emas, keyingi bargga): 1. Yangi test message matnini tekshirmaydi — faqat kodni. Kontrakt misolidagi matn drifti test bilan ushlanmaydi (hozir harfma-harf mos). 2. Tekshiruvlar TARTIBI test bilan qotirilmagan (mas. arxiv + noto'g'ri tur bir vaqtda). Kontraktda ProductionRunCreateConflict tartibni normativ deb aytmagani uchun (WorkClaimConflict'dan farqli — u 'UC3 tekshiruvlari tartibida' deydi) bu talab emas. 3. Qamrovdan tashqarida (team-lead allaqachon qayd etgan): buyurtmadan avto oqim HTTP yo'liga ega emas, bu gate uni qoplamaydi. KOD O'ZGARTIRILMADI (mutatsiyalar tiklandi). Kontrakt va erp/docs ga TEGILMADI. Parallel EP-07-T04 fayllariga tegilmadi — ular 58b5fec da o'z egasi tomonidan commit qilindi.

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

✓ Arxiv (`active=false`) variant uchun sikl ochishga urinish rad etiladi (R15 bilan izchil)

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

coder-ep-07-t02-s02 yangiladi

Yangilandi: commit_sha

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

✓ Arxiv (`active=false`) variant uchun sikl ochishga urinish rad etiladi (R15 bilan izchil)

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

2 ta fayl qayd etildi

coder-ep-07-t02-s02 statusni o'zgartirdi Tayyor → Ishlanyapti

Agent ishni boshladi

coder-ep-07-t02-s02 ishga oldi

team-lead-pm statusni o'zgartirdi Bloklangan → Tayyor

Kontrakt tuzatildi, AC3202 endi haqiqiy — keyingi sessiyada coder olsin.

team-lead-pm izoh qoldirdi

🔓 BLOK YECHILDI — foydalanuvchi biznes qarori 2026-08-25 + kontrakt tuzatildi. QAROR: arxivlangan (active=false) variantga ishlab chiqarish sikli OCHILMAYDI — fail-fast. Sabab: spec R15 to'siqni faqat production_in ga qo'ygan edi, natijada sikl ochilib homashyo sarflanardi, keyin kirim bloklanib WIP abadiy osilib qolardi. production_in to'sig'i o'z joyida QOLADI. KONTRAKT (tasks.yaml, qayta muzlatilgan, mock :4010 tekshirilgan): • :123-126 xato katalogi — VARIANT_ARCHIVED endi ikki gate uchun • :314-320 eski 'sikl ochilishi BLOKLANMAYDI' matni OLIB TASHLANDI, o'rniga fail-fast gate • :1114-1117, :1136-1140 — ProductionRunCreateConflict ga VARIANT_ARCHIVED + 'archived' misoli BAJARILISHI KERAK: ProductionRunCreationService da arxiv variant tekshiruvi -> 409 VARIANT_ARCHIVED; AC3202 endi HAQIQIY va bajarilishi shart. Qolgan 4 mezon allaqachon QA bilan tasdiqlangan. ⚠️ QAMROVDAN TASHQARIDA (bu bargda HAL QILINMAYDI): siklning ikkinchi ochilish yo'li — buyurtmadan avto oqim (outbox/@Scheduled, 06_orders chaqiradi) — HTTP yo'liga ega emas, shuning uchun bu gate uni qoplamaydi. Arxiv variantga buyurtma tushsa WIP tuzog'i qaytadi. Ochiq savol EP-07-T02 izohida. ⚠️ Doc qarzi: 07_tasks R15 matni hali eski — CLAUDE.md §5 sikli bilan yangilanadi.

contract-ep-07 izoh qoldirdi

KONTRAKT YANGILANDI (T00 tuzatish, 2026-08-25) — AC3202 endi HAQIQIY va bajarilishi kerak. Foydalanuvchi biznes qarori: arxivlangan (active=false) variantga ishlab chiqarish sikli OCHILMAYDI (fail-fast). openapi/v1/tasks.yaml da POST /api/v1/production-runs endi 409 VARIANT_ARCHIVED qaytaradi (ProductionRunCreateConflict ga qo'shildi, misol nomi 'archived'). Tekshiruv sikl va tasklar yaratilishidan OLDIN — rad etilganda hech qanday yozuv tug'ilmaydi. production_in dagi mavjud R15 to'sig'i O'ZGARMAYDI (ikkita gate, ikkinchisi zaxira). Sabab: kafolatlangan WIP boshi berk ko'chasi. Kontraktni QAYTA O'QI va S02 ni davom ettir (unblock qarori team-lead'da). Spec 07_tasks R15 matni hozircha eski — bu doc qarzi, erp/ §5 sikli bilan yangilanadi; kod KONTRAKTGA amal qiladi.

qa-ep-07-t02-s02 izoh qoldirdi

QA: 3198-3201 mustaqil tasdiqlandi (mutatsiya bilan), yagona qolgan to'siq AC3202 kontrakt ziddiyati. Testlar (qulf ostida, clean'siz, HEAD=da359c1): ProductionRunCreationServiceTest 10/10, ProductionOutboxNormOrRateMissingMigrationTest 5/5 (Testcontainers PG16), ProfessionNormServiceTest 17/17 — 32/32 yashil. Mutatsiya tekshiruvi — 9/9 O'LDIRILDI (har biridan keyin git checkout bilan tiklandi, ish daraxti toza): - M1 [3199] snapshot -> har task uchun findActiveRoute qayta o'qish => createManual_snapshotsActiveRouteOnce... FAIL (verify times(1)). - M2 [3200] berilgan organizationId e'tiborsiz => 2 FAIL + 6 ERROR, jumladan createManual_explicitOrganizationIdIsAuthoritative_employeeLookupNeverCalled. - M3 [3200] employee rezolyutsiyasi o'chirildi => createManual_nullOrganizationId_resolvesFromCreatorsEmployeeRecord + ..._rejectsWithValidationError FAIL. - M4 [3201] outbox enqueueIfAbsent o'chirildi => 2 FAIL. - M5 [3201] yashirin default (normMissing=false) => 2 FAIL. - M6 [3198] bitta operatsiya tashlab ketildi => hasSize(2) FAIL. - M7 [3198] task.required_qty siklnikidan farq qildi => FAIL. - M8 [3198] task.status=new emas => FAIL. - M9 [3201/DB] V29 CHECK qo'shimchasi o'chirildi => newEventTypeIsAcceptedByCheckConstraint + enqueueIfAbsentWritesRow... ERROR (haqiqiy PG CHECK). 3201 DB darajasi: faqat mock EMAS — ProductionOutboxNormOrRateMissingMigrationTest:125-155 haqiqiy PG'ga yozadi va production_outbox dan status='pending' + payload->>'norm_missing' ni raw JDBC bilan o'qiydi, dedup (ikkinchi chaqiruv 0 qator) ham tasdiqlangan. Servis->repository uchi esa ProductionRunCreationServiceTest:265-278 da event_type/object_type/dedup_key/payload literal bo'yicha qamrab olingan (ikki uchda bir xil literal — drift ushlanadi). 3200 chegaraviy holat: user'da employee yozuvi yo'q + organizationId=null => ProductionRunCreationService.java:191-197 ValidationException('VALIDATION_ERROR', field=organization_id) -> GlobalExceptionHandler.java:91-94 => 400 + fields. Test: ProductionRunCreationServiceTest:350-366 (verifyNoInteractions bilan — rezolyutsiya bosqichida to'xtaydi, DB'ga tegilmaydi). Javadoc va'dasi test bilan qoplangan. Cross-cutting: createManual @Transactional (ProductionRunCreationService.java:140) — outbox enqueueIfAbsent AYNI tranzaksiyada (REQUIRES_NEW yo'q), AR-3 bajarilgan. R19/Q22 chalkashmagan: findActiveRoute(productVariantId) — org-filtrsiz (ProductionRouteService.java:67), profession_norms/piece_rates esa (variant_operation_id x organization_id) bo'yicha node-lokal. tenant_id ustuni yo'q (AR-1). Precision: required_qty/produced_qty numeric(18,4), wip_cost numeric(18,2) (V24:36,42,46,48). Kontrakt izchil: ProductionRunCreateRequest.organization_id opsional+nullable (tasks.yaml:1525-1532) — kod aynan shunday ishlaydi. Qamrov bo'shlig'i (bloklovchi emas, keyingi bargga kuzatuv): rollback testi yo'q — biznes tranzaksiyasi qaytarilganda production_outbox qatori ham yo'qolishini isbotlovchi test qo'shilsa yaxshi bo'lardi (T02-S03 REST integratsiyasida tabiiy joy). Kod O'ZGARTIRILMADI. Status blocked'da qoldirildi.

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

✓ R16: normativ yo'q/0 bo'lgan operatsiya uchun ogohlantirish qatori yoziladi, default norm QO'YILMAYDI

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

✓ R17: `manual` da org-node yaratuvchi user node'idan olinadi

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

✓ `recipe_version_id` sikl ochilgandagi AKTIV versiya; keyin 02 da yangi versiya aktivlashsa sikl o'zgarmaydi (test)

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

✓ Sikl + har operatsiya uchun bitta task, hammasi `status=new`, `required_qty` bir xil

coder-ep-07-t02-s02 statusni o'zgartirdi Ishlanyapti → Bloklangan

AC3202 openapi/v1/tasks.yaml (createProductionRun, T00 muzlatilgan) va 07_tasks.md R15 matni bilan TO'G'RIDAN-TO'G'RI ZID (kontrakt: arxiv variant sikl ochilishida BLOKLANMAYDI, faqat production_in'da). Bu AC amalga oshirilmadi — kontraktni o'zgartirish huquqim yo'q. 4/5 mezon met (3198/3199/3200/3201) — 3200 worker-pm topilmasi bilan tuzatildi (commit feed72a): employees.user_id (nullable+UNIQUE, V20) orqali yaratuvchi user node'i rezolyutsiya qilinadi. Qaror kerak: AC3202 xato/eskirgan (olib tashlansin) YOKI biznes chindan arxiv blokini xohlaydi (kontrakt+katalog yangilanishi kerak, EP-07-T00 qayta ochiladi). Batafsil: oldingi comment'lar.

coder-ep-07-t02-s02 statusni o'zgartirdi Bloklangan → Ishlanyapti

AC3200 tuzatildi (worker-pm topilmasi), blocked_reason'ni yangilash uchun vaqtinchalik o'tish

coder-ep-07-t02-s02 izoh qoldirdi

worker-pm ko'rib chiqishi to'g'ri edi: AC3200 uchun mexanizm bor ekan — employees.user_id (nullable+UNIQUE, V20) -> employees.organization_id, bir qiymatli. PermissionScope (ALL/ko'p-node) satr-darajasidagi huquq doirasi uchun, org-node IDENTITY rezolyutsiyasi uchun emas — bu ikkisini aralashtirib, ortiqcha ehtiyotkorlik bilan bloklagan edim. Tuzatildi: createManual(...) organizationId endi OPSIONAL, null bo'lsa EmployeeAudienceOwnershipService#findOrganizationIdByUserId orqali rezolyutsiya qilinadi; hodim yozuvi topilmasa 400 VALIDATION_ERROR (organization_id) — bu holat ATAYLAB qoldirilgan (taxmin qilinmadi), chunki spec akkauntsiz/hodimsiz yaratuvchi holatini yozmagan. Commit feed72a, 10/10 + yangi EmployeeAudienceOwnershipServiceTest 4/4 yashil. 3200 endi met. Faqat 3202 qoladi (arxiv variant blok) — bu haligacha to'g'ridan-to'g'ri kontrakt ziddiyati (openapi/v1/tasks.yaml + R15 matni ikkalasi ham arxiv gate'ini faqat production_in'ga qo'yadi), kod tomonimdan yechilmaydi — lead/PM qarori kerak.

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

✓ R17: `manual` da org-node yaratuvchi user node'idan olinadi

coder-ep-07-t02-s02 yangiladi

Yangilandi: commit_sha

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

4 ta fayl qayd etildi

coder-ep-07-t02-s02 statusni o'zgartirdi Ishlanyapti → Bloklangan

AC3202 openapi/v1/tasks.yaml (createProductionRun, T00 muzlatilgan) bilan TO'G'RIDAN-TO'G'RI ZID (kontrakt: arxiv variant sikl ochilishida BLOKLANMAYDI, faqat production_in'da — R15 matni ham shu). AC3200 (R17 "manual"da org-node yaratuvchi user node'idan olinadi) uchun mexanizm yo'q — 09_auth bo'yicha users tenant-umumiy, org-node employees YOKI ko'p-qiymatli permission scope orqali, spec qaysi birini ishlatishni aytmaydi. Ikkalasi ham taxmin qilinmadi (CLAUDE.md 13 / TEAM.md 5). Batafsil izoh + qaror variantlari: oxirgi comment. 3198/3199/3201 yashil test bilan yopilgan, commit 3efea78 (feat/EP-07-erpbom-backend).

coder-ep-07-t02-s02 izoh qoldirdi

3198/3199/3201 yashil test bilan yopildi (ProductionRunCreationServiceTest — 8 test, ProductionOutboxNormOrRateMissingMigrationTest — 5 test, ProfessionNormServiceTest — +2 test). 3200 va 3202 — TAXMIN QILINMADI, ikkalasi ham spec bo'shligi/ziddiyati (blocked sababi): 1) AC3202 vs MUZLATILGAN kontrakt — TO'G'RIDAN-TO'G'RI ZID. openapi/v1/tasks.yaml `createProductionRun` tavsifi (T00, muzlatilgan) so'zma-so'z: "Variant active=false (arxiv) bo'lsa sikl ochilishi bu yerda BLOKLANMAYDI — R15 arxiv tekshiruvini production_in lahzasiga (oxirgi operatsiya tasdig'i) qo'yadi; spec undan boshqa joyda arxiv gate'i bermaydi." 409 ProductionRunCreateConflict faqat VARIANT_TYPE_INVALID/RECIPE_VERSION_MISSING'ni sanaydi — VARIANT_ARCHIVED yo'q. 07_tasks.md #5 R15 matnining o'zi ham FAQAT shuni deydi: "`active=false` variant `production_in` ni bloklaydi" — sikl ochilishi haqida hech narsa yo'q. AC3202 esa "R15 bilan izchil" deb aytib, aksini talab qiladi (sikl ochishda rad). Ikki manba (kontrakt matni + R15 matni) bir xil xulosaga keladi va AC3202 shu ikkalasiga ham zid. KONTRAKTNI O'ZGARTIRISH HUQUQIM YO'Q (T00 muzlatilgan) — shuning uchun bu tekshiruv YOZILMADI. Qaror kerak: (a) AC3202 xato/eskirgan — olib tashlansin, YOKI (b) biznes chindan sikl ochilishida ham arxiv blokini xohlaydi — u holda kontrakt R15 izohi va 409 ProductionRunCreateConflict'ga yangi kod (mas. VARIANT_ARCHIVED) qo'shilishi kerak (kontrakt o'zgarishi tartibi bilan, EP-07-T00 qayta ochiladi). 2) AC3200 (R17 "manual"da org-node yaratuvchi user node'idan olinadi) — MEXANIZM YO'Q. 07_tasks.md R17 "yaratuvchi user node'i" deydi, lekin 09_auth.md (R18 izohi, §6.1) ga ko'ra `users` TENANT-UMUMIY — foydalanuvchining "o'z org-node'i" degan YAGONA maydon YO'Q. Org-node bog'lanishi ikki mustaqil yo'l bilan: (a) `employees.organization_id` (agar yaratuvchida hodim yozuvi bo'lsa — lekin `production_run.create` huquqli "Ishlab chiqarish mas'uli" hodim yozuviga ega bo'lishi SHART emas, spec buni aytmaydi); (b) ruxsat scope'i (`PermissionScope` — `ALL` YOKI BIR NECHTA org-node bo'lishi mumkin, bitta qiymat emas — 09_auth §6.1). Qaysi mexanizm ishlatilishi, ko'p/ALL node holatida nima bo'lishi (400? birinchisi? majburiy organization_id?) va hodim yozuvi yo'q userda nima bo'lishi spec'da YOZILMAGAN. Shu sabab ProductionRunCreationService.createManual(...) da organizationId ATAYLAB MAJBURIY parametr qilib qoldirildi — fallback yozilmadi (taxmin qilinmadi). Berilgan organizationId to'g'ri saqlanishi (R17ning "storage" qismi) test bilan tasdiqlangan (createManual_storesGivenOrganizationIdOnRunAndAllTasks). Ikkala band ham ProductionRunCreationService.java Javadoc'ida ("Qamrovga KIRMAYDI" bo'limi) batafsil yozilgan. Keyingi EP-07-T02-S03 (REST controller) shu servisga ustidan quriladi: createManual(productVariantId, requiredQty, organizationId, createdByUserId) -> ProductionRunCreationResult(ProductionRun, List<Task>). organizationId'ni HTTP so'rovdan/kelajakdagi resolver'dan olib shu metodga MAJBURIY uzatish S03/keyingi ish.

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

✓ R16: normativ yo'q/0 bo'lgan operatsiya uchun ogohlantirish qatori yoziladi, default norm QO'YILMAYDI

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

✓ `recipe_version_id` sikl ochilgandagi AKTIV versiya; keyin 02 da yangi versiya aktivlashsa sikl o'zgarmaydi (test)

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

✓ Sikl + har operatsiya uchun bitta task, hammasi `status=new`, `required_qty` bir xil

coder-ep-07-t02-s02 yangiladi

Yangilandi: branch, commit_sha

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

9 ta fayl qayd etildi

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

Agent ishni boshladi

coder-ep-07-t02-s02 ishga oldi

pm-agent yaratdi

`ProductionRunCreationService` — sikl + task-per-operatsiya (R17/R19/R16)