users + currencies — akkaunt va valyuta katalogi

team-lead-ep-09 yangiladi

Yangilandi: pr_url

team-lead-ep-09 bog'liqlik qo'shdi

→ EP-09-T01-S01

team-lead-ep-09 yangiladi

Yangilandi: estimate_hours

qa-ep-09-t01-s03 statusni o'zgartirdi Review → Bajarildi

qa-ep-09-t01-s03 izoh qoldirdi

QA tekshiruvi — done. Spec: 09_auth.md #2.3b (users), #2.2.5+CROSS_CUTTING A (currencies), #2.3f (organization_settings forward-ref). Test: flock ostida `mvn -Dtest=UsersAndCurrenciesMigrationTest test` — 10/10 yashil (real Postgres 16, Testcontainers, V1-V4 Flyway toza o'tdi). Mutatsiya tekshiruvi (kodga vaqtincha buzilish kiritib, testni qizil ko'rdim, keyin `git checkout` bilan tikladim): - organization_settings FK olib tashlandi -> organizationSettingsUpdatedByUserIdReferencesUsers qizil bo'ldi (AssertionError: 'Expecting code to raise a throwable'). Tasdiqlandi. - chk_users_status CHECK'ga 'on_vacation' qo'shildi -> statusCheckRejectsInvalidValue qizil bo'ldi. Tasdiqlandi. Ikkalasi ham original holatga qaytarildi, `git diff` toza. Mezonlar (AC642-645) alohida tekshirildi: - AC642: UNIQUE(username) — real DB constraint, test bilan tasdiqlandi. - AC643: information_schema.columns so'rovi orqali tenant_id/mfa_secret/organization_id/email/is_system yo'qligi tasdiqlandi. Spec matni (2.3b 'Olib tashlangan ustunlar' jadvali + R49) bilan mos. - AC644: user_type va status CHECK'lari real Postgres orqali tasdiqlandi (mutatsiya bilan ham tekshirildi). - AC645 (a): task matni 'varchar(3)' deydi, lekin coder char(3) PK + format CHECK ishlatgan. CROSS_CUTTING A ni o'qib tasdiqladim: `code` ustuni kanonik ravishda `char(3)` PK (faqat FK qiluvchi currency_code ustunlari varchar(3)). 09_auth #2.2.5 aynan CROSS_CUTTING A ga 'To'liq jadval' deb havola qiladi. Demak coder qarori spec bilan mos — ziddiyat yo'q, AC645 mazmunan bajarilgan. - AC645 (b): organization_settings.updated_by_user_id ALTER — S01 dagi V2 migratsiyadagi qoldirilgan izoh bilan so'zma-so'z mos (forward-ref, jadval bo'sh bo'lgani uchun NOT NULL xavfsiz). FK ishlashi mutatsiya bilan ham tasdiqlandi. - c) Olib tashlangan ustunlar (mfa_secret/organization_id/email/is_system/tenant_id) — barchasi spec §2.3b va AR-1/R18/R49/Q7/Q9 bilan mos, users jadvalidagi barcha talab qilingan maydonlar (token_valid_after, failed_login_count, locked_until va h.k.) joyida. Cross-cutting (8 tekshiruv, migratsiya-only subtask): #1 (@Transactional) va #4 (outbox) — N/A, bu subtaskda service qatlami yo'q. #2 single-tenant — tenant_id/schema almashtirish yo'q, tasdiqlandi. #3 — bu migratsiyada tashqi katalogga FK yo'q (exchange_rates forward-ref sifatida bu epicda yaratilmagan, spec bilan mos). #5 nomlash — postgresql skill naqshlariga (fk_<jadval>_<maqsad>, chk_/uq_/idx_) to'liq mos. #6 precision — bu migratsiyada qty/narx ustuni yo'q, N/A. #7 soft-delete — users.deleted_at bor, currencies'da active flag (katalog uslubi, spec/skill §8 bilan mos, hard DELETE yo'q). #8 audit — password_hash saqlanadi, hech qanday parol/token log/test chiqishida ko'rinmaydi. Hammasi tasdiqlandi -> done.

qa-ep-09-t01-s03 qabul mezonini belgiladi

✓ `currencies.code` `varchar(3)` PK; `organization_settings.updated_by_user_id` → `users.id` FK ishlaydi

qa-ep-09-t01-s03 qabul mezonini belgiladi

✓ `status` va `user_type` CHECK cheklovlari noto'g'ri qiymatni rad etadi

qa-ep-09-t01-s03 qabul mezonini belgiladi

✓ `users` da `tenant_id`, `mfa_secret`, `organization_id`, `email`, `is_system` ustunlari YO'Q (grep/SQL tekshiruvi)

qa-ep-09-t01-s03 qabul mezonini belgiladi

✓ `UNIQUE(username)` butun jadval bo'ylab — ikkita bir xil username yozib bo'lmaydi

coder-ep-09-t01-s03 izoh qoldirdi

V4__users_and_currencies.sql: users (09_auth §2.3b) + currencies (CROSS_CUTTING A) yaratildi; shu faylda organization_settings.updated_by_user_id -> users.id FK ham ulandi (S01 forward-ref, jadval bo'sh bo'lgani uchun to'g'ridan-to'g'ri NOT NULL). currencies.code CROSS_CUTTING A bo'yicha char(3) (task tavsifidagi "varchar(3)" erkin ta'rif edi, kanonik jadval char(3) deydi). Test: UsersAndCurrenciesMigrationTest, 10/10 yashil (Testcontainers/PG16).

coder-ep-09-t01-s03 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-09-t01-s03 yangiladi

Yangilandi: branch, commit_sha

coder-ep-09-t01-s03 qabul mezonini belgiladi

✓ `currencies.code` `varchar(3)` PK; `organization_settings.updated_by_user_id` → `users.id` FK ishlaydi

coder-ep-09-t01-s03 qabul mezonini belgiladi

✓ `status` va `user_type` CHECK cheklovlari noto'g'ri qiymatni rad etadi

coder-ep-09-t01-s03 qabul mezonini belgiladi

✓ `users` da `tenant_id`, `mfa_secret`, `organization_id`, `email`, `is_system` ustunlari YO'Q (grep/SQL tekshiruvi)

coder-ep-09-t01-s03 qabul mezonini belgiladi

✓ `UNIQUE(username)` butun jadval bo'ylab — ikkita bir xil username yozib bo'lmaydi

coder-ep-09-t01-s03 fayllarni qayd etdi

2 ta fayl qayd etildi

coder-ep-09-t01-s03 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-09-t01-s03 ishga oldi

pm-agent bog'liqlik qo'shdi

→ EP-09-T01-S01

pm-agent yaratdi

users + currencies — akkaunt va valyuta katalogi