wuffle 0.64.0 → 0.66.0

package/lib/apps/auth-routes/AuthRoutes.js

@@ -83,7 +83,7 @@ export default function AuthRoutes(logger, router, securityContext) {
 
     return req.session.destroy(function(err) {
       return res.redirect(redirectTo);
-    });
+    }) && null;
 
   });
 
@@ -185,7 +185,7 @@ export default function AuthRoutes(logger, router, securityContext) {
     } = session;
 
     if (!githubUser) {
-      return res.type('json').json(null);
+      return res.type('json').json(null) && null;
     }
 
     const {
@@ -218,14 +218,14 @@ export default function AuthRoutes(logger, router, securityContext) {
         // access is not granted anymore, clear current session
         return req.session.destroy(function(err) {
           return res.type('json').json(null);
-        });
+        }) && null;
       }
     }
 
     return res.type('json').json({
       login,
       avatar_url
-    });
+    }) && null;
 
   });
 

package/lib/apps/background-sync/BackgroundSync.js

@@ -104,7 +104,7 @@ We automatically synchronize all repositories you granted us access to via the G
       const octokit = await githubClient.getOrgScoped(owner);
 
       const repositories = await octokit.paginate(
-        octokit.apps.listReposAccessibleToInstallation,
+        octokit.rest.apps.listReposAccessibleToInstallation,
         {
           per_page: 100
         }
@@ -152,7 +152,7 @@ We automatically synchronize all repositories you granted us access to via the G
 
             // open issues
             octokit.paginate(
-              octokit.issues.listForRepo,
+              octokit.rest.issues.listForRepo,
               {
                 ...params,
                 state: 'open'
@@ -162,7 +162,7 @@ We automatically synchronize all repositories you granted us access to via the G
 
             // closed issues, updated last 30 days
             octokit.paginate(
-              octokit.issues.listForRepo,
+              octokit.rest.issues.listForRepo,
               {
                 ...params,
                 state: 'closed',
@@ -173,7 +173,7 @@ We automatically synchronize all repositories you granted us access to via the G
 
             // open pulls, all
             octokit.paginate(
-              octokit.pulls.list,
+              octokit.rest.pulls.list,
               {
                 ...params,
                 state: 'open'
@@ -182,7 +182,7 @@ We automatically synchronize all repositories you granted us access to via the G
 
             // closed pulls, updated last 30 days
             octokit.paginate(
-              octokit.pulls.list,
+              octokit.rest.pulls.list,
               {
                 ...params,
                 state: 'closed'

package/lib/apps/board-api-routes/board-api-routes.js

@@ -174,17 +174,17 @@ export default async function BoardApiRoutes(
 
     return filterBoardItems(req, items).then(filteredItems => {
 
-      res.type('json').json({
+      return res.type('json').json({
         items: filteredItems,
         cursor
-      });
+      }) && null;
     }).catch(err => {
       log.error({
         err,
         cursor
       }, 'failed to retrieve cards');
 
-      res.status(500).json({ error : true });
+      return res.status(500).json({ error : true }) && null;
     });
   });
 
@@ -206,7 +206,7 @@ export default async function BoardApiRoutes(
         };
       }),
       name: name || 'Wuffle Board'
-    });
+    }) && null;
 
   });
 
@@ -236,7 +236,7 @@ export default async function BoardApiRoutes(
     const user = authRoutes.getGitHubUser(req);
 
     if (!user) {
-      return res.status(401).json({});
+      return res.status(401).json({}) && null;
     }
 
     const body = JSON.parse(req.body);
@@ -251,13 +251,13 @@ export default async function BoardApiRoutes(
     const issue = await store.getIssueById(id);
 
     if (!issue) {
-      return res.status(404).json({});
+      return res.status(404).json({}) && null;
     }
 
     const column = columns.getByName(columnName);
 
     if (!column) {
-      return res.status(404).json({});
+      return res.status(404).json({}) && null;
     }
 
     const repo = repoAndOwner(issue);
@@ -265,7 +265,7 @@ export default async function BoardApiRoutes(
     const canWrite = await userAccess.canWrite(user, repo);
 
     if (!canWrite) {
-      return res.status(403).json({});
+      return res.status(403).json({}) && null;
     }
 
     const octokit = await githubClient.getUserScoped(user);
@@ -288,7 +288,7 @@ export default async function BoardApiRoutes(
 
         res.status(500).json({ error : true });
       })
-    );
+    ) && null;
 
   });
 

package/lib/apps/github-app/GithubApp.js

@@ -245,7 +245,7 @@ export default function GithubApp(config, app, logger, injector) {
   function fetchInstallations() {
     return /** @type Promise<Array<Installation>> */ (getAppScopedClient().then(
       octokit => octokit.paginate(
-        octokit.apps.listInstallations,
+        octokit.rest.apps.listInstallations,
         { per_page: 100 }
       )
     ));

package/lib/apps/github-checks/GithubChecks.js

@@ -44,7 +44,7 @@ export default function GithubChecks(webhookEvents, events, githubClient, store)
 
       const {
         data: result
-      } = await github.checks.listForRef({
+      } = await github.rest.checks.listForRef({
         owner,
         repo,
         ref: sha,

package/lib/apps/github-client/GithubClient.js

@@ -1,4 +1,4 @@
-import { ProbotOctokit } from 'probot/lib/octokit/probot-octokit.js';
+import { ProbotOctokit } from 'probot';
 import { Cache } from '../../util/index.js';
 
 

package/lib/apps/github-issues/GithubIssues.js

@@ -77,7 +77,7 @@ export default function GithubIssues(config, logger, columns) {
 
     const params = context.repo({ issue_number });
 
-    return context.octokit.issues.get(params)
+    return context.octokit.rest.issues.get(params)
       .then(response => response.data)
       .catch(err => {
 
@@ -156,7 +156,7 @@ export default function GithubIssues(config, logger, columns) {
       log.info(addLabelParams, 'add labels');
 
       invocations.push(
-        context.octokit.issues.addLabels(addLabelParams)
+        context.octokit.rest.issues.addLabels(addLabelParams)
       );
     }
 
@@ -170,7 +170,7 @@ export default function GithubIssues(config, logger, columns) {
       log.info(params, 'update');
 
       invocations.push(
-        context.octokit.issues.update(params)
+        context.octokit.rest.issues.update(params)
       );
     }
 
@@ -187,7 +187,7 @@ export default function GithubIssues(config, logger, columns) {
       }, 'remove label');
 
       invocations.push(
-        context.octokit.issues.removeLabel(removeLabelParams).catch(err => {
+        context.octokit.rest.issues.removeLabel(removeLabelParams).catch(err => {
 
           // gracefully handle non-existing label
           // may have been already removed by other

package/lib/apps/github-reviews/GithubReviews.js

@@ -39,7 +39,7 @@ export default function GithubReviews(webhookEvents, events, githubClient, store
 
     const {
       data: reviews
-    } = await github.pulls.listReviews({
+    } = await github.rest.pulls.listReviews({
       owner,
       repo,
       pull_number: issue.number

package/lib/apps/github-statuses/GithubStatuses.js

@@ -46,7 +46,7 @@ export default function GithubStates(webhookEvents, events, githubClient, store)
 
       const {
         data: response
-      } = await github.repos.getCombinedStatusForRef({
+      } = await github.rest.repos.getCombinedStatusForRef({
         owner,
         repo,
         ref: sha

package/lib/apps/security-context/SecurityContext.js

@@ -17,7 +17,7 @@ export default function SecurityContext(githubClient) {
 
     const {
       data: user
-    } = await github.users.getAuthenticated();
+    } = await github.rest.users.getAuthenticated();
 
     return user;
   };

package/lib/apps/user-access/UserAccess.js

@@ -46,7 +46,7 @@ export default function UserAccess(
     const octokit = await githubClient.getUserScoped(user);
 
     return octokit.paginate(
-      octokit.apps.listInstallationsForAuthenticatedUser
+      octokit.rest.apps.listInstallationsForAuthenticatedUser
     );
   }
 
@@ -65,7 +65,7 @@ export default function UserAccess(
     const octokit = await githubClient.getUserScoped(user);
 
     return octokit.paginate(
-      octokit.apps.listInstallationReposForAuthenticatedUser,
+      octokit.rest.apps.listInstallationReposForAuthenticatedUser,
       {
         installation_id: installation.id,
         per_page: 100
@@ -184,7 +184,7 @@ export default function UserAccess(
 
     return githubClient.getOrgScoped(owner)
       .then(octokit => {
-        return octokit.repos.getCollaboratorPermissionLevel({
+        return octokit.rest.repos.getCollaboratorPermissionLevel({
           repo,
           owner,
           username: login

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wuffle",
-  "version": "0.64.0",
+  "version": "0.66.0",
   "description": "A multi-repository task board for GitHub issues",
   "author": {
     "name": "Nico Rehwaldt",
@@ -44,36 +44,36 @@
   "dependencies": {
     "@aws-sdk/client-s3": "^3.525.0",
     "async-didi": "^0.3.1",
-    "body-parser": "^1.20.0",
-    "compression": "^1.7.5",
-    "express-session": "^1.18.1",
+    "body-parser": "^2.0.0",
+    "compression": "^1.8.1",
+    "express-session": "^1.18.2",
     "fake-tag": "^5.0.0",
     "memorystore": "^1.6.7",
-    "min-dash": "^4.2.2",
+    "min-dash": "^4.2.3",
     "mkdirp": "^3.0.1",
     "p-defer": "^4.0.1",
     "prexit": "^2.3.0",
-    "probot": "^13.4.1",
-    "smee-client": "^2.0.4"
+    "probot": "^13.4.5",
+    "smee-client": "^4.0.0"
   },
   "devDependencies": {
-    "@graphql-eslint/eslint-plugin": "^4.3.0",
+    "@graphql-eslint/eslint-plugin": "^4.4.0",
     "@octokit/graphql-schema": "^15.25.0",
-    "@types/compression": "^1.7.2",
-    "@types/express-session": "^1.18.1",
+    "@types/compression": "^1.8.1",
+    "@types/express-session": "^1.18.2",
     "@types/mocha": "^10.0.10",
-    "chai": "^4.5.0",
-    "graphql": "^16.10.0",
-    "mocha": "^10.8.2",
-    "nock": "^13.5.6",
-    "nodemon": "^3.1.9",
-    "npm-run-all2": "^7.0.2",
-    "sinon": "^18.0.1",
-    "sinon-chai": "^3.7.0",
+    "chai": "^6.0.0",
+    "graphql": "^16.11.0",
+    "mocha": "^11.0.0",
+    "nock": "^14.0.10",
+    "nodemon": "^3.1.10",
+    "npm-run-all2": "^8.0.4",
+    "sinon": "^21.0.0",
+    "sinon-chai": "^4.0.0",
     "typescript": "^5.7.2"
   },
   "engines": {
-    "node": ">= 18"
+    "node": ">= 20"
   },
   "files": [
     "bin",
@@ -83,5 +83,5 @@
     "index.js",
     "wuffle.config.example.js"
   ],
-  "gitHead": "bf153a12861bce84a2562407218d9110a328977b"
+  "gitHead": "ce1d4f52d8ac699a03db60e3fd35a3f6f81f8397"
 }