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

package/dist/index.d.ts

@@ -433,6 +433,11 @@ declare class ProboCodeGenerator {
      * Get default viewport dimensions
      */
     private static getDefaultViewPort;
+    /**
+     * Validate and normalize viewport dimensions
+     * Returns a valid viewport object with numeric width and height, falling back to defaults if invalid
+     */
+    private static normalizeViewPort;
     /**
      * Generate code for a single scenario
      * Fetches scenario data, converts to Interaction[], generates code

package/dist/index.js

@@ -5848,6 +5848,19 @@ class ProboCodeGenerator {
     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
@@ -5866,7 +5879,7 @@ class ProboCodeGenerator {
         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
@@ -6070,7 +6083,52 @@ class TestSuiteRunner {
         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.existsSync(testSuiteDir)) {
+            try {
+                const nodeModulesPath = path.join(testSuiteDir, 'node_modules');
+                const hasNodeModules = fs.existsSync(nodeModulesPath);
+                // Temporarily move node_modules out of the way if it exists
+                let tempNodeModulesPath = null;
+                if (hasNodeModules) {
+                    tempNodeModulesPath = path.join(testSuiteDir, '..', `node_modules.temp.${testSuiteId}`);
+                    // Remove temp directory if it exists from a previous failed run
+                    if (fs.existsSync(tempNodeModulesPath)) {
+                        fs.rmSync(tempNodeModulesPath, { recursive: true, force: true });
+                    }
+                    fs.renameSync(nodeModulesPath, tempNodeModulesPath);
+                    console.log(`📦 Preserved node_modules temporarily`);
+                }
+                // Delete everything in the directory
+                const entries = fs.readdirSync(testSuiteDir, { withFileTypes: true });
+                for (const entry of entries) {
+                    const entryPath = path.join(testSuiteDir, entry.name);
+                    try {
+                        if (entry.isDirectory()) {
+                            fs.rmSync(entryPath, { recursive: true, force: true });
+                        }
+                        else {
+                            fs.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.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.join(testSuiteDir, 'tests');
         ensureDirectoryExists(testsDir);