feat(auth): Make the presentation anchor optional (#2545)

* feat(auth): Make the presentation anchor optional

* Fix crash

* added test

* Revert "added test"

This reverts commit 7684e0a.

* Add test

Author: royjit

Amplify/Categories/Auth/AuthCategory+ClientBehavior.swift

@@ -31,14 +31,14 @@ extension AuthCategory: AuthCategoryBehavior {
 
 #if canImport(AuthenticationServices)
     public func signInWithWebUI(
-        presentationAnchor: AuthUIPresentationAnchor,
+        presentationAnchor: AuthUIPresentationAnchor? = nil,
         options: AuthWebUISignInRequest.Options? = nil) async throws -> AuthSignInResult {
             return try await plugin.signInWithWebUI(presentationAnchor: presentationAnchor, options: options)
         }
 
     public func signInWithWebUI(
         for authProvider: AuthProvider,
-        presentationAnchor: AuthUIPresentationAnchor,
+        presentationAnchor: AuthUIPresentationAnchor? = nil,
         options: AuthWebUISignInRequest.Options? = nil) async throws -> AuthSignInResult {
         return try await plugin.signInWithWebUI(for: authProvider,
                                       presentationAnchor: presentationAnchor,

Amplify/Categories/Auth/AuthCategoryBehavior.swift

@@ -68,7 +68,7 @@ public protocol AuthCategoryBehavior: AuthCategoryUserBehavior, AuthCategoryDevi
     /// - Parameters:
     ///   - presentationAnchor: Anchor on which the UI is presented.
     ///   - options: Parameters specific to plugin behavior.
-    func signInWithWebUI(presentationAnchor: AuthUIPresentationAnchor,
+    func signInWithWebUI(presentationAnchor: AuthUIPresentationAnchor?,
                          options: AuthWebUISignInRequest.Options?) async throws -> AuthSignInResult
 
     /// SignIn using an auth provider on a web UI
@@ -82,7 +82,7 @@ public protocol AuthCategoryBehavior: AuthCategoryUserBehavior, AuthCategoryDevi
     ///   - presentationAnchor: Anchor on which the UI is presented.
     ///   - options: Parameters specific to plugin behavior.
     func signInWithWebUI(for authProvider: AuthProvider,
-                         presentationAnchor: AuthUIPresentationAnchor,
+                         presentationAnchor: AuthUIPresentationAnchor?,
                          options: AuthWebUISignInRequest.Options?) async throws -> AuthSignInResult
 #endif
 

Amplify/Categories/Auth/Request/AuthWebUISignInRequest.swift

@@ -14,15 +14,15 @@ import Foundation
 public struct AuthWebUISignInRequest: AmplifyOperationRequest {
 
     /// Presentation anchor on which the webUI is displayed
-    public let presentationAnchor: AuthUIPresentationAnchor
+    public let presentationAnchor: AuthUIPresentationAnchor?
 
     /// Optional auth provider to directly sign in with the provider
     public let authProvider: AuthProvider?
 
     /// Extra request options defined in `AuthWebUISignInRequest.Options`
     public var options: Options
 
-    public init(presentationAnchor: AuthUIPresentationAnchor,
+    public init(presentationAnchor: AuthUIPresentationAnchor?,
                 authProvider: AuthProvider? = nil,
                 options: Options) {
         self.presentationAnchor = presentationAnchor

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Actions/SignIn/HostedUI/InitializeHostedUISignIn.swift

@@ -28,19 +28,14 @@ struct InitializeHostedUISignIn: Action {
             return
         }
 
-        guard let presentationAnchor = options.presentationAnchor else {
-            fatalError("""
-        Should not happen, initialize hostedUISignIn should always start with presentationanchor
-        """)
-        }
         await initializeHostedUI(
-            presentationAnchor: presentationAnchor,
+            presentationAnchor: options.presentationAnchor,
             environment: environment,
             hostedUIEnvironment: hostedUIEnvironment,
             dispatcher: dispatcher)
     }
 
-    func initializeHostedUI(presentationAnchor: AuthUIPresentationAnchor,
+    func initializeHostedUI(presentationAnchor: AuthUIPresentationAnchor?,
                             environment: AuthEnvironment,
                             hostedUIEnvironment: HostedUIEnvironment,
                             dispatcher: EventDispatcher) async {

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/ClientBehavior/AWSCognitoAuthPlugin+ClientBehavior.swift

@@ -7,6 +7,7 @@
 
 import Foundation
 import Amplify
+import AuthenticationServices
 
 extension AWSCognitoAuthPlugin: AuthCategoryBehavior {
 
@@ -47,7 +48,7 @@ extension AWSCognitoAuthPlugin: AuthCategoryBehavior {
     }
 
 #if canImport(AuthenticationServices)
-    public func signInWithWebUI(presentationAnchor: AuthUIPresentationAnchor,
+    public func signInWithWebUI(presentationAnchor: AuthUIPresentationAnchor? = nil,
                                 options: AuthWebUISignInRequest.Options?) async throws -> AuthSignInResult {
         let options = options ?? AuthWebUISignInRequest.Options()
         let request = AuthWebUISignInRequest(presentationAnchor: presentationAnchor,
@@ -64,7 +65,7 @@ extension AWSCognitoAuthPlugin: AuthCategoryBehavior {
     }
 
     public func signInWithWebUI(for authProvider: AuthProvider,
-                                presentationAnchor: AuthUIPresentationAnchor,
+                                presentationAnchor: AuthUIPresentationAnchor? = nil,
                                 options: AuthWebUISignInRequest.Options?) async throws -> AuthSignInResult {
         let options = options ?? AuthWebUISignInRequest.Options()
         let request = AuthWebUISignInRequest(presentationAnchor: presentationAnchor,

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/StateMachine/CodeGen/States/HostedUISignInState.swift

@@ -37,7 +37,7 @@ struct HostedUISigningInState: Equatable {
 
     let codeChallenge: String
 
-    let presentationAnchor: AuthUIPresentationAnchor
+    let presentationAnchor: AuthUIPresentationAnchor?
 
     let options: HostedUIOptions
 }