@browserless/lighthouse 13.1.7 → 13.1.8

package/package.json

@@ -2,7 +2,7 @@
   "name": "@browserless/lighthouse",
   "description": "Run Google Lighthouse audits on websites using headless Chrome. Get performance, accessibility, and SEO metrics.",
   "homepage": "https://browserless.js.org",
-  "version": "13.1.7",
+  "version": "13.1.8",
   "main": "src",
   "author": {
     "email": "hello@microlink.io",
@@ -32,10 +32,11 @@
     "web-vitals"
   ],
   "dependencies": {
+    "@browserless/errors": "13.1.7",
     "lighthouse": "~13.4.0"
   },
   "devDependencies": {
-    "@browserless/test": "13.1.7",
+    "@browserless/test": "13.1.8",
     "ava": "5"
   },
   "engines": {
@@ -53,5 +54,5 @@
     "timeout": "2m",
     "workerThreads": false
   },
-  "gitHead": "cb9a0c6fc11176da209ba9bc33bca5a47c3aabbe"
+  "gitHead": "c47bf305151f14340462c59ae0f47aadeb993e16"
 }

package/src/lighthouse.js

@@ -1,8 +1,26 @@
 'use strict'
 
+const errors = require('@browserless/errors')
 const lighthouse = require('lighthouse/core/index.cjs')
 
+// Lighthouse instruments each phase with performance marks (via marky). When
+// the browser context is torn down mid-run, lighthouse's teardown calls
+// `performance.measure` for a phase whose start mark never fired, throwing a
+// `SyntaxError: The "start lh:…" performance mark has not been set`. That
+// message masks the real cause (a destroyed context), so it would otherwise
+// escape the retry machinery as an opaque error. Map it back to a
+// context-disconnected error to let `withPage` recreate the context and retry.
+const isPerfMarkError = error =>
+  error?.message?.endsWith('performance mark has not been set') ?? false
+
 module.exports = async ({ url, config, flags, page }) => {
-  const { lhr, report } = await lighthouse(url, flags, config, page)
-  return config.settings.output === 'json' ? lhr : report
+  try {
+    const { lhr, report } = await lighthouse(url, flags, config, page)
+    return config.settings.output === 'json' ? lhr : report
+  } catch (error) {
+    if (isPerfMarkError(error)) throw errors.contextDisconnected()
+    throw error
+  }
 }
+
+module.exports.isPerfMarkError = isPerfMarkError