Drop Node16 support (inrupt#901)

* Update devDependencies
* No longer run CI on Node 16
* Upgrade prod dependencies
* Storybook migration
* Changelog
* Lint
* Fix unit test issue
* Remove fetch polyfills

Author: NSeydoux

Diff for: .babelrc.json

@@ -0,0 +1,18 @@
+{
+  "sourceType": "unambiguous",
+  "presets": [
+    [
+      "@babel/preset-env",
+      {
+        "targets": {
+          "chrome": 100,
+          "safari": 15,
+          "firefox": 91
+        }
+      }
+    ],
+    "@babel/preset-typescript",
+    "@babel/preset-react"
+  ],
+  "plugins": []
+}

Diff for: .eslintrc.js

@@ -22,6 +22,11 @@ module.exports = {
     "no-shadow": ["warn", { allow: ["fetch"] }],
     "react/require-default-props": "warn",
     "react/default-props-match-prop-types": "warn",
+    "react/function-component-definition": "off",
     noImplicitAny: "off",
+    // FIXME: re-enable the following
+    "react/no-unstable-nested-components": "off",
+    "react/jsx-no-bind": "off",
+    "react/jsx-no-constructed-context-values": "off"
   },
 };

Diff for: .github/workflows/ci.yml

@@ -15,7 +15,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: ["20.x", "18.x", "16.x"]
+        node-version: ["20.x", "18.x"]
     steps:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v4

Diff for: .github/workflows/release.yml

@@ -15,7 +15,7 @@ jobs:
       - name: Prepare for publication to npm
         uses: actions/setup-node@v4
         with:
-          node-version: "16.x"
+          node-version-file: ".nvmrc"
           registry-url: "https://registry.npmjs.org"
           cache: npm
       - run: npm ci

Diff for: .storybook/main.js

@@ -1,27 +1,27 @@
 module.exports = {
-  core: {
-    builder: "webpack5",
+  framework: {
+    name: "@storybook/react-webpack5",
+    options: {}
   },
-  framework: '@storybook/react',
+
   staticDirs: ["../public"],
   stories: ["../stories/**/*.stories.@(tsx|mdx)"],
-  addons: [
-    {
-      name: '@storybook/addon-essentials',
-      // https://github.com/storybookjs/storybook/issues/17996#issuecomment-1134810729
-      options: {
-        'actions': false,
-      }
-    },
-    {
-      name: "@storybook/addon-storysource",
-      options: {
-        sourceLoaderOptions: {
-          injectStoryParameters: false,
-        },
+
+  addons: [{
+    name: '@storybook/addon-essentials',
+    // https://github.com/storybookjs/storybook/issues/17996#issuecomment-1134810729
+    options: {
+      'actions': false,
+    }
+  }, {
+    name: "@storybook/addon-storysource",
+    options: {
+      sourceLoaderOptions: {
+        injectStoryParameters: false,
       },
     },
-  ],
+  }, "@storybook/addon-mdx-gfm"],
+
   typescript: {
     check: false,
     checkOptions: {},
@@ -32,7 +32,12 @@ module.exports = {
         prop.parent ? !/node_modules/.test(prop.parent.fileName) : true,
     },
   },
+
   webpackFinal: async (config) => {
     return config;
   },
+
+  docs: {
+    autodocs: true
+  }
 };

Diff for: CHANGELOG.md

@@ -4,6 +4,10 @@ This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html
 
 ## Unreleased
 
+### Breaking changes
+
+- Node 16 is no longer supported, this requires Node 18 or higher to be built.
+
 ## 2.9.1 ( December 19, 2023)
 
 - Use non-deprecated events API from solid-client-authn-js

Diff for: jest.config.ts

@@ -1,23 +1,23 @@
-/**
- * Copyright 2020 Inrupt Inc.
- *
- * Permission is hereby granted, free of charge, to any person obtaining a copy
- * of this software and associated documentation files (the "Software"), to deal in
- * the Software without restriction, including without limitation the rights to use,
- * copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the
- * Software, and to permit persons to whom the Software is furnished to do so,
- * subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in
- * all copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED,
- * INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A
- * PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
- * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
- * OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
- * SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
- */
+//
+// Copyright Inrupt Inc.
+//
+// Permission is hereby granted, free of charge, to any person obtaining a copy
+// of this software and associated documentation files (the "Software"), to deal in
+// the Software without restriction, including without limitation the rights to use,
+// copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the
+// Software, and to permit persons to whom the Software is furnished to do so,
+// subject to the following conditions:
+//
+// The above copyright notice and this permission notice shall be included in
+// all copies or substantial portions of the Software.
+//
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED,
+// INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A
+// PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+// HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+// OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+// SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+//
 export default {
   // Automatically clear mock calls and instances between every test
   clearMocks: true,

Diff for: jest.setup.ts

@@ -1,22 +1,61 @@
-/**
- * Copyright 2020 Inrupt Inc.
- *
- * Permission is hereby granted, free of charge, to any person obtaining a copy
- * of this software and associated documentation files (the "Software"), to deal in
- * the Software without restriction, including without limitation the rights to use,
- * copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the
- * Software, and to permit persons to whom the Software is furnished to do so,
- * subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in
- * all copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED,
- * INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A
- * PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
- * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
- * OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
- * SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
- */
+//
+// Copyright Inrupt Inc.
+//
+// Permission is hereby granted, free of charge, to any person obtaining a copy
+// of this software and associated documentation files (the "Software"), to deal in
+// the Software without restriction, including without limitation the rights to use,
+// copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the
+// Software, and to permit persons to whom the Software is furnished to do so,
+// subject to the following conditions:
+//
+// The above copyright notice and this permission notice shall be included in
+// all copies or substantial portions of the Software.
+//
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED,
+// INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A
+// PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+// HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+// OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+// SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+//
 
-import "@inrupt/jest-jsdom-polyfills";
+// import "@inrupt/jest-jsdom-polyfills";
+if (
+    typeof globalThis.TextEncoder === "undefined" ||
+    typeof globalThis.TextDecoder === "undefined"
+  ) {
+    const utils = require("util");
+    globalThis.TextEncoder = utils.TextEncoder;
+    globalThis.TextDecoder = utils.TextDecoder;
+    // TextEncoder references a Uint8Array constructor different than the global
+    // one used by users in tests. The following enforces the same constructor to
+    // be referenced by both.
+    // FIXME: currently this doesn't work, and must be set in a custom environment.
+    globalThis.Uint8Array = Uint8Array;
+  }
+  
+  if (
+    typeof globalThis.crypto !== "undefined" &&
+    // jsdom doesn't implement the subtle Web Crypto API
+    typeof globalThis.crypto.subtle === "undefined"
+  ) {
+    // Requires OPENSSL_CONF=/dev/null (see https://github.com/nodejs/node/discussions/43184)
+    const {
+      Crypto: WCrypto,
+      CryptoKey: WCryptoKey,
+    } = require("@peculiar/webcrypto");
+    Object.assign(globalThis.crypto, new WCrypto());
+    globalThis.CryptoKey = WCryptoKey;
+  }
+  
+  // Node.js doesn't support Blob or File, so we're polyfilling those with
+  // https://github.com/web-std/io
+  if (typeof globalThis.Blob === "undefined") {
+    const stdBlob = require("@web-std/blob");
+    globalThis.Blob = stdBlob.Blob;
+  }
+  
+  if (typeof globalThis.File === "undefined") {
+    const stdFile = require("@web-std/file");
+    globalThis.File = stdFile.File;
+  }