wuffle 0.73.2 → 0.74.0

package/LICENSE

@@ -0,0 +1,21 @@
+The MIT License (MIT)
+
+Copyright (c) 2019-present Nico Rehwaldt
+
+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.

package/lib/apps/{automatic-dev-flow.js → automatic-dev-flow/AutomaticDevFlow.js}

@@ -1,6 +1,3 @@
-import { filterIssueOrPull } from '../filters.js';
-import { issueIdent } from '../util/meta.js';
-
 const DONE = 'DONE';
 const EXTERNAL_CONTRIBUTION = 'EXTERNAL_CONTRIBUTION';
 const IN_PROGRESS = 'IN_PROGRESS';
@@ -15,11 +12,11 @@ const CHANGES_REQUESTED = 'changes_requested';
  *
  * @constructor
  *
- * @param {import('./webhook-events/WebhookEvents.js').default} webhookEvents
- * @param {import('./github-issues/GithubIssues.js').default} githubIssues
- * @param {import('../columns.js').default} columns
- * @param {import('./issue-filter/IssueFilter.js').default} issueFilter
- * @param {import('../types.js').Logger } logger
+ * @param {import('../webhook-events/WebhookEvents.js').default} webhookEvents
+ * @param {import('../github-issues/GithubIssues.js').default} githubIssues
+ * @param {import('../../columns.js').default} columns
+ * @param {import('../issue-filter/IssueFilter.js').default} issueFilter
+ * @param {import('../../types.js').Logger } logger
  */
 export default function(webhookEvents, githubIssues, columns, issueFilter, logger) {
 
@@ -27,26 +24,7 @@ export default function(webhookEvents, githubIssues, columns, issueFilter, logge
     name: 'wuffle:automatic-dev-flow'
   });
 
-  function ifEnabled(webhookHandlerFn) {
-
-    return (context) => {
-
-      const payload = context.payload;
-
-      const issueOrPull = filterIssueOrPull(
-        payload.issue || payload.pull_request,
-        payload.repository
-      );
-
-      if (issueFilter.isIgnored(issueOrPull)) {
-        log.debug({ issue: issueIdent(issueOrPull) }, 'issue matching ignore filter');
-
-        return;
-      }
-
-      return webhookHandlerFn(context);
-    };
-  }
+  const ifEnabled = issueFilter.createWebhookFilter(log);
 
   webhookEvents.on([
     'issues.closed',

package/lib/apps/automatic-dev-flow/index.js

@@ -0,0 +1,6 @@
+import AutomaticDevFlow from './AutomaticDevFlow.js';
+
+export default {
+  __init__: [ 'automaticDevFlow' ],
+  automaticDevFlow: [ 'type', AutomaticDevFlow ]
+};

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

@@ -11,14 +11,13 @@ function isInternalError(error) {
  *
  * @constructor
  *
- * @param {import('../../types.js').Logger} logger
  * @param {Object} config
+ * @param {import('../../types.js').Logger} logger
  * @param {import('../../store.js').default} store
- * @param {import('../github-client/GithubClient.js').default} githubClient
- * @param {import('../github-app/GithubApp.js').default} githubApp
  * @param {import('../../events.js').default} events
+ * @param {import('./BackgroundSyncBackend.js').default} backgroundSyncBackend
  */
-export default function BackgroundSync(logger, config, store, githubClient, githubApp, events) {
+export default function BackgroundSync(config, logger, store, events, backgroundSyncBackend) {
 
   // 30 days
   const syncClosedLookback = (
@@ -101,18 +100,14 @@ We automatically synchronize all repositories you granted us access to via the G
     log.debug({ installation: owner }, 'processing');
 
     try {
-      const octokit = await githubClient.getOrgScoped(owner);
-
-      const repositories = await octokit.paginate(
-        octokit.rest.apps.listReposAccessibleToInstallation,
-        {
-          per_page: 100
-        }
-      );
+      const repositories = await backgroundSyncBackend.getInstallationRepositories(installation);
 
       for (const repository of repositories) {
 
-        const owner = repository.owner.login;
+        if (repository.owner.login !== owner) {
+          throw new Error('repository.owner !== installation.owner');
+        }
+
         const repo = repository.name;
 
         // log found repository
@@ -135,74 +130,12 @@ We automatically synchronize all repositories you granted us access to via the G
             repo
           }, 'processing');
 
-          const params = {
-            sort: /** @type { 'updated' } */ ('updated'),
-            direction: /** @type { 'desc' } */ ('desc'),
-            per_page: 100,
-            owner,
-            repo
-          };
-
-          const [
+          const {
             open_issues,
             closed_issues,
             open_pull_requests,
             closed_pull_requests
-          ] = await Promise.all([
-
-            // open issues
-            octokit.paginate(
-              octokit.rest.issues.listForRepo,
-              {
-                ...params,
-                state: 'open'
-              },
-              response => response.data.filter(issue => !('pull_request' in issue))
-            ),
-
-            // closed issues, updated last 30 days
-            octokit.paginate(
-              octokit.rest.issues.listForRepo,
-              {
-                ...params,
-                state: 'closed',
-                since: new Date(syncClosedSince).toISOString()
-              },
-              response => response.data.filter(issue => !('pull_request' in issue))
-            ),
-
-            // open pulls, all
-            octokit.paginate(
-              octokit.rest.pulls.list,
-              {
-                ...params,
-                state: 'open'
-              }
-            ),
-
-            // closed pulls, updated last 30 days
-            octokit.paginate(
-              octokit.rest.pulls.list,
-              {
-                ...params,
-                state: 'closed'
-              },
-              (response, done) => {
-
-                const pulls = response.data;
-
-                const filtered = pulls.filter(
-                  pull => new Date(pull.updated_at).getTime() > syncClosedSince
-                );
-
-                if (filtered.length !== pulls.length) {
-                  done();
-                }
-
-                return filtered;
-              }
-            )
-          ]);
+          } = await backgroundSyncBackend.getRepositoryIssuesAndPulls(repository, syncClosedSince);
 
           for (const issueOrPull of [
             ...open_issues,
@@ -217,25 +150,11 @@ We automatically synchronize all repositories you granted us access to via the G
 
               const update = filterIssueOrPull(issueOrPull, repository);
 
-              const {
-                id
-              } = update;
+              foundIssues[update.id] = update;
 
-              const existingIssue = await store.getIssueById(id);
-
-              if (existingIssue && existingIssue.updated_at >= update.updated_at) {
-                foundIssues[id] = null;
-
-                log.debug({
-                  [type]: `${owner}/${repo}#${issueOrPull.number}`
-                }, 'skipping, as up-to-date');
-              } else {
-                foundIssues[id] = update;
-
-                log.debug({
-                  [type]: `${owner}/${repo}#${issueOrPull.number}`
-                }, 'scheduled for update');
-              }
+              log.debug({
+                [type]: `${owner}/${repo}#${issueOrPull.number}`
+              }, 'scheduled for update');
             } catch (err) {
               log.error({
                 err,
@@ -395,12 +314,21 @@ We automatically synchronize all repositories you granted us access to via the G
 
     // update changed issues
 
-    await Promise.all(pendingUpdates.map(update => store.updateIssue(update)));
+    const updateTasks = pendingUpdates.map(
+      update => store.updateIssue(update)
+    );
+
+    // returns the update for updated issues or undefined
+    // where issues were not stored (filtered out) - we only filter
+    // for actual updated issues
+    const updatedIssues = await Promise.all(updateTasks).then(updates => {
+      return updates.filter(update => update);
+    });
 
-    // emit background sync event for all found issues
+    // emit background sync event for all updated issues
 
     await syncDetails(
-      Object.keys(foundIssues),
+      updatedIssues,
       getSyncClosedDetailsSince(),
       getSyncOpenDetailsSince()
     );
@@ -437,10 +365,12 @@ We automatically synchronize all repositories you granted us access to via the G
     );
   }
 
-  function syncDetails(issueIds, syncClosedSince, syncOpenSince) {
+  function syncDetails(updatedIssues, syncClosedSince, syncOpenSince) {
+
+    const jobs = updatedIssues.map(async updatedIssue => {
 
-    const jobs = issueIds.map(async id => {
-      const issue = await store.getIssueById(id);
+      // ensure we fetch latest version of issue (to prevent de-sync)
+      const issue = await store.getIssueById(updatedIssue.id);
 
       if (!issue) {
         return;
@@ -468,7 +398,7 @@ We automatically synchronize all repositories you granted us access to via the G
     log.info('start');
 
     try {
-      const installations = await githubApp.getInstallations();
+      const installations = await backgroundSyncBackend.getInstallations();
 
       await doSync(installations);
 

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

@@ -0,0 +1,140 @@
+/**
+ * @typedef { import('../github-app/types.js').Installation } Installation
+ */
+
+/**
+ * This component fetches GitHub data required for background synchronization.
+ *
+ * @constructor
+ *
+ * @param {import('../github-client/GithubClient.js').default} githubClient
+ * @param {import('../github-app/GithubApp.js').default} githubApp
+ */
+export default function BackgroundSyncBackend(githubClient, githubApp) {
+
+  /**
+   * Return available installations.
+   *
+   * @return {Promise<Installation[]>}
+   */
+  async function getInstallations(installation) {
+    return githubApp.getInstallations();
+  }
+
+  /**
+   * Return repositories accessible to a GitHub app installation.
+   *
+   * @param {Installation} installation
+   *
+   * @return {Promise<Array>}
+   */
+  async function getInstallationRepositories(installation) {
+    const owner = installation.account.login;
+    const octokit = await githubClient.getOrgScoped(owner);
+
+    return octokit.paginate(
+      octokit.rest.apps.listReposAccessibleToInstallation,
+      { per_page: 100 }
+    );
+  }
+
+  /**
+   * Fetch issues and pull requests for a repository.
+   *
+   * @param {{ owner: { login: string }, name: string }} repository
+   * @param {number} syncClosedSince
+   *
+   * @return {Promise<{ open_issues: Array, closed_issues: Array, open_pull_requests: Array, closed_pull_requests: Array }>}
+   */
+  async function getRepositoryIssuesAndPulls(repository, syncClosedSince) {
+    const owner = repository.owner.login;
+    const octokit = await githubClient.getOrgScoped(owner);
+
+    const repo = repository.name;
+
+    const params = {
+      sort: /** @type { 'updated' } */ ('updated'),
+      direction: /** @type { 'desc' } */ ('desc'),
+      per_page: 100,
+      owner,
+      repo
+    };
+
+    const [
+      open_issues,
+      closed_issues,
+      open_pull_requests,
+      closed_pull_requests
+    ] = await Promise.all([
+
+      // open issues
+      octokit.paginate(
+        octokit.rest.issues.listForRepo,
+        {
+          ...params,
+          state: 'open'
+        },
+        response => response.data.filter(issue => !('pull_request' in issue))
+      ),
+
+      // closed issues, updated since syncClosedSince
+      octokit.paginate(
+        octokit.rest.issues.listForRepo,
+        {
+          ...params,
+          state: 'closed',
+          since: new Date(syncClosedSince).toISOString()
+        },
+        response => response.data.filter(issue => !('pull_request' in issue))
+      ),
+
+      // open pulls, all
+      octokit.paginate(
+        octokit.rest.pulls.list,
+        {
+          ...params,
+          state: 'open'
+        }
+      ),
+
+      // closed pulls, updated since syncClosedSince
+      octokit.paginate(
+        octokit.rest.pulls.list,
+        {
+          ...params,
+          state: 'closed'
+        },
+        (response, done) => {
+
+          const pulls = response.data;
+
+          const filtered = pulls.filter(
+            pull => new Date(pull.updated_at).getTime() > syncClosedSince
+          );
+
+          if (filtered.length !== pulls.length) {
+            done();
+          }
+
+          return filtered;
+        }
+      )
+    ]);
+
+    return {
+      open_issues,
+      closed_issues,
+      open_pull_requests,
+      closed_pull_requests
+    };
+  }
+
+
+  // api ///////////////////
+
+  this.getInstallations = getInstallations;
+
+  this.getInstallationRepositories = getInstallationRepositories;
+
+  this.getRepositoryIssuesAndPulls = getRepositoryIssuesAndPulls;
+}

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

@@ -1,6 +1,8 @@
 import BackgroundSync from './BackgroundSync.js';
+import BackgroundSyncBackend from './BackgroundSyncBackend.js';
 
 export default {
   __init__: [ 'backgroundSync' ],
-  backgroundSync: [ 'type', BackgroundSync ]
+  backgroundSync: [ 'type', BackgroundSync ],
+  backgroundSyncBackend: [ 'type', BackgroundSyncBackend ]
 };

package/lib/apps/{events-sync.js → events-sync/EventsSync.js}

@@ -5,7 +5,7 @@ import {
   filterRepository,
   getIdentifier,
   getKey
-} from '../filters.js';
+} from '../../filters.js';
 
 
 /**
@@ -13,14 +13,14 @@ import {
  *
  * @constructor
  *
- * @param {import('./webhook-events/WebhookEvents.js').default} webhookEvents
- * @param {import('../store.js').default} store
- * @param {import('../types.js').Logger} logger
+ * @param {import('../webhook-events/WebhookEvents.js').default} webhookEvents
+ * @param {import('../../store.js').default} store
+ * @param {import('../../types.js').Logger} logger
  */
 export default function EventsSync(webhookEvents, store, logger) {
 
   const log = logger.child({
-    name: 'wuffle:user-access'
+    name: 'wuffle:events-sync'
   });
 
   // issues /////////////////////
@@ -104,6 +104,8 @@ export default function EventsSync(webhookEvents, store, logger) {
     'pull_request.converted_to_draft',
     'pull_request.assigned',
     'pull_request.unassigned',
+    'pull_request.milestoned',
+    'pull_request.demilestoned',
     'pull_request.synchronize',
     'pull_request.closed',
     'pull_request.review_requested',

package/lib/apps/events-sync/index.js

@@ -0,0 +1,6 @@
+import EventsSync from './EventsSync.js';
+
+export default {
+  __init__: [ 'eventsSync' ],
+  eventsSync: [ 'type', EventsSync ]
+};

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

@@ -28,6 +28,7 @@ const RequiredEvents = [
 
 /**
  * @typedef {import('./types.js').Installation} Installation
+ * @typedef {import('../../types.js').Octokit} Octokit
  */
 
 /**
@@ -53,8 +54,19 @@ export default function GithubApp(config, app, logger, injector) {
 
   // cached data //////////////////
 
+  /**
+   * @type {Promise<Installation[]> | null}
+   */
   let installations = null;
+
+  /**
+   * @type {Promise<Record<string, Installation>> | null}
+   */
   let installationsByLogin = null;
+
+  /**
+   * @type {Promise<Record<string, Installation>> | null}
+   */
   let installationsById = null;
 
   // reactivity ////////////////////
@@ -71,12 +83,17 @@ export default function GithubApp(config, app, logger, injector) {
   // functionality /////////////////
 
   /**
-   * @return {Promise<import('../../types.js').Octokit>}
+   * @return {Promise<Octokit>}
    */
   function getAppScopedClient() {
     return app.auth();
   }
 
+  /**
+   * Return installations
+   *
+   * @return {Promise<Installation[]>}
+   */
   function getInstallations() {
     installations = installations || fetchInstallations().then(installations => {
 
@@ -90,6 +107,11 @@ export default function GithubApp(config, app, logger, injector) {
     return installations;
   }
 
+  /**
+   * Return installations by ID
+   *
+   * @return {Promise<Record<string, Installation>>}
+   */
   function getInstallationsById() {
 
     installationsById = installationsById || getInstallations().then(
@@ -106,9 +128,9 @@ export default function GithubApp(config, app, logger, injector) {
   /**
    * Get an installation for the given id.
    *
-   * @param {string} id
+   * @param {string|number} id
    *
-   * @return {Promise<Installation?>}
+   * @return {Promise<Installation | undefined>}
    */
   function getInstallationById(id) {
     return getInstallationsById().then(byId => {
@@ -119,7 +141,7 @@ export default function GithubApp(config, app, logger, injector) {
   /**
    * Return map of installations, grouped by org / login.
    *
-   * @return {Promise<Object<String, Installation>>}
+   * @return {Promise<Record<string, Installation>>}
    */
   function getInstallationsByLogin() {
 
@@ -155,6 +177,13 @@ export default function GithubApp(config, app, logger, injector) {
     });
   }
 
+  /**
+   * Return true if installation is enabled.
+   *
+   * @param {Installation} installation
+   *
+   * @return {Promise<boolean>}
+   */
   function isInstallationEnabled(installation) {
 
     const {
@@ -240,10 +269,10 @@ export default function GithubApp(config, app, logger, injector) {
   /**
    * Fetch active installations.
    *
-   * @return {Promise<Array<Installation>>} installations
+   * @return {Promise<Installation[]>} installations
    */
   function fetchInstallations() {
-    return /** @type Promise<Array<Installation>> */ (getAppScopedClient().then(
+    return /** @type Promise<Installation[]> */ (getAppScopedClient().then(
       octokit => octokit.paginate(
         octokit.rest.apps.listInstallations,
         { per_page: 100 }
@@ -253,13 +282,47 @@ export default function GithubApp(config, app, logger, injector) {
 
   // api ///////////////////
 
+  /**
+   * Return an application-scoped octokit client.
+   *
+   * @return {Promise<Octokit>}
+   */
   this.getAppScopedClient = getAppScopedClient;
 
+  /**
+   * Return true if installation is enabled.
+   *
+   * @param {Installation} installation
+   *
+   * @return {Promise<boolean>}
+   */
   this.isInstallationEnabled = isInstallationEnabled;
 
+  /**
+   * Get an installation for the given login.
+   *
+   * This method throws if an installation for the given login does not exist.
+   *
+   * @param {string} login
+   *
+   * @return {Promise<Installation>}
+   */
   this.getInstallationByLogin = getInstallationByLogin;
+
+  /**
+   * Get an installation for the given id.
+   *
+   * @param {string} id
+   *
+   * @return {Promise<Installation | undefined>}
+   */
   this.getInstallationById = getInstallationById;
 
+  /**
+   * Return installations
+   *
+   * @return {Promise<Installation[]>}
+   */
   this.getInstallations = getInstallations;
 
 }

package/lib/apps/github-comments/GithubComments.js

@@ -12,8 +12,16 @@ import gql from 'fake-tag';
  * @param {import('../../events.js').default} events
  * @param {import('../github-client/GithubClient.js').default} githubClient
  * @param {import('../../store.js').default} store
+ * @param {import('../issue-filter/IssueFilter.js').default} issueFilter
+ * @param {import('../../types.js').Logger } logger
  */
-export default function GithubComments(webhookEvents, events, githubClient, store) {
+export default function GithubComments(webhookEvents, events, githubClient, store, issueFilter, logger) {
+
+  const log = logger.child({
+    name: 'wuffle:github-comments'
+  });
+
+  const ifEnabled = issueFilter.createWebhookFilter(log);
 
   // issues /////////////////////
 
@@ -108,7 +116,7 @@ export default function GithubComments(webhookEvents, events, githubClient, stor
 
   webhookEvents.on([
     'issue_comment'
-  ], async ({ payload }) => {
+  ], ifEnabled(async ({ payload }) => {
     const {
       action,
       comment: _comment,
@@ -169,7 +177,7 @@ export default function GithubComments(webhookEvents, events, githubClient, stor
       ...commentedIssue,
       comments
     });
-  });
+  }));
 
 }
 

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

@@ -11,8 +11,16 @@ import { filterUser, filterPull } from '../../filters.js';
  * @param {import('../../events.js').default} events
  * @param {import('../github-client/GithubClient.js').default} githubClient
  * @param {import('../../store.js').default} store
+ * @param {import('../issue-filter/IssueFilter.js').default} issueFilter
+ * @param {import('../../types.js').Logger } logger
  */
-export default function GithubReviews(webhookEvents, events, githubClient, store) {
+export default function GithubReviews(webhookEvents, events, githubClient, store, issueFilter, logger) {
+
+  const log = logger.child({
+    name: 'wuffle:github-reviews'
+  });
+
+  const ifEnabled = issueFilter.createWebhookFilter(log);
 
   // issues /////////////////////
 
@@ -54,7 +62,7 @@ export default function GithubReviews(webhookEvents, events, githubClient, store
 
   webhookEvents.on([
     'pull_request_review'
-  ], async ({ payload }) => {
+  ], ifEnabled(async ({ payload }) => {
     const {
       action,
       review: _review,
@@ -102,7 +110,7 @@ export default function GithubReviews(webhookEvents, events, githubClient, store
       ...pull_request,
       reviews
     });
-  });
+  }));
 
 }
 

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

@@ -11,8 +11,16 @@ import { filterPull } from '../../filters.js';
  * @param {import('../../events.js').default} events
  * @param {import('../github-client/GithubClient.js').default} githubClient
  * @param {import('../../store.js').default} store
+ * @param {import('../issue-filter/IssueFilter.js').default} issueFilter
+ * @param {import('../../types.js').Logger } logger
  */
-export default function GithubStates(webhookEvents, events, githubClient, store) {
+export default function GithubStatuses(webhookEvents, events, githubClient, store, issueFilter, logger) {
+
+  const log = logger.child({
+    name: 'wuffle:github-statuses'
+  });
+
+  const ifEnabled = issueFilter.createWebhookFilter(log);
 
   // issues /////////////////////
 
@@ -89,7 +97,7 @@ export default function GithubStates(webhookEvents, events, githubClient, store)
   webhookEvents.on([
     'pull_request.opened',
     'pull_request.synchronize'
-  ], async ({ payload }) => {
+  ], ifEnabled(async ({ payload }) => {
 
     const {
       pull_request: _pull_request,
@@ -108,7 +116,7 @@ export default function GithubStates(webhookEvents, events, githubClient, store)
       id,
       statuses
     });
-  });
+  }));
 
   webhookEvents.on([
     'status'

package/lib/apps/issue-filter/IssueFilter.js

@@ -2,12 +2,14 @@
  * @typedef { { ignoreFilter?: string } } StoreFilterConfig
  */
 
+import { filterIssueOrPull } from '../../filters.js';
+import { issueIdent } from '../../util/meta.js';
+
 /**
  * An component that configures the store to filter certain elements,
  * effectively making them invisible to the board and its users.
  *
  * @param { StoreFilterConfig } config
- *
  * @param { import('../../store.js').default } store
  * @param { import('../search/Search.js').default } search
  * @param { import('../../types.js').Logger } logger
@@ -21,21 +23,50 @@ export default function IssueFilter(config, store, search, logger) {
   /**
    * @type { import('../search/Search.js').FilterFn }
    */
-  let ignoreFilter = (issue) => false;
+  let isIgnored = (issue) => false;
 
 
   if ('ignoreFilter' in config) {
     const ignoreFilterFn = search.buildFilterFn(config.ignoreFilter);
 
     if (ignoreFilterFn) {
-      ignoreFilter = ignoreFilterFn;
+      isIgnored = ignoreFilterFn;
 
-      store.setIgnoreFilter(ignoreFilter);
+      store.setIgnoreFilter(isIgnored);
     } else {
       log.warn('unparseable <ignoreFilter> - please correct your board configuration');
     }
   }
 
+  /**
+   * @param {import('../../types.js').Logger } log
+   *
+   * @return { (filterFn) => (any) => any }
+   */
+  function createWebhookFilter(log) {
+
+    return function ifEnabled(webhookHandlerFn) {
+
+      return (context) => {
+
+        const payload = context.payload;
+
+        const issueOrPull = filterIssueOrPull(
+          payload.issue || payload.pull_request,
+          payload.repository
+        );
+
+        if (isIgnored(issueOrPull)) {
+          log.debug({ issue: issueIdent(issueOrPull) }, 'issue matching ignore filter');
+
+          return;
+        }
+
+        return webhookHandlerFn(context);
+      };
+    };
+  };
+
   /**
    * Figure whether the issue shall be ignored (by ignore filter rules)
    *
@@ -43,8 +74,12 @@ export default function IssueFilter(config, store, search, logger) {
    *
    * @return {boolean} true, if issue shall be ignored
    */
-  this.isIgnored = function(issue) {
-    return ignoreFilter(issue);
-  };
+  this.isIgnored = isIgnored;
 
+  /**
+   * @param {import('../../types.js').Logger } log
+   *
+   * @return { (filterFn) => (any) => any }
+   */
+  this.createWebhookFilter = createWebhookFilter;
 }

package/lib/apps/log-events.js

@@ -27,18 +27,18 @@ export default function LogEvents(logger, webhookEvents) {
 
   log.info('dumping webhook events to %s', path.resolve(eventsDir));
 
-  function write(event, payload) {
+  function write(name, payload) {
 
     const {
       action
     } = payload;
 
-    const name = action ? `${event}.${action}` : event;
+    const eventName = action ? `${name}.${action}` : name;
 
-    const data = JSON.stringify({ event, payload }, null, '  ');
+    const data = JSON.stringify({ name, payload }, null, '  ');
 
     return fs.mkdir(eventsDir, { recursive: true }).then(() => {
-      const fileName = path.join(eventsDir, `${Date.now()}-${counter++}-${name}.json`);
+      const fileName = path.join(eventsDir, `${Date.now()}-${counter++}-${eventName}.json`);
 
       return fs.writeFile(fileName, data, 'utf8');
     });

package/lib/apps/{reindex-store.js → reindex-store/ReindexStore.js}

@@ -1,16 +1,16 @@
 import {
   getPackageVersion,
   hash
-} from '../util/index.js';
+} from '../../util/index.js';
 
 
 /**
  * @constructor
  *
  * @param {any} config
- * @param {import('../types.js').Logger} logger
- * @param {import('../events.js').default} events
- * @param {import('../store.js').default} store
+ * @param {import('../../types.js').Logger} logger
+ * @param {import('../../events.js').default} events
+ * @param {import('../../store.js').default} store
  */
 export default function ReindexStore(config, logger, events, store) {
 
@@ -48,6 +48,8 @@ export default function ReindexStore(config, logger, events, store) {
       log.info('config changed');
 
       await reindexStore();
+    } else {
+      log.debug('config unchanged - skipping reindexing');
     }
   });
 

package/lib/apps/reindex-store/index.js

@@ -0,0 +1,6 @@
+import ReindexStore from './ReindexStore.js';
+
+export default {
+  __init__: [ 'reindexStore' ],
+  reindexStore: [ 'type', ReindexStore ]
+};

package/lib/index.js

@@ -11,7 +11,7 @@ const appModules = [
       ? import('./apps/dump-store/s3/index.js')
       : import('./apps/dump-store/local/index.js')
   ),
-  import('./apps/events-sync.js'),
+  import('./apps/events-sync/index.js'),
   import('./apps/github-app/index.js'),
   import('./apps/github-issues/index.js'),
   import('./apps/github-comments/index.js'),
@@ -24,11 +24,11 @@ const appModules = [
   import('./apps/user-access/index.js'),
   import('./apps/search/index.js'),
   import('./apps/background-sync/index.js'),
-  import('./apps/automatic-dev-flow.js'),
+  import('./apps/automatic-dev-flow/index.js'),
   import('./apps/auth-routes/index.js'),
   import('./apps/board-api-routes/index.js'),
   import('./apps/board-routes.js'),
-  import('./apps/reindex-store.js')
+  import('./apps/reindex-store/index.js')
 ];
 
 import loadConfig from './load-config.js';
@@ -78,17 +78,25 @@ export default function Wuffle(app, { getRouter }) {
     });
 
     const coreModule = {
-      'app': [ 'value', app ],
       'config': [ 'value', config ],
-      'router': [ 'value', router ],
       'logger': [ 'value', logger ],
       'columns': [ 'type', Columns ],
       'store': [ 'type', Store ],
       'events': [ 'value', events ]
     };
 
+    const webModule = {
+      'router': [ 'value', router ]
+    };
+
+    const probotModule = {
+      'app': [ 'value', app ],
+    };
+
     const injector = new AsyncInjector([
       coreModule,
+      probotModule,
+      webModule,
       ...modules
     ]);
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wuffle",
-  "version": "0.73.2",
+  "version": "0.74.0",
   "description": "A multi-repository task board for GitHub issues",
   "author": {
     "name": "Nico Rehwaldt",
@@ -35,7 +35,7 @@
     "all": "run-s lint test",
     "dev": "nodemon",
     "start": "NODE_ENV=production node ./bin/wuffle",
-    "test": "mocha 'test/**/*.js' --exit",
+    "test": "mocha -r test/helpers/globals.js 'test/**/*.js' --exit",
     "lint": "run-s lint:*",
     "lint:eslint": "eslint .",
     "lint:types": "tsc --pretty",
@@ -61,13 +61,14 @@
     "@types/compression": "^1.8.1",
     "@types/express-session": "^1.19.0",
     "@types/mocha": "^10.0.10",
+    "@types/sinon": "^21.0.1",
     "chai": "^6.2.2",
     "graphql": "^16.13.2",
     "mocha": "^11.7.5",
     "nock": "^14.0.13",
     "nodemon": "^3.1.14",
     "npm-run-all2": "^8.0.4",
-    "sinon": "^21.1.2",
+    "sinon": "^22.0.0",
     "sinon-chai": "^4.0.0",
     "typescript": "^5.9.3"
   },
@@ -81,5 +82,6 @@
     "app.yml",
     "index.js",
     "wuffle.config.example.js"
-  ]
+  ],
+  "gitHead": "ea9ac74526b02f9834b5e926df0e8e82536eba07"
 }