@vizzly-testing/cli 0.21.2 → 0.22.1

package/dist/cli.js

@@ -16,6 +16,8 @@ import { validateWhoamiOptions, whoamiCommand } from './commands/whoami.js';
 import { createPluginServices } from './plugin-api.js';
 import { loadPlugins } from './plugin-loader.js';
 import { createServices } from './services/index.js';
+import { generateStaticReport, getReportFileUrl } from './services/static-report-generator.js';
+import { openBrowser } from './utils/browser.js';
 import { colors } from './utils/colors.js';
 import { loadConfig } from './utils/config-loader.js';
 import { getContext } from './utils/context.js';
@@ -274,7 +276,7 @@ program.command('upload').description('Upload screenshots to Vizzly').argument('
 const tddCmd = program.command('tdd').description('Run tests in TDD mode with local visual comparisons');
 
 // TDD Start - Background server
-tddCmd.command('start').description('Start background TDD server with dashboard').option('--port <port>', 'Port for TDD server', '47392').option('--open', 'Open dashboard in browser').option('--baseline-build <id>', 'Use specific build as baseline').option('--baseline-comparison <id>', 'Use specific comparison as baseline').option('--environment <env>', 'Environment name', 'test').option('--threshold <number>', 'Comparison threshold', parseFloat).option('--timeout <ms>', 'Server timeout in milliseconds', '30000').option('--token <token>', 'API token override').option('--daemon-child', 'Internal: run as daemon child process').action(async options => {
+tddCmd.command('start').description('Start background TDD server with dashboard').option('--port <port>', 'Port for TDD server', '47392').option('--open', 'Open dashboard in browser').option('--baseline-build <id>', 'Use specific build as baseline').option('--baseline-comparison <id>', 'Use specific comparison as baseline').option('--environment <env>', 'Environment name', 'test').option('--threshold <number>', 'Comparison threshold', parseFloat).option('--timeout <ms>', 'Server timeout in milliseconds', '30000').option('--fail-on-diff', 'Fail tests when visual differences are detected').option('--token <token>', 'API token override').option('--daemon-child', 'Internal: run as daemon child process').action(async options => {
   const globalOptions = program.opts();
 
   // If this is a daemon child process, run the server directly
@@ -298,7 +300,7 @@ tddCmd.command('status').description('Check TDD server status').action(async opt
 });
 
 // TDD Run - One-off test run with ephemeral server (generates static report)
-tddCmd.command('run <command>').description('Run tests once in TDD mode with local visual comparisons').option('--port <port>', 'Port for TDD server', '47392').option('--branch <branch>', 'Git branch override').option('--environment <env>', 'Environment name', 'test').option('--threshold <number>', 'Comparison threshold', parseFloat).option('--token <token>', 'API token override').option('--timeout <ms>', 'Server timeout in milliseconds', '30000').option('--baseline-build <id>', 'Use specific build as baseline').option('--baseline-comparison <id>', 'Use specific comparison as baseline').option('--set-baseline', 'Accept current screenshots as new baseline (overwrites existing)').action(async (command, options) => {
+tddCmd.command('run <command>').description('Run tests once in TDD mode with local visual comparisons').option('--port <port>', 'Port for TDD server', '47392').option('--branch <branch>', 'Git branch override').option('--environment <env>', 'Environment name', 'test').option('--threshold <number>', 'Comparison threshold', parseFloat).option('--token <token>', 'API token override').option('--timeout <ms>', 'Server timeout in milliseconds', '30000').option('--baseline-build <id>', 'Use specific build as baseline').option('--baseline-comparison <id>', 'Use specific comparison as baseline').option('--set-baseline', 'Accept current screenshots as new baseline (overwrites existing)').option('--fail-on-diff', 'Fail tests when visual differences are detected').option('--no-open', 'Skip opening report in browser').action(async (command, options) => {
   const globalOptions = program.opts();
 
   // Validate options
@@ -315,21 +317,49 @@ tddCmd.command('run <command>').description('Run tests once in TDD mode with loc
     cleanup
   } = await tddCommand(command, options, globalOptions);
 
-  // Set up cleanup on process signals
+  // Track cleanup state to prevent double cleanup
+  let isCleaningUp = false;
   const handleCleanup = async () => {
+    if (isCleaningUp) return;
+    isCleaningUp = true;
     await cleanup();
   };
-  process.once('SIGINT', () => {
-    handleCleanup().then(() => process.exit(1));
-  });
-  process.once('SIGTERM', () => {
-    handleCleanup().then(() => process.exit(1));
-  });
-  if (result && !result.success && result.exitCode > 0) {
-    await cleanup();
-    process.exit(result.exitCode);
+
+  // Set up cleanup on process signals
+  const sigintHandler = () => {
+    handleCleanup().then(() => process.exit(result?.exitCode || 0));
+  };
+  const sigtermHandler = () => {
+    handleCleanup().then(() => process.exit(result?.exitCode || 0));
+  };
+  process.once('SIGINT', sigintHandler);
+  process.once('SIGTERM', sigtermHandler);
+
+  // If there are comparisons, generate static report
+  const hasComparisons = result?.comparisons?.length > 0;
+  if (hasComparisons) {
+    // Note: Tests have completed at this point, so report-data.json is stable.
+    // The report reflects the final state of all comparisons.
+    const reportResult = await generateStaticReport(process.cwd());
+    if (reportResult.success) {
+      const reportUrl = getReportFileUrl(reportResult.reportPath);
+      output.print(`  ${colors.brand.textTertiary('→')} Report: ${colors.blue(reportUrl)}`);
+      output.blank();
+
+      // Open report in browser unless --no-open
+      if (options.open !== false) {
+        await openBrowser(reportUrl);
+      }
+    } else {
+      output.warn(`Failed to generate static report: ${reportResult.error}`);
+    }
   }
-  await cleanup();
+
+  // Remove signal handlers before normal cleanup to prevent double cleanup
+  process.off('SIGINT', sigintHandler);
+  process.off('SIGTERM', sigtermHandler);
+  await handleCleanup();
+  process.exit(result?.exitCode || 0);
 });
 program.command('run').description('Run tests with Vizzly integration').argument('<command>', 'Test command to run').option('--port <port>', 'Port for screenshot server', '47392').option('-b, --build-name <name>', 'Custom build name').option('--branch <branch>', 'Git branch override').option('--commit <sha>', 'Git commit SHA').option('--message <msg>', 'Commit message').option('--environment <env>', 'Environment name', 'test').option('--token <token>', 'API token override').option('--wait', 'Wait for build completion').option('--timeout <ms>', 'Server timeout in milliseconds', '30000').option('--allow-no-token', 'Allow running without API token').option('--upload-all', 'Upload all screenshots without SHA deduplication').option('--parallel-id <id>', 'Unique identifier for parallel test execution').action(async (command, options) => {
   const globalOptions = program.opts();

package/dist/client/index.js

@@ -356,8 +356,8 @@ export function isVizzlyReady() {
  * @param {boolean} [config.enabled] - Enable/disable screenshots
  */
 export function configure(config = {}) {
-  if (config.serverUrl) {
-    currentClient = createSimpleClient(config.serverUrl);
+  if ('serverUrl' in config) {
+    currentClient = config.serverUrl ? createSimpleClient(config.serverUrl) : null;
   }
   if (typeof config.enabled === 'boolean') {
     setVizzlyEnabled(config.enabled);

package/dist/commands/tdd-daemon.js

@@ -56,7 +56,7 @@ export async function tddStartCommand(options = {}, globalOptions = {}) {
     // CLI entry point
     'tdd', 'start', '--daemon-child',
     // Special flag for child process
-    '--port', port.toString(), ...(options.open ? ['--open'] : []), ...(options.baselineBuild ? ['--baseline-build', options.baselineBuild] : []), ...(options.baselineComparison ? ['--baseline-comparison', options.baselineComparison] : []), ...(options.environment ? ['--environment', options.environment] : []), ...(options.threshold !== undefined ? ['--threshold', options.threshold.toString()] : []), ...(options.timeout ? ['--timeout', options.timeout] : []), ...(options.token ? ['--token', options.token] : []), ...(globalOptions.json ? ['--json'] : []), ...(globalOptions.verbose ? ['--verbose'] : []), ...(globalOptions.noColor ? ['--no-color'] : [])], {
+    '--port', port.toString(), ...(options.open ? ['--open'] : []), ...(options.baselineBuild ? ['--baseline-build', options.baselineBuild] : []), ...(options.baselineComparison ? ['--baseline-comparison', options.baselineComparison] : []), ...(options.environment ? ['--environment', options.environment] : []), ...(options.threshold !== undefined ? ['--threshold', options.threshold.toString()] : []), ...(options.timeout ? ['--timeout', options.timeout] : []), ...(options.failOnDiff ? ['--fail-on-diff'] : []), ...(options.token ? ['--token', options.token] : []), ...(globalOptions.json ? ['--json'] : []), ...(globalOptions.verbose ? ['--verbose'] : []), ...(globalOptions.noColor ? ['--no-color'] : [])], {
       detached: true,
       stdio: ['ignore', 'inherit', 'inherit', 'ipc'],
       cwd: process.cwd()
@@ -199,7 +199,8 @@ export async function runDaemonChild(options = {}, globalOptions = {}) {
     const serverInfo = {
       pid: process.pid,
       port: port,
-      startTime: Date.now()
+      startTime: Date.now(),
+      failOnDiff: options.failOnDiff || false
     };
     writeFileSync(join(vizzlyDir, 'server.json'), JSON.stringify(serverInfo, null, 2));