From 7594493772745a69f28d6970c171acc956642525 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jind=C5=99ich=20B=C3=A4r?= <jindrichbar@gmail.com>
Date: Mon, 20 Jan 2025 09:23:27 +0000
Subject: [PATCH] fix: throw from `BasicCrawler.run` on `CriticalError`

---
 packages/basic-crawler/src/internals/basic-crawler.ts | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/packages/basic-crawler/src/internals/basic-crawler.ts b/packages/basic-crawler/src/internals/basic-crawler.ts
index f4052742c6d2..26591052fd17 100644
--- a/packages/basic-crawler/src/internals/basic-crawler.ts
+++ b/packages/basic-crawler/src/internals/basic-crawler.ts
@@ -918,6 +918,8 @@ export class BasicCrawler<Context extends CrawlingContext = BasicCrawlingContext
         this.events.on(EventType.MIGRATING, boundPauseOnMigration);
         this.events.on(EventType.ABORTING, boundPauseOnMigration);
 
+        let stats: FinalStatistics = {} as FinalStatistics;
+
         try {
             await this.autoscaledPool!.run();
         } finally {
@@ -929,7 +931,7 @@ export class BasicCrawler<Context extends CrawlingContext = BasicCrawlingContext
             this.events.off(EventType.ABORTING, boundPauseOnMigration);
 
             const finalStats = this.stats.calculate();
-            const stats = {
+            stats = {
                 requestsFinished: this.stats.state.requestsFinished,
                 requestsFailed: this.stats.state.requestsFailed,
                 retryHistogram: this.stats.requestRetryHistogram,
@@ -970,10 +972,9 @@ export class BasicCrawler<Context extends CrawlingContext = BasicCrawlingContext
             );
             this.running = false;
             this.hasFinishedBefore = true;
-
-            // eslint-disable-next-line no-unsafe-finally -- Return from finally is safe (it's the only `return` in the function)
-            return stats;
         }
+
+        return stats;
     }
 
     async getRequestQueue() {