Skip to content

[#460] WebPageMetadataServiceImpl에서 Sendable 이슈를 해결한다#495

Merged
opficdev merged 1 commit into
developfrom
fix/#460-Sendable
May 25, 2026
Merged

[#460] WebPageMetadataServiceImpl에서 Sendable 이슈를 해결한다#495
opficdev merged 1 commit into
developfrom
fix/#460-Sendable

Conversation

@opficdev
Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

WebPageMetadataServiceImpl에서 NSItemProvider.loadObject 콜백 내부에 WebPageImageStore가 캡처되며 발생하는 Sendable 경고 제거

📝 작업 내용

📌 요약

  • 웹페이지 메타데이터 이미지 추출 흐름에서 non-Sendable 캡처 제거
  • 이미지 데이터 추출과 이미지 저장 책임 분리

🔍 상세

  • loadObject 콜백에서는 UIImageData로 변환하는 작업만 수행하도록 변경
  • WebPageImageStore.saveImage 호출을 continuation 외부의 async 흐름으로 이동
  • WebPageImageStore 프로토콜 계약 변경 없이 DevLogInfra 구현 내부에서 Sendable 경고를 해결

📸 영상 / 이미지 (Optional)

@opficdev opficdev self-assigned this May 25, 2026
Copy link
Copy Markdown
Contributor

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

Choose a reason for hiding this comment

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

Code Review

이번 PR은 WebPageMetadataServiceImpl에서 이미지 추출 및 저장 로직을 리팩토링하여 imageData 메서드를 분리하고 비동기 흐름을 개선했습니다. 리뷰를 통해 실패 경로에서의 Continuation 누수 가능성이 지적되었으며, Sendable 안정성과 성능 향상을 위해 UIImage 대신 loadDataRepresentation을 사용하는 개선 방안이 제시되었습니다.

@opficdev opficdev merged commit 9d33d26 into develop May 25, 2026
1 check passed
@opficdev opficdev deleted the fix/#460-Sendable branch May 25, 2026 12:34
@opficdev opficdev changed the title [#460] WebPageMetadatServiceImpl에서 Sendable 이슈를 해결한다 [#460] WebPageMetadataServiceImpl에서 Sendable 이슈를 해결한다 May 25, 2026
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.

WebPageMetadataServiceImpl에서 Sendable 이슈를 해결한다

1 participant