Báo cáo sai lệch — warehouse/export-excel-error-data — vòng 1

SVC = easyposbase:src/main/java/sds/ep/baseservice/services/impl/BaseWarehouseServiceImpl.java; đường dẫn mới tính từ warehouse-service:src/main/java/com/softdreams/easypos/warehouse/. (Báo cáo CHẶN lần trước — thiếu Apache POI — đã bị ghi đè khi viết báo cáo này; lý do CHẶN còn trong nhật ký status.md.)

0. Kết luận nhanh

• Điều kiện đạt: G1 ✅ G2 ✅ G3 ✅ G4 — G5 ✅ G6 ✅ → CHỜ KIỂM SONAR (máy đạt hết, chờ người kiểm G4)
• Trạng thái: SẴN SÀNG CHO NGƯỜI XEM
• Tầng rủi ro: C (tự động — không đọc/ghi DB) → người cần đọc: spec + báo cáo
• Số quy tắc: 9 · PASS 6 · DEVIATION có chủ đích 2 (R-01/R-02 định dạng lỗi theo starter, R-07 xoá file tạm) · DEVIATION KHÔNG chủ đích 0 · UNVERIFIED 1 (R-09)
• Coverage thấp nhất: F-01 100% dòng / 100% nhánh
• Finding logic: CRITICAL 0 · MAJOR 0 · MINOR 2 · UNSURE 0
• Nghi vấn bug code cũ (mục 3c): 4 (CAO 0) · Quyết định tự chốt cần xác nhận (mục 3d): 6
• Chạy bằng: /mig-auto (spec do spec-reviewer kiểm, chưa có người duyệt) · Số vòng: 1 / lần sửa 0/3
• Sự cố trong lượt: working tree repo đích bị GitHub Desktop stash (f14b118) giữa chừng; người đã khôi phục. Build vòng 0 lúc đó vô hiệu; snapshot dựng lại (89a6df6… = HEAD a54d38b + code import-excel-validate), verify vòng 1 chạy sau khi khôi phục.

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-1/flow-analysis.md

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

B-01 Bấm tải file lỗi khi không có dòng nào → "lỗi hệ thống" không có câu thông báo

• Mức nghi ngờ: TRUNG BÌNH — tham số truyền đảo chỗ rõ ràng.
• Ở đâu: code cũ SVC:1204-1206 (new BaseInternalServerException(DATA_NOT_FOUND, ENTITY_NAME), constructor là (entityName, errorKey)) → code mới application/warehouse/WarehouseImportErrorExportService.java (SystemException(DATA_NOT_FOUND_LEGACY))
• Code cũ đang làm gì: danh sách rỗng thì báo lỗi, nhưng mã lỗi nhận nhầm tên module sds.ep.warehouse thay vì DATA_NOT_FOUND.
• Ví dụ cụ thể: gửi [] → HTTP 500, message[0].code = "sds.ep.warehouse", reason = null — giao diện không có câu thông báo. Lẽ ra là mã DATA_NOT_FOUND (và có thể nên là 400).
• Vì sao nghi là sai: thứ tự tham số ngược với constructor.
• Ảnh hưởng nếu đúng là bug: chỉ khi client gửi danh sách rỗng; người dùng thấy lỗi không rõ nghĩa.
• Đã xử lý khi chuyển: GIỮ mã sds.ep.warehouse, HTTP 500. Test khoá: TC-03 (WarehouseImportErrorExportServiceTest#emptyRowsIsLegacyError), TC-04.
• Nếu muốn sửa: đổi DATA_NOT_FOUND_LEGACY thành DATA_NOT_FOUND + thêm text i18n; đổi TC-03, TC-04.
• Liên quan: D-03, R-02

B-02 Lỗi khi tạo file bị nuốt → tải về file 0 byte

• Mức nghi ngờ: TRUNG BÌNH
• Ở đâu: SVC:1207-1251 → PoiExcelTableWriter.write, WarehouseImportErrorExportService.exportErrors
• Code cũ đang làm gì: mọi lỗi khi dựng file chỉ in log, vẫn trả HTTP 200 với phần đã ghi (thường rỗng).
• Ví dụ cụ thể: body [{"name":"Kho 1"}, null] hoặc mô tả dài 40.000 ký tự → 200, file 0 byte, không mở được.
• Vì sao nghi là sai: người dùng nhận file hỏng thay vì thông báo lỗi.
• Ảnh hưởng nếu đúng là bug: hiếm (dữ liệu lạ); khó chẩn đoán.
• Đã xử lý khi chuyển: GIỮ. Test khoá: TC-05, TC-06, TC-07.
• Nếu muốn sửa: ném SystemException khi dựng file lỗi; đổi TC-05..TC-07.
• Liên quan: D-04, R-03

B-03 File lỗi ghi ô trạng thái trống thành "Ngừng hoạt động" — ngược cách import hiểu ô trống

• Mức nghi ngờ: THẤP — thực tế bước kiểm luôn điền hai giá trị này.
• Ở đâu: SVC:1300-1301 → WarehouseImportErrorExportService.toTableRow
• Code cũ đang làm gì: không có trạng thái → ghi "Ngừng hoạt động"; không có bán trực tiếp → "Không". Bước import hiểu ô trống là "đang hoạt động"/"có".
• Ví dụ cụ thể: dòng {name:"Kho 1"} → file ghi "Ngừng hoạt động", "Không"; sửa file rồi import lại → kho bị ngừng hoạt động.
• Vì sao nghi là sai: hai chiều xuất/nhập không khớp.
• Ảnh hưởng nếu đúng là bug: chỉ khi client tự dựng body thiếu hai field.
• Đã xử lý khi chuyển: GIỮ. Test khoá: TC-11, TC-10.
• Nếu muốn sửa: để trống ô khi không có giá trị; đổi TC-10, TC-11.
• Liên quan: D-05, R-05

B-04 Header tải file: tên file filename.extension, kiểu form-data

• Mức nghi ngờ: THẤP — client hiện tại có lẽ tự đặt tên file.
• Ở đâu: backend:web/rest/WarehouseResource.java:113-114 → WarehouseController.exportImportErrors
• Ví dụ cụ thể: Content-Disposition: form-data; name="attachment"; filename="filename.extension" — tải trực tiếp bằng trình duyệt ra file không đuôi .xlsx.
• Đã xử lý khi chuyển: GIỮ (hợp đồng API). Test khoá: TC-14.
• Nếu muốn sửa: ContentDisposition.attachment().filename("…xlsx"); đổi TC-14; kiểm client.
• Liên quan: D-07, R-06

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

Không có CRITICAL/MAJOR. MINOR (không qua xác minh, chuyển cho người): F-logic-1 (3 style thay 4), F-logic-2 (mã lỗi không có i18n) — nguyên văn ở verify/round-1/review-logic.md.

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

12 file (verify/round-1/changed-files.md): 6 mới ở src/main (ExcelTable, ExcelTableWriter, PoiExcelTableWriter, WarehouseImportErrorRow, WarehouseImportErrorExportService, WarehouseImportErrorRowRequest), 3 sửa ở src/main (WarehouseErrorCode, WarehouseWebMapper, WarehouseController), 2 test mới, WarehouseControllerTest sửa. Không file ngoài phạm vi; không chạm pom.xml.

9. Đề xuất bổ sung kinh nghiệm (/mig-learn gom vào proposal để người duyệt)