fix: making sessionkey wallet optional

fix: handling case where login is called primaryWallet is defined

Author: dafuga

src/kit.ts

@@ -170,12 +170,15 @@ export class SessionKit {
         // Initialize session key support if configured
         if (options.sessionKey) {
             this.sessionKeyManager = new SessionKeyManager(options.sessionKey, this.ui)
-            this.walletPlugins = [
-                ...this.walletPlugins,
-                new SessionKeyWalletPlugin({
-                    walletPlugins: this.walletPlugins,
-                }),
-            ]
+            // Only add SessionKeyWalletPlugin to wallet picker if not disabled
+            if (!options.sessionKey.disableWalletPlugin) {
+                this.walletPlugins = [
+                    ...this.walletPlugins,
+                    new SessionKeyWalletPlugin({
+                        walletPlugins: this.walletPlugins,
+                    }),
+                ]
+            }
         }
     }
 

src/sessionkey/types.ts

@@ -33,6 +33,8 @@ export interface SessionKeyConfig {
     ) => Promise<API.v1.AccountPermission | undefined>
     /** Skip showing consent UI to user during setup. Default is false. */
     skipConsent?: boolean
+    /** Disable SessionKeyWalletPlugin from appearing in the wallet picker. Default is false. */
+    disableWalletPlugin?: boolean
 }
 
 /**

src/sessionkey/wallet.ts

@@ -222,6 +222,11 @@ export class SessionKeyWalletPlugin
     }
 
     login(context: LoginContext): Promise<WalletPluginLoginResponse> {
+        if (!this.primaryWallet) {
+            throw new Error(
+                'Session key wallet is not configured. Please log in with a regular wallet first to set up a session key.'
+            )
+        }
         return this.primaryWallet.login(context)
     }