@opice/harness 0.2.1 → 0.2.2

package/dist/context.d.ts

@@ -3,8 +3,18 @@ import { type BrowserContext, type Page } from 'playwright';
 export declare function getPage(): Page;
 /** The active browser context (for cookies/storage, new tabs, etc.). */
 export declare function getContext(): BrowserContext;
-/** Launch a fresh isolated browser + context + page. Called from `beforeAll`. */
+/**
+ * Open a fresh isolated context + page for a scenario, reusing the shared
+ * browser. Called from `beforeAll`. Any context left over from a previous
+ * scenario whose teardown didn't complete is closed first so state never
+ * bleeds across scenarios.
+ */
 export declare function launchPage(): Promise<Page>;
-/** Close the page, context, and browser. Called from `afterAll`. */
+/**
+ * Close the scenario's context (and page); keep the shared browser alive for
+ * the next scenario. Called from `afterAll`. The browser itself is launched
+ * once and reaped by Playwright's own process-exit handler when `bun test`
+ * exits — see the `beforeExit` hook below for the graceful path.
+ */
 export declare function closePage(): Promise<void>;
 //# sourceMappingURL=context.d.ts.map

package/dist/context.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"context.d.ts","sourceRoot":"","sources":["../src/context.ts"],"names":[],"mappings":"AAAA,OAAO,EAA0B,KAAK,cAAc,EAAE,KAAK,IAAI,EAAE,MAAM,YAAY,CAAA;AAoBnF,4EAA4E;AAC5E,wBAAgB,OAAO,IAAI,IAAI,CAK9B;AAED,wEAAwE;AACxE,wBAAgB,UAAU,IAAI,cAAc,CAK3C;AAED,iFAAiF;AACjF,wBAAsB,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC,CAKhD;AAED,oEAAoE;AACpE,wBAAsB,SAAS,IAAI,OAAO,CAAC,IAAI,CAAC,CAS/C"}
+{"version":3,"file":"context.d.ts","sourceRoot":"","sources":["../src/context.ts"],"names":[],"mappings":"AAAA,OAAO,EAA0B,KAAK,cAAc,EAAE,KAAK,IAAI,EAAE,MAAM,YAAY,CAAA;AA2BnF,4EAA4E;AAC5E,wBAAgB,OAAO,IAAI,IAAI,CAK9B;AAED,wEAAwE;AACxE,wBAAgB,UAAU,IAAI,cAAc,CAK3C;AAUD;;;;;GAKG;AACH,wBAAsB,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC,CAUhD;AAED;;;;;GAKG;AACH,wBAAsB,SAAS,IAAI,OAAO,CAAC,IAAI,CAAC,CAO/C"}

package/dist/context.js

@@ -1,11 +1,18 @@
 import { chromium } from 'playwright';
 /**
- * The live Playwright page for the running scenario. `browserTest` launches a
- * fresh browser + context + page per scenario (`beforeAll`) and tears it down
- * (`afterAll`); the DSL — `el`, `byRole`, navigation — reads the current page
- * from here. This module replaces the old agent-browser CLI session handling:
- * there is no shell-out and no daemon, the browser runs in-process under
- * `bun test`.
+ * The live Playwright page for the running scenario.
+ *
+ * The browser process is launched **once** and reused across every scenario;
+ * each `browserTest` only opens a fresh isolated `context` + `page` in
+ * `beforeAll` and closes that context in `afterAll`. Launching (and tearing
+ * down) a whole chromium per scenario is expensive — on a constrained CI runner
+ * that per-scenario launch competes with the app/server for CPU and, when a
+ * teardown stalls, leaks a zombie browser that drags the rest of the suite
+ * down. A fresh context per scenario keeps the same isolation (separate
+ * storage/cookies) at a fraction of the cost.
+ *
+ * The DSL — `el`, `byRole`, navigation — reads the current page from here. The
+ * browser runs in-process under `bun test`; there is no shell-out and no daemon.
  */
 let browser = null;
 let context = null;
@@ -28,23 +35,51 @@ export function getContext() {
     }
     return context;
 }
-/** Launch a fresh isolated browser + context + page. Called from `beforeAll`. */
+/** Launch the shared browser once; reuse it on subsequent scenarios. */
+async function getBrowser() {
+    if (!browser || !browser.isConnected()) {
+        browser = await chromium.launch({ headless: !headed() });
+    }
+    return browser;
+}
+/**
+ * Open a fresh isolated context + page for a scenario, reusing the shared
+ * browser. Called from `beforeAll`. Any context left over from a previous
+ * scenario whose teardown didn't complete is closed first so state never
+ * bleeds across scenarios.
+ */
 export async function launchPage() {
-    browser = await chromium.launch({ headless: !headed() });
-    context = await browser.newContext();
+    if (context) {
+        await context.close().catch(() => { });
+        context = null;
+        page = null;
+    }
+    const b = await getBrowser();
+    context = await b.newContext();
     page = await context.newPage();
     return page;
 }
-/** Close the page, context, and browser. Called from `afterAll`. */
+/**
+ * Close the scenario's context (and page); keep the shared browser alive for
+ * the next scenario. Called from `afterAll`. The browser itself is launched
+ * once and reaped by Playwright's own process-exit handler when `bun test`
+ * exits — see the `beforeExit` hook below for the graceful path.
+ */
 export async function closePage() {
     try {
         await context?.close();
     }
     finally {
-        await browser?.close();
         page = null;
         context = null;
-        browser = null;
     }
 }
+// Graceful shutdown of the shared browser when the test process winds down. If
+// this doesn't fire (hard exit/signal), Playwright's own exit handler still
+// kills the chromium child, so the process never outlives the run.
+process.once('beforeExit', () => {
+    const b = browser;
+    browser = null;
+    void b?.close().catch(() => { });
+});
 //# sourceMappingURL=context.js.map

package/dist/context.js.map

@@ -1 +1 @@
-{"version":3,"file":"context.js","sourceRoot":"","sources":["../src/context.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAgD,MAAM,YAAY,CAAA;AAEnF;;;;;;;GAOG;AAEH,IAAI,OAAO,GAAmB,IAAI,CAAA;AAClC,IAAI,OAAO,GAA0B,IAAI,CAAA;AACzC,IAAI,IAAI,GAAgB,IAAI,CAAA;AAE5B,oFAAoF;AACpF,SAAS,MAAM;IACd,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC,IAAI,OAAO,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,CAAA;AACjE,CAAC;AAED,4EAA4E;AAC5E,MAAM,UAAU,OAAO;IACtB,IAAI,CAAC,IAAI,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CAAC,yEAAyE,CAAC,CAAA;IAC3F,CAAC;IACD,OAAO,IAAI,CAAA;AACZ,CAAC;AAED,wEAAwE;AACxE,MAAM,UAAU,UAAU;IACzB,IAAI,CAAC,OAAO,EAAE,CAAC;QACd,MAAM,IAAI,KAAK,CAAC,wEAAwE,CAAC,CAAA;IAC1F,CAAC;IACD,OAAO,OAAO,CAAA;AACf,CAAC;AAED,iFAAiF;AACjF,MAAM,CAAC,KAAK,UAAU,UAAU;IAC/B,OAAO,GAAG,MAAM,QAAQ,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,MAAM,EAAE,EAAE,CAAC,CAAA;IACxD,OAAO,GAAG,MAAM,OAAO,CAAC,UAAU,EAAE,CAAA;IACpC,IAAI,GAAG,MAAM,OAAO,CAAC,OAAO,EAAE,CAAA;IAC9B,OAAO,IAAI,CAAA;AACZ,CAAC;AAED,oEAAoE;AACpE,MAAM,CAAC,KAAK,UAAU,SAAS;IAC9B,IAAI,CAAC;QACJ,MAAM,OAAO,EAAE,KAAK,EAAE,CAAA;IACvB,CAAC;YAAS,CAAC;QACV,MAAM,OAAO,EAAE,KAAK,EAAE,CAAA;QACtB,IAAI,GAAG,IAAI,CAAA;QACX,OAAO,GAAG,IAAI,CAAA;QACd,OAAO,GAAG,IAAI,CAAA;IACf,CAAC;AACF,CAAC"}
+{"version":3,"file":"context.js","sourceRoot":"","sources":["../src/context.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAgD,MAAM,YAAY,CAAA;AAEnF;;;;;;;;;;;;;;GAcG;AAEH,IAAI,OAAO,GAAmB,IAAI,CAAA;AAClC,IAAI,OAAO,GAA0B,IAAI,CAAA;AACzC,IAAI,IAAI,GAAgB,IAAI,CAAA;AAE5B,oFAAoF;AACpF,SAAS,MAAM;IACd,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC,IAAI,OAAO,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,CAAA;AACjE,CAAC;AAED,4EAA4E;AAC5E,MAAM,UAAU,OAAO;IACtB,IAAI,CAAC,IAAI,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CAAC,yEAAyE,CAAC,CAAA;IAC3F,CAAC;IACD,OAAO,IAAI,CAAA;AACZ,CAAC;AAED,wEAAwE;AACxE,MAAM,UAAU,UAAU;IACzB,IAAI,CAAC,OAAO,EAAE,CAAC;QACd,MAAM,IAAI,KAAK,CAAC,wEAAwE,CAAC,CAAA;IAC1F,CAAC;IACD,OAAO,OAAO,CAAA;AACf,CAAC;AAED,wEAAwE;AACxE,KAAK,UAAU,UAAU;IACxB,IAAI,CAAC,OAAO,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,EAAE,CAAC;QACxC,OAAO,GAAG,MAAM,QAAQ,CAAC,MAAM,CAAC,EAAE,QAAQ,EAAE,CAAC,MAAM,EAAE,EAAE,CAAC,CAAA;IACzD,CAAC;IACD,OAAO,OAAO,CAAA;AACf,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,UAAU;IAC/B,IAAI,OAAO,EAAE,CAAC;QACb,MAAM,OAAO,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAA;QACrC,OAAO,GAAG,IAAI,CAAA;QACd,IAAI,GAAG,IAAI,CAAA;IACZ,CAAC;IACD,MAAM,CAAC,GAAG,MAAM,UAAU,EAAE,CAAA;IAC5B,OAAO,GAAG,MAAM,CAAC,CAAC,UAAU,EAAE,CAAA;IAC9B,IAAI,GAAG,MAAM,OAAO,CAAC,OAAO,EAAE,CAAA;IAC9B,OAAO,IAAI,CAAA;AACZ,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS;IAC9B,IAAI,CAAC;QACJ,MAAM,OAAO,EAAE,KAAK,EAAE,CAAA;IACvB,CAAC;YAAS,CAAC;QACV,IAAI,GAAG,IAAI,CAAA;QACX,OAAO,GAAG,IAAI,CAAA;IACf,CAAC;AACF,CAAC;AAED,+EAA+E;AAC/E,4EAA4E;AAC5E,mEAAmE;AACnE,OAAO,CAAC,IAAI,CAAC,YAAY,EAAE,GAAG,EAAE;IAC/B,MAAM,CAAC,GAAG,OAAO,CAAA;IACjB,OAAO,GAAG,IAAI,CAAA;IACd,KAAK,CAAC,EAAE,KAAK,EAAE,CAAC,KAAK,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAA;AAChC,CAAC,CAAC,CAAA"}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@opice/harness",
-	"version": "0.2.1",
+	"version": "0.2.2",
 	"description": "Runtime primitives for opice — AI-driven E2E browser tests on top of Playwright",
 	"type": "module",
 	"main": "./src/index.ts",

package/src/context.ts

@@ -1,12 +1,19 @@
 import { chromium, type Browser, type BrowserContext, type Page } from 'playwright'
 
 /**
- * The live Playwright page for the running scenario. `browserTest` launches a
- * fresh browser + context + page per scenario (`beforeAll`) and tears it down
- * (`afterAll`); the DSL — `el`, `byRole`, navigation — reads the current page
- * from here. This module replaces the old agent-browser CLI session handling:
- * there is no shell-out and no daemon, the browser runs in-process under
- * `bun test`.
+ * The live Playwright page for the running scenario.
+ *
+ * The browser process is launched **once** and reused across every scenario;
+ * each `browserTest` only opens a fresh isolated `context` + `page` in
+ * `beforeAll` and closes that context in `afterAll`. Launching (and tearing
+ * down) a whole chromium per scenario is expensive — on a constrained CI runner
+ * that per-scenario launch competes with the app/server for CPU and, when a
+ * teardown stalls, leaks a zombie browser that drags the rest of the suite
+ * down. A fresh context per scenario keeps the same isolation (separate
+ * storage/cookies) at a fraction of the cost.
+ *
+ * The DSL — `el`, `byRole`, navigation — reads the current page from here. The
+ * browser runs in-process under `bun test`; there is no shell-out and no daemon.
  */
 
 let browser: Browser | null = null
@@ -34,22 +41,52 @@ export function getContext(): BrowserContext {
 	return context
 }
 
-/** Launch a fresh isolated browser + context + page. Called from `beforeAll`. */
+/** Launch the shared browser once; reuse it on subsequent scenarios. */
+async function getBrowser(): Promise<Browser> {
+	if (!browser || !browser.isConnected()) {
+		browser = await chromium.launch({ headless: !headed() })
+	}
+	return browser
+}
+
+/**
+ * Open a fresh isolated context + page for a scenario, reusing the shared
+ * browser. Called from `beforeAll`. Any context left over from a previous
+ * scenario whose teardown didn't complete is closed first so state never
+ * bleeds across scenarios.
+ */
 export async function launchPage(): Promise<Page> {
-	browser = await chromium.launch({ headless: !headed() })
-	context = await browser.newContext()
+	if (context) {
+		await context.close().catch(() => {})
+		context = null
+		page = null
+	}
+	const b = await getBrowser()
+	context = await b.newContext()
 	page = await context.newPage()
 	return page
 }
 
-/** Close the page, context, and browser. Called from `afterAll`. */
+/**
+ * Close the scenario's context (and page); keep the shared browser alive for
+ * the next scenario. Called from `afterAll`. The browser itself is launched
+ * once and reaped by Playwright's own process-exit handler when `bun test`
+ * exits — see the `beforeExit` hook below for the graceful path.
+ */
 export async function closePage(): Promise<void> {
 	try {
 		await context?.close()
 	} finally {
-		await browser?.close()
 		page = null
 		context = null
-		browser = null
 	}
 }
+
+// Graceful shutdown of the shared browser when the test process winds down. If
+// this doesn't fire (hard exit/signal), Playwright's own exit handler still
+// kills the chromium child, so the process never outlives the run.
+process.once('beforeExit', () => {
+	const b = browser
+	browser = null
+	void b?.close().catch(() => {})
+})