@checksum-ai/runtime 4.18.1 → 5.0.0-beta.1

package/package.json

@@ -3,7 +3,7 @@
   "description": "Checksum.ai test runtime",
   "main": "index.js",
   "dependencies": {
-    "@playwright/test": "1.62.1",
+    "@playwright/test": "1.63.0",
     "dotenv": "17.4.2",
     "jsdom": "30.0.1",
     "playwright-extra": "4.3.6",
@@ -45,5 +45,5 @@
     "url": "https://github.com/checksum-ai/checksum-ai-monorepo/issues"
   },
   "homepage": "https://github.com/checksum-ai/checksum-ai-monorepo/tree/main/packages/runtime#readme",
-  "version": "4.18.1"
+  "version": "5.0.0-beta.1"
 }

package/scripts/patch.js

@@ -6,14 +6,19 @@ function getPlaywrightPatchesDirectory() {
   return join(__dirname, "playwright_patches");
 }
 
-const availablePatchVersions = fs
-  .readdirSync(getPlaywrightPatchesDirectory())
-  .map((patchFile) => path.basename(patchFile, ".js"))
-  .sort();
+function availablePatchVersions() {
+  return fs
+    .readdirSync(getPlaywrightPatchesDirectory())
+    .map((patchFile) => path.basename(patchFile, ".js"))
+    .sort();
+}
 
-function findSuitablePatchVersion(version) {
-  let suitablePatchVersion = availablePatchVersions[0];
-  for (const patchVersion of availablePatchVersions) {
+function findSuitablePatchVersion(
+  version,
+  available = availablePatchVersions()
+) {
+  let suitablePatchVersion = available[0];
+  for (const patchVersion of available) {
     if (version >= patchVersion) {
       suitablePatchVersion = patchVersion;
     } else {
@@ -23,26 +28,38 @@ function findSuitablePatchVersion(version) {
   return suitablePatchVersion;
 }
 
-const projectRootFromEnv = process.env.PROJECT_ROOT;
-const projectPaths = projectRootFromEnv
-  ? [projectRootFromEnv]
-  : [".", "../", "../.."].map((project) => join(process.cwd(), project));
+function patchAll() {
+  const projectRootFromEnv = process.env.PROJECT_ROOT;
+  const projectPaths = projectRootFromEnv
+    ? [projectRootFromEnv]
+    : [".", "../", "../.."].map((project) => join(process.cwd(), project));
 
-for (const projectPath of projectPaths) {
-  try {
-    if (fs.existsSync(join(projectPath, "node_modules", "playwright"))) {
-      const absolutePathToPackageJSON = resolve(
-        join(projectPath, "node_modules", "playwright", "package.json")
-      );
-      const packageJSON = require(absolutePathToPackageJSON);
-      const playwrightVersion = packageJSON.version;
-      const patchVersion = findSuitablePatchVersion(playwrightVersion);
-      const patchPath = resolve(
-        join(getPlaywrightPatchesDirectory(), `${patchVersion}.js`)
-      );
-      require(patchPath)(projectPath);
+  for (const projectPath of projectPaths) {
+    try {
+      if (fs.existsSync(join(projectPath, "node_modules", "playwright"))) {
+        const absolutePathToPackageJSON = resolve(
+          join(projectPath, "node_modules", "playwright", "package.json")
+        );
+        const packageJSON = require(absolutePathToPackageJSON);
+        const playwrightVersion = packageJSON.version;
+        const patchVersion = findSuitablePatchVersion(playwrightVersion);
+        const patchPath = resolve(
+          join(getPlaywrightPatchesDirectory(), `${patchVersion}.js`)
+        );
+        require(patchPath)(projectPath);
+      }
+    } catch (e) {
+      // Patching is best-effort: a failure here must not block the install.
     }
-  } catch (e) {
-    // Patching is best-effort: a failure here must not block the install.
   }
 }
+
+// Guarded like playwright_pins.js: the runtime always spawns this as
+// `node patch.js`, so requiring it must not patch anything as a side effect —
+// that is what lets playwright_patches.spec.ts test the real selection logic
+// instead of keeping a copy of it in sync by hand.
+if (require.main === module) {
+  patchAll();
+}
+
+module.exports = { availablePatchVersions, findSuitablePatchVersion };

package/scripts/playwright_patches/1.62.1.js

@@ -525,6 +525,39 @@ function htmlReporter(projectRoot) {
   replaceContent(file, originalContent, newContent);
 }
 
+// -------- [Shutdown safety] -------- //
+
+// Playwright leaves two teardown fixtures at `timeout: 0`, and TimeoutManager
+// reads that as `deadline = kMaxDeadline` — unbounded, not "use the default".
+// Every other phase is capped at the project timeout, so these are the only two
+// ways a worker can stop reporting for good: an unreaped browser leaves
+// `browser.close()` awaiting a `close` event that never arrives, which is how
+// run cc44a300 went silent for 10h42m until the k8s job deadline killed the pod.
+// Playwright's own 5-minute worker watchdog cannot catch it — heartbeats run on
+// an interval independent of any pending await, so a wedged worker keeps
+// heartbeating and the deadline slides forever.
+//
+// `> 0 ? : ` rather than `||`: a zero, negative or unparseable override must
+// fall back to the default, never restore the unbounded wait.
+function boundUnboundedFixtures(projectRoot) {
+  const file = join(projectRoot, "node_modules/playwright/lib/index.js");
+  if (!doesFileExist(file)) {
+    return;
+  }
+
+  let originalContent, newContent;
+
+  // ArtifactsRecorder.didFinishTest(): screenshot, trace stop, aria snapshot.
+  originalContent = `}, { auto: "all-hooks-included", title: "trace recording", box: true, timeout: 0 }],`;
+  newContent = `}, { auto: "all-hooks-included", title: "trace recording", box: true, timeout: Number(process.env["CHECKSUM_PW_ARTIFACTS_TIMEOUT_MS"]) > 0 ? Number(process.env["CHECKSUM_PW_ARTIFACTS_TIMEOUT_MS"]) : 120000 }],`;
+  replaceContent(file, originalContent, newContent);
+
+  // The `browser` fixture: launch on setup, `browser.close()` on teardown.
+  originalContent = `}, { scope: "worker", timeout: 0 }],`;
+  newContent = `}, { scope: "worker", timeout: Number(process.env["CHECKSUM_PW_BROWSER_FIXTURE_TIMEOUT_MS"]) > 0 ? Number(process.env["CHECKSUM_PW_BROWSER_FIXTURE_TIMEOUT_MS"]) : 180000 }],`;
+  replaceContent(file, originalContent, newContent);
+}
+
 // -------- [Run] -------- //
 
 const isRuntime = true || process.env.RUNTIME === "true";
@@ -549,6 +582,7 @@ function run(projectPath) {
           reportTraceFile(projectPath);
         }
         htmlReporter(projectPath);
+        boundUnboundedFixtures(projectPath);
       }
     } else {
       console.warn("Project path not found", projectPath);