chore: skip non-configurable properties in ContextPipeline

Author: barjin

packages/core/src/crawlers/context_pipeline.ts

@@ -1,5 +1,7 @@
 import type { Awaitable } from '@crawlee/types';
 
+import log from '@apify/log';
+
 import {
     ContextPipelineCleanupError,
     ContextPipelineInitializationError,
@@ -129,7 +131,18 @@ class ContextPipelineImpl<TContextBase, TCrawlingContext extends TContextBase> e
             for (const { action, cleanup } of middlewares) {
                 try {
                     const contextExtension = await action(crawlingContext);
-                    Object.defineProperties(crawlingContext, Object.getOwnPropertyDescriptors(contextExtension));
+
+                    const extensionDescriptors = Object.getOwnPropertyDescriptors(contextExtension);
+
+                    for (const [key, descriptor] of Object.entries(extensionDescriptors)) {
+                        try {
+                            if (Object.getOwnPropertyDescriptor(crawlingContext, key)?.configurable !== false) {
+                                Object.defineProperty(crawlingContext, key, descriptor);
+                            }
+                        } catch (error: any) {
+                            log.debug(`Context pipeline failed to define property ${key}:`, error);
+                        }
+                    }
 
                     if (cleanup) {
                         cleanupStack.push(cleanup);

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

@@ -521,20 +521,19 @@ export class AdaptivePlaywrightCrawler<
         };
 
         const subCrawlerContext = { ...context };
+
+        for (const [key, descriptor] of Object.entries(Object.getOwnPropertyDescriptors(resultBoundContextHelpers))) {
+            Object.defineProperty(subCrawlerContext, key, { ...descriptor, configurable: false });
+        }
+
         this.resultObjects.set(subCrawlerContext, result);
 
         try {
             const callAdaptiveRequestHandler = async () => {
                 if (renderingType === 'static') {
-                    await this.staticContextPipeline.call(subCrawlerContext, async (finalContext) => {
-                        Object.assign(finalContext, resultBoundContextHelpers);
-                        await this.requestHandler(finalContext);
-                    });
+                    await this.staticContextPipeline.call(subCrawlerContext, this.requestHandler.bind(this));
                 } else if (renderingType === 'clientOnly') {
-                    await this.browserContextPipeline.call(subCrawlerContext, async (finalContext) => {
-                        Object.assign(finalContext, resultBoundContextHelpers);
-                        await this.requestHandler(finalContext);
-                    });
+                    await this.browserContextPipeline.call(subCrawlerContext, this.requestHandler.bind(this));
                 }
             };