refactor: using strict, refactor types to pass tslint

src/machine.ts

@@ -20,7 +20,7 @@ import {
 } from './types';
 import logger from './logger';
 
-export const syncHandler = (_context, _event) => (callback, onReceive) => {
+export const syncHandler = () => (callback, onReceive) => {
   logger.debug('Sync handler instantiated');
   const iterator = syncToLatestBlock();
   const asyncCall: () => void = async () => {

src/types.ts

@@ -13,9 +13,9 @@ export interface Block {
 export interface DecodedScript {
   type: string;
   address: string;
-  timelock?: number;
-  value?: number;
-  tokenData?: number;
+  timelock?: number | undefined | null;
+  value?: number | undefined | null;
+  tokenData?: number | undefined | null;
 }
 
 export interface Input {
@@ -25,16 +25,16 @@ export interface Input {
   script: string;
   decoded: DecodedScript;
   index: number;
-  token?: string;
+  token?: string | undefined | null;
 }
 
 export interface Output {
   value: number;
   tokenData: number;
   script: string;
   decoded: DecodedScript;
-  token?: string;
-  spentBy?: string;
+  token?: string | undefined | null;
+  spentBy?: string | undefined | null;
 }
 
 export interface Token {
@@ -103,15 +103,41 @@ export interface HandlerEvent {
   type: string;
 }
 
-export interface StatusEvent {
+export type StatusEvent = {
+  type: 'finished';
+  success: boolean;
+  message?: string;
+} | {
+  type: 'block_success';
+  success: boolean;
+  height?: number;
+  blockId: string;
+  message?: string;
+  transactions: string[];
+} | {
+  type: 'transaction_failure';
+  success: boolean;
+  message?: string;
+} | {
+  type: 'reorg';
+  success: boolean;
+  message?: string;
+} | {
+  type: 'error';
+  success: boolean;
+  message?: string;
+  error?: string;
+}
+
+/* export interface StatusEvent {
   type: string;
   success: boolean;
   blockId?: string;
   height?: number;
   transactions?: string[];
   message?: string;
   error?: string;
-};
+}; */
 
 export interface GeneratorYieldResult<StatusEvent> {
   done?: boolean;
@@ -121,7 +147,7 @@ export interface GeneratorYieldResult<StatusEvent> {
 export interface PreparedDecodedScript {
   type: string;
   address: string;
-  timelock?: number;
+  timelock?: number | undefined | null;
   value?: number;
   token_data?: number;
 }
@@ -165,3 +191,8 @@ export interface PreparedTx {
   token_symbol?: string;
   raw?: string;
 }
+
+export interface RawTxResponse {
+  tx: any;
+  meta: any;
+};

src/utils.ts

@@ -18,6 +18,8 @@ import {
   PreparedTx,
   PreparedInput,
   PreparedOutput,
+  PreparedDecodedScript,
+  RawTxResponse,
 } from './types';
 import {
   downloadTx,
@@ -35,7 +37,7 @@ import logger from './logger';
 
 dotenv.config();
 
-const TX_CACHE_SIZE: number = parseInt(process.env.TX_CACHE_SIZE) || 200;
+const TX_CACHE_SIZE: number = parseInt(process.env.TX_CACHE_SIZE as string) || 200;
 
 /**
  * Recursively downloads all transactions that were confirmed by a given block
@@ -49,8 +51,8 @@ export const recursivelyDownloadTx = async (blockId: string, txIds: string[] = [
     return data;
   }
 
-  const txId = txIds.pop();
-  const txData = await downloadTx(txId);
+  const txId: string = txIds.pop() as string;
+  const txData: RawTxResponse = await downloadTx(txId);
   const { tx, meta } = txData;
   const parsedTx: FullTx = parseTx(tx);
 
@@ -89,8 +91,8 @@ export const prepareTx = (tx: FullTx | FullBlock): PreparedTx => {
         value: input.value,
         token_data: input.tokenData,
         script: input.script,
-        token: input.token,
-        decoded: input.decoded,
+        token: input.token as string,
+        decoded: input.decoded as PreparedDecodedScript,
         index: input.index,
       };
 
@@ -101,6 +103,10 @@ export const prepareTx = (tx: FullTx | FullBlock): PreparedTx => {
         };
       }
 
+      if (!tx.tokens || tx.tokens.length <= 0) {
+        throw new Error('Input is a token but there are no tokens in the tokens list.');
+      }
+
       const { uid } = tx.tokens[wallet.getTokenIndex(input.tokenData) - 1];
 
       return {
@@ -113,9 +119,9 @@ export const prepareTx = (tx: FullTx | FullBlock): PreparedTx => {
         value: output.value,
         token_data: output.tokenData,
         script: output.script,
-        token: output.token,
-        spent_by: output.spentBy,
-        decoded: output.decoded,
+        token: output.token as string,
+        spent_by: output.spentBy as string,
+        decoded: output.decoded as PreparedDecodedScript,
       };
 
       if (output.tokenData === 0) {
@@ -125,6 +131,10 @@ export const prepareTx = (tx: FullTx | FullBlock): PreparedTx => {
         };
       }
 
+      if (!tx.tokens || tx.tokens.length <= 0) {
+        throw new Error('Output is a token but there are no tokens in the tokens list.');
+      }
+
       const { uid } = tx.tokens[wallet.getTokenIndex(output.tokenData) - 1];
 
       return {

tsconfig.json

@@ -7,7 +7,9 @@
     "declaration": true,
     "sourceMap": true,
     "rootDir": "./src",
-    "strict": false,
+    "strict": true,
+    "strictFunctionTypes": false,
+    "noImplicitAny": false,
     "noImplicitReturns": true,
     "noFallthroughCasesInSwitch": true,
     "noUnusedLocals": true,