np-dms/lcbp3

Fork 0

Files

T

admin 7e8f4859cd

CI / CD Pipeline / build (push) Failing after 6m24s

Details

CI / CD Pipeline / deploy (push) Has been skipped

Details

feat(ai): add ADR-036 unified OCR architecture and frontend test coverage

- Add ADR-036 unified OCR architecture (typhoon-ocr via Ollama)
- Extend AI execution profiles for OCR sandbox configuration
- Add comprehensive frontend test coverage (components, hooks, services)
- Add backend test coverage for document-numbering services
- Update OCR sidecar with typhoon-ocr integration
- Add AI policy service and execution profile management
- Update AGENTS.md and architecture documentation

2026-06-14 06:34:07 +07:00

4.0 KiB

Raw Blame History

Session — 2026-06-13 (OCR Sandbox-Production Parity & Polish)

Summary

สำเร็จการทำงานใน Phase 9 (Polish) สำหรับ Feature-236 (Unified OCR Architecture & Sandbox Parity) โค้ดได้รับการปรับปรุงให้ Type-safe 100%, แก้ไขข้อผิดพลาดของ ESLint และ Prettier, และรันการทดสอบผ่าน 100% (256/256 tests) ทั้งใน Frontend และ Backend

ปัญหาที่พบ (Root Cause)

การ import ExecutionProfile ใน ai.controller.ts หายไป: ทำให้ ESLint ไม่สามารถแปลงและตรวจสอบประเภทของ ExecutionProfile ได้อย่างถูกต้อง ส่งผลให้เกิดข้อผิดพลาด Unsafe argument of type error typed...
การ Cast ประเภทแบบไม่ปลอดภัยใน getProductionProfile: การใช้ profileName as ExecutionProfile โดยตรงจากตัวแปร string ถูกตั้งข้อสังเกตจาก ESLint ในบางสภาพแวดล้อม
ข้อผิดพลาด Unsafe member access ใน ai.controller.spec.ts: ตัวแปร req ที่คืนค่าจาก context.switchToHttp().getRequest() ถูกอนุมานเป็น any โดยอัตโนมัติ ทำให้การกำหนดค่า req.user แจ้งเตือนข้อผิดพลาด
ความไม่สอดคล้องของ Prettier: พบปัญหาการจัดรูปแบบโค้ด (formatting) เล็กน้อยหลังจากการปรับแก้โค้ด

การแก้ไข (Fix)

ไฟล์	การเปลี่ยนแปลง
`backend/src/modules/ai/ai.controller.ts`	นำเข้า `ExecutionProfile` และปรับปรุง `getProductionProfile` ให้ตรวจสอบและจัดประเภท ExecutionProfile อย่างปลอดภัยหลีกเลี่ยงการ cast ตรงๆ
`backend/src/modules/ai/tests/ai.controller.spec.ts`	ใช้ generic type parameter ใน `getRequest<T>()` เพื่อระบุประเภทข้อมูลของ `user` อย่างถูกต้องและแก้ไข ESLint warning
`specs/88-logs/rollouts.md`	เพิ่มบันทึก rollout ของ Feature-236
`AGENTS.md`	อัปเดตชื่อโมเดลหลักและ OCR ให้ใช้ canonical names (`np-dms-ai:latest` และ `np-dms-ocr:latest`)

กฎที่ Lock แล้ว

การเข้าถึง Request User ใน Controller tests: ให้ใช้ generic type parameter ใน getRequest<{ user: { user_id: number; username: string } }>() ทุกครั้ง เพื่อความปลอดภัยของไทป์และป้องกัน ESLint warning
การแปลง String เป็น Union Type ใน Controller: ควรใช้การตรวจสอบด้วย runtime array (เช่น validProfiles.find()) เพื่อตรวจสอบขอบเขตและจัดกลุ่มประเภทข้อมูล แทนการบังคับแคสต์ตรง ๆ (as UnionType)

Verification

Backend compilation: ✅ npx tsc --noEmit ผ่านด้วย 0 errors
Backend linting: ✅ npm run lint ผ่านด้วย 0 errors / warnings
Frontend compilation: ✅ npx tsc --noEmit ผ่านด้วย 0 errors
Frontend linting: ✅ npm run lint ผ่านด้วย 0 errors / warnings
Automated Tests: ✅ Jest tests ผ่าน 256/256 ตัวสำเร็จ 100%

4.0 KiB Raw Blame History