@vizzly-testing/cli 0.38.1 → 0.39.1

package/dist/cli.js

@@ -28,15 +28,15 @@ import { 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 { withTimeout } from './utils/async-utils.js';
-import { openBrowser } from './utils/browser.js';
+import { runUpdatedCli } from './utils/cli-update.js';
 import { colors } from './utils/colors.js';
 import { loadConfig } from './utils/config-loader.js';
 import { getContext } from './utils/context.js';
 import { saveUserPath } from './utils/global-config.js';
 import * as output from './utils/output.js';
 import { getPackageVersion } from './utils/package-info.js';
+import { notifyUpdates } from './utils/update-notifications.js';
 
 // Custom help formatting with BearDen design system
 const formatHelp = (cmd, helper) => {
@@ -395,7 +395,7 @@ function writeCommanderError(text) {
     }
   }
 }
-program.name('vizzly').description('Vizzly CLI for visual regression testing').version(getPackageVersion()).option('-c, --config <path>', 'Config file path').option('--token <token>', 'Vizzly API token').option('-v, --verbose', 'Verbose output (shorthand for --log-level debug)').option('--log-level <level>', 'Log level: debug, info, warn, error (default: info, or VIZZLY_LOG_LEVEL env var)').option('--json [fields]', 'JSON output, optionally specify fields (e.g., --json id,status,branch)').option('--color', 'Force colored output (even in non-TTY)').option('--no-color', 'Disable colored output').option('--strict', 'Fail on any error (default: be resilient, warn on non-critical issues)').configureHelp({
+program.name('vizzly').description('Vizzly CLI for visual regression testing').version(getPackageVersion()).option('-c, --config <path>', 'Config file path').option('--token <token>', 'Vizzly API token').option('-v, --verbose', 'Verbose output (shorthand for --log-level debug)').option('--log-level <level>', 'Log level: debug, info, warn, error (default: info, or VIZZLY_LOG_LEVEL env var)').option('--json [fields]', 'JSON output, optionally specify fields (e.g., --json id,status,branch)').option('--color', 'Force colored output (even in non-TTY)').option('--no-color', 'Disable colored output').option('--no-update-notifier', 'Skip update checks and notices').option('--strict', 'Fail on any error (default: be resilient, warn on non-critical issues)').configureHelp({
   formatHelp
 });
 program.configureOutput({
@@ -404,6 +404,13 @@ program.configureOutput({
 program.showHelpAfterError('(Run vizzly --help for available commands)');
 program.showSuggestionAfterError();
 program.exitOverride();
+program.hook('preAction', async (_command, actionCommand) => {
+  if (actionCommand.name() === 'init') return;
+  let updatedCli = await notifyUpdates(actionCommand.optsWithGlobals());
+  if (!updatedCli) return;
+  let code = await runUpdatedCli(updatedCli, process.argv.slice(2), process.cwd());
+  process.exit(code);
+});
 
 // Load plugins before defining commands
 // We need to manually parse to get the config option early
@@ -430,38 +437,49 @@ output.configure({
   color: colorOverride,
   json: jsonArg
 });
-let config = await loadConfig(configPath, {
-  token: tokenArg
-});
-let services = createServices(config);
-let pluginServices = createPluginServices(services);
-let plugins = [];
-try {
-  plugins = await loadPlugins(configPath, config);
-  for (let plugin of plugins) {
-    try {
-      // Add timeout protection for plugin registration (5 seconds)
-      let registerPromise = plugin.register(program, {
-        config,
-        services: pluginServices,
-        output
-      });
-      await withTimeout(registerPromise, 5000, 'Plugin registration timeout (5s)');
-      output.debug(`Registered plugin: ${plugin.name}`);
-    } catch (error) {
-      output.warn(`Failed to register plugin ${plugin.name}: ${error.message}`);
+let setupCommand = findRequestedCommand(process.argv, new Set(['init', 'project']))?.name;
+let needsRuntimeBootstrap = !['init', 'project'].includes(setupCommand);
+if (needsRuntimeBootstrap) {
+  let config = await loadConfig(configPath, {
+    token: tokenArg
+  });
+  let services = createServices(config);
+  let pluginServices = createPluginServices(services);
+  try {
+    let plugins = await loadPlugins(configPath, config);
+    for (let plugin of plugins) {
+      try {
+        // Add timeout protection for plugin registration (5 seconds)
+        let registerPromise = plugin.register(program, {
+          config,
+          services: pluginServices,
+          output
+        });
+        await withTimeout(registerPromise, 5000, 'Plugin registration timeout (5s)');
+        output.debug(`Registered plugin: ${plugin.name}`);
+      } catch (error) {
+        output.warn(`Failed to register plugin ${plugin.name}: ${error.message}`);
+      }
     }
+  } catch (error) {
+    output.debug(`Plugin loading failed: ${error.message}`);
   }
-} catch (error) {
-  output.debug(`Plugin loading failed: ${error.message}`);
 }
-program.command('init').description('Initialize Vizzly in your project').option('--force', 'Overwrite existing configuration').option('--agent-skill', 'Install or refresh the repo-local Vizzly agent skill').option('--agent-guidance', 'Install or refresh the repo-local Vizzly skill and AGENTS.md guidance').option('--skip-agent-skill', 'Skip the Vizzly agent skill prompt').action(async options => {
+program.command('init').description('Initialize Vizzly in your project').option('--quick', 'Create an editable config without the setup wizard').option('--force', 'Overwrite existing configuration').option('--agent-skill', 'Install or refresh the Vizzly agent skill').option('--agent-guidance', 'Install or refresh the Vizzly skill and AGENTS.md guidance').option('--skip-agent-skill', 'Skip the Vizzly agent skill prompt').action(async options => {
   let globalOptions = getGlobalOptions();
-  await init({
-    ...globalOptions,
-    ...options,
-    plugins
-  });
+  try {
+    let result = await init({
+      ...globalOptions,
+      ...options
+    });
+    process.exitCode = result.exitCode || 0;
+  } catch (error) {
+    output.error(error.message, error, {
+      changes: error.completed || []
+    });
+    if (error.completed?.length) output.list(error.completed.map(change => change.path));
+    process.exitCode = error.code === 'INIT_INPUT_REQUIRED' ? 2 : 1;
+  }
 });
 program.command('upload').description('Upload screenshots to Vizzly').argument('<path>', 'Path to screenshots directory').option('-b, --build-name <name>', 'Build name for grouping').option('-m, --metadata <json>', 'Additional metadata as JSON').option('--batch-size <n>', 'Upload batch size', Number).option('--upload-timeout <ms>', 'Upload timeout in milliseconds', Number).option('--branch <branch>', 'Git branch').option('--commit <sha>', 'Git commit SHA').option('--message <msg>', 'Commit message').option('--environment <env>', 'Environment name', 'test').option('--threshold <number>', 'Comparison threshold', Number).option('--min-cluster-size <pixels>', 'Minimum changed-pixel cluster size', Number).option('--token <token>', 'API token override').option('--wait', 'Wait for build completion').option('--upload-all', 'Upload all screenshots without SHA deduplication').option('--parallel-id <id>', 'Unique identifier for parallel test execution').action(async (path, options) => {
   let globalOptions = getGlobalOptions();
@@ -555,7 +573,10 @@ tddCmd.command('run <command>').description('Run tests once in TDD mode with loc
   const {
     result,
     cleanup
-  } = await tddCommand(command, options, globalOptions);
+  } = await tddCommand(command, {
+    ...options,
+    autoPort: options.port === undefined
+  }, globalOptions);
 
   // Track cleanup state to prevent double cleanup
   let isCleaningUp = false;
@@ -575,26 +596,6 @@ tddCmd.command('run <command>').description('Run tests once in TDD mode with loc
   process.once('SIGINT', sigintHandler);
   process.once('SIGTERM', sigtermHandler);
 
-  // If the run captured screenshots, generate a static report for review.
-  let hasComparisons = result?.screenshotsCaptured > 0 || result?.comparisons?.length > 0 || result?.summary?.total > 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.info(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}`);
-    }
-  }
-
   // Remove signal handlers before normal cleanup to prevent double cleanup
   process.off('SIGINT', sigintHandler);
   process.off('SIGTERM', sigtermHandler);
@@ -950,7 +951,7 @@ program.command('whoami').description('Show current authentication status and us
 
 // Save PATH for the menubar app before commands read or update credentials
 // in the same config file.
-await saveUserPath().catch(() => {});
+if (needsRuntimeBootstrap) await saveUserPath().catch(() => {});
 let commandNames = new Set(program.commands.map(command => command.name()));
 let nestedCommandNames = new Map(program.commands.map(command => [command.name(), new Set(command.registeredArguments.length ? [] : command.commands.map(subcommand => subcommand.name()))]));
 let normalizedArgv = normalizeJsonArgv(process.argv, commandNames);
@@ -971,7 +972,7 @@ if (requestedCommand && !requestedCommand.known) {
   process.exit(1);
 }
 try {
-  program.parse(normalizedArgv);
+  await program.parseAsync(normalizedArgv);
 } catch (error) {
   if (error.exitCode == null) {
     output.error('Unexpected CLI error', error);