@swiss-ai-hub/web 0.318.0 → 0.319.0

package/README.md

@@ -155,8 +155,8 @@ The same single-instance requirement applies to **PrimeVue** (its theme system i
 > { "dependencies": { "vue-router": "4.6.4" } }
 > ```
 >
-> Verify with `pnpm why vue-router` (`@vueuse/router` must show `4.6.4`) and `pnpm build` (`nuxt generate` must succeed —
-> forcing a single `vue-router` major instead breaks `@nuxtjs/i18n`, which needs 5.x).
+> Verify with `pnpm why vue-router` (`@vueuse/router` must show `4.6.4`) and `pnpm build` (`nuxt generate` must succeed
+> — forcing a single `vue-router` major instead breaks `@nuxtjs/i18n`, which needs 5.x).
 
 ## Quick start
 

package/components/FormKit/LocaleInput.vue

@@ -7,6 +7,7 @@
         :options="localeOptions"
         option-label="label"
         option-value="value"
+        :allow-empty="false"
         size="small"
       >
         <template #option="{ option }">
@@ -100,15 +101,24 @@ const localeOptions = locales.map(lang => ({
 
 const activeLocale = ref<Locale>('en')
 
+function isLocale(value: unknown): value is Locale {
+  return locales.includes(value as Locale)
+}
+
 // Get the current locale string value
 const localeValue = computed<LocaleStringDto>(() => {
   return props.context.value ?? { de: null, en: null, fr: null, it: null }
 })
 
 // Current input value for the active locale
+// The guard is belt-and-braces next to `:allow-empty="false"`: with an unset `activeLocale`
+// this setter used to write under the key `"null"`, producing a value whose four locales
+// were all empty but which no longer looked like a locale object to the backend's
+// normalization — so it slipped past validation and saved a blank name (issue #135).
 const currentValue = computed({
-  get: () => localeValue.value[activeLocale.value] ?? '',
+  get: () => (isLocale(activeLocale.value) ? localeValue.value[activeLocale.value] ?? '' : ''),
   set: (newVal: string) => {
+    if (!isLocale(activeLocale.value)) return
     const updated: LocaleStringDto = {
       ...localeValue.value,
       [activeLocale.value]: newVal || null,

package/formkit.config.ts

@@ -10,9 +10,39 @@ import { en, de, fr, it } from '@formkit/i18n'
 import { createInput } from '@formkit/vue'
 import { primeInputs } from '@sfxcode/formkit-primevue'
 
+import type { FormKitNode } from '@formkit/core'
 import type { DefaultConfigOptions } from '@formkit/vue'
 
+const LOCALES = ['de', 'en', 'fr', 'it'] as const
+
+// FormKit's built-in `required` rule only asks whether a value is present, and a localeInput's
+// value is always a `{de, en, fr, it}` object — non-empty, so `required` passes even when every
+// locale inside it is blank. Backend `LocaleInput` elements emit `localeRequired` instead
+// (see packages/core/swiss_ai_hub/core/form/elements/locale_input.py).
+function localeRequired(node: FormKitNode): boolean {
+  const value = node.value as Record<string, string | null> | null | undefined
+  if (!value) return false
+  return LOCALES.some(locale => !!value[locale]?.trim())
+}
+
+// FormKit skips a rule entirely when the node's value is empty, unless the rule opts out —
+// which is why the built-in `required` sets this too. Without it the rule would never run on a
+// never-touched field, whose value is still `null`, and a blank Name would pass on a fresh
+// create form: precisely the case this rule exists to catch.
+localeRequired.skipEmpty = false
+
+const localeRequiredMessages = {
+  de: 'Mindestens eine Sprache muss ausgefüllt sein.',
+  en: 'At least one language must be filled in.',
+  fr: 'Au moins une langue doit être renseignée.',
+  it: 'Almeno una lingua deve essere compilata.',
+}
+
 const config: DefaultConfigOptions = {
+  rules: { localeRequired },
+  messages: Object.fromEntries(
+    LOCALES.map(locale => [locale, { validation: { localeRequired: localeRequiredMessages[locale] } }]),
+  ),
   inputs: {
     ...primeInputs,
     agentSelector: createInput(AgentSelector, {

package/package.json

@@ -3,7 +3,7 @@
   "license": "AGPL-3.0-or-later",
   "author": "bbv Software Services AG (https://www.bbv.ch)",
   "type": "module",
-  "version": "0.318.0",
+  "version": "0.319.0",
   "description": "Swiss AI Hub - Admin & Management UI (Nuxt 3 layer)",
   "main": "./nuxt.config.ts",
   "repository": {

package/pages/[tenant]/service/agents/[agent_class]-[agent_id]/configuration.vue

@@ -117,6 +117,9 @@ const submitConfiguration = async (formData: Record<string, unknown>) => {
   }
   catch (error) {
     console.error('Failed to save agent configuration:', error)
+    // No `detail` here on purpose: the SDK's global onResponseError (app.vue) already toasts the
+    // backend's `detail`, and formats pydantic's array-shaped errors while doing it. Passing
+    // `error.message` would only add a second toast reading `[PUT] "<url>": 400 Bad Request`.
     toast.add({
       severity: 'error',
       summary: t('agent.configuration.saveError'),

package/pages/[tenant]/service/processes/[process_class]-[process_id]/configuration.vue

@@ -96,6 +96,8 @@ const submitConfiguration = async (formData: Record<string, unknown>) => {
   }
   catch (error) {
     console.error('Failed to save process configuration:', error)
+    // See the agent configuration page: the SDK's global onResponseError already surfaces the
+    // backend `detail`, so adding `error.message` here would only duplicate the toast with a URL.
     toast.add({
       severity: 'error',
       summary: t('process.configuration.saveError'),

package/sdk/client/index.ts

@@ -313,6 +313,8 @@ export {
   type CreateTranscriptionErrors,
   type CreateTranscriptionResponse,
   type CreateTranscriptionResponses,
+  type CronInput,
+  type CronInputWritable,
   type CustomOutput,
   type DashboardDto,
   type DashboardItemDto,
@@ -743,6 +745,7 @@ export {
   type MailBatchDraftedEventWritable,
   type MailFetchedEvent,
   type MailFetchedEventWritable,
+  type MailMessageRef,
   type MailMovedEvent,
   type MailMovedEventWritable,
   type MemoriesResponse,
@@ -883,6 +886,8 @@ export {
   type RouterEventWritable,
   type RunStatistics,
   type RunStatisticsWritable,
+  type ScheduledStartEvent,
+  type ScheduledStartEventWritable,
   type SearchContextCostPerQueryDto,
   type SearchOrganizationMemoriesData,
   type SearchOrganizationMemoriesError,