Account for different Node.js version default stream high water mark values.

Also, enable testing Node.js v22 in CI.

Author: jaydenseric

.github/workflows/ci.yml

@@ -7,7 +7,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, macos-latest]
-        node: ["18", "20"]
+        node: ["18", "20", "22"]
     steps:
       - uses: actions/checkout@v3
       - name: Setup Node.js v${{ matrix.node }}

changelog.md

@@ -36,10 +36,11 @@
 
 - Updated the `package.json` field `repository` to conform to new npm requirements.
 - Updated GitHub Actions CI config:
-  - Updated the tested Node.js versions to v18, v20.
+  - Updated the tested Node.js versions to v18, v20, v22.
 - Migrated to the ESLint v9 CLI and “flat” config.
 - Integrated a new dev dependency [`eslint-plugin-jsdoc`](https://npm.im/eslint-plugin-jsdoc) and revised types.
 - Removed the Node.js CLI option `--unhandled-rejections=throw` in the package script `tests` as it’s now the default for all supported Node.js versions.
+- Avoid hardcoding a default value in the type `FileUploadCreateReadStreamOptions` property `highWaterMark` description and use the function `getDefaultHighWaterMark` from `node:stream` in tests.
 - Omit unused catch bindings in the function `processRequest`.
 - Corrected the JSDoc type `FileUploadCreateReadStreamOptions` in the module `processRequest.mjs`.
 - Avoid using `return` in the middleware.

processRequest.mjs

@@ -396,7 +396,8 @@ export default function processRequest(
  *   `base64url`, or `hex`. Defaults to `utf8`.
  * @property {ReadStreamOptions["highWaterMark"]} [highWaterMark] Maximum number
  *   of bytes to store in the internal buffer before ceasing to read from the
- *   underlying resource. Defaults to `16384`.
+ *   underlying resource. Defaults to the Node.js default high water mark for
+ *   non object mode streams.
  */
 
 /**

processRequest.test.mjs

@@ -11,6 +11,7 @@ import {
   throws,
 } from "node:assert";
 import { createServer } from "node:http";
+import { getDefaultHighWaterMark } from "node:stream";
 import { text } from "node:stream/consumers";
 import { describe, it } from "node:test";
 
@@ -85,7 +86,10 @@ describe(
 
           ok(stream instanceof ReadStream);
           strictEqual(stream.readableEncoding, null);
-          strictEqual(stream.readableHighWaterMark, 16384);
+          strictEqual(
+            stream.readableHighWaterMark,
+            getDefaultHighWaterMark(false),
+          );
           strictEqual(await text(stream), "a");
         } catch (error) {
           serverError = error;
@@ -143,7 +147,10 @@ describe(
 
           ok(stream instanceof ReadStream);
           strictEqual(stream.readableEncoding, null);
-          strictEqual(stream.readableHighWaterMark, 16384);
+          strictEqual(
+            stream.readableHighWaterMark,
+            getDefaultHighWaterMark(false),
+          );
           strictEqual(await text(stream), "a");
         } catch (error) {
           serverError = error;