refactor(receiver): 수신자 서버 거절 예외 4종을 ReceiverFailure.ServerRejection 하나로 압축 (#934 4-A) [base: develop]

[1hyok]

#934 의 4-A 단계만 담는다. 1·2단계는 #1051, 4-B(애프터노트)는 별도 PR — 이슈는 닫지 않는다.

무엇을

ReceiverServerRejectionException(sealed 루트) + 하위 3종(MasterKey·EmailAuth·DeliveryVerification) → ReceiverFailure.ServerRejection 하나. 파일 4개 → 1개.

왜 하위 타입을 지우나

나눠 둔 명분은 KDoc 상 "사유 code 별 분기를 넣을 때 when 이 누락을 잡아준다" 였는데, 셋을 갈라 보는 프로덕션 코드가 없었다. 소비처는 상위 타입만 보고, data 계층은 셋을 만들기만 했다. 어느 endpoint 가 거절했는지는 stack trace 에 남으므로 타입으로 나눌 이유가 없다. 구분이 실제로 필요해지면 그때 필드를 더한다.

묶기만 하지 않았다

toErrorPayload 와 shouldReportInReceiverFlow 는 단일 타입 캐스팅(as? / is)이라 sealed 이득이 0 이었다. 루트로 좁혀 when 을 exhaustive 하게 만들어, 수신자 실패 유형이 늘면 두 소비처가 컴파일 에러로 잡힌다.

판정 기준은 그대로다 — 4xx 대역·code allowlist·문구 유무 모두 손대지 않았으므로 화면 문구와 텔레메트리 기록 대상이 변하지 않는다.

회귀 가드

노출 게이트는 사용자가 보는 스낵바 문구를 직접 좌우한다(#651 이 고친 회귀). ErrorPayloadTest 6케이스·ReceiverFlowReportingTest 4케이스를 케이스 그대로 이행했다 — 5xx 문구 제외, 미등재 code 폴백, 문구 없는 4xx 폴백 등 조합이 전부 살아 있다.

주석 정리 동반

대상	처리
USER_DISPLAYABLE_SERVER_CODES 의 code 별 의미 나열	제거 — BE 소유이고 한 번 틀려 #654 로 정정한 자리다. "신규 code 는 문구 확인 후 등재" 규약만 남김
ErrorPayload KDoc 의 통일은 #446 소관	제거 — 오참조다. 실제 소관은 #604(그 본문이 #446 과 층이 다름을 명시)
status 만 보고 code 를 안 보면… 판정 근거	진입점 → 실제로 판정하는 함수 자리로 이동
shouldReportInReceiverFlow KDoc 판정표	제거 — 코드와 중복. 근거 3문단은 1문단으로 압축
AfternoteFailureStage 하단 이전 이력(#615)	제거

남긴 것: Crashlytics 8건 보관 한도(+공식 문서 링크), 마스터 키를 대신 획득하지 않는 이유(#454), 업로드 진행 중 잠금(#711), @Valid 실패가 리터럴 code=400 으로 온다는 실측(#600).

검증

게이트	결과
:app:compileDebugUnitTestKotlin	green
:app:compileDebugAndroidTestKotlin (#949 신설)	green
전 모듈 ktlintCheck	green
:feature:receiver:data · :feature:receiver:presentation · :feature:afternote:presentation testDebugUnitTest	green

시각 검증 해당 없음 — 리소스·Compose 변경 0(바뀐 presentation 파일은 상태 data class 와 ViewModel). baseline 사이클 미실행.

리뷰어에게

• AfternoteFailureReporting.kt 를 refactor(core): 인증·계정 예외 6종을 CoreAuthFailure sealed 루트로 압축 (#934 1·2단계) [base: develop] #1051 도 건드린다. 그쪽은 백틱 주석 안 심볼명 치환이고 여기선 그 문단을 지운다. 순차 머지 시 뒤쪽이 리베이스 한 번 필요하고 해결은 "삭제 채택".
• ServerRejection 이 여전히 status·serverCode 를 들고 있는 건 의도적으로 남긴 것이다. 도메인이 HTTP 를 아는 계층 위반이 맞지만, 화면이 그 값으로 판정하고 있어 필드만 지우면 판정 근거를 잃는다. 판정을 data 로 내리는 작업은 동작 변경을 동반하므로 수신자 도메인 예외가 HTTP status·서버 code 를 나른다 — 노출·기록 판정을 data 로 내리고 도메인 어휘로 번역 #1053 으로 분리했다(blocked by refactor(core): 수신자 입력 거절의 서버 원문을 고정 진단 문구로 바꾼다 #934).
• androidTest 2파일(AppAndReceiverCompletionAndroidTest·ReceiverVerificationAndroidTest)도 갱신했다 — androidTest 도메인 API 드리프트 44건+런타임 7건 수정, PR 컴파일 게이트 추가 #949 가 :app:compileDebugAndroidTestKotlin 을 필수 게이트에 넣어 빠뜨리면 깨진다.

[github-actions]

이 PR 은 develop 와 충돌 상태이고, 그래서 CI 가 돌지 않는다.

pull_request 워크플로는 base 와 head 를 합친 가상 머지 커밋(refs/pull/<N>/merge)에서
실행되는데, 충돌이면 GitHub 이 그 ref 를 만들지 못해 워크플로가 트리거조차 되지 않는다.
체크가 실패하는 것이 아니라 아예 생성되지 않으므로, 새 커밋을 올려도 no checks reported
인 채 최신 커밋이 한 번도 검증되지 않은 상태로 남는다 (#1028).

develop 를 병합해 충돌을 풀면 검증이 다시 돈다. 해소되면 이 라벨은 자동으로 떨어진다.