@vizzly-testing/cli 0.30.0 → 0.32.0

package/dist/commands/run.js

@@ -8,12 +8,44 @@ import { createBuild as defaultCreateApiBuild, createApiClient as defaultCreateA
 import { VizzlyError } from '../errors/vizzly-error.js';
 import { createServerManager as defaultCreateServerManager } from '../server-manager/index.js';
 import { createBuildObject as defaultCreateBuildObject } from '../services/build-manager.js';
-import { createUploader as defaultCreateUploader } from '../services/uploader.js';
 import { finalizeBuild as defaultFinalizeBuild, runTests as defaultRunTests } from '../test-runner/index.js';
+import { createUploader as defaultCreateUploader } from '../uploader/index.js';
+import { getAppBaseUrl } from '../utils/api-url.js';
 import { loadConfig as defaultLoadConfig } from '../utils/config-loader.js';
 import { detectBranch as defaultDetectBranch, detectCommit as defaultDetectCommit, detectCommitMessage as defaultDetectCommitMessage, detectPullRequestNumber as defaultDetectPullRequestNumber, generateBuildNameWithGit as defaultGenerateBuildNameWithGit } from '../utils/git.js';
 import * as defaultOutput from '../utils/output.js';
 import { writeSession as defaultWriteSession } from '../utils/session.js';
+export async function resolveBuildDisplayUrl({
+  result,
+  config,
+  createApiClient = defaultCreateApiClient,
+  getTokenContext = defaultGetTokenContext
+}) {
+  if (result.url) {
+    return result.url;
+  }
+  if (!config.apiKey) {
+    return undefined;
+  }
+  let baseUrl = getAppBaseUrl(config.apiUrl);
+  try {
+    let client = createApiClient({
+      baseUrl: config.apiUrl,
+      token: config.apiKey,
+      command: 'run'
+    });
+    let tokenContext = await getTokenContext(client);
+    if (tokenContext.organization?.slug && tokenContext.project?.slug) {
+      return `${baseUrl}/${tokenContext.organization.slug}/${tokenContext.project.slug}/builds/${result.buildId}`;
+    }
+  } catch {
+    return `${baseUrl}/builds/${result.buildId}`;
+  }
+  return undefined;
+}
+function buildContextCommand(buildId) {
+  return `vizzly context build ${buildId} --agent`;
+}
 
 /**
  * Run command implementation
@@ -275,27 +307,12 @@ export async function runCommand(testCommand, options = {}, globalOptions = {},
       // JSON output mode - output structured data and exit
       if (globalOptions.json) {
         let executionTimeMs = Date.now() - startTime;
-
-        // Get URL from result, or construct one as fallback
-        let displayUrl = result.url;
-        if (!displayUrl && config.apiKey) {
-          try {
-            let client = createApiClient({
-              baseUrl: config.apiUrl,
-              token: config.apiKey,
-              command: 'run'
-            });
-            let tokenContext = await getTokenContext(client);
-            let baseUrl = config.apiUrl.replace(/\/api.*$/, '');
-            if (tokenContext.organization?.slug && tokenContext.project?.slug) {
-              displayUrl = `${baseUrl}/${tokenContext.organization.slug}/${tokenContext.project.slug}/builds/${result.buildId}`;
-            }
-          } catch {
-            // Fallback to simple URL if context fetch fails
-            let baseUrl = config.apiUrl.replace(/\/api.*$/, '');
-            displayUrl = `${baseUrl}/builds/${result.buildId}`;
-          }
-        }
+        let displayUrl = await resolveBuildDisplayUrl({
+          result,
+          config,
+          createApiClient,
+          getTokenContext
+        });
         let jsonResult = {
           buildId: result.buildId,
           status: 'completed',
@@ -307,6 +324,7 @@ export async function runCommand(testCommand, options = {}, globalOptions = {},
             commit,
             message
           },
+          contextCommand: result.buildId ? buildContextCommand(result.buildId) : null,
           exitCode: 0
         };
         output.data(jsonResult);
@@ -323,32 +341,18 @@ export async function runCommand(testCommand, options = {}, globalOptions = {},
         output.blank();
         let colors = output.getColors();
         output.print(`  ${colors.brand.textTertiary('Screenshots')}  ${colors.white(result.screenshotsCaptured)}`);
-
-        // Get URL from result, or construct one as fallback
-        let displayUrl = result.url;
-        if (!displayUrl && config.apiKey) {
-          try {
-            let client = createApiClient({
-              baseUrl: config.apiUrl,
-              token: config.apiKey,
-              command: 'run'
-            });
-            let tokenContext = await getTokenContext(client);
-            let baseUrl = config.apiUrl.replace(/\/api.*$/, '');
-            if (tokenContext.organization?.slug && tokenContext.project?.slug) {
-              displayUrl = `${baseUrl}/${tokenContext.organization.slug}/${tokenContext.project.slug}/builds/${result.buildId}`;
-            }
-          } catch {
-            // Fallback to simple URL if context fetch fails
-            let baseUrl = config.apiUrl.replace(/\/api.*$/, '');
-            displayUrl = `${baseUrl}/builds/${result.buildId}`;
-          }
-        }
+        let displayUrl = await resolveBuildDisplayUrl({
+          result,
+          config,
+          createApiClient,
+          getTokenContext
+        });
         if (displayUrl) {
           output.print(`  ${colors.brand.textTertiary('Results')}      ${colors.cyan(colors.underline(displayUrl))}`);
         } else {
           output.print(`  ${colors.brand.textTertiary('Build')}        ${colors.dim(result.buildId)}`);
         }
+        output.print(`  ${colors.brand.textTertiary('Context')}     ${colors.dim(buildContextCommand(result.buildId))}`);
       }
     } catch (error) {
       // Test execution failed - build should already be finalized by test runner
@@ -358,7 +362,7 @@ export async function runCommand(testCommand, options = {}, globalOptions = {},
       if (error.code === 'TEST_COMMAND_FAILED' || error.code === 'TEST_COMMAND_INTERRUPTED') {
         // Extract exit code from error message if available
         let exitCodeMatch = error.message.match(/exited with code (\d+)/);
-        let exitCode = exitCodeMatch ? parseInt(exitCodeMatch[1], 10) : 1;
+        let exitCode = exitCodeMatch ? Number(exitCodeMatch[1]) : 1;
 
         // JSON output for test command failure
         if (globalOptions.json) {
@@ -427,26 +431,12 @@ export async function runCommand(testCommand, options = {}, globalOptions = {},
         // JSON output for --wait mode
         if (globalOptions.json) {
           let executionTimeMs = Date.now() - startTime;
-
-          // Get URL from result, or construct one as fallback
-          let displayUrl = result.url;
-          if (!displayUrl && config.apiKey) {
-            try {
-              let client = createApiClient({
-                baseUrl: config.apiUrl,
-                token: config.apiKey,
-                command: 'run'
-              });
-              let tokenContext = await getTokenContext(client);
-              let baseUrl = config.apiUrl.replace(/\/api.*$/, '');
-              if (tokenContext.organization?.slug && tokenContext.project?.slug) {
-                displayUrl = `${baseUrl}/${tokenContext.organization.slug}/${tokenContext.project.slug}/builds/${result.buildId}`;
-              }
-            } catch {
-              let baseUrl = config.apiUrl.replace(/\/api.*$/, '');
-              displayUrl = `${baseUrl}/builds/${result.buildId}`;
-            }
-          }
+          let displayUrl = await resolveBuildDisplayUrl({
+            result,
+            config,
+            createApiClient,
+            getTokenContext
+          });
           let exitCode = buildResult.failedComparisons > 0 ? 1 : 0;
           let jsonResult = {
             buildId: result.buildId,
@@ -466,6 +456,7 @@ export async function runCommand(testCommand, options = {}, globalOptions = {},
               identical: buildResult.identicalComparisons || 0
             },
             approvalStatus: buildResult.approvalStatus || 'pending',
+            contextCommand: buildContextCommand(result.buildId),
             exitCode
           };
           output.data(jsonResult);
@@ -544,28 +535,40 @@ export function validateRunOptions(testCommand, options) {
     errors.push('Test command is required');
   }
   if (options.port) {
-    let port = parseInt(options.port, 10);
-    if (Number.isNaN(port) || port < 1 || port > 65535) {
+    let port = Number(options.port);
+    if (!Number.isInteger(port) || port < 1 || port > 65535) {
       errors.push('Port must be a valid number between 1 and 65535');
     }
   }
   if (options.timeout) {
-    let timeout = parseInt(options.timeout, 10);
-    if (Number.isNaN(timeout) || timeout < 1000) {
+    let timeout = Number(options.timeout);
+    if (!Number.isInteger(timeout) || timeout < 1000) {
       errors.push('Timeout must be at least 1000 milliseconds');
     }
   }
   if (options.batchSize !== undefined) {
-    let n = parseInt(options.batchSize, 10);
-    if (!Number.isFinite(n) || n <= 0) {
+    let n = Number(options.batchSize);
+    if (!Number.isInteger(n) || n <= 0) {
       errors.push('Batch size must be a positive integer');
     }
   }
   if (options.uploadTimeout !== undefined) {
-    let n = parseInt(options.uploadTimeout, 10);
-    if (!Number.isFinite(n) || n <= 0) {
+    let n = Number(options.uploadTimeout);
+    if (!Number.isInteger(n) || n <= 0) {
       errors.push('Upload timeout must be a positive integer (milliseconds)');
     }
   }
+  if (options.threshold !== undefined) {
+    let threshold = Number(options.threshold);
+    if (!Number.isFinite(threshold) || threshold < 0) {
+      errors.push('Threshold must be a non-negative number (CIEDE2000 Delta E)');
+    }
+  }
+  if (options.minClusterSize !== undefined) {
+    let minClusterSize = Number(options.minClusterSize);
+    if (!Number.isInteger(minClusterSize) || minClusterSize < 1) {
+      errors.push('Min cluster size must be a positive integer');
+    }
+  }
   return errors;
 }

package/dist/commands/status.js

@@ -4,9 +4,204 @@
  */
 
 import { createApiClient as defaultCreateApiClient, getBuild as defaultGetBuild, getPreviewInfo as defaultGetPreviewInfo } from '../api/index.js';
+import { getAppBaseUrl } from '../utils/api-url.js';
 import { loadConfig as defaultLoadConfig } from '../utils/config-loader.js';
 import { getApiUrl as defaultGetApiUrl } from '../utils/environment-config.js';
 import * as defaultOutput from '../utils/output.js';
+function createStatusDeps(deps = {}) {
+  return {
+    loadConfig: deps.loadConfig || defaultLoadConfig,
+    createApiClient: deps.createApiClient || defaultCreateApiClient,
+    getBuild: deps.getBuild || defaultGetBuild,
+    getPreviewInfo: deps.getPreviewInfo || defaultGetPreviewInfo,
+    getApiUrl: deps.getApiUrl || defaultGetApiUrl,
+    output: deps.output || defaultOutput,
+    exit: deps.exit || (code => process.exit(code))
+  };
+}
+function configureOutput(output, globalOptions) {
+  output.configure({
+    json: globalOptions.json,
+    verbose: globalOptions.verbose,
+    color: !globalOptions.noColor
+  });
+}
+function createStatusClient({
+  createApiClient,
+  config
+}) {
+  return createApiClient({
+    baseUrl: config.apiUrl,
+    token: config.apiKey,
+    command: 'status'
+  });
+}
+export function normalizeBuildStatus(buildStatus) {
+  return buildStatus.build || buildStatus;
+}
+export function createStatusData(build, previewInfo = null) {
+  return {
+    buildId: build.id,
+    status: build.status,
+    name: build.name,
+    createdAt: build.created_at,
+    updatedAt: build.updated_at,
+    completedAt: build.completed_at,
+    environment: build.environment,
+    branch: build.branch,
+    commit: build.commit_sha,
+    commitMessage: build.commit_message,
+    screenshotsTotal: build.screenshot_count || 0,
+    comparisonsTotal: build.total_comparisons || 0,
+    newComparisons: build.new_comparisons || 0,
+    changedComparisons: build.changed_comparisons || 0,
+    identicalComparisons: build.identical_comparisons || 0,
+    approvalStatus: build.approval_status,
+    executionTime: build.execution_time_ms,
+    isBaseline: build.is_baseline,
+    userAgent: build.user_agent,
+    preview: previewInfo ? {
+      url: previewInfo.preview_url,
+      status: previewInfo.status,
+      fileCount: previewInfo.file_count,
+      expiresAt: previewInfo.expires_at
+    } : null
+  };
+}
+export function createBuildInfo(build) {
+  let buildInfo = {
+    Name: build.name || build.id,
+    Status: build.status.toUpperCase(),
+    Environment: build.environment
+  };
+  if (build.branch) {
+    buildInfo.Branch = build.branch;
+  }
+  if (build.commit_sha) {
+    buildInfo.Commit = `${build.commit_sha.substring(0, 8)} - ${build.commit_message || 'No message'}`;
+  }
+  return buildInfo;
+}
+export function createComparisonStats(build, colors) {
+  let stats = [];
+  let newCount = build.new_comparisons || 0;
+  let changedCount = build.changed_comparisons || 0;
+  let identicalCount = build.identical_comparisons || 0;
+  if (newCount > 0) {
+    stats.push(`${colors.brand.info(newCount)} new`);
+  }
+  if (changedCount > 0) {
+    stats.push(`${colors.brand.warning(changedCount)} changed`);
+  }
+  if (identicalCount > 0) {
+    stats.push(`${colors.brand.success(identicalCount)} identical`);
+  }
+  return stats.join(colors.brand.textMuted(' · '));
+}
+export function createBuildUrl(baseUrl, build) {
+  if (!baseUrl || !build.project_id) {
+    return null;
+  }
+  return `${getAppBaseUrl(baseUrl)}/projects/${build.project_id}/builds/${build.id}`;
+}
+export function shouldFailStatus(build) {
+  return build.status === 'failed' || build.failed_jobs > 0;
+}
+export function getProcessingProgress(build) {
+  if (build.status !== 'processing' && build.status !== 'pending') {
+    return null;
+  }
+  let completedJobs = build.completed_jobs || 0;
+  let failedJobs = build.failed_jobs || 0;
+  let processingScreenshots = build.processing_screenshots || 0;
+  let totalJobs = completedJobs + failedJobs + processingScreenshots;
+  if (totalJobs <= 0) {
+    return null;
+  }
+  return (completedJobs + failedJobs) / totalJobs * 100;
+}
+function writeStatusTiming({
+  build,
+  output
+}) {
+  if (build.created_at) {
+    output.hint(`Created ${new Date(build.created_at).toLocaleString()}`);
+  }
+  if (build.completed_at) {
+    output.hint(`Completed ${new Date(build.completed_at).toLocaleString()}`);
+  } else if (build.status !== 'completed' && build.status !== 'failed') {
+    output.hint(`Started ${new Date(build.started_at || build.created_at).toLocaleString()}`);
+  }
+  if (build.execution_time_ms) {
+    output.hint(`Took ${Math.round(build.execution_time_ms / 1000)}s`);
+  }
+}
+function createVerboseInfo(build) {
+  let verboseInfo = {};
+  if (build.approved_screenshots > 0 || build.rejected_screenshots > 0 || build.pending_screenshots > 0) {
+    verboseInfo.Approvals = `${build.approved_screenshots || 0} approved, ${build.rejected_screenshots || 0} rejected, ${build.pending_screenshots || 0} pending`;
+  }
+  if (build.avg_diff_percentage != null) {
+    verboseInfo['Avg Diff'] = `${(build.avg_diff_percentage * 100).toFixed(2)}%`;
+  }
+  if (build.github_pull_request_number) {
+    verboseInfo['GitHub PR'] = `#${build.github_pull_request_number}`;
+  }
+  if (build.is_baseline) {
+    verboseInfo.Baseline = 'Yes';
+  }
+  verboseInfo['User Agent'] = build.user_agent || 'Unknown';
+  verboseInfo['Build ID'] = build.id;
+  verboseInfo['Project ID'] = build.project_id;
+  return verboseInfo;
+}
+function writeHumanStatus({
+  build,
+  buildUrl,
+  globalOptions,
+  output,
+  previewInfo
+}) {
+  output.header('status', build.status);
+  output.keyValue(createBuildInfo(build));
+  output.blank();
+  let colors = output.getColors();
+  let comparisonStats = createComparisonStats(build, colors);
+  output.labelValue('Screenshots', String(build.screenshot_count || 0));
+  if (comparisonStats) {
+    output.labelValue('Comparisons', comparisonStats);
+  }
+  if (build.approval_status) {
+    output.labelValue('Approval', build.approval_status);
+  }
+  output.blank();
+  writeStatusTiming({
+    build,
+    output
+  });
+  if (buildUrl) {
+    output.blank();
+    output.labelValue('View', output.link('Build', buildUrl));
+  }
+  if (previewInfo?.preview_url) {
+    output.labelValue('Preview', output.link('Preview', previewInfo.preview_url));
+    if (previewInfo.expires_at) {
+      let expiresDate = new Date(previewInfo.expires_at);
+      output.hint(`Preview expires ${expiresDate.toLocaleDateString()}`);
+    }
+  }
+  if (globalOptions.verbose) {
+    output.blank();
+    output.divider();
+    output.blank();
+    output.keyValue(createVerboseInfo(build));
+  }
+  let progress = getProcessingProgress(build);
+  if (progress !== null) {
+    output.blank();
+    output.print(`  ${output.progressBar(progress, 100)} ${Math.round(progress)}%`);
+  }
+}
 
 /**
  * Status command implementation
@@ -17,19 +212,15 @@ import * as defaultOutput from '../utils/output.js';
  */
 export async function statusCommand(buildId, options = {}, globalOptions = {}, deps = {}) {
   let {
-    loadConfig = defaultLoadConfig,
-    createApiClient = defaultCreateApiClient,
-    getBuild = defaultGetBuild,
-    getPreviewInfo = defaultGetPreviewInfo,
-    getApiUrl = defaultGetApiUrl,
-    output = defaultOutput,
-    exit = code => process.exit(code)
-  } = deps;
-  output.configure({
-    json: globalOptions.json,
-    verbose: globalOptions.verbose,
-    color: !globalOptions.noColor
-  });
+    loadConfig,
+    createApiClient,
+    getBuild,
+    getPreviewInfo,
+    getApiUrl,
+    output,
+    exit
+  } = createStatusDeps(deps);
+  configureOutput(output, globalOptions);
   try {
     // Load configuration with CLI overrides
     let allOptions = {
@@ -41,15 +232,21 @@ export async function statusCommand(buildId, options = {}, globalOptions = {}, d
     // Validate API token
     if (!config.apiKey) {
       output.error('API token required. Use --token or set VIZZLY_TOKEN environment variable');
+      output.cleanup();
       exit(1);
+      return {
+        success: false,
+        result: {
+          reason: 'missing_token'
+        }
+      };
     }
 
     // Get build details via functional API
     output.startSpinner('Fetching build status...');
-    let client = createApiClient({
-      baseUrl: config.apiUrl,
-      token: config.apiKey,
-      command: 'status'
+    let client = createStatusClient({
+      createApiClient,
+      config
     });
     let buildStatus = await getBuild(client, buildId);
 
@@ -58,155 +255,35 @@ export async function statusCommand(buildId, options = {}, globalOptions = {}, d
     output.stopSpinner();
 
     // Extract build data from API response
-    let build = buildStatus.build || buildStatus;
+    let build = normalizeBuildStatus(buildStatus);
 
     // Output in JSON mode
     if (globalOptions.json) {
-      let statusData = {
-        buildId: build.id,
-        status: build.status,
-        name: build.name,
-        createdAt: build.created_at,
-        updatedAt: build.updated_at,
-        completedAt: build.completed_at,
-        environment: build.environment,
-        branch: build.branch,
-        commit: build.commit_sha,
-        commitMessage: build.commit_message,
-        screenshotsTotal: build.screenshot_count || 0,
-        comparisonsTotal: build.total_comparisons || 0,
-        newComparisons: build.new_comparisons || 0,
-        changedComparisons: build.changed_comparisons || 0,
-        identicalComparisons: build.identical_comparisons || 0,
-        approvalStatus: build.approval_status,
-        executionTime: build.execution_time_ms,
-        isBaseline: build.is_baseline,
-        userAgent: build.user_agent,
-        preview: previewInfo ? {
-          url: previewInfo.preview_url,
-          status: previewInfo.status,
-          fileCount: previewInfo.file_count,
-          expiresAt: previewInfo.expires_at
-        } : null
-      };
-      output.data(statusData);
+      output.data(createStatusData(build, previewInfo));
       output.cleanup();
       return;
     }
 
     // Human-readable output
-    output.header('status', build.status);
-
-    // Build info section
-    let buildInfo = {
-      Name: build.name || build.id,
-      Status: build.status.toUpperCase(),
-      Environment: build.environment
-    };
-    if (build.branch) {
-      buildInfo.Branch = build.branch;
-    }
-    if (build.commit_sha) {
-      buildInfo.Commit = `${build.commit_sha.substring(0, 8)} - ${build.commit_message || 'No message'}`;
-    }
-    output.keyValue(buildInfo);
-    output.blank();
-
-    // Comparison stats with visual indicators
-    let colors = output.getColors();
-    let stats = [];
-    let newCount = build.new_comparisons || 0;
-    let changedCount = build.changed_comparisons || 0;
-    let identicalCount = build.identical_comparisons || 0;
-    let screenshotCount = build.screenshot_count || 0;
-    output.labelValue('Screenshots', String(screenshotCount));
-    if (newCount > 0) {
-      stats.push(`${colors.brand.info(newCount)} new`);
-    }
-    if (changedCount > 0) {
-      stats.push(`${colors.brand.warning(changedCount)} changed`);
-    }
-    if (identicalCount > 0) {
-      stats.push(`${colors.brand.success(identicalCount)} identical`);
-    }
-    if (stats.length > 0) {
-      output.labelValue('Comparisons', stats.join(colors.brand.textMuted(' · ')));
-    }
-    if (build.approval_status) {
-      output.labelValue('Approval', build.approval_status);
-    }
-    output.blank();
-
-    // Timing info
-    if (build.created_at) {
-      output.hint(`Created ${new Date(build.created_at).toLocaleString()}`);
-    }
-    if (build.completed_at) {
-      output.hint(`Completed ${new Date(build.completed_at).toLocaleString()}`);
-    } else if (build.status !== 'completed' && build.status !== 'failed') {
-      output.hint(`Started ${new Date(build.started_at || build.created_at).toLocaleString()}`);
-    }
-    if (build.execution_time_ms) {
-      output.hint(`Took ${Math.round(build.execution_time_ms / 1000)}s`);
-    }
-
     // Show build URL if we can construct it
     let baseUrl = config.baseUrl || getApiUrl();
-    if (baseUrl && build.project_id) {
-      let buildUrl = baseUrl.replace('/api', '') + `/projects/${build.project_id}/builds/${build.id}`;
-      output.blank();
-      output.labelValue('View', output.link('Build', buildUrl));
-    }
-
-    // Show preview URL if available
-    if (previewInfo?.preview_url) {
-      output.labelValue('Preview', output.link('Preview', previewInfo.preview_url));
-      if (previewInfo.expires_at) {
-        let expiresDate = new Date(previewInfo.expires_at);
-        output.hint(`Preview expires ${expiresDate.toLocaleDateString()}`);
-      }
-    }
-
-    // Show additional info in verbose mode
-    if (globalOptions.verbose) {
-      output.blank();
-      output.divider();
-      output.blank();
-      let verboseInfo = {};
-      if (build.approved_screenshots > 0 || build.rejected_screenshots > 0 || build.pending_screenshots > 0) {
-        verboseInfo.Approvals = `${build.approved_screenshots || 0} approved, ${build.rejected_screenshots || 0} rejected, ${build.pending_screenshots || 0} pending`;
-      }
-      if (build.avg_diff_percentage !== null) {
-        verboseInfo['Avg Diff'] = `${(build.avg_diff_percentage * 100).toFixed(2)}%`;
-      }
-      if (build.github_pull_request_number) {
-        verboseInfo['GitHub PR'] = `#${build.github_pull_request_number}`;
-      }
-      if (build.is_baseline) {
-        verboseInfo.Baseline = 'Yes';
-      }
-      verboseInfo['User Agent'] = build.user_agent || 'Unknown';
-      verboseInfo['Build ID'] = build.id;
-      verboseInfo['Project ID'] = build.project_id;
-      output.keyValue(verboseInfo);
-    }
-
-    // Show progress if build is still processing
-    if (build.status === 'processing' || build.status === 'pending') {
-      let totalJobs = build.completed_jobs + build.failed_jobs + build.processing_screenshots;
-      if (totalJobs > 0) {
-        let progress = (build.completed_jobs + build.failed_jobs) / totalJobs;
-        output.blank();
-        output.print(`  ${output.progressBar(progress * 100, 100)} ${Math.round(progress * 100)}%`);
-      }
-    }
+    let buildUrl = createBuildUrl(baseUrl, build);
+    writeHumanStatus({
+      build,
+      buildUrl,
+      globalOptions,
+      output,
+      previewInfo
+    });
     output.cleanup();
 
     // Exit with appropriate code based on build status
-    if (build.status === 'failed' || build.failed_jobs > 0) {
+    if (shouldFailStatus(build)) {
       exit(1);
     }
   } catch (error) {
+    output.stopSpinner();
+
     // Don't fail CI for Vizzly infrastructure issues (5xx errors)
     let status = error.context?.status;
     if (status >= 500) {
@@ -220,6 +297,7 @@ export async function statusCommand(buildId, options = {}, globalOptions = {}, d
       };
     }
     output.error('Failed to get build status', error);
+    output.cleanup();
     exit(1);
     return {
       success: false,