fix: handle stale enum values when retrying tasks

When retrying a task from history, stale enum values (e.g. a removed
model) were sent directly to the server, causing a ValidationException.
The error was displayed as a generic unhelpful message.

This commit fixes four related issues:

- Fix typo in TaskTypeFields.vue that prevented optional enum fields
  from ever receiving their dropdown options (e.g. model selection in
  Advanced options was completely broken)

- Change "Try again" from history sidebar to load the task into the
  form instead of auto-submitting, so the user can review and fix
  stale values before resubmitting

- Add a watch in EnumField.vue that clears enum values not matching
  any available option, preventing stale values from persisting in
  the form state

- Show the server's actual validation error message in the toast
  notification instead of a generic "Something went wrong" message

Closes #400

Signed-off-by: Alexander Piskun <bigcat88@icloud.com>

Author: bigcat88

src/assistant.js

@@ -170,8 +170,12 @@ export async function openAssistantForm({
 				.catch(error => {
 					view.loading = false
 					view.showSyncTaskRunning = false
-					console.error('Assistant scheduling error', error?.response?.data?.ocs?.data?.message)
-					showError(t('assistant', 'Assistant error') + ': ' + t('assistant', 'Something went wrong when scheduling the task'))
+					const serverMessage = error?.response?.data?.ocs?.data?.message
+					console.error('Assistant scheduling error', serverMessage)
+					showError(
+						t('assistant', 'Assistant error') + ': '
+						+ (serverMessage || t('assistant', 'Something went wrong when scheduling the task')),
+					)
 				})
 		}
 		modalMountPoint.addEventListener('sync-submit', (data) => {
@@ -612,8 +616,12 @@ export async function openAssistantTask(
 			.catch(error => {
 				view.loading = false
 				view.showSyncTaskRunning = false
-				console.error('Assistant scheduling error', error?.response?.data?.ocs?.data?.message)
-				showError(t('assistant', 'Assistant error') + ': ' + t('assistant', 'Something went wrong when scheduling the task'))
+				const serverMessage = error?.response?.data?.ocs?.data?.message
+				console.error('Assistant scheduling error', serverMessage)
+				showError(
+					t('assistant', 'Assistant error') + ': '
+					+ (serverMessage || t('assistant', 'Something went wrong when scheduling the task')),
+				)
 			})
 	}
 	modalMountPoint.addEventListener('sync-submit', (data) => {

src/components/AssistantTextProcessingForm.vue

@@ -532,7 +532,10 @@ export default {
 			this.$emit('action-button-clicked', { button, output: this.myOutputs })
 		},
 		onHistoryTryAgain(e) {
-			this.$emit('try-again', e)
+			// Load the task into the form instead of auto-submitting.
+			// This lets the user review and fix stale enum values (e.g. removed models)
+			// before resubmitting.
+			this.$emit('load-task', e)
 		},
 		onHistoryLoadTask(e) {
 			this.$emit('load-task', e)

src/components/fields/EnumField.vue

@@ -70,6 +70,17 @@ export default {
 	},
 
 	watch: {
+		selectValue: {
+			handler(newVal) {
+				// If the current value doesn't match any available option, clear it.
+				// This handles stale enum values (e.g. a removed model) when loading
+				// a task from history.
+				if (this.value !== null && this.value !== undefined && this.value !== '' && newVal === undefined) {
+					this.$emit('update:value', undefined)
+				}
+			},
+			immediate: true,
+		},
 	},
 
 	mounted() {

src/components/fields/TaskTypeFields.vue

@@ -130,7 +130,7 @@ export default {
 		getOptionalInputFieldOptions(field, key) {
 			if (field.type === 'Enum'
 				&& this.optionalShapeOptions !== null
-				&& !Array.isArray(this.optionalShapeOptionsshapeOptions)
+				&& !Array.isArray(this.optionalShapeOptions)
 				&& this.optionalShapeOptions[key]
 			) {
 				return this.optionalShapeOptions[key]

src/views/AssistantPage.vue

@@ -151,8 +151,12 @@ export default {
 				.catch(error => {
 					this.loading = false
 					this.showSyncTaskRunning = false
-					console.error('Assistant scheduling error', error?.response?.data?.ocs?.data?.message)
-					showError(t('assistant', 'Assistant error') + ': ' + t('assistant', 'Something went wrong when scheduling the task'))
+					const serverMessage = error?.response?.data?.ocs?.data?.message
+					console.error('Assistant scheduling error', serverMessage)
+					showError(
+						t('assistant', 'Assistant error') + ': '
+						+ (serverMessage || t('assistant', 'Something went wrong when scheduling the task')),
+					)
 				})
 				.then(() => {
 				})