fix(amplify-codegen): iOS use default when PK.OptionallyManagedId (#469)

Author: lawmicha

packages/appsync-modelgen-plugin/src/__tests__/visitors/__snapshots__/appsync-swift-visitor.test.ts.snap

@@ -417,14 +417,8 @@ extension ModelExplicitDefaultPk {
 }
 
 extension ModelExplicitDefaultPk: ModelIdentifiable {
-  public typealias IdentifierFormat = ModelIdentifierFormat.Custom
-  public typealias IdentifierProtocol = ModelIdentifier<Self, ModelIdentifierFormat.Custom>
-}
-
-extension ModelExplicitDefaultPk.IdentifierProtocol {
-  public static func identifier(id: String) -> Self {
-    .make(fields:[(name: \\"id\\", value: id)])
-  }
+  public typealias IdentifierFormat = ModelIdentifierFormat.Default
+  public typealias IdentifierProtocol = DefaultModelIdentifier<Self>
 }"
 `;
 

packages/appsync-modelgen-plugin/src/visitors/appsync-swift-visitor.ts

@@ -320,7 +320,8 @@ export class AppSyncSwiftVisitor<
     let result: string[] = [];
     const primaryKeyField = model.fields.find(field => field.primaryKeyInfo)!;
     const { primaryKeyType, sortKeyFields } = primaryKeyField.primaryKeyInfo!;
-    const useDefaultExplicitID = primaryKeyType === CodeGenPrimaryKeyType.ManagedId;
+    const useDefaultExplicitID =
+      primaryKeyType === CodeGenPrimaryKeyType.ManagedId || primaryKeyType === CodeGenPrimaryKeyType.OptionallyManagedId;
 
     const identifiableExtension = new SwiftDeclarationBlock()
       .asKind('extension')