fix: Prevent browser-based crawling mode in AdaptivePlaywrightCrawler from swallowing logs (#3803)

closes #3447

Author: janbuchar

packages/playwright-crawler/src/internals/adaptive-playwright-crawler.ts

@@ -648,6 +648,7 @@ export class AdaptivePlaywrightCrawler<
                 throw browserRun.error;
             }
 
+            browserRun.logs?.forEach(([log, method, ...args]) => log[method](...(args as [any, any])));
             await this.commitResult(crawlingContext, browserRun.result);
 
             if (shouldDetectRenderingType) {

test/core/crawlers/adaptive_playwright_crawler.test.ts

@@ -1,7 +1,15 @@
 import type { Server } from 'node:http';
 import type { AddressInfo } from 'node:net';
 
-import { type Dictionary, EventType, KeyValueStore, serviceLocator } from '@crawlee/core';
+import {
+    BaseCrawleeLogger,
+    type CrawleeLogger,
+    type CrawleeLoggerOptions,
+    type Dictionary,
+    EventType,
+    KeyValueStore,
+    serviceLocator,
+} from '@crawlee/core';
 import type {
     AdaptivePlaywrightCrawlerContext,
     AdaptivePlaywrightCrawlerOptions,
@@ -14,6 +22,22 @@ import express from 'express';
 import { startExpressAppPromise } from '../../shared/_helper.js';
 import { MemoryStorageEmulator } from '../../shared/MemoryStorageEmulator.js';
 
+// A minimal logger that records every message into a shared array. Child loggers share the same
+// array, so messages emitted by the crawler's prefixed child logger are captured as well.
+class RecordingLogger extends BaseCrawleeLogger {
+    constructor(private readonly messages: string[]) {
+        super();
+    }
+
+    logWithLevel(_level: number, message: string): void {
+        this.messages.push(message);
+    }
+
+    protected createChild(_options: Partial<CrawleeLoggerOptions>): CrawleeLogger {
+        return new RecordingLogger(this.messages);
+    }
+}
+
 describe('AdaptivePlaywrightCrawler', () => {
     // Set up an express server that will serve test pages
     const HOSTNAME = '127.0.0.1';
@@ -175,6 +199,36 @@ describe('AdaptivePlaywrightCrawler', () => {
         });
     });
 
+    test.each([['static'], ['clientOnly']] as const)(
+        'should replay request handler logs (%s)',
+        async (renderingType) => {
+            const renderingTypePredictor = makeRiggedRenderingTypePredictor({
+                detectionProbabilityRecommendation: 0,
+                renderingType,
+            });
+            const url = new URL(`http://${HOSTNAME}:${port}/static`);
+
+            const messages: string[] = [];
+            const requestHandler: AdaptivePlaywrightCrawlerOptions['requestHandler'] = vi.fn(async ({ log }) => {
+                log.info('handler log message');
+            });
+
+            const crawler = await makeOneshotCrawler(
+                {
+                    requestHandler,
+                    renderingTypePredictor,
+                    logger: new RecordingLogger(messages),
+                },
+                [url.toString()],
+            );
+
+            await crawler.run();
+
+            expect(requestHandler).toHaveBeenCalled();
+            expect(messages).toContain('handler log message');
+        },
+    );
+
     test('should not store detection results on non-detection runs', async () => {
         const renderingTypePredictor = makeRiggedRenderingTypePredictor({
             detectionProbabilityRecommendation: 0,