Qaytarish va utilizatsiya ekranlari

finalize-ep-04 yangiladi

Yangilandi: commit_sha, pr_url

qa-ep-04-t10-s02 statusni o'zgartirdi Review → Bajarildi

qa-ep-04-t10-s02 izoh qoldirdi

QA (qa-EP-04-T10-S02) — commit 63a6a20, feat/EP-04-erpbom-web. VERDIKT: done. Testlar: `npx vitest run src/modules/warehouse/` -> 38 fayl / 268 test yashil. `vue-tsc --noEmit` toza. Ish daraxti toza (`git diff HEAD` bo'sh). == 5 MUTATSIYA TEKSHIRUVI — HAMMASI TEST TOMONIDAN O'LDIRILDI (yolg'on yashil YO'Q) == 1) AC2543 — StockWriteOffFormView.vue:105 `canConfirm` dan `reason.value.trim().length > 0` olib tashlandi -> "AC2543 ... disabled atributi bilan BLOKLANADI" YIQILDI (AssertionError: expected undefined to be defined). 2) AC2545 — StockReturnFormView.vue:90 `isEditable` -> `computed(() => true)` -> AC2545 testi YIQILDI. 3) AC2545 — StockWriteOffFormView.vue:85 `isEditable` -> `true` VA :101 `showConfirmAction` dan status sharti olib tashlandi -> AC2545 testi YIQILDI. 4) AC2544 — ikkala tasdiq dialogidan "ORTGA QAYTARIB BO'LMAYDI" matni olib tashlandi -> ikkala AC2544 testi (return + write-off) YIQILDI. 5) AC2542 — StockReturnLinesEditor.vue `emitLines` ga variant bo'yicha dedupe qo'shildi -> "AC2542 ... IKKALASI ham bor" (expected length 2, got 1) VA "AC2542 ... POST tanasida IKKALASI ham boradi" YIQILDI. Qo'shimcha: StockReturnLinesEditor.vue readonly shoxi o'chirildi -> AC2545 dagi "Qator qo'shish yo'q" assert'i va editor readonly testi YIQILDI — ya'ni bu "element yo'q" assert'i ham haqiqiy. Har mutatsiyadan keyin `git checkout -- <aniq fayl>` bilan tiklandi. == "ELEMENT YO'Q" ASSERT'LARI (S01 dagi nuqson turi) — TESKARI HOLAT QOPLANGAN == AC2545 testlari tugma YO'QLIGINI tasdiqlaydi, lekin yo'qlik sababi haqiqatan `status`: (a) beforeEach da authStore.setPermissions bilan stock_return.confirm / stock_write_off.confirm rule'lari BERILGAN; (b) shu fayldagi `draft` testlari AYNAN shu mount helper bilan tugma KO'RINISHINI tasdiqlaydi (ikki tomonlama qoplama); (c) test `linesEditor.props('readonly') === true` va `wrapper.text()).toContain('Yakunlangan')` bilan komponent haqiqatan render bo'lganini isbotlaydi (mount xatosi emas). AC2543 testi ham ikki tomonlama: yoqilgan -> bo'sh reason -> disabled -> qayta to'ldirilgan -> yoqilgan. == KONTRAKTGA MOSLIK (openapi/v1/warehouse.yaml) == - Holat o'tishi FAQAT alohida POST /confirm orqali: stock-returns.api.ts:139, stock-write-offs.api.ts:123. Hech bir request DTO'da `status` YO'Q (types.ts Create/Update interfeyslari); testlar `expect(capturedBody).not.toHaveProperty('status')` bilan qotiradi. `status:` faqat GET ro'yxat filtrida (store) — kontrakt ruxsat bergan query param. - Qator maydonlari: qaytarishda product_variant_id + qty + condition + unit_cost (StockReturnLineInput bilan bir xil); utilizatsiyada `unit_cost` YUBORILMAYDI (StockWriteOffLinesEditor.vue emitLines) — StockWriteOffLineInput va §2.12 ga mos (tannarx confirm da FIFO qatlamlaridan). write-off `reason` majburiy (minLength 1) — UI canSave/canConfirm da ham talab qiladi. write-off qatori default condition = `defective` (§2.12). - Ekranda raqamli variant ID ko'rsatilmaydi: variantDisplayLabel() `variant_code — variant_name` beradi; `#id` faqat server ikkalasini ham bermasa (readOnly maydonlar doim keladi). - 409 STOCK_INSUFFICIENT tafsiloti serverning `message` idan: STOCK_RETURN/WRITE_OFF_ERROR_MESSAGES xaritalari ataylab bo'sh, resolveErrorMessage `error.message` ni birinchi qaytaradi; test server matnini ("kerak 20, mavjud 14 (6 tasi band)") aynan tekshiradi — frontend raqam TO'QIMAYDI. - Fixture tekshiruvi: kontraktda YO'Q maydonga tayangan test topilmadi. /stock fixture'idagi qty_available / is_low / stock_value / created_at — hammasi StockResponse da bor (warehouse.yaml:3176-3270). == source_ref_id (worker-pm so'ragan qo'shimcha tekshiruv) — RAD ETILMADI, DEFEKT EMAS == 1) Kontraktda BOR: StockReturnCreateRequest/UpdateRequest/Response da `source_ref_id: integer(int64), nullable: true`, required ro'yxatida YO'Q -> IXTIYORIY. 2) Tanlash uchun lookup endpoint kontraktda YO'Q: openapi/v1/ da faqat auth/products/suppliers/warehouse yaml bor; buyurtma/task endpoint'i umuman mavjud emas. Ya'ni coder foydalanmagan vosita yo'q edi. 3) SPEC BO'SHLIG'I EMAS, shuning uchun `blocked` QILINMADI: 03_warehouse.md §8.5 p.1 "Ixtiyoriy source_ref_id (buyurtma/task)", §2.9 esa ochiq aytadi — "To'liq bog'lanish 06_orders/07_tasks da yakunlanadi (soft reference)". Spec bu bog'lanishni ATAYLAB keyingi modullarga qoldirgan. 4) "Raqamli ID ko'rsatilmasin" qoidasi buzilmagan: bu qoida product_variant_id ga taalluqli, chunki kontrakt unga variant_name/variant_code readOnly hamrohlarini beradi. source_ref_id boshqa HUJJATGA havola va kontraktda hech qanday nom hamrohi yo'q. UI halol: label "Manba hujjat ID", placeholder "Buyurtma/task ID (ixtiyoriy)" (StockReturnFormView.vue:427-435). == SPEC (§8.5/§8.6) MUVOFIQLIGI == draft -> confirmed yagona o'tish (§4.3/§4.4); confirmed da tahrir taqiqlangan; aralash qaytarish ikki qator bilan (§8.5 misoli 3 soz + 2 nosoz — testda aynan shu raqamlar); utilizatsiyada mavjud qoldiq GET /stock dan warehouse_id + product_variant_id + condition bilan so'ralib ko'rsatiladi, condition o'zgarsa qayta so'raladi. QA test yozmadi — mavjud qamrov mutatsiyaga chidamli.

qa-ep-04-t10-s02 qabul mezonini belgiladi

✓ Yakunlangan hujjat read-only bo'ladi

qa-ep-04-t10-s02 qabul mezonini belgiladi

✓ Yakunlashdan oldin orqaga qaytarib bo'lmasligi haqida ogohlantirish ko'rsatiladi

qa-ep-04-t10-s02 qabul mezonini belgiladi

✓ Utilizatsiyada `reason` bo'sh bo'lsa yakunlash tugmasi bloklanadi

qa-ep-04-t10-s02 qabul mezonini belgiladi

✓ Qaytarishda bitta tovarni `good` va `defective` sifatida ikki qatorda kiritish qulay ishlaydi

coder-ep-04-t10-s02 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-04-t10-s02 yangiladi

Yangilandi: branch, commit_sha

coder-ep-04-t10-s02 izoh qoldirdi

stock-returns + stock-write-offs ekranlari qo'shildi (03_warehouse.md §8.5/§8.6). S01 (warehouse-transfers) naqshiga izchil: api/store/lines-editor/list+form view, 4 ta AC ham qamrab olindi va mutatsiya bilan tekshirildi (AC2542 — emitLines'ga dedupe qo'shib test yiqilishini ko'rdim; AC2543 — canConfirm'dan reason shartini olib tashlab test yiqilishini ko'rdim; AC2545 — isEditable'ni har doim true qilib test yiqilishini ko'rdim; keyin qaytardim). Umumiy fayllar (types.ts/labels.ts/router/navigation.ts) faqat Edit bilan, mavjud qatorlar tegilmadi. npx vue-tsc -b --noEmit + eslint + npm run build + npx vitest run src/modules/warehouse (38 fayl, 268 test) — barchasi yashil. Taxmin: source_ref_id uchun alohida lookup yo'q (kontraktda ham yo'q) — oddiy InputNumber (soft reference ID).

coder-ep-04-t10-s02 qabul mezonini belgiladi

✓ Yakunlangan hujjat read-only bo'ladi

coder-ep-04-t10-s02 qabul mezonini belgiladi

✓ Yakunlashdan oldin orqaga qaytarib bo'lmasligi haqida ogohlantirish ko'rsatiladi

coder-ep-04-t10-s02 qabul mezonini belgiladi

✓ Utilizatsiyada `reason` bo'sh bo'lsa yakunlash tugmasi bloklanadi

coder-ep-04-t10-s02 qabul mezonini belgiladi

✓ Qaytarishda bitta tovarni `good` va `defective` sifatida ikki qatorda kiritish qulay ishlaydi

coder-ep-04-t10-s02 fayllarni qayd etdi

26 ta fayl qayd etildi

coder-ep-04-t10-s02 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-04-t10-s02 ishga oldi

pm-agent bog'liqlik qo'shdi

→ EP-04-T10-S01

pm-agent yaratdi

Qaytarish va utilizatsiya ekranlari