Fix update script (#289)

Author: fisker

.github/workflows/update.yml

@@ -39,6 +39,8 @@ jobs:
           check-latest: true
           node-version: latest
       - run: npm install
+      # https://github.com/puppeteer/puppeteer/issues/13853
+      - run: npx patch-package
       - uses: actions/download-artifact@v4
         with:
           name: data

package.json

@@ -54,6 +54,7 @@
 		"get-port": "^7.1.0",
 		"npm-run-all2": "^8.0.1",
 		"outdent": "^0.8.0",
+		"patch-package": "^8.0.0",
 		"puppeteer": "^24.8.1",
 		"shelljs": "^0.9.2",
 		"tsd": "^0.32.0",

patches/@puppeteer+browsers+2.10.3.patch

@@ -0,0 +1,13 @@
+diff --git a/node_modules/@puppeteer/browsers/lib/esm/install.js b/node_modules/@puppeteer/browsers/lib/esm/install.js
+index cf6db62..b2df977 100644
+--- a/node_modules/@puppeteer/browsers/lib/esm/install.js
++++ b/node_modules/@puppeteer/browsers/lib/esm/install.js
+@@ -128,7 +128,7 @@ async function installUrl(url, options) {
+     if (downloadProgressCallback === 'default') {
+         downloadProgressCallback = await makeProgressCallback(options.browser, options.buildIdAlias ?? options.buildId);
+     }
+-    const fileName = url.toString().split('/').pop();
++    const fileName = decodeURIComponent(url.toString().split('/').pop());
+     assert(fileName, `A malformed download URL was found: ${url}.`);
+     const cache = new Cache(options.cacheDir);
+     const browserRoot = cache.browserRoot(options.browser);

scripts/get-browser-globals.mjs

@@ -1,5 +1,6 @@
 import process from 'node:process';
 import http from 'node:http';
+import os from 'node:os';
 import assert from 'node:assert/strict';
 import puppeteer from 'puppeteer';
 import getPort from 'get-port';
@@ -235,10 +236,13 @@ async function runInWebWorker(function_) {
 			page.evaluate(() => new Worker('data:application/javascript,;'));
 		});
 
-		assert.ok(
-			await worker.evaluate(() => globalThis.isSecureContext),
-			'Expected a secure worker.',
-		);
+		// Unknown reason, it returns `undefined` instead of `true` on MacOS
+		if (os.platform() !== 'darwin') {
+			assert.ok(
+				await worker.evaluate(() => globalThis.isSecureContext),
+				'Expected a secure worker.',
+			);
+		}
 
 		return await worker.evaluate(function_);
 	} finally {