refactor: [LW-11802] enable strict null checks

apps/browser-extension-wallet/src/tsconfig.json

@@ -6,6 +6,7 @@
     "baseUrl": "../src",
     "resolveJsonModule": true,
     "noEmitOnError": false,
+    "strictNullChecks": false,
     "paths": {
       "@assets/*": ["assets/*"],
       "@components/*": ["components/*"],

apps/browser-extension-wallet/src/utils/format-number.ts

@@ -54,7 +54,7 @@ export const formatLocaleNumber = (value: string, decimalPlaces: number = DEFAUL
  * @param maxDecimals The maximum number of decimal places to include. Default is 0.
  * @returns The formatted number string.
  */
-export const formatNumberForDisplay = (value: string, maxDecimals = 0): string => {
+export const formatNumberForDisplay = (value?: string, maxDecimals = 0): string => {
   if (!value) return '0';
   // Remove any character that is not a dot or a number
   const parsedStringValue = value.replace(/[^\d.]/g, '');
@@ -137,7 +137,7 @@ export const handleFormattedValueChange = (
  * @param decimals The desired decimal places (default = 2)
  * @returns The formatted value with the desired decimals and the unit as a string
  */
-export const compactNumberWithUnit = (value: number | string, decimals = DEFAULT_DECIMALS): string => {
+export const compactNumberWithUnit = (value?: number | string, decimals = DEFAULT_DECIMALS): string => {
   const bigNumberValue = value ? new BigNumber(value) : new BigNumber(0);
 
   if (bigNumberValue.isNaN()) return formatLocaleNumber('0', decimals);

apps/browser-extension-wallet/src/views/browser-view/features/send-transaction/components/Form/CoinInput/useSelectedCoins.tsx

@@ -27,7 +27,7 @@ import { MAX_NFT_TICKER_LENGTH, MAX_TOKEN_TICKER_LENGTH } from '../../../constan
 
 interface InputFieldActionParams {
   id: string;
-  value: string;
+  value?: string;
   maxDecimals?: number;
 }
 
@@ -176,10 +176,10 @@ export const useSelectedCoins = ({
         coinBalance,
         spendableCoin?.toString(),
         tokensUsed[cardanoCoin.id] || '0',
-        assetInputItem?.value || '0'
+        assetInputItem?.value ?? '0'
       );
       const fiatValue = Wallet.util.convertAdaToFiat({
-        ada: assetInputItem?.value || '0',
+        ada: assetInputItem?.value ?? '0',
         fiat: prices?.cardano?.price
       });
       return {

apps/browser-extension-wallet/src/views/browser-view/features/staking/components/StakingInfo/StakePoolInfo.tsx

@@ -18,8 +18,8 @@ export const StakePoolInfo = ({
   onClick?: () => void;
   popupView: boolean;
 }): React.ReactElement => {
-  const title = name || ticker || '-';
-  const subTitle: string | React.ReactElement = ticker || (
+  const title = name ?? ticker ?? '-';
+  const subTitle: string | React.ReactElement = ticker ?? (
     <Ellipsis className={styles.id} text={id} beforeEllipsis={6} afterEllipsis={8} />
   );
 

apps/browser-extension-wallet/test/__mocks__/set-env-vars.js

@@ -20,3 +20,7 @@ process.env.USE_POSTHOG_ANALYTICS = 'true';
 process.env.USE_POSTHOG_ANALYTICS_FOR_OPTED_OUT = 'false';
 process.env.POSTHOG_HOST = 'https://e.lw.iog.io';
 process.env.POSTHOG_DEV_TOKEN = 'test-token';
+process.env.MIN_NUMBER_OF_COSIGNERS = '2';
+process.env.TWITTER_URL = 'TWITTER_URL';
+process.env.YOUTUBE_URL = 'YOUTUBE_URL';
+process.env.DISCORD_URL = 'DISCORD_URL';

packages/cardano/src/ui/components/StakePoolMetricsBrowser/StakePoolMetricsBrowser.tsx

@@ -27,7 +27,7 @@ export const StakePoolMetricsBrowser = ({ data, popupView }: StakePoolMetricsBro
           {t}
         </div>
         <div className={styles.statBody} data-testid={`${testId}-value`}>
-          {unit ? formatNumericValue(value, unit) : value}
+          {unit && value ? formatNumericValue(value, unit) : value}
         </div>
       </div>
     ))}

packages/cardano/src/ui/components/StakePoolNameBrowser/StakePoolNameBrowser.tsx

@@ -32,9 +32,9 @@ export const StakePoolNameBrowser = ({
   isOversaturated,
   translations
 }: StakePoolNameBrowserProps): React.ReactElement => {
-  const title = name || ticker || '-';
-  const subTitle: string | React.ReactElement = ticker || (
-    <Ellipsis className={styles.id} text={id} beforeEllipsis={6} afterEllipsis={8} />
+  const title = name ?? ticker ?? '-';
+  const subTitle: string | React.ReactElement = ticker ?? (
+    <Ellipsis className={styles.id} text={id ?? ''} beforeEllipsis={6} afterEllipsis={8} />
   );
 
   return (
@@ -48,13 +48,15 @@ export const StakePoolNameBrowser = ({
           <p className={styles.subTitle} data-testid="stake-pool-item-ticker">
             {subTitle}
           </p>
-          <StatusLogo
-            status={status}
-            isDelegated={isDelegated}
-            isOversaturated={isOversaturated}
-            className={styles.statusLogo}
-            translations={translations}
-          />
+          {status && (
+            <StatusLogo
+              status={status}
+              isDelegated={!!isDelegated}
+              isOversaturated={isOversaturated}
+              className={styles.statusLogo}
+              translations={translations}
+            />
+          )}
         </div>
       </div>
     </div>

packages/cardano/src/ui/components/StakePoolNameBrowser/__tests__/StakePoolNameBrowser.test.tsx

@@ -5,17 +5,19 @@ import { StakePoolNameBrowser, StakePoolNameBrowserProps } from '../StakePoolNam
 import '@testing-library/jest-dom';
 
 describe('Testing StakePoolNameBrowser component', () => {
+  const name = 'name';
+  const ticker = 'ticker';
   const props: StakePoolNameBrowserProps = {
-    name: 'name',
-    ticker: 'ticker',
+    name,
+    ticker,
     isDelegated: true,
     translations: { retiring: 'retiring', retired: 'retired', delegating: 'delegating', saturated: 'saturated' }
   };
   test('should display all stake pool metrics with icons', async () => {
     const { findByText, findByTestId } = render(<StakePoolNameBrowser {...props} />);
     const roiLabel = await findByTestId('stake-pool-item-logo');
-    const nameValue = await findByText(props.name);
-    const tickerValue = await findByText(props.ticker);
+    const nameValue = await findByText(name);
+    const tickerValue = await findByText(ticker);
 
     expect(roiLabel).toBeVisible();
     expect(nameValue).toBeVisible();

packages/cardano/src/ui/components/StakePoolStatusLogo/StatusLogo.tsx

@@ -44,8 +44,8 @@ export const StatusLogo = ({
   if (status === 'retired' || status === 'retiring') {
     currentStatus = status;
   }
-  const icon = statusIcons[currentStatus];
-  const description = statusInfo[currentStatus];
+  const icon = currentStatus ? statusIcons[currentStatus] : undefined;
+  const description = currentStatus ? statusInfo[currentStatus] : undefined;
 
   return (
     <>

packages/cardano/src/ui/components/Voting/CatalystConfirmationStep.tsx

@@ -2,7 +2,7 @@ import React from 'react';
 import styles from './Voting.module.scss';
 import { Button } from '@lace/common';
 import CatalystLogo from '../../assets/images/catalyst-logo.png';
-import Icon, { InfoCircleOutlined } from '@ant-design/icons';
+import { InfoCircleOutlined } from '@ant-design/icons';
 import { TranslationsFor } from '@wallet/util/types';
 
 export interface CatalystConfirmationStepProps {
@@ -52,7 +52,7 @@ export const CatalystConfirmationStep = ({
         <>
           <div className={styles.fee}>
             <p>
-              {translations.totalFee} <Icon component={InfoCircleOutlined} />
+              {translations.totalFee} <InfoCircleOutlined />
             </p>
             <b>{fee}</b>
           </div>

packages/cardano/src/ui/components/Voting/CatalystScanStep.tsx

@@ -1,6 +1,6 @@
 import React, { useEffect, useRef } from 'react';
 import { Button } from '@lace/common';
-import Icon, { DownloadOutlined } from '@ant-design/icons';
+import { DownloadOutlined } from '@ant-design/icons';
 import QRCodeStyling from 'qr-code-styling';
 import styles from './Voting.module.scss';
 import { TranslationsFor } from '@wallet/util/types';
@@ -16,12 +16,12 @@ export const CatalystScanStep = ({
   certificate,
   translations
 }: CatalystScanStepProps): React.ReactElement => {
-  const qrCodeRef = useRef(null);
-  const qrCode = useRef<QRCodeStyling>(null);
+  const qrCodeRef = useRef<HTMLDivElement | null>(null);
+  const qrCode = useRef<QRCodeStyling | null>(null);
 
   useEffect(() => {
     qrCode.current = new QRCodeStyling({ data: certificate, width: 240, height: 240 });
-    qrCode.current.append(qrCodeRef.current);
+    qrCode.current.append(qrCodeRef.current || undefined);
   }, [certificate]);
 
   return (
@@ -34,10 +34,10 @@ export const CatalystScanStep = ({
           <div ref={qrCodeRef} />
           <div>
             <Button
-              onClick={() => qrCode.current.download({ name: 'certificate', extension: 'png' })}
+              onClick={() => qrCode.current?.download({ name: 'certificate', extension: 'png' })}
               color="secondary"
             >
-              <Icon component={DownloadOutlined} /> {translations.downloadButton}
+              <DownloadOutlined /> {translations.downloadButton}
             </Button>
           </div>
         </div>

packages/cardano/src/wallet/lib/__tests__/build-transaction.test.ts

@@ -27,7 +27,7 @@ describe.skip('Testing build-transaction', () => {
 
     expect(transaction).toBeDefined();
     expect(minimumCoinQuantities).toBeDefined();
-    expect(minimumCoinQuantities.get(output).coinMissing).toBe(BigInt(0));
+    expect(minimumCoinQuantities.get(output)?.coinMissing).toBe(BigInt(0));
   });
 
   test('should validate and build a transaction successfully with coins missing', async () => {
@@ -39,7 +39,7 @@ describe.skip('Testing build-transaction', () => {
 
     expect(transaction).toBeDefined();
     expect(minimumCoinQuantities).toBeDefined();
-    expect(minimumCoinQuantities.get(output).coinMissing).toBeGreaterThan(BigInt(0));
+    expect(minimumCoinQuantities.get(output)?.coinMissing).toBeGreaterThan(BigInt(0));
   });
 
   test('should throw an error if tx initialization fails', async () => {

packages/cardano/src/wallet/lib/__tests__/get-inputs-value.test.ts

@@ -49,7 +49,7 @@ describe('Testing getTxInputsValueAndAddress function', () => {
         signed$: of([])
       }
     }
-  } as ObservableWallet;
+  } as unknown as ObservableWallet;
 
   const multiDelegationTransactionMock: Cardano.HydratedTx = {
     inputSource: Cardano.InputSource.inputs,
@@ -194,11 +194,11 @@ describe('Testing getTxInputsValueAndAddress function', () => {
         'addr_test1qrmavlv5c7za68rq6n6r0hrcqd604j9zsr6tycwtjf3lef0wc4p9nt0e0wwytcy30mxar65892w3k77e5g2uyhx29lhqjt267p'
       );
       // From matched output in mocked transaction
-      expect(inputs[0].value.coins.toString()).toBe('3000000');
+      expect(inputs[0].value?.coins.toString()).toBe('3000000');
       expect(
-        inputs[0].value.assets
-          .get(Cardano.AssetId('6b8d07d69639e9413dd637a1a815a7323c69c86abbafb66dbfdb1aa7'))
-          .toString()
+        inputs[0].value?.assets
+          ?.get(Cardano.AssetId('6b8d07d69639e9413dd637a1a815a7323c69c86abbafb66dbfdb1aa7'))
+          ?.toString()
       ).toBe('30');
     });
   });
@@ -225,11 +225,11 @@ describe('Testing getTxInputsValueAndAddress function', () => {
         'addr_test1qrmavlv5c7za68rq6n6r0hrcqd604j9zsr6tycwtjf3lef0wc4p9nt0e0wwytcy30mxar65892w3k77e5g2uyhx29lhqjt267p'
       );
       // From matched output in mocked transaction
-      expect(inputs[0].value.coins.toString()).toBe('3000000');
+      expect(inputs[0].value?.coins.toString()).toBe('3000000');
       expect(
-        inputs[0].value.assets
-          .get(Cardano.AssetId('6b8d07d69639e9413dd637a1a815a7323c69c86abbafb66dbfdb1aa7'))
-          .toString()
+        inputs[0].value?.assets
+          ?.get(Cardano.AssetId('6b8d07d69639e9413dd637a1a815a7323c69c86abbafb66dbfdb1aa7'))
+          ?.toString()
       ).toBe('30');
     });
   });
@@ -261,7 +261,7 @@ describe('Testing getTxInputsValueAndAddress function', () => {
       const inputs = await result;
       expect(transactionsByHashes).toBeCalledTimes(1);
       expect(inputs[0].address).toBe('address');
-      expect(inputs[0].value.coins.toString()).toBe('3000000');
+      expect(inputs[0].value?.coins.toString()).toBe('3000000');
     });
   });
 
@@ -329,11 +329,11 @@ describe('Testing getTxInputsValueAndAddress function', () => {
       expect(inputs[0].address).toBe(
         'addr_test1qrmavlv5c7za68rq6n6r0hrcqd604j9zsr6tycwtjf3lef0wc4p9nt0e0wwytcy30mxar65892w3k77e5g2uyhx29lhqjt267p'
       );
-      expect(inputs[0].value.coins.toString()).toBe('3000000');
+      expect(inputs[0].value?.coins.toString()).toBe('3000000');
       expect(
-        inputs[0].value.assets
-          .get(Cardano.AssetId('6b8d07d69639e9413dd637a1a815a7323c69c86abbafb66dbfdb1aa7'))
-          .toString()
+        inputs[0].value?.assets
+          ?.get(Cardano.AssetId('6b8d07d69639e9413dd637a1a815a7323c69c86abbafb66dbfdb1aa7'))
+          ?.toString()
       ).toBe('30');
 
       // Assert the properties for the second input
@@ -342,12 +342,12 @@ describe('Testing getTxInputsValueAndAddress function', () => {
       );
 
       expect(
-        inputs[1].value.assets
-          .get(Cardano.AssetId('6b8d07d69639e9413dd637a1a815a7323c69c86abbafb66dbfdb1aa7'))
-          .toString()
+        inputs[1].value?.assets
+          ?.get(Cardano.AssetId('6b8d07d69639e9413dd637a1a815a7323c69c86abbafb66dbfdb1aa7'))
+          ?.toString()
       ).toBe('10');
 
-      expect(inputs[1].value.coins.toString()).toBe('100000');
+      expect(inputs[1].value?.coins.toString()).toBe('100000');
 
       expect(inputs[2].address).toBe(
         'addr_test1qrrx8s34r6m0w835qe9tj8mqa4ugkwhllw5l4hwpmhakpy8hukqufzmfnrvvr24tschssxw96z8dq9dz09xkg9eghtkqe07423'

packages/cardano/src/wallet/lib/build-transaction-props.ts

@@ -18,7 +18,7 @@ type OutputsMap = Map<string, CardanoOutput>;
 type TokenBalanceMap = Map<Cardano.AssetId, bigint>;
 
 export const convertAssetsToBigInt = (
-  assets: CardanoOutput['value']['assets'],
+  assets: Map<Cardano.AssetId, string>,
   assetsInfo: Assets = new Map()
 ): TokenBalanceMap => {
   const assetMap: TokenBalanceMap = new Map();

packages/cardano/src/wallet/lib/build-transaction.ts

@@ -17,7 +17,7 @@ export const buildTransaction = async (
   wallet: ObservableWallet
 ): Promise<InitializedCardanoTransaction> => {
   const util = createWalletUtil(wallet);
-  const minimumCoinQuantities = await util.validateOutputs(txProps.outputs);
+  const minimumCoinQuantities = await util.validateOutputs(txProps.outputs || []);
   const transaction = await wallet.initializeTx(txProps);
 
   return { transaction, minimumCoinQuantities };

packages/cardano/src/wallet/lib/get-inputs-value.ts

@@ -21,7 +21,7 @@ export const getTxInputsValueAndAddress = async (
 
   for (const input of inputs) {
     const output = await util.resolveInput(input);
-    resolvedInputs.push([{ address: output.address, ...input }, output]);
+    if (output) resolvedInputs.push([{ address: output.address, ...input }, output]);
   }
 
   return resolvedInputs.map((utxo) => {

packages/cardano/src/wallet/lib/hardware-wallet.ts

@@ -18,8 +18,8 @@ const DEFAULT_COMMUNICATION_TYPE = KeyManagement.CommunicationType.Web;
 
 // https://github.com/trezor/connect/blob/develop/docs/index.md#trezor-connect-manifest
 export const manifest: KeyManagement.TrezorConfig['manifest'] = {
-  appUrl: process.env.WEBSITE_URL,
-  email: process.env.EMAIL_ADDRESS
+  appUrl: process.env.WEBSITE_URL as unknown as string,
+  email: process.env.EMAIL_ADDRESS as unknown as string
 };
 
 const initializeTrezor = () =>
@@ -28,15 +28,16 @@ const initializeTrezor = () =>
     communicationType: DEFAULT_COMMUNICATION_TYPE
   });
 
-const connectDevices: Record<HardwareWallets, () => Promise<DeviceConnection>> = {
+const connectDevices: Partial<Record<HardwareWallets, () => Promise<DeviceConnection>>> = {
   [WalletType.Ledger]: async () =>
     await HardwareLedger.LedgerKeyAgent.checkDeviceConnection(DEFAULT_COMMUNICATION_TYPE),
   ...(AVAILABLE_WALLETS.includes(WalletType.Trezor) && {
     [WalletType.Trezor]: async () => await initializeTrezor()
   })
 };
 
-export const connectDevice = async (model: HardwareWallets): Promise<DeviceConnection> => await connectDevices[model]();
+export const connectDevice = async (model: HardwareWallets): Promise<DeviceConnection> =>
+  await connectDevices[model]?.();
 
 type Descriptor = Partial<USBDevice>;
 type DescriptorEntries<T extends Descriptor> = [keyof T, T[keyof T]][];
@@ -111,18 +112,18 @@ export const getHwExtendedAccountPublicKey = async (
 };
 
 type DeviceSpec = {
-  model: string;
+  model?: string;
   firmwareVersion?: string;
   cardanoAppVersion?: string;
 };
 
 const makeVersion = (major: number, minor: number, patch: number) => `${major}.${minor}.${patch}`;
 
 export const getDeviceSpec = async (connection: HardwareWalletConnection): Promise<DeviceSpec> => {
-  if (connection.type === WalletType.Ledger) {
+  if (connection.type === WalletType.Ledger && typeof connection.value !== 'undefined') {
     const { version } = await connection.value.getVersion();
     return {
-      model: connection.value.transport.deviceModel.id,
+      model: connection.value.transport.deviceModel?.id,
       cardanoAppVersion: makeVersion(version.major, version.minor, version.patch)
     };
   }

packages/cardano/src/wallet/test/mocks/ProviderStub.ts

@@ -193,9 +193,6 @@ export const mockWalletProvider = (): NetworkInfoProvider &
   RewardsProvider &
   StakePoolProvider &
   UtxoProvider => ({
-  protocolParameters: jest.fn().mockResolvedValue(protocolParameters),
-  genesisParameters: jest.fn().mockResolvedValue(genesisParameters),
-  ledgerTip: jest.fn().mockResolvedValue(ledgerTip),
   blocksByHashes: jest
     .fn()
     .mockResolvedValue([{ epoch: Cardano.EpochNo(currentEpoch.number - 3) } as Cardano.ExtendedBlockInfo]),
@@ -215,7 +212,6 @@ export const mockWalletProvider = (): NetworkInfoProvider &
       retiring: 5
     }
   }),
-  healthCheck: jest.fn().mockResolvedValue({ ok: true }),
   utxoByAddresses: jest.fn().mockResolvedValue(utxo),
   rewardAccountBalance: jest.fn().mockResolvedValue(rewards),
   ...networkInfoProviderStub()