Merge pull request #16293 from Snuffleupagus/Node-18

[api-minor] Update the minimum supported Node.js version to 18

external/.eslintrc

@@ -6,8 +6,4 @@
   "env": {
     "node": true,
   },
-
-  "rules": {
-    "unicorn/prefer-at": "off",
-  },
 }

external/builder/builder.js

@@ -72,7 +72,7 @@ function preprocess(inFilename, outFilename, defines) {
       ? outFilename
       : function (line) {
           if (!line || AllWhitespaceRegexp.test(line)) {
-            const prevLine = out[out.length - 1];
+            const prevLine = out.at(-1);
             if (!prevLine || AllWhitespaceRegexp.test(prevLine)) {
               return; // Avoid adding consecutive blank lines.
             }

external/builder/preprocessor2.js

@@ -244,8 +244,8 @@ function postprocessNode(ctx, node) {
       const block = node.body;
       if (
         block.body.length > 0 &&
-        block.body[block.body.length - 1].type === "ReturnStatement" &&
-        !block.body[block.body.length - 1].argument
+        block.body.at(-1).type === "ReturnStatement" &&
+        !block.body.at(-1).argument
       ) {
         // Function body ends with return without arg -- removing it.
         block.body.pop();

external/cmapscompress/compress.js

@@ -215,7 +215,7 @@ function parseCMap(binaryData) {
     },
     readHexSigned(size) {
       const num = this.readHexNumber(size);
-      const sign = fromHexDigit(num[num.length - 1]) & 1 ? 15 : 0;
+      const sign = fromHexDigit(num.at(-1)) & 1 ? 15 : 0;
       let c = 0;
       let result = "";
       for (const digit of num) {

external/cmapscompress/optimize.js

@@ -151,10 +151,7 @@ exports.optimizeCMap = function (data) {
       const maxDistance = 100,
         minItems = 10,
         itemsPerBucket = 50;
-      if (
-        subitems.length > minItems &&
-        codes[codes.length - 1] - codes[0] > maxDistance
-      ) {
+      if (subitems.length > minItems && codes.at(-1) - codes[0] > maxDistance) {
         const gapsCount = Math.max(2, (subitems.length / itemsPerBucket) | 0);
         const gaps = [];
         for (let q = 0; q < gapsCount; q++) {

gulpfile.js

@@ -80,7 +80,7 @@ const ENV_TARGETS = [
   "Chrome >= 88",
   "Firefox ESR",
   "Safari >= 14.1",
-  "Node >= 16",
+  "Node >= 18",
   "> 1%",
   "not IE > 0",
   "not dead",
@@ -2165,10 +2165,7 @@ function packageJson() {
     license: DIST_LICENSE,
     optionalDependencies: {
       canvas: "^2.11.2",
-    },
-    dependencies: {
       "path2d-polyfill": "^2.0.1",
-      "web-streams-polyfill": "^3.2.1",
     },
     browser: {
       canvas: false,
@@ -2184,7 +2181,7 @@ function packageJson() {
       url: DIST_REPO_URL,
     },
     engines: {
-      node: ">=16",
+      node: ">=18",
     },
   };
 

package.json

@@ -55,7 +55,6 @@
     "typogr": "^0.6.8",
     "vinyl": "^3.0.0",
     "vinyl-fs": "^3.0.3",
-    "web-streams-polyfill": "^3.2.1",
     "webpack": "^5.80.0",
     "webpack-stream": "^7.0.0",
     "wintersmith": "^2.5.0",
@@ -69,7 +68,7 @@
     "url": "git://github.com/mozilla/pdf.js.git"
   },
   "engines": {
-    "node": ">=16"
+    "node": ">=18"
   },
   "license": "Apache-2.0"
 }

src/core/worker.js

@@ -129,24 +129,6 @@ class WorkerMessageHandler {
             "; thus breaking e.g. `for...in` iteration of `Array`s."
         );
       }
-
-      // Ensure that (primarily) Node.js users won't accidentally attempt to use
-      // a non-translated/non-polyfilled build of the library, since that would
-      // quickly fail anyway because of missing functionality.
-      if (
-        (isNodeJS && typeof Path2D === "undefined") ||
-        typeof ReadableStream === "undefined"
-      ) {
-        const partialMsg =
-          "The browser/environment lacks native support for critical " +
-          "functionality used by the PDF.js library " +
-          "(e.g. `Path2D` and/or `ReadableStream`); ";
-
-        if (isNodeJS) {
-          throw new Error(partialMsg + "please use a `legacy`-build instead.");
-        }
-        throw new Error(partialMsg + "please update to a supported browser.");
-      }
     }
     const workerHandlerName = docId + "_worker";
     let handler = new MessageHandler(workerHandlerName, docId, port);

src/shared/compatibility.js

@@ -36,39 +36,24 @@ import { isNodeJS } from "./is_node.js";
   polyfillPath2D(globalThis);
 })();
 
-// Support: Node.js<18.0.0
-(function checkReadableStream() {
-  if (globalThis.ReadableStream || !isNodeJS) {
-    return;
-  }
-  globalThis.ReadableStream = __non_webpack_require__(
-    "web-streams-polyfill/dist/ponyfill.js"
-  ).ReadableStream;
-})();
-
-// Support: Firefox<90, Chrome<92, Safari<15.4, Node.js<16.6.0
+// Support: Chrome<92, Safari<15.4
 (function checkArrayAt() {
   if (Array.prototype.at) {
     return;
   }
   require("core-js/es/array/at.js");
 })();
 
-// Support: Firefox<90, Chrome<92, Safari<15.4, Node.js<16.6.0
+// Support: Chrome<92, Safari<15.4
 (function checkTypedArrayAt() {
   if (Uint8Array.prototype.at) {
     return;
   }
   require("core-js/es/typed-array/at.js");
 })();
 
-// Support: Firefox<94, Chrome<98, Safari<15.4, Node.js<17.0.0
+// Support: Chrome<98, Safari<15.4
 (function checkStructuredClone() {
-  if (typeof PDFJSDev !== "undefined" && PDFJSDev.test("IMAGE_DECODERS")) {
-    // The current image decoders are synchronous, hence `structuredClone`
-    // shouldn't need to be polyfilled for the IMAGE_DECODERS build target.
-    return;
-  }
   if (globalThis.structuredClone) {
     return;
   }

test/test.js

@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-/* eslint-disable no-var, unicorn/prefer-at */
+/* eslint-disable no-var */
 
 "use strict";
 
@@ -750,9 +750,7 @@ function refTestPostHandler(req, res) {
       });
     }
 
-    var isDone =
-      taskResults[taskResults.length - 1] &&
-      taskResults[taskResults.length - 1][lastPageNum - 1];
+    var isDone = taskResults.at(-1) && taskResults.at(-1)[lastPageNum - 1];
     if (isDone) {
       checkRefTestResults(browser, id, taskResults);
       session.remaining--;