@testdriverai/agent 7.11.255-test → 7.11.257-test

package/agent/index.js

@@ -1691,8 +1691,9 @@ ${regression}
       createNew = true;
     }
 
-    // If CI environment variable is true, always create a new sandbox
-    if (this.config.CI && !requireSandbox) {
+    // If CI environment variable is true, create a new sandbox unless the
+    // caller explicitly named one to connect to
+    if (this.config.CI && !requireSandbox && !this.explicitSandboxId) {
       createNew = true;
       this.emitter.emit(
         events.log.log,
@@ -1769,7 +1770,8 @@ ${regression}
       return;
     } else if (
       this.sandboxId &&
-      (requireSandbox || (!createNew && !this.config.CI))
+      (requireSandbox ||
+        (!createNew && (!this.config.CI || this.explicitSandboxId)))
     ) {
       // Only attempt to connect to existing sandbox if not in CI mode and not creating new
       // Attempt to connect to known instance

package/docs/bring-your-own-machine.mdx

@@ -171,10 +171,6 @@ describe('My machine', () => {
 
 On a Windows machine, also pass `os: 'windows'`.
 
-<Warning>
-  Run these tests with the `CI` environment variable unset. When `CI` is set, TestDriver always requests a new sandbox and ignores `sandboxId`, so the test waits for a runner that never connects.
-</Warning>
-
 Run the test from any computer:
 
 ```bash

package/docs/cache.mdx

@@ -163,10 +163,6 @@ Or for a whole run:
 TD_NO_CACHE=true npx vitest run
 ```
 
-<Warning>
-  `cache: { enabled: false }` doesn't turn the cache off. Use `cache: false` or `TD_NO_CACHE=true`.
-</Warning>
-
 ## Inspect a result
 
 Read the cache details from the element that `find()` returns:

package/docs/options.mdx

@@ -197,7 +197,7 @@ These options control the element cache, which reuses element positions from a p
 <ParamField path="cache" type="boolean | object" default="true">
   Pass `false` to turn the cache off, or an object to set the match thresholds.
 
-  `cache: { enabled: false }` does not turn the cache off. Use `cache: false` or `TD_NO_CACHE=true`.
+  `cache: { enabled: false }` and `TD_NO_CACHE=true` also turn it off.
 
   <Expandable title="advanced config">
     <ParamField path="thresholds" type="object">

package/docs/parse.mdx

@@ -69,10 +69,6 @@ None. `parse()` analyzes the screen as it is when you call it.
   </Expandable>
 </ResponseField>
 
-<Warning>
-  `interactivity` is a boolean, although the TypeScript types in `sdk.d.ts` declare it as a string. Compare it with `true`, not with a string like `'clickable'`.
-</Warning>
-
 ## Errors
 
 `parse()` throws if the screen analysis fails. The error message comes from the TestDriver API.

package/docs/screenshots.mdx

@@ -78,17 +78,13 @@ Each automatic screenshot is named:
 The description comes from:
 
 - The description passed to `find()` or `findAll()`.
-- The first 20 characters typed by `type()`.
+- The first 20 characters typed by `type()`, unless you pass `secret: true`.
 - The keys passed to `pressKeys()`, joined with `+`.
 - The direction passed to `scroll()`.
 - The first 30 characters of the text passed to `assert()` or `extract()`.
 
 TestDriver lowercases it, drops anything other than letters, digits, spaces, and hyphens, turns spaces into hyphens, and keeps the first 30 characters.
 
-<Warning>
-  The description for `type()` uses the typed text even when you pass `secret: true`. Don't turn on `autoScreenshots` for tests that type passwords, or the start of the password ends up in a file name.
-</Warning>
-
 ### Example
 
 This test:

package/docs/secrets.mdx

@@ -45,11 +45,9 @@ With `secret: true`, TestDriver replaces the text with `****`:
 - In the SDK's console output.
 - In the record of the action that TestDriver stores for the test run.
 
-The text is still typed into the sandbox, so `secret: true` can't hide what the screen shows. If the field displays its contents, the value appears in the [Dashcam](/dashcam) recording and in screenshots. Password fields usually mask what you type, but plain text fields don't.
+With [`autoScreenshots`](/options#param-auto-screenshots) on, the typed text is also left out of screenshot file names.
 
-<Warning>
-  With [`autoScreenshots`](/options#param-auto-screenshots) on, screenshot file names include the first characters of the typed text, even when `secret` is `true`. Leave `autoScreenshots` off in tests that type secrets.
-</Warning>
+The text is still typed into the sandbox, so `secret: true` can't hide what the screen shows. If the field displays its contents, the value appears in the [Dashcam](/dashcam) recording and in screenshots. Password fields usually mask what you type, but plain text fields don't.
 
 ## Store secrets locally
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testdriverai/agent",
-  "version": "7.11.255-test",
+  "version": "7.11.257-test",
   "description": "Next generation autonomous AI agent for end-to-end testing of web & desktop",
   "main": "sdk.js",
   "types": "sdk.d.ts",

package/sdk.d.ts

@@ -435,8 +435,8 @@ export interface ParsedElement {
   type: string;
   /** Text content or description */
   content: string;
-  /** Interactivity level (e.g. "clickable", "non-interactive") */
-  interactivity: string;
+  /** Whether the element appears to be clickable */
+  interactivity: boolean;
   /** Bounding box in pixel coordinates */
   bbox: ParsedElementBBox;
   /** Bounding box as {left, top, width, height} */
@@ -1416,7 +1416,7 @@ export default class TestDriverSDK {
    * @example
    * // Find clickable elements
    * const result = await testdriver.parse();
-   * const clickable = result.elements.filter(e => e.interactivity === 'clickable');
+   * const clickable = result.elements.filter(e => e.interactivity);
    *
    * @example
    * // Find text content

package/sdk.js

@@ -1576,7 +1576,9 @@ class TestDriverSDK {
     // To enable cache, provide a cacheKey when calling find() or findAll()
     // Also support TD_NO_CACHE environment variable and cache: false option for backwards compatibility
     const cacheExplicitlyDisabled =
-      options.cache === false || process.env.TD_NO_CACHE === "true";
+      options.cache === false ||
+      options.cache?.enabled === false ||
+      process.env.TD_NO_CACHE === "true";
 
     // Track whether cache was explicitly disabled (not just default)
     this._cacheExplicitlyDisabled = cacheExplicitlyDisabled;
@@ -2130,6 +2132,11 @@ CAPTCHA_SOLVER_EOF`,
     };
 
     // Set agent properties for buildEnv to use
+    // A sandboxId the user passed (not one loaded from .testdriver/last-sandbox)
+    // is honored even in CI, where buildEnv otherwise always creates a new sandbox.
+    this.agent.explicitSandboxId = !!(
+      connectOptions.sandboxId || this.options?.sandboxId
+    );
     if (connectOptions.sandboxId) {
       this.agent.sandboxId = connectOptions.sandboxId;
     } else if (this.sandboxId) {
@@ -2797,7 +2804,8 @@ CAPTCHA_SOLVER_EOF`,
       
       switch (methodName) {
         case "type":
-          // For type, use the text being typed (truncated)
+          // For type, use the text being typed (truncated), masked if secret
+          if (args[1] && typeof args[1] === "object" && args[1].secret) return "****";
           return typeof firstArg === "string" ? firstArg.substring(0, 20) : "";
         case "pressKeys":
           // For pressKeys, show the keys
@@ -3047,7 +3055,7 @@ CAPTCHA_SOLVER_EOF`,
    * @example
    * // Find clickable elements
    * const result = await testdriver.parse();
-   * const clickable = result.elements.filter(e => e.interactivity === 'clickable');
+   * const clickable = result.elements.filter(e => e.interactivity);
    *
    * @example
    * // Find text content