chore: consolidate test directories in packages/core (#1621)

.changeset/giant-sloths-invent.md

@@ -0,0 +1,5 @@
+---
+"@browserbasehq/stagehand": patch
+---
+
+chore: move Vitest unit tests to packages/core/tests

packages/core/package.json

@@ -12,10 +12,10 @@
     "typecheck": "tsc --noEmit",
     "build": "pnpm run gen-version && pnpm run build-dom-scripts && pnpm run build-js && pnpm run typecheck",
     "example": "node --import tsx -e \"const args=process.argv.slice(1).filter(a=>a!=='--'); const [p]=args; const n=(p||'example').replace(/^\\.\\//,'').replace(/\\.ts$/i,''); import(new URL(require('node:path').resolve('examples', n + '.ts'), 'file:'));\" --",
-    "test": "playwright test --config=lib/v3/tests/v3.playwright.config.ts",
-    "e2e": "playwright test --config=lib/v3/tests/v3.local.playwright.config.ts",
-    "e2e:local": "playwright test --config=lib/v3/tests/v3.local.playwright.config.ts",
-    "e2e:bb": "playwright test --config=lib/v3/tests/v3.bb.playwright.config.ts",
+    "test": "playwright test --config=tests/integration/v3.playwright.config.ts",
+    "e2e": "playwright test --config=tests/integration/v3.local.playwright.config.ts",
+    "e2e:local": "playwright test --config=tests/integration/v3.local.playwright.config.ts",
+    "e2e:bb": "playwright test --config=tests/integration/v3.bb.playwright.config.ts",
     "lint": "cd ../.. && prettier --check packages/core && cd packages/core && eslint .",
     "format": "prettier --write .",
     "test:vitest": "turbo run build && vitest run --config vitest.config.ts"

packages/core/lib/v3/tests/agent-abort-signal.spec.ts → packages/core/tests/integration/agent-abort-signal.spec.ts

@@ -1,7 +1,7 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
-import { AgentAbortError } from "../types/public/sdkErrors";
+import { AgentAbortError } from "../../lib/v3/types/public/sdkErrors";
 
 test.describe("Stagehand agent abort signal", () => {
   let v3: V3;

packages/core/lib/v3/tests/agent-cache-self-heal.spec.ts → packages/core/tests/integration/agent-cache-self-heal.spec.ts

@@ -1,13 +1,13 @@
 import { test, expect } from "@playwright/test";
 import fs from "fs/promises";
 import path from "path";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 import type {
   AgentReplayActStep,
   AgentReplayFillFormStep,
   CachedAgentEntry,
-} from "../types/private/cache";
+} from "../../lib/v3/types/private/cache";
 
 test.describe("Agent cache self-heal (e2e)", () => {
   let v3: V3;

packages/core/lib/v3/tests/agent-callbacks.spec.ts → packages/core/tests/integration/agent-callbacks.spec.ts

@@ -1,8 +1,8 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 import type { StepResult, ToolSet } from "ai";
-import { StreamingCallbacksInNonStreamingModeError } from "../types/public/sdkErrors";
+import { StreamingCallbacksInNonStreamingModeError } from "../../lib/v3/types/public/sdkErrors";
 
 test.describe("Stagehand agent callbacks behavior", () => {
   let v3: V3;

packages/core/lib/v3/tests/agent-experimental-validation.spec.ts → packages/core/tests/integration/agent-experimental-validation.spec.ts

@@ -1,12 +1,12 @@
 import { test, expect } from "@playwright/test";
 import { z } from "zod";
 import { tool } from "ai";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 import {
   ExperimentalNotConfiguredError,
   StagehandInvalidArgumentError,
-} from "../types/public/sdkErrors";
+} from "../../lib/v3/types/public/sdkErrors";
 
 // Define a mock custom tool for testing
 const mockCustomTool = tool({

packages/core/lib/v3/tests/agent-hybrid-mode.spec.ts → packages/core/tests/integration/agent-hybrid-mode.spec.ts

@@ -1,8 +1,8 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
-import { createAgentTools } from "../agent/tools";
-import { buildAgentSystemPrompt } from "../agent/prompts/agentSystemPrompt";
+import { createAgentTools } from "../../lib/v3/agent/tools";
+import { buildAgentSystemPrompt } from "../../lib/v3/agent/prompts/agentSystemPrompt";
 import type { StepResult, ToolSet } from "ai";
 
 test.describe("Stagehand agent hybrid mode", () => {

packages/core/lib/v3/tests/agent-message-continuation.spec.ts → packages/core/tests/integration/agent-message-continuation.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 import type { ModelMessage } from "ai";
 

packages/core/lib/v3/tests/agent-streaming.spec.ts → packages/core/tests/integration/agent-streaming.spec.ts

@@ -1,7 +1,7 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
-import type { AgentResult } from "../types/public/agent";
+import type { AgentResult } from "../../lib/v3/types/public/agent";
 
 test.describe("Stagehand agent streaming behavior", () => {
   let v3: V3;

packages/core/lib/v3/tests/click-count.spec.ts → packages/core/tests/integration/click-count.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Locator and Page click methods", () => {

packages/core/lib/v3/tests/connect-to-existing-browser.spec.ts → packages/core/tests/integration/connect-to-existing-browser.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3BBTestConfig } from "./v3.bb.config";
 
 const PAGE_TARGET_COUNT = 5;

packages/core/lib/v3/tests/context-addInitScript.spec.ts → packages/core/tests/integration/context-addInitScript.spec.ts

@@ -1,7 +1,7 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
-import { V3Context } from "../understudy/context";
+import { V3Context } from "../../lib/v3/understudy/context";
 
 const toDataUrl = (html: string): string =>
   `data:text/html,${encodeURIComponent(html)}`;

packages/core/lib/v3/tests/default-page-tracking.spec.ts → packages/core/tests/integration/default-page-tracking.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3DynamicTestConfig } from "./v3.dynamic.config";
 
 test.describe.configure({ mode: "parallel" });

packages/core/lib/v3/tests/downloads.spec.ts → packages/core/tests/integration/downloads.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import Browserbase from "@browserbasehq/sdk";
 import AdmZip from "adm-zip";
 import { v3DynamicTestConfig } from "./v3.dynamic.config";

packages/core/lib/v3/tests/frame-get-location-and-click.spec.ts → packages/core/tests/integration/frame-get-location-and-click.spec.ts

@@ -1,5 +1,5 @@
 import { expect, test } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Coordinate-based clicking", () => {

packages/core/lib/v3/tests/keyboard.spec.ts → packages/core/tests/integration/keyboard.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3DynamicTestConfig } from "./v3.dynamic.config";
 
 function dataUrl(html: string): string {

packages/core/lib/v3/tests/locator-backend-node-id.spec.ts → packages/core/tests/integration/locator-backend-node-id.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Locator.backendNodeId() - CDP DOM node ID", () => {

packages/core/lib/v3/tests/locator-content-methods.spec.ts → packages/core/tests/integration/locator-content-methods.spec.ts

@@ -1,5 +1,5 @@
 import { expect, test } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Locator content methods (textContent, innerHtml, innerText, inputValue)", () => {

packages/core/lib/v3/tests/locator-count-iframe.spec.ts → packages/core/tests/integration/locator-count-iframe.spec.ts

@@ -1,5 +1,5 @@
 import { expect, test } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3DynamicTestConfig } from "./v3.dynamic.config";
 
 test.describe("Locator count() method with iframes", () => {

packages/core/lib/v3/tests/locator-count.spec.ts → packages/core/tests/integration/locator-count.spec.ts

@@ -1,5 +1,5 @@
 import { expect, test } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3DynamicTestConfig } from "./v3.dynamic.config";
 
 test.describe("Locator count() method tests", () => {

packages/core/lib/v3/tests/locator-input-methods.spec.ts → packages/core/tests/integration/locator-input-methods.spec.ts

@@ -1,5 +1,5 @@
 import { expect, test } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Locator input methods (fill, type, hover, isVisible, isChecked)", () => {

packages/core/lib/v3/tests/locator-nth.spec.ts → packages/core/tests/integration/locator-nth.spec.ts

@@ -1,5 +1,5 @@
 import { expect, test } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3DynamicTestConfig } from "./v3.dynamic.config";
 
 test.describe("Locator nth() method tests", () => {

packages/core/lib/v3/tests/locator-select-option.spec.ts → packages/core/tests/integration/locator-select-option.spec.ts

@@ -1,5 +1,5 @@
 import { expect, test } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Locator.selectOption() method", () => {

packages/core/lib/v3/tests/logger-initialization.spec.ts → packages/core/tests/integration/logger-initialization.spec.ts

@@ -4,8 +4,8 @@ import {
   unbindInstanceLogger,
   withInstanceLogContext,
   v3Logger,
-} from "../logger";
-import type { LogLine } from "../types/public/logs";
+} from "../../lib/v3/logger";
+import type { LogLine } from "../../lib/v3/types/public/logs";
 
 test.describe("V3 Logger Instance Routing", () => {
   test.afterEach(() => {

packages/core/lib/v3/tests/multi-instance-logger.spec.ts → packages/core/tests/integration/multi-instance-logger.spec.ts

@@ -1,7 +1,7 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { getV3DynamicTestConfig } from "./v3.dynamic.config";
-import type { LogLine } from "../types/public/logs";
+import type { LogLine } from "../../lib/v3/types/public/logs";
 
 test.describe("V3 Multi-Instance Logger Isolation", () => {
   // Run tests serially to avoid resource exhaustion from creating many Chrome instances

packages/core/lib/v3/tests/nested-div.spec.ts → packages/core/tests/integration/nested-div.spec.ts

@@ -1,6 +1,6 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
-import { captureHybridSnapshot } from "../understudy/a11y/snapshot";
+import { V3 } from "../../lib/v3";
+import { captureHybridSnapshot } from "../../lib/v3/understudy/a11y/snapshot";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("tests captureHybridSnapshot() does not break due to -32000 Failed to convert response to JSON: CBOR: stack limit exceeded", () => {

packages/core/lib/v3/tests/page-addInitScript.spec.ts → packages/core/tests/integration/page-addInitScript.spec.ts

@@ -1,7 +1,7 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
-import { V3Context } from "../understudy/context";
+import { V3Context } from "../../lib/v3/understudy/context";
 
 const EXAMPLE_URL = "https://example.com";
 

packages/core/lib/v3/tests/page-console.spec.ts → packages/core/tests/integration/page-console.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Page console events", () => {

packages/core/lib/v3/tests/page-drag-and-drop.spec.ts → packages/core/tests/integration/page-drag-and-drop.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Page.dragAndDrop() - dragging elements", () => {

packages/core/lib/v3/tests/page-goto-response.spec.ts → packages/core/tests/integration/page-goto-response.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Page.goto() response surface", () => {

packages/core/lib/v3/tests/page-hover.spec.ts → packages/core/tests/integration/page-hover.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Page.hover() - mouse hover at coordinates", () => {

packages/core/lib/v3/tests/page-screenshot.spec.ts → packages/core/tests/integration/page-screenshot.spec.ts

@@ -2,9 +2,9 @@ import { test, expect } from "@playwright/test";
 import { promises as fs } from "fs";
 import * as os from "os";
 import * as path from "path";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
-import { Frame } from "../understudy/frame";
+import { Frame } from "../../lib/v3/understudy/frame";
 
 const wait = (ms: number) => new Promise((resolve) => setTimeout(resolve, ms));
 

packages/core/lib/v3/tests/page-scroll.spec.ts → packages/core/tests/integration/page-scroll.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Page.scroll() - mouse wheel scrolling", () => {

packages/core/lib/v3/tests/page-send-cdp.spec.ts → packages/core/tests/integration/page-send-cdp.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 test.describe("Page sendCDP method", () => {

packages/core/lib/v3/tests/perform-understudy-method.spec.ts → packages/core/tests/integration/perform-understudy-method.spec.ts

@@ -1,7 +1,7 @@
 import { expect, test } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3DynamicTestConfig } from "./v3.dynamic.config";
-import { performUnderstudyMethod } from "../handlers/handlerUtils/actHandlerUtils";
+import { performUnderstudyMethod } from "../../lib/v3/handlers/handlerUtils/actHandlerUtils";
 
 test.describe("tests performUnderstudyMethod", () => {
   let v3: V3;

packages/core/lib/v3/tests/setinputfiles.spec.ts → packages/core/tests/integration/setinputfiles.spec.ts

@@ -3,8 +3,8 @@ import { Buffer } from "buffer";
 import { promises as fs } from "fs";
 import path from "path";
 import crypto from "crypto";
-import type { Page as V3Page } from "../understudy/page";
-import { V3 } from "../v3";
+import type { Page as V3Page } from "../../lib/v3/understudy/page";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 
 const FILE_UPLOAD_IFRAME_URL =

packages/core/lib/v3/tests/shadow-iframe.spec.ts → packages/core/tests/integration/shadow-iframe.spec.ts

@@ -1,10 +1,10 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import puppeteer from "puppeteer-core";
 import { chromium as playwrightChromium } from "playwright";
 import { chromium as patchrightChromium } from "patchright-core";
-import { Action } from "../types/public/methods";
-import { AnyPage } from "../types/public/page";
+import { Action } from "../../lib/v3/types/public/methods";
+import { AnyPage } from "../../lib/v3/types/public/page";
 import { v3DynamicTestConfig } from "./v3.dynamic.config";
 
 /**

packages/core/lib/v3/tests/text-selector-innermost.spec.ts → packages/core/tests/integration/text-selector-innermost.spec.ts

@@ -1,6 +1,6 @@
 import { expect, test } from "@playwright/test";
 import { Protocol } from "devtools-protocol";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3DynamicTestConfig } from "./v3.dynamic.config";
 
 test.describe("Text selector innermost element matching", () => {

packages/core/lib/v3/tests/timeouts.spec.ts → packages/core/tests/integration/timeouts.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3DynamicTestConfig } from "./v3.dynamic.config";
 import { z } from "zod";
 

packages/core/lib/v3/tests/user-data-dir.spec.ts → packages/core/tests/integration/user-data-dir.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from "@playwright/test";
-import { V3 } from "../v3";
+import { V3 } from "../../lib/v3";
 import { v3TestConfig } from "./v3.config";
 import * as fs from "fs";
 import * as path from "path";

packages/core/lib/v3/tests/v3.bb.config.ts → packages/core/tests/integration/v3.bb.config.ts

@@ -1,5 +1,5 @@
-import type { V3Options } from "../types/public/options";
-import type { LogLine } from "../types/public/logs";
+import type { V3Options } from "../../lib/v3/types/public/options";
+import type { LogLine } from "../../lib/v3/types/public/logs";
 import dotenv from "dotenv";
 import path from "path";
 

packages/core/lib/v3/tests/v3.bb.playwright.config.ts → packages/core/tests/integration/v3.bb.playwright.config.ts

@@ -5,8 +5,8 @@ import path from "path";
 // Load environment variables before setting TEST_ENV
 dotenv.config();
 
-// Try loading from repo root (packages/core/lib/v3/tests -> repo root = 5 levels up)
-const repoRootEnvPath = path.resolve(__dirname, "../../../../../.env");
+// Try loading from repo root (packages/core/tests/integration -> repo root = 4 levels up)
+const repoRootEnvPath = path.resolve(__dirname, "../../../../.env");
 dotenv.config({ path: repoRootEnvPath, override: false });
 
 // Set TEST_ENV before tests run

packages/core/lib/v3/tests/v3.config.ts → packages/core/tests/integration/v3.config.ts

@@ -1,5 +1,5 @@
-import type { V3Options } from "../types/public/options";
-import type { LogLine } from "../types/public/logs";
+import type { V3Options } from "../../lib/v3/types/public/options";
+import type { LogLine } from "../../lib/v3/types/public/logs";
 
 export const v3TestConfig: V3Options = {
   env: "LOCAL",

packages/core/lib/v3/tests/v3.dynamic.config.ts → packages/core/tests/integration/v3.dynamic.config.ts

@@ -1,5 +1,5 @@
-import type { V3Options } from "../types/public/options";
-import type { LogLine } from "../types/public/logs";
+import type { V3Options } from "../../lib/v3/types/public/options";
+import type { LogLine } from "../../lib/v3/types/public/logs";
 import dotenv from "dotenv";
 import path from "path";
 

packages/core/lib/v3/tests/v3.local.playwright.config.ts → packages/core/tests/integration/v3.local.playwright.config.ts

@@ -5,8 +5,8 @@ import path from "path";
 // Load environment variables before setting TEST_ENV
 dotenv.config();
 
-// Try loading from repo root (packages/core/lib/v3/tests -> repo root = 5 levels up)
-const repoRootEnvPath = path.resolve(__dirname, "../../../../../.env");
+// Try loading from repo root (packages/core/tests/integration -> repo root = 4 levels up)
+const repoRootEnvPath = path.resolve(__dirname, "../../../../.env");
 dotenv.config({ path: repoRootEnvPath, override: false });
 
 // Set TEST_ENV before tests run