removed web code

Author: fan-zhang-sv

packages/client/src/CoinbaseWalletSDK.native.test.ts

@@ -1,70 +1,56 @@
 import { CoinbaseWalletProvider } from './CoinbaseWalletProvider';
 import { CoinbaseWalletSDK } from './CoinbaseWalletSDK';
-import { ProviderInterface } from ':core/provider/interface';
-import { getFavicon } from ':core/type/util';
-import { getCoinbaseInjectedProvider } from ':util/provider';
+import { MOBILE_SDK_RESPONSE_PATH } from ':core/constants';
 
-jest.mock(':core/type/util');
-jest.mock(':util/provider');
 jest.mock('./CoinbaseWalletProvider');
+jest.mock(':core/storage/ScopedAsyncStorage');
 
 describe('CoinbaseWalletSDK', () => {
-  test('@makeWeb3Provider - return Coinbase Injected Provider', () => {
-    const injectedProvider = {} as unknown as ProviderInterface;
-    (getCoinbaseInjectedProvider as jest.Mock).mockReturnValue(injectedProvider);
-
-    const SDK = new CoinbaseWalletSDK({
-      appName: 'Test',
-      appLogoUrl: 'http://coinbase.com/wallet-logo.png',
-    });
-
-    expect(SDK.makeWeb3Provider()).toBe(injectedProvider);
+  const mockAppDeeplinkUrl = 'https://example.com';
+  const mockMetadata = {
+    appName: 'TestDapp',
+    appLogoUrl: 'https://example.com/logo.png',
+    appChainIds: [1, 42],
+    appDeeplinkUrl: mockAppDeeplinkUrl,
+  };
+
+  beforeEach(() => {
+    jest.clearAllMocks();
   });
 
-  test('@makeWeb3Provider - return new CoinbaseWalletProvider', () => {
-    (getCoinbaseInjectedProvider as jest.Mock).mockReturnValue(undefined);
-
-    const SDK = new CoinbaseWalletSDK({
-      appName: 'Test',
-      appLogoUrl: 'http://coinbase.com/wallet-logo.png',
+  describe('constructor', () => {
+    it('should throw an error if appDeeplinkUrl is not provided', () => {
+      expect(() => new CoinbaseWalletSDK({})).toThrow('appDeeplinkUrl is required on Mobile');
     });
 
-    SDK.makeWeb3Provider();
-
-    expect(CoinbaseWalletProvider).toHaveBeenCalledWith({
-      metadata: {
-        appName: 'Test',
-        appLogoUrl: 'http://coinbase.com/wallet-logo.png',
+    it('should initialize metadata with default values', () => {
+      const sdk = new CoinbaseWalletSDK({ appDeeplinkUrl: mockAppDeeplinkUrl });
+      expect(sdk['metadata']).toEqual({
+        appName: 'Dapp',
+        appLogoUrl: null,
         appChainIds: [],
-        appDeeplinkUrl: null,
-      },
-      preference: {
-        options: 'all',
-      },
+        appDeeplinkUrl: `${mockAppDeeplinkUrl}/${MOBILE_SDK_RESPONSE_PATH}`,
+      });
     });
-  });
-
-  test('@makeWeb3Provider - default values for metadata', () => {
-    (getFavicon as jest.Mock).mockReturnValue('https://dapp.xyz/pic.png');
-    (getCoinbaseInjectedProvider as jest.Mock).mockReturnValue(undefined);
 
-    const SDK = new CoinbaseWalletSDK({
-      appName: '',
-      appLogoUrl: '',
+    it('should initialize metadata with provided values', () => {
+      const sdk = new CoinbaseWalletSDK(mockMetadata);
+      expect(sdk['metadata']).toEqual({
+        ...mockMetadata,
+        appDeeplinkUrl: `${mockAppDeeplinkUrl}/${MOBILE_SDK_RESPONSE_PATH}`,
+      });
     });
+  });
 
-    SDK.makeWeb3Provider();
-
-    expect(CoinbaseWalletProvider).toHaveBeenCalledWith({
-      metadata: {
-        appName: 'Dapp',
-        appLogoUrl: 'https://dapp.xyz/pic.png',
-        appChainIds: [],
-        appDeeplinkUrl: null,
-      },
-      preference: {
-        options: 'all',
-      },
+  describe('makeWeb3Provider', () => {
+    it('should return a new CoinbaseWalletProvider instance', () => {
+      const sdk = new CoinbaseWalletSDK(mockMetadata);
+      const provider = sdk.makeWeb3Provider();
+      expect(provider).toBeInstanceOf(CoinbaseWalletProvider);
+      expect(CoinbaseWalletProvider).toHaveBeenCalledWith({
+        metadata: sdk['metadata'],
+        preference: { options: 'smartWalletOnly' },
+      });
     });
   });
 });

packages/client/src/CoinbaseWalletSDK.native.ts

@@ -2,31 +2,40 @@
 
 import { LogoType, walletLogo } from './assets/wallet-logo';
 import { CoinbaseWalletProvider } from './CoinbaseWalletProvider';
-import { AppMetadata, Preference, ProviderInterface } from './core/provider/interface';
+import { AppMetadata, ProviderInterface } from './core/provider/interface';
 import { LIB_VERSION } from './version';
+import { MOBILE_SDK_RESPONSE_PATH } from ':core/constants';
 import { ScopedAsyncStorage } from ':core/storage/ScopedAsyncStorage';
-import { getFavicon } from ':core/type/util';
-import { getCoinbaseInjectedProvider } from ':util/provider';
 
-// for backwards compatibility
 type CoinbaseWalletSDKOptions = Partial<AppMetadata>;
 
 export class CoinbaseWalletSDK {
   private metadata: AppMetadata;
 
   constructor(metadata: Readonly<CoinbaseWalletSDKOptions>) {
+    if (!metadata.appDeeplinkUrl) {
+      throw new Error('appDeeplinkUrl is required on Mobile');
+    }
+
+    const url = new URL(metadata.appDeeplinkUrl);
+    url.pathname += url.pathname.endsWith('/')
+      ? MOBILE_SDK_RESPONSE_PATH
+      : `/${MOBILE_SDK_RESPONSE_PATH}`;
+
     this.metadata = {
       appName: metadata.appName || 'Dapp',
-      appLogoUrl: metadata.appLogoUrl || getFavicon(),
+      appLogoUrl: metadata.appLogoUrl || null,
       appChainIds: metadata.appChainIds || [],
-      appDeeplinkUrl: null,
+      appDeeplinkUrl: url.toString(),
     };
     this.storeLatestVersion();
   }
 
-  public makeWeb3Provider(preference: Preference = { options: 'all' }): ProviderInterface {
-    const params = { metadata: this.metadata, preference };
-    return getCoinbaseInjectedProvider(params) ?? new CoinbaseWalletProvider(params);
+  public makeWeb3Provider(): ProviderInterface {
+    return new CoinbaseWalletProvider({
+      metadata: this.metadata,
+      preference: { options: 'smartWalletOnly' },
+    });
   }
 
   /**