@vizzly-testing/cli 0.21.1 → 0.21.2

package/dist/services/config-service.js

@@ -20,7 +20,8 @@ import * as output from '../utils/output.js';
  */
 let DEFAULT_CONFIG = {
   comparison: {
-    threshold: 2.0
+    threshold: 2.0,
+    minClusterSize: 2
   },
   server: {
     port: 47392,
@@ -102,6 +103,10 @@ export function createConfigService({
       config.comparison.threshold = parseFloat(process.env.VIZZLY_THRESHOLD);
       sources.comparison = 'env';
     }
+    if (process.env.VIZZLY_MIN_CLUSTER_SIZE) {
+      config.comparison.minClusterSize = parseInt(process.env.VIZZLY_MIN_CLUSTER_SIZE, 10);
+      sources.comparison = 'env';
+    }
     if (process.env.VIZZLY_PORT) {
       config.server.port = parseInt(process.env.VIZZLY_PORT, 10);
       sources.server = 'env';
@@ -243,6 +248,16 @@ export default defineConfig(${JSON.stringify(newConfig, null, 2)});
       }
     }
 
+    // Validate minClusterSize
+    if (config.comparison?.minClusterSize !== undefined) {
+      let minClusterSize = config.comparison.minClusterSize;
+      if (!Number.isInteger(minClusterSize) || minClusterSize < 1) {
+        errors.push('comparison.minClusterSize must be a positive integer (1 or greater)');
+      } else if (minClusterSize > 100) {
+        warnings.push('comparison.minClusterSize above 100 may filter out most differences');
+      }
+    }
+
     // Validate port
     if (config.server?.port !== undefined) {
       let port = config.server.port;

package/dist/tdd/tdd-service.js

@@ -863,6 +863,20 @@ export class TddService {
     return metadata;
   }
 
+  /**
+   * Upsert a comparison result - replaces existing if same ID, otherwise appends.
+   * This prevents stale results from accumulating in daemon mode.
+   * @private
+   */
+  _upsertComparison(result) {
+    let existingIndex = this.comparisons.findIndex(c => c.id === result.id);
+    if (existingIndex >= 0) {
+      this.comparisons[existingIndex] = result;
+    } else {
+      this.comparisons.push(result);
+    }
+  }
+
   /**
    * Compare a screenshot against baseline
    */
@@ -959,7 +973,7 @@ export class TddService {
         currentPath: currentImagePath,
         properties: validatedProperties
       });
-      this.comparisons.push(result);
+      this._upsertComparison(result);
       return result;
     }
 
@@ -982,7 +996,7 @@ export class TddService {
           minClusterSize: effectiveMinClusterSize,
           honeydiffResult
         });
-        this.comparisons.push(result);
+        this._upsertComparison(result);
         return result;
       } else {
         let hotspotAnalysis = this.getHotspotForScreenshot(name);
@@ -1007,7 +1021,7 @@ export class TddService {
         output.debug('comparison', `${sanitizedName}: ${result.status}`, {
           diff: diffInfo
         });
-        this.comparisons.push(result);
+        this._upsertComparison(result);
         return result;
       }
     } catch (error) {
@@ -1035,7 +1049,7 @@ export class TddService {
           currentPath: currentImagePath,
           properties: validatedProperties
         });
-        this.comparisons.push(result);
+        this._upsertComparison(result);
         return result;
       }
       output.debug('comparison', `${sanitizedName}: error - ${error.message}`);
@@ -1047,7 +1061,7 @@ export class TddService {
         properties: validatedProperties,
         errorMessage: error.message
       });
-      this.comparisons.push(result);
+      this._upsertComparison(result);
       return result;
     }
   }
@@ -1389,7 +1403,7 @@ export class TddService {
       properties,
       signature
     };
-    this.comparisons.push(result);
+    this._upsertComparison(result);
     output.info(`Baseline created for ${name}`);
     return result;
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vizzly-testing/cli",
-  "version": "0.21.1",
+  "version": "0.21.2",
   "description": "Visual review platform for UI developers and designers",
   "keywords": [
     "visual-testing",