POST /orders/{id}/start-picking — picking taski (R5, R18)

finalize-ep-10 yangiladi

Yangilandi: commit_sha, pr_url

qa-ep-10-t05-s01 izoh qoldirdi

QA (qa-EP-10-T05-S01) — izolyatsiyalangan worktree /tmp/qa-EP-10-T05-S01 @ 0a956d5. VERDIKT: done. TEST: mvn -Dtest=OrderStartPickingControllerTest test -> Tests run: 8, Failures: 0, Errors: 0. BUILD SUCCESS. MEZONLAR (kod darajasida): * AC3705 — OrderService.java:770 pickingRateRepository.findActiveByOrganizationId(order.getOrganizationId()).orElseThrow(... PICKING_RATE_MISSING). So'rov (PickingRateRepository:23) IKKALA shartni ham qo'yadi: r.active = true AND r.organizationId = :organizationId. Fixture xilma-xilligi TO'LIQ: stavka umuman yo'q / faqat active=false stavka / faol stavka BOSHQA org-node da — uchchala shoxobcha ham alohida testda. * AC3706 — OrderService.java:766 !STATUS_READY_TO_DELIVER.equals(order.getStatus()) -> INVALID_STATUS_TRANSITION. Test qolgan SAKKIZ holatni ham aylanib chiqadi (created/received/in_progress/picking/picked/delivering/delivered/cancelled) va har biri uchun holat o'zgarmaganini + task/outbox yozilmaganini tekshiradi. * AC3707 — TaskService.createPickingTask: task_type=picking, order_id to'ldirilgan, olti maydon (production_run_id/variant_operation_id/product_variant_id/recipe_version_id/required_qty/profession_id) yozilmaydi -> NULL; V25 chk_tasks_type_fields bilan mos. Test har olti maydonni alohida isNull() bilan tekshiradi. MUTATSIYA TEKSHIRUVI — 7 mutant, 7 O'LDIRILDI, 0 tirik qoldi (har biri kutilgan qatorda yiqildi): 1. PickingRateRepository so'rovidan "r.active = true" olib tashlandi -> test:138 yiqildi (AC3705 inactive shoxobchasi HAQIQATAN qoplangan). 2. So'rovdan org filtri neytrallashtirildi -> test:153 yiqildi (node-lokallik qoplangan). 3. Holat gate'i "if (false)" ga aylantirildi -> test:170 (status=created) va test:236 yiqildi (AC3706 qoplangan). 4. outbox enqueueIfAbsent bloklandi -> test:215 yiqildi (order.picking_started da'vosi qoplangan). 5. task.setOrderId(orderId + 1) -> test:197 va test:239 yiqildi. MUHIM: tasks.order_id da FK YO'Q (V25 izohi), ya'ni bu qiymatni DB ushlay olmaydi — faqat AC3707 assert'i ushladi. 6. @RequirePermission("order.start_picking") olib tashlandi -> test:256 yiqildi (403 qoplangan). 7. task status "new" -> "in_progress" (chk_tasks_status BU qiymatga RUXSAT beradi, ya'ni DB ushlamaydi) -> test:207 yiqildi (maydon-darajasidagi assert'lar haqiqiy, DB CHECK'ga tayanmaydi). Yakunda "grep -rn MUTANT src/" BO'SH, worktree 0a956d5 ga bit-ma-bit tiklandi (git status toza). CROSS-CUTTING: (1) startPicking @Transactional, createPickingTask REQUIRED bilan AYNI tranzaksiyaga qo'shiladi; buyurtma findByIdForUpdate (FOR UPDATE) bilan qulflanadi. (2) tenant_id / schema almashtirish YO'Q. (3) permission_code "order.start_picking" R__permissions_and_currencies_seed.sql:111 da katalogda bor. (4) Asinxron ish outbox orqali — enqueueIfAbsent ON CONFLICT(dedup_key) DO NOTHING, dedup_key = order.picking_started:order:<id>; tashqi broker ishlatilmagan; event_type V34 CHECK ro'yxatida (V34:114). (5-7) Bu commit'da yangi migratsiya yo'q; fizik DELETE yo'q. (8) log/parol/token yozuvi qo'shilmagan. KONTRAKT: openapi/v1/orders.yaml o'zgarmagan (git diff bo'sh). Kod qaytaradigan ikki 409 kodi OrderStartPickingConflict (orders.yaml:1767-1772) ro'yxati bilan AYNAN mos. KUZATUV (bloklamaydi, AC emas): startPicking dagi uchta yozuv (tasks INSERT + orders.status + outbox) atomikligi test bilan ISBOTLANMAGAN — @Transactional olib tashlansa ham to'plam yashil qoladi, chunki majburiy uzilish nuqtasi yo'q. Kelgusi barglarda (assign-driver, R6 accrual) shu naqsh uchun rollback testi qo'shilsa foydali.

qa-ep-10-t05-s01 statusni o'zgartirdi Review → Bajarildi

qa-ep-10-t05-s01 qabul mezonini belgiladi

✓ Yaratilgan task `task_type='picking'`, `order_id` to'ldirilgan, qolgan olti maydon NULL

qa-ep-10-t05-s01 qabul mezonini belgiladi

✓ `ready_to_deliver` bo'lmagan holatdan `start_picking` rad etiladi

qa-ep-10-t05-s01 qabul mezonini belgiladi

✓ Faol `picking_rates` bo'lmagan node'da `start_picking` bloklanadi (AC12)

coder-ep-10-t05-s01 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-10-t05-s01 izoh qoldirdi

start-picking qoshildi: gate (PICKING_RATE_MISSING), holat gate (INVALID_STATUS_TRANSITION), picking task 07_tasks TaskService.createPickingTask orqali (yangi cross-module metod, mavjud TaskRepository qayta ishlatildi, yangi mexanizm yoq). order.picking_started outbox qatori bir tranzaksiyada. 8 ta integration test (Testcontainers) yashil; PICKING_RATE_MISSING gate uchun mutatsiya sinovi otkazildi va tikladi. tasks.order_id FK qoshilmadi - TasksTableMigrationTest hozircha FK yoqligini aniq tekshiradi ("06_orders epicida qoshiladi" izohi bilan), uni ozgartirish shu bargning doirasidan tashqarida.

coder-ep-10-t05-s01 yangiladi

Yangilandi: branch, commit_sha

coder-ep-10-t05-s01 qabul mezonini belgiladi

✓ Yaratilgan task `task_type='picking'`, `order_id` to'ldirilgan, qolgan olti maydon NULL

coder-ep-10-t05-s01 qabul mezonini belgiladi

✓ `ready_to_deliver` bo'lmagan holatdan `start_picking` rad etiladi

coder-ep-10-t05-s01 qabul mezonini belgiladi

✓ Faol `picking_rates` bo'lmagan node'da `start_picking` bloklanadi (AC12)

coder-ep-10-t05-s01 fayllarni qayd etdi

4 ta fayl qayd etildi

coder-ep-10-t05-s01 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-10-t05-s01 ishga oldi

pm-agent yaratdi

POST /orders/{id}/start-picking — picking taski (R5, R18)