Merge pull request #5 from argentlabs/fix/argent-mobile-adapter

fix: argent mobile login return ConnectedStarknetWindowObject

src/connectors/argentMobile/modal/index.ts

@@ -1,4 +1,7 @@
-import type { StarknetWindowObject } from "get-starknet-core"
+import type {
+  StarknetWindowObject,
+  ConnectedStarknetWindowObject,
+} from "get-starknet-core"
 
 import { login } from "./login"
 import type { IArgentLoginOptions } from "./login"
@@ -8,4 +11,4 @@ export type { StarknetWindowObject, IArgentLoginOptions }
 
 export const getStarknetWindowObject = async (
   options: IArgentLoginOptions,
-): Promise<StarknetWindowObject> => login(options, StarknetAdapter)
+): Promise<ConnectedStarknetWindowObject> => login(options, StarknetAdapter)

src/connectors/argentMobile/modal/login.ts

@@ -8,6 +8,7 @@ const Network: typeof constants.NetworkName = constants.NetworkName
 
 import type { NamespaceAdapter, NamespaceAdapterOptions } from "./adapter"
 import { argentModal } from "./argentModal"
+import { resetWalletConnect } from "../../../helpers/resetWalletConnect"
 
 export interface IArgentLoginOptions {
   projectId?: string
@@ -79,6 +80,10 @@ export const login = async <TAdapter extends NamespaceAdapter>(
     }
 
     const params = { requiredNamespaces: adapter.getRequiredNamespaces() }
+
+    resetWalletConnect()
+    // wait for cookies to be removed and reset the websocket for walletconnect
+    await new Promise((resolve) => setTimeout(resolve, 200))
     const { uri, approval } = await client.connect(params)
 
     // Open QRCode modal if a URI was returned (i.e. we're not connecting an existing pairing).

src/connectors/argentMobile/modal/starknet/adapter.ts

@@ -126,8 +126,9 @@ export class StarknetAdapter
 
   // eslint-disable-next-line @typescript-eslint/ban-ts-comment
   // @ts-ignore
-  get isConnected(): boolean {
-    return this.signerConnection.connected
+  get isConnected(): true {
+    // needed to be implemented returning true since it's implementing ConnectedStarknetWindowObject
+    return true
   }
 
   async isPreauthorized(): Promise<boolean> {