@arcblock/did-connect-js 1.28.5 → 1.28.7

package/README.md

@@ -25,7 +25,7 @@ The following diagram demonstrates how a typical DID Connect Session works:
 ```sh
 npm install @arcblock/did-connect-js
 // or
-pnpm install @arcblock/did-connect-js
+bun install @arcblock/did-connect-js
 ```
 
 ## Usage

package/lib/authenticator/base.js

@@ -1,8 +1,6 @@
-/* eslint-disable no-promise-executor-return */
 const Jwt = require('@arcblock/jwt');
 const { isValid, fromJSON } = require('@ocap/wallet');
 
-// eslint-disable-next-line
 const debug = require('debug')(`${require('../../package.json').name}:authenticator:base`);
 
 class BaseAuthenticator {
@@ -49,7 +47,7 @@ class BaseAuthenticator {
         fullWallet.sk = fullWallet.secretKey;
       }
       return fullWallet;
-    } catch (err) {
+    } catch (_err) {
       // If fromJSON fails, return the original object for backward compatibility
       return wallet;
     }

package/lib/authenticator/wallet.js

@@ -1,7 +1,4 @@
-/* eslint-disable no-underscore-dangle */
-/* eslint-disable indent */
-/* eslint-disable object-curly-newline */
-const qs = require('querystring');
+const qs = require('node:querystring');
 const pick = require('lodash/pick');
 const random = require('lodash/random');
 const shuffle = require('lodash/shuffle');
@@ -14,7 +11,6 @@ const { toAddress } = require('@arcblock/did');
 
 const BaseAuthenticator = require('./base');
 
-// eslint-disable-next-line
 const debug = require('debug')(`${require('../../package.json').name}:authenticator:wallet`);
 
 const { DEFAULT_CHAIN_INFO } = BaseAuthenticator;
@@ -30,18 +26,18 @@ const formatDisplay = (display) => {
   }
 
   // object like
-  if (display && display.type && display.content) {
+  if (display?.type && display.content) {
     return JSON.stringify(pick(display, ['type', 'content']));
   }
 
   // string like
   try {
     const parsed = JSON.parse(display);
-    if (parsed && parsed.type && parsed.content) {
+    if (parsed?.type && parsed.content) {
       return display;
     }
     return '';
-  } catch (err) {
+  } catch (_err) {
     return '';
   }
 };
@@ -659,7 +655,7 @@ class WalletAuthenticator extends BaseAuthenticator {
     const description = desc || 'Prepare and sign this transaction to continue.';
 
     // We have to encode the transaction
-    if (type && type.endsWith('Tx')) {
+    if (type?.endsWith('Tx')) {
       if (!chainInfo.host) {
         throw new Error('Invalid chainInfo when trying to encoding partial transaction');
       }
@@ -744,7 +740,6 @@ class WalletAuthenticator extends BaseAuthenticator {
     const inferredLabel = label || asyncFn.name || asyncFn.toString();
     const invocationStack = new Error(`Timeout at: ${inferredLabel}`).stack;
 
-    // eslint-disable-next-line no-async-promise-executor
     return new Promise(async (resolve, reject) => {
       const timer = setTimeout(() => {
         const error = new Error(`Async operation (${inferredLabel}) did not complete within ${timeout} ms`);

package/lib/handlers/base.js

@@ -1,4 +1,4 @@
-const { EventEmitter } = require('events');
+const { EventEmitter } = require('node:events');
 const omit = require('lodash/omit');
 
 const { PROTECTED_KEYS } = require('../protocol');

package/lib/handlers/util.js

@@ -1,7 +1,3 @@
-/* eslint-disable no-underscore-dangle */
-/* eslint-disable prefer-destructuring */
-/* eslint-disable object-curly-newline */
-/* eslint-disable consistent-return */
 const url = require('url');
 const get = require('lodash/get');
 const set = require('lodash/set');
@@ -35,8 +31,7 @@ const isUrl = (input) => {
   return ABSOLUTE_URL_REGEX.test(input);
 };
 
-// eslint-disable-next-line
-const debug = require('debug')(`${require('../../package.json').name}:handlers:util`);
+const _debug = require('debug')(`${require('../../package.json').name}:handlers:util`);
 
 const sha3 = Mcrypto.Hasher.SHA3.hash256;
 const getLocale = (req) => (req.acceptsLanguages('en-US', 'zh-CN') || 'en-US').split('-').shift();
@@ -232,11 +227,9 @@ module.exports = function createHandlers({
     const firstClaim = defaultSteps[0][keys[0]];
     if (Array.isArray(firstClaim)) {
       if (firstClaim[0] === 'authPrincipal') {
-        // eslint-disable-next-line no-param-reassign
         authPrincipal = false;
       }
     } else if (keys[0] === 'authPrincipal') {
-      // eslint-disable-next-line no-param-reassign
       authPrincipal = false;
     }
   }
@@ -282,7 +275,7 @@ module.exports = function createHandlers({
   }
 
   // Whether we can skip the authPrincipal step safely
-  const canSkipConnect = defaultSteps[0] && defaultSteps[0].authPrincipal && defaultSteps[0].authPrincipal.skippable;
+  const canSkipConnect = defaultSteps[0]?.authPrincipal?.skippable;
 
   const createExtraParams = (locale, params, extra = {}) => {
     const finalParams = { ...params, ...(extra || {}) };
@@ -300,7 +293,6 @@ module.exports = function createHandlers({
 
   const createSessionUpdater =
     (token, params) =>
-    // eslint-disable-next-line require-await
     async (key, value, secure = false) => {
       const getUpdate = (k, v) => {
         if (secure && params[encKey]) {
@@ -313,7 +305,7 @@ module.exports = function createHandlers({
 
       // If key is an object, update multiple keys
       if (typeof key === 'object') {
-        secure = value; // eslint-disable-line no-param-reassign
+        secure = value;
         const keys = Object.keys(key);
         const updates = Object.assign(...keys.map((k) => getUpdate(k, key[k])));
         return tokenStorage.update(token, updates);
@@ -525,7 +517,6 @@ module.exports = function createHandlers({
     return false;
   };
 
-  // eslint-disable-next-line consistent-return
   const onAuthRequest = async (req, res) => {
     const { locale, token, store, params, didwallet } = req.context;
     const extraParams = createExtraParams(locale, params, get(store, 'extraParams', {}));
@@ -614,7 +605,6 @@ module.exports = function createHandlers({
     }
   };
 
-  // eslint-disable-next-line consistent-return
   const onAuthResponse = async (req, res) => {
     const { locale, token, store, params, didwallet } = req.context;
 
@@ -773,7 +763,7 @@ module.exports = function createHandlers({
         // Instead, save the relationship between the two
         // Then, mark both session as complete on nextWorkflow complete
         // In theory, we can use this mechanism to concat infinite sessions
-        if (result && result.nextToken && result.nextWorkflow) {
+        if (result?.nextToken && result.nextWorkflow) {
           try {
             await tokenStorage.update(result.nextToken, { prevToken: token });
           } catch (err) {
@@ -850,7 +840,7 @@ module.exports = function createHandlers({
     }
   };
 
-  const ensureContext = async (req, res, next) => {
+  const ensureContext = async (req, _res, next) => {
     const didwallet = parseWalletUA(req.query['user-agent'] || req.headers['user-agent']);
     const params = { ...req.body, ...req.query, ...req.params };
     const token = params[tokenKey];

package/lib/handlers/wallet.js

@@ -1,5 +1,3 @@
-/* eslint-disable object-curly-newline */
-// eslint-disable-next-line
 const debug = require('debug')(`${require('../../package.json').name}:handlers:wallet`);
 
 const cors = require('cors');
@@ -84,7 +82,6 @@ class WalletHandlers extends BaseHandler {
     onDecline = noop,
     onComplete = noop,
     onExpire = noop,
-    // eslint-disable-next-line no-console
     onError = console.error,
     authPrincipal = true,
     persistentDynamicClaims = false,
@@ -105,7 +102,6 @@ class WalletHandlers extends BaseHandler {
     const pathname = `${prefix}/${action}/auth`;
     debug('attach routes', { action, prefix, pathname });
 
-    // eslint-disable-next-line consistent-return
     const onConnectWrapped = async (...args) => {
       if (typeof this.onConnect === 'function') {
         await this.onConnect(...args);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@arcblock/did-connect-js",
   "description": "Helper function to setup DID Connect support on a node.js web server",
-  "version": "1.28.5",
+  "version": "1.28.7",
   "author": {
     "name": "wangshijun",
     "email": "shijun@arcblock.io",
@@ -18,29 +18,28 @@
     "access": "public"
   },
   "dependencies": {
+    "@arcblock/did": "1.28.7",
+    "@arcblock/jwt": "1.28.7",
+    "@arcblock/validator": "1.28.7",
+    "@ocap/client": "1.28.7",
+    "@ocap/mcrypto": "1.28.7",
+    "@ocap/util": "1.28.7",
+    "@ocap/wallet": "1.28.7",
     "cors": "^2.8.5",
     "debug": "^4.3.6",
     "json-stable-stringify": "^1.0.1",
     "lodash": "^4.17.21",
     "semver": "^7.6.3",
-    "tweetnacl-sealedbox-js": "^1.2.0",
-    "@arcblock/did": "1.28.5",
-    "@arcblock/jwt": "1.28.5",
-    "@arcblock/validator": "1.28.5",
-    "@ocap/client": "1.28.5",
-    "@ocap/mcrypto": "1.28.5",
-    "@ocap/util": "1.28.5",
-    "@ocap/wallet": "1.28.5"
+    "tweetnacl-sealedbox-js": "^1.2.0"
   },
   "devDependencies": {
     "@arcblock/did-agent-storage-memory": "^1.8.0",
     "@arcblock/did-connect-storage-memory": "^1.8.0",
+    "@ocap/e2e-test": "1.28.7",
     "axios": "^1.7.5",
-    "jest": "^29.7.0",
     "remark-cli": "^10.0.1",
     "remark-preset-github": "^4.0.4",
-    "tweetnacl": "^1.0.3",
-    "@ocap/e2e-test": "1.28.5"
+    "tweetnacl": "^1.0.3"
   },
   "remarkConfig": {
     "plugins": [
@@ -68,19 +67,19 @@
     "type": "git",
     "url": "https://github.com/ArcBlock/blockchain/tree/master/did/did-connect"
   },
-  "resolutions": {
-    "tweetnacl": "1.0.3"
-  },
-  "gitHead": "87990c8b5e215107fc587c1ced0d6b3e2cd2483e",
   "scripts": {
-    "lint": "eslint lib tests",
-    "lint:fix": "eslint --fix lib tests",
-    "docs": "pnpm run gen-dts && pnpm run gen-docs && pnpm run cleanup-docs && pnpm run format-docs",
+    "lint": "biome check",
+    "lint:fix": "biome check --write",
+    "docs": "bun run gen-dts && bun run gen-docs && bun run cleanup-docs && bun run format-docs",
     "cleanup-docs": "node ../../scripts/cleanup-docs.js docs/README.md $npm_package_name",
     "gen-docs": "jsdoc2md lib/**/*.js lib/**/**/*.js > docs/README.md",
     "gen-dts": "j2d lib/index.js",
     "format-docs": "remark . -o",
-    "test": "jest --forceExit --detectOpenHandles",
+    "test": "bun test",
     "coverage": "npm run test -- --coverage"
-  }
-}
+  },
+  "resolutions": {
+    "tweetnacl": "1.0.3"
+  },
+  "gitHead": "87990c8b5e215107fc587c1ced0d6b3e2cd2483e"
+}