update dev dependencies

Author: haroun

.github/workflows/main.yml

@@ -5,9 +5,9 @@ name: tests
 # Controls when the action will run.
 on:
   push:
-    branches: [ '*' ]
+    branches: ['*']
   pull_request:
-    branches: [ '*' ]
+    branches: ['*']
 
   # Allows you to run this workflow manually from the Actions tab
   workflow_dispatch:
@@ -20,26 +20,26 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: [14, 16, 18]
-        mongodb-version: [4.2, 4.4, 5.0]
+        node-version: [18, 20, 22]
+        mongodb-version: [5.0, 6.0, 7.0, 8.0]
 
     # Steps represent a sequence of tasks that will be executed as part of the job
     steps:
-    - name: Git checkout
-      uses: actions/checkout@v2
+      - name: Git checkout
+        uses: actions/checkout@v4
 
-    - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v1
-      with:
-        node-version: ${{ matrix.node-version }}
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
 
-    - name: Start MongoDB
-      uses: supercharge/mongodb-github-action@1.3.0
-      with:
-        mongodb-version: ${{ matrix.mongodb-version }}
+      - name: Start MongoDB
+        uses: supercharge/mongodb-github-action@1.11.0
+        with:
+          mongodb-version: ${{ matrix.mongodb-version }}
 
-    - run: npm install
+      - run: npm install
 
-    - run: npm test
-      env:
-        CI: true
+      - run: npm test
+        env:
+          CI: true

index.js

@@ -33,7 +33,9 @@ module.exports = {
           // that export a Strategy property, and directly passing in a strategy property
           // in the spec
           const strategyModule = spec.module && await self.apos.root.import(spec.module);
-          const Strategy = strategyModule ? (strategyModule.Strategy || strategyModule) : spec.Strategy;
+          const Strategy = strategyModule
+            ? (strategyModule.Strategy || strategyModule)
+            : spec.Strategy;
           if (!Strategy) {
             throw new Error('@apostrophecms/passport-bridge: each strategy must have a "module" setting\n' +
               'giving the name of an npm module installed in your project that\n' +
@@ -111,10 +113,12 @@ module.exports = {
         return (absolute ? (self.apos.baseUrl + self.apos.prefix) : '') + '/auth/' + strategyName + '/connect/' + token;
       },
 
-      // Adds the login route, which will be `/auth/strategyname/login`, where the strategy name
+      // Adds the login route
+      // which will be `/auth/strategyname/login`, where the strategy name
       // depends on the passport module being used.
       //
-      // Redirect users to this URL to start the process of logging them in via each strategy
+      // Redirect users to this URL
+      // to start the process of logging them in via each strategy
       addLoginRoute(spec) {
         self.apos.app.get(self.getLoginUrl(spec), (req, res, next) => {
           if (req.query.newLocale) {
@@ -208,7 +212,7 @@ module.exports = {
         self.apos.app.get(self.getFailureUrl(spec), function (req, res) {
           // Gets i18n'd in the template
           return self.sendPage(req, 'error', {
-            spec: spec,
+            spec,
             message: 'aposPassportBridge:rejected'
           });
         });
@@ -289,7 +293,12 @@ module.exports = {
             return callback(null, false);
           }
           try {
-            const user = await self.apos.user.find(adminReq, criteria).toObject() || (self.options.create && !connectingUserId && await self.createUser(spec, profile));
+            const user = await self.apos.user.find(adminReq, criteria).toObject() ||
+              (
+                self.options.create &&
+                !connectingUserId &&
+                await self.createUser(spec, profile)
+              );
             // Legacy, incompatible with Passport 0.6
             if (self.options.retainAccessTokenInSession && user && req) {
               req.session.accessToken = accessToken;
@@ -473,7 +482,9 @@ module.exports = {
           }
           let route;
           if (doc) {
-            const action = self.apos.page.isPage(doc) ? self.apos.page.action : self.apos.doc.getManager(doc).action;
+            const action = self.apos.page.isPage(doc)
+              ? self.apos.page.action
+              : self.apos.doc.getManager(doc).action;
             route = `${action}/${doc._id}/locale/${newLocale}`;
           } else {
             // Fall back to home page, with appropriate prefix

modules/@apostrophecms/user-passport-bridge/index.js

@@ -75,7 +75,9 @@ module.exports = {
           return await fn(accessToken);
         } catch (e) {
           if (e.status && e.status === 401) {
-            const { accessToken } = await self.refreshTokens(user, strategy, refreshToken);
+            const {
+              accessToken
+            } = await self.refreshTokens(user, strategy, refreshToken);
             // On the second try, failure is failure
             // We don't need "await" because we are already returning
             // a promise

package.json

@@ -16,15 +16,15 @@
   "author": "Apostrophe Technologies",
   "license": "MIT",
   "devDependencies": {
-    "eslint": "^7.9.0",
-    "eslint-config-apostrophe": "^3.4.0",
-    "eslint-config-standard": "^14.1.1",
+    "eslint": "^8.57.1",
+    "eslint-config-apostrophe": "^4.3.0",
+    "eslint-config-standard": "^17.1.0",
     "eslint-plugin-import": "^2.22.0",
     "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-promise": "^4.2.1",
+    "eslint-plugin-promise": "^6.6.0",
     "eslint-plugin-standard": "^4.0.1",
-    "stylelint": "^13.7.1",
-    "stylelint-config-apostrophe": "^1.0.0"
+    "stylelint": "^16.9.0",
+    "stylelint-config-apostrophe": "^4.1.0"
   },
   "dependencies": {
     "humanname": "^0.2.2",