@vizzly-testing/cli 0.3.1 → 0.4.0

package/dist/utils/console-ui.js

@@ -158,11 +158,12 @@ export class ConsoleUI {
   startSpinner(message) {
     if (this.json || !process.stdout.isTTY) return;
     this.stopSpinner();
+    this.currentMessage = message;
     const frames = ['⠋', '⠙', '⠹', '⠸', '⠼', '⠴', '⠦', '⠧', '⠇', '⠏'];
     let i = 0;
     this.spinner = setInterval(() => {
       const frame = frames[i++ % frames.length];
-      const line = `${this.colors.blue(frame)} ${message}`;
+      const line = `${this.colors.blue(frame)} ${this.currentMessage || message}`;
 
       // Clear previous line and write new one
       process.stdout.write('\r' + ' '.repeat(this.lastLine.length) + '\r');
@@ -211,16 +212,5 @@ export class ConsoleUI {
   }
 }
 
-// Ensure spinner is cleaned up on process exit
-process.on('exit', () => {
-  // Clear any remaining spinner
-  if (process.stdout.isTTY) {
-    process.stdout.write('\r' + ' '.repeat(80) + '\r');
-  }
-});
-process.on('SIGINT', () => {
-  if (process.stdout.isTTY) {
-    process.stdout.write('\r' + ' '.repeat(80) + '\r');
-  }
-  process.exit(1);
-});
+// Note: Global process event listeners are handled in individual commands
+// to avoid interference between tests and proper cleanup

package/docs/api-reference.md

@@ -288,6 +288,7 @@ Upload screenshots from a directory.
 - `--threshold <number>` - Comparison threshold (0-1)
 - `--token <token>` - API token override
 - `--wait` - Wait for build completion
+- `--upload-all` - Upload all screenshots without SHA deduplication
 
 **Exit Codes:**
 - `0` - Success (all approved or no changes)
@@ -316,18 +317,14 @@ Run tests with Vizzly integration.
 
 *Processing Options:*
 - `--wait` - Wait for build completion and exit with appropriate code
-- `--eager` - Create build immediately (default: lazy creation)
 - `--threshold <number>` - Comparison threshold (0-1, default: 0.01)
 - `--upload-timeout <ms>` - Upload wait timeout in ms (default: from config or 30000)
+- `--upload-all` - Upload all screenshots without SHA deduplication
 
 *Development & Testing:*
-- `--tdd` - Enable TDD mode with local comparisons
 - `--allow-no-token` - Allow running without API token
 - `--token <token>` - API token override
 
-*Baseline Configuration:*
-- `--baseline-build <id>` - Use specific build as baseline for comparisons
-- `--baseline-comparison <id>` - Use specific comparison as baseline
 
 **Environment Variables Set:**
 - `VIZZLY_SERVER_URL` - Local server URL

package/docs/getting-started.md

@@ -59,7 +59,7 @@ npx vizzly upload ./screenshots --build-name "Release v1.2.3"
 npx vizzly run "npm test"
 
 # Use TDD mode for local development
-npx vizzly run "npm test" --tdd
+npx vizzly tdd "npm test"
 ```
 
 ### 6. In your test code

package/docs/tdd-mode.md

@@ -145,11 +145,11 @@ vizzly tdd "npm test" --timeout 60000
 npx vizzly run "npm test" --wait
 
 # 2. Start TDD development
-npx vizzly run "npm test" --tdd
+npx vizzly tdd "npm test"
 
 # 3. Make changes and iterate
 # Edit code...
-npx vizzly run "npm test" --tdd
+npx vizzly tdd "npm test"
 
 # 4. Upload when satisfied
 npx vizzly run "npm test" --wait
@@ -159,13 +159,13 @@ npx vizzly run "npm test" --wait
 
 ```bash
 # Start with latest baselines
-npx vizzly run "npm test" --tdd
+npx vizzly tdd "npm test"
 
 # Develop new feature with immediate feedback
 while [ $? -ne 0 ]; do
   # Edit code to fix visual differences
   vim src/components/NewFeature.js
-  npx vizzly run "npm test" --tdd
+  npx vizzly tdd "npm test"
 done
 
 # Upload completed feature
@@ -176,7 +176,7 @@ npx vizzly run "npm test" --build-name "Feature: New Dashboard"
 
 ```bash
 # Use TDD mode to verify fixes
-npx vizzly run "npm test" --tdd
+npx vizzly tdd "npm test"
 
 # Tests should pass when bug is fixed
 # Then upload the fix
@@ -216,7 +216,7 @@ npx vizzly status  # Shows latest build info
 Download new baselines from a different build:
 
 ```bash
-npx vizzly run "npm test" --tdd --baseline-build build-xyz789
+npx vizzly tdd "npm test" --baseline-build build-xyz789
 ```
 
 ### Force Baseline Refresh
@@ -225,7 +225,7 @@ Delete local baselines to force re-download:
 
 ```bash
 rm -rf .vizzly/baselines/
-npx vizzly run "npm test" --tdd
+npx vizzly tdd "npm test"
 ```
 
 ## Advanced Usage
@@ -276,7 +276,7 @@ jobs:
       # Use TDD mode for PR builds (faster, no uploads)
       - name: TDD Visual Tests (PR)
         if: github.event_name == 'pull_request'
-        run: npx vizzly run "npm test" --tdd
+        run: npx vizzly tdd "npm test"
         env:
           VIZZLY_TOKEN: ${{ secrets.VIZZLY_TOKEN }}
       
@@ -337,7 +337,7 @@ Error: Failed to compare 'homepage': baseline image not found
 **Solution**: Refresh baselines:
 ```bash
 rm -rf .vizzly/baselines/
-npx vizzly run "npm test" --tdd
+npx vizzly tdd "npm test"
 ```
 
 ### Odiff Not Found

package/docs/test-integration.md

@@ -199,9 +199,9 @@ vizzly run "npm test" --branch "feature/new-ui"
 vizzly run "npm test" --wait
 ```
 
-**`--eager`** - Create build immediately (default: lazy)
+**`--upload-all`** - Upload all screenshots without SHA deduplication
 ```bash
-vizzly run "npm test" --eager
+vizzly run "npm test" --upload-all
 ```
 
 **`--threshold <number>`** - Comparison threshold (0-1, default: 0.01)
@@ -211,10 +211,7 @@ vizzly run "npm test" --threshold 0.02
 
 ### Development Options
 
-**`--tdd`** - Enable TDD mode (see [TDD Mode Guide](./tdd-mode.md))
-```bash
-vizzly run "npm test" --tdd
-```
+For TDD mode, use the dedicated `vizzly tdd` command. See [TDD Mode Guide](./tdd-mode.md) for details.
 
 **`--allow-no-token`** - Allow running without API token
 ```bash
@@ -226,17 +223,6 @@ vizzly run "npm test" --allow-no-token
 vizzly run "npm test" --token "your-token-here"
 ```
 
-### Baseline Configuration
-
-**`--baseline-build <id>`** - Use specific build as baseline
-```bash
-vizzly run "npm test" --baseline-build "build_123"
-```
-
-**`--baseline-comparison <id>`** - Use specific comparison as baseline
-```bash
-vizzly run "npm test" --baseline-comparison "comp_456"
-```
 
 ## Screenshot Properties
 

package/docs/upload-command.md

@@ -72,6 +72,13 @@ This will:
 - Show progress and results
 - Exit with appropriate status code
 
+**`--upload-all`** - Upload all screenshots without SHA deduplication
+```bash
+vizzly upload ./screenshots --upload-all
+```
+
+By default, Vizzly deduplicates screenshots based on their SHA hash to avoid unnecessary uploads. Use this flag to force upload of all screenshots regardless of their content.
+
 **`--metadata <json>`** - Additional metadata as JSON
 ```bash
 vizzly upload ./screenshots --metadata '{"browser": "chrome", "viewport": "1920x1080"}'

package/package.json

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

package/dist/screenshot-wrapper.js

@@ -1,68 +0,0 @@
-/**
- * Simple factory for creating Vizzly instances with shared configuration
- * Users handle their own screenshots, just pass the buffer to Vizzly
- */
-
-import { createVizzly as createVizzlySDK } from './vizzly.js';
-
-/**
- * Create a factory that pre-configures Vizzly instances
- *
- * @param {Object} config - Shared configuration
- * @param {Object} [config.defaultProperties] - Default metadata for all screenshots
- * @param {number} [config.defaultThreshold] - Default comparison threshold
- *
- * @example
- * // test-setup.js - Configure once
- * export const createVizzly = vizzlyFactory({
- *   defaultProperties: {
- *     framework: 'playwright',
- *     project: 'web-app'
- *   }
- * });
- *
- * // my-test.spec.js - Use everywhere
- * const vizzly = createVizzly();
- *
- * const screenshot = await page.screenshot({ fullPage: true }); // Your method
- * await vizzly.screenshot({
- *   name: 'homepage',
- *   image: screenshot, // Your buffer
- *   properties: { browser: 'chrome' } // Merges with defaults
- * });
- */
-export function vizzlyFactory(globalConfig) {
-  const {
-    defaultProperties = {},
-    defaultThreshold,
-    ...vizzlyConfig
-  } = globalConfig;
-  return function createVizzly(overrideConfig = {}) {
-    const vizzly = createVizzlySDK({
-      ...vizzlyConfig,
-      ...overrideConfig
-    });
-    return {
-      ...vizzly,
-      /**
-       * Take a screenshot with default properties merged in
-       *
-       * @param {Object} screenshot - Screenshot object
-       * @param {string} screenshot.name - Screenshot name
-       * @param {Buffer} screenshot.image - Image buffer from YOUR screenshot method
-       * @param {Object} [screenshot.properties] - Additional metadata (merged with defaults)
-       * @param {number} [screenshot.threshold] - Comparison threshold (defaults to global)
-       */
-      async screenshot(screenshot) {
-        return await vizzly.screenshot({
-          ...screenshot,
-          properties: {
-            ...defaultProperties,
-            ...screenshot.properties
-          },
-          threshold: screenshot.threshold || defaultThreshold
-        });
-      }
-    };
-  };
-}