chore: use set to check supported algorithms (#336)

* feat: ignore pnpm

* fix: use set

* chore: reduce

.gitignore

@@ -1,5 +1,6 @@
 package-lock.json
 yarn.lock
+pnpm-lock.yaml
 
 .clinic
 .nyc_output

src/signer.js

@@ -15,9 +15,9 @@ const { TokenError } = require('./error')
 const { getAsyncKey, ensurePromiseCallback } = require('./utils')
 const { createPrivateKey, createSecretKey } = require('crypto')
 
-const supportedAlgorithms = Array.from(
-  new Set([...hsAlgorithms, ...esAlgorithms, ...rsaAlgorithms, ...edAlgorithms, 'none'])
-).join(', ')
+const supportedAlgorithms = new Set([...hsAlgorithms, ...esAlgorithms, ...rsaAlgorithms, ...edAlgorithms, 'none'])
+
+const supportedAlgorithmsList = Array.from(supportedAlgorithms).join(', ')
 
 function checkIsCompatibleAlgorithm(expected, actual) {
   const expectedType = expected.slice(0, 2)
@@ -193,15 +193,11 @@ module.exports = function createSigner(options) {
   // Validate options
   if (
     algorithm &&
-    algorithm !== 'none' &&
-    !hsAlgorithms.includes(algorithm) &&
-    !esAlgorithms.includes(algorithm) &&
-    !rsaAlgorithms.includes(algorithm) &&
-    !edAlgorithms.includes(algorithm)
+    !supportedAlgorithms.has(algorithm)
   ) {
     throw new TokenError(
       TokenError.codes.invalidOption,
-      `The algorithm option must be one of the following values: ${supportedAlgorithms}.`
+      `The algorithm option must be one of the following values: ${supportedAlgorithmsList}.`
     )
   }
 
@@ -285,8 +281,11 @@ module.exports = function createSigner(options) {
   }
 
   const fpo = { jti, aud, iss, sub, nonce }
-  const fixedPayload = Object.keys(fpo).reduce((obj, key) => {
-    return fpo[key] !== undefined ? Object.assign(obj, { [key]: fpo[key] }) : obj
+  const fixedPayload = Object.entries(fpo).reduce((obj, [key, value]) => {
+    if (value !== undefined) {
+      obj[key] = value
+    }
+    return obj
   }, {})
 
   // Return the signer