consume security scoped bookmarks to read urls (#314)

* consume security scoped bookmarks to read urls

* move entitlements to correct directory

Author: davidkoski

MLXChatExample.entitlements …XChatExample/MLXChatExample.entitlementsMLXChatExample.entitlements renamed to Applications/MLXChatExample/MLXChatExample.entitlements MLXChatExample.entitlements renamed to Applications/MLXChatExample/MLXChatExample.entitlements

@@ -8,6 +8,8 @@
 	<true/>
 	<key>com.apple.security.files.downloads.read-write</key>
 	<true/>
+	<key>com.apple.security.files.user-selected.read-only</key>
+	<true/>
 	<key>com.apple.security.network.client</key>
 	<true/>
 </dict>

Applications/MLXChatExample/Services/MLXService.swift

@@ -107,7 +107,8 @@ class MLXService {
         }
 
         // Prepare input for model processing
-        let userInput = UserInput(chat: chat)
+        let userInput = UserInput(
+            chat: chat, processing: .init(resize: .init(width: 1024, height: 1024)))
 
         // Generate response using the model
         return try await modelContainer.perform { (context: ModelContext) in

Applications/MLXChatExample/ViewModels/ChatViewModel.swift

@@ -160,15 +160,29 @@ class MediaSelection {
     var isShowing = false
 
     /// Currently selected image URLs
-    var images: [URL] = []
+    var images: [URL] = [] {
+        didSet {
+            didSetURLs(oldValue, images)
+        }
+    }
 
     /// Currently selected video URLs
-    var videos: [URL] = []
+    var videos: [URL] = [] {
+        didSet {
+            didSetURLs(oldValue, videos)
+        }
+    }
 
     /// Whether any media is currently selected
     var isEmpty: Bool {
         images.isEmpty && videos.isEmpty
     }
+
+    private func didSetURLs(_ old: [URL], _ new: [URL]) {
+        // the urls we get from fileImporter require SSB calls to access
+        new.filter { !old.contains($0) }.forEach { _ = $0.startAccessingSecurityScopedResource() }
+        old.filter { !new.contains($0) }.forEach { $0.stopAccessingSecurityScopedResource() }
+    }
 }
 
 /// Options for clearing different aspects of the chat state

mlx-swift-examples.xcodeproj/project.pbxproj

@@ -191,7 +191,6 @@
 /* End PBXCopyFilesBuildPhase section */
 
 /* Begin PBXFileReference section */
-		0A7C100B2DC37CEA00FD5155 /* MLXChatExample.entitlements */ = {isa = PBXFileReference; lastKnownFileType = text.plist.entitlements; path = MLXChatExample.entitlements; sourceTree = "<group>"; };
 		0AC74EBB2D136221003C90A7 /* VLMEval.app */ = {isa = PBXFileReference; explicitFileType = wrapper.application; includeInIndex = 0; path = VLMEval.app; sourceTree = BUILT_PRODUCTS_DIR; };
 		0AC74EC92D13622A003C90A7 /* Preview Assets.xcassets */ = {isa = PBXFileReference; lastKnownFileType = folder.assetcatalog; path = "Preview Assets.xcassets"; sourceTree = "<group>"; };
 		0AC74ECB2D13622A003C90A7 /* Assets.xcassets */ = {isa = PBXFileReference; lastKnownFileType = folder.assetcatalog; path = Assets.xcassets; sourceTree = "<group>"; };
@@ -528,7 +527,6 @@
 		C39273672B60697700368D5D = {
 			isa = PBXGroup;
 			children = (
-				0A7C100B2DC37CEA00FD5155 /* MLXChatExample.entitlements */,
 				C325DE3F2B648CDB00628871 /* README.md */,
 				F8D7023A2BB4E223003D7CF5 /* Package.swift */,
 				C3C36A6C2CA714600099FFA4 /* Configuration */,
@@ -1319,7 +1317,7 @@
 				CLANG_WARN_DOCUMENTATION_COMMENTS = YES;
 				CLANG_WARN_OBJC_ROOT_CLASS = YES_ERROR;
 				CLANG_WARN_UNGUARDED_AVAILABILITY = YES_AGGRESSIVE;
-				CODE_SIGN_ENTITLEMENTS = MLXChatExample.entitlements;
+				CODE_SIGN_ENTITLEMENTS = Applications/MLXChatExample/MLXChatExample.entitlements;
 				CODE_SIGN_STYLE = Automatic;
 				COPY_PHASE_STRIP = NO;
 				CURRENT_PROJECT_VERSION = 1;
@@ -1385,7 +1383,7 @@
 				CLANG_WARN_DOCUMENTATION_COMMENTS = YES;
 				CLANG_WARN_OBJC_ROOT_CLASS = YES_ERROR;
 				CLANG_WARN_UNGUARDED_AVAILABILITY = YES_AGGRESSIVE;
-				CODE_SIGN_ENTITLEMENTS = MLXChatExample.entitlements;
+				CODE_SIGN_ENTITLEMENTS = Applications/MLXChatExample/MLXChatExample.entitlements;
 				CODE_SIGN_STYLE = Automatic;
 				COPY_PHASE_STRIP = NO;
 				CURRENT_PROJECT_VERSION = 1;