@checksum-ai/runtime 4.18.0-beta.1 → 4.18.0-beta.2

package/package.json

@@ -3,7 +3,7 @@
   "description": "Checksum.ai test runtime",
   "main": "index.js",
   "dependencies": {
-    "@playwright/test": "1.63.0",
+    "@playwright/test": "1.62.1",
     "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.0-beta.1"
+  "version": "4.18.0-beta.2"
 }

package/scripts/patch.js

@@ -6,19 +6,14 @@ function getPlaywrightPatchesDirectory() {
   return join(__dirname, "playwright_patches");
 }
 
-function availablePatchVersions() {
-  return fs
-    .readdirSync(getPlaywrightPatchesDirectory())
-    .map((patchFile) => path.basename(patchFile, ".js"))
-    .sort();
-}
+const availablePatchVersions = fs
+  .readdirSync(getPlaywrightPatchesDirectory())
+  .map((patchFile) => path.basename(patchFile, ".js"))
+  .sort();
 
-function findSuitablePatchVersion(
-  version,
-  available = availablePatchVersions()
-) {
-  let suitablePatchVersion = available[0];
-  for (const patchVersion of available) {
+function findSuitablePatchVersion(version) {
+  let suitablePatchVersion = availablePatchVersions[0];
+  for (const patchVersion of availablePatchVersions) {
     if (version >= patchVersion) {
       suitablePatchVersion = patchVersion;
     } else {
@@ -28,38 +23,26 @@ function findSuitablePatchVersion(
   return suitablePatchVersion;
 }
 
-function patchAll() {
-  const projectRootFromEnv = process.env.PROJECT_ROOT;
-  const projectPaths = projectRootFromEnv
-    ? [projectRootFromEnv]
-    : [".", "../", "../.."].map((project) => join(process.cwd(), project));
+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);
-      }
-    } catch (e) {
-      // Patching is best-effort: a failure here must not block the install.
+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.
   }
 }
-
-// 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,39 +525,6 @@ 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";
@@ -582,7 +549,6 @@ function run(projectPath) {
           reportTraceFile(projectPath);
         }
         htmlReporter(projectPath);
-        boundUnboundedFixtures(projectPath);
       }
     } else {
       console.warn("Project path not found", projectPath);