@midscene/cli 1.10.12-beta-20260813050119.0 → 1.10.12

package/dist/es/framework/index.mjs

@@ -7,7 +7,7 @@ import { ReportMergingTool } from "@midscene/core";
 import { getMidsceneRunSubDir } from "@midscene/shared/common";
 import { getDebug } from "@midscene/shared/logger";
 import { ScriptPlayer, parseYamlScript, resolveWebTarget } from "@midscene/core/yaml";
-import { buildChromeArgs, buildDownloadBehavior, captureSessionStorageSnapshot, defaultViewportHeight, defaultViewportWidth, puppeteerAgentForTarget } from "@midscene/web/puppeteer-agent-launcher";
+import { buildChromeArgs, buildDownloadBehavior, defaultViewportHeight, defaultViewportWidth, puppeteerAgentForTarget } from "@midscene/web/puppeteer-agent-launcher";
 import lodash_merge from "lodash.merge";
 import puppeteer from "puppeteer";
 import { createServer } from "http-server";
@@ -2403,13 +2403,10 @@ function createNotExecutedYamlResult(file) {
         error: notExecutedError
     };
 }
-function isYamlPlayerSuccessful(player) {
-    return 'error' !== player.status && !player.taskStatusList?.some((task)=>'error' === task.status);
-}
 function createExecutedYamlResult(options) {
     const { file, player, duration } = options;
+    const hasFailedTasks = player.taskStatusList?.some((task)=>'error' === task.status) ?? false;
     const hasPlayerError = 'error' === player.status;
-    const hasFailedTasks = !isYamlPlayerSuccessful(player) && !hasPlayerError;
     let success;
     let resultType;
     if (hasPlayerError) {
@@ -2823,7 +2820,7 @@ async function createYamlPlayer(file, script, options) {
                 const { agent, freeFn: newFreeFn } = await puppeteerAgentForTarget(webTarget, {
                     ...preference,
                     ...buildAgentOptions(clonedYamlScript.agent, preference.reportFileName, fileName)
-                }, cdpBrowser, options?.page, options?.sessionStorageSnapshot);
+                }, cdpBrowser, options?.page);
                 const cleanFreeFn = newFreeFn.filter((f)=>'puppeteer_browser' !== f.name);
                 if (!options?.browser) cleanFreeFn.push({
                     name: 'cdp_browser_disconnect',
@@ -2839,7 +2836,7 @@ async function createYamlPlayer(file, script, options) {
                 const { agent, freeFn: newFreeFn } = await puppeteerAgentForTarget(webTarget, {
                     ...preference,
                     ...buildAgentOptions(clonedYamlScript.agent, preference.reportFileName, fileName)
-                }, options?.browser, options?.page, options?.sessionStorageSnapshot);
+                }, options?.browser, options?.page);
                 freeFn.push(...newFreeFn);
                 return {
                     agent,
@@ -3418,9 +3415,6 @@ function yaml_batch_executor_define_property(obj, key, value) {
     else obj[key] = value;
     return obj;
 }
-const batchWarning = getDebug('yaml-batch-executor', {
-    console: true
-});
 class YamlBatchExecutor {
     async run(options = {}) {
         const generateSummary = options.generateSummary ?? true;
@@ -3432,6 +3426,7 @@ class YamlBatchExecutor {
         let setupContext;
         const fileContextList = [];
         let browser = null;
+        let sharedPage = null;
         try {
             if (setup) {
                 const fileConfig = await this.loadFileConfig(setup);
@@ -3487,7 +3482,11 @@ class YamlBatchExecutor {
                         acceptInsecureCerts: globalWebConfig?.acceptInsecureCerts
                     });
                 }
-                for (const context of allContexts)context.options.browser = browser;
+                sharedPage = await browser.newPage();
+                for (const context of allContexts){
+                    context.options.browser = browser;
+                    context.options.page = sharedPage;
+                }
             }
             const { executedResults, notExecutedContexts } = await this.executeFiles(setupContext, fileContextList);
             this.results = await this.processResults(executedResults, notExecutedContexts);
@@ -3545,7 +3544,7 @@ class YamlBatchExecutor {
             ttyRenderer.start();
         }
         try {
-            const executeFile = async (context, keepPageOpen = false)=>{
+            const executeFile = async (context)=>{
                 const allFileContext = allFileContexts.find((c)=>c.file === context.file);
                 if (!allFileContext) throw new Error(`Player not found for file: ${context.file}`);
                 if (!isTTY) {
@@ -3553,44 +3552,23 @@ class YamlBatchExecutor {
                     console.log(mergedText);
                 }
                 if (context.outputPath) allFileContext.player.output = context.outputPath;
-                const ownedPage = await this.createPageForContext(context);
-                let executionFailed = false;
-                try {
-                    const startTime = Date.now();
-                    await allFileContext.player.run();
-                    const endTime = Date.now();
-                    const duration = endTime - startTime;
-                    const executedContext = {
-                        file: context.file,
-                        player: allFileContext.player,
-                        duration
-                    };
-                    if (!isTTY) console.log(contextTaskListSummary(allFileContext.player.taskStatusList, executedContext));
-                    return executedContext;
-                } catch (error) {
-                    executionFailed = true;
-                    throw error;
-                } finally{
-                    if (ownedPage && !keepPageOpen && !this.config.keepWindow) await this.closePage(ownedPage, executionFailed);
-                }
+                const startTime = Date.now();
+                await allFileContext.player.run();
+                const endTime = Date.now();
+                const duration = endTime - startTime;
+                const executedContext = {
+                    file: context.file,
+                    player: allFileContext.player,
+                    duration
+                };
+                if (!isTTY) console.log(contextTaskListSummary(allFileContext.player.taskStatusList, executedContext));
+                return executedContext;
             };
             let setupFailed = false;
             if (setupContext) {
-                let setupExecutionFailed = false;
-                try {
-                    const executedContext = await executeFile(setupContext, true);
-                    executedResults.push(executedContext);
-                    setupFailed = !isYamlPlayerSuccessful(executedContext.player);
-                    if (!setupFailed && setupContext.options.page) {
-                        const sessionStorageSnapshot = await captureSessionStorageSnapshot(setupContext.options.page);
-                        for (const context of fileContextList)context.options.sessionStorageSnapshot = sessionStorageSnapshot;
-                    }
-                } catch (error) {
-                    setupExecutionFailed = true;
-                    throw error;
-                } finally{
-                    if (setupContext.options.page && !this.config.keepWindow) await this.closePage(setupContext.options.page, setupExecutionFailed);
-                }
+                const executedContext = await executeFile(setupContext);
+                executedResults.push(executedContext);
+                setupFailed = 'error' === executedContext.player.status;
             }
             if (setupFailed) for (const context of fileContextList)notExecutedContexts.push({
                 file: context.file,
@@ -3609,32 +3587,6 @@ class YamlBatchExecutor {
             notExecutedContexts
         };
     }
-    async createPageForContext(context) {
-        const webTarget = resolveWebTarget(context.executionConfig)?.target;
-        if (!context.options.browser || !webTarget || webTarget.bridgeMode) return;
-        const page = await context.options.browser.newPage();
-        context.options.page = page;
-        return page;
-    }
-    async closePage(page, preservePriorError = false) {
-        try {
-            if (!page.isClosed()) await page.close();
-        } catch (error) {
-            if (preservePriorError) return void batchWarning('failed to close a YAML execution page after execution had already failed; preserving the original error', error);
-            throw new Error('Failed to close a YAML execution page', {
-                cause: error
-            });
-        }
-    }
-    throwUnexpectedExecutionError(settledResults) {
-        const rejected = settledResults.find((result)=>'rejected' === result.status);
-        if (rejected) {
-            if (rejected.reason instanceof Error) throw rejected.reason;
-            throw new Error('Unexpected YAML execution failure', {
-                cause: rejected.reason
-            });
-        }
-    }
     async executeConcurrently(fileContextList, executeFile, executedResults, notExecutedContexts) {
         const limit = pLimit(this.config.concurrent);
         if (this.config.continueOnError) {
@@ -3642,8 +3594,7 @@ class YamlBatchExecutor {
                     const executedContext = await executeFile(context);
                     executedResults.push(executedContext);
                 }));
-            const settledResults = await Promise.allSettled(tasks);
-            this.throwUnexpectedExecutionError(settledResults);
+            await Promise.allSettled(tasks);
         } else {
             let shouldStop = false;
             const stopLock = {
@@ -3661,8 +3612,7 @@ class YamlBatchExecutor {
                         shouldStop = true;
                     }
                 }));
-            const settledResults = await Promise.allSettled(tasks);
-            this.throwUnexpectedExecutionError(settledResults);
+            await Promise.allSettled(tasks);
             if (shouldStop) {
                 for (const context of fileContextList)if (!executedResults.some((r)=>r.file === context.file) && !notExecutedContexts.some((ctx)=>ctx.file === context.file)) notExecutedContexts.push({
                     file: context.file,