@vizzly-testing/cli 0.16.4 → 0.18.0

package/dist/sdk/index.js

@@ -10,14 +10,14 @@
  * @description Full SDK for custom integrations and advanced usage
  */
 
-import { EventEmitter } from 'events';
-import { resolveImageBuffer } from '../utils/file-helpers.js';
-import { createUploader } from '../services/uploader.js';
-import { createTDDService } from '../services/tdd-service.js';
+import { EventEmitter } from 'node:events';
+import { VizzlyError } from '../errors/vizzly-error.js';
 import { ScreenshotServer } from '../services/screenshot-server.js';
+import { createTDDService } from '../services/tdd-service.js';
+import { createUploader } from '../services/uploader.js';
 import { loadConfig } from '../utils/config-loader.js';
+import { resolveImageBuffer } from '../utils/file-helpers.js';
 import * as output from '../utils/output.js';
-import { VizzlyError } from '../errors/vizzly-error.js';
 
 /**
  * Create a new Vizzly instance with custom configuration
@@ -59,15 +59,15 @@ export function createVizzly(config = {}, options = {}) {
   });
 
   // Merge with loaded config
-  let resolvedConfig = {
+  const resolvedConfig = {
     ...config
   };
 
   /**
    * Initialize SDK with config loading
    */
-  let init = async () => {
-    let fileConfig = await loadConfig();
+  const init = async () => {
+    const fileConfig = await loadConfig();
     Object.assign(resolvedConfig, fileConfig, config); // CLI config takes precedence
     return resolvedConfig;
   };
@@ -75,7 +75,7 @@ export function createVizzly(config = {}, options = {}) {
   /**
    * Create uploader service
    */
-  let createUploaderService = (uploaderOptions = {}) => {
+  const createUploaderService = (uploaderOptions = {}) => {
     return createUploader({
       apiKey: resolvedConfig.apiKey,
       apiUrl: resolvedConfig.apiUrl
@@ -88,7 +88,7 @@ export function createVizzly(config = {}, options = {}) {
   /**
    * Create TDD service
    */
-  let createTDDServiceInstance = (tddOptions = {}) => {
+  const createTDDServiceInstance = (tddOptions = {}) => {
     return createTDDService(resolvedConfig, {
       ...options,
       ...tddOptions
@@ -98,16 +98,16 @@ export function createVizzly(config = {}, options = {}) {
   /**
    * Upload screenshots (convenience method)
    */
-  let upload = async uploadOptions => {
-    let uploader = createUploaderService();
+  const upload = async uploadOptions => {
+    const uploader = createUploaderService();
     return uploader.upload(uploadOptions);
   };
 
   /**
    * Start TDD mode (convenience method)
    */
-  let startTDD = async (tddOptions = {}) => {
-    let tddService = createTDDServiceInstance();
+  const startTDD = async (tddOptions = {}) => {
+    const tddService = createTDDServiceInstance();
     return tddService.start(tddOptions);
   };
   return {
@@ -195,7 +195,7 @@ export class VizzlySDK extends EventEmitter {
     }
 
     // Create a simple build manager for screenshot collection
-    let buildManager = {
+    const buildManager = {
       screenshots: new Map(),
       currentBuildId: null,
       async addScreenshot(buildId, screenshot) {
@@ -210,8 +210,8 @@ export class VizzlySDK extends EventEmitter {
     };
     this.server = new ScreenshotServer(this.config, buildManager);
     await this.server.start();
-    let port = this.config.server?.port || 3000;
-    let serverInfo = {
+    const port = this.config.server?.port || 3000;
+    const serverInfo = {
       port,
       url: `http://localhost:${port}`
     };
@@ -235,15 +235,15 @@ export class VizzlySDK extends EventEmitter {
     }
 
     // Resolve Buffer or file path using shared utility
-    let buffer = resolveImageBuffer(imageBuffer, 'screenshot');
+    const buffer = resolveImageBuffer(imageBuffer, 'screenshot');
 
     // Generate or use provided build ID
-    let buildId = options.buildId || this.currentBuildId || 'default';
+    const buildId = options.buildId || this.currentBuildId || 'default';
     this.currentBuildId = buildId;
 
     // Convert Buffer to base64 for JSON transport
-    let imageBase64 = buffer.toString('base64');
-    let screenshotData = {
+    const imageBase64 = buffer.toString('base64');
+    const screenshotData = {
       buildId,
       name,
       image: imageBase64,
@@ -251,9 +251,9 @@ export class VizzlySDK extends EventEmitter {
     };
 
     // POST to the local screenshot server
-    let serverUrl = `http://localhost:${this.config.server?.port || 3000}`;
+    const serverUrl = `http://localhost:${this.config.server?.port || 3000}`;
     try {
-      let response = await fetch(`${serverUrl}/screenshot`, {
+      const response = await fetch(`${serverUrl}/screenshot`, {
         method: 'POST',
         headers: {
           'Content-Type': 'application/json'
@@ -261,7 +261,7 @@ export class VizzlySDK extends EventEmitter {
         body: JSON.stringify(screenshotData)
       });
       if (!response.ok) {
-        let errorData = await response.json().catch(() => ({
+        const errorData = await response.json().catch(() => ({
           error: 'Unknown error'
         }));
         throw new VizzlyError(`Screenshot capture failed: ${errorData.error}`, 'SCREENSHOT_FAILED', {
@@ -302,8 +302,8 @@ export class VizzlySDK extends EventEmitter {
     }
 
     // Get the screenshots directory from config or default
-    let screenshotsDir = options.screenshotsDir || this.config?.upload?.screenshotsDir || './screenshots';
-    let uploadOptions = {
+    const screenshotsDir = options.screenshotsDir || this.config?.upload?.screenshotsDir || './screenshots';
+    const uploadOptions = {
       screenshotsDir,
       buildName: options.buildName || this.config.buildName,
       branch: options.branch || this.config.branch,
@@ -319,7 +319,7 @@ export class VizzlySDK extends EventEmitter {
       }
     };
     try {
-      let result = await this.services.uploader.upload(uploadOptions);
+      const result = await this.services.uploader.upload(uploadOptions);
       this.emit('upload:completed', result);
       return result;
     } catch (error) {
@@ -343,9 +343,9 @@ export class VizzlySDK extends EventEmitter {
     }
 
     // Resolve Buffer or file path using shared utility
-    let buffer = resolveImageBuffer(imageBuffer, 'compare');
+    const buffer = resolveImageBuffer(imageBuffer, 'compare');
     try {
-      let result = await this.services.tddService.compareScreenshot(name, buffer);
+      const result = await this.services.tddService.compareScreenshot(name, buffer);
       this.emit('comparison:completed', result);
       return result;
     } catch (error) {
@@ -357,11 +357,9 @@ export class VizzlySDK extends EventEmitter {
     }
   }
 }
-
-// Re-export key utilities and errors
-export { loadConfig } from '../utils/config-loader.js';
-export * as output from '../utils/output.js';
-
+export { createTDDService } from '../services/tdd-service.js';
 // Export service creators for advanced usage
 export { createUploader } from '../services/uploader.js';
-export { createTDDService } from '../services/tdd-service.js';
+// Re-export key utilities and errors
+export { loadConfig } from '../utils/config-loader.js';
+export * as output from '../utils/output.js';

package/dist/server/handlers/api-handler.js

@@ -1,8 +1,8 @@
-import { Buffer } from 'buffer';
-import { existsSync, readFileSync } from 'fs';
-import { resolve } from 'path';
-import * as output from '../../utils/output.js';
+import { Buffer } from 'node:buffer';
+import { existsSync, readFileSync } from 'node:fs';
+import { resolve } from 'node:path';
 import { detectImageInputType } from '../../utils/image-input-detector.js';
+import * as output from '../../utils/output.js';
 
 /**
  * API Handler - Non-blocking screenshot upload

package/dist/server/handlers/tdd-handler.js

@@ -1,11 +1,11 @@
-import { Buffer } from 'buffer';
-import { writeFileSync, readFileSync, existsSync } from 'fs';
-import { join, resolve } from 'path';
+import { Buffer } from 'node:buffer';
+import { existsSync, readFileSync, writeFileSync } from 'node:fs';
+import { join, resolve } from 'node:path';
 import { getDimensionsSync } from '@vizzly-testing/honeydiff';
-import * as output from '../../utils/output.js';
 import { TddService } from '../../services/tdd-service.js';
-import { sanitizeScreenshotName, validateScreenshotProperties } from '../../utils/security.js';
 import { detectImageInputType } from '../../utils/image-input-detector.js';
+import * as output from '../../utils/output.js';
+import { sanitizeScreenshotName, validateScreenshotProperties } from '../../utils/security.js';
 
 /**
  * Group comparisons by screenshot name with variant structure
@@ -141,7 +141,7 @@ export const createTddHandler = (config, workingDir, baselineBuild, baselineComp
       if (existingIndex >= 0) {
         // Preserve initialStatus from the original comparison
         // This keeps sort order stable when status changes (e.g., after approval)
-        let initialStatus = reportData.comparisons[existingIndex].initialStatus;
+        const initialStatus = reportData.comparisons[existingIndex].initialStatus;
         reportData.comparisons[existingIndex] = {
           ...newComparison,
           initialStatus: initialStatus || newComparison.status
@@ -200,7 +200,7 @@ export const createTddHandler = (config, workingDir, baselineBuild, baselineComp
       output.debug('tdd', `using baseline: ${baseline.buildName}`);
     }
   };
-  const handleScreenshot = async (buildId, name, image, properties = {}) => {
+  const handleScreenshot = async (_buildId, name, image, properties = {}) => {
     // Validate and sanitize screenshot name
     let sanitizedName;
     try {
@@ -217,7 +217,7 @@ export const createTddHandler = (config, workingDir, baselineBuild, baselineComp
     }
 
     // Unwrap double-nested properties if needed (client SDK wraps options in properties field)
-    // This happens when test helper passes { properties: {...}, threshold: 0.1 }
+    // This happens when test helper passes { properties: {...}, threshold: 2.0 }
     // and client SDK wraps it as { properties: options }
     let unwrappedProperties = properties;
     if (properties.properties && typeof properties.properties === 'object') {
@@ -504,7 +504,7 @@ export const createTddHandler = (config, workingDir, baselineBuild, baselineComp
             try {
               const {
                 unlinkSync
-              } = await import('fs');
+              } = await import('node:fs');
               unlinkSync(baselinePath);
               deletedBaselines++;
               // Silent deletion
@@ -521,7 +521,7 @@ export const createTddHandler = (config, workingDir, baselineBuild, baselineComp
             try {
               const {
                 unlinkSync
-              } = await import('fs');
+              } = await import('node:fs');
               unlinkSync(currentPath);
               deletedCurrents++;
               // Silent deletion
@@ -538,7 +538,7 @@ export const createTddHandler = (config, workingDir, baselineBuild, baselineComp
             try {
               const {
                 unlinkSync
-              } = await import('fs');
+              } = await import('node:fs');
               unlinkSync(diffPath);
               deletedDiffs++;
               output.debug(`Deleted diff for ${comparison.name}`);
@@ -555,7 +555,7 @@ export const createTddHandler = (config, workingDir, baselineBuild, baselineComp
         try {
           const {
             unlinkSync
-          } = await import('fs');
+          } = await import('node:fs');
           unlinkSync(metadataPath);
           output.debug('Deleted baseline metadata');
         } catch (error) {

package/dist/server/http-server.js

@@ -3,29 +3,28 @@
  * Thin dispatcher that routes requests to modular routers
  */
 
-import { createServer } from 'http';
+import { createServer } from 'node:http';
 import * as output from '../utils/output.js';
 
 // Middleware
 import { corsMiddleware } from './middleware/cors.js';
 import { sendError } from './middleware/response.js';
-
-// Routers
-import { createHealthRouter } from './routers/health.js';
 import { createAssetsRouter } from './routers/assets.js';
-import { createDashboardRouter } from './routers/dashboard.js';
-import { createScreenshotRouter } from './routers/screenshot.js';
+import { createAuthRouter } from './routers/auth.js';
 import { createBaselineRouter } from './routers/baseline.js';
+import { createCloudProxyRouter } from './routers/cloud-proxy.js';
 import { createConfigRouter } from './routers/config.js';
-import { createAuthRouter } from './routers/auth.js';
+import { createDashboardRouter } from './routers/dashboard.js';
+// Routers
+import { createHealthRouter } from './routers/health.js';
 import { createProjectsRouter } from './routers/projects.js';
-import { createCloudProxyRouter } from './routers/cloud-proxy.js';
-export let createHttpServer = (port, screenshotHandler, services = {}) => {
+import { createScreenshotRouter } from './routers/screenshot.js';
+export const createHttpServer = (port, screenshotHandler, services = {}) => {
   let server = null;
-  let defaultBuildId = services.buildId || null;
+  const defaultBuildId = services.buildId || null;
 
   // Extract services
-  let {
+  const {
     configService,
     authService,
     projectService,
@@ -33,7 +32,7 @@ export let createHttpServer = (port, screenshotHandler, services = {}) => {
   } = services;
 
   // Create router context
-  let routerContext = {
+  const routerContext = {
     port,
     screenshotHandler,
     defaultBuildId,
@@ -45,9 +44,9 @@ export let createHttpServer = (port, screenshotHandler, services = {}) => {
   };
 
   // Initialize routers
-  let routers = [createHealthRouter(routerContext), createAssetsRouter(routerContext), createScreenshotRouter(routerContext), createBaselineRouter(routerContext), createConfigRouter(routerContext), createAuthRouter(routerContext), createProjectsRouter(routerContext), createCloudProxyRouter(routerContext), createDashboardRouter(routerContext) // Catch-all for SPA routes - must be last
+  const routers = [createHealthRouter(routerContext), createAssetsRouter(routerContext), createScreenshotRouter(routerContext), createBaselineRouter(routerContext), createConfigRouter(routerContext), createAuthRouter(routerContext), createProjectsRouter(routerContext), createCloudProxyRouter(routerContext), createDashboardRouter(routerContext) // Catch-all for SPA routes - must be last
   ];
-  let handleRequest = async (req, res) => {
+  const handleRequest = async (req, res) => {
     // Apply CORS middleware
     if (corsMiddleware(req, res)) {
       return;
@@ -57,13 +56,13 @@ export let createHttpServer = (port, screenshotHandler, services = {}) => {
     res.setHeader('Content-Type', 'application/json');
 
     // Parse URL
-    let parsedUrl = new URL(req.url, `http://${req.headers.host}`);
-    let pathname = parsedUrl.pathname;
+    const parsedUrl = new URL(req.url, `http://${req.headers.host}`);
+    const pathname = parsedUrl.pathname;
 
     // Try each router in order
-    for (let router of routers) {
+    for (const router of routers) {
       try {
-        let handled = await router(req, res, pathname, parsedUrl);
+        const handled = await router(req, res, pathname, parsedUrl);
         if (handled) {
           return;
         }
@@ -79,7 +78,7 @@ export let createHttpServer = (port, screenshotHandler, services = {}) => {
     // No router handled the request
     sendError(res, 404, 'Not found');
   };
-  let start = () => {
+  const start = () => {
     return new Promise((resolve, reject) => {
       server = createServer(async (req, res) => {
         try {
@@ -112,7 +111,7 @@ export let createHttpServer = (port, screenshotHandler, services = {}) => {
       });
     });
   };
-  let stop = () => {
+  const stop = () => {
     if (server) {
       return new Promise(resolve => {
         server.close(() => {
@@ -129,10 +128,10 @@ export let createHttpServer = (port, screenshotHandler, services = {}) => {
    * Finish build - flush any pending background operations
    * Call this before finalizing a build to ensure all uploads complete
    */
-  let finishBuild = async _buildId => {
+  const finishBuild = async _buildId => {
     // Flush screenshot handler if it has a flush method (API mode)
     if (screenshotHandler?.flush) {
-      let stats = await screenshotHandler.flush();
+      const stats = await screenshotHandler.flush();
       if (stats.uploaded > 0 || stats.failed > 0) {
         output.debug('upload', 'flushed', {
           uploaded: stats.uploaded,

package/dist/server/middleware/json-parser.js

@@ -24,7 +24,7 @@ export function parseJsonBody(req) {
         return;
       }
       try {
-        let data = JSON.parse(body);
+        const data = JSON.parse(body);
         resolve(data);
       } catch {
         reject(new Error('Invalid JSON'));

package/dist/server/routers/assets.js

@@ -3,14 +3,14 @@
  * Serves static assets (bundle files, images)
  */
 
-import { readFileSync, existsSync } from 'fs';
-import { join, dirname } from 'path';
-import { fileURLToPath } from 'url';
-import { sendFile, sendError, sendNotFound } from '../middleware/response.js';
+import { existsSync, readFileSync } from 'node:fs';
+import { dirname, join } from 'node:path';
+import { fileURLToPath } from 'node:url';
 import * as output from '../../utils/output.js';
-let __filename = fileURLToPath(import.meta.url);
-let __dirname = dirname(__filename);
-let PROJECT_ROOT = join(__dirname, '..', '..', '..');
+import { sendError, sendFile, sendNotFound } from '../middleware/response.js';
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+const PROJECT_ROOT = join(__dirname, '..', '..', '..');
 
 /**
  * Create assets router
@@ -25,10 +25,10 @@ export function createAssetsRouter() {
 
     // Serve React bundle JS
     if (pathname === '/reporter-bundle.js') {
-      let bundlePath = join(PROJECT_ROOT, 'dist', 'reporter', 'reporter-bundle.iife.js');
+      const bundlePath = join(PROJECT_ROOT, 'dist', 'reporter', 'reporter-bundle.iife.js');
       if (existsSync(bundlePath)) {
         try {
-          let bundle = readFileSync(bundlePath, 'utf8');
+          const bundle = readFileSync(bundlePath, 'utf8');
           sendFile(res, bundle, 'application/javascript');
           return true;
         } catch (error) {
@@ -46,10 +46,10 @@ export function createAssetsRouter() {
 
     // Serve React bundle CSS
     if (pathname === '/reporter-bundle.css') {
-      let cssPath = join(PROJECT_ROOT, 'dist', 'reporter', 'reporter-bundle.css');
+      const cssPath = join(PROJECT_ROOT, 'dist', 'reporter', 'reporter-bundle.css');
       if (existsSync(cssPath)) {
         try {
-          let css = readFileSync(cssPath, 'utf8');
+          const css = readFileSync(cssPath, 'utf8');
           sendFile(res, css, 'text/css');
           return true;
         } catch (error) {
@@ -67,11 +67,11 @@ export function createAssetsRouter() {
 
     // Serve images from .vizzly directory
     if (pathname.startsWith('/images/')) {
-      let imagePath = pathname.replace('/images/', '');
-      let fullImagePath = join(process.cwd(), '.vizzly', imagePath);
+      const imagePath = pathname.replace('/images/', '');
+      const fullImagePath = join(process.cwd(), '.vizzly', imagePath);
       if (existsSync(fullImagePath)) {
         try {
-          let imageData = readFileSync(fullImagePath);
+          const imageData = readFileSync(fullImagePath);
           sendFile(res, imageData, 'image/png');
           return true;
         } catch (error) {

package/dist/server/routers/auth.js

@@ -3,9 +3,9 @@
  * Handles authentication endpoints (device flow login, logout, status)
  */
 
-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 auth router
@@ -26,10 +26,10 @@ export function createAuthRouter({
     // Get auth status and user info
     if (req.method === 'GET' && pathname === '/api/auth/status') {
       try {
-        let isAuthenticated = await authService.isAuthenticated();
+        const isAuthenticated = await authService.isAuthenticated();
         let user = null;
         if (isAuthenticated) {
-          let whoami = await authService.whoami();
+          const whoami = await authService.whoami();
           user = whoami.user;
         }
         sendSuccess(res, {
@@ -50,10 +50,10 @@ export function createAuthRouter({
     // Initiate device flow login
     if (req.method === 'POST' && pathname === '/api/auth/login') {
       try {
-        let deviceFlow = await authService.initiateDeviceFlow();
+        const deviceFlow = await authService.initiateDeviceFlow();
 
         // Transform snake_case to camelCase for frontend
-        let response = {
+        const response = {
           deviceCode: deviceFlow.device_code,
           userCode: deviceFlow.user_code,
           verificationUri: deviceFlow.verification_uri,
@@ -73,8 +73,8 @@ export function createAuthRouter({
     // Poll device authorization status
     if (req.method === 'POST' && pathname === '/api/auth/poll') {
       try {
-        let body = await parseJsonBody(req);
-        let {
+        const body = await parseJsonBody(req);
+        const {
           deviceCode
         } = body;
         if (!deviceCode) {
@@ -86,7 +86,7 @@ export function createAuthRouter({
           result = await authService.pollDeviceAuthorization(deviceCode);
         } catch (error) {
           // Handle "Authorization pending" as a valid response
-          if (error.message && error.message.includes('Authorization pending')) {
+          if (error.message?.includes('Authorization pending')) {
             sendSuccess(res, {
               status: 'pending'
             });
@@ -96,11 +96,11 @@ export function createAuthRouter({
         }
 
         // Check if authorization is complete by looking for tokens
-        if (result.tokens && result.tokens.accessToken) {
-          let tokensData = result.tokens;
-          let tokenExpiresIn = tokensData.expiresIn || tokensData.expires_in;
-          let tokenExpiresAt = tokenExpiresIn ? new Date(Date.now() + tokenExpiresIn * 1000).toISOString() : result.expires_at || result.expiresAt;
-          let tokens = {
+        if (result.tokens?.accessToken) {
+          const tokensData = result.tokens;
+          const tokenExpiresIn = tokensData.expiresIn || tokensData.expires_in;
+          const tokenExpiresAt = tokenExpiresIn ? new Date(Date.now() + tokenExpiresIn * 1000).toISOString() : result.expires_at || result.expiresAt;
+          const tokens = {
             accessToken: tokensData.accessToken || tokensData.access_token,
             refreshToken: tokensData.refreshToken || tokensData.refresh_token,
             expiresAt: tokenExpiresAt,

package/dist/server/routers/baseline.js

@@ -3,9 +3,9 @@
  * Handles baseline management (accept, accept-all, reset)
  */
 
-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 baseline router
@@ -28,7 +28,7 @@ export function createBaselineRouter({
         return true;
       }
       try {
-        let {
+        const {
           id
         } = await parseJsonBody(req);
         if (!id) {
@@ -55,7 +55,7 @@ export function createBaselineRouter({
         return true;
       }
       try {
-        let result = await screenshotHandler.acceptAllBaselines();
+        const result = await screenshotHandler.acceptAllBaselines();
         sendSuccess(res, {
           success: true,
           message: `Accepted ${result.count} baselines`,
@@ -96,8 +96,8 @@ export function createBaselineRouter({
         return true;
       }
       try {
-        let body = await parseJsonBody(req);
-        let {
+        const body = await parseJsonBody(req);
+        const {
           buildId,
           organizationSlug,
           projectSlug
@@ -111,7 +111,7 @@ export function createBaselineRouter({
         // If organizationSlug and projectSlug are provided, use OAuth-based download
         if (organizationSlug && projectSlug && authService) {
           try {
-            let result = await tddService.downloadBaselinesWithAuth(buildId, organizationSlug, projectSlug, authService);
+            const result = await tddService.downloadBaselinesWithAuth(buildId, organizationSlug, projectSlug, authService);
             sendSuccess(res, {
               success: true,
               message: `Baselines downloaded from build ${buildId}`,
@@ -138,7 +138,7 @@ export function createBaselineRouter({
         }
 
         // Fall back to API token-based download (when no OAuth info or OAuth auth failed)
-        let result = await tddService.downloadBaselines('test',
+        const result = await tddService.downloadBaselines('test',
         // environment
         null,
         // branch (not needed when buildId is specified)

package/dist/server/routers/cloud-proxy.js

@@ -9,9 +9,9 @@
  * - Returns proxied response to React app
  */
 
-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 cloud proxy router
@@ -53,7 +53,7 @@ export function createCloudProxyRouter({
     // Route: GET /api/cloud/projects - List user's projects
     if (req.method === 'GET' && pathname === '/api/cloud/projects') {
       try {
-        let response = await proxyRequest('/api/cli/projects', {
+        const response = await proxyRequest('/api/cli/projects', {
           method: 'GET'
         });
         sendSuccess(res, {
@@ -76,19 +76,19 @@ export function createCloudProxyRouter({
     }
 
     // Route: GET /api/cloud/organizations/:org/projects/:project/builds
-    let buildsMatch = pathname.match(/^\/api\/cloud\/organizations\/([^/]+)\/projects\/([^/]+)\/builds$/);
+    const buildsMatch = pathname.match(/^\/api\/cloud\/organizations\/([^/]+)\/projects\/([^/]+)\/builds$/);
     if (req.method === 'GET' && buildsMatch) {
       try {
-        let organizationSlug = decodeURIComponent(buildsMatch[1]);
-        let projectSlug = decodeURIComponent(buildsMatch[2]);
-        let limit = parsedUrl.searchParams.get('limit') || '20';
-        let branch = parsedUrl.searchParams.get('branch');
-        let queryParams = new URLSearchParams();
+        const organizationSlug = decodeURIComponent(buildsMatch[1]);
+        const projectSlug = decodeURIComponent(buildsMatch[2]);
+        const limit = parsedUrl.searchParams.get('limit') || '20';
+        const branch = parsedUrl.searchParams.get('branch');
+        const queryParams = new URLSearchParams();
         if (limit) queryParams.append('limit', limit);
         if (branch) queryParams.append('branch', branch);
-        let query = queryParams.toString();
-        let endpoint = `/api/cli/organizations/${organizationSlug}/projects/${projectSlug}/builds${query ? `?${query}` : ''}`;
-        let response = await proxyRequest(endpoint, {
+        const query = queryParams.toString();
+        const endpoint = `/api/cli/organizations/${organizationSlug}/projects/${projectSlug}/builds${query ? `?${query}` : ''}`;
+        const response = await proxyRequest(endpoint, {
           method: 'GET'
         });
         sendSuccess(res, {
@@ -105,8 +105,8 @@ export function createCloudProxyRouter({
     // Route: POST /api/cloud/baselines/download - Download baselines from build
     if (req.method === 'POST' && pathname === '/api/cloud/baselines/download') {
       try {
-        let body = await parseJsonBody(req);
-        let {
+        const body = await parseJsonBody(req);
+        const {
           buildId,
           screenshotNames
         } = body;
@@ -116,7 +116,7 @@ export function createCloudProxyRouter({
         }
 
         // Download baselines from the specified build
-        let response = await proxyRequest('/api/cli/baselines/download', {
+        const response = await proxyRequest('/api/cli/baselines/download', {
           method: 'POST',
           headers: {
             'Content-Type': 'application/json'