Add some comments/clarifications and fix missing function param

Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>

Author: lukasdotcom

lib/Controller/ChattyLLMController.php

@@ -655,7 +655,7 @@ public function regenerateForSession(int $sessionId, int $messageId): JSONRespon
 	}
 
 	/**
-	 * Check the status of a generation task
+	 * Check the status of a generation task. The value of slow_pickup will be set to true if the task is not being picked up.
 	 *
 	 * Used by the frontend to poll a generation task status. If the task succeeds, a new message is stored and returned.
 	 *
@@ -711,7 +711,7 @@ public function checkMessageGenerationTask(int $taskId, int $sessionId): JSONRes
 			}
 		} elseif ($task->getstatus() === Task::STATUS_RUNNING || $task->getstatus() === Task::STATUS_SCHEDULED) {
 			$startTime = $task->getStartedAt() ?? time();
-			$slowPickup = $task->getScheduledAt() + 60 * 5 < $startTime;
+			$slowPickup = ($task->getScheduledAt() + (60 * 5)) < $startTime;
 			return new JSONResponse(['task_status' => $task->getstatus(), 'slow_pickup' => $slowPickup], Http::STATUS_EXPECTATION_FAILED);
 		} elseif ($task->getstatus() === Task::STATUS_FAILED || $task->getstatus() === Task::STATUS_CANCELLED) {
 			return new JSONResponse(['error' => 'task_failed_or_canceled', 'task_status' => $task->getstatus()], Http::STATUS_BAD_REQUEST);

openapi.json

@@ -3544,7 +3544,7 @@
         "/ocs/v2.php/apps/assistant/chat/check_generation": {
             "get": {
                 "operationId": "chattyllm-check-message-generation-task",
-                "summary": "Check the status of a generation task",
+                "summary": "Check the status of a generation task. The value of slow_pickup will be set to true if the task is not being picked up.",
                 "description": "Used by the frontend to poll a generation task status. If the task succeeds, a new message is stored and returned.",
                 "tags": [
                     "chat_api"

package-lock.json

@@ -48,10 +48,10 @@
 		"@nextcloud/event-bus": "^3.1.0",
 		"@nextcloud/files": "^3.11.0",
 		"@nextcloud/initial-state": "^2.0.0",
-		"@nextcloud/l10n": "^3.1.0",
+		"@nextcloud/l10n": "~3.3.0",
 		"@nextcloud/moment": "^1.3.1",
 		"@nextcloud/router": "^3.0.0",
-		"@nextcloud/vue": "^9.0.0-rc.4",
+		"@nextcloud/vue": "^9.0.0-rc.5",
 		"extendable-media-recorder": "^9.2.11",
 		"extendable-media-recorder-wav-encoder": "^7.0.129",
 		"moment": "^2.30.1",

package.json

@@ -110,7 +110,7 @@ export default {
 
 	computed: {
 		tooLongForScheduling() {
-			return this.scheduledAt !== null && this.scheduledAt + 60 * 5 < this.now
+			return this.scheduledAt !== null && (this.scheduledAt + (60 * 5)) < this.now
 		},
 		TASK_STATUS_STRING() {
 			return TASK_STATUS_STRING
@@ -133,6 +133,7 @@ export default {
 	},
 
 	mounted() {
+		// Need to use a timer to update the state for now otherwise the component won't re-render when the time changes
 		this.timer = setInterval(() => {
 			console.debug('scheduledAt', this.scheduledAt)
 			console.debug('status', this.taskStatus)

src/components/RunningEmptyContent.vue

@@ -196,7 +196,7 @@ export default {
 					this.task.completionExpectedAt = updatedTask.completionExpectedAt
 					this.task.scheduledAt = updatedTask.scheduledAt
 
-					pollTask(updatedTask.id, this).then(finishedTask => {
+					pollTask(updatedTask.id, this, this.updateTask).then(finishedTask => {
 						console.debug('pollTask.then', finishedTask)
 						if (finishedTask.status === TASK_STATUS_STRING.successful) {
 							this.task.output = finishedTask?.output