Ta'minotchi ro'yxati / kartochka / tahrir

team-lead-pm bog'liqlik qo'shdi

→ EP-03-T02-S01

team-lead-pm yangiladi

Yangilandi: estimate_hours

team-lead-pm bog'liqlik qo'shdi

→ EP-03-T02-S01

team-lead-pm yangiladi

Yangilandi: estimate_hours

team-lead-pm bog'liqlik qo'shdi

→ EP-03-T02-S01

team-lead-pm yangiladi

Yangilandi: estimate_hours

finalize-ep-03 yangiladi

Yangilandi: pr_url

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

qa-ep-03-t02-s02 izoh qoldirdi

QA qayta tekshiruv (2026-08-19) — QABUL QILINDI. A. Asosiy defekt TUZALGAN. ?sort=name,asc -> 200 va HAQIQIY tartib assert qilinadi (containsExactly, orderedItemNames). ?sort=inn,desc, ?sort=created_at,desc, takroriy ?sort=name,asc&sort=inn,desc — hammasi ishlaydi. Whitelist saqlangan: sort=debt_limit,desc va sort=id,asc -> 400. SQL injection xavfsiz: SORTABLE_FIELDS Map orqali property'ga o'giriladi, xom satr Sort.by ga tushmaydi (SupplierService.java:250-271). B. REGRESSIYA YO'Q. SortQueryArgumentResolver.supportsParameter (SortQueryArgumentResolver.java:30-33) IKKI shartni talab qiladi: List turi VA @RawQueryValues annotatsiyasi. Repo'dagi qolgan 14 controller'ning sort parametri @RequestParam bilan qolgan — resolver ularga tegmaydi. PageQueryArgumentResolver bilan to'qnashuv yo'q (u PageQuery.class ga bog'langan). AMALDA tasdiqlandi: PriceTypeControllerIntegrationTest 12/12, PermissionGroupControllerIntegrationTest 10/10 yashil. C. MUTATSIYA TEKSHIRUVI o'tkazildi: - @RawQueryValues -> @RequestParam qaytarilganda AYNAN 3 ta yangi sort testi yiqildi (listSortsByNameAscendingWithSingleSortValue, listSortsByInnDescendingWithSingleSortValue, listSortsByCreatedAtDescendingWithSingleSortValue). Testlar qo'g'irchoq EMAS. - AC1701 testi haqiqiy DB holatini tekshiradi (xom SQL SELECT ... FROM suppliers): so'rovda default_payment_method=cash yuborilgan, DB'da 'transfer' qolgani assert qilinadi; debt_limit=1.00 yuborilgan, DB'da NULL qolgani assert qilinadi. D. Kontrakt mos: SupplierUpdateRequest DTO = kontrakt (name majburiy, address/phone/inn opsional, maxLength'lar); SupplierResponse 12 maydon = kontrakt; sort whitelist name/inn/created_at/updated_at = suppliers.yaml:148; default name,asc = :149. ?sort=name,asc,extra -> 400 (kontrakt vergul-yopishtirishni qo'llamaydi). E. Gigiyena toza: 3b68c88 faqat 6 fayl, hammasi doira ichida. openapi/ ga, erp/docs ga tegilmagan; boshqa modul controller/service fayllari o'zgartirilmagan; boshqa agentning supplier_documents/V13 ishi commit'ga tushmagan. Testlar (QA o'zi yugurtirdi, flock ostida): SupplierCreateTest 13/13 + SupplierQueryTest 19/19 = 32/32 BUILD SUCCESS. Mezonlar 1700/1701/1702/1703 — hammasi tasdiqlandi. 🟡 Bloklovchi bo'lmagan qamrov bo'shliqlari (keyingi bargda yoki texnik qarzga): 1. Default saralash (sort berilmasa name,asc — suppliers.yaml:149) HECH BIR testda tartib bo'yicha assert qilinmagan. SupplierService.java:252 da to'g'ri yozilgan, lekin mutatsiyani ushlaydigan test yo'q. 2. saveAndFlush -> save ga qaytarilganda 32/32 test baribir YASHIL qoldi — 5-band (409 o'rniga 500) testsiz 'tuzatilgan'. Kod o'zi to'g'ri (SupplierService.java:238), lekin bu yo'l faqat haqiqiy INN poygasida ishga tushgani uchun deterministik test yo'q; determinatsiyalangan 409 pre-check orqali qoplangan.

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

✓ Qidiruv `inn` va `phone` bo'yicha qisman moslikda ishlaydi

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

✓ `supplier.view` / `supplier.update` rule'siz mos amallar `403`

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

✓ So'rovda `default_term_days`/`debt_limit` yuborilsa ular E'TIBORSIZ qoldiriladi yoki `400` — cache ustunlari mijoz tomonidan yozilmaydi

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

✓ Ro'yxat so'rovi org-node bo'yicha filtrlanmaydi — turli org-node useri bir xil to'plamni oladi (§12 AC, 2026-08-04)

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

coder-ep-03-t02-s02 yangiladi

Yangilandi: commit_sha

coder-ep-03-t02-s02 izoh qoldirdi

QA topilmalari tuzatildi. (1) BLOKLOVCHI: GET /suppliers?sort=name,asc (bitta qiymat) 400 berardi — sabab @RequestParam List<String> ni Spring StringToCollectionConverter vergul boyicha bolib yuborishi edi. Yechim: RawQueryValues annotatsiyasi + SortQueryArgumentResolver (faqat shu annotatsiya bilan izohlangan parametrga ulanadi, HttpServletRequest.getParameterValues orqali xom qiymat oladi) — FAQAT SupplierController#list dagi sort parametriga ulandi, boshqa modul (PriceType/ProductVariant/Category/Brand/Sector/SellingPrice) fayllariga tegilmadi. (2) Test bosligi yopildi: sort happy-path (bitta va takrorlangan qiymat: name,asc / inn,desc / created_at,desc), whitelist xavfsizligi (debt_limit,desc va id,asc -> 400) testlari qoshildi. (3) AC1701 kengaytirildi: debt_limit va default_payment_method ham assert qilinadi. (4) active va name filtrlariga test qoshildi. (5) SupplierService#saveSupplierOrThrowDuplicate save() -> saveAndFlush(): update() da entity managed bolgani uchun UPDATE odatda commit paytida chiqar edi, real poygada DataIntegrityViolationException try/catch tashqarisida qolib 409 orniga 500 chiqishi mumkin edi; saveAndFlush flushni tryning ichiga majburlaydi. Test: mvn -Dtest=SupplierCreateTest,SupplierQueryTest test -> 32/32 yashil (SupplierQueryTest 11->19). Eslatma: openapi/v1/suppliers.yaml indeksda staged edi (mening ishim emas), commitga pathspec bilan kiritilmadi, tegilmadi.

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

6 ta fayl qayd etildi

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

qa-ep-03-t02-s02 izoh qoldirdi

QA (mutatsiya tekshiruvi bilan) — 4 ta AC ✓ TASDIQLANDI, lekin KONTRAKT BUZILISHI topildi -> qaytarildi. ✅ AC1700/1701/1702/1703 — hammasi kod darajasida tasdiqlangan va mutatsiyaga CHIDAMLI: M1 (SupplierService.list ga org-scope filtri) -> listReturnsSameSetRegardlessOfViewersOrgNodeScope YIQILDI (2 -> 0). Test haqiqatan ikki HAR XIL org-node useri bilan ishlaydi (root vs yangi child org). M2 (SupplierUpdateRequest ga default_term_days qo'shib service'da yozdirish) -> updateIgnoresReadOnlyCacheFieldsSentInBody YIQILDI. Test DB holatini haqiqatan tekshiradi. M3 (@RequirePermission olib tashlash) -> 3 ta 403 testi YIQILDI. M4 (LIKE -> =) -> listFiltersByPartialInnMatch + ...PhoneMatch YIQILDI. Barcha mutatsiyalar qaytarildi (git status toza). 🔴 DEFEKT — sort kontraktdan chetga chiqadi (400, 200 o'rniga): Fayl: src/main/java/uz/erpbom/suppliers/controller/SupplierController.java:49 (`@RequestParam(required=false) List<String> sort`). GET /api/v1/suppliers?sort=name,asc -> 400 VALIDATION_ERROR: fields=[{sort:"ruxsat etilmagan qiymat: name"},{sort:"ruxsat etilmagan qiymat: asc"}] Sabab: Spring'ning StringToCollectionConverter'i BITTA `sort` parametri qiymatini vergul bo'yicha bo'lib yuboradi -> ["name","asc"], keyin SupplierService.resolveSort (SupplierService.java:236-257) har ikkalasini ham yaroqsiz deb hisoblaydi. Kontrakt (openapi/common.yaml SortParam): `type: array`, `items.pattern = ^[a-z][a-z0-9_]*,(asc|desc)$`, explode=true -> bitta mezon AYNAN `?sort=name,asc` ko'rinishida yuboriladi; suppliers.yaml GET /api/v1/suppliers tavsifi ham `name/inn/created_at/updated_at` ni ruxsat etadi. Ya'ni klientning eng oddiy holati (bitta ustun bo'yicha saralash) hozir ISHLAMAYDI. Empirik: takroriy shakl `?sort=name,asc&sort=inn,desc` -> 200 va to'g'ri saralanadi (2+ qiymatda Spring String[] beradi, bo'lish bo'lmaydi). Demak faqat BITTA mezonli sort siniq. Xavfsizlik tomoni yaxshi: whitelist ishlaydi (`?sort=debt_limit,desc` va `?sort=id,asc` -> 400), SQL injection yo'q. Tuzatish (taklif): `sort` xom qiymatlarini `HttpServletRequest.getParameterValues("sort")` orqali olish (PageQueryArgumentResolver naqshi kabi umumiy `SortQueryArgumentResolver`), `List<String>` bilan bog'lamaslik. DIQQAT: bu naqsh butun repo'da takrorlangan (PriceTypeService:145, ProductVariantService:544, ProductCategoryService:185, Brand/Sector/SellingPrice controllerlari) — umumiy yechim afzal; boshqa modullar uchun PM alohida task ochishi mumkin. Test ham kerak: sort happy-path (`?sort=name,asc` -> 200 + tartib) SupplierQueryTest'da UMUMAN yo'q edi. 🟡 Kichik (blocker emas, keyingi safar kuchaytirish): 1) SupplierQueryTest.java:154-164 — AC1701 testi DB'da faqat default_term_days/active/user_id ni tekshiradi; `debt_limit` va `default_payment_method` assert QILINMAGAN. QA o'z probe'i bilan tekshirdi: ular ham haqiqatan o'zgarmaydi (DTO'da yo'q) — xatti-harakat TO'G'RI, faqat test qamrovi AC matnidan tor. 2) SupplierService.java:107 — `update()` da `saveSupplierOrThrowDuplicate` boshqariladigan (managed) entity ustida ishlaydi, flush commit paytida bo'ladi -> haqiqiy poyga (race) holatida DataIntegrityViolationException try/catch'dan tashqarida qoladi va 500 chiqishi mumkin (409 o'rniga). Pre-check odatdagi holatni yopadi. 3) `active` va `name` filtrlari kontraktda bor, ammo testda qamralmagan. ✅ Regressiya: SupplierCreateTest 13/13 yashil (S01 create oqimi buzilmagan). SupplierQueryTest 11/11 yashil. Jami 24/24 — coder da'vosi to'g'ri. ✅ Gigiyena: `git show --stat 64c4467` — faqat 5 ta java fayl; openapi/ ga ham, /var/www/erp/docs ga ham tegilmagan; boshqa agentning supplier_documents/V13 ishi bu commit'ga tushmagan. ✅ Kontrakt moslik (sort'dan tashqari): SupplierResponse maydonlari/tiplari, SupplierUpdateRequest (faqat name/address/phone/inn, required=[name]), PageResponse konverti (items/page/size/total_items/total_pages), 200/400/401/403/404/409 + DUPLICATE_INN kodi — hammasi mos.

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

✓ Qidiruv `inn` va `phone` bo'yicha qisman moslikda ishlaydi

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

✓ `supplier.view` / `supplier.update` rule'siz mos amallar `403`

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

✓ So'rovda `default_term_days`/`debt_limit` yuborilsa ular E'TIBORSIZ qoldiriladi yoki `400` — cache ustunlari mijoz tomonidan yozilmaydi

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

✓ Ro'yxat so'rovi org-node bo'yicha filtrlanmaydi — turli org-node useri bir xil to'plamni oladi (§12 AC, 2026-08-04)

coder-ep-03-t02-s02 yangiladi

Yangilandi: branch, commit_sha

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

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

✓ Qidiruv `inn` va `phone` bo'yicha qisman moslikda ishlaydi

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

✓ `supplier.view` / `supplier.update` rule'siz mos amallar `403`

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

✓ So'rovda `default_term_days`/`debt_limit` yuborilsa ular E'TIBORSIZ qoldiriladi yoki `400` — cache ustunlari mijoz tomonidan yozilmaydi

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

✓ Ro'yxat so'rovi org-node bo'yicha filtrlanmaydi — turli org-node useri bir xil to'plamni oladi (§12 AC, 2026-08-04)

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

5 ta fayl qayd etildi

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

Agent ishni boshladi

coder-ep-03-t02-s02 ishga oldi

pm-agent bog'liqlik qo'shdi

→ EP-03-T02-S01

pm-agent yaratdi

Ta'minotchi ro'yxati / kartochka / tahrir