feat(bot): Allow reactions by bots

[nickvergessen]

☑️ Resolves

• Task from 🐣 Bots part 2 #10094 (comment)

🏁 Checklist

• ⛑️ Tests (unit and/or integration) are included or not required
• 📘 API documentation in docs/ has been updated or is not required
• 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required
• 🔖 Capability is added or not needed

[Altahrim]

I am looking for a way to avoid repetition here and keep the same logic in one place. Maybe a wrapper like this:

private function handleInvalidArgumentException(\InvalidArgumentException $e) {
	$response = new DataResponse([], $e->getCode());
	if ($e->getCode() === Http::STATUS_UNAUTHORIZED) {
		$response->throttle(['action' => 'bot']);
	}
	return $response;
}

Block from l. 130 to l. 138 is used several times. I wonder we could extract it in a method, but then we will have something like:

$bot = $this->loadBot($token, $message);
if ($bot instanceof DataResponse) {
	return $bot;
}

And I don't like it ^^

[nickvergessen]

yeah, I don't like mixed return types, that's why I went for the throwing instead.