Reacting app_mention and message events about slack-edge

[seratch]

Hi @t6adev, thanks again. This is because your app subscribes both app_mention and message events. Regardless whether your app subscribes app_mention events, message events delivers all visible message events to your app endpoint. If you want to skip the bot mentioned message events, you can check message.text string if it does not have the mention.

Let me close this issue now, but if you have further to ask, feel free to write in!

[t6adev]

Ok, thank you.
Since the app.use (= postAuthorizeMiddleware) runs before handling events, I believe we can mark the mentioned message inside Cloudflare worker's fetch (it's messy😅)

export default {
  async fetch(request: Request, env: Env, ctx: ExecutionContext): Promise<Response> {
    ...
    const current = { isAppMention: false };
    app.use(async ({ body }) => {
      const slackRequestBody = body as SlackRequestBody;
      if (slackRequestBody.type === PayloadType.EventsAPI) {
        if (slackRequestBody.event?.type === 'app_mention') {
          current.isAppMention = true;
        }
      }
    });
    ...
  }
}

slack-edge/src/app.ts

Lines 638 to 647 in 2785568

	for (const middlware of this.postAuthorizeMiddleware) {
	const response = await middlware(baseRequest);
	if (response) {
	return toCompleteResponse(response);
	}
	}
	const payload = body as SlackRequestBody;
	if (body.type === PayloadType.EventsAPI) {

[seratch]

Since the fetch handler can be invoked per request, your approach never works as you expect. In other words, your fetch function is always stateless (as long as you don't store the state in a database) so that your message listener never knows the current was set by a different app_mention request.

Alternatively, your message event listener can check if your app's bot user ID is mentioned in the message event. This is not specific to this slack-edge library. When you build a Slack app that subscribes both app_mention and message events, your code is responsible to handle these events.

If you need more assistance this, I am happy to help you in Japanese. Feel free to mention @seratch_ja (@seratch for English) on X. Since this is the issue tracker for managing this library's issues (bugs, enhancement ideas), I would like to stop discussing this here. It'd be appreciated if you could understand this.

[t6adev]

Thank you, I respect your opinion on how you manage this repo. (I also prefer to discuss something on the GitHub discussion page.)