fix: skip unneccessary browser context generation for api scrapers

Signed-off-by: Eiko Wagenknecht <[email protected]>

src/services/scraper/base/scraper.ts

@@ -128,8 +128,6 @@ export abstract class BaseScraper {
    * const offers = await scraper.scrape();
    */
   public async scrape(): Promise<NewOffer[]> {
-    this.context = browserService.getContext();
-
     try {
       const offers = await this.readOffers();
       const cleanedOffers = this.cleanOffers(offers);
@@ -189,12 +187,7 @@ export abstract class BaseScraper {
     pageReadySelector: string;
     pageLoadedHook?: (page: Page) => Promise<void>;
   }): Promise<Omit<NewOffer, "category">[]> {
-    if (!this.context) {
-      throw new ScraperError(
-        "Browser context not initialized. Call initialize() first.",
-        this.getScraperName(),
-      );
-    }
+    this.context = browserService.getContext();
 
     const offers: Omit<NewOffer, "category">[] = [];
     let page: Page | null = null;

src/services/scraper/implementations/epicApi.ts

@@ -6,7 +6,6 @@ import {
   OfferType,
 } from "@/types/basic";
 import type { NewOffer } from "@/types/database";
-import { logger } from "@/utils/logger";
 import {
   ApolloClient,
   type DefaultOptions,
@@ -164,21 +163,12 @@ export class EpicGamesApiScraper extends BaseScraper {
 
   override async readOffers(): Promise<Omit<NewOffer, "category">[]> {
     const client = this.createClient();
-    try {
-      const response = await client.query<CatalogData, { count: number }>({
-        query: FREEGAMES_QUERY,
-        variables: { count: 1000 },
-      });
+    const response = await client.query<CatalogData, { count: number }>({
+      query: FREEGAMES_QUERY,
+      variables: { count: 1000 },
+    });
 
-      return this.parseOffers(response.data);
-    } catch (error) {
-      if (error instanceof Error) {
-        logger.error("Error fetching free games:", error.message);
-      } else {
-        logger.error("Unknown error occurred while fetching free games");
-      }
-      return [];
-    }
+    return this.parseOffers(response.data);
   }
 
   protected override shouldAlwaysHaveOffers(): boolean {