sodalive-backend-spring-boot/docs/20261109_감지번역공급자공통화/reviews/phase-2-review.md

코드 기반 공급자 순서 Phase 2 리뷰

항목	내용
기준	PRD LP-011/012, 계획 Phase 2
일자	2026-11-09
리뷰어	독립 읽기 전용 gate reviewer
판정	APPROVE

판정 근거

• LanguageProviderOrder의 두 독립 val List<String>이 production 순서의 기준이며 기본값은 각각 papago다.
• LanguageProviderProperties는 typed-list 생성자와 @Autowired 기본 생성자만 제공한다. 우선순위용 String overload, @Value, 문자열 파싱이나 mutable 전역 설정이 없다.
• YAML 삭제는 우선순위 블록 5줄뿐이다. 인증 환경변수는 유지한다.
• 실제 Spring 기본 경로에서 과거 property/env 값은 순서에 영향을 주지 않는다.
• 독립 순서·invalid 목록·중복 공급자·H2 귀속/재사용의 기존 검증을 유지한다.
• 현재 가이드가 코드 편집 지점·빌드/배포·기존 override 무시 동작과 일치한다.
• enum 변경은 사용자가 적합성만 질문한 별도 미승인 범위로, 이번 구현 누락이 아니다.

확정 발견 사항 없음. 수정 Task 전환 항목 없음.

검증 근거와 한계

변경 전 의미 있는 RED 2건, 구현 후 88/88 통과와 실제 lint/bootJar 성공 기록을 검토했다. 부모가 XML 15개를 직접 대조했고, focused 54건을 --rerun하여 exit 0(36초)을 확인했다. 부모의 별도 Java/Spring 실행은 충돌 property/env 무시, 감지 B/A/C·번역 C/B/A, invalid 목록/공급자 거부를 통과했다. Spring context는 종료됐으며 외부 API 요청은 없었다. reviewer git diff --check는 exit 0이다.

리뷰는 코드·테스트·설정을 변경하거나 테스트/빌드를 재실행하지 않았다. 전체 테스트·LSP·유료 API·운영 MySQL은 미검증이며 기존 H2 claim SQL 대역 한계를 그대로 명시한다.