Fix @typescript-eslint/no-misused-promises

Author: peachbits

eslint.config.mjs

@@ -542,7 +542,6 @@ export default [
       '@typescript-eslint/ban-ts-comment': 'warn',
       '@typescript-eslint/explicit-function-return-type': 'warn',
       '@typescript-eslint/no-dynamic-delete': 'warn',
-      '@typescript-eslint/no-misused-promises': 'warn',
       '@typescript-eslint/strict-boolean-expressions': 'warn',
       '@typescript-eslint/use-unknown-in-catch-callback-variable': 'warn'
     }

src/actions/CountryListActions.tsx

@@ -28,22 +28,26 @@ export const checkAndSetRegion = (props: {
   countryCode: string
   stateProvinceCode?: string
 }): ThunkAction<void> => {
-  return async (dispatch, getState) => {
+  return (dispatch, getState) => {
     const { countryCode, stateProvinceCode } = props
 
     if (countryCode == null || countryCode === '') {
       // If no countryCode, always show country selection
-      await dispatch(showCountrySelectionModal(props))
+      dispatch(showCountrySelectionModal(props)).catch((error: unknown) => {
+        showError(error)
+      })
     } else {
       // Show state province selection if stateProvinceCode is required according
       // to the country data
       const countryData = COUNTRY_CODES.find(
         cc => cc['alpha-2'] === countryCode
       )
       if (countryData != null && stateProvinceCode == null) {
-        await dispatch(
+        dispatch(
           showCountrySelectionModal({ ...props, skipCountry: true })
-        )
+        ).catch((error: unknown) => {
+          showError(error)
+        })
       }
     }
   }

src/actions/LoginActions.tsx

@@ -216,11 +216,15 @@ export function initializeAccount(
           )}
         />
       ))
-        .finally(async () => {
+        .finally(() => {
           for (const pluginId of pluginIdsNeedingUserAction) {
             const currencyConfig = account.currencyConfig[pluginId]
             const { userSettings = {} } = currencyConfig
-            await currencyConfig.changeUserSettings(userSettings)
+            currencyConfig
+              .changeUserSettings(userSettings)
+              .catch((error: unknown) => {
+                showError(error)
+              })
           }
         })
         .catch(err => {

src/actions/PaymentProtoActions.tsx

@@ -405,7 +405,7 @@ export async function launchPaymentProto(
     tokenId,
     lockTilesMap: { amount: true, address: true, fee: requiredFeeRate != null },
     onBack,
-    onDone: async (error: Error | null, edgeTransaction?: EdgeTransaction) => {
+    onDone: (error: Error | null, edgeTransaction?: EdgeTransaction) => {
       if (error)
         showError(
           `${lstrings.create_wallet_account_error_sending_transaction}: ${error.message}`

src/app.ts

@@ -14,6 +14,7 @@ import { getVersion } from 'react-native-device-info'
 import RNFS from 'react-native-fs'
 
 import { initDeviceSettings } from './actions/DeviceSettingsActions'
+import { showError } from './components/services/AirshipInstance'
 import { changeTheme, getTheme } from './components/services/ThemeContext'
 import { ENV } from './env'
 import type { NumberMap } from './types/types'
@@ -250,7 +251,7 @@ if (ENV.DEBUG_THEME) {
         method: 'GET'
       }
       let themeJson = ''
-      setInterval(async () => {
+      const updateTheme = async (): Promise<void> => {
         try {
           const response = await realFetch(url, getOptions)
           const overrideTheme = await response.json()
@@ -264,6 +265,11 @@ if (ENV.DEBUG_THEME) {
         } catch (e: any) {
           console.log(`Failed get theme`, e.message)
         }
+      }
+      setInterval(() => {
+        updateTheme().catch((error: unknown) => {
+          showError(error)
+        })
       }, 3000)
     } catch (e: any) {
       console.log(`Failed to access theme server`)

src/components/cards/EarnOptionCard.tsx

@@ -22,7 +22,7 @@ interface Props {
   /** If false, show "Stake"/"Earn"
    * If true, show "Staked"/"Earned" */
   isOpenPosition?: boolean
-  onPress?: () => void
+  onPress?: () => Promise<void>
 }
 
 export function EarnOptionCard(props: Props) {

src/components/modals/CategoryModal.tsx

@@ -20,6 +20,7 @@ import { useDispatch, useSelector } from '../../types/reactRedux'
 import { scale } from '../../util/scaling'
 import { MinimalButton } from '../buttons/MinimalButton'
 import { EdgeTouchableOpacity } from '../common/EdgeTouchableOpacity'
+import { showError } from '../services/AirshipInstance'
 import { cacheStyles, type Theme, useTheme } from '../services/ThemeContext'
 import { DividerLine } from '../themed/DividerLine'
 import { EdgeText } from '../themed/EdgeText'
@@ -117,9 +118,11 @@ export function CategoryModal(props: Props) {
     bridge.resolve(fullCategory)
   }
 
-  const handleSubmit = useHandler(async () => {
+  const handleSubmit = useHandler(() => {
     const fullCategory = joinCategory({ category, subcategory })
-    await handleCategoryUpdate(fullCategory)
+    handleCategoryUpdate(fullCategory).catch((error: unknown) => {
+      showError(error)
+    })
   })
 
   const keyExtractor = useHandler((row: CategoryRow) => row.raw)

src/components/modals/PasswordReminderModal.tsx

@@ -64,28 +64,32 @@ export class PasswordReminderModalComponent extends React.PureComponent<
     }
   }
 
-  handleSubmit = async () => {
+  handleSubmit = (): void => {
     const { bridge, account } = this.props
     const { password } = this.state
 
-    const isValidPassword = await account.checkPassword(password).catch(err => {
-      showError(err)
-    })
-    if (isValidPassword) {
-      this.props.dispatch({
-        type: 'PASSWORD_REMINDER_MODAL/CHECK_PASSWORD_SUCCESS'
+    account
+      .checkPassword(password)
+      .then(isValidPassword => {
+        if (isValidPassword) {
+          this.props.dispatch({
+            type: 'PASSWORD_REMINDER_MODAL/CHECK_PASSWORD_SUCCESS'
+          })
+          this.setState({ checkingPassword: false })
+          showToast(lstrings.password_reminder_great_job)
+          setTimeout(() => {
+            bridge.resolve()
+          }, 10)
+        } else {
+          this.setState({
+            errorMessage: lstrings.password_reminder_invalid,
+            checkingPassword: false
+          })
+        }
       })
-      this.setState({ checkingPassword: false })
-      showToast(lstrings.password_reminder_great_job)
-      setTimeout(() => {
-        bridge.resolve()
-      }, 10)
-    } else {
-      this.setState({
-        errorMessage: lstrings.password_reminder_invalid,
-        checkingPassword: false
+      .catch(err => {
+        showError(err)
       })
-    }
   }
 
   handleChangeText = (password: string) => {

src/components/modals/SwapVerifyTermsModal.tsx

@@ -6,7 +6,7 @@ import FastImage from 'react-native-fast-image'
 
 import { lstrings } from '../../locales/strings'
 import { getSwapPluginIconUri } from '../../util/CdnUris'
-import { Airship } from '../services/AirshipInstance'
+import { Airship, showError } from '../services/AirshipInstance'
 import { cacheStyles, type Theme, useTheme } from '../services/ThemeContext'
 import { UnscaledText } from '../text/UnscaledText'
 import { Paragraph } from '../themed/EdgeText'
@@ -88,6 +88,12 @@ function SwapVerifyTermsModal(props: Props) {
   const theme = useTheme()
   const styles = getStyles(theme)
 
+  const handleLink = (url: string): void => {
+    Linking.openURL(url).catch((error: unknown) => {
+      showError(error)
+    })
+  }
+
   return (
     <EdgeModal
       bridge={bridge}
@@ -125,23 +131,29 @@ function SwapVerifyTermsModal(props: Props) {
         {termsUri == null ? null : (
           <UnscaledText
             style={styles.linkText}
-            onPress={async () => await Linking.openURL(termsUri)}
+            onPress={() => {
+              handleLink(termsUri)
+            }}
           >
             {lstrings.swap_terms_terms_link}
           </UnscaledText>
         )}
         {privacyUri == null ? null : (
           <UnscaledText
             style={styles.linkText}
-            onPress={async () => await Linking.openURL(privacyUri)}
+            onPress={() => {
+              handleLink(privacyUri)
+            }}
           >
             {lstrings.swap_terms_privacy_link}
           </UnscaledText>
         )}
         {kycUri == null ? null : (
           <UnscaledText
             style={styles.linkText}
-            onPress={async () => await Linking.openURL(kycUri)}
+            onPress={() => {
+              handleLink(kycUri)
+            }}
           >
             {lstrings.swap_terms_kyc_link}
           </UnscaledText>

src/components/modals/TransferModal.tsx

@@ -12,7 +12,7 @@ import { useHandler } from '../../hooks/useHandler'
 import { lstrings } from '../../locales/strings'
 import { config } from '../../theme/appConfig'
 import { useDispatch } from '../../types/reactRedux'
-import { Airship } from '../services/AirshipInstance'
+import { Airship, showError } from '../services/AirshipInstance'
 import { type Theme, useTheme } from '../services/ThemeContext'
 import { SelectableRow } from '../themed/SelectableRow'
 import { EdgeModal } from './EdgeModal'
@@ -55,42 +55,51 @@ export const TransferModal = ({
     Airship.clear()
   })
 
-  const handleSend = useHandler(async () => {
-    const result = await Airship.show<WalletListResult>(bridge => (
+  const handleSend = useHandler(() => {
+    Airship.show<WalletListResult>(bridge => (
       <WalletListModal
         bridge={bridge}
         headerTitle={lstrings.select_wallet_to_send_from}
         navigation={navigation}
       />
     ))
-    if (result?.type === 'wallet') {
-      const { walletId, tokenId } = result
-      navigation.push('send2', {
-        walletId,
-        tokenId,
-        hiddenFeaturesMap: { scamWarning: false }
+      .then(result => {
+        if (result?.type === 'wallet') {
+          const { walletId, tokenId } = result
+          navigation.push('send2', {
+            walletId,
+            tokenId,
+            hiddenFeaturesMap: { scamWarning: false }
+          })
+        }
+        Airship.clear()
+      })
+      .catch((error: unknown) => {
+        showError(error)
       })
-    }
-    Airship.clear()
   })
 
-  const handleReceive = useHandler(async () => {
+  const handleReceive = useHandler(() => {
     Airship.clear()
 
-    const result = await Airship.show<WalletListResult>(bridge => (
+    Airship.show<WalletListResult>(bridge => (
       <WalletListModal
         bridge={bridge}
         headerTitle={lstrings.select_receive_asset}
         navigation={navigation}
         showCreateWallet
       />
     ))
-
-    if (result?.type === 'wallet') {
-      const { walletId, tokenId } = result
-      await dispatch(selectWalletToken({ navigation, walletId, tokenId }))
-      navigation.navigate('request', { tokenId, walletId })
-    }
+      .then(async result => {
+        if (result?.type === 'wallet') {
+          const { walletId, tokenId } = result
+          await dispatch(selectWalletToken({ navigation, walletId, tokenId }))
+          navigation.navigate('request', { tokenId, walletId })
+        }
+      })
+      .catch((error: unknown) => {
+        showError(error)
+      })
   })
 
   const handleCancel = useHandler(() => {