[OnlyReadOneBook]

Closes #930

파일 2개짜리 core:di 가 존재하는 탓에 core:data 의 repository 구현 6개가 전부 public 으로 열려 있었다. internal 은 모듈 스코프인데 바인딩이 다른 모듈에 있으니 닫을 방법이 없었다. 바인딩을 구현 옆으로 옮기고 모듈을 삭제한다.

변경

1. CoreRepositoryModule → core:data 로 이동하면서 3개로 분할

모듈	선언	바인딩	왜
CoreUserRepositoryModule	abstract class + internal abstract @Binds	Auth · User · UserProfile	androidTest 가 @TestInstallIn(replaces = …) 로 대체하는 대상
CoreAccountRepositoryModule	internal interface	Account	대체 대상 아님 → 통째로 닫음
CoreUploadRepositoryModule	internal interface	Photo · Video	위와 동일

분할한 이유는 replaces 가 전부 아니면 전무이기 때문이다. 통짜 모듈이면 androidTest 가 실물로 쓰는 3종(Account·Photo·Video)을 테스트 쪽에서 다시 바인딩해야 하고, 그 인자 타입으로 구현체가 노출돼 internal 화가 원천 봉쇄된다. 쪼개면 페이크로 갈아끼울 모듈만 replaces 하면 되고 나머지는 손댈 일이 없다.

CoreUserRepositoryModule 만 interface 가 아닌 이유 — replaces 는 모듈 클래스를 참조하므로 이 모듈은 public 이어야 한다. 그런데 바인딩 메서드가 public 이면 'public' function exposes its 'internal' parameter type 으로 컴파일되지 않고, interface 멤버에는 가시성을 줄 수 없다. abstract class 라야 클래스는 열어 두고 메서드만 internal 로 닫는다. KDoc 에 근거를 남겨 뒀다.

2. 구현체 5개를 internal 로 전환

UserRepositoryImpl 만 public 으로 남겼다 — SettingCompletionAndroidTest 가 Hilt 를 우회해 UserRepositoryImpl(userApi, authRepository) 를 직접 조립한다. 코드에 사유 주석을 달았고, 닫으려면 그 테스트의 페이크 기반 개조가 선행돼야 한다(이슈 본문에 선택지 3개 정리).

3. 나머지 정리

• DispatcherModule → core:common (IoDispatcher 한정자가 이미 거기 있다)
• app 의 core:di 의존을 core:data 로 교체, settings.gradle.kts include 제거, core/di 디렉터리 삭제
• 삭제된 core/di/build.gradle.kts 가 선언하던 kakao.sdk.auth·work.runtime.ktx 는 두 소스 어디서도 쓰지 않았다. 전자는 app 이, 후자는 core:common·core:startup 이 각각 직접 선언하고 있어 영향 없다.

검증

태스크	결과
:core:data:testDebugUnitTest	✅ 통과 — internal 구현체를 같은 모듈 test 소스셋이 정상 접근
:app:compileDebugUnitTestKotlin	✅ 통과
:app:assembleRelease	✅ BUILD SUCCESSFUL
:core:data:kspDebug/ReleaseKotlin · :app:kspDebug/ReleaseKotlin	✅ 실제 재실행 — Dagger 가 internal abstract @Binds 를 정상 처리, Hilt 그래프 생성 통과
ktlintFormat --rerun	✅ 포맷 변경 0

시각 검증은 해당 없음 — DI 배선과 가시성 변경뿐이고 Compose 본문·디자인 토큰·리소스 변경이 0건이라 screenshotTest 커버를 지나지 않는다.

리뷰어에게

:app:compileDebugAndroidTestKotlin 은 이 브랜치에서 red 인데, develop 이 원래 그렇다. #949 가 고치는 도메인 API 드리프트 44건이 그대로 남아 있다. 제 변경 탓이 아님을 다음과 같이 확인했다.

• origin/develop 을 별도 워크트리로 띄워 같은 태스크를 돌리고 에러 집합을 diff → 완전히 동일(추가 0 · 제거 0)
• 그 44건을 내는 파일 5개를 임시로 치우고 컴파일 → BUILD SUCCESSFUL. 이 PR 이 건드린 TestHiltModules.kt 와, UserRepositoryImpl 을 public 으로 남긴 판단이 함께 검증된다

#949 가 먼저 develop 에 들어가면 이 브랜치는 리베이스만으로 해당 게이트가 green 이 된다. 두 변경은 서로를 필요로 하지 않아 스택으로 묶지 않았다 — 파일이 겹치는 SettingCompletionAndroidTest.kt 는 이 PR 이 수정하지 않는다.

#949 머지 후에도 그 파일은 UserRepositoryImpl 을 직접 조립하므로, 위의 public 유지 판단은 그대로 유효하다.

[github-actions]

🚀 QA 배포 권장

• 위험도: high
• 포함 이슈: fix(mindrecord): 임시저장 선택 삭제 실패가 무시되고 완료 처리됨 #442, refactor(app): 홈·수신자 홈 화면 feature 모듈 이전 — app 잔존 presentation 정리 #546, fix(mindrecord): 홈 MEMORIES 카드·추억 공간 화면이 고정 더미 — 실데이터 미연동 #559, fix(mindrecord): 데일리질문 생성·수정 응답의 userDailyQuestionId 를 BaseResponse<Unit> 으로 버림 #573, fix(mindrecord): 주간리포트 상단 카운트와 요약 기록일수 불일치 — diaryAmount 는 1인데 week[] 에 미포함 #590, fix(mindrecord): 데일리질문 목록 상단 '오늘의 추천 질문' 영역 미구현 — 조회한 todayQuestion 을 화면이 소비하지 않음 #592, refactor(afternote): 수신자 흐름 58개 파일을 feature:receiver 로 이전 — 애프터노트 모듈이 타 도메인 열람 관문 소유 #615, fix(mindrecord): 홈 TODAY'S QUESTION 카드 '데일리질문 답변하기' 무동작 — 카드 데이터는 정상 #627, fix(mindrecord): data 계층 runCatching 이 CancellationException 을 Result 로 소비 — 취소 전파 보존 #670, fix(mindrecord): 화면 배경이 시안 #FAFAFA 대신 Material3 기본 surface #FEF7FF 로 렌더 — 탭 간 배경 불일치 #691, fix(mindrecord): 보기 스위치·캘린더 탐색 상태가 콘텐츠와 불일치 #724, fix(mindrecord): 주간리포트 주차 드롭다운이 5개로 끝나 시안의 추가 이력·스크롤 미노출 #729, fix(mindrecord): 주간리포트 요약 문구 공백 소실 — '이번 주,이름님은N일의…'로 붙어 표시 #732, fix(mindrecord): 주간리포트 캘린더가 감정 이모티콘과 점을 함께 표시 — 시안·디자이너 정의는 배타적 #749, fix(mindrecord): 주간리포트 데일리질문 날짜 파싱 실패가 오늘 날짜로 무음 폴백 — 틀린 날짜를 정상처럼 표시 #751, fix(mindrecord): 일기 날짜 선택 제목이 '발송 날짜' — 시안은 '기록 날짜'로 수정됨 #760, fix(mindrecord): 작성 툴바 '임시저장' 카운트가 항상 0 — 두 작성 화면 모두 draftCount 미전달 #769, refactor(mindrecord): DTO 기본값·불필요 nullable 제거 — 누락 계약을 정상값으로 은폐 #789, 로그아웃 시 로컬 저장소 3개가 지워지지 않음 — DataStore 수명(scope) 계약 도입 #912, chore(build): 빌드·테스트 클래스패스 bouncycastle 1.84 상향 — dependency-review critical(GHSA-574f-3g2m-x479) 해소 #921, NavigationDebugLogger 삭제 — 역할 끝난 일회성 디버그 프로브 #927, core:startup 모듈 삭제 — Initializer 2개를 app 으로 이동 #929, core:di 모듈 삭제 — 바인딩을 구현 옆으로 옮기고 repository impl 을 internal 로 #930, develop androidTest 컴파일 실패 44건 — 배치 머지 API 드리프트가 미검증 착륙, 컴파일 게이트 부재 #936, 추모 미디어 업로드 경로 통합 — Outcome 타입 제거, 사진·영상 중복 구현 1개로 #938, fix(core): presigned URL 요청에 필수 contentLength 가 빠져 이미지 업로드가 전부 실패 #950, fix(mindrecord): 수신자 데일리질문 isDraft 를 파싱하지 않고 false 하드코딩 — draft 노출 방어가 무효 #956, fix(build): androidx-compose-material3 선언 버전과 Gradle 해석 버전을 일치시킨다 #973, fix(build): coroutines-core 선언 버전과 Gradle 해석 버전을 일치시킨다 #974, fix(build): io.netty:netty-codec 보안 권고 대응 필요 #975, fix(build): io.netty:netty-codec-http 보안 권고 대응 필요 #976, fix(build): io.netty:netty-codec-http2 보안 권고 대응 필요 #977, fix(build): io.netty:netty-common 보안 권고 대응 필요 #978, fix(build): io.netty:netty-handler 보안 권고 대응 필요 #979, fix(build): io.netty:netty-handler-proxy 보안 권고 대응 필요 #980, fix(build): org.apache.commons:commons-lang3 보안 권고 대응 필요 #981, fix(build): org.bitbucket.b_c:jose4j 보안 권고 대응 필요 #982, fix(build): org.bouncycastle:bcpkix-jdk18on 보안 권고 대응 필요 #983, fix(build): org.bouncycastle:bcprov-jdk18on 보안 권고 대응 필요 #984, fix(build): org.jdom:jdom2 보안 권고 대응 필요 #985, fix(ci): 릴리스·WIF canary 배포가 같은 Firebase 앱에 동시 업로드 가능 — concurrency group 분리 #995, perf(ci): PR 검증 4개 job이 같은 debug variant 를 각자 컴파일 — build cache 가 전역 off #996, chore(ci): 실서비스 secret workflow 3종의 검증·decode 블록 3변종 중복 — 공유 composite 로 통합 #997, chore(ci): dependency-graph job correlator 불일치 — develop base PR 의 dependency-review 가 의존성 전량을 신규 도입으로 오탐 #1009
• 판단 근거: 고위험 변경이 포함됐습니다: .github/workflows/release-distribution.yml, PR fix(ci): 릴리스·WIF canary 가 같은 Firebase 앱에 동시 업로드 가능 (closes 995) [base: develop] #1001, PR chore(ci): 실서비스 secret workflow 3종의 검증·decode 3변종을 composite 로 통합 (closes 997) [base: develop] #1002, PR [fix] #627 홈 TODAY'S QUESTION 답변 CTA 를 작성 화면에 연결한다 #894. 다른 변경을 기다리지 않고 QA 배포합니다.

QA 포인트

• core:di 모듈 삭제 — 바인딩을 구현 옆으로 옮기고 repository impl 을 internal 로 #930 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• refactor(app): 홈·수신자 홈 화면 feature 모듈 이전 — app 잔존 presentation 정리 #546 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• refactor(afternote): 수신자 흐름 58개 파일을 feature:receiver 로 이전 — 애프터노트 모듈이 타 도메인 열람 관문 소유 #615 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 보기 스위치·캘린더 탐색 상태가 콘텐츠와 불일치 #724 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• perf(ci): PR 검증 4개 job이 같은 debug variant 를 각자 컴파일 — build cache 가 전역 off #996 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(ci): 릴리스·WIF canary 배포가 같은 Firebase 앱에 동시 업로드 가능 — concurrency group 분리 #995 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• chore(ci): 실서비스 secret workflow 3종의 검증·decode 블록 3변종 중복 — 공유 composite 로 통합 #997 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 홈 TODAY'S QUESTION 카드 '데일리질문 답변하기' 무동작 — 카드 데이터는 정상 #627 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): data 계층 runCatching 이 CancellationException 을 Result 로 소비 — 취소 전파 보존 #670 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 작성 툴바 '임시저장' 카운트가 항상 0 — 두 작성 화면 모두 draftCount 미전달 #769 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 화면 배경이 시안 #FAFAFA 대신 Material3 기본 surface #FEF7FF 로 렌더 — 탭 간 배경 불일치 #691 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 주간리포트 요약 문구 공백 소실 — '이번 주,이름님은N일의…'로 붙어 표시 #732 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 데일리질문 목록 상단 '오늘의 추천 질문' 영역 미구현 — 조회한 todayQuestion 을 화면이 소비하지 않음 #592 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 홈 MEMORIES 카드·추억 공간 화면이 고정 더미 — 실데이터 미연동 #559 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 주간리포트 캘린더가 감정 이모티콘과 점을 함께 표시 — 시안·디자이너 정의는 배타적 #749 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 주간리포트 상단 카운트와 요약 기록일수 불일치 — diaryAmount 는 1인데 week[] 에 미포함 #590 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 임시저장 선택 삭제 실패가 무시되고 완료 처리됨 #442 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 일기 날짜 선택 제목이 '발송 날짜' — 시안은 '기록 날짜'로 수정됨 #760 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(core): presigned URL 요청에 필수 contentLength 가 빠져 이미지 업로드가 전부 실패 #950 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• refactor(mindrecord): DTO 기본값·불필요 nullable 제거 — 누락 계약을 정상값으로 은폐 #789 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 수신자 데일리질문 isDraft 를 파싱하지 않고 false 하드코딩 — draft 노출 방어가 무효 #956 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 주간리포트 주차 드롭다운이 5개로 끝나 시안의 추가 이력·스크롤 미노출 #729 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 데일리질문 생성·수정 응답의 userDailyQuestionId 를 BaseResponse<Unit> 으로 버림 #573 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 주간리포트 데일리질문 날짜 파싱 실패가 오늘 날짜로 무음 폴백 — 틀린 날짜를 정상처럼 표시 #751 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• chore(ci): dependency-graph job correlator 불일치 — develop base PR 의 dependency-review 가 의존성 전량을 신규 도입으로 오탐 #1009 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• develop androidTest 컴파일 실패 44건 — 배치 머지 API 드리프트가 미검증 착륙, 컴파일 게이트 부재 #936 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• 로그아웃 시 로컬 저장소 3개가 지워지지 않음 — DataStore 수명(scope) 계약 도입 #912 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• chore(build): 빌드·테스트 클래스패스 bouncycastle 1.84 상향 — dependency-review critical(GHSA-574f-3g2m-x479) 해소 #921 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): org.bouncycastle:bcpkix-jdk18on 보안 권고 대응 필요 #983 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): org.bouncycastle:bcprov-jdk18on 보안 권고 대응 필요 #984 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• NavigationDebugLogger 삭제 — 역할 끝난 일회성 디버그 프로브 #927 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• core:startup 모듈 삭제 — Initializer 2개를 app 으로 이동 #929 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• 추모 미디어 업로드 경로 통합 — Outcome 타입 제거, 사진·영상 중복 구현 1개로 #938 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): androidx-compose-material3 선언 버전과 Gradle 해석 버전을 일치시킨다 #973 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): coroutines-core 선언 버전과 Gradle 해석 버전을 일치시킨다 #974 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-codec 보안 권고 대응 필요 #975 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-codec-http 보안 권고 대응 필요 #976 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-codec-http2 보안 권고 대응 필요 #977 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-common 보안 권고 대응 필요 #978 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-handler 보안 권고 대응 필요 #979 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-handler-proxy 보안 권고 대응 필요 #980 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): org.apache.commons:commons-lang3 보안 권고 대응 필요 #981 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): org.bitbucket.b_c:jose4j 보안 권고 대응 필요 #982 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): org.jdom:jdom2 보안 권고 대응 필요 #985 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인

이 판단은 배포를 자동 실행하지 않습니다. deploy일 때 Release Distribution을 수동 실행하세요.