small formatting changes to make the code in line with the existing code

Signed-off-by: Anna Visman <a.c.visman@student.tudelft.nl>

Author: annaneliaa

lib/Controller/ChattyLLMController.php

@@ -480,13 +480,14 @@ public function deleteMessage(int $messageId, int $sessionId): JSONResponse {
 	#[NoAdminRequired]
 	#[OpenAPI(scope: OpenAPI::SCOPE_DEFAULT, tags: ['chat_api'])]
 	public function searchMessages(string $query): JSONResponse {
+
 		try {
 			$result = $this->chatService->searchMessages($this->userId, $query);
 			return new JSONResponse($result);
 		} catch (InternalException $e) {
 			$this->logger->warning('Failed to search chat messages', ['exception' => $e]);
 			return new JSONResponse(['error' => $this->l10n->t('Failed to search chat messages')], Http::STATUS_INTERNAL_SERVER_ERROR);
-		} catch (UnauthorizedException $e) {
+		} catch (\OCA\Assistant\Service\UnauthorizedException $e) {
 			return new JSONResponse(['error' => $this->l10n->t('User not logged in')], Http::STATUS_UNAUTHORIZED);
 		}
 	}

lib/Db/ChattyLLM/MessageMapper.php

@@ -193,18 +193,9 @@ public function searchMessages(string $userId, string $query, int $limit = 100):
 		$qb = $this->db->getQueryBuilder();
 		$qb->select(Message::$columns)
 			->from($this->getTableName(), 'm')
-			->join('m', 'assistant_chat_sns', 's',
-				$qb->expr()->eq('m.session_id', 's.id')
-			)
-			->where($qb->expr()->eq('s.user_id',
-				$qb->createPositionalParameter($userId, IQueryBuilder::PARAM_STR)
-			))
-			->andWhere($qb->expr()->iLike('m.content',
-				$qb->createPositionalParameter(
-					'%' . $this->db->escapeLikeParameter($query) . '%',
-					IQueryBuilder::PARAM_STR
-				)
-			))
+			->join('m', 'assistant_chat_sns', 's', $qb->expr()->eq('m.session_id', 's.id'))
+			->where($qb->expr()->eq('s.user_id', $qb->createPositionalParameter($userId, IQueryBuilder::PARAM_STR)))
+			->andWhere($qb->expr()->iLike('m.content', $qb->createPositionalParameter('%' . $this->db->escapeLikeParameter($query) . '%', IQueryBuilder::PARAM_STR)))
 			->orderBy('m.timestamp', 'DESC')
 			->setMaxResults($limit);
 

lib/Service/ChatService.php

@@ -177,6 +177,7 @@ public function searchMessages(?string $userId, string $query): array {
 		if ($userId === null) {
 			throw new UnauthorizedException($this->l10n->t('Unauthorized'));
 		}
+		// For empty queries return two empty lists right away
 		if (trim($query) === '') {
 			return ['messages' => [], 'sessionIds' => []];
 		}
@@ -189,7 +190,7 @@ public function searchMessages(?string $userId, string $query): array {
 			array_map(fn(Message $m) => $m->getSessionId(), $messages)
 		));
 		return [
-			'messages' => array_map(fn(Message $m) => $m->jsonSerialize(), $messages),
+			'messages' => array_map(fn(Message $m) => $m->jsonSerialize(), $messages), // convert Message objects into plain arrays
 			'sessionIds' => $sessionIds,
 		];
 	}