fixup! add thread summary ui

Signed-off-by: hamza221 <[email protected]>
nextcloud · Aug 5, 2023 · b029448 · b029448
1 parent 652418a
commit b029448
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 13 deletions.
diff --git a/appinfo/routes.php b/appinfo/routes.php
@@ -367,7 +367,7 @@
 		],
 		[
 			'name' => 'thread#summarize',
-			'url' => '/api/thread/{id}/summary/',
+			'url' => '/api/thread/{id}/summary',
 			'verb' => 'GET'
 		],
 		[

diff --git a/lib/Controller/ThreadController.php b/lib/Controller/ThreadController.php
@@ -125,23 +125,26 @@ public function delete(int $id): JSONResponse {
 	/**
 	 * @NoAdminRequired
 	 *
-	 * @param string $id
-	 * @param int $accountId
+	 * @param int $id
 	 *
 	 * @return JSONResponse
 	 */
-	public function summarize(string $id, int $accountId): JSONResponse {
+	public function summarize(int $id): JSONResponse {
 		try {
-			$account = $this->accountService->find($this->currentUserId, $accountId);
-			$messages = $this->mailManager->getThread($account, $id);
-		} catch (ClientException $e) {
+			$message = $this->mailManager->getMessage($this->currentUserId, $id);
+			$mailbox = $this->mailManager->getMailbox($this->currentUserId, $message->getMailboxId());
+			$account = $this->accountService->find($this->currentUserId, $mailbox->getAccountId());
+		} catch (DoesNotExistException $e) {
 			return new JSONResponse([], Http::STATUS_FORBIDDEN);
 		}
-
+		if (empty($message->getThreadRootId())) {
+			return new JSONResponse([], Http::STATUS_NOT_FOUND);
+		}
+		$thread = $this->mailManager->getThread($account, $message->getThreadRootId());
 		try {
 			$summary = $this->aiIntergrationsService->summarizeThread(
-				$id,
-				$messages,
+				$message->getThreadRootId(),
+				$thread,
 				$this->currentUserId,
 			);
 		} catch (\Throwable $e) {

diff --git a/src/components/Thread.vue b/src/components/Thread.vue
@@ -176,7 +176,7 @@ export default {
 			if (this.thread.length < 2 && !this.enabledThreadSummary) return
 
 			this.summaryLoading = true
-			const summary = await summarizeThread(this.thread[0].threadRootId, this.accountId)
+			const summary = await summarizeThread(this.thread[0].databaseId)
 			this.summaryText = summary
 			this.summaryLoading = false
 		},

diff --git a/src/service/AiIntergrationsService.js b/src/service/AiIntergrationsService.js
@@ -2,9 +2,8 @@ import { generateUrl } from '@nextcloud/router'
 import axios from '@nextcloud/axios'
 
 export const summarizeThread = async (threadId, accountId) => {
-	const url = generateUrl('/apps/mail/api/thread/{threadId}/summary?accountId={accountId}', {
+	const url = generateUrl('/apps/mail/api/thread/{threadId}/summary', {
 		threadId,
-		accountId,
 	})
 
 	const resp = await axios.get(url)