refactor: 웹페이지 이미지 저장 책임을 스토어와 유즈케이스로 분리

Author: opficdev

DevLog/App/Assembler/InfraAssembler.swift

@@ -53,7 +53,9 @@ final class InfraAssembler: Assembler {
         }
 
         container.register(WebPageMetadataService.self) {
-            WebPageMetadataService()
+            WebPageMetadataService(
+                store: container.resolve(WebPageImageStore.self)
+            )
         }
 
         container.register(NWPathConnectivityProvider.self) {

DevLog/Infra/Service/WebPageMetadataService.swift

@@ -10,8 +10,13 @@ import LinkPresentation
 import UIKit
 
 final class WebPageMetadataService {
+    private let imageStore: WebPageImageStore
     private let logger = Logger(category: "WebPageMetadataService")
 
+    init(store: WebPageImageStore) {
+        self.imageStore = store
+    }
+
     func fetchMetadata(from urlString: String) async throws -> WebPageMetadataResponse {
         logger.info("Fetching metadata for URL: \(urlString)")
 
@@ -46,12 +51,7 @@ final class WebPageMetadataService {
         }
 
         do {
-            let removed = try await Task.detached(priority: .utility) {
-                let fileURL = try Self.cacheFileURL(for: url)
-                guard FileManager.default.fileExists(atPath: fileURL.path) else { return false }
-                try FileManager.default.removeItem(at: fileURL)
-                return true
-            }.value
+            let removed = try imageStore.removeImage(for: url)
 
             if removed {
                 logger.info("Removed cached image for URL: \(urlString)")
@@ -66,11 +66,12 @@ final class WebPageMetadataService {
             throw URLError(.badURL)
         }
 
-        return try Self.cacheFileURL(for: url)
+        return try imageStore.cachedImageURL(for: url)
     }
 
     private func extractImageURL(from imageProvider: NSItemProvider?, url: URL) async throws -> URL? {
         guard let imageProvider else { return nil }
+        let imageStore = self.imageStore
 
         return try await withCheckedThrowingContinuation { continuation in
             imageProvider.loadObject(ofClass: UIImage.self) { image, error in
@@ -86,44 +87,12 @@ final class WebPageMetadataService {
                 }
 
                 do {
-                    let fileURL = try Self.cacheFileURL(for: url)
-                    Task.detached { [data, fileURL] in
-                        do {
-                            try data.write(to: fileURL, options: [.atomic])
-                            continuation.resume(returning: fileURL)
-                        } catch {
-                            continuation.resume(throwing: error)
-                        }
-                    }
+                    let fileURL = try imageStore.saveImage(data, for: url)
+                    continuation.resume(returning: fileURL)
                 } catch {
                     continuation.resume(throwing: error)
                 }
             }
         }
     }
-
-    private static func cacheFileURL(for url: URL) throws -> URL {
-        let imageDir = try imageDirectoryURL()
-        let fileName = url.absoluteString
-            .addingPercentEncoding(withAllowedCharacters: .alphanumerics) ?? UUID().uuidString
-
-        return imageDir
-            .appendingPathComponent(fileName)
-            .appendingPathExtension("jpeg")
-    }
-
-    private static func imageDirectoryURL() throws -> URL {
-        let directory = try FileManager.default.url(
-            for: .applicationSupportDirectory,
-            in: .userDomainMask,
-            appropriateFor: nil,
-            create: true
-        )
-        let imageDir = directory.appendingPathComponent("webPageImages", isDirectory: true)
-        if !FileManager.default.fileExists(atPath: imageDir.path) {
-            try FileManager.default.createDirectory(at: imageDir, withIntermediateDirectories: true)
-        }
-
-        return imageDir
-    }
 }