Remove screenshot capture on navigation failure in executeStep function to streamline error handling and reduce unnecessary file generation.

lablnet · lablnet · commit 7e1725bbbe55 · 2025-10-28T07:13:13.000+05:00
diff --git a/src/step-executor.ts b/src/step-executor.ts
@@ -354,8 +354,6 @@ export async function executeStep(
           try {
             await newPage.goto(href, { waitUntil: 'networkidle' });
           } catch (err: any) {
-            // take page screenshot
-            await newPage?.screenshot({ path: `screenshot-${href}.png` });
             console.log(`   ⚠️  Navigation failed for ${href}: ${err.message}`);
             throw err;
           }

Original file line number	Diff line number	Diff line change
`@@ -354,8 +354,6 @@ export async function executeStep(`
`354`	`354`	`try {`
`355`	`355`	`await newPage.goto(href, { waitUntil: 'networkidle' });`
`356`	`356`	`} catch (err: any) {`
`357`		`- // take page screenshot`
`358`		- await newPage?.screenshot({ path: `screenshot-${href}.png` });
`359`	`357`	console.log(` ⚠️ Navigation failed for ${href}: ${err.message}`);
`360`	`358`	`throw err;`
`361`	`359`	`}`