Potential fixes on Reown and Metmask

Packages/io.chainsafe.web3-unity.web3auth/Runtime/Web3AuthProvider.cs

@@ -26,8 +26,9 @@ public Web3AuthProvider(
         Web3AuthWalletConfig config,
         Web3Environment environment,
         IChainConfig chainConfig,
-        IOperationTracker operationTracker)
-        : base(environment, chainConfig, operationTracker)
+        IOperationTracker operationTracker,
+        IOperatingSystemMediator operatingSystemMediator)
+        : base(environment, chainConfig, operationTracker, operatingSystemMediator)
     {
         this.operationTracker = operationTracker;
         _config = config;

Packages/io.chainsafe.web3-unity.web3auth/Runtime/Web3AuthWebGLProvider.cs

@@ -12,8 +12,8 @@ public class Web3AuthWebGLProvider : Web3AuthProvider
 {
     private readonly Web3AuthWalletConfig _config;
 
-    public Web3AuthWebGLProvider(Web3AuthWalletConfig config, Web3Environment environment, IChainConfig chainConfig, IOperationTracker operationTracker)
-        : base(config, environment, chainConfig, operationTracker)
+    public Web3AuthWebGLProvider(Web3AuthWalletConfig config, Web3Environment environment, IChainConfig chainConfig, IOperationTracker operationTracker, IOperatingSystemMediator operatingSystemMediator)
+        : base(config, environment, chainConfig, operationTracker, operatingSystemMediator)
     {
         _config = config;
     }

Packages/io.chainsafe.web3-unity.web3auth/Runtime/chainsafe.web3-unity.web3auth.asmdef

@@ -3,13 +3,15 @@
     "rootNamespace": "",
     "references": [
         "GUID:5426c6b788696eb4c88f4198b59839eb",
-        "Unity.TextMeshPro"
+        "GUID:6055be8ebefd69e48b49212b09b47b2f"
     ],
     "includePlatforms": [],
     "excludePlatforms": [],
     "allowUnsafeCode": false,
     "overrideReferences": false,
-    "precompiledReferences": [],
+    "precompiledReferences": [
+        "Nethereum.Signer.dll"
+    ],
     "autoReferenced": true,
     "defineConstraints": [],
     "versionDefines": [],

Packages/io.chainsafe.web3-unity/Runtime/Scripts/Reown/AppKit/ReownWebGLProvider.cs

@@ -40,7 +40,7 @@ public class ReownWebGLProvider : WalletProvider, ILifecycleParticipant, IConnec
 
         public ReownWebGLProvider(ReownHttpClient httpClient, IChainConfigSet chains, IReownConfig reownConfig,
             ILogWriter logWriter, Web3Environment web3Environment, IChainManager chainManager,
-            IOperationTracker operationTracker) : base(web3Environment, chainManager.Current, operationTracker)
+            IOperationTracker operationTracker, IOperatingSystemMediator operatingSystemMediator) : base(web3Environment, chainManager.Current, operationTracker, operatingSystemMediator)
         {
             _httpClient = httpClient;
             _chains = chains;

src/ChainSafe.Gaming.HyperPlay/HyperPlayProvider.cs

@@ -31,14 +31,16 @@ public class HyperPlayProvider : WalletProvider
         /// <param name="environment">Injected <see cref="environment"/>.</param>
         /// <param name="chainConfig">ChainConfig to fetch chain data.</param>
         /// <param name="operationTracker">Injected <see cref="IOperationTracker"/>.</param>
+        /// <param name="operatingSystemMediator">Injected <see cref="IOperatingSystemMediator"/>.</param>
         public HyperPlayProvider(
             IHyperPlayConfig config,
             IHyperPlayData data,
             ILocalStorage localStorage,
             Web3Environment environment,
             IChainConfig chainConfig,
-            IOperationTracker operationTracker)
-            : base(environment, chainConfig, operationTracker)
+            IOperationTracker operationTracker,
+            IOperatingSystemMediator operatingSystemMediator)
+            : base(environment, chainConfig, operationTracker, operatingSystemMediator)
         {
             this.config = config;
             this.data = data;

src/ChainSafe.Gaming.Reown/ReownProvider.cs

@@ -71,8 +71,9 @@ public ReownProvider(
             Web3Environment environment,
             ReownHttpClient reownHttpClient,
             IOperationTracker operationTracker,
-            IRpcProvider rpcProvider)
-            : base(environment, chainConfig, operationTracker)
+            IRpcProvider rpcProvider,
+            IOperatingSystemMediator operatingSystemMediator)
+            : base(environment, chainConfig, operationTracker, operatingSystemMediator)
         {
             this.operationTracker = operationTracker;
             this.rpcProvider = rpcProvider;
@@ -139,62 +140,61 @@ private async Task Initialize()
 
             ReownLogger.Instance = new ReownLogWriter(logWriter, config);
 
-            using (operationTracker.TrackOperation("Initializing the Reown module..."))
-            {
-                var storage = await ReownStorageFactory.Build(environment);
-                var signClientOptions = new SignClientOptions
-                {
-                    ProjectId = config.ProjectId,
-                    Name = config.ProjectName,
-                    Metadata = config.Metadata,
-                    BaseContext = config.BaseContext,
-                    Storage = storage,
-                    KeyChain = new KeyChain(storage),
-                    ConnectionBuilder = config.ConnectionBuilder,
-                    RelayUrlBuilder = config.RelayUrlBuilder,
-                };
 
-                SignClient = await SignClient.Init(signClientOptions);
-                await SignClient.AddressProvider.LoadDefaultsAsync();
+            var storage = await ReownStorageFactory.Build(environment);
+            var signClientOptions = new SignClientOptions
+            {
+                ProjectId = config.ProjectId,
+                Name = config.ProjectName,
+                Metadata = config.Metadata,
+                BaseContext = config.BaseContext,
+                Storage = storage,
+                KeyChain = new KeyChain(storage),
+                ConnectionBuilder = config.ConnectionBuilder,
+                RelayUrlBuilder = config.RelayUrlBuilder,
+            };
+
+            SignClient = await SignClient.Init(signClientOptions);
+            await SignClient.AddressProvider.LoadDefaultsAsync();
+
+            if (config.OnRelayErrored is not null)
+            {
+                SignClient.CoreClient.Relayer.OnErrored += config.OnRelayErrored;
+            }
 
-                if (config.OnRelayErrored is not null)
+            var optionalNamespace =
+                new ProposedNamespace // todo using optional namespaces like AppKit does, should they be required?
                 {
-                    SignClient.CoreClient.Relayer.OnErrored += config.OnRelayErrored;
-                }
-
-                var optionalNamespace =
-                    new ProposedNamespace // todo using optional namespaces like AppKit does, should they be required?
+                    Chains = chainConfigSet.Configs
+                        .Select(chainEntry => chainEntry.ChainId)
+                        .ToArray(),
+                    Methods = new[]
                     {
-                        Chains = chainConfigSet.Configs
-                            .Select(chainEntry => chainEntry.ChainId)
-                            .ToArray(),
-                        Methods = new[]
-                        {
-                            "eth_sign",
-                            "personal_sign",
-                            "eth_signTypedData",
-                            "eth_signTransaction",
-                            "eth_sendTransaction",
-                            "eth_chainId",
-                            "eth_getTransactionByHash",
-                            "wallet_switchEthereumChain",
-                            "wallet_addEthereumChain",
-                            "eth_blockNumber",
-                        },
-                        Events = new[]
-                        {
-                            "chainChanged",
-                            "accountsChanged",
-                        },
-                    };
-
-                optionalNamespaces = new Dictionary<string, ProposedNamespace>
-                {
-                    { EvmNamespace, optionalNamespace },
+                        "eth_sign",
+                        "personal_sign",
+                        "eth_signTypedData",
+                        "eth_signTransaction",
+                        "eth_sendTransaction",
+                        "eth_chainId",
+                        "eth_getTransactionByHash",
+                        "wallet_switchEthereumChain",
+                        "wallet_addEthereumChain",
+                        "eth_blockNumber",
+                    },
+                    Events = new[]
+                    {
+                        "chainChanged",
+                        "accountsChanged",
+                    },
                 };
 
-                initialized = true;
-            }
+            optionalNamespaces = new Dictionary<string, ProposedNamespace>
+            {
+                { EvmNamespace, optionalNamespace },
+            };
+
+            initialized = true;
+
         }
 
         protected override Task<string> GetWalletChainId()
@@ -314,13 +314,9 @@ private async Task<Session> ConnectSession()
         {
             ConnectedData connectedData;
             IConnectionHandler connectionHandler;
-
-            using (operationTracker.TrackOperation("Initializing a new Reown session..."))
-            {
-                var connectOptions = new ConnectOptions { OptionalNamespaces = optionalNamespaces };
-                connectedData = await SignClient.Connect(connectOptions);
-                connectionHandler = await config.ConnectionHandlerProvider.ProvideHandler();
-            }
+            var connectOptions = new ConnectOptions { OptionalNamespaces = optionalNamespaces };
+            connectedData = await SignClient.Connect(connectOptions);
+            connectionHandler = await config.ConnectionHandlerProvider.ProvideHandler();
 
             try
             {

src/ChainSafe.Gaming.Unity.MetaMask/MetaMaskProvider.cs

@@ -28,12 +28,15 @@ public class MetaMaskProvider : WalletProvider
         /// <param name="environment">Injected <see cref="Web3Environment"/>.</param>
         /// <param name="chainConfig">Injected <see cref="IChainConfig"/>.</param>
         /// <param name="chainRegistryProvider">Injected <see cref="ChainRegistryProvider"/>.</param>
+        /// <param name="operationTracker">Injected <see cref="IOperationTracker"/>.</param>
+        /// <param name="operatingSystemMediator">Injected <see cref="IOperatingSystemMediator"/></param>
         public MetaMaskProvider(
             Web3Environment environment,
             IChainConfig chainConfig,
             ChainRegistryProvider chainRegistryProvider,
-            IOperationTracker operationTracker)
-            : base(environment, chainConfig, operationTracker)
+            IOperationTracker operationTracker,
+            IOperatingSystemMediator operatingSystemMediator)
+            : base(environment, chainConfig, operationTracker, operatingSystemMediator)
         {
             logWriter = environment.LogWriter;
             this.chainConfig = chainConfig;