@vizzly-testing/cli 0.39.0 → 0.39.2

package/dist/{tdd/tdd-service.js → domains/tdd/tdd.service.js}

@@ -5,25 +5,31 @@
  * This is a thin orchestration layer - most logic lives in the modules.
  *
  * CRITICAL: Signature/filename generation MUST stay in sync with the cloud!
- * See src/tdd/core/signature.js for details.
+ * See src/domains/screenshots/screenshot-identity.js for details.
  */
 
 import { existsSync as defaultExistsSync, mkdirSync as defaultMkdirSync, readFileSync as defaultReadFileSync, writeFileSync as defaultWriteFileSync } from 'node:fs';
 import { compareSync } from '@vizzly-testing/honeydiff';
-import { createApiClient as defaultCreateApiClient, getBuilds as defaultGetBuilds, getComparison as defaultGetComparison, getTddBaselines as defaultGetTddBaselines } from '../api/index.js';
-import { NetworkError, VizzlyError } from '../errors/vizzly-error.js';
-import { colors as defaultColors } from '../utils/colors.js';
-import { fetchWithTimeout as defaultFetchWithTimeout } from '../utils/fetch-utils.js';
-import { getDefaultBranch as defaultGetDefaultBranch } from '../utils/git.js';
-import * as defaultOutput from '../utils/output.js';
-import { safePath as defaultSafePath, sanitizeScreenshotName as defaultSanitizeScreenshotName, validatePathSecurity as defaultValidatePathSecurity, validateScreenshotProperties as defaultValidateScreenshotProperties } from '../utils/security.js';
+import { colors as defaultColors } from '../../shared/colors.js';
+import { fetchWithTimeout as defaultFetchWithTimeout } from '../../shared/fetch-utils.js';
+import { getDefaultBranch as defaultGetDefaultBranch } from '../../shared/git.js';
+import * as defaultOutput from '../../shared/output.js';
+import { safePath as defaultSafePath, validatePathSecurity as defaultValidatePathSecurity } from '../../shared/paths.js';
+import { VizzlyError } from '../../shared/vizzly-error.js';
+import { createApiClient as defaultCreateApiClient } from '../auth/cloud-client.js';
+import { baselineExists as defaultBaselineExists, clearBaselineData as defaultClearBaselineData, clearRunData as defaultClearRunData, getBaselinePath as defaultGetBaselinePath, getCurrentPath as defaultGetCurrentPath, getDiffPath as defaultGetDiffPath, initializeDirectories as defaultInitializeDirectories, saveBaseline as defaultSaveBaseline, saveCurrent as defaultSaveCurrent } from '../baselines/baseline.service.js';
+import { installBaselineBuild } from '../baselines/baseline-download.js';
+import { createEmptyBaselineMetadata as defaultCreateEmptyBaselineMetadata, loadBaselineMetadata as defaultLoadBaselineMetadata, saveBaselineMetadata as defaultSaveBaselineMetadata, upsertScreenshotInMetadata as defaultUpsertScreenshotInMetadata } from '../baselines/baseline-metadata.js';
+import { writeBaselineScreenshot } from '../baselines/baseline-write.js';
+import { getTddBaselines as defaultGetTddBaselines } from '../baselines/cloud-baselines.js';
+import { getBuilds as defaultGetBuilds } from '../builds/build.service.js';
+import { getComparison as defaultGetComparison } from '../comparisons/cloud-comparisons.js';
+import { buildErrorComparison as defaultBuildErrorComparison, buildFailedComparison as defaultBuildFailedComparison, buildNewComparison as defaultBuildNewComparison, buildPassedComparison as defaultBuildPassedComparison, compareImages as defaultCompareImages, isDimensionMismatchError as defaultIsDimensionMismatchError } from '../comparisons/comparison.service.js';
+import { buildResults as defaultBuildResults, getFailedComparisons as defaultGetFailedComparisons, getNewComparisons as defaultGetNewComparisons } from '../comparisons/comparison-results.js';
+import { sanitizeScreenshotName as defaultSanitizeScreenshotName, validateScreenshotProperties as defaultValidateScreenshotProperties } from '../screenshots/screenshot.validations.js';
 // Import from extracted modules
-import { generateBaselineFilename as defaultGenerateBaselineFilename, generateComparisonId as defaultGenerateComparisonId, generateScreenshotSignature as defaultGenerateScreenshotSignature } from './core/signature.js';
-import { createEmptyBaselineMetadata as defaultCreateEmptyBaselineMetadata, loadBaselineMetadata as defaultLoadBaselineMetadata, saveBaselineMetadata as defaultSaveBaselineMetadata, upsertScreenshotInMetadata as defaultUpsertScreenshotInMetadata } from './metadata/baseline-metadata.js';
+import { generateBaselineFilename as defaultGenerateBaselineFilename, generateComparisonId as defaultGenerateComparisonId, generateScreenshotSignature as defaultGenerateScreenshotSignature } from '../screenshots/screenshot-identity.js';
 import { buildTddDependencyOps, resolveTddPaths, resolveTddWorkingDirectory } from './runtime-context.js';
-import { baselineExists as defaultBaselineExists, clearBaselineData as defaultClearBaselineData, clearRunData as defaultClearRunData, getBaselinePath as defaultGetBaselinePath, getCurrentPath as defaultGetCurrentPath, getDiffPath as defaultGetDiffPath, initializeDirectories as defaultInitializeDirectories, saveBaseline as defaultSaveBaseline, saveCurrent as defaultSaveCurrent } from './services/baseline-manager.js';
-import { buildErrorComparison as defaultBuildErrorComparison, buildFailedComparison as defaultBuildFailedComparison, buildNewComparison as defaultBuildNewComparison, buildPassedComparison as defaultBuildPassedComparison, compareImages as defaultCompareImages, isDimensionMismatchError as defaultIsDimensionMismatchError } from './services/comparison-service.js';
-import { buildResults as defaultBuildResults, getFailedComparisons as defaultGetFailedComparisons, getNewComparisons as defaultGetNewComparisons } from './services/result-service.js';
 
 /**
  * Create a new TDD service instance
@@ -150,20 +156,13 @@ export class TddService {
     // Destructure dependencies
     let {
       output,
-      colors,
       getDefaultBranch,
       getTddBaselines,
       getBuilds,
       getComparison,
       clearBaselineData,
       generateScreenshotSignature,
-      generateBaselineFilename,
-      sanitizeScreenshotName,
-      safePath,
-      existsSync,
-      fetchWithTimeout,
-      writeFileSync,
-      saveBaselineMetadata
+      generateBaselineFilename
     } = this._deps;
 
     // If no branch specified, detect default branch
@@ -281,181 +280,7 @@ export class TddService {
         baselineBuild = apiResponse.build;
         baselineBuild.screenshots = apiResponse.screenshots;
       }
-      let buildDetails = baselineBuild;
-      if (!buildDetails.screenshots || buildDetails.screenshots.length === 0) {
-        output.warn('No screenshots found in baseline build');
-        return null;
-      }
-      output.info(`Using baseline from build: ${colors.info(baselineBuild.name || 'Unknown')} (${baselineBuild.id || 'Unknown ID'})`);
-      output.info(`Checking ${colors.info(buildDetails.screenshots.length)} baseline screenshots...`);
-
-      // Check existing baseline metadata for SHA comparison
-      let existingBaseline = await this.loadBaseline();
-      let existingShaMap = new Map();
-      if (existingBaseline) {
-        existingBaseline.screenshots.forEach(s => {
-          if (s.sha256 && s.filename) {
-            existingShaMap.set(s.filename, s.sha256);
-          }
-        });
-      }
-
-      // Download screenshots
-      let downloadedCount = 0;
-      let skippedCount = 0;
-      let errorCount = 0;
-      let batchSize = 5;
-      let screenshotsToProcess = [];
-      for (let screenshot of buildDetails.screenshots) {
-        let sanitizedName;
-        try {
-          sanitizedName = sanitizeScreenshotName(screenshot.name);
-        } catch (error) {
-          output.warn(`Skipping screenshot with invalid name '${screenshot.name}': ${error.message}`);
-          errorCount++;
-          continue;
-        }
-        let filename = screenshot.filename;
-        if (!filename) {
-          output.warn(`Screenshot ${sanitizedName} has no filename from API - skipping`);
-          errorCount++;
-          continue;
-        }
-        let imagePath = safePath(this.baselinePath, filename);
-
-        // Check SHA
-        if (existsSync(imagePath) && screenshot.sha256) {
-          let storedSha = existingShaMap.get(filename);
-          if (storedSha === screenshot.sha256) {
-            downloadedCount++;
-            skippedCount++;
-            continue;
-          }
-        }
-        let downloadUrl = screenshot.original_url || screenshot.url;
-        if (!downloadUrl) {
-          output.warn(`Screenshot ${sanitizedName} has no download URL - skipping`);
-          errorCount++;
-          continue;
-        }
-        screenshotsToProcess.push({
-          screenshot,
-          sanitizedName,
-          imagePath,
-          downloadUrl,
-          filename
-        });
-      }
-
-      // Process downloads in batches
-      if (screenshotsToProcess.length > 0) {
-        output.info(`📥 Downloading ${screenshotsToProcess.length} new/updated screenshots...`);
-        for (let i = 0; i < screenshotsToProcess.length; i += batchSize) {
-          let batch = screenshotsToProcess.slice(i, i + batchSize);
-          let batchNum = Math.floor(i / batchSize) + 1;
-          let totalBatches = Math.ceil(screenshotsToProcess.length / batchSize);
-          output.info(`📦 Processing batch ${batchNum}/${totalBatches}`);
-          let downloadPromises = batch.map(async ({
-            sanitizedName,
-            imagePath,
-            downloadUrl
-          }) => {
-            try {
-              let response = await fetchWithTimeout(downloadUrl);
-              if (!response.ok) {
-                throw new NetworkError(`Failed to download ${sanitizedName}: ${response.statusText}`);
-              }
-              let arrayBuffer = await response.arrayBuffer();
-              let imageBuffer = Buffer.from(arrayBuffer);
-              writeFileSync(imagePath, imageBuffer);
-              return {
-                success: true,
-                name: sanitizedName
-              };
-            } catch (error) {
-              output.warn(`Failed to download ${sanitizedName}: ${error.message}`);
-              return {
-                success: false,
-                name: sanitizedName,
-                error: error.message
-              };
-            }
-          });
-          let batchResults = await Promise.all(downloadPromises);
-          let batchSuccesses = batchResults.filter(r => r.success).length;
-          let batchFailures = batchResults.filter(r => !r.success).length;
-          downloadedCount += batchSuccesses;
-          errorCount += batchFailures;
-        }
-      }
-      if (downloadedCount === 0 && skippedCount === 0) {
-        output.error('No screenshots were successfully downloaded');
-        return null;
-      }
-
-      // Store baseline metadata
-      this.baselineData = {
-        buildId: baselineBuild.id,
-        buildName: baselineBuild.name,
-        environment,
-        branch,
-        threshold: this.threshold,
-        signatureProperties: this.signatureProperties,
-        createdAt: new Date().toISOString(),
-        buildInfo: {
-          commitSha: baselineBuild.commit_sha,
-          commitMessage: baselineBuild.commit_message,
-          visual_review: baselineBuild.visual_review || null,
-          completedAt: baselineBuild.completed_at
-        },
-        screenshots: buildDetails.screenshots.filter(s => s.filename).map(s => ({
-          name: sanitizeScreenshotName(s.name),
-          originalName: s.name,
-          sha256: s.sha256,
-          id: s.id,
-          filename: s.filename,
-          path: safePath(this.baselinePath, s.filename),
-          browser: s.browser,
-          viewport_width: s.viewport_width,
-          originalUrl: s.original_url,
-          fileSize: s.file_size_bytes,
-          dimensions: {
-            width: s.width,
-            height: s.height
-          }
-        }))
-      };
-      saveBaselineMetadata(this.baselinePath, this.baselineData);
-
-      // Save baseline build metadata for MCP plugin
-      let baselineMetadataPath = safePath(this.workingDir, '.vizzly', 'baseline-metadata.json');
-      writeFileSync(baselineMetadataPath, JSON.stringify({
-        buildId: baselineBuild.id,
-        buildName: baselineBuild.name,
-        branch,
-        environment,
-        commitSha: baselineBuild.commit_sha,
-        commitMessage: baselineBuild.commit_message,
-        visual_review: baselineBuild.visual_review || null,
-        completedAt: baselineBuild.completed_at,
-        downloadedAt: new Date().toISOString()
-      }, null, 2));
-
-      // Summary
-      let actualDownloads = downloadedCount - skippedCount;
-      if (skippedCount > 0) {
-        if (actualDownloads === 0) {
-          output.info(`All ${skippedCount} baselines up-to-date`);
-        } else {
-          output.info(`Downloaded ${actualDownloads} new screenshots, ${skippedCount} already up-to-date`);
-        }
-      } else {
-        output.info(`Downloaded ${downloadedCount}/${buildDetails.screenshots.length} screenshots successfully`);
-      }
-      if (errorCount > 0) {
-        output.warn(`${errorCount} screenshots failed to download`);
-      }
-      return this.baselineData;
+      return await this._installBaselineBuild(baselineBuild, environment, branch);
     } catch (error) {
       output.error(`Failed to download baseline: ${error.message}`);
       throw error;
@@ -473,14 +298,7 @@ export class TddService {
     // Destructure dependencies
     let {
       output,
-      colors,
-      clearBaselineData,
-      sanitizeScreenshotName,
-      safePath,
-      existsSync,
-      fetchWithTimeout,
-      writeFileSync,
-      saveBaselineMetadata
+      clearBaselineData
     } = this._deps;
 
     // Clear local state before downloading
@@ -506,181 +324,47 @@ export class TddService {
       output.warn(`Build ${buildId} has status: ${baselineBuild.status} (expected: completed)`);
     }
     baselineBuild.screenshots = apiResponse.screenshots;
-    let buildDetails = baselineBuild;
-    if (!buildDetails.screenshots || buildDetails.screenshots.length === 0) {
+    return this._installBaselineBuild(baselineBuild, 'test', null);
+  }
+  async _installBaselineBuild(baselineBuild, environment, branch) {
+    let {
+      output,
+      colors,
+      fetchWithTimeout,
+      existsSync,
+      writeFileSync,
+      safePath,
+      sanitizeScreenshotName,
+      saveBaselineMetadata
+    } = this._deps;
+    if (!baselineBuild.screenshots || baselineBuild.screenshots.length === 0) {
       output.warn('No screenshots found in baseline build');
       return null;
     }
     output.info(`Using baseline from build: ${colors.info(baselineBuild.name || 'Unknown')} (${baselineBuild.id || 'Unknown ID'})`);
-    output.info(`Checking ${colors.info(buildDetails.screenshots.length)} baseline screenshots...`);
-
-    // Check existing baseline metadata for SHA comparison
+    output.info(`Checking ${colors.info(baselineBuild.screenshots.length)} baseline screenshots...`);
     let existingBaseline = await this.loadBaseline();
-    let existingShaMap = new Map();
-    if (existingBaseline) {
-      existingBaseline.screenshots.forEach(s => {
-        if (s.sha256 && s.filename) {
-          existingShaMap.set(s.filename, s.sha256);
-        }
-      });
-    }
-
-    // Download screenshots
-    let downloadedCount = 0;
-    let skippedCount = 0;
-    let errorCount = 0;
-    let batchSize = 5;
-    let screenshotsToProcess = [];
-    for (let screenshot of buildDetails.screenshots) {
-      let sanitizedName;
-      try {
-        sanitizedName = sanitizeScreenshotName(screenshot.name);
-      } catch (error) {
-        output.warn(`Skipping screenshot with invalid name '${screenshot.name}': ${error.message}`);
-        errorCount++;
-        continue;
-      }
-      let filename = screenshot.filename;
-      if (!filename) {
-        output.warn(`Screenshot ${sanitizedName} has no filename from API - skipping`);
-        errorCount++;
-        continue;
-      }
-      let imagePath = safePath(this.baselinePath, filename);
-
-      // Check SHA
-      if (existsSync(imagePath) && screenshot.sha256) {
-        let storedSha = existingShaMap.get(filename);
-        if (storedSha === screenshot.sha256) {
-          downloadedCount++;
-          skippedCount++;
-          continue;
-        }
-      }
-      let downloadUrl = screenshot.original_url || screenshot.url;
-      if (!downloadUrl) {
-        output.warn(`Screenshot ${sanitizedName} has no download URL - skipping`);
-        errorCount++;
-        continue;
-      }
-      screenshotsToProcess.push({
-        screenshot,
-        sanitizedName,
-        imagePath,
-        downloadUrl,
-        filename
-      });
-    }
-
-    // Process downloads in batches
-    if (screenshotsToProcess.length > 0) {
-      output.info(`📥 Downloading ${screenshotsToProcess.length} new/updated screenshots...`);
-      for (let i = 0; i < screenshotsToProcess.length; i += batchSize) {
-        let batch = screenshotsToProcess.slice(i, i + batchSize);
-        let batchNum = Math.floor(i / batchSize) + 1;
-        let totalBatches = Math.ceil(screenshotsToProcess.length / batchSize);
-        output.info(`📦 Processing batch ${batchNum}/${totalBatches}`);
-        let downloadPromises = batch.map(async ({
-          sanitizedName,
-          imagePath,
-          downloadUrl
-        }) => {
-          try {
-            let response = await fetchWithTimeout(downloadUrl);
-            if (!response.ok) {
-              throw new Error(`Failed to download ${sanitizedName}: ${response.statusText}`);
-            }
-            let arrayBuffer = await response.arrayBuffer();
-            let imageBuffer = Buffer.from(arrayBuffer);
-            writeFileSync(imagePath, imageBuffer);
-            return {
-              success: true,
-              name: sanitizedName
-            };
-          } catch (error) {
-            output.warn(`Failed to download ${sanitizedName}: ${error.message}`);
-            return {
-              success: false,
-              name: sanitizedName,
-              error: error.message
-            };
-          }
-        });
-        let batchResults = await Promise.all(downloadPromises);
-        let batchSuccesses = batchResults.filter(r => r.success).length;
-        let batchFailures = batchResults.filter(r => !r.success).length;
-        downloadedCount += batchSuccesses;
-        errorCount += batchFailures;
-      }
-    }
-    if (downloadedCount === 0 && skippedCount === 0) {
-      output.error('No screenshots were successfully downloaded');
-      return null;
-    }
-
-    // Store baseline metadata
-    this.baselineData = {
-      buildId: baselineBuild.id,
-      buildName: baselineBuild.name,
-      environment: 'test',
-      branch: null,
+    let installedBaseline = await installBaselineBuild({
+      baselineBuild,
+      baselinePath: this.baselinePath,
+      workingDir: this.workingDir,
       threshold: this.threshold,
       signatureProperties: this.signatureProperties,
-      createdAt: new Date().toISOString(),
-      buildInfo: {
-        commitSha: baselineBuild.commit_sha,
-        commitMessage: baselineBuild.commit_message,
-        visual_review: baselineBuild.visual_review || null,
-        completedAt: baselineBuild.completed_at
-      },
-      screenshots: buildDetails.screenshots.filter(s => s.filename).map(s => ({
-        name: sanitizeScreenshotName(s.name),
-        originalName: s.name,
-        sha256: s.sha256,
-        id: s.id,
-        filename: s.filename,
-        path: safePath(this.baselinePath, s.filename),
-        browser: s.browser,
-        viewport_width: s.viewport_width,
-        originalUrl: s.original_url,
-        fileSize: s.file_size_bytes,
-        dimensions: {
-          width: s.width,
-          height: s.height
-        }
-      }))
-    };
-    saveBaselineMetadata(this.baselinePath, this.baselineData);
-
-    // Save baseline build metadata for MCP plugin
-    let baselineMetadataPath = safePath(this.workingDir, '.vizzly', 'baseline-metadata.json');
-    writeFileSync(baselineMetadataPath, JSON.stringify({
-      buildId: baselineBuild.id,
-      buildName: baselineBuild.name,
-      branch: null,
-      environment: 'test',
-      commitSha: baselineBuild.commit_sha,
-      commitMessage: baselineBuild.commit_message,
-      visual_review: baselineBuild.visual_review || null,
-      completedAt: baselineBuild.completed_at,
-      downloadedAt: new Date().toISOString()
-    }, null, 2));
-
-    // Summary
-    let actualDownloads = downloadedCount - skippedCount;
-    if (skippedCount > 0) {
-      if (actualDownloads === 0) {
-        output.info(`All ${skippedCount} baselines up-to-date`);
-      } else {
-        output.info(`Downloaded ${actualDownloads} new screenshots, ${skippedCount} already up-to-date`);
+      existingBaseline,
+      environment,
+      branch,
+      output,
+      fetchWithTimeout,
+      operations: {
+        existsSync,
+        writeFileSync,
+        safePath,
+        sanitizeScreenshotName,
+        saveBaselineMetadata
       }
-    } else {
-      output.info(`Downloaded ${downloadedCount}/${buildDetails.screenshots.length} screenshots successfully`);
-    }
-    if (errorCount > 0) {
-      output.warn(`${errorCount} screenshots failed to download`);
-    }
-    return this.baselineData;
+    });
+    if (installedBaseline) this.baselineData = installedBaseline;
+    return installedBaseline;
   }
 
   /**
@@ -1079,6 +763,26 @@ export class TddService {
     }
     return results;
   }
+  _writeBaselineScreenshot(screenshot, imageBuffer) {
+    let {
+      writeFileSync,
+      createEmptyBaselineMetadata,
+      upsertScreenshotInMetadata
+    } = this._deps;
+    this.baselineData = writeBaselineScreenshot({
+      imageBuffer,
+      screenshot,
+      metadata: this.baselineData,
+      baselineOptions: {
+        threshold: this.threshold,
+        signatureProperties: this.signatureProperties
+      }
+    }, {
+      writeFileSync,
+      createEmptyBaselineMetadata,
+      upsertScreenshotInMetadata
+    });
+  }
 
   /**
    * Update all baselines with current screenshots
@@ -1094,9 +798,7 @@ export class TddService {
       getBaselinePath,
       existsSync,
       readFileSync,
-      writeFileSync,
       createEmptyBaselineMetadata,
-      upsertScreenshotInMetadata,
       saveBaselineMetadata
     } = this._deps;
     if (this.comparisons.length === 0) {
@@ -1132,14 +834,12 @@ export class TddService {
       let baselineImagePath = getBaselinePath(this.baselinePath, filename);
       try {
         let currentBuffer = readFileSync(current);
-        writeFileSync(baselineImagePath, currentBuffer);
-        let screenshotEntry = {
+        this._writeBaselineScreenshot({
           name: sanitizedName,
           properties: validatedProperties,
           path: baselineImagePath,
           signature
-        };
-        upsertScreenshotInMetadata(this.baselineData, screenshotEntry, signature);
+        }, currentBuffer);
         updatedCount++;
         output.info(`Updated baseline for ${sanitizedName}`);
       } catch (error) {
@@ -1171,9 +871,6 @@ export class TddService {
       existsSync,
       readFileSync,
       mkdirSync,
-      writeFileSync,
-      createEmptyBaselineMetadata,
-      upsertScreenshotInMetadata,
       saveBaselineMetadata
     } = this._deps;
     let comparison;
@@ -1219,22 +916,12 @@ export class TddService {
 
     // Update the baseline (filename already includes .png extension)
     let baselineImagePath = safePath(this.baselinePath, filename);
-    writeFileSync(baselineImagePath, imageBuffer);
-
-    // Update baseline metadata
-    if (!this.baselineData) {
-      this.baselineData = createEmptyBaselineMetadata({
-        threshold: this.threshold,
-        signatureProperties: this.signatureProperties
-      });
-    }
-    let screenshotEntry = {
+    this._writeBaselineScreenshot({
       name: sanitizedName,
       properties,
       path: baselineImagePath,
       signature
-    };
-    upsertScreenshotInMetadata(this.baselineData, screenshotEntry, signature);
+    }, imageBuffer);
     saveBaselineMetadata(this.baselinePath, this.baselineData);
     return {
       name: sanitizedName,
@@ -1253,27 +940,16 @@ export class TddService {
       output,
       generateScreenshotSignature,
       generateComparisonId,
-      writeFileSync,
-      createEmptyBaselineMetadata,
-      upsertScreenshotInMetadata,
       saveBaselineMetadata
     } = this._deps;
     output.info(`Creating baseline for ${name}`);
-    writeFileSync(baselineImagePath, imageBuffer);
-    if (!this.baselineData) {
-      this.baselineData = createEmptyBaselineMetadata({
-        threshold: this.threshold,
-        signatureProperties: this.signatureProperties
-      });
-    }
     let signature = generateScreenshotSignature(name, properties || {}, this.signatureProperties);
-    let screenshotEntry = {
+    this._writeBaselineScreenshot({
       name,
       properties: properties || {},
       path: baselineImagePath,
       signature
-    };
-    upsertScreenshotInMetadata(this.baselineData, screenshotEntry, signature);
+    }, imageBuffer);
     saveBaselineMetadata(this.baselinePath, this.baselineData);
     let result = {
       id: generateComparisonId(signature),