@vizzly-testing/cli 0.35.3-beta.1 → 0.35.3-beta.3

package/dist/context/local-workspace-provider.js

@@ -43,7 +43,7 @@ function mapComparisonResult(status) {
   }
   return status || 'unknown';
 }
-function mapApprovalStatus(status) {
+function mapReviewState(status) {
   if (status === 'failed' || status === 'new') {
     return 'pending';
   }
@@ -88,50 +88,6 @@ function resolveAssetReference(assetPath, snapshot) {
   }
   return assetPath;
 }
-function normalizeConfirmedRegions(regions = []) {
-  return regions.map((region, index) => ({
-    id: region.id || `local-region-${index}`,
-    x1: region.x1 ?? region.x ?? null,
-    y1: region.y1 ?? region.y ?? null,
-    x2: region.x2 ?? (region.x != null && region.width != null ? region.x + region.width : null),
-    y2: region.y2 ?? (region.y != null && region.height != null ? region.y + region.height : null),
-    label: region.label || null
-  }));
-}
-function mergeConfirmedRegions(snapshot, comparisonName, details = {}) {
-  let workspaceRegions = snapshot.regions?.[comparisonName]?.confirmed || [];
-  let detailRegions = details.confirmedRegions || [];
-  let merged = [...workspaceRegions, ...detailRegions];
-  let seen = new Set();
-  return normalizeConfirmedRegions(merged).filter(region => {
-    let key = `${region.label || ''}:${region.x1}:${region.y1}:${region.x2}:${region.y2}`;
-    if (seen.has(key)) {
-      return false;
-    }
-    seen.add(key);
-    return true;
-  });
-}
-function buildHotspotAnalysis(snapshot, comparisonName, details = {}) {
-  let hotspotMetadata = snapshot.hotspots?.[comparisonName] || null;
-  let hotspotAnalysis = details.hotspotAnalysis || null;
-  if (!hotspotMetadata && !hotspotAnalysis) {
-    return {
-      regions: [],
-      total_builds_analyzed: 0,
-      confidence: 'no_data',
-      confidence_score: null,
-      data_source: 'local_workspace'
-    };
-  }
-  return {
-    regions: hotspotMetadata?.regions || [],
-    total_builds_analyzed: 1,
-    confidence: hotspotAnalysis?.confidence || hotspotMetadata?.confidence || 'workspace',
-    confidence_score: hotspotAnalysis?.confidenceScore || null,
-    data_source: 'local_workspace'
-  };
-}
 function buildComparisonLinks(snapshot, comparisonId) {
   if (!snapshot.serverInfo?.port) {
     return {};
@@ -160,7 +116,7 @@ function buildBuildSnapshot(snapshot) {
     branch: 'local',
     commit_sha: null,
     commit_message: null,
-    approval_status: snapshot.serverInfo ? 'pending' : 'approved',
+    review_state: snapshot.serverInfo ? 'pending' : 'approved',
     status: snapshot.serverInfo ? 'running' : 'completed',
     created_at: null
   };
@@ -181,7 +137,7 @@ function buildBaselineSnapshot(snapshot) {
       branch: metadata.branch || 'local',
       commit_sha: metadata.buildInfo?.commitSha || null,
       commit_message: metadata.buildInfo?.commitMessage || null,
-      approval_status: metadata.buildInfo?.approvalStatus || 'approved',
+      review_state: metadata.buildInfo?.visual_review?.state || 'approved',
       status: metadata.buildInfo?.completedAt ? 'completed' : 'local',
       created_at: metadata.createdAt || null,
       completed_at: metadata.buildInfo?.completedAt || null
@@ -192,7 +148,7 @@ function buildBaselineSnapshot(snapshot) {
 }
 function buildReviewState(build, reviewSummary) {
   let reasons = [];
-  if (build.approval_status === 'pending') {
+  if (build.review_state === 'pending') {
     reasons.push('build_pending_approval');
   }
   if (reviewSummary.pending > 0) {
@@ -208,12 +164,10 @@ function buildReviewState(build, reviewSummary) {
 function mapLocalComparison(snapshot, comparison) {
   let details = snapshot.comparisonDetails[comparison.id] || {};
   let comparisonName = comparison.originalName || comparison.name;
-  let confirmedRegions = mergeConfirmedRegions(snapshot, comparisonName, details);
-  let hotspotAnalysis = buildHotspotAnalysis(snapshot, comparisonName, details);
   let properties = comparison.properties || {};
   let buildSnapshot = buildBuildSnapshot(snapshot);
   let result = mapComparisonResult(comparison.status);
-  let approvalStatus = mapApprovalStatus(comparison.status);
+  let reviewState = mapReviewState(comparison.status);
   let baselineBuildId = snapshot.baselineMetadata?.buildId || null;
   let diffImageUrl = resolveAssetReference(comparison.diff, snapshot);
   let diffRegions = details.diffClusters || [];
@@ -223,8 +177,8 @@ function mapLocalComparison(snapshot, comparison) {
     screenshot_name: comparisonName,
     status: comparison.status,
     result,
-    approval_status: approvalStatus,
-    needs_review: approvalStatus === 'pending' && ['changed', 'new'].includes(result),
+    review_state: reviewState,
+    needs_review: reviewState === 'pending' && ['changed', 'new'].includes(result),
     build_id: buildSnapshot.id,
     build_name: buildSnapshot.name,
     build_branch: buildSnapshot.branch,
@@ -275,14 +229,11 @@ function mapLocalComparison(snapshot, comparison) {
       } : null,
       diff_lines: null,
       fingerprint_hash: null,
-      fingerprint_data: null,
-      hotspot_analysis: hotspotAnalysis,
-      region_analysis: details.regionAnalysis || null,
-      confirmed_regions: confirmedRegions
+      fingerprint_data: null
     }
   };
 }
-function projectLocalScreenshot(screenshot, baseline) {
+function formatLocalScreenshot(screenshot, baseline) {
   if (!screenshot) return null;
   return {
     id: screenshot.id,
@@ -306,8 +257,8 @@ function projectLocalScreenshot(screenshot, baseline) {
     } : null
   };
 }
-function projectLocalDiff(diff, includeDiffs) {
-  let projected = {
+function formatLocalDiff(diff, includeDiffs) {
+  let formatted = {
     percentage: diff?.percentage ?? null,
     changed_pixels: diff?.changed_pixels ?? null,
     total_pixels: diff?.total_pixels ?? null,
@@ -317,32 +268,32 @@ function projectLocalDiff(diff, includeDiffs) {
     region_count: Array.isArray(diff?.regions) ? diff.regions.length : null
   };
   if (includeDiffs) {
-    projected.regions = diff?.regions || [];
-    projected.cluster_metadata = diff?.cluster_metadata ?? null;
-    projected.fingerprint_data = diff?.fingerprint_data ?? null;
-    projected.diff_lines = diff?.diff_lines ?? [];
+    formatted.regions = diff?.regions || [];
+    formatted.cluster_metadata = diff?.cluster_metadata ?? null;
+    formatted.fingerprint_data = diff?.fingerprint_data ?? null;
+    formatted.diff_lines = diff?.diff_lines ?? [];
   }
-  return projected;
+  return formatted;
 }
-function projectLocalEvidence(comparison, includeDiffs) {
+function formatLocalEvidence(comparison, includeDiffs) {
   return {
     type: 'comparison',
     id: comparison.id,
     screenshot_name: comparison.screenshot_name,
     status: comparison.status,
     result: comparison.result,
-    approval_status: comparison.approval_status,
+    review_state: comparison.review_state,
     needs_review: comparison.needs_review,
     build_id: comparison.build_id,
     build_name: comparison.build_name,
     build_branch: comparison.build_branch,
     build_commit_sha: comparison.build_commit_sha,
     build_created_at: comparison.build_created_at,
-    screenshot: projectLocalScreenshot(comparison.screenshot, comparison.baseline),
-    diff: projectLocalDiff(comparison.diff, includeDiffs)
+    screenshot: formatLocalScreenshot(comparison.screenshot, comparison.baseline),
+    diff: formatLocalDiff(comparison.diff, includeDiffs)
   };
 }
-function projectLocalFocusedComparison(comparison, includeDiffs) {
+function formatLocalFocusedComparison(comparison, includeDiffs) {
   if (includeDiffs) return comparison;
   let analysis = comparison.analysis || {};
   let {
@@ -354,11 +305,11 @@ function projectLocalFocusedComparison(comparison, includeDiffs) {
   } = analysis;
   return {
     ...comparison,
-    diff: projectLocalDiff(comparison.diff, false),
+    diff: formatLocalDiff(comparison.diff, false),
     analysis: summaryAnalysis
   };
 }
-function projectLocalHistoryItem(comparison) {
+function formatLocalHistoryItem(comparison) {
   return {
     id: comparison.id,
     screenshot_name: comparison.screenshot_name,
@@ -369,7 +320,7 @@ function projectLocalHistoryItem(comparison) {
     build_name: comparison.build_name,
     build_branch: comparison.build_branch,
     build_created_at: comparison.build_created_at,
-    screenshot: projectLocalScreenshot(comparison.screenshot, comparison.baseline),
+    screenshot: formatLocalScreenshot(comparison.screenshot, comparison.baseline),
     diff: {
       percentage: comparison.diff?.percentage ?? null,
       fingerprint_hash: comparison.diff?.fingerprint_hash ?? null,
@@ -384,43 +335,19 @@ function compactLocalCollection(items) {
     details_available: Array.isArray(items) && items.length > 0
   };
 }
-function createLocalDynamicRegions(comparison = null) {
-  let confirmedRegions = comparison?.analysis?.confirmed_regions;
-  let hotspotAnalysis = comparison?.analysis?.hotspot_analysis;
-  return {
-    decision: null,
-    patterns: compactLocalCollection([]),
-    confirmed_regions: compactLocalCollection(confirmedRegions),
-    exclusions: {
-      total: null,
-      included: false,
-      details_available: false
-    },
-    hotspot_analysis: hotspotAnalysis ? {
-      total_builds_analyzed: hotspotAnalysis.total_builds_analyzed ?? null,
-      confidence: hotspotAnalysis.confidence ?? null,
-      confidence_score: hotspotAnalysis.confidence_score ?? null,
-      data_source: hotspotAnalysis.data_source ?? null,
-      coverage: hotspotAnalysis.coverage ?? null,
-      confirmed_region_coverage: hotspotAnalysis.confirmed_region_coverage ?? null
-    } : null
-  };
-}
 function createLocalSnapshotRevision(snapshot) {
   let revisionInput = {
     serverInfo: snapshot.serverInfo,
     reportData: snapshot.reportData,
     comparisonDetails: snapshot.comparisonDetails,
-    baselineMetadata: snapshot.baselineMetadata,
-    hotspotFile: snapshot.hotspotFile,
-    regionFile: snapshot.regionFile
+    baselineMetadata: snapshot.baselineMetadata
   };
   return createHash('sha256').update(JSON.stringify(revisionInput)).digest('base64url');
 }
 function buildReviewSummary(comparisons = []) {
-  let approved = comparisons.filter(comparison => mapApprovalStatus(comparison.status) === 'approved').length;
-  let rejected = comparisons.filter(comparison => mapApprovalStatus(comparison.status) === 'rejected').length;
-  let pending = comparisons.filter(comparison => mapApprovalStatus(comparison.status) === 'pending').length;
+  let approved = comparisons.filter(comparison => mapReviewState(comparison.status) === 'approved').length;
+  let rejected = comparisons.filter(comparison => mapReviewState(comparison.status) === 'rejected').length;
+  let pending = comparisons.filter(comparison => mapReviewState(comparison.status) === 'pending').length;
   return {
     total: comparisons.length,
     pending,
@@ -457,12 +384,8 @@ export function createLocalWorkspaceContextProvider(options = {}, deps = {}) {
       serverInfo: readJson(join(vizzlyDir, 'server.json')),
       reportData: normalizeReportData(readJson(join(vizzlyDir, 'report-data.json')) || createEmptyReportData()),
       comparisonDetails: readJson(join(vizzlyDir, 'comparison-details.json')) || {},
-      baselineMetadata: readJson(join(vizzlyDir, 'baselines', 'metadata.json')),
-      hotspotFile: readJson(join(vizzlyDir, 'hotspots.json')),
-      regionFile: readJson(join(vizzlyDir, 'regions.json'))
+      baselineMetadata: readJson(join(vizzlyDir, 'baselines', 'metadata.json'))
     };
-    snapshotCache.hotspots = snapshotCache.hotspotFile?.hotspots || null;
-    snapshotCache.regions = snapshotCache.regionFile?.regions || null;
     return snapshotCache;
   }
   function isAvailable(snapshot = loadSnapshot()) {
@@ -486,7 +409,7 @@ export function createLocalWorkspaceContextProvider(options = {}, deps = {}) {
       return Boolean(findComparison(snapshot, target));
     }
     if (command === 'screenshot') {
-      return Boolean(snapshot.reportData.comparisons.some(comparison => (comparison.originalName || comparison.name) === target) || snapshot.regions?.[target] || snapshot.hotspots?.[target]);
+      return Boolean(snapshot.reportData.comparisons.some(comparison => (comparison.originalName || comparison.name) === target));
     }
     if (command === 'review-queue') {
       return snapshot.reportData.comparisons.length > 0;
@@ -572,7 +495,7 @@ export function createLocalWorkspaceContextProvider(options = {}, deps = {}) {
       throw createLocalWorkspaceError(`Local workspace context is only available for the active local build (${resolvedBuild.id})`);
     }
     let mappedComparisons = snapshot.reportData.comparisons.map(comparison => mapLocalComparison(snapshot, comparison));
-    let mappedScreenshots = mappedComparisons.map(comparison => projectLocalScreenshot(comparison.screenshot, comparison.baseline));
+    let mappedScreenshots = mappedComparisons.map(comparison => formatLocalScreenshot(comparison.screenshot, comparison.baseline));
     let reviewSummary = buildReviewSummary(snapshot.reportData.comparisons);
     let reviewState = buildReviewState(resolvedBuild, reviewSummary);
     let context = {
@@ -612,7 +535,7 @@ export function createLocalWorkspaceContextProvider(options = {}, deps = {}) {
     let cursorQuery = `${query.details}:${limit}`;
     let cursor = readLocalCursor(query.cursor, 'build_context', resolvedBuild.id, cursorQuery, revision, ['evidence']);
     let includeDiffs = query.details === 'diffs';
-    let evidence = mappedComparisons.map(comparison => projectLocalEvidence(comparison, includeDiffs));
+    let evidence = mappedComparisons.map(comparison => formatLocalEvidence(comparison, includeDiffs));
     return {
       resource: 'build_context',
       source: LOCAL_CONTEXT_SOURCE,
@@ -623,14 +546,6 @@ export function createLocalWorkspaceContextProvider(options = {}, deps = {}) {
       summary: context.summary,
       preview: null,
       signature_properties: [],
-      dynamic_regions: {
-        exclusions: {
-          total: null,
-          included: false,
-          details_available: false
-        },
-        item_details_included: false
-      },
       evidence: createLocalPage(evidence, {
         limit,
         offset: cursor?.offset || 0,
@@ -664,9 +579,7 @@ export function createLocalWorkspaceContextProvider(options = {}, deps = {}) {
       comparison: mappedComparison,
       history: {
         similar_by_fingerprint: [],
-        recent_by_name: history,
-        hotspot_analysis: buildHotspotAnalysis(snapshot, comparisonName, snapshot.comparisonDetails[comparison.id] || {}),
-        confirmed_regions: mergeConfirmedRegions(snapshot, comparisonName, snapshot.comparisonDetails[comparison.id] || {})
+        recent_by_name: history
       },
       review: {
         review_summary: {
@@ -699,8 +612,7 @@ export function createLocalWorkspaceContextProvider(options = {}, deps = {}) {
       scope: context.scope,
       build: context.build,
       signature_properties: [],
-      comparison: projectLocalFocusedComparison(context.comparison, query.details === 'diffs'),
-      dynamic_regions: createLocalDynamicRegions(context.comparison),
+      comparison: formatLocalFocusedComparison(context.comparison, query.details === 'diffs'),
       history: {
         active_stream: activeStream || null,
         similar_by_fingerprint: createLocalPage([], {
@@ -712,7 +624,7 @@ export function createLocalWorkspaceContextProvider(options = {}, deps = {}) {
           stream: 'similar_by_fingerprint',
           revision
         }),
-        recent_by_name: createLocalPage(history.map(projectLocalHistoryItem), {
+        recent_by_name: createLocalPage(history.map(formatLocalHistoryItem), {
           limit,
           offset: recentOffset,
           resource: 'comparison_context',
@@ -735,7 +647,7 @@ export function createLocalWorkspaceContextProvider(options = {}, deps = {}) {
   function getScreenshotContext(screenshotName) {
     let snapshot = loadSnapshot();
     let matches = snapshot.reportData.comparisons.filter(comparison => (comparison.originalName || comparison.name) === screenshotName).map(comparison => mapLocalComparison(snapshot, comparison));
-    if (matches.length === 0 && !snapshot.regions?.[screenshotName] && !snapshot.hotspots?.[screenshotName]) {
+    if (matches.length === 0) {
       throw createLocalWorkspaceError(`No local screenshot context found for "${screenshotName}"`);
     }
     return {
@@ -745,8 +657,6 @@ export function createLocalWorkspaceContextProvider(options = {}, deps = {}) {
       screenshot: {
         name: screenshotName
       },
-      hotspot_analysis: buildHotspotAnalysis(snapshot, screenshotName),
-      confirmed_regions: mergeConfirmedRegions(snapshot, screenshotName),
       history: {
         recent_comparisons: matches
       }