@vizzly-testing/cli 0.39.1 → 0.39.2

package/dist/services/config-service.js

@@ -1,313 +0,0 @@
-/**
- * Config Service
- * Manages configuration for the TDD dashboard settings page
- *
- * Provides read/write access to:
- * - Merged config (read-only combination of all sources)
- * - Project config (vizzly.config.js in working directory)
- * - Global config (~/.vizzly/config.json)
- */
-
-import { existsSync } from 'node:fs';
-import { writeFile } from 'node:fs/promises';
-import { join } from 'node:path';
-import { cosmiconfigSync } from 'cosmiconfig';
-import { CONFIG_DEFAULTS } from '../config/core.js';
-import { loadGlobalConfig, saveGlobalConfig } from '../utils/global-config.js';
-import * as output from '../utils/output.js';
-
-/**
- * Create a config service instance
- * @param {Object} options
- * @param {string} options.workingDir - Working directory for project config
- * @returns {Object} Config service with getConfig, updateConfig, validateConfig methods
- */
-export function createConfigService({
-  workingDir
-}) {
-  let projectConfigPath = null;
-  let projectConfigFormat = 'js'; // 'js' or 'json'
-
-  // Find project config file
-  let explorer = cosmiconfigSync('vizzly');
-  let searchResult = explorer.search(workingDir);
-  if (searchResult?.filepath) {
-    projectConfigPath = searchResult.filepath;
-    projectConfigFormat = searchResult.filepath.endsWith('.json') ? 'json' : 'js';
-  }
-
-  /**
-   * Get configuration by type
-   * @param {'merged'|'project'|'global'} type
-   * @returns {Promise<Object>}
-   */
-  async function getConfig(type) {
-    if (type === 'merged') {
-      return getMergedConfig();
-    } else if (type === 'project') {
-      return getProjectConfig();
-    } else if (type === 'global') {
-      return getGlobalConfigData();
-    }
-    throw new Error(`Unknown config type: ${type}`);
-  }
-
-  /**
-   * Get merged configuration with source tracking
-   */
-  async function getMergedConfig() {
-    let config = cloneConfigValue(CONFIG_DEFAULTS);
-    let sources = {};
-
-    // Layer 1: Global config
-    let globalConfig = await loadGlobalConfig();
-    if (globalConfig.settings) {
-      mergeWithTracking(config, globalConfig.settings, sources, 'global');
-    }
-
-    // Layer 2: Project config
-    if (projectConfigPath && existsSync(projectConfigPath)) {
-      try {
-        let result = explorer.load(projectConfigPath);
-        let projectConfig = result?.config?.default || result?.config || {};
-        mergeWithTracking(config, projectConfig, sources, 'project');
-      } catch (error) {
-        output.debug('config-service', `Error loading project config: ${error.message}`);
-      }
-    }
-
-    // Layer 3: Environment variables
-    if (process.env.VIZZLY_THRESHOLD) {
-      config.comparison.threshold = Number(process.env.VIZZLY_THRESHOLD);
-      sources.comparison = 'env';
-    }
-    if (process.env.VIZZLY_MIN_CLUSTER_SIZE) {
-      config.comparison.minClusterSize = Number(process.env.VIZZLY_MIN_CLUSTER_SIZE);
-      sources.comparison = 'env';
-    }
-    if (process.env.VIZZLY_PORT) {
-      config.server.port = Number(process.env.VIZZLY_PORT);
-      sources.server = 'env';
-    }
-    return {
-      config,
-      sources
-    };
-  }
-
-  /**
-   * Get project-level configuration only
-   */
-  async function getProjectConfig() {
-    if (!projectConfigPath || !existsSync(projectConfigPath)) {
-      return {
-        config: {},
-        path: null
-      };
-    }
-    try {
-      let result = explorer.load(projectConfigPath);
-      let config = result?.config?.default || result?.config || {};
-      return {
-        config,
-        path: projectConfigPath
-      };
-    } catch (error) {
-      output.debug('config-service', `Error loading project config: ${error.message}`);
-      return {
-        config: {},
-        path: projectConfigPath,
-        error: error.message
-      };
-    }
-  }
-
-  /**
-   * Get global configuration only
-   */
-  async function getGlobalConfigData() {
-    let globalConfig = await loadGlobalConfig();
-    return {
-      config: globalConfig.settings || {},
-      path: join(process.env.VIZZLY_HOME || join(process.env.HOME || '', '.vizzly'), 'config.json')
-    };
-  }
-
-  /**
-   * Update configuration by type
-   * @param {'project'|'global'} type
-   * @param {Object} updates - Config updates to apply
-   * @returns {Promise<Object>}
-   */
-  async function updateConfig(type, updates) {
-    if (type === 'project') {
-      return updateProjectConfig(updates);
-    } else if (type === 'global') {
-      return updateGlobalConfig(updates);
-    }
-    throw new Error(`Cannot update config type: ${type}`);
-  }
-
-  /**
-   * Update project configuration (vizzly.config.js)
-   */
-  async function updateProjectConfig(updates) {
-    // If no project config exists, create one
-    if (!projectConfigPath) {
-      projectConfigPath = join(workingDir, 'vizzly.config.js');
-      projectConfigFormat = 'js';
-    }
-
-    // Read existing config
-    let existingConfig = {};
-    if (existsSync(projectConfigPath)) {
-      try {
-        let result = explorer.load(projectConfigPath);
-        existingConfig = result?.config?.default || result?.config || {};
-      } catch {
-        // Start fresh if corrupted
-      }
-    }
-
-    // Merge updates
-    let newConfig = mergeDeep(existingConfig, updates);
-
-    // Write based on format
-    if (projectConfigFormat === 'json') {
-      await writeFile(projectConfigPath, JSON.stringify(newConfig, null, 2));
-    } else {
-      // Write as ES module
-      let content = `import { defineConfig } from '@vizzly-testing/cli/config';
-
-export default defineConfig(${JSON.stringify(newConfig, null, 2)});
-`;
-      await writeFile(projectConfigPath, content);
-    }
-
-    // Clear cosmiconfig cache so next read gets fresh data
-    explorer.clearCaches();
-    return {
-      success: true,
-      path: projectConfigPath
-    };
-  }
-
-  /**
-   * Update global configuration (~/.vizzly/config.json)
-   */
-  async function updateGlobalConfig(updates) {
-    let globalConfig = await loadGlobalConfig();
-    if (!globalConfig.settings) {
-      globalConfig.settings = {};
-    }
-    globalConfig.settings = mergeDeep(globalConfig.settings, updates);
-    await saveGlobalConfig(globalConfig);
-    return {
-      success: true
-    };
-  }
-
-  /**
-   * Validate configuration
-   * @param {Object} config - Config to validate
-   * @returns {Promise<Object>}
-   */
-  async function validateConfig(config) {
-    let errors = [];
-    let warnings = [];
-
-    // Validate threshold
-    if (config.comparison?.threshold !== undefined) {
-      let threshold = config.comparison.threshold;
-      if (typeof threshold !== 'number' || !Number.isFinite(threshold) || threshold < 0) {
-        errors.push('comparison.threshold must be a non-negative number');
-      } else if (threshold > 100) {
-        warnings.push('comparison.threshold above 100 may cause all comparisons to pass');
-      }
-    }
-
-    // Validate minClusterSize
-    if (config.comparison?.minClusterSize !== undefined) {
-      let minClusterSize = config.comparison.minClusterSize;
-      if (!Number.isInteger(minClusterSize) || minClusterSize < 1) {
-        errors.push('comparison.minClusterSize must be a positive integer (1 or greater)');
-      } else if (minClusterSize > 100) {
-        warnings.push('comparison.minClusterSize above 100 may filter out most differences');
-      }
-    }
-
-    // Validate port
-    if (config.server?.port !== undefined) {
-      let port = config.server.port;
-      if (!Number.isInteger(port) || port < 1 || port > 65535) {
-        errors.push('server.port must be an integer between 1 and 65535');
-      } else if (port < 1024) {
-        warnings.push('server.port below 1024 may require elevated privileges');
-      }
-    }
-
-    // Validate timeout
-    if (config.server?.timeout !== undefined) {
-      let timeout = config.server.timeout;
-      if (!Number.isInteger(timeout) || timeout < 0) {
-        errors.push('server.timeout must be a non-negative integer');
-      }
-    }
-    return {
-      valid: errors.length === 0,
-      errors,
-      warnings
-    };
-  }
-  return {
-    getConfig,
-    updateConfig,
-    validateConfig
-  };
-}
-
-/**
- * Deep merge two objects
- */
-function mergeDeep(target, source) {
-  let result = cloneConfigValue(target);
-  for (let key in source) {
-    if (source[key] && typeof source[key] === 'object' && !Array.isArray(source[key])) {
-      result[key] = mergeDeep(result[key] || {}, source[key]);
-    } else if (Array.isArray(source[key])) {
-      result[key] = [...source[key]];
-    } else {
-      result[key] = source[key];
-    }
-  }
-  return result;
-}
-function cloneConfigValue(value) {
-  if (Array.isArray(value)) {
-    return value.map(cloneConfigValue);
-  }
-  if (value && typeof value === 'object') {
-    let clone = {};
-    for (let key in value) {
-      clone[key] = cloneConfigValue(value[key]);
-    }
-    return clone;
-  }
-  return value;
-}
-
-/**
- * Merge config with source tracking
- */
-function mergeWithTracking(target, source, sources, sourceName, sectionName) {
-  for (let key in source) {
-    let sourceSection = sectionName || key;
-    if (source[key] && typeof source[key] === 'object' && !Array.isArray(source[key])) {
-      if (!target[key]) target[key] = {};
-      mergeWithTracking(target[key], source[key], sources, sourceName, sourceSection);
-    } else {
-      target[key] = source[key];
-      sources[sourceSection] = sourceName;
-    }
-  }
-}

package/dist/services/project-service.js

@@ -1,88 +0,0 @@
-/**
- * Project Service
- * Wraps project operations for use by the HTTP server
- *
- * Provides the interface expected by src/server/routers/projects.js:
- * - listProjects() - Returns [] if not authenticated
- * - getRecentBuilds(projectSlug, organizationSlug, options) - Returns [] if not authenticated
- *
- * Error handling:
- * - API methods (listProjects, getRecentBuilds) return empty arrays when not authenticated
- */
-
-import { createAuthClient } from '../auth/client.js';
-import * as projectOps from '../project/operations.js';
-import { getApiUrl } from '../utils/environment-config.js';
-import { getAuthTokens } from '../utils/global-config.js';
-
-/**
- * Create a project service instance
- * @param {Object} [options]
- * @param {string} [options.apiUrl] - API base URL (defaults to VIZZLY_API_URL or https://app.vizzly.dev)
- * @param {Object} [options.httpClient] - Injectable HTTP client (for testing)
- * @param {Function} [options.getAuthTokens] - Injectable token getter (for testing)
- * @returns {Object} Project service
- */
-export function createProjectService(options = {}) {
-  let apiUrl = options.apiUrl || getApiUrl();
-
-  // Create HTTP client once at service creation (not per-request)
-  // Allow injection for testing
-  let httpClient = options.httpClient || createAuthClient({
-    baseUrl: apiUrl
-  });
-
-  // Allow injection of getAuthTokens for testing
-  let tokenGetter = options.getAuthTokens || getAuthTokens;
-
-  /**
-   * Create an OAuth client with current access token
-   * @returns {Promise<Object|null>} OAuth client or null if not authenticated
-   */
-  async function createOAuthClient() {
-    let auth = await tokenGetter();
-    if (!auth?.accessToken) {
-      return null;
-    }
-
-    // Wrap authenticatedRequest to auto-inject the access token
-    return {
-      authenticatedRequest: (endpoint, fetchOptions = {}) => httpClient.authenticatedRequest(endpoint, auth.accessToken, fetchOptions)
-    };
-  }
-  return {
-    /**
-     * List all projects from API
-     * Returns empty array if not authenticated (projectOps handles null oauthClient)
-     * @returns {Promise<Array>} Array of projects, empty if not authenticated
-     */
-    async listProjects() {
-      let oauthClient = await createOAuthClient();
-      // projectOps.listProjects handles null oauthClient by returning []
-      return projectOps.listProjects({
-        oauthClient,
-        apiClient: null
-      });
-    },
-    /**
-     * Get recent builds for a project
-     * Returns empty array if not authenticated (projectOps handles null oauthClient)
-     * @param {string} projectSlug - Project slug
-     * @param {string} organizationSlug - Organization slug
-     * @param {Object} options - Query options
-     * @returns {Promise<Array>} Array of builds, empty if not authenticated
-     */
-    async getRecentBuilds(projectSlug, organizationSlug, options = {}) {
-      let oauthClient = await createOAuthClient();
-      // projectOps.getRecentBuilds handles null oauthClient by returning []
-      return projectOps.getRecentBuilds({
-        oauthClient,
-        apiClient: null,
-        projectSlug,
-        organizationSlug,
-        limit: options.limit,
-        branch: options.branch
-      });
-    }
-  };
-}

package/dist/test-runner/core.js

@@ -1,293 +0,0 @@
-/**
- * Test Runner Core - Pure functions for test execution logic
- *
- * No I/O, no side effects - just data transformations.
- */
-
-// ============================================================================
-// Environment Building
-// ============================================================================
-
-/**
- * Build environment variables for test process execution
- * @param {Object} options - Options
- * @param {number} options.port - Server port
- * @param {string} options.buildId - Build ID
- * @param {boolean} [options.setBaseline] - Whether to set baseline
- * @param {boolean} [options.failOnDiff] - Whether visual diffs should fail tests
- * @param {Object} [options.baseEnv] - Base environment (defaults to process.env)
- * @returns {Object} Environment variables object
- */
-export function buildTestEnv({
-  port,
-  buildId,
-  setBaseline = false,
-  failOnDiff = false,
-  baseEnv = process.env
-}) {
-  return {
-    ...baseEnv,
-    VIZZLY_SERVER_URL: `http://localhost:${port}`,
-    VIZZLY_BUILD_ID: buildId,
-    VIZZLY_ENABLED: 'true',
-    VIZZLY_SET_BASELINE: setBaseline ? 'true' : 'false',
-    VIZZLY_FAIL_ON_DIFF: failOnDiff ? 'true' : 'false'
-  };
-}
-
-/**
- * Build environment for disabled Vizzly (allowNoToken mode)
- * @param {Object} [baseEnv] - Base environment (defaults to process.env)
- * @returns {Object} Environment variables object
- */
-export function buildDisabledEnv(baseEnv = process.env) {
-  return {
-    ...baseEnv,
-    VIZZLY_ENABLED: 'false'
-  };
-}
-
-// ============================================================================
-// Build Payload Building
-// ============================================================================
-
-/**
- * Build API build payload from options
- * @param {Object} options - Run options
- * @param {Object} [comparisonConfig] - Comparison configuration (threshold, minClusterSize)
- * @returns {Object} Build payload for API
- */
-export function buildApiBuildPayload(options, comparisonConfig = null) {
-  let payload = {
-    name: options.buildName || `Test Run ${new Date().toISOString()}`,
-    branch: options.branch || 'main',
-    environment: options.environment || 'test',
-    commit_sha: options.commit,
-    commit_message: options.message,
-    commit_author_name: options.commitAuthorName,
-    commit_author_email: options.commitAuthorEmail,
-    github_pull_request_number: options.pullRequestNumber,
-    parallel_id: options.parallelId
-  };
-
-  // Only include metadata if we have meaningful config to send
-  if (comparisonConfig?.threshold != null || comparisonConfig?.minClusterSize != null) {
-    payload.metadata = {
-      comparison: {
-        threshold: comparisonConfig.threshold,
-        minClusterSize: comparisonConfig.minClusterSize
-      }
-    };
-  }
-  return payload;
-}
-
-// ============================================================================
-// Mode Determination
-// ============================================================================
-
-/**
- * Determine if we should skip Vizzly integration entirely
- * @param {Object} options - Options
- * @param {boolean} options.allowNoToken - Whether to allow running without token
- * @param {boolean} options.hasApiKey - Whether an API key is available
- * @param {boolean} options.tdd - Whether in TDD mode
- * @returns {boolean} True if Vizzly should be disabled
- */
-export function shouldDisableVizzly({
-  allowNoToken,
-  hasApiKey,
-  tdd
-}) {
-  return allowNoToken && !hasApiKey && !tdd;
-}
-
-/**
- * Determine build mode
- * @param {boolean} tdd - Whether TDD mode is enabled
- * @returns {'tdd'|'api'} Build mode
- */
-export function determineBuildMode(tdd) {
-  return tdd ? 'tdd' : 'api';
-}
-
-// ============================================================================
-// Result Building
-// ============================================================================
-
-/**
- * Build result object for disabled run (no Vizzly integration)
- * @returns {Object} Result object
- */
-export function buildDisabledRunResult(reason = null) {
-  return {
-    testsPassed: 1,
-    testsFailed: 0,
-    screenshotsCaptured: 0,
-    ...(reason ? {
-      capture: {
-        setup: 'disabled',
-        reason,
-        accepted: 0,
-        rejected: 0,
-        errors: 0
-      }
-    } : {})
-  };
-}
-
-/**
- * Build final run result object
- * @param {Object} options - Options
- * @param {string|null} options.buildId - Build ID
- * @param {string|null} options.buildUrl - Build URL
- * @param {boolean} options.testSuccess - Whether tests passed
- * @param {number} options.screenshotCount - Number of screenshots
- * @param {Object|null} options.tddResults - TDD results (comparisons, etc.)
- * @param {Object|null} [options.uploadStats] - Cumulative cloud upload outcomes,
- * exposed as `uploads` independently of test and local comparison results.
- * @returns {Object} Final result object
- */
-export function buildRunResult({
-  buildId,
-  buildUrl,
-  testSuccess,
-  screenshotCount,
-  tddResults,
-  uploadStats
-}) {
-  return {
-    buildId,
-    url: buildUrl,
-    testsPassed: testSuccess ? 1 : 0,
-    testsFailed: testSuccess ? 0 : 1,
-    screenshotsCaptured: screenshotCount,
-    ...(uploadStats ? {
-      uploads: uploadStats
-    } : {}),
-    comparisons: tddResults?.comparisons || null,
-    failed: (tddResults?.failed || 0) > 0,
-    ...(tddResults ? {
-      summary: {
-        total: tddResults.total ?? screenshotCount,
-        passed: tddResults.passed || 0,
-        failed: tddResults.failed || 0,
-        new: tddResults.new || 0,
-        errors: tddResults.errors || 0
-      },
-      capture: {
-        setup: 'ready',
-        reason: tddResults.error || null,
-        accepted: tddResults.capture?.accepted ?? screenshotCount,
-        rejected: tddResults.capture?.rejected || 0,
-        errors: tddResults.errors || 0
-      }
-    } : {})
-  };
-}
-
-// ============================================================================
-// Validation
-// ============================================================================
-
-/**
- * Validate test command is provided
- * @param {string|undefined} testCommand - Test command to validate
- * @returns {{ valid: boolean, error: string|null }}
- */
-export function validateTestCommand(testCommand) {
-  if (!testCommand) {
-    return {
-      valid: false,
-      error: 'No test command provided'
-    };
-  }
-  return {
-    valid: true,
-    error: null
-  };
-}
-
-/**
- * Validate daemon mode requirements
- * @param {Object} options - Options
- * @param {boolean} options.tdd - Whether TDD mode is enabled
- * @param {boolean} options.daemon - Whether daemon mode is enabled
- * @returns {{ valid: boolean, error: string|null }}
- */
-export function validateDaemonMode({
-  tdd,
-  daemon
-}) {
-  if (!tdd || !daemon) {
-    return {
-      valid: false,
-      error: 'Initialize method is only for TDD daemon mode'
-    };
-  }
-  return {
-    valid: true,
-    error: null
-  };
-}
-
-// ============================================================================
-// Client Options Building
-// ============================================================================
-
-/**
- * Build API client options from config
- * @param {Object} config - Configuration object
- * @returns {Object|null} Client options or null if no API key
- */
-export function buildClientOptions(config) {
-  if (!config?.apiKey) {
-    return null;
-  }
-  return {
-    baseUrl: config.apiUrl,
-    token: config.apiKey,
-    command: 'run'
-  };
-}
-
-/**
- * Check if API key is available
- * @param {Object} config - Configuration object
- * @returns {boolean} Whether API key exists
- */
-export function hasApiKey(config) {
-  return Boolean(config?.apiKey);
-}
-
-// ============================================================================
-// Spawn Options Building
-// ============================================================================
-
-/**
- * Build spawn options for test command execution
- * @param {Object} env - Environment variables
- * @param {Object} [options] - Spawn display options
- * @param {boolean} [options.json] - Keep stdout reserved for CLI JSON output
- * @returns {Object} Spawn options
- */
-export function buildSpawnOptions(env, options = {}) {
-  return {
-    env,
-    stdio: options.json ? ['inherit', 'pipe', 'pipe'] : 'inherit',
-    shell: true
-  };
-}
-
-// ============================================================================
-// Set Baseline Normalization
-// ============================================================================
-
-/**
- * Normalize setBaseline option (handles both camelCase and kebab-case)
- * @param {Object} options - Options object
- * @returns {boolean} Whether to set baseline
- */
-export function normalizeSetBaseline(options) {
-  return Boolean(options?.setBaseline || options?.['set-baseline']);
-}

package/dist/test-runner/index.js

@@ -1,13 +0,0 @@
-/**
- * Test Runner Module - Public exports
- *
- * Provides functional test runner primitives:
- * - core.js: Pure functions for building env, payloads, results
- * - operations.js: Test execution operations with dependency injection
- */
-
-// Core pure functions
-export { buildApiBuildPayload, buildClientOptions, buildDisabledEnv, buildDisabledRunResult, buildRunResult, buildSpawnOptions, buildTestEnv, determineBuildMode, hasApiKey, normalizeSetBaseline, shouldDisableVizzly, validateDaemonMode, validateTestCommand } from './core.js';
-
-// Test runner operations (take dependencies as parameters)
-export { cancelTests, createBuild, executeTestCommand, fetchBuildUrl, finalizeBuild, initializeDaemon, runTests } from './operations.js';