Fix the issue of no reply in a threaded mentioned message

[maxshine]

Dear reviewer

Issue Summary

This PR is to fix an issue that the bot ignore following mentioned messages in a threaded conversation
The opened issue is #70, which this PR is targeted.

Fix details

The is_no_mention_thread method defined in slack_ops.py returns the negated value than the correct one. This cause the app deems the message in a thread as a not mentioned one and ignore it. So a negate operation was added to fix it.

Would you please kindly review it?

Thanks.

[seratch]

Hi @maxshine, thanks for sharing the issue. The pattern you reported should be supported, but changing this method's behavior breaks an intenational design of this app, which does not allow involving ChatGPT in the middle of a thread convo (this could be pretty annoying because after that, the bot will reply to every single message in the thread.)

The internal method name looks confusing, so I will refactor the code to suppor your use case later

[maxshine]

Hi @maxshine, thanks for sharing the issue. The pattern you reported should be supported, but changing this method's behavior breaks an intenational design of this app, which does not allow involving ChatGPT in the middle of a thread convo (this could be pretty annoying because after that, the bot will reply to every single message in the thread.)

The internal method name looks confusing, so I will refactor the code to suppor your use case later

Hey @seratch

Thank you for the reply.

I think the current logic is to let bot reply to the messages mentioning itself, not to every single one in the thread. The is_not_mention_thread check the immediate parent message if it has the bot user id, which is the latest message sent in the thread.

Can you please evaluate it if this is a reasonable case?

Thank you.

[seratch]

I mean, when you use this app in a channel, this app requires mentioning the app's bot user to start the conversation thread with ChatGPT. In the thread, you don't need to mention the bot. Modifying the internal method breaks the behavior because the method is used in the app_mention event handler too.

[maxshine]

I mean, when you use this app in a channel, this app requires mentioning the app's bot user to start the conversation thread with ChatGPT. In the thread, you don't need to mention the bot. Modifying the internal method breaks the behavior because the method is used in the app_mention event handler too.

Thanks for the clarification. I think I get your point. :)

[seratch]

It seems this not relevant. Let me close this PR. Thank you for your time and efforts on this!