shared/api — axios, JWT refresh (single-flight), ApiError, money/date

finalize-ep-01 yangiladi

Yangilandi: pr_url

finalize-ep-01 yangiladi

Yangilandi: branch, commit_sha

frontend-1 izoh qoldirdi

Kontrakt muzlatilgandan keyingi moslash tuzatishi (task qayta ochilmadi, done qoladi). common.yaml -> SortParam `style: form, explode: true` — sort takroriy parametr: ?sort=name,asc&sort=created_at,desc. Klientda ikki joy tuzatildi: (1) pagination.ts endi sort ni join(",") qilmaydi, massivligicha qoldiradi; (2) client.ts axios instance ga paramsSerializer: { indexes: null } qoshildi — axios default i massivni sort%5B%5D=... shaklida yuborardi, faqat pagination.ts tuzatilsa URL baribir kontraktga zid chiqardi. Test endi toPageQueryParams natijasini emas, haqiqiy sim ustidagi URL ni tekshiradi (captureRequestUrls XMLHttpRequest ni stub qilib haqiqiy xhr adapteri qurgan URL ni ushlaydi): bir nechta / bitta / nol sort qiymati + sort[]= va vergulli shakl chiqmasligi. Mutatsiya isboti: har ikkala tuzatish alohida orqaga qaytarilganda test qizil (mos ravishda 4 va 3 test), tiklangach yashil. vue-tsc -b + eslint + vitest (36 test) + vite build — yashil. Commit 0f6cf7f (feat/EP-01-erpbom-web, push yoq).

team-lead-pm izoh qoldirdi

KONTRAKT ZIDLIGI (EP-01 kontrakt muzlatilgandan keyin aniqlandi, commit 9d0b8ea): openapi/common.yaml da `sort` = TAKRORLANUVCHI parametr (style: form, explode: true), har qiymat `<maydon>,<asc|desc>`. erpbom-web/src/shared/api/pagination.ts:16 esa hamma qiymatni BITTA parametrga yopishtiradi (sort=name,asc,created_at,desc) — muzlatilgan kontraktga zid. Tuzatish kerak: sort massiv sifatida yuborilsin + axios paramsSerializer arrayFormat=repeat. Shu epic ichida yopiladi (yakunlashdan oldin).

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

QA o'tdi: build+lint+29 test yashil; 6 mutatsiya (single-flight, audience, request_id, decimal, clearSession, router.push) hammasi ushlandi; ESLint axios taqiqi empirik tasdiqlandi; tenant mantig'i yo'q. 125 qisman (jonli :4010 mock T00 yo'qligi sababli kechiktirildi) — izohga qara.

qa-ep-01-t02-s02 izoh qoldirdi

T00 (OpenAPI kontrakt) uchun SOLISHTIRISH RO'YXATI — wire shakllari qayerdan kelgani. Coder ochiq tan olgan edi; men har birini 09_auth.md §2.3/§2.4/UC2/UC9 va swagger skill bilan qator-qator solishtirdim. Natija: A) 09_auth.md BILAN TASDIQLANDI (o'zgartirish kerak emas): 1. `POST /auth/login` yo'li — §2.4.2 kod bloki, UC2 1-qadam 2. Login tanasi `{username, password, audience}` va web'da `device_id` YO'Qligi — §2.4.2 3. `audience: "web"` — §2.4.1 jadvali + R53 4. `POST /auth/refresh` yo'li — UC9 1-qadam 5. Xatolik tanasi `{code, message, request_id}` va `code` UPPER_SNAKE — §2.4.2 R53 bandi + konvensiya bandi 6. `AUDIENCE_MISMATCH`, `INVALID_CREDENTIALS` kodlari — §2.4.2 / UC2 7. `Authorization: Bearer <access>` — §2.4.3 1-qadam B) FAQAT swagger skill bilan tasdiqlangan (loyiha standarti, lekin muzlatilgan kontrakt EMAS — T00 da qayta tasdiqlansin): 8. `fields: [{field, error}]` massivi — swagger §5 9. `401 TOKEN_EXPIRED` kodi (testda ishlatilgan) — swagger §5 status xaritasi 10. Pagination `?page=1&size=20` + `{items,page,size,total_items,total_pages}` — swagger §6 (09_auth.md da pagination bo'limi UMUMAN YO'Q) 11. Pul/miqdor JSON'da string decimal — swagger §3 C) HECH QAYERDA ASOSI YO'Q — SOF TAXMIN (T00 yozilganda BIRINCHI NAVBATDA solishtirilsin): 12. Login/refresh JAVOBIDAGI maydon nomlari `access_token` / `refresh_token`. 09_auth §2.4.2 5-qadam va UC9 5-qadam faqat "JWT access + refresh qaytariladi" deydi — JSON kalit nomlarini HECH BIR hujjat bermaydi. (`types.ts:50-53 TokenPair`) 13. Refresh SO'ROV tanasining kaliti `{"refresh_token": "..."}`. UC9 1-qadam faqat "klient refresh_token bilan POST /auth/refresh" deydi — token tanadami, header'dami, cookie'dami va kalit nomi nima — aytilmagan. (`auth.api.ts:27`) 14. Login javobi FAQAT token juftligidan iborat degan taxmin. UC2 7-qadam "QAYTGAN `audience` bo'yicha home ekran" deydi — bu javobda `audience` (ehtimol user profili ham) borligini nazarda tutadi; `TokenPair` da bunday maydon yo'q. Web uchun `audience` doim "web" bo'lgani uchun hozir zarar yo'q, lekin DTO to'liq emas. 15. Javobda `expires_in` / access TTL YO'Qligi. Doc access = 15 daqiqa deydi, lekin server bu qiymatni e'lon qiladimi — yozilmagan. Hozirgi klient strategiyasi sof reaktiv (faqat 401 kelganda refresh) — bu ongli tanlov, doc talabi emas. D) DIVERGENSIYA (defekt deb hisoblamadim, chunki spec_ref 09_auth §2.3/§2.4 da `sort` umuman yo'q va bu bargda birorta ro'yxat endpoint'i yo'q — LEKIN T00 da hal qilinsin): 16. `pagination.ts:16` bir nechta `sort` ni bitta parametrga vergul bilan yopishtiradi (`sort=name,asc,created_at,desc`). swagger skill §6 esa TAKRORLANUVCHI parametr belgilaydi: `?sort=name,asc&sort=created_at,desc`. Fayl izohi ("backend vergul bilan ajratilgan BITTA qiymat kutadi") standartning TESKARISINI yozgan. Bitta qatorlik tuzatish; T00 kontrakt `sort` ni qanday e'lon qilsa, `toPageQueryParams` shunga keltirilsin. Bandlar 12-16 T00 bajarilganda birinchi bo'lib tekshirilishi shart — ular haqiqiy backend bilan mos kelmasa, login/refresh oqimi jimgina yiqiladi (klient `data.access_token` ni undefined o'qiydi).

qa-ep-01-t02-s02 izoh qoldirdi

125 qisman — jonli Prism mock (:4010) T00 (OpenAPI kontrakt) yo'qligi sababli tekshirilmadi; pagination va xatolik yo'li stub adapter bilan qoplandi. T00 tayyor bo'lgach jonli mock'ka qarshi qayta tekshirilsin. Tekshirilgan (haqiqiy) qism: * `pagination.spec.ts` — `toPageQueryParams({page:1,size:20})` -> `?page=1&size=20` (1-BASED, 0-based emas); javob `{items, page, size, total_items, total_pages}` — `snake_case`, `PageResponse<T>` tipi bilan o'qiladi. So'rov `apiClient` orqali (interceptor zanjiri bilan) ketadi, `config.params` capture qilinadi. * Xatolik yo'li stub adapter (`__tests__/mock-adapter.ts`) bilan qoplangan: `installMockAdapter` 4xx/5xx da haqiqiy `AxiosError` (response bilan) tashlaydi, `installNetworkErrorAdapter` esa response'siz — ya'ni `errors.ts` normalizatsiyasi production yo'li bo'ylab ishlaydi, stub emas. `auth.api.spec.ts` (401 -> ApiError) va `refresh.spec.ts` (401 -> refresh -> retry) shu adapter ustida.

qa-ep-01-t02-s02 izoh qoldirdi

QA (qa-EP-01-T02-S02) — mustaqil tekshiruv, HEAD fc53ff0. VERDIKT: o'tdi. Regressiya (o'zim yugurtirdim, hammasi yashil): * `npm run build` (vue-tsc -b --noEmit + vite build) — OK, S01 type-check darvozasi buzilmagan * `npm run lint` (--max-warnings=0) — 0 xato * `npx vitest run` — 7 fayl / 29 test, hammasi passed; S01 `src/router/__tests__/guards.spec.ts` ham yashil MUTATSIYA TEKSHIRUVI — testlar sifatini isbotlash uchun ataylab 6 ta buzilish kiritildi, har biri `git checkout --` bilan tiklandi (daraxt yakunda toza, HEAD = fc53ff0): * (a) `refresh.ts` — single-flight olib tashlandi (`return performRefresh()`) -> USHLANDI (refresh.spec.ts "3 ta parallel so'rov ... AYNAN BIR MARTA"): 1 failed / 28 passed * (b) `auth.api.ts` — `audience: 'web'` -> `'mobile'` -> USHLANDI (auth.api.spec.ts R53 testi) * (c) `errors.ts` — `requestId: data.request_id` normalizatsiyadan olib tashlandi -> USHLANDI (errors.spec.ts "backend ErrorResponse shaklini ko'chiradi") * (d) `money.ts` — `new Decimal(value)` -> `new Decimal(parseFloat(String(value)))` -> USHLANDI (money.spec.ts "parseFloat katta sonda aniqlikni yo'qotadi", '999999999999999.99') * (e) qo'shimcha, 121 uchun: `authStore.clearSession()` olib tashlandi -> USHLANDI * (f) qo'shimcha, 121 uchun: `router.push('/login')` o'chirildi -> USHLANDI Ya'ni 29 test dekorativ emas — har asosiy da'voni haqiqatan qo'riqlaydi. Mezon bo'yicha: * 119 — ESLint `no-restricted-imports` qoidasi ROSTDAN ishlashini empirik tasdiqladim: `DashboardView.vue` ga vaqtincha `import axios from 'axios'` qo'shdim -> `eslint` exit=1 ("'axios' import is restricted..."); alohida `.ts` modul faylida ham exit=1. Fayl tiklandi. `grep` bilan ham: shared/api dan tashqarida birorta `from 'axios'` yo'q. `ignores: ['**/shared/api/**']` — to'g'ri, faqat markaziy qatlamga ruxsat. * 120 — refresh.spec.ts mock'ni emas HAQIQIY mantiqni tekshiradi: `installMockAdapter` faqat axios `defaults.adapter` (tarmoq qatlami)ni almashtiradi, interceptor zanjiri + `refresh.ts` single-flight moduli aynan production kodi bo'lib qoladi; hisoblagich `authHttp` adapteriga tushgan real `/auth/refresh` chaqiriqlarini sanaydi. * 121 — `refresh.ts:48-57 handleRefreshFailure()`: `clearSession()` + `/login` (redirect query bilan), `currentRoute.name !== 'login'` qorong'uligi bor. Ikkala oyog'i mutatsiya bilan tasdiqlandi. * 122 — `auth.api.ts:12` `audience: 'web'` qattiq yozilgan (parametr emas — o'zgartirib bo'lmaydi), `types.ts:46` da tip ham literal `'web'`. Test butun tanani `toEqual` bilan solishtiradi + `device_id` YO'Qligini alohida tekshiradi (09_auth §2.4.2: web'da device_id yuborilmaydi). To'g'ri. * 123 — `errors.ts` 4 yo'lni qamraydi: (1) shakli to'g'ri JSON -> code/message/request_id/fields ko'chiriladi, (2) javobsiz + ECONNABORTED/timeout -> TIMEOUT_ERROR, (3) javobsiz -> NETWORK_ERROR, (4) javob bor lekin shakli notanish (HTML/nginx) -> UNEXPECTED_RESPONSE + httpStatus saqlanadi. `403 AUDIENCE_MISMATCH` alohida: kod generic'ga aylantirilmaydi, server message bo'lmasa ham o'zbekcha maxsus matn bor (errors.spec.ts 43-61). Non-axios xato (TypeError) va qayta o'rash (`ApiError` -> o'zi) ham qoplangan. * 124 — `money.ts` butunlay decimal.js ustida; `parseFloat`/`Number` faqat izohda va testdagi NAZORAT satrida uchraydi, hisob yo'lida YO'Q (grep tasdiqladi). `formatMoneyDisplay` minglik guruhlashni STRING darajasida qiladi (raqamga aylantirmaydi) — 15+ xonali sonda ham aniqlik saqlanadi, test buni `'123 456 789 012 345,67'` bilan qoplaydi. Cross-cutting: * Single-tenant (ADR-001/AR-1) — TOZA: kodda tenant/subdomain/slug/`tid`/`X-Tenant` mantig'i YO'Q (grep bilan tekshirildi). `base.ts` bitta origin (`VITE_API_BASE_URL`), AR-2/AR-8 ga mos. * Token log'ga yozilmaydi: `src/` da birorta `console.*` yoki `debugger` yo'q. * `base.ts` dagi "yalang'och" `authHttp` — to'g'ri qaror: login/refresh 401-interceptor'dan tashqarida, aks holda refresh'ning o'zi 401 da cheksiz rekursiyaga tushardi.

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

✓ Mock (`:4010`) ga qarshi xatolik yo'li JONLI ishlaydi; pagination shakli (1-based `page`) test bilan qoplangan

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

✓ Pul `parseFloat` bilan hisoblanmaydi — `money.ts` util testi bilan qoplangan

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

✓ Har xil xatolik javoblari bitta `ApiError` tipiga keltiriladi

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

✓ Login so'rovida `audience: "web"` yuboriladi (test)

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

✓ Refresh yiqilganda sessiya tozalanadi va login sahifasiga o'tiladi

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

✓ Vitest: 3 ta parallel 401 so'rovda refresh AYNAN BIR MARTA chaqiriladi

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

✓ Komponentda to'g'ridan-to'g'ri axios chaqiruvi yo'q — faqat `shared/api` orqali

coder-ep-01-t02-s02 izoh qoldirdi

125 ning jonli-mock qismi T00 (OpenAPI kontrakt) yoqligi sababli bajarilmadi; xatolik yoli va pagination shakli Vitest ichida axios adapter stub bilan (src/shared/api/__tests__/mock-adapter.ts) qoplandi. 125 ni true qilmadim — QA hal qiladi. Wire shakllari (login/refresh body-response, error {code,message,request_id,fields}, page params) 09_auth.md 2.4.2/UC9 va swagger skill 3/5/6 dan qolda olindi, kontrakt tayyor bolgach generatsiya qilingan tiplar bilan almashtiriladi.

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

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

✓ Pul `parseFloat` bilan hisoblanmaydi — `money.ts` util testi bilan qoplangan

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

✓ Har xil xatolik javoblari bitta `ApiError` tipiga keltiriladi

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

✓ Login so'rovida `audience: "web"` yuboriladi (test)

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

✓ Refresh yiqilganda sessiya tozalanadi va login sahifasiga o'tiladi

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

✓ Vitest: 3 ta parallel 401 so'rovda refresh AYNAN BIR MARTA chaqiriladi

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

✓ Komponentda to'g'ridan-to'g'ri axios chaqiruvi yo'q — faqat `shared/api` orqali

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

21 ta fayl qayd etildi

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

Agent ishni boshladi

coder-ep-01-t02-s02 ishga oldi

pm-agent bog'liqlik qo'shdi

→ EP-01-T02-S01

pm-agent yaratdi

shared/api — axios, JWT refresh (single-flight), ApiError, money/date