@probolabs/playwright 1.4.0-rc.4 → 1.4.0-rc.6

package/dist/index.cjs

@@ -5867,6 +5867,19 @@ export default class ProboReporter implements Reporter {
         static getDefaultViewPort() {
             return { width: 1280, height: 720 };
         }
+        /**
+         * Validate and normalize viewport dimensions
+         * Returns a valid viewport object with numeric width and height, falling back to defaults if invalid
+         */
+        static normalizeViewPort(viewPort) {
+            const defaultViewPort = this.getDefaultViewPort();
+            if (!viewPort) {
+                return defaultViewPort;
+            }
+            const width = typeof viewPort.width === 'number' && !isNaN(viewPort.width) ? viewPort.width : defaultViewPort.width;
+            const height = typeof viewPort.height === 'number' && !isNaN(viewPort.height) ? viewPort.height : defaultViewPort.height;
+            return { width, height };
+        }
         /**
          * Generate code for a single scenario
          * Fetches scenario data, converts to Interaction[], generates code
@@ -5885,7 +5898,7 @@ export default class ProboReporter implements Reporter {
             const interactions = this.convertBackendInteractionsToInteractionFormat(scenarioData.interactions || []);
             // Get settings and viewport
             const settings = (options === null || options === void 0 ? void 0 : options.recorderSettings) || this.getDefaultRecorderSettings();
-            const viewPort = (options === null || options === void 0 ? void 0 : options.viewPort) || scenarioData.viewPort || this.getDefaultViewPort();
+            const viewPort = this.normalizeViewPort((options === null || options === void 0 ? void 0 : options.viewPort) || scenarioData.viewPort);
             // Get parameter table rows
             const rows = scenarioData.parameterTable || [];
             // Generate code
@@ -6089,7 +6102,52 @@ export default class ProboReporter implements Reporter {
             if (codeGenResult.scenarios.length === 0) {
                 throw new Error(`No scenarios found in test suite ${testSuiteId}. Cannot generate test files.`);
             }
-            // Ensure directories exist
+            // Delete everything in the test suite directory except node_modules
+            // This ensures a clean state while preserving dependencies for faster subsequent runs
+            if (fs__namespace.existsSync(testSuiteDir)) {
+                try {
+                    const nodeModulesPath = path__namespace.join(testSuiteDir, 'node_modules');
+                    const hasNodeModules = fs__namespace.existsSync(nodeModulesPath);
+                    // Temporarily move node_modules out of the way if it exists
+                    let tempNodeModulesPath = null;
+                    if (hasNodeModules) {
+                        tempNodeModulesPath = path__namespace.join(testSuiteDir, '..', `node_modules.temp.${testSuiteId}`);
+                        // Remove temp directory if it exists from a previous failed run
+                        if (fs__namespace.existsSync(tempNodeModulesPath)) {
+                            fs__namespace.rmSync(tempNodeModulesPath, { recursive: true, force: true });
+                        }
+                        fs__namespace.renameSync(nodeModulesPath, tempNodeModulesPath);
+                        console.log(`📦 Preserved node_modules temporarily`);
+                    }
+                    // Delete everything in the directory
+                    const entries = fs__namespace.readdirSync(testSuiteDir, { withFileTypes: true });
+                    for (const entry of entries) {
+                        const entryPath = path__namespace.join(testSuiteDir, entry.name);
+                        try {
+                            if (entry.isDirectory()) {
+                                fs__namespace.rmSync(entryPath, { recursive: true, force: true });
+                            }
+                            else {
+                                fs__namespace.unlinkSync(entryPath);
+                            }
+                        }
+                        catch (error) {
+                            console.warn(`⚠️  Failed to delete ${entry.name}:`, error);
+                        }
+                    }
+                    console.log(`🗑️  Cleaned test suite directory (preserved node_modules)`);
+                    // Move node_modules back
+                    if (hasNodeModules && tempNodeModulesPath) {
+                        fs__namespace.renameSync(tempNodeModulesPath, nodeModulesPath);
+                        console.log(`📦 Restored node_modules`);
+                    }
+                }
+                catch (error) {
+                    console.warn(`⚠️  Failed to clean test suite directory ${testSuiteDir}:`, error);
+                    // Continue anyway - we'll overwrite files as needed
+                }
+            }
+            // Ensure directories exist (will recreate after deletion)
             ensureDirectoryExists(testSuiteDir);
             const testsDir = path__namespace.join(testSuiteDir, 'tests');
             ensureDirectoryExists(testsDir);