refactor(relayer): Reduce inventory log spam in looping mode (#1808)

If there are no cross-chain transfers to update then don't bother
logging about it.

src/clients/bridges/CrossChainTransferClient.ts

@@ -82,6 +82,10 @@ export class CrossChainTransferClient {
   async update(l1Tokens: string[], chainIds = this.getEnabledL2Chains()): Promise<void> {
     const enabledChainIds = this.getEnabledL2Chains();
     chainIds = chainIds.filter((chainId) => enabledChainIds.includes(chainId));
+    if (chainIds.length === 0) {
+      return;
+    }
+
     this.log("Updating cross chain transfers", { chainIds });
 
     const outstandingTransfersPerChain = await Promise.all(

src/relayer/index.ts

@@ -34,9 +34,13 @@ export async function runRelayer(_logger: winston.Logger, baseSigner: Signer): P
   const simulate = !config.sendingRelaysEnabled;
   const enableSlowFills = config.sendingSlowRelaysEnabled;
 
-  let run = 1;
-  let txnReceipts: { [chainId: number]: Promise<string[]> };
   const { acrossApiClient, inventoryClient, profitClient, spokePoolClients, tokenClient } = relayerClients;
+  const inventoryChainIds =
+    config.pollingDelay === 0 ? Object.values(spokePoolClients).map(({ chainId }) => chainId) : [];
+
+  let txnReceipts: { [chainId: number]: Promise<string[]> };
+  let run = 1;
+
   try {
     do {
       if (loop) {
@@ -67,8 +71,6 @@ export async function runRelayer(_logger: winston.Logger, baseSigner: Signer): P
       // disable inventory management, but does make it ignorant of in-flight cross-chain transfers. The rebalancer is
       // assumed to run separately from the relayer and with pollingDelay 0, so it doesn't loop and will track transfers
       // correctly to avoid repeat rebalances.
-      const inventoryChainIds =
-        config.pollingDelay === 0 ? Object.values(spokePoolClients).map(({ chainId }) => chainId) : [];
       await Promise.all([
         acrossApiClient.update(config.ignoreLimits),
         inventoryClient.update(inventoryChainIds),