@desplega.ai/qa-use 2.4.1 → 2.5.0

package/lib/api/browser.ts

@@ -2,27 +2,27 @@
  * BrowserApiClient - Client for the desplega.ai Browser API (/browsers/v1/)
  */
 
-import axios from 'axios';
 import type { AxiosInstance } from 'axios';
+import axios from 'axios';
+import type { ExtendedStep } from '../../src/types/test-definition.js';
 import { getEnv } from '../env/index.js';
 import type {
-  BrowserSession,
-  BrowserAction,
   ActionResult,
-  SnapshotResult,
-  SnapshotOptions,
-  UrlResult,
   BlocksResult,
-  CreateBrowserSessionOptions,
+  BrowserAction,
+  BrowserSession,
   BrowserSessionStatus,
-  GenerateTestOptions,
-  GenerateTestResult,
   ConsoleLogsOptions,
   ConsoleLogsResult,
+  CreateBrowserSessionOptions,
+  GenerateTestOptions,
+  GenerateTestResult,
   NetworkLogsOptions,
   NetworkLogsResult,
+  SnapshotOptions,
+  SnapshotResult,
+  UrlResult,
 } from './browser-types.js';
-import type { ExtendedStep } from '../../src/types/test-definition.js';
 
 export class BrowserApiClient {
   private readonly client: AxiosInstance;
@@ -49,7 +49,7 @@ export class BrowserApiClient {
 
   setApiKey(apiKey: string): void {
     this.apiKey = apiKey;
-    this.client.defaults.headers['Authorization'] = `Bearer ${apiKey}`;
+    this.client.defaults.headers.Authorization = `Bearer ${apiKey}`;
   }
 
   getApiKey(): string | null {
@@ -382,21 +382,21 @@ export class BrowserApiClient {
 
 // Re-export types for convenience
 export type {
-  BrowserSession,
-  BrowserAction,
   ActionResult,
-  SnapshotResult,
-  SnapshotOptions,
-  SnapshotFilterStats,
-  UrlResult,
-  CreateBrowserSessionOptions,
+  BrowserAction,
+  BrowserSession,
   BrowserSessionStatus,
-  GenerateTestOptions,
-  GenerateTestResult,
+  ConsoleLogEntry,
   ConsoleLogsOptions,
   ConsoleLogsResult,
-  ConsoleLogEntry,
+  CreateBrowserSessionOptions,
+  GenerateTestOptions,
+  GenerateTestResult,
+  NetworkLogEntry,
   NetworkLogsOptions,
   NetworkLogsResult,
-  NetworkLogEntry,
+  SnapshotFilterStats,
+  SnapshotOptions,
+  SnapshotResult,
+  UrlResult,
 } from './browser-types.js';

package/lib/api/index.test.ts

@@ -0,0 +1,169 @@
+/**
+ * ApiClient Unit Tests for importTestDefinition
+ */
+
+import { beforeEach, describe, expect, it, mock } from 'bun:test';
+import { ApiClient } from './index.js';
+
+// Mock axios
+const mockAxiosInstance = {
+  get: mock(() => Promise.resolve({ data: {} })),
+  post: mock(() => Promise.resolve({ data: {} })),
+  delete: mock(() => Promise.resolve({ data: {} })),
+  defaults: {
+    baseURL: 'https://api.desplega.ai',
+    headers: {} as Record<string, string>,
+  },
+};
+
+mock.module('axios', () => ({
+  default: {
+    create: () => mockAxiosInstance,
+    isAxiosError: (err: unknown) =>
+      typeof err === 'object' && err !== null && 'response' in err && 'isAxiosError' in err,
+  },
+}));
+
+describe('ApiClient', () => {
+  let client: ApiClient;
+
+  beforeEach(() => {
+    // Reset mocks
+    mockAxiosInstance.get.mockReset();
+    mockAxiosInstance.post.mockReset();
+    mockAxiosInstance.delete.mockReset();
+
+    // Clear environment variables
+    delete process.env.QA_USE_API_KEY;
+    delete process.env.QA_USE_API_URL;
+
+    client = new ApiClient();
+    client.setApiKey('test-key');
+  });
+
+  describe('importTestDefinition', () => {
+    const testDef = {
+      name: 'Test Definition',
+      steps: [{ goto: 'https://example.com' }],
+    };
+
+    it('should pass force option when provided', async () => {
+      const mockResult = {
+        success: true,
+        imported: [{ name: 'Test', id: '123', action: 'created', version_hash: 'abc123' }],
+        errors: [],
+      };
+      mockAxiosInstance.post.mockResolvedValueOnce({ data: mockResult });
+
+      await client.importTestDefinition([testDef], { upsert: true, force: true });
+
+      expect(mockAxiosInstance.post).toHaveBeenCalledWith('/vibe-qa/cli/import', {
+        test_definitions: [testDef],
+        upsert: true,
+        dry_run: false,
+        force: true,
+      });
+    });
+
+    it('should default force to false', async () => {
+      const mockResult = {
+        success: true,
+        imported: [{ name: 'Test', id: '123', action: 'created' }],
+        errors: [],
+      };
+      mockAxiosInstance.post.mockResolvedValueOnce({ data: mockResult });
+
+      await client.importTestDefinition([testDef], { upsert: true });
+
+      expect(mockAxiosInstance.post).toHaveBeenCalledWith('/vibe-qa/cli/import', {
+        test_definitions: [testDef],
+        upsert: true,
+        dry_run: false,
+        force: false,
+      });
+    });
+
+    it('should handle conflict action in response', async () => {
+      const mockResult = {
+        success: true,
+        imported: [
+          {
+            name: 'Test',
+            id: '123',
+            action: 'conflict',
+            message: 'Remote modified since last sync',
+            prev_version_hash: 'old123',
+            version_hash: 'new456',
+          },
+        ],
+        errors: [],
+      };
+      mockAxiosInstance.post.mockResolvedValueOnce({ data: mockResult });
+
+      const result = await client.importTestDefinition([testDef], { upsert: true });
+
+      expect(result.success).toBe(true);
+      expect(result.imported[0].action).toBe('conflict');
+      expect(result.imported[0].message).toBe('Remote modified since last sync');
+      expect(result.imported[0].prev_version_hash).toBe('old123');
+      expect(result.imported[0].version_hash).toBe('new456');
+    });
+
+    it('should handle unchanged action in response', async () => {
+      const mockResult = {
+        success: true,
+        imported: [
+          {
+            name: 'Test',
+            id: '123',
+            action: 'unchanged',
+            version_hash: 'same123',
+          },
+        ],
+        errors: [],
+      };
+      mockAxiosInstance.post.mockResolvedValueOnce({ data: mockResult });
+
+      const result = await client.importTestDefinition([testDef], { upsert: true });
+
+      expect(result.success).toBe(true);
+      expect(result.imported[0].action).toBe('unchanged');
+    });
+
+    it('should pass dry_run option', async () => {
+      const mockResult = {
+        success: true,
+        imported: [{ name: 'Test', id: '123', action: 'created' }],
+        errors: [],
+      };
+      mockAxiosInstance.post.mockResolvedValueOnce({ data: mockResult });
+
+      await client.importTestDefinition([testDef], { dry_run: true });
+
+      expect(mockAxiosInstance.post).toHaveBeenCalledWith('/vibe-qa/cli/import', {
+        test_definitions: [testDef],
+        upsert: true,
+        dry_run: true,
+        force: false,
+      });
+    });
+
+    it('should use default options when none provided', async () => {
+      const mockResult = {
+        success: true,
+        imported: [],
+        errors: [],
+      };
+      mockAxiosInstance.post.mockResolvedValueOnce({ data: mockResult });
+
+      await client.importTestDefinition([testDef]);
+
+      expect(mockAxiosInstance.post).toHaveBeenCalledWith('/vibe-qa/cli/import', {
+        test_definitions: [testDef],
+        upsert: true,
+        dry_run: false,
+        force: false,
+      });
+    });
+  });
+});

package/lib/api/index.ts

@@ -1,23 +1,23 @@
-import axios from 'axios';
 import type { AxiosInstance, AxiosResponse } from 'axios';
+import axios from 'axios';
 import 'dotenv/config';
-import { getEnv } from '../env/index.js';
+import type { TestDefinition } from '../../src/types/test-definition.js';
 import type {
-  TestAgentV2Session,
+  IssueReport,
   IssueType,
   Severity,
-  IssueReport,
+  TestAgentV2Session,
   TestCreatorDoneIntent,
 } from '../../src/types.js';
+import type { BlockSummary, EnhancedTestSummary } from '../../src/utils/summary.js';
 import {
-  generateEnhancedTestSummary,
+  categorizeIssues,
   formatEnhancedTestReport,
+  generateEnhancedTestSummary,
   generateIssueStatistics,
-  categorizeIssues,
 } from '../../src/utils/summary.js';
-import type { EnhancedTestSummary, BlockSummary } from '../../src/utils/summary.js';
-import { streamSSE, type SSEEvent } from './sse.js';
-import type { TestDefinition } from '../../src/types/test-definition.js';
+import { getEnv } from '../env/index.js';
+import { type SSEEvent, streamSSE } from './sse.js';
 
 // Re-export new types for external consumers
 export type { IssueType, Severity, IssueReport, TestCreatorDoneIntent };
@@ -285,12 +285,16 @@ export interface ValidationResult {
 export interface ImportOptions {
   upsert?: boolean;
   dry_run?: boolean;
+  force?: boolean; // Override version conflicts
 }
 
 export interface ImportedTest {
   name: string;
   id: string;
-  action: 'created' | 'updated' | 'skipped';
+  action: 'created' | 'updated' | 'skipped' | 'unchanged' | 'conflict';
+  message?: string; // Warning/conflict details
+  prev_version_hash?: string; // Hash before operation
+  version_hash?: string; // Hash after operation
 }
 
 export interface ImportResult {
@@ -331,7 +335,7 @@ export class ApiClient {
 
   setApiKey(apiKey: string): void {
     this.apiKey = apiKey;
-    this.client.defaults.headers['Authorization'] = `Bearer ${apiKey}`;
+    this.client.defaults.headers.Authorization = `Bearer ${apiKey}`;
   }
 
   getApiKey(): string | null {
@@ -936,6 +940,7 @@ export class ApiClient {
         test_definitions: definitions,
         upsert: options.upsert ?? true,
         dry_run: options.dry_run ?? false,
+        force: options.force ?? false,
       });
 
       return response.data as ImportResult;
@@ -944,6 +949,15 @@ export class ApiClient {
         const statusCode = error.response?.status;
         const errorData = error.response?.data;
 
+        // Handle validation errors (detail is an array of error objects)
+        if (Array.isArray(errorData?.detail)) {
+          const messages = errorData.detail.map((e: { msg?: string; loc?: string[] }) => {
+            const field = e.loc?.slice(2).join('.') || 'unknown';
+            return `${field}: ${e.msg || 'validation error'}`;
+          });
+          throw new Error(`Validation failed:\n  ${messages.join('\n  ')}`);
+        }
+
         throw new Error(
           errorData?.message ||
             errorData?.detail ||

package/lib/browser/index.ts

@@ -1,10 +1,9 @@
+import { fork } from 'node:child_process';
+import fs from 'node:fs';
+import { createRequire } from 'node:module';
+import path from 'node:path';
+import type { Browser, BrowserServer } from 'playwright';
 import { chromium } from 'playwright';
-import type { BrowserServer, Browser } from 'playwright';
-import { fork } from 'child_process';
-import path from 'path';
-import fs from 'fs';
-import { createRequire } from 'module';
-import { fileURLToPath } from 'url';
 
 export interface BrowserInstallationStatus {
   installed: boolean;
@@ -90,7 +89,7 @@ export class BrowserManager {
 
     try {
       await session.browserServer.close();
-    } catch (error) {
+    } catch {
       // Silently handle cleanup errors
     }
   }
@@ -102,7 +101,7 @@ export class BrowserManager {
 
     try {
       return await chromium.connect(this.session.wsEndpoint);
-    } catch (error) {
+    } catch {
       return null;
     }
   }
@@ -127,7 +126,7 @@ export class BrowserManager {
       });
       await browser.close();
       return true;
-    } catch (error) {
+    } catch {
       this.session.isActive = false;
       return false;
     }

package/lib/env/index.ts

@@ -6,9 +6,9 @@
  * 2. Config file (~/.qa-use.json) with structure: { "env": { "VAR_NAME": "value" } }
  */
 
-import { readFileSync, existsSync } from 'fs';
-import { join } from 'path';
-import { homedir } from 'os';
+import { existsSync, readFileSync } from 'node:fs';
+import { homedir } from 'node:os';
+import { join } from 'node:path';
 
 interface QaUseConfig {
   env?: Record<string, string>;
@@ -133,7 +133,7 @@ export function logConfigSources(): void {
   const sources: string[] = [];
 
   if (apiKey.value) {
-    const maskedKey = apiKey.value.slice(0, 8) + '...' + apiKey.value.slice(-4);
+    const maskedKey = `${apiKey.value.slice(0, 8)}...${apiKey.value.slice(-4)}`;
     sources.push(`  API Key: ${maskedKey} (from ${getSourceDescription(apiKey.source)})`);
   }
 

package/lib/tunnel/index.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, beforeEach, mock } from 'bun:test';
+import { beforeEach, describe, expect, it, mock } from 'bun:test';
 import { TunnelManager } from './index';
 
 // Mock the localtunnel module

package/lib/tunnel/index.ts

@@ -1,7 +1,7 @@
+import crypto from 'node:crypto';
+import https from 'node:https';
+import { URL } from 'node:url';
 import localtunnel from '@desplega.ai/localtunnel';
-import { URL } from 'url';
-import https from 'https';
-import crypto from 'crypto';
 import { getEnv } from '../env/index.js';
 
 export interface TunnelSession {
@@ -124,7 +124,7 @@ export class TunnelManager {
 
     try {
       await session.tunnel.close();
-    } catch (error) {
+    } catch {
       // Silently handle cleanup errors
     }
   }
@@ -155,7 +155,7 @@ export class TunnelManager {
       clearTimeout(timeout);
       // 426 = Upgrade Required (expected for WebSocket endpoint)
       return response.ok || response.status === 426;
-    } catch (error) {
+    } catch {
       this.session.isActive = false;
       return false;
     }
@@ -190,7 +190,7 @@ export class TunnelManager {
       return (
         this.session.publicUrl.replace('https://', 'wss://').replace('http://', 'ws://') + wsPath
       );
-    } catch (error) {
+    } catch {
       return null;
     }
   }

package/lib/tunnel/integration.test.ts

@@ -1,6 +1,6 @@
-import { describe, it, expect, beforeEach, afterEach } from 'bun:test';
+import { afterEach, beforeEach, describe, expect, it } from 'bun:test';
+import http from 'node:http';
 import { TunnelManager } from './index';
-import http from 'http';
 
 describe('TunnelManager Integration Tests', () => {
   let tunnelManager: TunnelManager;
@@ -12,7 +12,7 @@ describe('TunnelManager Integration Tests', () => {
 
     // Create a simple test HTTP server
     return new Promise<void>((resolve) => {
-      testServer = http.createServer((req, res) => {
+      testServer = http.createServer((_req, res) => {
         res.writeHead(200, { 'Content-Type': 'text/plain' });
         res.end('Hello from test server');
       });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@desplega.ai/qa-use",
-  "version": "2.4.1",
+  "version": "2.5.0",
   "packageManager": "bun@^1.3.4",
   "description": "QA automation tool for browser testing with MCP server support",
   "type": "module",
@@ -26,10 +26,12 @@
     "start": "bun run build && node dist/src/index.js",
     "inspector": "bun run build && npx -y @modelcontextprotocol/inspector node dist/src/index.js",
     "inspector:env": "bun run build && npx dotenv-cli -e .env -- npx -y @modelcontextprotocol/inspector node dist/src/index.js",
-    "format": "prettier --write \"src/**/*.{ts,tsx,js,jsx,json}\" \"lib/**/*.{ts,tsx,js,jsx,json}\"",
-    "format:check": "prettier --check \"src/**/*.{ts,tsx,js,jsx,json}\" \"lib/**/*.{ts,tsx,js,jsx,json}\"",
-    "lint": "eslint \"src/**/*.{ts,tsx,js,jsx}\" \"lib/**/*.{ts,tsx,js,jsx}\"",
-    "lint:fix": "eslint \"src/**/*.{ts,tsx,js,jsx}\" \"lib/**/*.{ts,tsx,js,jsx}\" --fix",
+    "format": "biome format --write src lib",
+    "format:check": "biome format src lib",
+    "lint": "biome lint src lib && oxlint src lib",
+    "lint:fix": "biome lint --write src lib && oxlint src lib --fix",
+    "check": "biome check src lib",
+    "check:fix": "biome check --write src lib",
     "typecheck": "tsc --noEmit",
     "test": "bun test --preload ./test-setup.ts",
     "test:unit": "node scripts/test-unit.js",
@@ -66,25 +68,19 @@
     "access": "public"
   },
   "devDependencies": {
-    "@eslint/js": "^9.36.0",
+    "@biomejs/biome": "^2.3.13",
     "@types/express": "^5.0.0",
     "@types/node": "^24.5.2",
     "@types/react": "^19.1.14",
     "@types/ws": "^8.18.1",
-    "@typescript-eslint/eslint-plugin": "^8.44.1",
-    "@typescript-eslint/parser": "^8.44.1",
     "@vercel/node": "^3.2.23",
     "dotenv-cli": "^7.4.2",
-    "eslint": "^9.36.0",
-    "eslint-config-prettier": "^10.1.8",
-    "eslint-plugin-prettier": "^5.5.4",
     "ignore": "^7.0.5",
     "json-schema-to-typescript": "^15.0.4",
-    "prettier": "^3.6.2",
+    "oxlint": "^1.42.0",
     "ts-node": "^10.9.2",
     "tsx": "^4.20.6",
-    "typescript": "^5.9.2",
-    "typescript-eslint": "^8.44.1"
+    "typescript": "^5.9.2"
   },
   "dependencies": {
     "@desplega.ai/localtunnel": "^2.2.0",