Skip to content

[#706] 계정별로 캐시용 디렉토리를 분리한다#711

Merged
opficdev merged 4 commits into
developfrom
refactor/#706
Jul 8, 2026
Merged

[#706] 계정별로 캐시용 디렉토리를 분리한다#711
opficdev merged 4 commits into
developfrom
refactor/#706

Conversation

@opficdev

@opficdev opficdev commented Jul 8, 2026

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🔗 연관된 이슈

🎯 의도

  • WebPage 썸네일 캐시가 로그인 계정 간 같은 로컬 디렉토리를 공유하지 않도록 계정별 저장 위치를 분리

📝 작업 내용

📌 요약

  • WebPageImageStoreWebPageMetadataServiceaccountID scope 전달 추가
  • WebPageRepositoryImplWebPageImageRepositoryImpl에서 현재 AuthService.uid 기준으로 캐시 저장, 조회, 삭제 연결
  • WebPageImageStoreImpl에서 계정 ID 해시 기반 하위 디렉토리 사용
  • 비피쳐 테스트 제거 후 WebPage 캐시 피쳐 테스트 중심으로 테스트 정리

🔍 상세

  • WebPage 저장, 복구, 삭제 시 현재 계정의 cache URL만 사용하도록 accountID 전달 경로 정리
  • 계정 ID가 없거나 빈 값인 경우 기존 webPageImages 경로를 유지해 기존 동작 보존
  • 기존 파일 URL이 현재 계정 cache 경로와 다르면 metadata를 재생성해 다른 계정 캐시 재사용 방지
  • 설정 캐시 삭제가 현재 계정 디렉토리만 비우도록 Persistence 계정 scope 검증 추가
  • 구현 단위 테스트 제거 및 빈 DomainTests 타깃 제거
  • 전체 workspace 스킴 build-only 순회로 생성 프로젝트와 테스트 타깃 구성 확인

📸 영상 / 이미지 (Optional)

  • 없음

@opficdev opficdev self-assigned this Jul 8, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

이번 풀 리퀘스트는 웹페이지 이미지 캐시를 계정별(accountID)로 격리하여 관리하도록 개선합니다. WebPageImageStoreWebPageMetadataService 프로토콜과 구현체에 accountID 매개변수를 추가하고, AuthService를 통해 획득한 사용자 UID를 기반으로 SHA256 해시된 계정별 서브디렉토리에 이미지를 저장하도록 변경했습니다. 또한 관련 테스트 코드를 추가하고 불필요한 기존 테스트 파일들을 정리했습니다. 제공된 리뷰 코멘트가 없으므로 추가로 제시할 피드백은 없습니다.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@opficdev opficdev merged commit 27a7106 into develop Jul 8, 2026
7 checks passed
@opficdev opficdev deleted the refactor/#706 branch July 8, 2026 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

계정별로 캐시용 디렉토리를 분리한다

1 participant