@vizzly-testing/cli 0.17.0 → 0.18.0

package/dist/server/routers/config.js

@@ -3,9 +3,9 @@
  * Handles configuration management endpoints
  */
 
-import { parseJsonBody } from '../middleware/json-parser.js';
-import { sendSuccess, sendError, sendServiceUnavailable } from '../middleware/response.js';
 import * as output from '../../utils/output.js';
+import { parseJsonBody } from '../middleware/json-parser.js';
+import { sendError, sendServiceUnavailable, sendSuccess } from '../middleware/response.js';
 
 /**
  * Create config router
@@ -26,7 +26,7 @@ export function createConfigRouter({
     // Get merged config with sources
     if (req.method === 'GET' && pathname === '/api/config') {
       try {
-        let configData = await configService.getConfig('merged');
+        const configData = await configService.getConfig('merged');
         sendSuccess(res, configData);
         return true;
       } catch (error) {
@@ -41,7 +41,7 @@ export function createConfigRouter({
     // Get project-level config
     if (req.method === 'GET' && pathname === '/api/config/project') {
       try {
-        let configData = await configService.getConfig('project');
+        const configData = await configService.getConfig('project');
         sendSuccess(res, configData);
         return true;
       } catch (error) {
@@ -56,7 +56,7 @@ export function createConfigRouter({
     // Get global config
     if (req.method === 'GET' && pathname === '/api/config/global') {
       try {
-        let configData = await configService.getConfig('global');
+        const configData = await configService.getConfig('global');
         sendSuccess(res, configData);
         return true;
       } catch (error) {
@@ -71,8 +71,8 @@ export function createConfigRouter({
     // Update project config
     if (req.method === 'POST' && pathname === '/api/config/project') {
       try {
-        let body = await parseJsonBody(req);
-        let result = await configService.updateConfig('project', body);
+        const body = await parseJsonBody(req);
+        const result = await configService.updateConfig('project', body);
         sendSuccess(res, {
           success: true,
           ...result
@@ -90,8 +90,8 @@ export function createConfigRouter({
     // Update global config
     if (req.method === 'POST' && pathname === '/api/config/global') {
       try {
-        let body = await parseJsonBody(req);
-        let result = await configService.updateConfig('global', body);
+        const body = await parseJsonBody(req);
+        const result = await configService.updateConfig('global', body);
         sendSuccess(res, {
           success: true,
           ...result
@@ -109,8 +109,8 @@ export function createConfigRouter({
     // Validate config
     if (req.method === 'POST' && pathname === '/api/config/validate') {
       try {
-        let body = await parseJsonBody(req);
-        let result = await configService.validateConfig(body);
+        const body = await parseJsonBody(req);
+        const result = await configService.validateConfig(body);
         sendSuccess(res, result);
         return true;
       } catch (error) {

package/dist/server/routers/dashboard.js

@@ -3,13 +3,13 @@
  * Serves the React SPA for all dashboard routes
  */
 
-import { readFileSync, existsSync } from 'fs';
-import { join } from 'path';
-import { sendHtml, sendSuccess } from '../middleware/response.js';
+import { existsSync, readFileSync } from 'node:fs';
+import { join } from 'node:path';
 import * as output from '../../utils/output.js';
+import { sendHtml, sendSuccess } from '../middleware/response.js';
 
 // SPA routes that should serve the dashboard HTML
-let SPA_ROUTES = ['/', '/dashboard', '/stats', '/settings', '/projects', '/builds'];
+const SPA_ROUTES = ['/', '/dashboard', '/stats', '/settings', '/projects', '/builds'];
 
 /**
  * Create dashboard router
@@ -24,10 +24,10 @@ export function createDashboardRouter() {
 
     // API endpoint for fetching report data
     if (pathname === '/api/report-data') {
-      let reportDataPath = join(process.cwd(), '.vizzly', 'report-data.json');
+      const reportDataPath = join(process.cwd(), '.vizzly', 'report-data.json');
       if (existsSync(reportDataPath)) {
         try {
-          let data = readFileSync(reportDataPath, 'utf8');
+          const data = readFileSync(reportDataPath, 'utf8');
           res.setHeader('Content-Type', 'application/json');
           res.statusCode = 200;
           res.end(data);
@@ -50,8 +50,8 @@ export function createDashboardRouter() {
 
     // API endpoint for real-time status
     if (pathname === '/api/status') {
-      let reportDataPath = join(process.cwd(), '.vizzly', 'report-data.json');
-      let baselineMetadataPath = join(process.cwd(), '.vizzly', 'baselines', 'metadata.json');
+      const reportDataPath = join(process.cwd(), '.vizzly', 'report-data.json');
+      const baselineMetadataPath = join(process.cwd(), '.vizzly', 'baselines', 'metadata.json');
       let reportData = null;
       let baselineInfo = null;
       if (existsSync(reportDataPath)) {
@@ -84,11 +84,11 @@ export function createDashboardRouter() {
 
     // Serve React SPA for dashboard routes
     if (SPA_ROUTES.includes(pathname) || pathname.startsWith('/comparison/')) {
-      let reportDataPath = join(process.cwd(), '.vizzly', 'report-data.json');
+      const reportDataPath = join(process.cwd(), '.vizzly', 'report-data.json');
       let reportData = null;
       if (existsSync(reportDataPath)) {
         try {
-          let data = readFileSync(reportDataPath, 'utf8');
+          const data = readFileSync(reportDataPath, 'utf8');
           reportData = JSON.parse(data);
         } catch (error) {
           output.debug('Could not read report data:', {
@@ -96,7 +96,7 @@ export function createDashboardRouter() {
           });
         }
       }
-      let dashboardHtml = `
+      const dashboardHtml = `
 <!DOCTYPE html>
 <html>
 <head>

package/dist/server/routers/health.js

@@ -3,8 +3,8 @@
  * Health check endpoint with diagnostics
  */
 
-import { readFileSync, existsSync } from 'fs';
-import { join } from 'path';
+import { existsSync, readFileSync } from 'node:fs';
+import { join } from 'node:path';
 import { sendSuccess } from '../middleware/response.js';
 
 /**
@@ -22,8 +22,8 @@ export function createHealthRouter({
     if (req.method !== 'GET' || pathname !== '/health') {
       return false;
     }
-    let reportDataPath = join(process.cwd(), '.vizzly', 'report-data.json');
-    let baselineMetadataPath = join(process.cwd(), '.vizzly', 'baselines', 'metadata.json');
+    const reportDataPath = join(process.cwd(), '.vizzly', 'report-data.json');
+    const baselineMetadataPath = join(process.cwd(), '.vizzly', 'baselines', 'metadata.json');
     let reportData = null;
     let baselineInfo = null;
     if (existsSync(reportDataPath)) {

package/dist/server/routers/projects.js

@@ -3,9 +3,9 @@
  * Handles project management and builds endpoints
  */
 
-import { parseJsonBody } from '../middleware/json-parser.js';
-import { sendSuccess, sendError, sendServiceUnavailable } from '../middleware/response.js';
 import * as output from '../../utils/output.js';
+import { parseJsonBody } from '../middleware/json-parser.js';
+import { sendError, sendServiceUnavailable, sendSuccess } from '../middleware/response.js';
 
 /**
  * Create projects router
@@ -26,7 +26,7 @@ export function createProjectsRouter({
     // List all projects from API
     if (req.method === 'GET' && pathname === '/api/projects') {
       try {
-        let projects = await projectService.listProjects();
+        const projects = await projectService.listProjects();
         sendSuccess(res, {
           projects
         });
@@ -43,7 +43,7 @@ export function createProjectsRouter({
     // List project directory mappings
     if (req.method === 'GET' && pathname === '/api/projects/mappings') {
       try {
-        let mappings = await projectService.listMappings();
+        const mappings = await projectService.listMappings();
         sendSuccess(res, {
           mappings
         });
@@ -60,15 +60,15 @@ export function createProjectsRouter({
     // Create or update project mapping
     if (req.method === 'POST' && pathname === '/api/projects/mappings') {
       try {
-        let body = await parseJsonBody(req);
-        let {
+        const body = await parseJsonBody(req);
+        const {
           directory,
           projectSlug,
           organizationSlug,
           token,
           projectName
         } = body;
-        let mapping = await projectService.createMapping(directory, {
+        const mapping = await projectService.createMapping(directory, {
           projectSlug,
           organizationSlug,
           token,
@@ -91,7 +91,7 @@ export function createProjectsRouter({
     // Delete project mapping
     if (req.method === 'DELETE' && pathname.startsWith('/api/projects/mappings/')) {
       try {
-        let directory = decodeURIComponent(pathname.replace('/api/projects/mappings/', ''));
+        const directory = decodeURIComponent(pathname.replace('/api/projects/mappings/', ''));
         await projectService.removeMapping(directory);
         sendSuccess(res, {
           success: true,
@@ -114,15 +114,15 @@ export function createProjectsRouter({
         return true;
       }
       try {
-        let currentDir = process.cwd();
-        let mapping = await projectService.getMapping(currentDir);
+        const currentDir = process.cwd();
+        const mapping = await projectService.getMapping(currentDir);
         if (!mapping || !mapping.projectSlug || !mapping.organizationSlug) {
           sendError(res, 400, 'No project configured for this directory');
           return true;
         }
-        let limit = parseInt(parsedUrl.searchParams.get('limit') || '10', 10);
-        let branch = parsedUrl.searchParams.get('branch') || undefined;
-        let builds = await projectService.getRecentBuilds(mapping.projectSlug, mapping.organizationSlug, {
+        const limit = parseInt(parsedUrl.searchParams.get('limit') || '10', 10);
+        const branch = parsedUrl.searchParams.get('branch') || undefined;
+        const builds = await projectService.getRecentBuilds(mapping.projectSlug, mapping.organizationSlug, {
           limit,
           branch
         });
@@ -140,14 +140,14 @@ export function createProjectsRouter({
     }
 
     // Get builds for a specific project (used by /builds page)
-    let projectBuildsMatch = pathname.match(/^\/api\/projects\/([^/]+)\/([^/]+)\/builds$/);
+    const projectBuildsMatch = pathname.match(/^\/api\/projects\/([^/]+)\/([^/]+)\/builds$/);
     if (req.method === 'GET' && projectBuildsMatch) {
       try {
-        let organizationSlug = decodeURIComponent(projectBuildsMatch[1]);
-        let projectSlug = decodeURIComponent(projectBuildsMatch[2]);
-        let limit = parseInt(parsedUrl.searchParams.get('limit') || '20', 10);
-        let branch = parsedUrl.searchParams.get('branch') || undefined;
-        let builds = await projectService.getRecentBuilds(projectSlug, organizationSlug, {
+        const organizationSlug = decodeURIComponent(projectBuildsMatch[1]);
+        const projectSlug = decodeURIComponent(projectBuildsMatch[2]);
+        const limit = parseInt(parsedUrl.searchParams.get('limit') || '20', 10);
+        const branch = parsedUrl.searchParams.get('branch') || undefined;
+        const builds = await projectService.getRecentBuilds(projectSlug, organizationSlug, {
           limit,
           branch
         });

package/dist/server/routers/screenshot.js

@@ -3,9 +3,9 @@
  * Handles screenshot uploads and legacy baseline accept
  */
 
-import { parseJsonBody } from '../middleware/json-parser.js';
-import { sendJson, sendError } from '../middleware/response.js';
 import * as output from '../../utils/output.js';
+import { parseJsonBody } from '../middleware/json-parser.js';
+import { sendError, sendJson } from '../middleware/response.js';
 
 /**
  * Create screenshot router
@@ -26,8 +26,8 @@ export function createScreenshotRouter({
     // Main screenshot upload endpoint
     if (pathname === '/screenshot') {
       try {
-        let body = await parseJsonBody(req);
-        let {
+        const body = await parseJsonBody(req);
+        const {
           buildId,
           name,
           properties,
@@ -39,8 +39,8 @@ export function createScreenshotRouter({
         }
 
         // Use buildId from request body, or fall back to server's buildId
-        let effectiveBuildId = buildId || defaultBuildId;
-        let result = await screenshotHandler.handleScreenshot(effectiveBuildId, name, image, properties);
+        const effectiveBuildId = buildId || defaultBuildId;
+        const result = await screenshotHandler.handleScreenshot(effectiveBuildId, name, image, properties);
         sendJson(res, result.statusCode, result.body);
         return true;
       } catch (error) {
@@ -55,8 +55,8 @@ export function createScreenshotRouter({
     // Legacy accept-baseline endpoint
     if (pathname === '/accept-baseline') {
       try {
-        let body = await parseJsonBody(req);
-        let {
+        const body = await parseJsonBody(req);
+        const {
           id
         } = body;
         if (!id) {
@@ -64,7 +64,7 @@ export function createScreenshotRouter({
           return true;
         }
         if (screenshotHandler.acceptBaseline) {
-          let result = await screenshotHandler.acceptBaseline(id);
+          const result = await screenshotHandler.acceptBaseline(id);
           sendJson(res, 200, {
             success: true,
             ...result

package/dist/services/api-service.js

@@ -3,12 +3,12 @@
  * Handles HTTP requests to the Vizzly API
  */
 
-import { URLSearchParams } from 'url';
-import { VizzlyError, AuthError } from '../errors/vizzly-error.js';
-import crypto from 'crypto';
-import { getPackageVersion } from '../utils/package-info.js';
-import { getApiUrl, getApiToken, getUserAgent } from '../utils/environment-config.js';
+import crypto from 'node:crypto';
+import { URLSearchParams } from 'node:url';
+import { AuthError, VizzlyError } from '../errors/vizzly-error.js';
+import { getApiToken, getApiUrl, getUserAgent } from '../utils/environment-config.js';
 import { getAuthTokens, saveAuthTokens } from '../utils/global-config.js';
+import { getPackageVersion } from '../utils/package-info.js';
 
 /**
  * ApiService class for direct API communication
@@ -66,11 +66,11 @@ export class ApiService {
       // Handle authentication errors with automatic token refresh
       if (response.status === 401 && !isRetry) {
         // Attempt to refresh token if we have refresh token in global config
-        let auth = await getAuthTokens();
-        if (auth && auth.refreshToken) {
+        const auth = await getAuthTokens();
+        if (auth?.refreshToken) {
           try {
             // Attempt token refresh
-            let refreshResponse = await fetch(`${this.baseUrl}/api/auth/cli/refresh`, {
+            const refreshResponse = await fetch(`${this.baseUrl}/api/auth/cli/refresh`, {
               method: 'POST',
               headers: {
                 'Content-Type': 'application/json',
@@ -81,7 +81,7 @@ export class ApiService {
               })
             });
             if (refreshResponse.ok) {
-              let refreshData = await refreshResponse.json();
+              const refreshData = await refreshResponse.json();
 
               // Save new tokens to global config
               await saveAuthTokens({
@@ -128,7 +128,7 @@ export class ApiService {
    * @returns {Promise<Object>} Comparison data
    */
   async getComparison(comparisonId) {
-    let response = await this.request(`/api/sdk/comparisons/${comparisonId}`);
+    const response = await this.request(`/api/sdk/comparisons/${comparisonId}`);
     return response.comparison;
   }
 
@@ -145,7 +145,7 @@ export class ApiService {
     if (!name || typeof name !== 'string') {
       throw new VizzlyError('name is required and must be a non-empty string');
     }
-    let {
+    const {
       branch,
       limit = 50,
       offset = 0
@@ -274,7 +274,7 @@ export class ApiService {
 
     // Check if this SHA with signature already exists
     const checkResult = await this.checkShas(screenshotCheck, buildId);
-    if (checkResult.existing && checkResult.existing.includes(sha256)) {
+    if (checkResult.existing?.includes(sha256)) {
       // File already exists with same signature, screenshot record was automatically created
       const screenshot = checkResult.screenshots?.find(s => s.sha256 === sha256);
       return {
@@ -366,13 +366,13 @@ export class ApiService {
    * @returns {Promise<Object>} Hotspot analysis data
    */
   async getScreenshotHotspots(screenshotName, options = {}) {
-    let {
+    const {
       windowSize = 20
     } = options;
-    let queryParams = new URLSearchParams({
+    const queryParams = new URLSearchParams({
       windowSize: String(windowSize)
     });
-    let encodedName = encodeURIComponent(screenshotName);
+    const encodedName = encodeURIComponent(screenshotName);
     return this.request(`/api/sdk/screenshots/${encodedName}/hotspots?${queryParams}`);
   }
 
@@ -385,7 +385,7 @@ export class ApiService {
    * @returns {Promise<Object>} Hotspots keyed by screenshot name
    */
   async getBatchHotspots(screenshotNames, options = {}) {
-    let {
+    const {
       windowSize = 20
     } = options;
     return this.request('/api/sdk/screenshots/hotspots', {

package/dist/services/auth-service.js

@@ -5,8 +5,8 @@
 
 import { AuthError, VizzlyError } from '../errors/vizzly-error.js';
 import { getApiUrl } from '../utils/environment-config.js';
+import { clearAuthTokens, getAuthTokens, saveAuthTokens } from '../utils/global-config.js';
 import { getPackageVersion } from '../utils/package-info.js';
-import { saveAuthTokens, clearAuthTokens, getAuthTokens } from '../utils/global-config.js';
 
 /**
  * AuthService class for CLI authentication
@@ -24,12 +24,12 @@ export class AuthService {
    * @returns {Promise<Object>} Response data
    */
   async request(endpoint, options = {}) {
-    let url = `${this.baseUrl}${endpoint}`;
-    let headers = {
+    const url = `${this.baseUrl}${endpoint}`;
+    const headers = {
       'User-Agent': this.userAgent,
       ...options.headers
     };
-    let response = await fetch(url, {
+    const response = await fetch(url, {
       ...options,
       headers
     });
@@ -37,8 +37,8 @@ export class AuthService {
       let errorText = '';
       let errorData = null;
       try {
-        let contentType = response.headers.get('content-type');
-        if (contentType && contentType.includes('application/json')) {
+        const contentType = response.headers.get('content-type');
+        if (contentType?.includes('application/json')) {
           errorData = await response.json();
           errorText = errorData.error || errorData.message || '';
         } else {
@@ -65,26 +65,26 @@ export class AuthService {
    * @returns {Promise<Object>} Response data
    */
   async authenticatedRequest(endpoint, options = {}) {
-    let auth = await getAuthTokens();
+    const auth = await getAuthTokens();
     if (!auth || !auth.accessToken) {
       throw new AuthError('No authentication token found. Please run "vizzly login" first.');
     }
-    let url = `${this.baseUrl}${endpoint}`;
-    let headers = {
+    const url = `${this.baseUrl}${endpoint}`;
+    const headers = {
       'User-Agent': this.userAgent,
       Authorization: `Bearer ${auth.accessToken}`,
       ...options.headers
     };
-    let response = await fetch(url, {
+    const response = await fetch(url, {
       ...options,
       headers
     });
     if (!response.ok) {
       let errorText = '';
       try {
-        let contentType = response.headers.get('content-type');
-        if (contentType && contentType.includes('application/json')) {
-          let errorData = await response.json();
+        const contentType = response.headers.get('content-type');
+        if (contentType?.includes('application/json')) {
+          const errorData = await response.json();
           errorText = errorData.error || errorData.message || '';
         } else {
           errorText = await response.text();
@@ -151,11 +151,11 @@ export class AuthService {
    * @returns {Promise<Object>} New tokens
    */
   async refresh() {
-    let auth = await getAuthTokens();
+    const auth = await getAuthTokens();
     if (!auth || !auth.refreshToken) {
       throw new AuthError('No refresh token found. Please run "vizzly login" first.');
     }
-    let response = await this.request('/api/auth/cli/refresh', {
+    const response = await this.request('/api/auth/cli/refresh', {
       method: 'POST',
       headers: {
         'Content-Type': 'application/json'
@@ -180,8 +180,8 @@ export class AuthService {
    * @returns {Promise<void>}
    */
   async logout() {
-    let auth = await getAuthTokens();
-    if (auth && auth.refreshToken) {
+    const auth = await getAuthTokens();
+    if (auth?.refreshToken) {
       try {
         // Attempt to revoke tokens on server
         await this.request('/api/auth/cli/logout', {

package/dist/services/build-manager.js

@@ -3,7 +3,7 @@
  * Manages the build lifecycle and coordinates test execution
  */
 
-import crypto from 'crypto';
+import crypto from 'node:crypto';
 import { VizzlyError } from '../errors/vizzly-error.js';
 
 /**
@@ -127,7 +127,7 @@ export class BuildManager {
     this.buildQueue = [];
   }
   async createBuild(buildOptions) {
-    let build = createBuildObject(buildOptions);
+    const build = createBuildObject(buildOptions);
     this.currentBuild = build;
     return build;
   }
@@ -156,7 +156,7 @@ export class BuildManager {
     return this.currentBuild;
   }
   queueBuild(buildOptions) {
-    let queuedBuild = createQueuedBuild(buildOptions);
+    const queuedBuild = createQueuedBuild(buildOptions);
     this.buildQueue.push(queuedBuild);
   }
   async clear() {