V20 — `employees` + `employee_professions`

finalize-ep-06 yangiladi

Yangilandi: pr_url

qa-ep-06-t01-s02 statusni o'zgartirdi Review → Bajarildi

qa-ep-06-t01-s02 izoh qoldirdi

QA (qa-EP-06-T01-S02) — V20 QABUL QILINDI. Test: EmployeesMigrationTest — 18/18 yashil (coder 15 + QA 3), real PG16 Testcontainers. MEZONLAR (har biri kod darajasida + mutatsiya bilan tekshirildi): - 2809 uq_employees_user_id partial (V20:60-62); ikkala yarim ham haqiqiy assert. - 2810 chk_employees_status IN ('active','suspended','fired') (V20:46-47) — 'invited' YO'Q (FP-16). - 2811 uq_employee_professions_employee_id_profession_id_active WHERE active (V20:90-92). - 2812 organization_id bigint NOT NULL (V20:36). - 2813 flyway_schema_history V20 success=true; V20 faqat YANGI jadval yaratadi (ALTER/backfill yo'q), shuning uchun mavjud (V19) bazaga ham xavfsiz. - Spec talabi tasdiqlandi: `users` jadvalida `employee_id` ustuni YO'Q (V4 da ham, ALTER TABLE users ham yo'q). - Kontrakt (openapi/v1/employees.yaml): maydon nomi/tip/enum V20 bilan mos (EmployeeStatus enum [active,suspended,fired], user_id int64 nullable, EmployeeProfessionResponse maydonlari). MUTATSIYA (har biri V20 ga kiritilib, testlar yiqilishi KO'RILDI, so'ng git checkout bilan tiklandi; V20 commit d8a347f bilan bayt-aynan): - unique indeksni oddiy INDEX qilish -> bindingSameUserToTwoEmployeesIsRejected YIQILDI. - CHECK ga 'invited' qo'shish -> statusInvitedIsRejectedByCheckConstraint YIQILDI. - organization_id ni nullable qilish -> organizationIdIsNotNull YIQILDI. - employee_professions unique'dan WHERE active olib tashlash -> reassigningProfessionAfterDeactivationIsAllowed YIQILDI. - ep unique'ni oddiy INDEX qilish -> assigningSameActiveProfessionTwiceIsRejected YIQILDI. - 'WHERE user_id IS NOT NULL' ni olib tashlash -> testlar YASHIL QOLDI (survived). Bu test nuqsoni EMAS, EKVIVALENT MUTANT: PostgreSQL oddiy unique indeksda NULL'larni o'zaro FARQLI deb hisoblaydi, ya'ni predikatsiz ham bir necha NULL user_id o'tadi. Tasdiq: indeksni 'NULLS NOT DISTINCT' ga o'zgartirganda severalEmployeesWithoutAccountAreAllowed YIQILDI — demak (a) SQL tahriri haqiqatan konteynerga yetib boradi, (b) 'bir necha akkauntsiz hodim' yarmi haqiqiy assert. QA QO'SHGAN TESTLAR (EmployeesMigrationTest, bo'shliq to'ldirildi — V20 ga tegilmadi): - reactivatingExistingAssignmentIsAllowed — §2.3 SPEC-KANONIK qayta biriktirish yo'li (mavjud nofaol qatorni active=true ga qaytarish, yangi INSERT emas) qoplanmagan edi; coder testi faqat yangi-qator yo'lini tekshirardi, kontrakt esa (EmployeeProfessionResponse.assigned_at) aynan tiriltirishni talab qiladi. - reactivatingAssignmentIsRejectedWhenAnotherActiveRowExists — partial unique UPDATE yo'lida ham himoya qilishi. - employeesStatusIndexIsPartialOnNotDeletedRows — idx_employees_status indeksi umuman tekshirilmagan edi; endi pg_indexes.indexdef orqali PARTIAL ('deleted_at IS NULL') ekani tasdiqlanadi. Uchalasi ham mutatsiya bilan o'ldiriladi (WHERE NOT active / status indeksidan WHERE olib tashlash / CHECK (active)). CROSS-CUTTING: tenant_id ustuni yo'q (AR-1) · katalogga oddiy FK (users/organizations/professions, AR-10) · snake_case, jadval ko'plikda, FK to'liq nomdan · soft-delete employees.deleted_at, fizik DELETE yo'q · pul/miqdor ustuni yo'q (precision N/A) · parol/token ustuni yo'q · outbox/@Transactional bu subtaskda N/A (faqat migratsiya). Izoh (bloklamaydi, kelajakdagi servis qatlami uchun): uq_employees_user_id da 'WHERE deleted_at IS NULL' YO'Q — soft-delete qilingan hodimning user_id si band qolaveradi, ya'ni o'sha akkauntni yangi hodimga bog'lab bo'lmaydi. Bu subtask tavsifiga AYNAN mos (spec shu shaklni bergan), shuning uchun defekt sifatida qaytarilmadi.

qa-ep-06-t01-s02 qabul mezonini belgiladi

✓ V20 toza va mavjud bazada xatosiz o'tadi

qa-ep-06-t01-s02 qabul mezonini belgiladi

✓ `employees` da `organization_id` NOT NULL (node-lokal — CROSS_CUTTING J)

qa-ep-06-t01-s02 qabul mezonini belgiladi

✓ Bir hodimga bir xil kasbni ikkinchi marta FAOL biriktirishga urinish rad etiladi; `active=false` qator qolgan holda qayta biriktirish o'tadi

qa-ep-06-t01-s02 qabul mezonini belgiladi

✓ `status` ga 'invited' yozishga urinish CHECK bilan rad etiladi (FP-16)

qa-ep-06-t01-s02 qabul mezonini belgiladi

✓ Bitta `users` qatorini ikki hodimga bog'lashga urinish unique violation; `user_id IS NULL` bo'lgan bir necha hodim muammosiz kiritiladi (partial indeks testi)

coder-ep-06-t01-s02 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-06-t01-s02 yangiladi

Yangilandi: branch, commit_sha

coder-ep-06-t01-s02 qabul mezonini belgiladi

✓ V20 toza va mavjud bazada xatosiz o'tadi

coder-ep-06-t01-s02 qabul mezonini belgiladi

✓ `employees` da `organization_id` NOT NULL (node-lokal — CROSS_CUTTING J)

coder-ep-06-t01-s02 qabul mezonini belgiladi

✓ Bir hodimga bir xil kasbni ikkinchi marta FAOL biriktirishga urinish rad etiladi; `active=false` qator qolgan holda qayta biriktirish o'tadi

coder-ep-06-t01-s02 qabul mezonini belgiladi

✓ `status` ga 'invited' yozishga urinish CHECK bilan rad etiladi (FP-16)

coder-ep-06-t01-s02 qabul mezonini belgiladi

✓ Bitta `users` qatorini ikki hodimga bog'lashga urinish unique violation; `user_id IS NULL` bo'lgan bir necha hodim muammosiz kiritiladi (partial indeks testi)

coder-ep-06-t01-s02 fayllarni qayd etdi

2 ta fayl qayd etildi

coder-ep-06-t01-s02 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-06-t01-s02 ishga oldi

pm-agent bog'liqlik qo'shdi

→ EP-06-T01-S01

pm-agent yaratdi

V20 — `employees` + `employee_professions`