feat: WalletAccount non-breaking temp solution

src/wallet/account.ts

@@ -1,11 +1,13 @@
 import type {
-  Signature,
   AccountChangeEventHandler,
   AddStarknetChainParameters,
   NetworkChangeEventHandler,
+  Signature,
   WatchAssetParameters,
 } from 'starknet-types-07';
+
 import { Account, AccountInterface } from '../account';
+import { StarknetChainId } from '../constants';
 import { ProviderInterface } from '../provider';
 import {
   AllowArray,
@@ -34,20 +36,35 @@ import {
   watchAsset,
 } from './connect';
 import { StarknetWalletProvider } from './types';
-import { StarknetChainId } from '../constants';
 
+// TODO: Remove non address constructor in next major version
 // Represent 'Selected Active' Account inside Connected Wallet
 export class WalletAccount extends Account implements AccountInterface {
   public address: string = '';
 
   public walletProvider: StarknetWalletProvider;
 
+  /**
+   * @deprecated Use static method WalletAccount.connect or WalletAccount.connectSilent instead. Constructor {@link WalletAccount.(format:2)}.
+   */
   constructor(
     providerOrOptions: ProviderOptions | ProviderInterface,
     walletProvider: StarknetWalletProvider,
     cairoVersion?: CairoVersion
+  );
+  constructor(
+    providerOrOptions: ProviderOptions | ProviderInterface,
+    walletProvider: StarknetWalletProvider,
+    cairoVersion?: CairoVersion,
+    address?: string
+  );
+  constructor(
+    providerOrOptions: ProviderOptions | ProviderInterface,
+    walletProvider: StarknetWalletProvider,
+    cairoVersion?: CairoVersion,
+    address: string = ''
   ) {
-    super(providerOrOptions, '', '', cairoVersion); // At this point unknown address
+    super(providerOrOptions, address, '', cairoVersion); // At this point unknown address
     this.walletProvider = walletProvider;
 
     // Update Address on change
@@ -64,17 +81,14 @@ export class WalletAccount extends Account implements AccountInterface {
       this.channel.setChainId(res as StarknetChainId);
     });
 
-    // Get and Set Address !!! Post constructor initial empty string
-    walletProvider
-      .request({
-        type: 'wallet_requestAccounts',
-        params: {
-          silent_mode: false,
-        },
-      })
-      .then((res) => {
-        this.address = res[0].toLowerCase();
+    if (!address.length) {
+      console.warn(
+        '@deprecated Use static method WalletAccount.connect or WalletAccount.connectSilent instead. Constructor {@link WalletAccount.(format:2)}.'
+      );
+      requestAccounts(this.walletProvider).then(([accountAddress]) => {
+        this.address = accountAddress.toLowerCase();
       });
+    }
   }
 
   /**
@@ -170,5 +184,23 @@ export class WalletAccount extends Account implements AccountInterface {
     return signMessage(this.walletProvider, typedData);
   }
 
+  static async connect(
+    provider: ProviderInterface,
+    walletProvider: StarknetWalletProvider,
+    cairoVersion?: CairoVersion,
+    silentMode: boolean = false
+  ) {
+    const [accountAddress] = await requestAccounts(walletProvider, silentMode);
+    return new WalletAccount(provider, walletProvider, cairoVersion, accountAddress);
+  }
+
+  static async connectSilent(
+    provider: ProviderInterface,
+    walletProvider: StarknetWalletProvider,
+    cairoVersion?: CairoVersion
+  ) {
+    return WalletAccount.connect(provider, walletProvider, cairoVersion, true);
+  }
+
   // TODO: MISSING ESTIMATES
 }