Yaratish/tahrir formasi + `active` toggle

team-lead-pm bog'liqlik qo'shdi

→ EP-03-T06-S01

team-lead-pm yangiladi

Yangilandi: estimate_hours

team-lead-pm bog'liqlik qo'shdi

→ EP-03-T06-S01

team-lead-pm yangiladi

Yangilandi: estimate_hours

team-lead-pm bog'liqlik qo'shdi

→ EP-03-T06-S01

team-lead-pm yangiladi

Yangilandi: estimate_hours

finalize-ep-03 yangiladi

Yangilandi: commit_sha, pr_url

qa-ep-03-t06-s02 statusni o'zgartirdi Review → Bajarildi

qa-ep-03-t06-s02 izoh qoldirdi

QA (qa-EP-03-T06-S02) — yakuniy tor doirali tekshiruv, commit 3dbfdde. Verdikt: DONE. 1️⃣ Commit qamrovi TOZA. `git show --stat 3dbfdde` → 1 fayl, +9 qator, faqat kutilgan fayl: src/modules/suppliers/components/__tests__/SupplierFormDialog.spec.ts Ishlab chiqarish kodiga, boshqa testga yoki konfiguratsiyaga BIRON o'zgarish tushmagan. Ya'ni AC1735/1736/1738 xatti-harakati o'zgarmagan (ular qayta ishlanmadi, oldingi tekshiruv kuchda). 2️⃣ MUTATSIYA QAYTA YURGIZILDI (o'z mutatsiyam, coder so'ziga tayanmasdan). supplier-errors.ts dan `DUPLICATE_INN: 'inn'` olib tashlandi → ILGARI (3105d07): test 6/6 O'TARDI — ko'r edi. ENDI (3dbfdde): × AC1737 YIQILDI — `AssertionError: expected '' to contain 'Bu INN bilan ta'minotchi allaqachon …'` Assert endi yolg'onni ushlaydi. `git checkout` bilan tiklandi. 3️⃣ YANGI ASSERT'NING O'ZI KO'R EMASLIGI — 3 ta alohida tekshiruv (sen so'raganingdek `?? ''` fallback jimgina o'tkazib yubormaydimi): (a) 🔴 eng kuchlisi — xato BOSHQA maydonga xaritalandi (`DUPLICATE_INN: 'name'`): test YIQILDI. Ya'ni assert "biror joyda xato bor" emas, aynan `inn` maydoni konteynerini tekshiradi — xabar `name` maydoni ostiga tushsa ham ushlaydi. Mezonning "TEGISHLI maydon ostida" yarmi haqiqatan himoyalangan. (b) xato sinf selektori buzildi (`.supplier-form__field-error` → mavjud bo'lmagan sinf): test YIQILDI. (c) konteyner selektori buzildi (`.closest('.supplier-form__field')` → mavjud bo'lmagan): test YIQILDI. Xulosa: `?? ''` fallback XAVFSIZ yo'nalishda ishlaydi — selektor hech narsa topmasa `''` qoladi va `toContain` YIQILADI (jimgina o'tmaydi). Uchala holatda ham xato xabari bir xil: `expected '' to contain ...`. 4️⃣ REGRESSIYA: `npx vitest run` → 93 fayl / 507 test yashil (kutilgan 507+ bajarildi). Qo'shimcha: `vue-tsc -b --noEmit` → 0, `eslint . --max-warnings=0` → 0. Barcha QA mutatsiyalari `git checkout` bilan tiklandi; `git status` toza, `git diff HEAD` bo'sh. 5️⃣ BOARD METAMA'LUMOTI — qisman tuzatilgan: ✅ `branch` = feat/EP-03-erpbom-web ✅ `commit_sha` = 3dbfdde928e14b78a7e3e89a6e278d86235fd3e8 ⚠️ `files` — to'g'ri nisbiy yo'llar (`src/modules/suppliers/...`, 11 ta) QO'SHILGAN, lekin eski noto'g'ri prefiksli qatorlar (`apps/erpbom-web/src/...`, 11 ta) O'CHIRILMAGAN. Hozir ro'yxatda 22 qator bor — har fayl ikki marta, biri noto'g'ri yo'l bilan. Bu kod/test defekti EMAS va biror qabul mezoniga tegmaydi, shuning uchun ish `done` qilindi. Tozalash worker-pm/coder zimmasida: eski `apps/erpbom-web/...` qatorlari olib tashlansin. (Men board yozuvini o'zim tozalamadim — `files` endpoint'i qo'shadimi yoki almashtiradimi aniq emas, taxmin bilan board holatini buzmaslik uchun tegmadim.) XULOSA: AC1735/1736/1737/1738 — hammasi bajarilgan va endi to'rttasi ham mutatsiyaga chidamli test bilan qoplangan. Ish `done`.

coder-ep-03-t06-s02 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-03-t06-s02 yangiladi

Yangilandi: branch, commit_sha

coder-ep-03-t06-s02 fayllarni qayd etdi

11 ta fayl qayd etildi

qa-ep-03-t06-s02 statusni o'zgartirdi Review → Ishlanyapti

qa-ep-03-t06-s02 izoh qoldirdi

QA (qa-EP-03-T06-S02) — commit 3105d07. Verdikt: QAYTARILDI (bitta test sifat darvozasida yiqildi). 🔴 MUHIM: to'rtala mezonning XATTI-HARAKATI to'g'ri — board'da 1735/1736/1737/1738 `true`. Qaytarish sababi bitta: AC1737 testi mutatsiya nazoratidan O'TMADI, ya'ni mezonning ajratuvchi yarmi ("tegishli maydon ostida") himoyalanmagan. Kod tuzatish emas, BITTA assert kuchaytirish kerak. ━━━ MEZONLAR ━━━ ✅ AC1735 — anketa + shartnoma BIR so'rovda. SupplierFormDialog.vue:209-220 — create tarmog'i `suppliersStore.create({...})` ni BIR marta, anketa + term_days/payment_method/payment_basis/debt_limit/split% bilan chaqiradi. suppliers.store.ts:93-97 — `createSupplier()` bir marta, keyingi `refresh()` — GET (yozuv emas). Ketma-ket ikkinchi shartnoma so'rovi YO'Q. Kontrakt (suppliers.yaml SupplierCreateRequest) bilan aynan mos: required [name, term_days, payment_method]; split % maydon nomlari `default_cash_percent`/`default_transfer_percent` va tipi **decimal STRING** (types.ts:64-74 — to'g'ri, number emas). Jonli Prism :4010 tekshiruvi — dialog yuboradigan tana → 201; minimal majburiy → 201; null opsionallar → 201. Mock rostdan validatsiya qiladi (name yo'q → 400, term_days string → 400, payment_method="barter" → 400, debt_limit raqam → 400), ya'ni 201 bejiz emas. Mutatsiya: tanadan term_days/payment_method olib tashlandi → SupplierFormDialog.spec.ts:95 yiqildi ✓ ✅ AC1736 — default_term_days/debt_limit tahrirlanmaydi. Ikki darajada tasdiqlandi (faqat `disabled` emas): (a) UI — SupplierFormDialog.vue:359-384, uchala maydon `disabled` va `:model-value` (v-model EMAS), ya'ni forma state'iga umuman ulanmagan, to'g'ridan-to'g'ri props.supplier dan o'qiydi; 354-357 da izoh Message ("yangi shartnoma versiyasini yarating"). (b) YUK — SupplierFormDialog.vue:199-204, PUT tanasi FAQAT {name, address, phone, inn}. `debt_limit`/`default_*` yukka TUSHMAYDI. Buni SupplierFormDialog.spec.ts:182 `toEqual` (toMatchObject emas — ortiqcha maydon ham ushlanadi) qat'iy tekshiradi. Kontrakt SupplierUpdateRequest ham aynan shu 4 maydon (active/user_id/default_* ATAYLAB yo'q) — types.ts:81-86 mos. Kuzatuv: edit rejimida resetForm() form.debt_limit ni tozalamaydi (SupplierFormDialog.vue:135-151), lekin PUT yukiga tushmagani uchun sizib chiqmaydi — hozir zararsiz. ✅ AC1738 — huquqsiz userga tugmalar ko'rinmaydi. SupplierListView.vue:146 `v-can="'supplier.create'"`, :227 `supplier.update`, :236 `supplier.toggle_active`. Direktiva (shared/directives/can.directive.ts:26-29) `authStore.can()` ni watchEffect bilan kuzatib `display:none` qo'yadi. Rule nomlari TAXMIN EMAS — 09_auth.md §7 katalogida uchalasi ham bor: supplier.create (:217), supplier.update (:218), supplier.toggle_active (:219). Nomuvofiqlik yo'q. `active` toggle useConfirmAction tasdiq oynasi ortida (SupplierListView.vue:102-115) — tasdiqsiz chaqirilmaydi. ✅ Split % validatsiyasi (chegara holatlari QA tomonidan alohida yurgizildi): QABUL: 99/1, 0/100, 100/0, bo'sh/bo'sh, 33.33/66.67 → submit ochiq. RAD: 99/2, 50/49, faqat bittasi to'ldirilgan, noto'g'ri format → submit bloklangan. `toDecimal(...).plus(...).equals(100)` (decimal.js) ishlatilgan, parseFloat EMAS — 33.33+66.67 to'g'ri o'tadi. Mutatsiya 3a (yig'indi=100 tekshiruvi o'chirildi) → test yiqildi ✓; Mutatsiya 3b ("faqat bittasi" tekshiruvi o'chirildi) → test yiqildi ✓ ━━━ ❌ QAYTARISH SABABI: AC1737 testi ko'r ━━━ Xatti-harakat TO'G'RI (buni alohida probe bilan tasdiqladim): 409 DUPLICATE_INN xabari aynan `inn` maydoni ostidagi `<small class="supplier-form__field-error">` ichida chiqadi, forma to'ldirilgan holicha qoladi (resetForm faqat `visible` true bo'lganda — SupplierFormDialog.vue:157-164). LEKIN test buni ushlamaydi. Fayl: src/modules/suppliers/components/__tests__/SupplierFormDialog.spec.ts:216 expect(wrapper.text()).toContain("Bu INN bilan ta'minotchi allaqachon mavjud") `wrapper.text()` butun oynaning matni — xabar `inn` maydoni ostida chiqsa ham, yuqoridagi UMUMIY banner'da (`errorMessage`) chiqsa ham bir xil o'tadi. Ya'ni mezonning aynan ajratuvchi yarmi ("tegishli maydon ostida") tekshirilmagan. Mutatsiya isboti: supplier-errors.ts:29-31 dagi `DUPLICATE_INN: 'inn'` xaritasini olib tashladim (applyApiError endi umumiy banner'ga tushadi): → coder testi: 6/6 O'TDI (butun suppliers moduli: 35/35 o'tdi) — ko'r. → QA probe (qat'iy assert): YIQILDI, `inn` maydoni ostidagi xato matni "" bo'lib chiqdi. Nega bu real xavf: `SUPPLIER_ERROR_FIELD` — bitta yozuvli xarita va `resolveSupplierErrorMessage` fallback'i borligi uchun refaktorda "keraksiz kod" deb o'chirilishi juda oson. O'chirilsa AC1737 jimgina buziladi, CI yashil qolaveradi. S01 dagi tautologik test bilan bir xil sinf. TUZATISH (aynan shu assert'ni qo'shing, SupplierFormDialog.spec.ts:216 dan keyin): const innField = wrapper.find('#supplier-inn').element.closest('.supplier-form__field') const innError = innField?.querySelector('.supplier-form__field-error')?.textContent ?? '' expect(innError).toContain("Bu INN bilan ta'minotchi allaqachon mavjud") Bu assert bilan yuqoridagi mutatsiya yiqiladi (tekshirdim). Mavjud assert'lar (forma qiymatlari saqlanishi, update:visible emitilmagani) yaxshi — ularni qoldiring. ━━━ REGRESSIYA ━━━ `npx vitest run` → 93 fayl / 507 test yashil (S01 da 92/491 edi; +1 fayl, +16 test). S01 funksionalligi buzilmagan: suppliers.api.spec.ts va SupplierListView.spec.ts (filtr/qidiruv/sahifalash, nofaol badge) yashil. `vue-tsc -b --noEmit` → 0, `eslint --max-warnings=0` → 0. Barcha QA mutatsiyalari `git checkout` bilan tiklandi, probe fayllari o'chirildi — `git diff HEAD` bo'sh. 📌 Bloklamaydigan kuzatuv: board'dagi `files` ro'yxatida yo'llar `apps/erpbom-web/...` prefiksi bilan yozilgan, repo ildizi esa /var/www/apps/erpbom-web — ya'ni repo ichidagi nisbiy yo'l `src/...` bo'lishi kerak. Shuningdek `branch`/`commit_sha` board'da `null` (S01 da to'ldirilgan edi). Kod defekti emas, board metama'lumoti.

qa-ep-03-t06-s02 qabul mezonini belgiladi

✓ Huquqi bo'lmagan userga `active` toggle va tahrir tugmalari ko'rinmaydi

qa-ep-03-t06-s02 qabul mezonini belgiladi

✓ `409` (INN dublikat) javobi tegishli maydon ostida ko'rsatiladi, forma ma'lumoti yo'qolmaydi

qa-ep-03-t06-s02 qabul mezonini belgiladi

✓ Tahrir formasida `default_term_days`/`debt_limit` maydonlari tahrirlanmaydi (disabled + izoh)

qa-ep-03-t06-s02 qabul mezonini belgiladi

✓ Yaratishda anketa + shartnoma shartlari BITTA so'rovda yuboriladi (§8.1)

coder-ep-03-t06-s02 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-03-t06-s02 qabul mezonini belgiladi

✓ Huquqi bo'lmagan userga `active` toggle va tahrir tugmalari ko'rinmaydi

coder-ep-03-t06-s02 qabul mezonini belgiladi

✓ `409` (INN dublikat) javobi tegishli maydon ostida ko'rsatiladi, forma ma'lumoti yo'qolmaydi

coder-ep-03-t06-s02 qabul mezonini belgiladi

✓ Tahrir formasida `default_term_days`/`debt_limit` maydonlari tahrirlanmaydi (disabled + izoh)

coder-ep-03-t06-s02 qabul mezonini belgiladi

✓ Yaratishda anketa + shartnoma shartlari BITTA so'rovda yuboriladi (§8.1)

coder-ep-03-t06-s02 fayllarni qayd etdi

11 ta fayl qayd etildi

coder-ep-03-t06-s02 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-03-t06-s02 ishga oldi

pm-agent bog'liqlik qo'shdi

→ EP-03-T06-S01

pm-agent yaratdi

Yaratish/tahrir formasi + `active` toggle