@vizzly-testing/cli 0.13.4 → 0.15.0

package/dist/types/utils/output.d.ts

@@ -0,0 +1,84 @@
+/**
+ * Configure output settings
+ * Call this once at CLI startup with global options
+ */
+export function configure(options?: {}): void;
+/**
+ * Show command header (e.g., "vizzly · tdd · local")
+ * Only shows once per command execution
+ */
+export function header(command: any, mode?: any): void;
+/**
+ * Get current colors instance (for custom formatting)
+ */
+export function getColors(): {
+    success: any;
+    error: any;
+    warning: any;
+    info: any;
+};
+/**
+ * Show a success message
+ */
+export function success(message: any, data?: {}): void;
+/**
+ * Show final result summary (e.g., "✓ 5 screenshots · 234ms")
+ */
+export function result(message: any): void;
+/**
+ * Show an info message
+ */
+export function info(message: any, data?: {}): void;
+/**
+ * Show a warning message (goes to stderr)
+ */
+export function warn(message: any, data?: {}): void;
+/**
+ * Show an error message (goes to stderr)
+ * Does NOT exit - caller decides whether to exit
+ */
+export function error(message: any, err?: any, data?: {}): void;
+/**
+ * Print a blank line for spacing
+ */
+export function blank(): void;
+/**
+ * Print raw text without any formatting
+ */
+export function print(text: any): void;
+/**
+ * Print raw text to stderr
+ */
+export function printErr(text: any): void;
+/**
+ * Output structured data
+ */
+export function data(obj: any): void;
+/**
+ * Start a spinner with message
+ */
+export function startSpinner(message: any): void;
+/**
+ * Update spinner message
+ */
+export function updateSpinner(message: any, current?: number, total?: number): void;
+/**
+ * Stop the spinner
+ */
+export function stopSpinner(): void;
+/**
+ * Show progress update
+ */
+export function progress(message: any, current?: number, total?: number): void;
+/**
+ * Log debug message with component prefix (only shown in verbose mode)
+ *
+ * @param {string} component - Component name (e.g., 'server', 'config', 'build')
+ * @param {string} message - Debug message
+ * @param {Object} data - Optional data object to display inline
+ */
+export function debug(component: string, message: string, data?: any): void;
+/**
+ * Clean up (stop spinner, flush logs)
+ */
+export function cleanup(): void;

package/dist/utils/config-loader.js

@@ -3,7 +3,8 @@ import { resolve } from 'path';
 import { getApiToken, getApiUrl, getParallelId } from './environment-config.js';
 import { validateVizzlyConfigWithDefaults } from './config-schema.js';
 import { getAccessToken, getProjectMapping } from './global-config.js';
-const DEFAULT_CONFIG = {
+import * as output from './output.js';
+let DEFAULT_CONFIG = {
   // API Configuration
   apiKey: undefined,
   // Will be set from env, global config, or CLI overrides
@@ -37,8 +38,8 @@ const DEFAULT_CONFIG = {
 };
 export async function loadConfig(configPath = null, cliOverrides = {}) {
   // 1. Load from config file using cosmiconfig
-  const explorer = cosmiconfigSync('vizzly');
-  const result = configPath ? explorer.load(configPath) : explorer.search();
+  let explorer = cosmiconfigSync('vizzly');
+  let result = configPath ? explorer.load(configPath) : explorer.search();
   let fileConfig = {};
   if (result && result.config) {
     // Handle ESM default export (cosmiconfig wraps it in { default: {...} })
@@ -46,10 +47,10 @@ export async function loadConfig(configPath = null, cliOverrides = {}) {
   }
 
   // 2. Validate config file using Zod schema
-  const validatedFileConfig = validateVizzlyConfigWithDefaults(fileConfig);
+  let validatedFileConfig = validateVizzlyConfigWithDefaults(fileConfig);
 
   // Create a proper clone of the default config to avoid shared object references
-  const config = {
+  let config = {
     ...DEFAULT_CONFIG,
     server: {
       ...DEFAULT_CONFIG.server
@@ -74,11 +75,8 @@ export async function loadConfig(configPath = null, cliOverrides = {}) {
 
   // 3. Check project mapping for current directory (if no CLI flag)
   if (!cliOverrides.token) {
-    const currentDir = process.cwd();
-    if (process.env.DEBUG_CONFIG) {
-      console.log('[CONFIG] Looking up project mapping for:', currentDir);
-    }
-    const projectMapping = await getProjectMapping(currentDir);
+    let currentDir = process.cwd();
+    let projectMapping = await getProjectMapping(currentDir);
     if (projectMapping && projectMapping.token) {
       // Handle both string tokens and token objects (backward compatibility)
       let token;
@@ -93,59 +91,37 @@ export async function loadConfig(configPath = null, cliOverrides = {}) {
       config.apiKey = token;
       config.projectSlug = projectMapping.projectSlug;
       config.organizationSlug = projectMapping.organizationSlug;
-
-      // Debug logging
-      if (process.env.DEBUG_CONFIG) {
-        console.log('[CONFIG] Found project mapping:', {
-          dir: currentDir,
-          projectSlug: projectMapping.projectSlug,
-          hasToken: !!projectMapping.token,
-          tokenType: typeof projectMapping.token,
-          tokenPrefix: token ? token.substring(0, 8) + '***' : 'none'
-        });
-        console.log('[CONFIG] Set config.apiKey to:', config.apiKey ? config.apiKey.substring(0, 8) + '***' : 'NONE');
-      }
-    } else if (process.env.DEBUG_CONFIG) {
-      console.log('[CONFIG] No project mapping found for:', currentDir);
+      output.debug('Using project mapping', {
+        project: projectMapping.projectSlug,
+        org: projectMapping.organizationSlug
+      });
     }
   }
 
   // 3.5. Check global config for user access token (if no CLI flag)
   if (!config.apiKey && !cliOverrides.token) {
-    const globalToken = await getAccessToken();
+    let globalToken = await getAccessToken();
     if (globalToken) {
       config.apiKey = globalToken;
     }
   }
 
   // 4. Override with environment variables (higher priority than fallbacks)
-  const envApiKey = getApiToken();
-  const envApiUrl = getApiUrl();
-  const envParallelId = getParallelId();
-  if (process.env.DEBUG_CONFIG) {
-    console.log('[CONFIG] Step 4 - env vars:', JSON.stringify({
-      hasEnvApiKey: !!envApiKey,
-      envApiKeyPrefix: envApiKey ? envApiKey.substring(0, 8) + '***' : 'none',
-      configApiKeyBefore: config.apiKey ? config.apiKey.substring(0, 8) + '***' : 'NONE'
-    }));
+  let envApiKey = getApiToken();
+  let envApiUrl = getApiUrl();
+  let envParallelId = getParallelId();
+  if (envApiKey) {
+    config.apiKey = envApiKey;
+    output.debug('Using API token from environment');
   }
-  if (envApiKey) config.apiKey = envApiKey;
   if (envApiUrl !== 'https://app.vizzly.dev') config.apiUrl = envApiUrl;
   if (envParallelId) config.parallelId = envParallelId;
 
   // 5. Apply CLI overrides (highest priority)
-  if (process.env.DEBUG_CONFIG) {
-    console.log('[CONFIG] Step 5 - before CLI overrides:', {
-      configApiKey: config.apiKey ? config.apiKey.substring(0, 8) + '***' : 'NONE',
-      cliToken: cliOverrides.token ? cliOverrides.token.substring(0, 8) + '***' : 'none'
-    });
+  if (cliOverrides.token) {
+    output.debug('Using API token from --token flag');
   }
   applyCLIOverrides(config, cliOverrides);
-  if (process.env.DEBUG_CONFIG) {
-    console.log('[CONFIG] Step 6 - after CLI overrides:', {
-      configApiKey: config.apiKey ? config.apiKey.substring(0, 8) + '***' : 'NONE'
-    });
-  }
   return config;
 }
 
@@ -191,7 +167,7 @@ function applyCLIOverrides(config, cliOverrides = {}) {
   if (cliOverrides.allowNoToken !== undefined) config.allowNoToken = cliOverrides.allowNoToken;
 }
 function mergeConfig(target, source) {
-  for (const key in source) {
+  for (let key in source) {
     if (source[key] && typeof source[key] === 'object' && !Array.isArray(source[key])) {
       if (!target[key]) target[key] = {};
       mergeConfig(target[key], source[key]);
@@ -201,7 +177,7 @@ function mergeConfig(target, source) {
   }
 }
 export function getScreenshotPaths(config) {
-  const screenshotsDir = config.upload?.screenshotsDir || './screenshots';
-  const paths = Array.isArray(screenshotsDir) ? screenshotsDir : [screenshotsDir];
+  let screenshotsDir = config.upload?.screenshotsDir || './screenshots';
+  let paths = Array.isArray(screenshotsDir) ? screenshotsDir : [screenshotsDir];
   return paths.map(p => resolve(process.cwd(), p));
 }

package/dist/utils/global-config.js

@@ -7,6 +7,7 @@ import { homedir } from 'os';
 import { join, dirname, parse } from 'path';
 import { readFile, writeFile, mkdir, chmod } from 'fs/promises';
 import { existsSync } from 'fs';
+import * as output from './output.js';
 
 /**
  * Get the path to the global Vizzly directory
@@ -57,7 +58,7 @@ export async function loadGlobalConfig() {
     }
 
     // Log warning about corrupted config but don't crash
-    console.warn('Warning: Global config file is corrupted, ignoring');
+    output.warn('Global config file is corrupted, ignoring');
     return {};
   }
 }
@@ -176,9 +177,6 @@ export async function getAccessToken() {
 export async function getProjectMapping(directoryPath) {
   let config = await loadGlobalConfig();
   if (!config.projects) {
-    if (process.env.DEBUG_CONFIG) {
-      console.log('[MAPPING] No projects in global config');
-    }
     return null;
   }
 
@@ -187,18 +185,8 @@ export async function getProjectMapping(directoryPath) {
   let {
     root
   } = parse(currentPath);
-  if (process.env.DEBUG_CONFIG) {
-    console.log('[MAPPING] Starting lookup from:', currentPath);
-    console.log('[MAPPING] Available mappings:', Object.keys(config.projects));
-  }
   while (currentPath !== root) {
-    if (process.env.DEBUG_CONFIG) {
-      console.log('[MAPPING] Checking:', currentPath);
-    }
     if (config.projects[currentPath]) {
-      if (process.env.DEBUG_CONFIG) {
-        console.log('[MAPPING] Found match at:', currentPath);
-      }
       return config.projects[currentPath];
     }
 
@@ -210,9 +198,6 @@ export async function getProjectMapping(directoryPath) {
     }
     currentPath = parentPath;
   }
-  if (process.env.DEBUG_CONFIG) {
-    console.log('[MAPPING] No mapping found');
-  }
   return null;
 }