Báo cáo sai lệch — warehouse/import-excel-save-data — vòng 3

0. Kết luận nhanh

• Điều kiện đạt: G1 ✅ G2 ✅ G3 ✅ G4 — G5 ✅ G6 ✅ → CHỜ KIỂM SONAR
• Trạng thái: SẴN SÀNG CHO NGƯỜI XEM
• Tầng rủi ro: A (ghi nhiều dòng, sinh mã, tenant R-03/R-04) → người cần đọc: spec + báo cáo + toàn bộ diff
• Số quy tắc: 10 · PASS 4 · DEVIATION có chủ đích 4 · DEVIATION KHÔNG chủ đích 0 · UNVERIFIED 2
• Coverage thấp nhất: F-01 98,7% dòng / 90,9% nhánh
• Finding logic: CRITICAL 0 · MAJOR 0 còn mở (1 đã sửa) · MINOR 2 · UNSURE 0
• Nghi vấn bug code cũ (mục 3c): 7 (CAO 1) · Quyết định tự chốt cần xác nhận (mục 3d): 11
• Chạy bằng: /mig-auto (spec do spec-reviewer kiểm, chưa có người duyệt) · Số vòng: 3 verify / lần sửa 2/3

1. Điều kiện đạt

Không chặn nhưng cần người: Integration test CHƯA CHẠY · Golden master CHƯA CHẠY (GM-01)

Coverage theo luồng

2. Ma trận truy vết

3. Người cần xem

3b. Phân tích luồng sau chuyển

Chi tiết: verify/round-2/flow-analysis.md (vòng 3 không đổi)

3c. Nghi vấn bug / sai ở code cũ (đã GIỮ nguyên khi chuyển)

B-01 Tạo kho qua import bị bỏ qua (hoặc lỗi) nếu công ty khác đã dùng mã kho đó

• Mức nghi ngờ: CAO — câu kiểm tra thiếu điều kiện công ty, trong khi mọi câu khác đều lọc công ty; bước validate chỉ kiểm trong công ty nên báo hợp lệ.
• Ở đâu: code cũ base:repositories/BaseWarehouseRepository.java:115-116 (findBySku), base:services/impl/BaseWarehouseServiceImpl.java:1121-1122 → code mới (giữ nguyên) WarehouseRelationalRepository.countBySkuAllCompanies, WarehouseImportService.createWarehouses
• Code cũ đang làm gì: trước khi tạo kho, tìm kho có cùng mã (SKU) ở mọi công ty. Có 1 kho → bỏ qua dòng đó, không báo gì. Có từ 2 kho → lỗi hệ thống, không kho nào được tạo.
• Ví dụ cụ thể: công ty B đã có kho mã KHO01. Công ty A import file có dòng KHO01 — màn validate báo hợp lệ; bấm lưu → báo "Nhập danh sách kho hàng thành công" nhưng không có kho nào được tạo (count=0). Nếu cả công ty B và C đều có KHO01 → công ty A nhận "Lỗi hệ thống" cho cả file. Đúng nghiệp vụ thì kho KHO01 của công ty A phải được tạo.
• Vì sao nghi là sai: mã kho là duy nhất trong một công ty (các API thêm/sửa kho chỉ kiểm trong công ty); câu findBySkuIn ngay cạnh có com_id.
• Ảnh hưởng nếu đúng là bug: công ty dùng mã phổ biến (K01, KHO1…) không import tạo kho được, không có thông báo lỗi; khó tự phát hiện.
• Đã xử lý khi chuyển: GIỮ. Test khoá: TC-17 (WarehouseImportServiceTest#existingSkuAnywhereIsSkipped), TC-18 (#duplicatedSkuAnywhereIsSystemError), TC-28 (WarehouseRelationalRepositoryTest$ImportSku#countBySkuAllCompanies).
• Nếu muốn sửa: thêm AND com_id = :comId vào countBySkuAllCompanies (đổi tên), báo lỗi/ghi rõ dòng bị bỏ qua; đổi TC-17, TC-18, TC-28. Hệ thống cũ còn chạy song song sẽ lệch kết quả.
• Liên quan: D-04, R-06

B-02 Lưu import không kiểm lại dữ liệu: lưu cả dòng bị đánh dấu lỗi, tên rỗng, SKU trùng

• Mức nghi ngờ: TRUNG BÌNH — dựa hoàn toàn vào client gửi đúng kết quả validate.
• Ở đâu: backend:web/rest/WarehouseResource.java:117-121; base:services/impl/BaseWarehouseServiceImpl.java:1100-1143 → WarehouseController.saveImportData, WarehouseImportService
• Code cũ đang làm gì: không validate, không lọc dòng statusImport=false, không kiểm trùng tên/SKU trong file và với DB.
• Ví dụ cụ thể: client gửi dòng {name:"", sku:"K09", statusImport:false} → tạo kho K09 tên rỗng. Hai dòng cùng SKU K11 → tạo 2 kho trùng SKU. Dòng {sku:"WH7"} và dòng không SKU (mã sinh ra WH7) → 2 kho cùng SKU WH7.
• Vì sao nghi là sai: API thêm kho thường chặn các trường hợp này.
• Ảnh hưởng nếu đúng là bug: dữ liệu kho trùng/rỗng khi client lỗi hoặc request tự dựng.
• Đã xử lý khi chuyển: GIỮ. Test khoá: TC-01, TC-23.
• Nếu muốn sửa: lọc statusImport, kiểm lại tên/SKU; đổi TC-01, TC-23.
• Liên quan: D-05, R-01, R-06

B-03 Cập nhật qua import đổi tên kho nhưng không cập nhật "tên tìm kiếm"

• Mức nghi ngờ: TRUNG BÌNH
• Ở đâu: base:services/impl/BaseWarehouseServiceImpl.java:1114-1115 → Warehouse.applyImport
• Code cũ đang làm gì: chép dữ liệu mới vào kho nhưng giữ cột normalized_name cũ.
• Ví dụ cụ thể: kho K01 "Kho cũ", import đổi thành "Kho Hà Nội" → ô tìm kiếm danh sách kho gõ "ha noi" không ra K01, gõ "kho cu" vẫn ra.
• Vì sao nghi là sai: thêm/sửa kho thường luôn tính lại tên tìm kiếm.
• Ảnh hưởng nếu đúng là bug: tìm kiếm kho sai sau khi import cập nhật.
• Đã xử lý khi chuyển: GIỮ. Test khoá: TC-08.
• Nếu muốn sửa: tính lại normalizedName trong applyImport; đổi TC-08; có thể cần chạy lại tên tìm kiếm cho dữ liệu cũ.
• Liên quan: D-06, R-05

B-04 Import cập nhật có thể ngừng kho mặc định; không ghi trạng thái là ngừng kho

• Mức nghi ngờ: TRUNG BÌNH
• Ở đâu: base:services/impl/BaseWarehouseServiceImpl.java:1114-1115 → Warehouse.applyImport
• Code cũ đang làm gì: cột trạng thái trống được hiểu là "ngừng hoạt động"; không áp luật kho mặc định WH1; không kiểm đơn chưa xử lý.
• Ví dụ cụ thể: dòng {sku:"WH1", name:"Kho tổng"} (không có trạng thái) → kho mặc định bị ngừng hoạt động, "cho bán" thành trống. Lẽ ra bị chặn như API sửa kho (UPDATE_WAREHOUSE_DEFAULT_INVALID).
• Vì sao nghi là sai: API sửa kho chặn đúng trường hợp này; bước validate có chặn nhưng bước lưu không kiểm lại.
• Ảnh hưởng nếu đúng là bug: cửa hàng mất kho bán mặc định.
• Đã xử lý khi chuyển: GIỮ. Test khoá: TC-09.
• Nếu muốn sửa: gọi validateDefaultWarehouse sau applyImport; đổi TC-09.
• Liên quan: D-07, R-05

B-05 Mã kho trong file khác hoa thường với mã đang lưu → "Lỗi hệ thống" cả file

• Mức nghi ngờ: THẤP — phụ thuộc collation DB (UNVERIFIED).
• Ở đâu: base:services/impl/BaseWarehouseServiceImpl.java:1108-1116 → WarehouseImportService.applyUpdates
• Code cũ đang làm gì: DB tìm thấy kho (so không phân biệt hoa thường) nhưng bước ghép dòng so phân biệt hoa thường → lỗi.
• Ví dụ cụ thể: kho lưu k01, file ghi K01 → cả file lỗi, không dòng nào được cập nhật. Lẽ ra cập nhật kho k01.
• Đã xử lý khi chuyển: GIỮ (500 EXCEPTION_ERROR). Test khoá: TC-13.
• Nếu muốn sửa: ghép không phân biệt hoa thường; đổi TC-13.
• Liên quan: D-08, R-05

B-06 Tạo kho qua import không ghi trạng thái → kho ngừng hoạt động

• Mức nghi ngờ: THẤP — bước validate thường điền trạng thái từ cột Excel.
• Ở đâu: base:dto/warehouse/WarehouseSaveRequest.java (getStatus) → Warehouse.createFromImport
• Ví dụ cụ thể: dòng {name:"Kho mới", sku:"K10"} → kho K10 ngừng hoạt động, "cho bán" trống. Thêm kho thường thì luôn hoạt động.
• Đã xử lý khi chuyển: GIỮ. Test khoá: TC-22.
• Nếu muốn sửa: mặc định hoạt động; đổi TC-22.
• Liên quan: D-09, R-06

B-07 Loại import sai vẫn báo "Nhập thành công"

• Mức nghi ngờ: THẤP
• Ở đâu: base:services/impl/BaseWarehouseServiceImpl.java:1104-1142 → WarehouseImportService.saveImportData
• Ví dụ cụ thể: type=3 với 5 dòng → "Nhập danh sách kho hàng thành công", không lưu gì.
• Đã xử lý khi chuyển: GIỮ. Test khoá: TC-24.
• Nếu muốn sửa: 400 khi type sai; đổi TC-24.
• Liên quan: D-10, R-07

3d. Quyết định AI tự chốt và thay đổi tự động (cần người xác nhận)

4. Finding đã xác nhận

F-logic-1 — MAJOR, data-integrity — CONFIRMED vòng 1 → ĐÃ HẾT vòng 2

Lưu từng kho, mỗi kho cập nhật thêm 1 SELECT (N+1). Sửa: WarehouseRepository.saveAll (một findAllById, một saveAll kho mới); khoá bằng TC-08-EXTRA-1/2.

F-logic-2, F-logic-3 — MINOR (mục 3 #6, #7)

5. Finding bị loại

6. Xử lý của người

7. So với vòng trước

8. Thay đổi trong repo đích

22 file (verify/round-3/changed-files.md): 15 main Java + 2 i18n + 5 test, trong src/main, src/test. Mới: WarehouseCodeGenerator, WarehouseImportService, WarehouseImportCommand, WarehouseImportResult, WarehouseImportSaveRequest, WarehouseImportRowRequest, WarehousePageReader, WarehouseImportServiceTest. Sửa code của unit khác: WarehouseManagementService (dùng generator), WarehouseRelationalRepository (tách đọc trang). Không có file ngoài phạm vi.

9. Đề xuất bổ sung kinh nghiệm