Remove optional chaining

src/mappings/HistoryElements.ts

@@ -1,13 +1,12 @@
-import {SubstrateEvent, SubstrateExtrinsic} from '@subql/types';
+import {SubstrateExtrinsic} from '@subql/types';
 import {HistoryElement, Transfer} from "../types";
 import {
     callFromProxy, callsFromBatch,
-    exportFeeFromDepositEvent,
+    exportFeeFromDepositEventAsString,
     extrinsicIdFromBlockAndIdx, isBatch, isProxy,
     isTransfer,
     timestamp
 } from "./common";
-import {Balance} from "@polkadot/types/interfaces";
 import {CallBase} from "@polkadot/types/types/calls";
 import {AnyTuple} from "@polkadot/types/types/codec";
 import {u64} from "@polkadot/types";
@@ -50,7 +49,7 @@ async function saveExtrinsic(extrinsic: SubstrateExtrinsic): Promise<void> {
         module: extrinsic.extrinsic.method.section,
         call: extrinsic.extrinsic.method.method,
         success: extrinsic.success,
-        fee: exportFeeFromDepositEvent(extrinsic)?.toString()
+        fee: exportFeeFromDepositEventAsString(extrinsic)
     }
     await element.save()
 }
@@ -74,7 +73,7 @@ function findFailedTransferCalls(extrinsic: SubstrateExtrinsic): Transfer[] | nu
             from: sender.toString(),
             to: tuple[0],
             block: blockNumber,
-            fee: exportFeeFromDepositEvent(extrinsic).toString(),
+            fee: exportFeeFromDepositEventAsString(extrinsic),
             extrinsicId: extrinsicIdFromBlockAndIdx(blockNumber, extrinsic.idx.toString()),
             success: false
         }

src/mappings/Rewards.ts

@@ -160,7 +160,7 @@ async function handleSlashForTxHistory(slashEvent: SubstrateEvent): Promise<void
     })
     const validatorsSet = new Set(validatorsInSlashEra)
 
-    const initialValidator: string | null = null
+    const initialValidator: string = ""
 
     await buildRewardEvents(
         slashEvent.block,

src/mappings/StakeChanged.ts

@@ -12,7 +12,7 @@ export async function handleBonded(event: SubstrateEvent): Promise<void> {
     let accumulatedAmount = await handleAccumulatedStake(address, amountBalance)
 
     const element = new StakeChange(eventId(event));
-    element.timestamp = timestamp(event.extrinsic.block)
+    element.timestamp = timestamp(event.block)
     element.address = address
     element.amount = amountBalance.toString()
     element.accumulatedAmount = accumulatedAmount.toString()
@@ -29,7 +29,7 @@ export async function handleUnbonded(event: SubstrateEvent): Promise<void> {
     let accumulatedAmount = await handleAccumulatedStake(address, -amountBalance)
 
     const element = new StakeChange(eventId(event));
-    element.timestamp = timestamp(event.extrinsic.block)
+    element.timestamp = timestamp(event.block)
     element.address = address
     element.amount = amountBalance.toString()
     element.accumulatedAmount = accumulatedAmount.toString()

src/mappings/Transfers.ts

@@ -1,6 +1,6 @@
 import {HistoryElement} from '../types';
 import {SubstrateEvent} from "@subql/types";
-import {blockNumber, eventId, exportFeeFromDepositEvent, extrinsicId, timestamp} from "./common";
+import {blockNumber, eventId, exportFeeFromDepositEventAsString, extrinsicId, timestamp} from "./common";
 
 export async function handleTransfer(event: SubstrateEvent): Promise<void> {
     const {event: {data: [from, to, ]}} = event;
@@ -27,7 +27,7 @@ async function populateTransfer(element: HistoryElement, event: SubstrateEvent):
         from: from.toString(),
         to: to.toString(),
         block: blockNumber(event),
-        fee: exportFeeFromDepositEvent(event.extrinsic).toString(),
+        fee: exportFeeFromDepositEventAsString(event.extrinsic),
         extrinsicId: extrinsicId(event),
         success: true
     }

src/mappings/common.ts

@@ -4,8 +4,6 @@ import {Balance} from "@polkadot/types/interfaces";
 import {CallBase} from "@polkadot/types/types/calls";
 import {AnyTuple} from "@polkadot/types/types/codec";
 import { Vec } from '@polkadot/types';
-import {EventRecord} from "@polkadot/types/interfaces/system/types"
-
 
 const batchCalls = ["batch", "batchAll"]
 const transferCalls = ["transfer", "transferKeepAlive"]
@@ -43,7 +41,8 @@ export function eventIdFromBlockAndIdx(blockNumber: string, eventIdx: string) {
 }
 
 export function extrinsicId(event: SubstrateEvent): string {
-    return `${blockNumber(event)}-${event.extrinsic.idx.toString()}`
+    let idx: string = event.extrinsic ? event.extrinsic.idx.toString() : event.idx.toString()
+    return `${blockNumber(event)}-${idx}`
 }
 
 export function blockNumber(event: SubstrateEvent): string {
@@ -71,3 +70,12 @@ export function exportFeeFromDepositEvent(extrinsic: SubstrateExtrinsic): Balanc
         return null
     }
 }
+
+export function exportFeeFromDepositEventAsString(extrinsic?: SubstrateExtrinsic): string {
+    if (extrinsic) {
+        let fee = exportFeeFromDepositEvent(extrinsic)
+        return fee ? fee.toString() : "0"
+    } else {
+        return "0"
+    } 
+}

tsconfig.json

@@ -9,7 +9,9 @@
     "module": "commonjs",
     "outDir": "dist",
     "rootDir": "src",
-    "target": "es2020"
+    "target": "es2020",
+    "skipLibCheck": true,
+    "strictNullChecks": true
   },
   "include": [
     "src/**/*",