feat(bots): Add events for enabling and disabling bots

[sanskar-soni-9]

☑️ Resolves

• Fix Requiring an event whenever a bot is added to or removed from a chatroom #11461

Overview

• The PR introduces BotEnabledEvent and BotDisabledEvent.
• These events are dispatched when a bot is enabled or disabled through controller or occ.
• And sends an asynchronous request to the bot server when these events occur.

This allows the bot server to be notified when a bot is enabled or disabled, enabling it to take appropriate action.

Demo Example

2024-06-20.17-30-59.mp4

🏁 Checklist

• ⛑️ Tests (unit and/or integration) are included or not possible
• 📘 API documentation in docs/ has been updated or is not required
• 🔖 Capability is added or not needed

[nickvergessen]

This should also be documented in https://github.com/nextcloud/spreed/blob/main/docs/bots.md if possible

[sanskar-soni-9]

@nickvergessen resolved all the suggestions and also updated the docs please verify, specially the bots.md :)

[nickvergessen]

Thanks @sanskar-soni-9
Looks good now and I set it to merge after CI ran.

[sanskar-soni-9]

fixed linting

[sanskar-soni-9]

@nickvergessen Integration test is failing, with occ talk:bot:setup in case of invalid room token, the value of room with token will be null, so in case of federatedConvesation and RoomNotFound should it directly return from function here instead of ignoring it and adding bot with invalid token?

[nickvergessen]

And similarly we need to check if the room existed here:

Suggested change

	$event = new BotDisabledEvent($this->roomManager->getRoomByToken($token), $botServer);
	try {
	$room = $this->roomManager->getRoomByToken($token);
	} catch (RoomNotFoundException) {
	continue;
	}
	$event = new BotDisabledEvent($room, $botServer);

(+ importing the OCA\Talk\Exceptions\RoomNotFoundException exception)

[sanskar-soni-9]

oops dont know how that happened, let me fix this

[sanskar-soni-9]

also rebased to the current head

[nickvergessen]

oops dont know how that happened, let me fix this

It was always broken, just was not a problem so far as the missing room object was not attempted to be used.

[sanskar-soni-9]

It was always broken, just was not a problem so far as the missing room object was not attempted to be used.

No no I used both reset HEAD~ and --amend flag so previous commit was also included 😅, I fixed that now

[github-actions]

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)