From 95fcfcade2248a342f1fa58f79ec6d2f68c8dd2b Mon Sep 17 00:00:00 2001
From: shardAstronaut <127413534+shardAstronaut@users.noreply.github.com>
Date: Thu, 27 Jun 2024 10:16:15 +0100
Subject: [PATCH] fix: Maximum call stack size exceeded with array.push (Bug:
 last_indexed_block reset to Zero after reaching the seed blocks #300)

When indexing contracts with a big number of events the array.push method will throw an error. When you use spread operator all items of the source array are stored in the stack as arguments list, so having a large number of items (~ > 100K) will cause the this stack size exceed. The most simple work-around is to manually push all items one by one.
---
 src/providers/starknet/provider.ts | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/providers/starknet/provider.ts b/src/providers/starknet/provider.ts
index 0615208..7516457 100644
--- a/src/providers/starknet/provider.ts
+++ b/src/providers/starknet/provider.ts
@@ -217,7 +217,7 @@ export class StarknetProvider extends BaseProvider {
     }
 
     let lastSources = this.instance.getCurrentSources(blockNumber);
-    const sourcesQueue = [...lastSources];
+    let sourcesQueue = [...lastSources];
 
     let source: ContractSourceConfig | undefined;
     while ((source = sourcesQueue.shift())) {
@@ -299,7 +299,7 @@ export class StarknetProvider extends BaseProvider {
         nextSource => !lastSources.find(lastSource => lastSource.contract === nextSource.contract)
       );
 
-      sourcesQueue.push(...newSources);
+      sourcesQueue = sourcesQueue.concat(newSources);
       lastSources = nextSources;
     }
 
@@ -307,7 +307,7 @@ export class StarknetProvider extends BaseProvider {
   }
 
   private async getEvents(blockNumber: number): Promise<EventsMap> {
-    const events: Event[] = [];
+    let events: Event[] = [];
 
     let continuationToken: string | undefined;
     do {
@@ -318,7 +318,7 @@ export class StarknetProvider extends BaseProvider {
         continuation_token: continuationToken
       });
 
-      events.push(...result.events);
+      events = events.concat(result.events);
 
       continuationToken = result.continuation_token;
     } while (continuationToken);
@@ -345,7 +345,7 @@ export class StarknetProvider extends BaseProvider {
     toBlock: number,
     address: string
   ): Promise<CheckpointRecord[]> {
-    const events: Event[] = [];
+    let events: Event[] = [];
 
     let continuationToken: string | undefined;
     do {
@@ -357,7 +357,7 @@ export class StarknetProvider extends BaseProvider {
         continuation_token: continuationToken
       });
 
-      events.push(...result.events);
+      events = events.concat(result.events);
 
       continuationToken = result.continuation_token;
     } while (continuationToken);
@@ -369,7 +369,7 @@ export class StarknetProvider extends BaseProvider {
   }
 
   async getCheckpointsRange(fromBlock: number, toBlock: number): Promise<CheckpointRecord[]> {
-    const events: CheckpointRecord[] = [];
+    let events: CheckpointRecord[] = [];
 
     for (const source of this.instance.getCurrentSources(fromBlock)) {
       const addressEvents = await this.getCheckpointsRangeForAddress(
@@ -377,7 +377,7 @@ export class StarknetProvider extends BaseProvider {
         toBlock,
         source.contract
       );
-      events.push(...addressEvents);
+      events = events.concat(addressEvents);
     }
 
     return events;