@vizzly-testing/cli 0.16.3 → 0.16.4

package/dist/services/server-manager.js

@@ -6,7 +6,7 @@
 import { createHttpServer } from '../server/http-server.js';
 import { createTddHandler } from '../server/handlers/tdd-handler.js';
 import { createApiHandler } from '../server/handlers/api-handler.js';
-import { writeFileSync, mkdirSync } from 'fs';
+import { writeFileSync, mkdirSync, unlinkSync, existsSync } from 'fs';
 import { join } from 'path';
 export class ServerManager {
   constructor(config, options = {}) {
@@ -85,6 +85,16 @@ export class ServerManager {
         // Don't throw - cleanup errors shouldn't fail the stop process
       }
     }
+
+    // Clean up server.json so the client SDK doesn't try to connect to a dead server
+    try {
+      let serverFile = join(process.cwd(), '.vizzly', 'server.json');
+      if (existsSync(serverFile)) {
+        unlinkSync(serverFile);
+      }
+    } catch {
+      // Non-fatal - cleanup errors shouldn't fail the stop process
+    }
   }
 
   // Expose server interface for compatibility
@@ -94,4 +104,13 @@ export class ServerManager {
       finishBuild: buildId => this.httpServer?.finishBuild?.(buildId)
     };
   }
+
+  /**
+   * Get TDD results (comparisons, screenshot count, etc.)
+   * Only available in TDD mode after tests have run
+   */
+  async getTddResults() {
+    if (!this.tddMode || !this.handler?.getResults) return null;
+    return await this.handler.getResults();
+  }
 }

package/dist/services/test-runner.js

@@ -112,8 +112,25 @@ export class TestRunner extends EventEmitter {
       // Error in setup phase
       testError = error;
       testSuccess = false;
-    } finally {
-      // Always finalize the build and stop the server
+    }
+
+    // Get TDD results before stopping the server (comparisons, screenshot count)
+    let tddResults = null;
+    if (tdd) {
+      try {
+        tddResults = await this.serverManager.getTddResults();
+        if (tddResults) {
+          screenshotCount = tddResults.total || 0;
+        }
+      } catch (tddError) {
+        output.debug('tdd', 'failed to get results', {
+          error: tddError.message
+        });
+      }
+    }
+
+    // Always finalize the build and stop the server (cleanup phase)
+    try {
       const executionTime = Date.now() - startTime;
       if (buildId) {
         try {
@@ -127,6 +144,8 @@ export class TestRunner extends EventEmitter {
       if (!tdd && this.serverManager.server?.getScreenshotCount) {
         screenshotCount = this.serverManager.server.getScreenshotCount(buildId) || 0;
       }
+    } finally {
+      // Always stop the server, even if finalization fails
       try {
         await this.serverManager.stop();
       } catch (stopError) {
@@ -144,7 +163,9 @@ export class TestRunner extends EventEmitter {
       url: buildUrl,
       testsPassed: testSuccess ? 1 : 0,
       testsFailed: testSuccess ? 0 : 1,
-      screenshotsCaptured: screenshotCount
+      screenshotsCaptured: screenshotCount,
+      comparisons: tddResults?.comparisons || null,
+      failed: (tddResults?.failed || 0) > 0
     };
   }
   async createBuild(options, tdd) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vizzly-testing/cli",
-  "version": "0.16.3",
+  "version": "0.16.4",
   "description": "Visual review platform for UI developers and designers",
   "keywords": [
     "visual-testing",