@samsara-dev/appwright 0.9.13 → 0.9.16

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # appwright
 
+## 0.9.16
+
+### Patch Changes
+
+- 7e73c82: Fix persistent device video download race condition that caused the process to hang when a worker had only one test (common with retries). When the 5-second delay wasn't enough for endTime to be written, the code misclassified the last test as intermediate and waited for a file that only the endTime download path would create. The intermediate test branch now polls for both the video file on disk and endTime being set, and all downloads go through a per-session lock to prevent concurrent writes.
+
+## 0.9.15
+
+### Patch Changes
+
+- 7e70815: Fix stale testId in GPT Driver when using persistent device fixtures
+
+  Update testId on the cached gpt-driver-node instance before every API call
+  so persistent device fixtures (scope: "worker") report the correct test in
+  GPT Driver sessions. Previously the testId was baked in on first use and
+  never updated, causing all subsequent tests and resetToHome teardowns to
+  report the same stale testId on the MobileBoost dashboard.
+
+## 0.9.14
+
+### Patch Changes
+
+- c7b7722: Switch GPT Driver cachingMode from FULL_SCREEN to INTERACTION_REGION
+
 ## 0.9.13
 
 ### Patch Changes

package/dist/gptDriver/index.d.ts

@@ -2,7 +2,7 @@ import type { Client as WebDriverClient } from "webdriver";
 import type { GptDriverConfig } from "../types";
 /**
  * Options for aiExecute. Intentionally exposes only appiumHandler;
- * cachingMode is set globally at provider init (FULL_SCREEN),
+ * cachingMode is set globally at provider init (INTERACTION_REGION),
  * and useSmartLoop is not yet surfaced.
  */
 export interface AiExecuteOptions {
@@ -32,6 +32,7 @@ export declare class GptDriverProvider implements GptDriverApi {
     private webDriverClient;
     private options?;
     private driver;
+    private testIdWarned;
     constructor(webDriverClient: WebDriverClient, options?: GptDriverConfig | undefined);
     private getAppiumUrl;
     private getDriver;

package/dist/gptDriver/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/gptDriver/index.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,MAAM,IAAI,eAAe,EAAE,MAAM,WAAW,CAAC;AAG3D,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,UAAU,CAAC;AAIhD;;;;GAIG;AACH,MAAM,WAAW,gBAAgB;IAC/B,aAAa,CAAC,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CACrC;AAED;;GAEG;AACH,MAAM,WAAW,YAAY;IAC3B,SAAS,CAAC,WAAW,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,gBAAgB,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAC1E,MAAM,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IACzC,UAAU,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAChD,SAAS,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,CAAC;IAClE,OAAO,CAAC,WAAW,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,CAAC;CAC9D;AAED;;;;;;;;;GASG;AACH,qBAAa,iBAAkB,YAAW,YAAY;IAIlD,OAAO,CAAC,eAAe;IACvB,OAAO,CAAC,OAAO,CAAC;IAJlB,OAAO,CAAC,MAAM,CAA0B;gBAG9B,eAAe,EAAE,eAAe,EAChC,OAAO,CAAC,EAAE,eAAe,YAAA;IAGnC,OAAO,CAAC,YAAY;IAQpB,OAAO,CAAC,SAAS;IA0BjB,OAAO,CAAC,aAAa;IAYrB,OAAO,CAAC,mBAAmB;IAW3B,OAAO,CAAC,kBAAkB;IAS1B,OAAO,CAAC,aAAa;IAMf,SAAS,CACb,WAAW,EAAE,MAAM,EACnB,OAAO,CAAC,EAAE,gBAAgB,GACzB,OAAO,CAAC,IAAI,CAAC;IAkBV,MAAM,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAaxC,UAAU,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC;IAa/C,SAAS,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAajE,OAAO,CAAC,WAAW,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;CAWnE"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/gptDriver/index.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,MAAM,IAAI,eAAe,EAAE,MAAM,WAAW,CAAC;AAG3D,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,UAAU,CAAC;AAIhD;;;;GAIG;AACH,MAAM,WAAW,gBAAgB;IAC/B,aAAa,CAAC,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CACrC;AAED;;GAEG;AACH,MAAM,WAAW,YAAY;IAC3B,SAAS,CAAC,WAAW,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,gBAAgB,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAC1E,MAAM,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IACzC,UAAU,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAChD,SAAS,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,CAAC;IAClE,OAAO,CAAC,WAAW,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,CAAC;CAC9D;AAED;;;;;;;;;GASG;AACH,qBAAa,iBAAkB,YAAW,YAAY;IAKlD,OAAO,CAAC,eAAe;IACvB,OAAO,CAAC,OAAO,CAAC;IALlB,OAAO,CAAC,MAAM,CAA0B;IACxC,OAAO,CAAC,YAAY,CAAS;gBAGnB,eAAe,EAAE,eAAe,EAChC,OAAO,CAAC,EAAE,eAAe,YAAA;IAGnC,OAAO,CAAC,YAAY;IAQpB,OAAO,CAAC,SAAS;IA0BjB,OAAO,CAAC,aAAa;IA8BrB,OAAO,CAAC,mBAAmB;IAW3B,OAAO,CAAC,kBAAkB;IAS1B,OAAO,CAAC,aAAa;IAMf,SAAS,CACb,WAAW,EAAE,MAAM,EACnB,OAAO,CAAC,EAAE,gBAAgB,GACzB,OAAO,CAAC,IAAI,CAAC;IAkBV,MAAM,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAaxC,UAAU,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC;IAa/C,SAAS,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAajE,OAAO,CAAC,WAAW,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;CAWnE"}

package/dist/gptDriver/index.js

@@ -77,6 +77,7 @@ let GptDriverProvider = (() => {
         webDriverClient = __runInitializers(this, _instanceExtraInitializers);
         options;
         driver = null;
+        testIdWarned = false;
         constructor(webDriverClient, options) {
             this.webDriverClient = webDriverClient;
             this.options = options;
@@ -100,7 +101,7 @@ let GptDriverProvider = (() => {
                 // gpt-driver-node expects webdriverio Browser type but webdriver Client is API-compatible at runtime
                 driver: this.webDriverClient,
                 serverConfig: { url: this.getAppiumUrl() },
-                cachingMode: "FULL_SCREEN",
+                cachingMode: "INTERACTION_REGION",
                 testId: test_1.default.info()?.testId ?? `test-${Date.now()}`,
                 ...(this.options?.additionalUserContext != null && {
                     additionalUserContext: this.options.additionalUserContext,
@@ -115,6 +116,21 @@ let GptDriverProvider = (() => {
                 test_1.default.skip(true, "GPT Driver not configured. Set GPT_DRIVER_API_KEY environment variable.");
                 throw new Error("GPT Driver not configured");
             }
+            // Update testId to current test context so persistent device fixtures
+            // report the correct test in GPT Driver API calls.
+            // gpt-driver-node declares testId as private in TS but it's a plain
+            // JS property at runtime — safe to mutate directly.
+            // TODO: Replace with public setTestId() if gpt-driver-node exposes one.
+            const currentTestId = test_1.default.info()?.testId;
+            if (currentTestId && "testId" in driver) {
+                driver.testId = currentTestId;
+            }
+            else if (currentTestId && !this.testIdWarned) {
+                console.warn("[GptDriver] Cannot update testId — property not found on driver instance. " +
+                    "GPT Driver sessions may be attributed to the wrong test. " +
+                    "Check if gpt-driver-node renamed or removed the testId field.");
+                this.testIdWarned = true;
+            }
             return driver;
         }
         validateInstruction(instruction, methodName) {

package/dist/reporter.d.ts

@@ -1,11 +1,21 @@
 import type { Reporter, TestCase, TestResult } from "@playwright/test/reporter";
 declare class VideoDownloader implements Reporter {
     private downloadPromises;
+    private sessionDownloadLocks;
     onBegin(): void;
     onTestBegin(test: TestCase, result: TestResult): void;
     onTestEnd(test: TestCase, result: TestResult): void;
     onEnd(): Promise<void>;
     private trimAndAttachPersistentDeviceVideo;
+    /**
+     * Ensures exactly one download per session. The first caller triggers the
+     * download; concurrent callers await the same promise. Returns the
+     * downloaded file path, or null if the download failed.
+     *
+     * Keyed by sessionId (not workerIndex) because Playwright can reuse
+     * workerIndex across worker restarts with different BrowserStack sessions.
+     */
+    private ensureWorkerVideoDownloaded;
     private downloadAndAttachDeviceVideo;
     private providerSupportsVideo;
 }

package/dist/reporter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"reporter.d.ts","sourceRoot":"","sources":["../src/reporter.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,QAAQ,EAAE,UAAU,EAAE,MAAM,2BAA2B,CAAC;AAUhF,cAAM,eAAgB,YAAW,QAAQ;IACvC,OAAO,CAAC,gBAAgB,CAAsB;IAE9C,OAAO;IAQP,WAAW,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU;IAU9C,SAAS,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU;IAuGtC,KAAK;YAKG,kCAAkC;IA6ChD,OAAO,CAAC,4BAA4B;IAyBpC,OAAO,CAAC,qBAAqB;CAU9B;AAwHD,eAAe,eAAe,CAAC"}
+{"version":3,"file":"reporter.d.ts","sourceRoot":"","sources":["../src/reporter.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,QAAQ,EAAE,UAAU,EAAE,MAAM,2BAA2B,CAAC;AAUhF,cAAM,eAAgB,YAAW,QAAQ;IACvC,OAAO,CAAC,gBAAgB,CAAsB;IAK9C,OAAO,CAAC,oBAAoB,CAA6C;IAEzE,OAAO;IAQP,WAAW,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU;IAU9C,SAAS,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU;IAgItC,KAAK;YAKG,kCAAkC;IA6ChD;;;;;;;OAOG;YACW,2BAA2B;IAqBzC,OAAO,CAAC,4BAA4B;IAyBpC,OAAO,CAAC,qBAAqB;CAU9B;AA0ID,eAAe,eAAe,CAAC"}

package/dist/reporter.js

@@ -13,6 +13,11 @@ const utils_1 = require("./utils");
 const workerInfo_1 = require("./fixture/workerInfo");
 class VideoDownloader {
     downloadPromises = [];
+    // Ensures only one download per session — prevents concurrent downloads
+    // when multiple tests on the same worker resolve endTime simultaneously.
+    // Keyed by sessionId (not workerIndex) because Playwright can reuse
+    // workerIndex across worker restarts with different sessions.
+    sessionDownloadLocks = new Map();
     onBegin() {
         if (fs_1.default.existsSync((0, utils_1.basePath)())) {
             fs_1.default.rmSync((0, utils_1.basePath)(), {
@@ -53,7 +58,7 @@ class VideoDownloader {
                 type: "workerInfo",
                 description: `Ran on worker #${workerIndex}.`,
             });
-            // The `onTestEnd` is method is called before the worker ends and
+            // The `onTestEnd` method is called before the worker ends and
             // the worker's `endTime` is saved to disk. We add a 5 secs delay
             // to prevent a harmful race condition.
             const workerDownload = getWorkerInfo(workerIndex)
@@ -77,21 +82,37 @@ class VideoDownloader {
                         return; // Nothing to do here
                     }
                     const workerVideoBaseName = `worker-${workerIndex}-${sessionId}-video`;
+                    const expectedWorkerVideoPath = path_1.default.join((0, utils_1.basePath)(), `${workerVideoBaseName}.mp4`);
                     if (endTime) {
-                        // This is the last test in the worker, so let's download the video
-                        const provider = (0, providers_1.getProviderClass)(providerName);
-                        const downloaded = await provider.downloadVideo(sessionId, (0, utils_1.basePath)(), workerVideoBaseName);
-                        if (!downloaded) {
+                        // Last test in the worker — download through the lock
+                        // so concurrent endTime resolutions don't race.
+                        const downloadedPath = await this.ensureWorkerVideoDownloaded(sessionId, providerName, workerVideoBaseName);
+                        if (!downloadedPath) {
                             return;
                         }
-                        return this.trimAndAttachPersistentDeviceVideo(test, result, downloaded.path);
+                        return this.trimAndAttachPersistentDeviceVideo(test, result, downloadedPath);
                     }
                     else {
-                        // This is an intermediate test in the worker, so let's wait for the
-                        // video file to be found on disk. Once it is, we trim and attach it.
-                        const expectedWorkerVideoPath = path_1.default.join((0, utils_1.basePath)(), `${workerVideoBaseName}.mp4`);
-                        await waitFor(() => fs_1.default.existsSync(expectedWorkerVideoPath));
-                        return this.trimAndAttachPersistentDeviceVideo(test, result, expectedWorkerVideoPath);
+                        // This looks like an intermediate test, but endTime may not
+                        // have been written yet (race condition — common when a worker
+                        // has only one test, e.g. retries). Poll for EITHER the video
+                        // file on disk (written by the last test's download) OR
+                        // endTime appearing (meaning we need to download ourselves).
+                        const resolved = await waitForFileOrEndTime(expectedWorkerVideoPath, workerIndex);
+                        if (resolved === "file") {
+                            return this.trimAndAttachPersistentDeviceVideo(test, result, expectedWorkerVideoPath);
+                        }
+                        else {
+                            // endTime was set but file doesn't exist yet. Use the
+                            // per-session lock so only one test triggers the download;
+                            // others await the same promise.
+                            const downloadedPath = await this.ensureWorkerVideoDownloaded(sessionId, providerName, workerVideoBaseName);
+                            if (!downloadedPath) {
+                                logger_1.logger.error(`Video download returned null for session ${sessionId}, skipping attachment`);
+                                return;
+                            }
+                            return this.trimAndAttachPersistentDeviceVideo(test, result, downloadedPath);
+                        }
                     }
                 });
             })
@@ -146,6 +167,25 @@ class VideoDownloader {
             name: "video",
         });
     }
+    /**
+     * Ensures exactly one download per session. The first caller triggers the
+     * download; concurrent callers await the same promise. Returns the
+     * downloaded file path, or null if the download failed.
+     *
+     * Keyed by sessionId (not workerIndex) because Playwright can reuse
+     * workerIndex across worker restarts with different BrowserStack sessions.
+     */
+    async ensureWorkerVideoDownloaded(sessionId, providerName, workerVideoBaseName) {
+        if (!this.sessionDownloadLocks.has(sessionId)) {
+            const downloadPromise = (async () => {
+                const provider = (0, providers_1.getProviderClass)(providerName);
+                const downloaded = await provider.downloadVideo(sessionId, (0, utils_1.basePath)(), workerVideoBaseName);
+                return downloaded?.path ?? null;
+            })();
+            this.sessionDownloadLocks.set(sessionId, downloadPromise);
+        }
+        return this.sessionDownloadLocks.get(sessionId);
+    }
     downloadAndAttachDeviceVideo(test, result, providerClass, sessionId) {
         const videoFileName = `${sessionId}-${test.id}`;
         if (!providerClass.downloadVideo) {
@@ -174,21 +214,37 @@ class VideoDownloader {
         return !!provider.downloadVideo;
     }
 }
-function waitFor(condition, timeout = 60 * 60 * 1000) {
-    return new Promise((resolve, reject) => {
-        let interval;
-        const timeoutId = setTimeout(() => {
-            clearInterval(interval);
-            reject(new Error("Timed out waiting for condition"));
-        }, timeout);
-        interval = setInterval(() => {
-            if (condition()) {
-                clearInterval(interval);
-                clearTimeout(timeoutId);
-                resolve();
+/**
+ * Poll until the video file appears on disk OR the worker's endTime is set.
+ *
+ * When a worker has only one test (common with retries), the 5-second delay
+ * before checking endTime may not be enough — the worker teardown can take
+ * longer than 5 seconds. In that case onTestEnd
+ * incorrectly treats the last test as "intermediate" and waits for a video file
+ * that will never be created. This function resolves the race by checking both
+ * conditions in parallel.
+ *
+ * Uses validateMp4 instead of fs.existsSync to ensure the file is fully
+ * written (has a valid moov atom) before returning.
+ */
+async function waitForFileOrEndTime(filePath, workerIndex, timeout = 5 * 60 * 1000) {
+    const deadline = Date.now() + timeout;
+    while (Date.now() < deadline) {
+        if (fs_1.default.existsSync(filePath) && validateMp4(filePath)) {
+            return "file";
+        }
+        try {
+            const info = await getWorkerInfo(workerIndex);
+            if (info?.endTime) {
+                return "endTime";
             }
-        }, 500);
-    });
+        }
+        catch {
+            // getWorkerInfo can fail transiently; keep polling
+        }
+        await new Promise((r) => setTimeout(r, 500));
+    }
+    throw new Error(`Timed out waiting for video file or worker ${workerIndex} endTime`);
 }
 /**
  * Validate that an MP4 file has a valid moov atom (metadata header).

package/dist/tests/gptDriver.spec.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=gptDriver.spec.d.ts.map

package/dist/tests/gptDriver.spec.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"gptDriver.spec.d.ts","sourceRoot":"","sources":["../../src/tests/gptDriver.spec.ts"],"names":[],"mappings":""}

package/dist/tests/gptDriver.spec.js

@@ -0,0 +1,131 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const vitest_1 = require("vitest");
+const test_1 = __importDefault(require("@playwright/test"));
+// --- Mock gpt-driver-node ---
+// vi.hoisted runs before vi.mock hoisting, so variables are available in the factory
+const { MockGptDriver, mockAiExecute } = vitest_1.vi.hoisted(() => {
+    const mockAiExecute = vitest_1.vi.fn().mockResolvedValue(undefined);
+    // Must use function (not arrow) so it can be called with `new`
+    const MockGptDriver = vitest_1.vi.fn(function (config) {
+        this.aiExecute = mockAiExecute;
+        this.assert = vitest_1.vi.fn().mockResolvedValue(undefined);
+        this.assertBulk = vitest_1.vi.fn().mockResolvedValue(undefined);
+        this.checkBulk = vitest_1.vi.fn().mockResolvedValue({});
+        this.extract = vitest_1.vi.fn().mockResolvedValue({});
+        this.testId = config.testId;
+    });
+    return { MockGptDriver, mockAiExecute };
+});
+vitest_1.vi.mock("gpt-driver-node", () => ({ default: MockGptDriver }));
+// --- Mock Playwright test helpers ---
+let currentTestId = "test-initial";
+test_1.default.step = vitest_1.vi.fn(async (_name, body) => await body());
+test_1.default.info = () => currentTestId ? { testId: currentTestId } : undefined;
+test_1.default.skip = vitest_1.vi.fn();
+const gptDriver_1 = require("../gptDriver");
+function createProvider() {
+    return new gptDriver_1.GptDriverProvider({
+        options: {
+            protocol: "http",
+            hostname: "localhost",
+            port: 4723,
+            path: "/wd/hub",
+        },
+    });
+}
+(0, vitest_1.describe)("GptDriverProvider", () => {
+    (0, vitest_1.beforeEach)(() => {
+        vitest_1.vi.clearAllMocks();
+        currentTestId = "test-initial";
+        process.env.GPT_DRIVER_API_KEY = "fake-key";
+    });
+    (0, vitest_1.afterEach)(() => {
+        delete process.env.GPT_DRIVER_API_KEY;
+    });
+    (0, vitest_1.describe)("testId tracks current test context", () => {
+        (0, vitest_1.test)("initializes testId from first call's test context", async () => {
+            const provider = createProvider();
+            currentTestId = "test-first";
+            await provider.aiExecute("login");
+            const driver = provider.driver;
+            (0, vitest_1.expect)(driver.testId).toBe("test-first");
+        });
+        (0, vitest_1.test)("updates testId when a different test uses the same device", async () => {
+            const provider = createProvider();
+            // Test A runs
+            currentTestId = "test-aaa";
+            await provider.aiExecute("tap the submit button");
+            (0, vitest_1.expect)(provider.driver.testId).toBe("test-aaa");
+            // Test B runs on the same persistent device
+            currentTestId = "test-bbb";
+            await provider.aiExecute("verify the results screen");
+            (0, vitest_1.expect)(provider.driver.testId).toBe("test-bbb");
+        });
+        (0, vitest_1.test)("fixture teardown uses the current test's ID, not the first", async () => {
+            const provider = createProvider();
+            // Test A runs
+            currentTestId = "test-aaa";
+            await provider.aiExecute("perform action");
+            // Fixture teardown runs in Test A's context
+            await provider.aiExecute("navigate back to home");
+            (0, vitest_1.expect)(provider.driver.testId).toBe("test-aaa");
+            // Test B starts with new context
+            currentTestId = "test-bbb";
+            await provider.aiExecute("perform another action");
+            (0, vitest_1.expect)(provider.driver.testId).toBe("test-bbb");
+            // Fixture teardown runs in Test B's context
+            await provider.aiExecute("navigate back to home");
+            (0, vitest_1.expect)(provider.driver.testId).toBe("test-bbb");
+        });
+        (0, vitest_1.test)("preserves testId when test.info() returns undefined", async () => {
+            const provider = createProvider();
+            currentTestId = "test-known";
+            await provider.aiExecute("do something");
+            (0, vitest_1.expect)(provider.driver.testId).toBe("test-known");
+            // Worker teardown — no test context
+            currentTestId = undefined;
+            await provider.aiExecute("cleanup");
+            (0, vitest_1.expect)(provider.driver.testId).toBe("test-known");
+        });
+        (0, vitest_1.test)("warns once if testId property is missing from driver", async () => {
+            const provider = createProvider();
+            const warnSpy = vitest_1.vi.spyOn(console, "warn").mockImplementation(() => { });
+            currentTestId = "test-aaa";
+            await provider.aiExecute("initialize");
+            // Remove testId to simulate SDK rename/removal
+            delete provider.driver.testId;
+            currentTestId = "test-bbb";
+            await provider.aiExecute("second call");
+            (0, vitest_1.expect)(warnSpy).toHaveBeenCalledTimes(1);
+            (0, vitest_1.expect)(warnSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining("Cannot update testId"));
+            // Should not warn again on subsequent calls
+            currentTestId = "test-ccc";
+            await provider.aiExecute("third call");
+            (0, vitest_1.expect)(warnSpy).toHaveBeenCalledTimes(1);
+            warnSpy.mockRestore();
+        });
+    });
+    (0, vitest_1.describe)("lazy singleton", () => {
+        (0, vitest_1.test)("creates GptDriver instance only once", async () => {
+            const provider = createProvider();
+            currentTestId = "test-1";
+            await provider.aiExecute("first");
+            currentTestId = "test-2";
+            await provider.aiExecute("second");
+            currentTestId = "test-3";
+            await provider.aiExecute("third");
+            (0, vitest_1.expect)(MockGptDriver).toHaveBeenCalledTimes(1);
+        });
+        (0, vitest_1.test)("does not create GptDriver when API key is missing", async () => {
+            delete process.env.GPT_DRIVER_API_KEY;
+            const provider = createProvider();
+            // Should call test.skip and throw
+            await (0, vitest_1.expect)(provider.aiExecute("anything")).rejects.toThrow("GPT Driver not configured");
+            (0, vitest_1.expect)(MockGptDriver).not.toHaveBeenCalled();
+        });
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@samsara-dev/appwright",
-  "version": "0.9.13",
+  "version": "0.9.16",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"