sodalive-ios/docs/20261006_선물_UI_추가/reviews/phase6-tracking-entry-validation.md

Phase 6 운송장 등록 진입 검증 리뷰

1. 리뷰 정보

항목	내용
리뷰 대상	Phase 6 / P6-T1, P6-R1
기준 commit 또는 working tree	미커밋 working tree
리뷰 일자	2026-10-06
리뷰어	Sisyphus, Oracle 기능 리뷰, Oracle 정적 UI 리뷰
기준 문서	prd.md, plan-task.md
리뷰 상태	수정 검증 완료

2. 리뷰 목적과 범위

목적

• 운송장 등록 화면이 상세를 재조회하고 등록 가능 조건을 fail-closed로 판정하는지 확인한다.
• 개인정보 경계, guarded navigation, 반응형 너비와 긴 한국어 문구 접근성을 확인한다.

포함 범위

• 코드: SodaLive/Sources/V2/Gift/Tracking/**, 관련 display policy와 route helper.
• 테스트: production ViewModel을 직접 컴파일하는 임시 assertion harness.
• 문서: TRACK-001, TRACK-002, P6-T1.
• 수동 검증: 소스 기반 상태 전이와 정적 SwiftUI 검토.

제외 범위

• P6-T2 택배사·운송장 입력 UI와 P6-T3 등록 mutation.
• 로그인·실서버·시뮬레이터 screenshot·VoiceOver 실행 검증.

3. 판정 기준

심각도	기준
Blocker	핵심 흐름 불능, 개인정보 노출 또는 완료 판정 무효
High	확정 요구사항·API 계약 위반 또는 주요 회귀
Medium	제한된 lifecycle·화면 크기·접근성 조건의 기능 문제
Low	비핵심 유지보수성 또는 문서 정합성 문제

4. 검토한 근거

문서와 코드

• 요구사항: TRACK-001, TRACK-002.
• 계획: P6-T1, P6-R1.
• 코드: GiftTrackingRegistrationViewModel.swift, GiftTrackingRegistrationView.swift, GiftDisplayPolicy.swift, AppState.swift, BaseView.swift.

실행 환경

OS: macOS
Xcode SDK: iOS 26.0
대상: generic/platform=iOS, Debug, CODE_SIGNING_ALLOWED=NO
Browser/viewport: 해당 없음

실행한 검증

명령 또는 수동 검증	결과	핵심 증거
production ViewModel swiftc harness	성공	GREEN assertions: 56
xcodebuild / SodaLive-dev	성공	BUILD SUCCEEDED
xcodebuild / SodaLive	성공	BUILD SUCCEEDED
plutil -lint SodaLive.xcodeproj/project.pbxproj	성공	OK
swiftc -parse / git diff --check	성공	exit code 0
Oracle 기능 재리뷰	성공	APPROVE, Blocker/High/Medium 없음
Oracle 정적 UI 재리뷰	성공	APPROVED_STATIC, 차단 항목 없음
screenshot·VoiceOver·실서버	불가	인증 가능한 runtime 미확보

5. 발견 사항 요약

ID	심각도	상태	제목	소유 Task	후속 goal
REV-P6-001	Medium	수정 완료	비활성 중 toast 종료 시 상세 복귀 유실	P6-R1	완료
REV-P6-002	Medium	수정 완료	긴 오류 문구에서 재시도 버튼 세로 overflow 가능	P6-R1	완료

6. 발견 사항 상세

REV-P6-001 — 비활성 중 toast 종료 시 상세 복귀 유실

• 심각도: Medium
• 상태: 수정 완료
• 관련 요구사항: TRACK-001
• 소유 Task: P6-R1

관찰 내용

비활성 상태에서 toast가 닫히면 shouldReturnToDetail이 남아도 기존 hasAppeared guard 때문에 재등장 시 callback을 소비하지 못할 수 있었다.

권장 조치와 결과

모든 appearance에서 최초 조회 guard보다 먼저 errorToastDidDismiss()를 호출하고, ViewModel의 guard와 선행 flag 해제로 exactly-once를 유지했다. 수정 후 기능 재리뷰가 승인됐다.

REV-P6-002 — 긴 오류 문구에서 재시도 버튼 세로 overflow 가능

• 심각도: Medium
• 상태: 수정 완료
• 관련 요구사항: TRACK-002
• 소유 Task: P6-R1

관찰 내용

서버 오류 문구는 길이 제한이 없지만 오류 분기가 scroll되지 않아 작은 화면·큰 글자에서 재시도 버튼이 화면 밖으로 밀릴 수 있었다.

권장 조치와 결과

오류 메시지와 재시도 버튼을 기존 스타일 그대로 세로 ScrollView에 배치했다. 수정 후 정적 UI 재리뷰가 승인됐다.

7. 확정 항목의 plan·goal 전환

• 두 확정 항목을 plan-task.md의 완료된 회귀 Task P6-R1로 기록했다.

8. 리뷰 종료 판정

판정 항목	결과	근거
리뷰 범위 전체 확인	충족	코드·문서·검증 결과 대조
후보 항목 판정 완료	충족	두 Medium 항목 수정 완료
확정 항목 plan 반영	충족	P6-R1
보류 항목의 담당·재개 조건 기록	충족	인증 runtime 확보 후 screenshot·VoiceOver·실서버 확인
검증 명령과 결과 기록	충족	56 assertions, 양 스킴 build, lint·parse·diff check

최종 결론: 수정 검증 완료.

남은 항목: 로그인·실서버·screenshot·VoiceOver 검증은 최종 Gate에서 수행한다.

9. 수정 후 검증 기록

1차 수정 검증 — 2026-10-06

• 무엇을: REV-P6-001, REV-P6-002를 수정했다.
• 왜: 화면 lifecycle과 큰 글자에서 상세 복귀·재시도 접근성을 유지하기 위해서다.
• 어떻게:
  • production-source harness — 56 assertions 성공.
  • SodaLive-dev, SodaLive generic iOS Debug build — 성공.
  • Oracle 기능 재리뷰 — APPROVE.
  • Oracle 정적 UI 재리뷰 — APPROVED_STATIC.
• 남은 항목: 인증 runtime 기반 실제 화면·VoiceOver·실서버 검증.