@vizzly-testing/cli 0.17.0 → 0.19.0

package/dist/commands/login.js

@@ -3,10 +3,10 @@
  * Authenticates user via OAuth device flow
  */
 
-import * as output from '../utils/output.js';
 import { AuthService } from '../services/auth-service.js';
-import { getApiUrl } from '../utils/environment-config.js';
 import { openBrowser } from '../utils/browser.js';
+import { getApiUrl } from '../utils/environment-config.js';
+import * as output from '../utils/output.js';
 
 /**
  * Login command implementation using OAuth device flow
@@ -19,31 +19,31 @@ export async function loginCommand(options = {}, globalOptions = {}) {
     verbose: globalOptions.verbose,
     color: !globalOptions.noColor
   });
-  let colors = output.getColors();
+  const colors = output.getColors();
   try {
     output.info('Starting Vizzly authentication...');
     output.blank();
 
     // Create auth service
-    let authService = new AuthService({
+    const authService = new AuthService({
       baseUrl: options.apiUrl || getApiUrl()
     });
 
     // Initiate device flow
     output.startSpinner('Connecting to Vizzly...');
-    let deviceFlow = await authService.initiateDeviceFlow();
+    const deviceFlow = await authService.initiateDeviceFlow();
     output.stopSpinner();
 
     // Handle both snake_case and camelCase field names
-    let verificationUri = deviceFlow.verification_uri || deviceFlow.verificationUri;
-    let userCode = deviceFlow.user_code || deviceFlow.userCode;
-    let deviceCode = deviceFlow.device_code || deviceFlow.deviceCode;
+    const verificationUri = deviceFlow.verification_uri || deviceFlow.verificationUri;
+    const userCode = deviceFlow.user_code || deviceFlow.userCode;
+    const deviceCode = deviceFlow.device_code || deviceFlow.deviceCode;
     if (!verificationUri || !userCode || !deviceCode) {
       throw new Error('Invalid device flow response from server');
     }
 
     // Build URL with pre-filled code
-    let urlWithCode = `${verificationUri}?code=${userCode}`;
+    const urlWithCode = `${verificationUri}?code=${userCode}`;
 
     // Display user code prominently
     output.blank();
@@ -61,7 +61,7 @@ export async function loginCommand(options = {}, globalOptions = {}) {
     output.blank();
 
     // Try to open browser with pre-filled code
-    let browserOpened = await openBrowser(urlWithCode);
+    const browserOpened = await openBrowser(urlWithCode);
     if (browserOpened) {
       output.info('Opening browser...');
     } else {
@@ -83,12 +83,12 @@ export async function loginCommand(options = {}, globalOptions = {}) {
 
     // Check authorization status
     output.startSpinner('Checking authorization status...');
-    let pollResponse = await authService.pollDeviceAuthorization(deviceCode);
+    const pollResponse = await authService.pollDeviceAuthorization(deviceCode);
     output.stopSpinner();
     let tokenData = null;
 
     // Check if authorization was successful by looking for tokens
-    if (pollResponse.tokens && pollResponse.tokens.accessToken) {
+    if (pollResponse.tokens?.accessToken) {
       // Success! We got tokens
       tokenData = pollResponse;
     } else if (pollResponse.status === 'pending') {
@@ -103,10 +103,10 @@ export async function loginCommand(options = {}, globalOptions = {}) {
 
     // Complete device flow and save tokens
     // Handle both snake_case and camelCase for token data, and nested tokens object
-    let tokensData = tokenData.tokens || tokenData;
-    let tokenExpiresIn = tokensData.expiresIn || tokensData.expires_in;
-    let tokenExpiresAt = tokenExpiresIn ? new Date(Date.now() + tokenExpiresIn * 1000).toISOString() : tokenData.expires_at || tokenData.expiresAt;
-    let tokens = {
+    const tokensData = tokenData.tokens || tokenData;
+    const tokenExpiresIn = tokensData.expiresIn || tokensData.expires_in;
+    const tokenExpiresAt = tokenExpiresIn ? new Date(Date.now() + tokenExpiresIn * 1000).toISOString() : tokenData.expires_at || tokenData.expiresAt;
+    const tokens = {
       accessToken: tokensData.accessToken || tokensData.access_token,
       refreshToken: tokensData.refreshToken || tokensData.refresh_token,
       expiresAt: tokenExpiresAt,
@@ -129,7 +129,7 @@ export async function loginCommand(options = {}, globalOptions = {}) {
     if (tokens.organizations && tokens.organizations.length > 0) {
       output.blank();
       output.info('Organizations:');
-      for (let org of tokens.organizations) {
+      for (const org of tokens.organizations) {
         output.print(`  - ${org.name}${org.slug ? ` (@${org.slug})` : ''}`);
       }
     }
@@ -137,11 +137,11 @@ export async function loginCommand(options = {}, globalOptions = {}) {
     // Show token expiry info
     if (tokens.expiresAt) {
       output.blank();
-      let expiresAt = new Date(tokens.expiresAt);
-      let msUntilExpiry = expiresAt.getTime() - Date.now();
-      let daysUntilExpiry = Math.floor(msUntilExpiry / (1000 * 60 * 60 * 24));
-      let hoursUntilExpiry = Math.floor(msUntilExpiry / (1000 * 60 * 60));
-      let minutesUntilExpiry = Math.floor(msUntilExpiry / (1000 * 60));
+      const expiresAt = new Date(tokens.expiresAt);
+      const msUntilExpiry = expiresAt.getTime() - Date.now();
+      const daysUntilExpiry = Math.floor(msUntilExpiry / (1000 * 60 * 60 * 24));
+      const hoursUntilExpiry = Math.floor(msUntilExpiry / (1000 * 60 * 60));
+      const minutesUntilExpiry = Math.floor(msUntilExpiry / (1000 * 60));
       if (daysUntilExpiry > 0) {
         output.info(`Token expires in ${daysUntilExpiry} day${daysUntilExpiry !== 1 ? 's' : ''} (${expiresAt.toLocaleDateString()})`);
       } else if (hoursUntilExpiry > 0) {
@@ -180,7 +180,7 @@ export async function loginCommand(options = {}, globalOptions = {}) {
  * @param {Object} options - Command options
  */
 export function validateLoginOptions() {
-  let errors = [];
+  const errors = [];
 
   // No specific validation needed for login command
   // OAuth device flow handles everything via browser

package/dist/commands/logout.js

@@ -3,10 +3,10 @@
  * Clears stored authentication tokens
  */
 
-import * as output from '../utils/output.js';
 import { AuthService } from '../services/auth-service.js';
 import { getApiUrl } from '../utils/environment-config.js';
 import { getAuthTokens } from '../utils/global-config.js';
+import * as output from '../utils/output.js';
 
 /**
  * Logout command implementation
@@ -21,7 +21,7 @@ export async function logoutCommand(options = {}, globalOptions = {}) {
   });
   try {
     // Check if user is logged in
-    let auth = await getAuthTokens();
+    const auth = await getAuthTokens();
     if (!auth || !auth.accessToken) {
       output.info('You are not logged in');
       output.cleanup();
@@ -30,7 +30,7 @@ export async function logoutCommand(options = {}, globalOptions = {}) {
 
     // Logout
     output.startSpinner('Logging out...');
-    let authService = new AuthService({
+    const authService = new AuthService({
       baseUrl: options.apiUrl || getApiUrl()
     });
     await authService.logout();
@@ -58,7 +58,7 @@ export async function logoutCommand(options = {}, globalOptions = {}) {
  * @param {Object} options - Command options
  */
 export function validateLogoutOptions() {
-  let errors = [];
+  const errors = [];
 
   // No specific validation needed for logout command
 

package/dist/commands/project.js

@@ -3,12 +3,12 @@
  * Select, list, and manage project tokens
  */
 
-import * as output from '../utils/output.js';
+import { resolve } from 'node:path';
+import readline from 'node:readline';
 import { AuthService } from '../services/auth-service.js';
 import { getApiUrl } from '../utils/environment-config.js';
-import { getAuthTokens, saveProjectMapping, getProjectMapping, getProjectMappings, deleteProjectMapping } from '../utils/global-config.js';
-import { resolve } from 'path';
-import readline from 'readline';
+import { deleteProjectMapping, getAuthTokens, getProjectMapping, getProjectMappings, saveProjectMapping } from '../utils/global-config.js';
+import * as output from '../utils/output.js';
 
 /**
  * Project select command - configure project for current directory
@@ -23,20 +23,20 @@ export async function projectSelectCommand(options = {}, globalOptions = {}) {
   });
   try {
     // Check authentication
-    let auth = await getAuthTokens();
+    const auth = await getAuthTokens();
     if (!auth || !auth.accessToken) {
       output.error('Not authenticated');
       output.blank();
       output.info('Run "vizzly login" to authenticate first');
       process.exit(1);
     }
-    let authService = new AuthService({
+    const authService = new AuthService({
       baseUrl: options.apiUrl || getApiUrl()
     });
 
     // Get user info to show organizations
     output.startSpinner('Fetching organizations...');
-    let userInfo = await authService.whoami();
+    const userInfo = await authService.whoami();
     output.stopSpinner();
     if (!userInfo.organizations || userInfo.organizations.length === 0) {
       output.error('No organizations found');
@@ -53,12 +53,12 @@ export async function projectSelectCommand(options = {}, globalOptions = {}) {
       output.print(`  ${index + 1}. ${org.name} (@${org.slug})`);
     });
     output.blank();
-    let orgChoice = await promptNumber('Enter number', 1, userInfo.organizations.length);
-    let selectedOrg = userInfo.organizations[orgChoice - 1];
+    const orgChoice = await promptNumber('Enter number', 1, userInfo.organizations.length);
+    const selectedOrg = userInfo.organizations[orgChoice - 1];
 
     // List projects for organization
     output.startSpinner(`Fetching projects for ${selectedOrg.name}...`);
-    let response = await makeAuthenticatedRequest(`${options.apiUrl || getApiUrl()}/api/project`, {
+    const response = await makeAuthenticatedRequest(`${options.apiUrl || getApiUrl()}/api/project`, {
       headers: {
         Authorization: `Bearer ${auth.accessToken}`,
         'X-Organization': selectedOrg.slug
@@ -67,7 +67,7 @@ export async function projectSelectCommand(options = {}, globalOptions = {}) {
     output.stopSpinner();
 
     // Handle both array response and object with projects property
-    let projects = Array.isArray(response) ? response : response.projects || [];
+    const projects = Array.isArray(response) ? response : response.projects || [];
     if (projects.length === 0) {
       output.error('No projects found');
       output.blank();
@@ -83,12 +83,12 @@ export async function projectSelectCommand(options = {}, globalOptions = {}) {
       output.print(`  ${index + 1}. ${project.name} (${project.slug})`);
     });
     output.blank();
-    let projectChoice = await promptNumber('Enter number', 1, projects.length);
-    let selectedProject = projects[projectChoice - 1];
+    const projectChoice = await promptNumber('Enter number', 1, projects.length);
+    const selectedProject = projects[projectChoice - 1];
 
     // Create API token for project
     output.startSpinner(`Creating API token for ${selectedProject.name}...`);
-    let tokenResponse = await makeAuthenticatedRequest(`${options.apiUrl || getApiUrl()}/api/project/${selectedProject.slug}/tokens`, {
+    const tokenResponse = await makeAuthenticatedRequest(`${options.apiUrl || getApiUrl()}/api/project/${selectedProject.slug}/tokens`, {
       method: 'POST',
       headers: {
         Authorization: `Bearer ${auth.accessToken}`,
@@ -103,7 +103,7 @@ export async function projectSelectCommand(options = {}, globalOptions = {}) {
     output.stopSpinner();
 
     // Save project mapping
-    let currentDir = resolve(process.cwd());
+    const currentDir = resolve(process.cwd());
     await saveProjectMapping(currentDir, {
       token: tokenResponse.token,
       projectSlug: selectedProject.slug,
@@ -135,8 +135,8 @@ export async function projectListCommand(_options = {}, globalOptions = {}) {
     color: !globalOptions.noColor
   });
   try {
-    let mappings = await getProjectMappings();
-    let paths = Object.keys(mappings);
+    const mappings = await getProjectMappings();
+    const paths = Object.keys(mappings);
     if (paths.length === 0) {
       output.info('No projects configured');
       output.blank();
@@ -151,14 +151,14 @@ export async function projectListCommand(_options = {}, globalOptions = {}) {
     }
     output.info('Configured projects:');
     output.blank();
-    let currentDir = resolve(process.cwd());
-    for (let path of paths) {
-      let mapping = mappings[path];
-      let isCurrent = path === currentDir;
-      let marker = isCurrent ? '→' : ' ';
+    const currentDir = resolve(process.cwd());
+    for (const path of paths) {
+      const mapping = mappings[path];
+      const isCurrent = path === currentDir;
+      const marker = isCurrent ? '→' : ' ';
 
       // Extract token string (handle both string and object formats)
-      let tokenStr = typeof mapping.token === 'string' ? mapping.token : mapping.token?.token || '[invalid token]';
+      const tokenStr = typeof mapping.token === 'string' ? mapping.token : mapping.token?.token || '[invalid token]';
       output.print(`${marker} ${path}`);
       output.print(`  Project: ${mapping.projectName} (${mapping.projectSlug})`);
       output.print(`  Organization: ${mapping.organizationSlug}`);
@@ -187,8 +187,8 @@ export async function projectTokenCommand(_options = {}, globalOptions = {}) {
     color: !globalOptions.noColor
   });
   try {
-    let currentDir = resolve(process.cwd());
-    let mapping = await getProjectMapping(currentDir);
+    const currentDir = resolve(process.cwd());
+    const mapping = await getProjectMapping(currentDir);
     if (!mapping) {
       output.error('No project configured for this directory');
       output.blank();
@@ -197,7 +197,7 @@ export async function projectTokenCommand(_options = {}, globalOptions = {}) {
     }
 
     // Extract token string (handle both string and object formats)
-    let tokenStr = typeof mapping.token === 'string' ? mapping.token : mapping.token?.token || '[invalid token]';
+    const tokenStr = typeof mapping.token === 'string' ? mapping.token : mapping.token?.token || '[invalid token]';
     if (globalOptions.json) {
       output.data({
         token: tokenStr,
@@ -224,11 +224,11 @@ export async function projectTokenCommand(_options = {}, globalOptions = {}) {
  * Helper to make authenticated API request
  */
 async function makeAuthenticatedRequest(url, options = {}) {
-  let response = await fetch(url, options);
+  const response = await fetch(url, options);
   if (!response.ok) {
     let errorText = '';
     try {
-      let errorData = await response.json();
+      const errorData = await response.json();
       errorText = errorData.error || errorData.message || '';
     } catch {
       errorText = await response.text();
@@ -243,14 +243,14 @@ async function makeAuthenticatedRequest(url, options = {}) {
  */
 function promptNumber(message, min, max) {
   return new Promise(resolve => {
-    let rl = readline.createInterface({
+    const rl = readline.createInterface({
       input: process.stdin,
       output: process.stdout
     });
-    let ask = () => {
+    const ask = () => {
       rl.question(`${message} (${min}-${max}): `, answer => {
-        let num = parseInt(answer, 10);
-        if (isNaN(num) || num < min || num > max) {
+        const num = parseInt(answer, 10);
+        if (Number.isNaN(num) || num < min || num > max) {
           output.print(`Please enter a number between ${min} and ${max}`);
           ask();
         } else {
@@ -275,8 +275,8 @@ export async function projectRemoveCommand(_options = {}, globalOptions = {}) {
     color: !globalOptions.noColor
   });
   try {
-    let currentDir = resolve(process.cwd());
-    let mapping = await getProjectMapping(currentDir);
+    const currentDir = resolve(process.cwd());
+    const mapping = await getProjectMapping(currentDir);
     if (!mapping) {
       output.info('No project configured for this directory');
       output.cleanup();
@@ -290,7 +290,7 @@ export async function projectRemoveCommand(_options = {}, globalOptions = {}) {
     output.print(`  Organization: ${mapping.organizationSlug}`);
     output.print(`  Directory: ${currentDir}`);
     output.blank();
-    let confirmed = await promptConfirm('Remove this project configuration?');
+    const confirmed = await promptConfirm('Remove this project configuration?');
     if (!confirmed) {
       output.info('Cancelled');
       output.cleanup();
@@ -312,7 +312,7 @@ export async function projectRemoveCommand(_options = {}, globalOptions = {}) {
  */
 function promptConfirm(message) {
   return new Promise(resolve => {
-    let rl = readline.createInterface({
+    const rl = readline.createInterface({
       input: process.stdin,
       output: process.stdout
     });

package/dist/commands/run.js

@@ -1,7 +1,7 @@
-import { loadConfig } from '../utils/config-loader.js';
-import * as output from '../utils/output.js';
 import { createServices } from '../services/index.js';
+import { loadConfig } from '../utils/config-loader.js';
 import { detectBranch, detectCommit, detectCommitMessage, detectPullRequestNumber, generateBuildNameWithGit } from '../utils/git.js';
+import * as output from '../utils/output.js';
 
 /**
  * Run command implementation
@@ -21,7 +21,7 @@ export async function runCommand(testCommand, options = {}, globalOptions = {})
   let isTddMode = false;
 
   // Ensure cleanup on exit
-  let cleanup = async () => {
+  const cleanup = async () => {
     output.cleanup();
 
     // Cancel test runner (kills process and stops server)
@@ -36,40 +36,40 @@ export async function runCommand(testCommand, options = {}, globalOptions = {})
     // Finalize build if we have one
     if (testRunner && buildId) {
       try {
-        let executionTime = Date.now() - (startTime || Date.now());
+        const executionTime = Date.now() - (startTime || Date.now());
         await testRunner.finalizeBuild(buildId, isTddMode, false, executionTime);
       } catch {
         // Silent fail on cleanup
       }
     }
   };
-  let sigintHandler = async () => {
+  const sigintHandler = async () => {
     await cleanup();
     process.exit(1);
   };
-  let exitHandler = () => output.cleanup();
+  const exitHandler = () => output.cleanup();
   process.on('SIGINT', sigintHandler);
   process.on('exit', exitHandler);
   try {
     // Load configuration with CLI overrides
-    let allOptions = {
+    const allOptions = {
       ...globalOptions,
       ...options
     };
     output.debug('[RUN] Loading config', {
       hasToken: !!allOptions.token
     });
-    let config = await loadConfig(globalOptions.config, allOptions);
+    const config = await loadConfig(globalOptions.config, allOptions);
     output.debug('[RUN] Config loaded', {
       hasApiKey: !!config.apiKey,
-      apiKeyPrefix: config.apiKey ? config.apiKey.substring(0, 8) + '***' : 'NONE'
+      apiKeyPrefix: config.apiKey ? `${config.apiKey.substring(0, 8)}***` : 'NONE'
     });
     if (globalOptions.verbose) {
       output.info('Token check:');
       output.debug('Token details', {
         hasApiKey: !!config.apiKey,
         apiKeyType: typeof config.apiKey,
-        apiKeyPrefix: typeof config.apiKey === 'string' && config.apiKey ? config.apiKey.substring(0, 10) + '...' : 'none',
+        apiKeyPrefix: typeof config.apiKey === 'string' && config.apiKey ? `${config.apiKey.substring(0, 10)}...` : 'none',
         projectSlug: config.projectSlug || 'none',
         organizationSlug: config.organizationSlug || 'none'
       });
@@ -82,11 +82,11 @@ export async function runCommand(testCommand, options = {}, globalOptions = {})
     }
 
     // Collect git metadata and build info
-    let branch = await detectBranch(options.branch);
-    let commit = await detectCommit(options.commit);
-    let message = options.message || (await detectCommitMessage());
-    let buildName = await generateBuildNameWithGit(options.buildName);
-    let pullRequestNumber = detectPullRequestNumber();
+    const branch = await detectBranch(options.branch);
+    const commit = await detectCommit(options.commit);
+    const message = options.message || (await detectCommitMessage());
+    const buildName = await generateBuildNameWithGit(options.buildName);
+    const pullRequestNumber = detectPullRequestNumber();
     if (globalOptions.verbose) {
       output.info('Configuration loaded');
       output.debug('Config details', {
@@ -104,7 +104,7 @@ export async function runCommand(testCommand, options = {}, globalOptions = {})
 
     // Create service container and get test runner service
     output.startSpinner('Initializing test runner...');
-    let configWithVerbose = {
+    const configWithVerbose = {
       ...config,
       verbose: globalOptions.verbose,
       uploadAll: options.uploadAll || false
@@ -112,7 +112,7 @@ export async function runCommand(testCommand, options = {}, globalOptions = {})
     output.debug('[RUN] Creating services', {
       hasApiKey: !!configWithVerbose.apiKey
     });
-    let services = createServices(configWithVerbose, 'run');
+    const services = createServices(configWithVerbose, 'run');
     testRunner = services.testRunner;
     output.stopSpinner();
 
@@ -121,7 +121,7 @@ export async function runCommand(testCommand, options = {}, globalOptions = {})
 
     // Set up event handlers
     testRunner.on('progress', progressData => {
-      let {
+      const {
         message: progressMessage
       } = progressData;
       output.progress(progressMessage || 'Running tests...');
@@ -164,7 +164,7 @@ export async function runCommand(testCommand, options = {}, globalOptions = {})
     });
 
     // Prepare run options
-    let runOptions = {
+    const runOptions = {
       testCommand,
       port: config.server.port,
       timeout: config.server.timeout,
@@ -210,8 +210,8 @@ export async function runCommand(testCommand, options = {}, globalOptions = {})
       // Check if it's a test command failure (as opposed to setup failure)
       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;
+        const exitCodeMatch = error.message.match(/exited with code (\d+)/);
+        const exitCode = exitCodeMatch ? parseInt(exitCodeMatch[1], 10) : 1;
         output.error('Test run failed');
         return {
           success: false,
@@ -233,10 +233,10 @@ export async function runCommand(testCommand, options = {}, globalOptions = {})
       if (runOptions.wait) {
         output.info('Waiting for build completion...');
         output.startSpinner('Processing comparisons...');
-        let {
+        const {
           uploader
         } = services;
-        let buildResult = await uploader.waitForBuild(result.buildId);
+        const buildResult = await uploader.waitForBuild(result.buildId);
         output.success('Build processing completed');
 
         // Exit with appropriate code based on comparison results
@@ -255,11 +255,11 @@ export async function runCommand(testCommand, options = {}, globalOptions = {})
 
     // Provide more context about where the error occurred
     let errorContext = 'Test run failed';
-    if (error.message && error.message.includes('build')) {
+    if (error.message?.includes('build')) {
       errorContext = 'Build creation failed';
-    } else if (error.message && error.message.includes('screenshot')) {
+    } else if (error.message?.includes('screenshot')) {
       errorContext = 'Screenshot processing failed';
-    } else if (error.message && error.message.includes('server')) {
+    } else if (error.message?.includes('server')) {
       errorContext = 'Server startup failed';
     }
     output.error(errorContext, error);
@@ -277,30 +277,30 @@ export async function runCommand(testCommand, options = {}, globalOptions = {})
  * @param {Object} options - Command options
  */
 export function validateRunOptions(testCommand, options) {
-  let errors = [];
+  const errors = [];
   if (!testCommand || testCommand.trim() === '') {
     errors.push('Test command is required');
   }
   if (options.port) {
-    let port = parseInt(options.port, 10);
-    if (isNaN(port) || port < 1 || port > 65535) {
+    const port = parseInt(options.port, 10);
+    if (Number.isNaN(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 (isNaN(timeout) || timeout < 1000) {
+    const timeout = parseInt(options.timeout, 10);
+    if (Number.isNaN(timeout) || timeout < 1000) {
       errors.push('Timeout must be at least 1000 milliseconds');
     }
   }
   if (options.batchSize !== undefined) {
-    let n = parseInt(options.batchSize, 10);
+    const n = parseInt(options.batchSize, 10);
     if (!Number.isFinite(n) || n <= 0) {
       errors.push('Batch size must be a positive integer');
     }
   }
   if (options.uploadTimeout !== undefined) {
-    let n = parseInt(options.uploadTimeout, 10);
+    const n = parseInt(options.uploadTimeout, 10);
     if (!Number.isFinite(n) || n <= 0) {
       errors.push('Upload timeout must be a positive integer (milliseconds)');
     }

package/dist/commands/status.js

@@ -1,7 +1,7 @@
-import { loadConfig } from '../utils/config-loader.js';
-import * as output from '../utils/output.js';
 import { createServices } from '../services/index.js';
+import { loadConfig } from '../utils/config-loader.js';
 import { getApiUrl } from '../utils/environment-config.js';
+import * as output from '../utils/output.js';
 
 /**
  * Status command implementation
@@ -19,11 +19,11 @@ export async function statusCommand(buildId, options = {}, globalOptions = {}) {
     output.info(`Checking status for build: ${buildId}`);
 
     // Load configuration with CLI overrides
-    let allOptions = {
+    const allOptions = {
       ...globalOptions,
       ...options
     };
-    let config = await loadConfig(globalOptions.config, allOptions);
+    const config = await loadConfig(globalOptions.config, allOptions);
 
     // Validate API token
     if (!config.apiKey) {
@@ -33,17 +33,17 @@ export async function statusCommand(buildId, options = {}, globalOptions = {}) {
 
     // Get API service
     output.startSpinner('Fetching build status...');
-    let services = createServices(config, 'status');
-    let {
+    const services = createServices(config, 'status');
+    const {
       apiService
     } = services;
 
     // Get build details via unified ApiService
-    let buildStatus = await apiService.getBuild(buildId);
+    const buildStatus = await apiService.getBuild(buildId);
     output.stopSpinner();
 
     // Extract build data from API response
-    let build = buildStatus.build || buildStatus;
+    const build = buildStatus.build || buildStatus;
 
     // Display build summary
     output.success(`Build: ${build.name || build.id}`);
@@ -77,15 +77,15 @@ export async function statusCommand(buildId, options = {}, globalOptions = {}) {
     }
 
     // Show build URL if we can construct it
-    let baseUrl = config.baseUrl || getApiUrl();
+    const baseUrl = config.baseUrl || getApiUrl();
     if (baseUrl && build.project_id) {
-      let buildUrl = baseUrl.replace('/api', '') + `/projects/${build.project_id}/builds/${build.id}`;
+      const buildUrl = baseUrl.replace('/api', '') + `/projects/${build.project_id}/builds/${build.id}`;
       output.info(`View Build: ${buildUrl}`);
     }
 
     // Output JSON data for --json mode
     if (globalOptions.json) {
-      let statusData = {
+      const statusData = {
         buildId: build.id,
         status: build.status,
         name: build.name,
@@ -131,9 +131,9 @@ export async function statusCommand(buildId, options = {}, globalOptions = {}) {
 
     // 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;
+      const totalJobs = build.completed_jobs + build.failed_jobs + build.processing_screenshots;
       if (totalJobs > 0) {
-        let progress = (build.completed_jobs + build.failed_jobs) / totalJobs;
+        const progress = (build.completed_jobs + build.failed_jobs) / totalJobs;
         output.info(`Progress: ${Math.round(progress * 100)}% complete`);
       }
     }
@@ -155,7 +155,7 @@ export async function statusCommand(buildId, options = {}, globalOptions = {}) {
  * @param {Object} options - Command options
  */
 export function validateStatusOptions(buildId) {
-  let errors = [];
+  const errors = [];
   if (!buildId || buildId.trim() === '') {
     errors.push('Build ID is required');
   }