@craft-native/ios 0.0.243 → 0.0.244

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@craft-native/ios",
-  "version": "0.0.243",
+  "version": "0.0.244",
   "homepage": "https://github.com/craft-native/craft/tree/main/packages/ios#readme",
   "repository": {
     "type": "git",

package/templates/CraftApp.swift

@@ -670,6 +670,7 @@ final class CraftChrome: ObservableObject {
 
     @Published var tabs: [Tab] = []
     @Published var tabAccessory: Tab?
+    private weak var tabAccessoryOwner: AnyObject?
     @Published var selected: String?
     let tabSwipeProgress = CraftTabSwipeProgress()
     private(set) var tabSwipePosition: CGFloat? {
@@ -771,7 +772,7 @@ final class CraftChrome: ObservableObject {
         tabs[index] = next
     }
     /// A destination beside the tab pill, outside the tab stacks and swipe order.
-    func updateTabAccessory(_ item: [String: Any]?) {
+    func updateTabAccessory(_ item: [String: Any]?, owner: AnyObject? = nil) {
         let next: Tab?
         if let item, let path = item["path"] as? String,
            path.hasPrefix("/"), !path.hasPrefix("//"),
@@ -782,10 +783,17 @@ final class CraftChrome: ObservableObject {
         } else {
             next = nil
         }
+        tabAccessoryOwner = owner
         guard tabAccessory != next else { return }
         tabAccessory = next
     }
 
+    /// A retained or prepared screen cannot clear another screen's chrome.
+    func clearTabAccessory(ownedBy owner: AnyObject) {
+        guard tabAccessoryOwner === owner else { return }
+        updateTabAccessory(nil)
+    }
+
     func setHiddenByScreen(_ hidden: Bool) {
         guard hiddenByScreen != hidden else { return }
         hiddenByScreen = hidden

package/templates/CraftNativeMutation.swift

@@ -29,6 +29,7 @@ struct CraftNativeMutationResult {
 final class CraftNativeMutationDocument {
     private struct Node {
         let id: String
+        let key: String?
         var type: String
         var props: [String: Any]
         var style: [String: Any]
@@ -177,6 +178,7 @@ final class CraftNativeMutationDocument {
         }
         nodes[id] = Node(
             id: id,
+            key: value["key"] as? String,
             type: type,
             props: dictionary(value["props"]),
             style: dictionary(value["style"]),
@@ -330,6 +332,7 @@ final class CraftNativeMutationDocument {
             "style": node.style,
             "events": node.events,
         ]
+        if let key = node.key { value["key"] = key }
         let text: [Any] = node.text
         let children: [Any] = node.children.compactMap { materialize($0, nodes: nodes) }
         value["children"] = text + children
@@ -350,6 +353,7 @@ final class CraftNativeMutationDocument {
         let children = value["children"] as? [Any] ?? []
         nodes[id] = Node(
             id: id,
+            key: value["key"] as? String,
             type: value["type"] as? String ?? "View",
             props: props,
             style: dictionary(value["style"]),

package/templates/CraftNativeScreen.swift

@@ -1435,21 +1435,47 @@ private final class CraftNativeMapView: MKMapView, MKMapViewDelegate {
 
     private func configureMarkerView(_ view: MKAnnotationView, id: String) {
         if let bearing = markerBearings[id] {
-            let configuration = UIImage.SymbolConfiguration(pointSize: 16, weight: .semibold)
-                .applying(UIImage.SymbolConfiguration(paletteColors: [.white, .systemGreen]))
-            // MapKit otherwise tints this as a monochrome template and loses
-            // the contrasting route arrow when displayed over map content.
-            view.image = UIImage(systemName: "arrow.up.circle.fill", withConfiguration: configuration)?
-                .withRenderingMode(.alwaysOriginal)
+            let color = (markerPinColors[id] ?? .systemGreen).resolvedColor(with: traitCollection)
+            view.tintColor = color
+            view.image = Self.markerImage(color: color, direction: true)
             view.transform = CGAffineTransform(rotationAngle: CGFloat((bearing - camera.heading) * .pi / 180))
         } else {
-            view.tintColor = markerPinColors[id] ?? .systemRed
-            view.image = UIImage(systemName: "mappin.circle.fill", withConfiguration: UIImage.SymbolConfiguration(pointSize: 28))?
-                .withTintColor(view.tintColor, renderingMode: .alwaysOriginal)
+            let color = (markerPinColors[id] ?? .systemRed).resolvedColor(with: traitCollection)
+            view.tintColor = color
+            view.image = Self.markerImage(color: color, direction: false)
             view.transform = .identity
         }
     }
 
+    private static let markerImages: NSCache<NSString, UIImage> = {
+        let cache = NSCache<NSString, UIImage>()
+        cache.countLimit = 64
+        return cache
+    }()
+
+    private static func markerImage(color: UIColor, direction: Bool) -> UIImage {
+        let key = "\(direction ? "direction" : "pin"):\(color.description)" as NSString
+        if let cached = markerImages.object(forKey: key) { return cached }
+        let size = direction ? CGSize(width: 20, height: 24) : CGSize(width: 28, height: 28)
+        // Explicit native drawing preserves the light backing independently
+        // of MapKit's tint and SF Symbols' multicolor rendering preferences.
+        let image = UIGraphicsImageRenderer(size: size).image { _ in
+            let border = UIBezierPath(roundedRect: CGRect(origin: .zero, size: size).insetBy(dx: 1, dy: 1),
+                                      cornerRadius: (size.width - 2) / 2)
+            UIColor.white.setFill()
+            border.fill()
+            color.setStroke()
+            border.lineWidth = 1.5
+            border.stroke()
+            let symbol = UIImage(systemName: direction ? "arrow.up" : "mappin",
+                withConfiguration: UIImage.SymbolConfiguration(pointSize: 16, weight: .semibold))?
+                .withTintColor(color, renderingMode: .alwaysOriginal)
+            symbol?.draw(in: direction ? CGRect(x: 5, y: 6, width: 10, height: 12) : CGRect(x: 7, y: 5, width: 14, height: 18))
+        }.withRenderingMode(.alwaysOriginal)
+        markerImages.setObject(image, forKey: key)
+        return image
+    }
+
     override func layoutSubviews() {
         super.layoutSubviews()
         if tileTemplate != nil {
@@ -4621,10 +4647,9 @@ final class CraftNativeScreenController: UIViewController, UIScrollViewDelegate,
         super.viewWillDisappear(animated)
         screenVisibilityChanged(false)
         stopCelebrations(in: view)
-        // The accessory belongs to the visible native screen. Clear the
-        // shared shell state before a hybrid/web route appears, otherwise a
-        // destination from the outgoing screen remains tappable on top of it.
-        CraftChrome.shared.updateTabAccessory(nil)
+        // Release only this screen's accessory. A retained or prepared screen
+        // can disappear after the destination has already published its own.
+        CraftChrome.shared.clearTabAccessory(ownedBy: self)
         // Popped (or its stack dismissed) while the bar was away: it comes
         // back for whatever shows next, a web page included.
         if tabBarHidden, isMovingFromParent || isBeingDismissed || navigationController?.isBeingDismissed == true {
@@ -4669,7 +4694,7 @@ final class CraftNativeScreenController: UIViewController, UIScrollViewDelegate,
     private func restoreShellChrome() {
         guard !isAppScript else { return }
         CraftChrome.shared.setHiddenByScreen(tabBarHidden)
-        CraftChrome.shared.updateTabAccessory(tabAccessory)
+        CraftChrome.shared.updateTabAccessory(tabAccessory, owner: self)
     }
 
     /// The screen came on screen or went off it: a screen pushed over it was
@@ -5340,7 +5365,7 @@ final class CraftNativeScreenController: UIViewController, UIScrollViewDelegate,
         if options.keys.contains("tabAccessory") {
             tabAccessory = options["tabAccessory"] as? [String: Any]
             if navigationController?.topViewController === self, view.window != nil {
-                CraftChrome.shared.updateTabAccessory(tabAccessory)
+                CraftChrome.shared.updateTabAccessory(tabAccessory, owner: self)
             }
         }
         if options.keys.contains("title") {
@@ -6865,7 +6890,11 @@ final class CraftNativeScreenController: UIViewController, UIScrollViewDelegate,
 
     private func explicitKey(_ node: [String: Any], props: [String: Any]) -> String? {
         if let authored = authoredKey(node, props: props) { return authored }
-        return (node["id"] as? String).flatMap { $0.isEmpty ? nil : $0 }
+        // Repeated test selectors are valid for list rows. Retain each row by
+        // its unique protocol ID before considering legacy selector identity.
+        return [node["id"], props["testID"]]
+            .compactMap { $0 as? String }
+            .first { !$0.isEmpty }
     }
 
     /// The key a node's author gave it. A wrapper of one child (a
@@ -6877,7 +6906,7 @@ final class CraftNativeScreenController: UIViewController, UIScrollViewDelegate,
     private func authoredKey(_ node: [String: Any], props: [String: Any]) -> String? {
         // The current @stacksjs/stx native compiler emits key in props; accept the IR
         // field too so keyed children keep working when the compiler adopts it.
-        let direct = [node["key"], props["key"], props["testID"]]
+        let direct = [node["key"], props["key"]]
             .compactMap { $0 as? String }
             .first { !$0.isEmpty }
         if let direct { return direct }

package/templates/testing/DriverTests.swift.template

@@ -448,7 +448,11 @@ final class DriverTests: XCTestCase {
             var found = element(containing: arg)
             var swipes = 0
             while (found == nil || !(found!.isHittable)) && swipes < 8 {
-                app.swipeUp()
+                if let target = found, !target.frame.isEmpty, target.frame.midY < app.frame.midY {
+                    app.swipeDown()
+                } else {
+                    app.swipeUp()
+                }
                 swipes += 1
                 found = element(containing: arg)
             }