Automatically invoke the InstallationStore's deleteInstallation function for relevant events

[misscoded]

Description

Currently, deleteInstallation of a developer-provided InstallationStore must be invoked manually. It would be more intuitive behavior to have it invoked automatically when the app_uninstalled and token_revoked events are received.

Aside from a couple of community members raising questions around this (#1201, #1112), it's my understanding that the Java and Python implementations already support this.

What type of issue is this? (place an x in one of the [ ])

• bug
• enhancement (feature request)
• question
• documentation related
• example code related
• testing related
• discussion

Requirements (place an x in each of the [ ])

• I've read and understood the Contributing guidelines and have done my best effort to follow them.
• I've read and agree to the Code of Conduct.
• I've searched for any related issues and avoided creating a duplicate issue.

[seratch]

FYI, we've already implemented this in bolt-python and bolt-java this way:

• https://github.com/slackapi/bolt-python/releases/tag/v1.5.0
  • Fix #254 Add built-in tokens_revoked/app_uninstalled event handlers bolt-python#289
• https://github.com/slackapi/java-slack-sdk/releases/tag/v1.7.0
  • Add built-in tokens_revoked/app_uninstalled event handlers java-slack-sdk#703

[github-actions]

👋 It looks like this issue has been open for 30 days with no activity. We'll mark this as stale for now, and wait 10 days for an update or for further comment before closing this issue out. If you think this issue needs to be prioritized, please comment to get the thread going again! Maintainers also review issues marked as stale on a regular basis and comment or adjust status if the issue needs to be reprioritized.

[uap-dev]

Excited for this change for the js sdk!

[ElMatella]

Hey, thank you for this issue. I have a question until this gets implemented, how am I suppose to call deleteInstallation manually?

Currently, I use the app_uninstalled event like this:

app.event('app_uninstalled', async ({ event, logger, context }) => {
  try {
    await database.doc(context.enterpriseId || context.teamId).delete()
  }
  catch (error) {
    logger.error(error);
  }
});

I don't use my installationStore.deleteInstallation at all.. And I'm not sur that the code I made supports enterprise correctly. Could you give me a hand on that one?

Thank you very much

[seratch]

@ElMatella I would suggest implementing the deleteInstallation method for easier migration when the built-in handlers are released. You can check the type definitions related to the method signature below:

• https://github.com/slackapi/node-slack-sdk/blob/%40slack/oauth%402.5.2/packages/oauth/src/installation-query.ts
• https://github.com/slackapi/node-slack-sdk/blob/%40slack/oauth%402.5.2/packages/oauth/src/installation-stores/interface.ts#L13-L14