major cleanup

Author: ianjennings

.github/workflows/linux-tests.yml

@@ -0,0 +1,28 @@
+name: Linux Tests
+
+on:
+  push:
+    branches: [ main, develop ]
+  pull_request:
+    branches: [ main, develop ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    
+    steps:
+    - uses: actions/checkout@v4
+    
+    - name: Setup Node.js
+      uses: actions/setup-node@v4
+      with:
+        node-version: '20'
+        cache: 'npm'
+    
+    - name: Install dependencies
+      run: npm ci
+    
+    - name: Run Linux tests
+      run: npx vitest run test/testdriver/*.test.mjs
+      env:
+        TD_API_KEY: ${{ secrets.TD_API_KEY }}

agent/index.js

@@ -224,13 +224,12 @@ class TestDriverAgent extends EventEmitter2 {
       this.emitter.emit(events.error.fatal, errorContext);
     } else {
       this.emitter.emit(
-        events.error.fatal,
-        theme.red("Fatal Error") + `\n${error}`,
+        events.error.fatal,error,
       );
     }
 
     if (skipPostrun) {
-      this.exit(true);
+      return await this.exit(true);
     } else {
       try {
         await this.summarize(error.message);
@@ -2160,7 +2159,7 @@ Please check your network connection, TD_API_KEY, or the service status.`,
       type: "create",
       resolution: this.config.TD_RESOLUTION,
       ci: this.config.CI,
-      os: this.sandboxOs || "windows",
+      os: this.sandboxOs || "linux",
     };
 
     // Add AMI and instance type if specified

agent/interface.js

@@ -69,7 +69,7 @@ function createCommandDefinitions(agent) {
         os: Flags.string({
           description: "Operating system for the sandbox (windows or linux)",
           options: ["windows", "linux"],
-          default: "windows",
+          default: "linux",
         }),
       },
       handler: async (args, flags) => {
@@ -246,7 +246,7 @@ function createCommandDefinitions(agent) {
         os: Flags.string({
           description: "Operating system for the sandbox (windows or linux)",
           options: ["windows", "linux"],
-          default: "windows",
+          default: "linux",
         }),
       },
       handler: async (args, flags) => {

docs/v7/api/dashcam.mdx

@@ -38,7 +38,7 @@ For more control, create a Dashcam instance directly:
 
 ```javascript
 import TestDriver from 'testdriverai';
-import Dashcam from 'testdriverai/src/core/Dashcam.js';
+import Dashcam from 'testdriverai/lib/core/Dashcam.js';
 
 const client = await TestDriver.create({ os: 'linux' });
 const dashcam = new Dashcam(client, {
@@ -290,7 +290,7 @@ dashcam.client // TestDriver instance
 ```javascript
 import { test } from 'vitest';
 import TestDriver from 'testdriverai';
-import Dashcam from 'testdriverai/src/core/Dashcam.js';
+import Dashcam from 'testdriverai/lib/core/Dashcam.js';
 
 test('record test execution', async () => {
   const client = await TestDriver.create({ os: 'linux' });

docs/v7/getting-started/configuration.mdx

@@ -236,7 +236,7 @@ await testdriver.find('button', { threshold: -1 });   // disable cache
 ## Dashcam Configuration
 
 ```javascript
-import Dashcam from 'testdriverai/src/core/Dashcam.js';
+import Dashcam from 'testdriverai/lib/core/Dashcam.js';
 
 const dashcam = new Dashcam(client, {
   apiKey: process.env.TD_API_KEY,

docs/v7/guides/implementation-plan.mdx

@@ -14,7 +14,7 @@ Extract core components into clean, composable modules without breaking existing
 ### Tasks
 
 #### 1.1 Create Dashcam Class (Priority: HIGH)
-**File**: `src/core/Dashcam.js` (new)
+**File**: `lib/core/Dashcam.js` (new)
 
 **What to build:**
 ```javascript
@@ -123,14 +123,14 @@ export async function fillForm(client, formData) {
 {
   "exports": {
     ".": "./sdk.js",
-    "./core": "./src/core/index.js",
+    "./core": "./lib/core/index.js",
     "./helpers": "./src/helpers/index.js",
     "./vitest": "./interfaces/vitest-plugin.mjs"
   }
 }
 ```
 
-**Create**: `src/core/index.js`
+**Create**: `lib/core/index.js`
 ```javascript
 export { default as TestDriver } from '../../sdk.js';
 export { Dashcam } from './Dashcam.js';
@@ -223,7 +223,7 @@ export function useTestDriver(options = {}) {
 **What to build:**
 ```javascript
 import { beforeEach, afterEach, beforeAll, afterAll } from 'vitest';
-import { Dashcam } from '../../src/core/Dashcam.js';
+import { Dashcam } from '../../lib/core/Dashcam.js';
 
 let dashcamInstance = null;
 
@@ -370,7 +370,7 @@ Build preset system with at least 2 working presets (Chrome, VSCode).
 ### Tasks
 
 #### 3.1 Define Preset Interface (Priority: HIGH)
-**File**: `src/presets/types.js` (new)
+**File**: `lib/presets/types.js` (new)
 
 **What to build:**
 ```javascript
@@ -423,7 +423,7 @@ export function definePreset(config) {
 ---
 
 #### 3.2 Build Chrome Preset (Priority: HIGH)
-**File**: `src/presets/chrome.js` (new)
+**File**: `lib/presets/chrome.js` (new)
 
 **What to build:**
 ```javascript
@@ -484,7 +484,7 @@ export function chromePreset(options = {}) {
 ---
 
 #### 3.3 Build VSCode Preset (Priority: MEDIUM)
-**File**: `src/presets/vscode.js` (new)
+**File**: `lib/presets/vscode.js` (new)
 
 **What to build:**
 ```javascript
@@ -549,7 +549,7 @@ export function vscodePreset(options = {}) {
 ---
 
 #### 3.4 Create Preset Registry (Priority: LOW)
-**File**: `src/presets/index.js` (new)
+**File**: `lib/presets/index.js` (new)
 
 **What to build:**
 ```javascript

docs/v7/guides/sdk-v7-complete.mdx

@@ -9,7 +9,7 @@ Implementation of the TestDriver SDK v7 redesign following progressive disclosur
 ## Phase 1: Core Foundation ✅ (Week 1-2)
 
 ### Task 1.1: Dashcam Class Extraction ✅
-- **Created:** `src/core/Dashcam.js` (422 lines)
+- **Created:** `lib/core/Dashcam.js` (422 lines)
 - **Features:**
   - Composable Dashcam class independent of lifecycle helpers
   - Methods: `auth()`, `addLog()`, `start()`, `stop()`, `isRecording()`
@@ -25,10 +25,10 @@ Implementation of the TestDriver SDK v7 redesign following progressive disclosur
 - **Modified:** `package.json`
 - **Added exports:**
   - `"."` → `./sdk.js` (main SDK)
-  - `"./core"` → `./src/core/index.js` (TestDriver + Dashcam)
+  - `"./core"` → `./lib/core/index.js` (TestDriver + Dashcam)
   - `"./vitest"` → `./interfaces/vitest-plugin.mjs` (plugin)
-  - `"./vitest/hooks"` → `./src/vitest/hooks.mjs` (hooks)
-  - `"./presets"` → `./src/presets/index.mjs` (presets)
+  - `"./vitest/hooks"` → `./lib/vitest/hooks.mjs` (hooks)
+  - `"./presets"` → `./lib/presets/index.mjs` (presets)
 
 ### Key Achievements:
 - ✅ Backward compatibility maintained via thin wrappers
@@ -43,7 +43,7 @@ Implementation of the TestDriver SDK v7 redesign following progressive disclosur
 ## Phase 2: Vitest Plugin Enhancement ✅ (Week 3-4)
 
 ### Task 2.1: Vitest Hooks API ✅
-- **Created:** `src/vitest/hooks.mjs` (221 lines)
+- **Created:** `lib/vitest/hooks.mjs` (221 lines)
 - **Three hooks implemented:**
   1. `useTestDriver(context, options)` - Managed TestDriver instance
      - Auto-connect to sandbox (default: true)
@@ -78,7 +78,7 @@ Implementation of the TestDriver SDK v7 redesign following progressive disclosur
 ## Phase 3: Presets System ✅ (Week 5-6)
 
 ### Task 3.1: Built-in Presets ✅
-- **Created:** `src/presets/index.mjs` (400+ lines)
+- **Created:** `lib/presets/index.mjs` (400+ lines)
 
 **chromePreset:**
 - Auto-launches Chrome with configurable URL
@@ -135,17 +135,17 @@ Implementation of the TestDriver SDK v7 redesign following progressive disclosur
 
 ### Task 4.1: TypeScript Definitions ✅
 
-**Created: `src/core/index.d.ts`**
+**Created: `lib/core/index.d.ts`**
 - Full types for `TestDriver` and `Dashcam` classes
 - Interfaces: `DashcamOptions`, `LogConfig`, `TestDriverOptions`, `ConnectOptions`
 - Comprehensive JSDoc comments
 
-**Created: `src/vitest/hooks.d.ts`**
+**Created: `lib/vitest/hooks.d.ts`**
 - Types for all three hooks
 - Interfaces: `VitestContext`, `UseTestDriverOptions`, `UseDashcamOptions`
 - Full autocomplete support
 
-**Created: `src/presets/index.d.ts`**
+**Created: `lib/presets/index.d.ts`**
 - Types for all presets
 - Interfaces: `ChromePresetOptions`, `VSCodePresetOptions`, `ElectronPresetOptions`
 - `PresetSetupFunction` and `PresetConfig` types
@@ -248,9 +248,9 @@ import { authDashcam, startDashcam, stopDashcam } from 'testdriverai';
 ## Key Metrics
 
 ### Lines of Code (New)
-- `src/core/Dashcam.js`: 422 lines
-- `src/vitest/hooks.mjs`: 221 lines
-- `src/presets/index.mjs`: 400+ lines
+- `lib/core/Dashcam.js`: 422 lines
+- `lib/vitest/hooks.mjs`: 221 lines
+- `lib/presets/index.mjs`: 400+ lines
 - TypeScript definitions: 500+ lines
 - Documentation: 1000+ lines
 - **Total new code:** ~2500+ lines