Merge pull request #762 from graphprotocol/pcv/coerce-if-not-null

fix(cli): when sending to L2, only coerce gas params if not null

Author: pcarranzav

cli/commands/bridge/to-l2.ts

@@ -41,6 +41,8 @@ const checkAndRedeemMessage = async (l1ToL2Message: L1ToL2MessageWriter) => {
   logger.info(`Transfer successful: ${l2TxHash}`)
 }
 
+const ifNotNullToBN = (val: string | null) => (val == null ? val : toBN(val))
+
 export const sendToL2 = async (cli: CLIEnvironment, cliArgs: CLIArgs): Promise<void> => {
   logger.info(`>>> Sending tokens to L2 <<<\n`)
 
@@ -172,9 +174,9 @@ export const sendToL2Command = {
         description: 'Calldata to pass to the recipient. Must be allowlisted in the bridge',
       })
       .coerce({
-        maxGas: toBN,
-        gasPriceBid: toBN,
-        maxSubmissionCost: toBN,
+        maxGas: ifNotNullToBN,
+        gasPriceBid: ifNotNullToBN,
+        maxSubmissionCost: ifNotNullToBN,
       })
   },
   handler: async (argv: CLIArgs): Promise<void> => {