chore: lint fix

MetaMask · Jan 23, 2025 · 5e4b506 · 5e4b506
1 parent 4a9a8a2
commit 5e4b506
Show file tree

Hide file tree

Showing 17 changed files with 110 additions and 97 deletions.
diff --git a/app/components/UI/AccountApproval/styles.ts b/app/components/UI/AccountApproval/styles.ts
@@ -1,7 +1,6 @@
 import { StyleSheet } from 'react-native';
 import Device from '../../../util/device';
-import type { ThemeColors } from '@metamask/design-tokens';
-import type { ThemeTypography } from '@metamask/design-tokens';
+import type { ThemeColors, ThemeTypography } from '@metamask/design-tokens';
 
 const createStyles = (colors: ThemeColors, typography: ThemeTypography) =>
   StyleSheet.create({

diff --git a/app/components/UI/Notification/Modal/styles.ts b/app/components/UI/Notification/Modal/styles.ts
@@ -1,7 +1,6 @@
 // Third party dependencies.
 import { StyleSheet, TextStyle } from 'react-native';
-import { typography } from '@metamask/design-tokens';
-import { ThemeColors } from '@metamask/design-tokens';
+import { typography, ThemeColors } from '@metamask/design-tokens';
 
 /**
  * Style sheet function for AmbiguousAddressSheet component.

diff --git a/app/components/UI/Ramp/components/PaymentMethodIcon.tsx b/app/components/UI/Ramp/components/PaymentMethodIcon.tsx
@@ -85,12 +85,18 @@ function getIconComponent(icon: PaymentIcon) {
 }
 
 /*
-* With the integration of Expo, it introduced a compatibility layer (https://github.com/expo/vector-icons) 
-* around react-native-vector-icons which doesn't expose hasIcon anymore so we need to build our own based on
-* the implementation https://github.com/oblador/react-native-vector-icons/blob/master/lib/create-icon-set.js#L158
-*/
-function hasIcon(IconComponent: { glyphMap: { [key: string]: number } }, name: string) {
-  return IconComponent?.glyphMap && Object.prototype.hasOwnProperty.call(IconComponent.glyphMap, name);
+ * With the integration of Expo, it introduced a compatibility layer (https://github.com/expo/vector-icons)
+ * around react-native-vector-icons which doesn't expose hasIcon anymore so we need to build our own based on
+ * the implementation https://github.com/oblador/react-native-vector-icons/blob/master/lib/create-icon-set.js#L158
+ */
+function hasIcon(
+  IconComponent: { glyphMap: { [key: string]: number } },
+  name: string,
+) {
+  return (
+    IconComponent?.glyphMap &&
+    Object.prototype.hasOwnProperty.call(IconComponent.glyphMap, name)
+  );
 }
 
 function getIcon(icon: PaymentIcon) {

diff --git a/app/components/UI/SDKFeedback/index.tsx b/app/components/UI/SDKFeedback/index.tsx
@@ -1,5 +1,4 @@
-import type { ThemeColors } from '@metamask/design-tokens';
-import type { ThemeTypography } from '@metamask/design-tokens';
+import type { ThemeColors, ThemeTypography } from '@metamask/design-tokens';
 import Icon, {
   IconName,
   IconSize,

diff --git a/app/components/Views/SDK/SDKDisconnectModal/SDKDisconnectModal.tsx b/app/components/Views/SDK/SDKDisconnectModal/SDKDisconnectModal.tsx
@@ -2,8 +2,7 @@
 import React, { useMemo, useRef } from 'react';
 
 // External dependencies
-import type { ThemeColors } from '@metamask/design-tokens';
-import type { ThemeTypography } from '@metamask/design-tokens';
+import type { ThemeColors, ThemeTypography } from '@metamask/design-tokens';
 import { useNavigation } from '@react-navigation/native';
 import { StyleSheet, View } from 'react-native';
 import { EdgeInsets, useSafeAreaInsets } from 'react-native-safe-area-context';

diff --git a/app/components/Views/SDK/SDKSessionModal/SDKSessionModal.tsx b/app/components/Views/SDK/SDKSessionModal/SDKSessionModal.tsx
@@ -2,8 +2,7 @@
 import React, { useEffect, useMemo, useRef, useState } from 'react';
 
 // External dependencies
-import type { ThemeColors } from '@metamask/design-tokens';
-import type { ThemeTypography } from '@metamask/design-tokens';
+import type { ThemeColors, ThemeTypography } from '@metamask/design-tokens';
 import { useNavigation } from '@react-navigation/native';
 import { StyleSheet, View } from 'react-native';
 import { EdgeInsets, useSafeAreaInsets } from 'react-native-safe-area-context';

diff --git a/app/components/Views/SDK/SDKSessionsManager/SDKSessionAccountListItem.tsx b/app/components/Views/SDK/SDKSessionsManager/SDKSessionAccountListItem.tsx
@@ -1,5 +1,4 @@
-import type { ThemeColors } from '@metamask/design-tokens';
-import type { ThemeTypography } from '@metamask/design-tokens';
+import type { ThemeColors, ThemeTypography } from '@metamask/design-tokens';
 import React, { useEffect, useState } from 'react';
 import { StyleSheet, View } from 'react-native';
 import { EdgeInsets, useSafeAreaInsets } from 'react-native-safe-area-context';

diff --git a/app/components/Views/SDK/SDKSessionsManager/SDKSessionItem.tsx b/app/components/Views/SDK/SDKSessionsManager/SDKSessionItem.tsx
@@ -1,5 +1,4 @@
-import type { ThemeColors } from '@metamask/design-tokens';
-import type { ThemeTypography } from '@metamask/design-tokens';
+import type { ThemeColors, ThemeTypography } from '@metamask/design-tokens';
 import { useNavigation } from '@react-navigation/native';
 import React, { useCallback, useEffect, useState } from 'react';
 import { StyleSheet, View } from 'react-native';

diff --git a/app/components/Views/SDK/SDKSessionsManager/SDKSessionsManager.tsx b/app/components/Views/SDK/SDKSessionsManager/SDKSessionsManager.tsx
@@ -5,8 +5,7 @@ import { EdgeInsets, useSafeAreaInsets } from 'react-native-safe-area-context';
 import { strings } from '../../../../../locales/i18n';
 import { useTheme } from '../../../../util/theme';
 
-import type { ThemeColors } from '@metamask/design-tokens';
-import type { ThemeTypography } from '@metamask/design-tokens';
+import type { ThemeColors, ThemeTypography } from '@metamask/design-tokens';
 import { RouteProp, useNavigation, useRoute } from '@react-navigation/native';
 import { useSelector } from 'react-redux';
 import Button, {

diff --git a/app/components/Views/SDK/SDKSessionsManager/getSharedStyles.ts b/app/components/Views/SDK/SDKSessionsManager/getSharedStyles.ts
@@ -1,5 +1,4 @@
-import type { ThemeColors } from '@metamask/design-tokens';
-import type { ThemeTypography } from '@metamask/design-tokens';
+import type { ThemeColors, ThemeTypography } from '@metamask/design-tokens';
 import { TextStyle } from 'react-native';
 import type { EdgeInsets } from 'react-native-safe-area-context';
 

diff --git a/app/components/Views/Settings/Contacts/AmbiguousAddressSheet/AmbiguousAddressSheet.styles.ts b/app/components/Views/Settings/Contacts/AmbiguousAddressSheet/AmbiguousAddressSheet.styles.ts
@@ -1,7 +1,7 @@
 // Third party dependencies.
 import { StyleSheet, TextStyle } from 'react-native';
-import { typography } from '@metamask/design-tokens';
-import type { ThemeColors } from '@metamask/design-tokens';
+import { typography, ThemeColors } from '@metamask/design-tokens';
+
 /**
  * Style sheet function for AmbiguousAddressSheet component.
  *

diff --git a/app/components/Views/Settings/PermissionsSettings/PermissionsManager.tsx b/app/components/Views/Settings/PermissionsSettings/PermissionsManager.tsx
@@ -7,8 +7,7 @@ import { EdgeInsets, useSafeAreaInsets } from 'react-native-safe-area-context';
 import { strings } from '../../../../../locales/i18n';
 import { useTheme } from '../../../../util/theme';
 
-import type { ThemeColors } from '@metamask/design-tokens';
-import type { ThemeTypography } from '@metamask/design-tokens';
+import type { ThemeColors, ThemeTypography } from '@metamask/design-tokens';
 import { SDKSelectorsIDs } from '../../../../../e2e/selectors/Settings/SDK.selectors';
 import Icon, {
   IconName,

diff --git a/app/core/NotificationsManager.test.ts b/app/core/NotificationsManager.test.ts
@@ -123,33 +123,37 @@ describe('NotificationManager', () => {
       });
 
       expect(title).toBe(strings('notifications.default_message_title'));
-      expect(message).toBe(strings('notifications.default_message_description'));
+      expect(message).toBe(
+        strings('notifications.default_message_description'),
+      );
     });
   });
-  
+
   describe('smartTransactionListener', () => {
     const mockTransactionController = {
       getTransactions: jest.fn(),
       state: {
-        transactions: [{
-          id: '0x123',
-          txParams: {
-            nonce: '0x1',
-            from: '0x123'
+        transactions: [
+          {
+            id: '0x123',
+            txParams: {
+              nonce: '0x1',
+              from: '0x123',
+            },
+            chainId: '0x1',
+            time: 123,
+            status: 'failed' as TransactionMeta['status'],
+            error: { message: 'test error', rpc: { code: 0 }, name: 'Error' },
           },
-          chainId: '0x1',
-          time: 123,
-          status: 'failed' as TransactionMeta['status'],
-          error: { message: 'test error', rpc: { code: 0 }, name: 'Error' }
-        }]
-      }
+        ],
+      },
     };
 
     const mockControllerMessenger = {
       subscribe: jest.fn(),
       unsubscribe: jest.fn(),
       subscribeOnceIf: jest.fn(),
-      tryUnsubscribe: jest.fn()
+      tryUnsubscribe: jest.fn(),
     };
 
     let showNotificationSpy: jest.SpyInstance;
@@ -160,19 +164,19 @@ describe('NotificationManager', () => {
         value: {
           TransactionController: mockTransactionController,
         },
-        writable: true
+        writable: true,
       });
 
       Object.defineProperty(Engine, 'controllerMessenger', {
         value: mockControllerMessenger,
-        writable: true
+        writable: true,
       });
     });
 
     beforeEach(() => {
       // Clear all mock interactions before each test
       jest.clearAllMocks();
-      
+
       // Reset the notification manager before each test
       notificationManager = NotificationManager.init({
         navigation: mockNavigation,
@@ -183,7 +187,10 @@ describe('NotificationManager', () => {
       });
 
       // Create spy on the instance method
-      showNotificationSpy = jest.spyOn(notificationManager, '_showNotification');
+      showNotificationSpy = jest.spyOn(
+        notificationManager,
+        '_showNotification',
+      );
     });
 
     afterAll(() => {
@@ -197,16 +204,17 @@ describe('NotificationManager', () => {
         transactionId: '0x123',
       };
 
-      notificationManager.watchSubmittedTransaction({ 
+      notificationManager.watchSubmittedTransaction({
         id: '0x123',
         txParams: {
-          nonce: '0x1'
+          nonce: '0x1',
         },
-        silent: false
+        silent: false,
       });
-      
+
       // Get the subscriber callback
-      const subscriberCallback = mockControllerMessenger.subscribe.mock.calls[0][1];
+      const subscriberCallback =
+        mockControllerMessenger.subscribe.mock.calls[0][1];
       await subscriberCallback(transaction);
 
       expect(showNotificationSpy).toHaveBeenCalledWith({
@@ -217,29 +225,32 @@ describe('NotificationManager', () => {
     });
 
     it('shows a cancelled notification for cancelled smart transactions', async () => {
-      const mockTransaction = { 
+      const mockTransaction = {
         id: '0x123',
         txParams: {
-          nonce: '0x1'
-        }
+          nonce: '0x1',
+        },
       };
-      mockTransactionController.getTransactions.mockReturnValue([mockTransaction]);
+      mockTransactionController.getTransactions.mockReturnValue([
+        mockTransaction,
+      ]);
 
       const smartTransaction = {
         status: SmartTransactionStatuses.CANCELLED,
         transactionId: '0x123',
       };
 
-      notificationManager.watchSubmittedTransaction({ 
+      notificationManager.watchSubmittedTransaction({
         id: '0x123',
         txParams: {
-          nonce: '0x1'
+          nonce: '0x1',
         },
-        silent: false
+        silent: false,
       });
-      
+
       // Get the subscriber callback
-      const subscriberCallback = mockControllerMessenger.subscribe.mock.calls[0][1];
+      const subscriberCallback =
+        mockControllerMessenger.subscribe.mock.calls[0][1];
       await subscriberCallback(smartTransaction);
 
       expect(showNotificationSpy).toHaveBeenCalledWith({
@@ -256,31 +267,32 @@ describe('NotificationManager', () => {
         transactionId: '0x123',
       };
 
-      notificationManager.watchSubmittedTransaction({ 
+      notificationManager.watchSubmittedTransaction({
         id: '0x123',
         txParams: {
-          nonce: '0x1'
+          nonce: '0x1',
         },
-        silent: false
+        silent: false,
       });
-      
+
       // Get the subscriber callback
-      const subscriberCallback = mockControllerMessenger.subscribe.mock.calls[0][1];
+      const subscriberCallback =
+        mockControllerMessenger.subscribe.mock.calls[0][1];
       await subscriberCallback(transaction);
 
       expect(mockControllerMessenger.unsubscribe).toHaveBeenCalledWith(
         'SmartTransactionsController:smartTransaction',
-        subscriberCallback
+        subscriberCallback,
       );
     });
 
     it('sets up transaction event listeners correctly', () => {
-      const transaction = { 
+      const transaction = {
         id: '0x123',
         txParams: {
-          nonce: '0x1'
+          nonce: '0x1',
         },
-        silent: false
+        silent: false,
       };
 
       notificationManager.watchSubmittedTransaction(transaction);
@@ -289,17 +301,17 @@ describe('NotificationManager', () => {
       expect(mockControllerMessenger.subscribeOnceIf).toHaveBeenCalledWith(
         'TransactionController:transactionConfirmed',
         expect.any(Function),
-        expect.any(Function)
+        expect.any(Function),
       );
       expect(mockControllerMessenger.subscribeOnceIf).toHaveBeenCalledWith(
         'TransactionController:transactionFailed',
         expect.any(Function),
-        expect.any(Function)
+        expect.any(Function),
       );
       expect(mockControllerMessenger.subscribeOnceIf).toHaveBeenCalledWith(
         'TransactionController:speedupTransactionAdded',
         expect.any(Function),
-        expect.any(Function)
+        expect.any(Function),
       );
     });
   });

diff --git a/app/core/SDKConnect/utils/isUUID.ts b/app/core/SDKConnect/utils/isUUID.ts
@@ -2,4 +2,4 @@ export const isUUID = (str: string) => {
   const uuidRegex =
     /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
   return uuidRegex.test(str);
-};
+};
diff --git a/app/util/notifications/services/NotificationService.ts b/app/util/notifications/services/NotificationService.ts
@@ -6,7 +6,12 @@ import notifee, {
   AndroidChannel,
 } from '@notifee/react-native';
 
-import { HandleNotificationCallback, LAUNCH_ACTIVITY, Notification, PressActionId } from '../types';
+import {
+  HandleNotificationCallback,
+  LAUNCH_ACTIVITY,
+  Notification,
+  PressActionId,
+} from '../types';
 
 import { Linking, Platform, Alert as NativeAlert } from 'react-native';
 import {
@@ -222,11 +227,11 @@ class NotificationsService {
   };
 
   getInitialNotification = async (
-    callback: HandleNotificationCallback
+    callback: HandleNotificationCallback,
   ): Promise<void> => {
-    const event = await notifee.getInitialNotification()
+    const event = await notifee.getInitialNotification();
     if (event) {
-      callback(event.notification.data as Notification['data'])
+      callback(event.notification.data as Notification['data']);
     }
   };
 
@@ -241,12 +246,12 @@ class NotificationsService {
     channelId,
     title,
     body,
-    data
+    data,
   }: {
-    channelId: ChannelId
-    title: string
-    body?: string
-    data?: Notification['data']
+    channelId: ChannelId;
+    title: string;
+    body?: string;
+    data?: Notification['data'];
   }): Promise<void> => {
     await notifee.displayNotification({
       title,
@@ -259,7 +264,7 @@ class NotificationsService {
         pressAction: {
           id: PressActionId.OPEN_NOTIFICATIONS_VIEW,
           launchActivity: LAUNCH_ACTIVITY,
-        }
+        },
       },
       ios: {
         launchImageName: 'Default',