fix: 웹페이지 썸네일 캐시 복구 로직 수정 및 삭제 시 캐시 정리

Author: opficdev

DevLog/Data/Repository/WebPageRepositoryImpl.swift

@@ -6,6 +6,7 @@
 //
 
 import Foundation
+import UIKit
 
 final class WebPageRepositoryImpl: WebPageRepository {
     private let webPageService: WebPageService
@@ -25,22 +26,19 @@ final class WebPageRepositoryImpl: WebPageRepository {
         pages.reserveCapacity(responses.count)
 
         for response in responses {
-            if needsImageRestore(response), let restored = try? await restoreWebPage(response) {
-                pages.append(restored)
+            if await needsImageRestore(response) {
+                if let restored = try? await restoreWebPage(response) {
+                    pages.append(restored)
+                } else if let page = try? responseWithoutImage(response).toDomain() {
+                    pages.append(page)
+                }
                 continue
             }
             if let page = try? response.toDomain() {
                 pages.append(page)
             }
         }
 
-        // 쿼리가 비어있을 때 모든 웹페이지를 fetch 해오기 때문에 가능한 것
-        if query.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty {
-            await metadataService.removeUnusedCachedImages(
-                keeping: responses.map(\.imageURL)
-            )
-        }
-
         return pages
     }
 
@@ -62,13 +60,24 @@ final class WebPageRepositoryImpl: WebPageRepository {
 }
 
 private extension WebPageRepositoryImpl {
-    func needsImageRestore(_ response: WebPageResponse) -> Bool {
+    func needsImageRestore(_ response: WebPageResponse) async -> Bool {
         guard !response.imageURL.isEmpty,
               let url = URL(string: response.imageURL),
               url.isFileURL else {
             return false
         }
-        return !FileManager.default.fileExists(atPath: url.path)
+
+        return await Task.detached(priority: .utility) {
+            guard FileManager.default.fileExists(atPath: url.path) else {
+                return true
+            }
+
+            guard let imageData = try? Data(contentsOf: url) else {
+                return true
+            }
+
+            return UIImage(data: imageData) == nil
+        }.value
     }
 
     func restoreWebPage(_ response: WebPageResponse) async throws -> WebPage? {
@@ -91,4 +100,14 @@ private extension WebPageRepositoryImpl {
 
         return try? newResponse.toDomain()
     }
+
+    func responseWithoutImage(_ response: WebPageResponse) -> WebPageResponse {
+        WebPageResponse(
+            id: response.id,
+            title: response.title,
+            url: response.url,
+            displayURL: response.displayURL,
+            imageURL: ""
+        )
+    }
 }

DevLog/Infra/Service/WebPageMetadataService.swift

@@ -61,43 +61,6 @@ final class WebPageMetadataService {
         }
     }
 
-    func removeUnusedCachedImages(keeping imageURLStrings: [String]) async {
-        let keptPaths: Set<String> = Set(
-            imageURLStrings.compactMap { imageURLString in
-                guard let url = URL(string: imageURLString), url.isFileURL else {
-                    return nil
-                }
-                return url.path
-            }
-        )
-
-        do {
-            let removedCount = try await Task.detached(priority: .utility) {
-                let imageDir = try Self.imageDirectoryURL()
-                guard FileManager.default.fileExists(atPath: imageDir.path) else { return 0 }
-
-                let cachedFileURLs = try FileManager.default.contentsOfDirectory(
-                    at: imageDir,
-                    includingPropertiesForKeys: nil,
-                    options: [.skipsHiddenFiles]
-                )
-
-                var removedCount = 0
-                for fileURL in cachedFileURLs where !keptPaths.contains(fileURL.path) {
-                    try FileManager.default.removeItem(at: fileURL)
-                    removedCount += 1
-                }
-                return removedCount
-            }.value
-
-            if 0 < removedCount {
-                logger.info("Removed \(removedCount) unused cached images")
-            }
-        } catch {
-            logger.error("Failed to remove unused cached images", error: error)
-        }
-    }
-
     private func extractImageURL(from imageProvider: NSItemProvider?, url: URL) async throws -> URL? {
         guard let imageProvider else { return nil }
 

DevLog/UI/Common/Component/WebItemRow.swift

@@ -15,11 +15,7 @@ struct WebItemRow: View {
 
     var body: some View {
         HStack {
-            CacheableImage(url: item.imageURL) {
-                Image(systemName: "globe")
-                    .resizable()
-                    .scaledToFit()
-            }
+            thumbnail
             .frame(width: sceneWidth / 10, height: sceneWidth / 10)
             .clipShape(RoundedRectangle(cornerRadius: 10))
 
@@ -41,4 +37,30 @@ struct WebItemRow: View {
         }
         .padding(.vertical, 4)
     }
+
+    @ViewBuilder
+    private var thumbnail: some View {
+        if let imageURL = item.imageURL {
+            AsyncImage(url: imageURL) { phase in
+                switch phase {
+                case .success(let image):
+                    image
+                        .resizable()
+                        .scaledToFill()
+                case .empty:
+                    ProgressView()
+                default:
+                    placeholderImage
+                }
+            }
+        } else {
+            placeholderImage
+        }
+    }
+
+    private var placeholderImage: some View {
+        Image(systemName: "globe")
+            .resizable()
+            .scaledToFit()
+    }
 }