🐛 Fix TDD comparison caching and cloud minClusterSize (#158, #160)

**Issue #158 - TDD still-failing screenshots aren't updated:**
Added `_upsertComparison()` helper that replaces existing comparisons
by ID instead of always appending. This fixes the bug where re-running
tests in TDD daemon mode would accumulate stale comparison results,
causing fixed screenshots to still appear as failures.

**Issue #160 - minClusterSize not sent to cloud API:**
Added `minClusterSize` to the `runOptions` object in the run command.
The setting was already loaded from config but wasn't being passed
through to the API payload builder.

Author: Robdel12

src/commands/run.js

@@ -236,6 +236,7 @@ export async function runCommand(
       message,
       environment: config.build.environment,
       threshold: config.comparison.threshold,
+      minClusterSize: config.comparison.minClusterSize,
       eager: config.eager || false,
       allowNoToken: config.allowNoToken || false,
       wait: config.wait || options.wait || false,

src/tdd/tdd-service.js

@@ -259,6 +259,9 @@ export class TddService {
     // Track whether results have been printed (to avoid duplicate output)
     this._resultsPrinted = false;
 
+    // Bind the helper method
+    this._upsertComparison = this._upsertComparison.bind(this);
+
     if (this.setBaseline) {
       output.info(
         '[vizzly] Baseline update mode - will overwrite existing baselines with new ones'
@@ -1131,6 +1134,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
    */
@@ -1248,7 +1265,7 @@ export class TddService {
         properties: validatedProperties,
       });
 
-      this.comparisons.push(result);
+      this._upsertComparison(result);
       return result;
     }
 
@@ -1288,7 +1305,7 @@ export class TddService {
           honeydiffResult,
         });
 
-        this.comparisons.push(result);
+        this._upsertComparison(result);
         return result;
       } else {
         let hotspotAnalysis = this.getHotspotForScreenshot(name);
@@ -1315,7 +1332,7 @@ export class TddService {
           diff: diffInfo,
         });
 
-        this.comparisons.push(result);
+        this._upsertComparison(result);
         return result;
       }
     } catch (error) {
@@ -1356,7 +1373,7 @@ export class TddService {
           properties: validatedProperties,
         });
 
-        this.comparisons.push(result);
+        this._upsertComparison(result);
         return result;
       }
 
@@ -1371,7 +1388,7 @@ export class TddService {
         errorMessage: error.message,
       });
 
-      this.comparisons.push(result);
+      this._upsertComparison(result);
       return result;
     }
   }
@@ -1792,7 +1809,7 @@ export class TddService {
       signature,
     };
 
-    this.comparisons.push(result);
+    this._upsertComparison(result);
     output.info(`Baseline created for ${name}`);
     return result;
   }