Fix crash when CloudKit returns records without fields

Sources/BearCLICore/Models.swift

@@ -111,6 +111,21 @@ public struct CKRecord: Decodable {
     public let modified: CKTimestamp?
     public let deleted: Bool?
 
+    private enum CodingKeys: String, CodingKey {
+        case recordName, recordType, fields, recordChangeTag, created, modified, deleted
+    }
+
+    public init(from decoder: Decoder) throws {
+        let container = try decoder.container(keyedBy: CodingKeys.self)
+        recordName = try container.decode(String.self, forKey: .recordName)
+        recordType = try container.decodeIfPresent(String.self, forKey: .recordType)
+        fields = try container.decodeIfPresent([String: CKRecordField].self, forKey: .fields) ?? [:]
+        recordChangeTag = try container.decodeIfPresent(String.self, forKey: .recordChangeTag)
+        created = try container.decodeIfPresent(CKTimestamp.self, forKey: .created)
+        modified = try container.decodeIfPresent(CKTimestamp.self, forKey: .modified)
+        deleted = try container.decodeIfPresent(Bool.self, forKey: .deleted)
+    }
+
     public init(
         recordName: String,
         recordType: String? = nil,

Tests/BearCLITests/ModelsTests.swift

@@ -200,6 +200,97 @@ final class NoteCacheTests: XCTestCase {
     }
 }
 
+final class CKRecordDecodingTests: XCTestCase {
+
+    func testDecodesRecordWithFields() throws {
+        let json = """
+        {
+            "recordName": "ABC-123",
+            "recordType": "SFNote",
+            "fields": {
+                "title": { "value": "Hello", "type": "STRING" }
+            },
+            "recordChangeTag": "tag1"
+        }
+        """.data(using: .utf8)!
+
+        let record = try JSONDecoder().decode(CKRecord.self, from: json)
+
+        XCTAssertEqual(record.recordName, "ABC-123")
+        XCTAssertEqual(record.recordType, "SFNote")
+        XCTAssertEqual(record.fields["title"]?.value.stringValue, "Hello")
+        XCTAssertEqual(record.recordChangeTag, "tag1")
+    }
+
+    func testDecodesRecordWithoutFieldsKey() throws {
+        let json = """
+        {
+            "recordName": "ABC-123",
+            "recordType": "SFNote"
+        }
+        """.data(using: .utf8)!
+
+        let record = try JSONDecoder().decode(CKRecord.self, from: json)
+
+        XCTAssertEqual(record.recordName, "ABC-123")
+        XCTAssertTrue(record.fields.isEmpty)
+    }
+
+    func testDecodesRecordWithEmptyFields() throws {
+        let json = """
+        {
+            "recordName": "ABC-123",
+            "fields": {}
+        }
+        """.data(using: .utf8)!
+
+        let record = try JSONDecoder().decode(CKRecord.self, from: json)
+
+        XCTAssertEqual(record.recordName, "ABC-123")
+        XCTAssertTrue(record.fields.isEmpty)
+    }
+
+    func testDecodesDeletedRecordWithoutFields() throws {
+        let json = """
+        {
+            "recordName": "DEL-456",
+            "deleted": true
+        }
+        """.data(using: .utf8)!
+
+        let record = try JSONDecoder().decode(CKRecord.self, from: json)
+
+        XCTAssertEqual(record.recordName, "DEL-456")
+        XCTAssertEqual(record.deleted, true)
+        XCTAssertTrue(record.fields.isEmpty)
+    }
+
+    func testDecodesLookupResponseWithMixedRecords() throws {
+        let json = """
+        {
+            "records": [
+                {
+                    "recordName": "FOUND-1",
+                    "recordType": "SFNote",
+                    "fields": {
+                        "title": { "value": "My Note", "type": "STRING" }
+                    }
+                },
+                {
+                    "recordName": "NOT-FOUND-2"
+                }
+            ]
+        }
+        """.data(using: .utf8)!
+
+        let response = try JSONDecoder().decode(CKRecordLookupResponse.self, from: json)
+
+        XCTAssertEqual(response.records.count, 2)
+        XCTAssertEqual(response.records[0].fields["title"]?.value.stringValue, "My Note")
+        XCTAssertTrue(response.records[1].fields.isEmpty)
+    }
+}
+
 final class SyncStatsTests: XCTestCase {
 
     func testSummaryEmpty() {