fixup! feat: ai message summary

Signed-off-by: SebastianKrupinski <[email protected]>
nextcloud · Dec 17, 2024 · e2f47a5 · e2f47a5
1 parent ab9040d
commit e2f47a5
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 7 deletions.
diff --git a/lib/Listener/TaskProcessingListener.php b/lib/Listener/TaskProcessingListener.php
@@ -44,9 +44,21 @@ public function handle(Event $event): void {
 			return;
 		}
 
-		[$type, $id] = explode(':', $task->getCustomId());
-		$userId = $task->getUserId();
-		$summary = $task->getOutput()['output'];
+		if ($task->getCustomId() !== null) {
+			[$type, $id] = explode(':', $task->getCustomId());
+		} else {
+			$this->logger->info('Error handling task processing event custom id missing');
+		}
+		if ($task->getUserId() !== null) {
+			$userId = $task->getUserId();
+		} else {
+			$this->logger->info('Error handling task processing event user id missing');
+		}
+		if ($task->getOutput() !== null && isset($task->getOutput()['output'])) {
+			$summary = $task->getOutput()['output'];
+		} else {
+			$this->logger->info('Error handling task processing event output missing');
+		}
 
 		if ($type === 'message') {
 			$this->handleMessageSummary($userId, (int)$id, $summary);

diff --git a/lib/Service/AiIntegrations/AiIntegrationsService.php b/lib/Service/AiIntegrations/AiIntegrationsService.php
@@ -63,11 +63,8 @@ public function __construct(
 	 *
 	 * @param Account $account
 	 * @param array<Message> $messages
-	 * @param Horde_Imap_Client_Socket $client
 	 *
-	 * @return null|string
-	 *
-	 * @throws ServiceException
+	 * @return void
 	 */
 	public function summarizeMessages(Account $account, array $messages): void {
 		try {