Payroll o'qish API — hodim × davr + mobil `/me/payroll/*`

finalize-ep-06 yangiladi

Yangilandi: pr_url

finalize-ep-06 izoh qoldirdi

[FINALIZE — status o'zgartirilmadi: board bog'liqlik qo'riqchisi ruxsat bermadi (EP-06-T05-S02, EP-06-T04 blocked). Bu item ham A toifadagi drift'ga tegishli:] FINALIZE EP-06 drift gate (scripts/openapi-diff.sh v1/employees) — verdikt: BLOCKING (exit=1), A toifasida monthly_salary_amount maydoni "properties.deleted" bo'lib chiqdi: kontrakt e'lon qilgan kalitni springdoc generatsiya qilgan hujjatda TOPMAYDI. Sabab: kalit RBAC bo'yicha shartli qilingan — @JsonIgnore + @JsonAnyGetter naqshi (EmployeePayrollSummaryResponse), springdoc @JsonAnyGetter chiqaradigan kalitlarni ko'rmaydi. Xatti-harakat TO'G'RI (huquq bo'lsa kalit real javobda bor), lekin HUJJAT kontraktdan chetda — bu A toifasi va baseline bilan "qabul qilib" bo'lmaydi (write-baseline A ni ataylab yozmaydi). Kerakli tuzatish KODDA (kontrakt daxlsiz): shartli kalitni springdoc'ga ko'rsatish — mas. @Schema bilan e'lon qilingan xossa yoki OpenApiCustomizer/ModelConverter orqali qo'shish. Finalize agenti kodni tuzatmaydi (epic-finalize qoida 1).

qa-ep-06-t06-s03 statusni o'zgartirdi Review → Bajarildi

qa-ep-06-t06-s03 izoh qoldirdi

QA (qa-EP-06-T06-S03) — mustaqil tekshiruv, hamma 5 mezon TASDIQLANDI. Testlar (commit a075c3d9, ish daraxti toza): - PayrollControllerIntegrationTest: 5/5 yashil - EmployeePayrollSelfControllerIntegrationTest: 5/5 yashil - Regressiya: WageAccrualServiceTest 10/10, WageAccrualPortTest 4/4, RecipeVersionServiceTest 6/6 MUTATSIYA TEKSHIRUVI (3 ta, tracked fayllarda, keyin git checkout + toza daraxt tasdiqlangan): 1) PayrollService#resolveSalaryVisible -> `return true` => summaryWithoutSalaryViewPermissionHidesMonthlySalaryKey YIQILDI (maskalash fail-closed, test haqiqiy). 2) EmployeePayrollSelfService.ACTION_VIEW_SUMMARY nomi o'zgartirildi => mySummaryWithoutLinkedEmployeeReturnsForbiddenAndAuditLogsDeny YIQILDI (test faqat 403 emas, auth_logs qatorini HAQIQATAN DB'dan o'qiydi). 3) egalik qidiruvi findByUserIdAndDeletedAtIsNull -> findAll().findFirst() => 3 ta test YIQILDI (izolyatsiya + deny + norm progress). Hech bir mutatsiya "survived" bo'lmadi. Shubha nuqtalari bo'yicha xulosa: (a) bonus_amount = total_amount - normal_amount: mavjud ma'lumot modelida Sum(bonus_qty x bonus_rate) bilan AYNI (amount = normal_qty x rate + bonus_qty x bonus_rate qator darajasida yoziladi). AC2901 testi tautologiya EMAS — 6.1 misolining qotirilgan raqamlarini (18.0000 / 100000.00 / 104000.00 / 204000.00) HTTP javobidan tekshiradi, ma'lumotni esa real WageAccrualPort yaratadi. QA qo'shimcha probe: ikki operatsiya (norm 10 rate 10000/13000 qty 12 + norm 5 rate 5000/6000 qty 3) => normal 115000.00, bonus 26000.00, jami 141000.00 — to'g'ri. bonus_qty=0 va normativdan past holatlar ham 0 bonus beradi. (b) RecipeVersionService#operationNamesByIds — sof additive yangi metod (26 qator, 0 o'chirilgan), @Transactional(readOnly=true), mavjud xatti-harakat o'zgarmagan; modul chegarasi (employees -> products repository'siga tegmaslik) uchun oqlanadi. RecipeVersionServiceTest 6/6 yashil. (c) Commit'da javadoc `*/` tuzatishi YO'Q — butun commit'dagi yagona o'chirilgan qator WageEntryRepository.java javadoc sarlavhasi ("...EP-06-T06-S01)." -> "...EP-06-T06-S01; 6.1/6.2/10.1 o'qish qo'shimchalari — EP-06-T06-S03)."). Hech narsa yo'qolmagan; kod kompilyatsiya bo'ladi. (d) Maskalash: EmployeeService/EmployeeResponse (current_salary, T05-S02) commit'da UMUMAN tegilmagan. Bu endpoint uchun kontrakt kaliti — monthly_salary_amount (current_salary emas); huquq yo'q bo'lsa kalit @JsonAnyGetter orqali JSON'ga UMUMAN chiqmaydi (null emas), autentifikatsiya yo'q bo'lsa false (fail-closed). `piece_rate` blokini `piece_rate.view` bilan yopish talabi kontraktga ZID bo'lardi: 7-bo'limda bunday rule yo'q (faqat piece_rate.manage, 02_products egaligida) va kontraktda piece_rate `required`, izohda "ishbay qismi baribir qaytadi" deyilgan. (e) AC2902: EmployeePayrollSummaryResponse = employee_id, date_from, date_to, [monthly_salary_amount], piece_rate; WageTotals = qty/normal_amount/bonus_amount/total_amount/entry_count. Proratsiya/to'lov/avans/yaxlitlash/pending maydonlari YO'Q; kontrakt boshidagi SPEC BO'SHLIG'I hurmat qilingan — coder "kutilayotgan" qismni o'zi to'qib qo'shmagan. (f) AC2904: qo'lda auditService.log(...) YO'Q — faqat OwnershipDeniedException.asForbidden, markaziy GlobalExceptionHandler yozadi (mutatsiya 2 buni isbotladi). Cross-cutting: barcha DB metodlari @Transactional(readOnly=true); migratsiya yo'q (tenant_id/FK/precision o'zgarmagan); soft-delete hurmat qilingan (findByIdAndDeletedAtIsNull / findByUserIdAndDeletedAtIsNull); log'ga token/parol yozilmaydi; JSON snake_case + BigDecimal string (JacksonConfig). Kontrakt: yo'llar, query parametrlari, maydon nomlari/tiplari va xatolik kodlari (400/403/404) openapi/v1/employees.yaml bilan mos. Kontrakt o'zgartirilmagan. TAVSIYA (bloklamaydi, keyingi ish uchun): AC2905 doimiy testi faqat BITTA operatsiyani qamraydi (hasSize(1)) — "operatsiya kesimida alohida qatorlar" semantikasini QA probe'i bilan tasdiqladim, lekin doimiy regressiya testi sifatida ikki operatsiyali holat qo'shilsa yaxshi bo'lardi. Shuningdek WageEntryView schema nomi kontraktdagi WageEntryResponse'dan farq qiladi (maydonlar aynan mos) — epic oxiridagi OpenAPI drift ratchet'ida ko'rib chiqilsin.

qa-ep-06-t06-s03 qabul mezonini belgiladi

✓ Bugungi normativ progress operatsiya kesimida `qty`/`norm_per_day` juftligini qaytaradi

qa-ep-06-t06-s03 qabul mezonini belgiladi

✓ `/me/payroll/summary` faqat so'rovchi hodimning ma'lumotini qaytaradi; boshqa hodim id'si bilan urinish `OwnershipDeniedException` orqali rad etiladi va `auth_logs` da `deny` qatori paydo bo'ladi

qa-ep-06-t06-s03 qabul mezonini belgiladi

✓ `payroll.view` siz so'rov 403; `employee_salary.view` siz maosh maydoni javobda yo'q

qa-ep-06-t06-s03 qabul mezonini belgiladi

✓ Javob sxemasida proratsiya/to'lov maydonlari YO'Q (08_finance chegarasi)

qa-ep-06-t06-s03 qabul mezonini belgiladi

✓ Web payroll javobida oylik maosh + ishbay jami + bonus ulushi to'g'ri (§6.1 misolidan yig'ilgan raqamlar bilan test)

coder-ep-06-t06-s03 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-06-t06-s03 yangiladi

Yangilandi: branch, commit_sha

coder-ep-06-t06-s03 qabul mezonini belgiladi

✓ Bugungi normativ progress operatsiya kesimida `qty`/`norm_per_day` juftligini qaytaradi

coder-ep-06-t06-s03 qabul mezonini belgiladi

✓ `/me/payroll/summary` faqat so'rovchi hodimning ma'lumotini qaytaradi; boshqa hodim id'si bilan urinish `OwnershipDeniedException` orqali rad etiladi va `auth_logs` da `deny` qatori paydo bo'ladi

coder-ep-06-t06-s03 qabul mezonini belgiladi

✓ `payroll.view` siz so'rov 403; `employee_salary.view` siz maosh maydoni javobda yo'q

coder-ep-06-t06-s03 qabul mezonini belgiladi

✓ Javob sxemasida proratsiya/to'lov maydonlari YO'Q (08_finance chegarasi)

coder-ep-06-t06-s03 qabul mezonini belgiladi

✓ Web payroll javobida oylik maosh + ishbay jami + bonus ulushi to'g'ri (§6.1 misolidan yig'ilgan raqamlar bilan test)

coder-ep-06-t06-s03 fayllarni qayd etdi

15 ta fayl qayd etildi

coder-ep-06-t06-s03 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-06-t06-s03 ishga oldi

worker-ep-06-t06 izoh qoldirdi

worker-EP-06-T06 yakuni: S01/S02/S04 done. S03 BOSHLANMADI — board bogliqlikni ozi rad etdi (claim -> 409 "Bogliqliklar yopilmagan: EP-06-T05-S02"). Bu spec boshligi EMAS, texnik bogliqlik: AC2903 (employee_salary.view bolmasa maosh maydoni javobda YOQ) aynan T05-S02 maskalash mexanizmiga tayanadi. Kodda tasdiqlandi: employee_salary.view uchun main kodda 0 ta ishlatish; EmployeeResponse.java:15, EmployeeService.java:426, EmployeeSalaryResponse.java:11 javadoclarida maskalash ochiq "EP-06-T05 zonasi" deb qoldirilgan. blocked QILINMADI — T05-S02 yopilgach S03 avtomatik claimable boladi (qolda ochish shart emas). Eslatma: /me/payroll/summary dagi kutilayotgan ish haqi spec boshligi S03 ga TOSIQ EMAS — kontrakt (openapi/v1/employees.yaml) pending qismni ataylab istisno qiladi, barcha summalar tasdiqlangan wage_entries dan.

pm-agent bog'liqlik qo'shdi

→ EP-06-T05-S02

pm-agent bog'liqlik qo'shdi

→ EP-06-T06-S01

pm-agent yaratdi

Payroll o'qish API — hodim × davr + mobil `/me/payroll/*`