@swiss-ai-hub/web 0.317.1 → 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/Agent/Card.vue

@@ -6,9 +6,7 @@
   >
     <div class="flex items-center justify-between gap-4">
       <div class="flex items-center justify-start gap-2">
-        <div
-          class="flex items-center justify-center rounded-full bg-white p-3 dark:bg-surface-900"
-        >
+        <div class="flex items-center justify-center rounded-full bg-white p-3 dark:bg-surface-900">
           <Icon
             :name="agent.agent_config.icon"
             size="1.5em"
@@ -45,7 +43,7 @@
         />
         <Button
           v-tooltip.top="t('agent.export.button')"
-          icon="pi pi-download"
+          icon="pi pi-file-export"
           severity="secondary"
           text
           rounded

package/components/FormKit/DynamicConfiguration.vue

@@ -54,6 +54,13 @@ import type { FormKitSchemaDefinition } from '@formkit/core'
 
 const { t } = useI18n()
 
+// Fields with a known platform issue, keyed by the backend element id. Frontend-only on
+// purpose: the notice is temporary and carries no config semantics, so it stays out of the
+// agent's form schema. Drop the entry once the underlying issue is fixed.
+const FIELD_WARNING_KEYS: Record<string, string> = {
+  org_memory: 'form.warnings.org_memory_performance',
+}
+
 const props = defineProps<{
   form: FormkitElement[]
   initialData?: Record<string, unknown>
@@ -98,9 +105,15 @@ function replaceLabelVariables(label: string): string {
   })
 }
 
+function fieldWarning(element: FormElement): string | undefined {
+  const warningKey = FIELD_WARNING_KEYS[element.id as string]
+  return warningKey ? t(warningKey) : undefined
+}
+
 const schema = computed<FormKitSchemaDefinition>(() => {
   return buildFormKitSchema(props.form as FormElement[], {
     labelTransform: replaceLabelVariables,
+    fieldWarning,
   })
 })
 
@@ -138,6 +151,10 @@ async function submitHandler() {
   @apply pt-3 pb-1;
 }
 
+.content :deep(.formkit-field-warning) {
+  @apply flex items-start gap-1.5 pb-1 text-xs text-amber-600 dark:text-amber-400;
+}
+
 .content :deep(.formkit-group-fieldset) {
   @apply border border-surface-300 dark:border-surface-600 rounded-lg p-4 mb-4;
 }

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/components/FormKit/TenantSelect.vue

@@ -36,11 +36,11 @@ const props = defineProps<TenantSelectProps>()
 const { t } = useI18n()
 
 const { tenants, tenantsAreLoading } = useTenantMemberships()
-const { activeTenant, activeTenantIsLoading } = useActiveTenantQuery()
+const { tenantId } = useTenant()
 
 const placeholder = computed(() => props.context.placeholder)
 const filter = computed(() => props.context.filter ?? true)
-const isLoading = computed(() => tenantsAreLoading.value || activeTenantIsLoading.value)
+const isLoading = computed(() => tenantsAreLoading.value)
 
 const selectedTenant = computed({
   get: () => props.context.value ?? null,
@@ -52,10 +52,10 @@ const selectedTenant = computed({
 // Pre-select the user's active tenant on a fresh field, while leaving an
 // already-configured value (editing an existing instance) untouched.
 watch(
-  [activeTenant, () => props.context.value],
+  [tenantId, () => props.context.value],
   ([active, current]) => {
-    if (!current && active?.id) {
-      props.context.node.input(active.id)
+    if (!current && active) {
+      props.context.node.input(active)
     }
   },
   { immediate: true },

package/components/Knowledge/Document/List.vue

@@ -18,6 +18,7 @@
   <DataTable
     v-model:selection="checkedDocuments"
     :value="documents"
+    data-key="id"
     table-style="min-width: 50rem"
     :row-class="getRowClass"
     :sort-field="sortField ?? undefined"
@@ -185,11 +186,16 @@ watch(
 const deletableDocuments = computed(() => props.documents.filter(isDocumentDeletable))
 
 const allDeletableSelected = computed(
-  () => deletableDocuments.value.length > 0 && checkedDocuments.value.length === deletableDocuments.value.length,
+  () => deletableDocuments.value.length > 0
+    && deletableDocuments.value.every(document => checkedDocuments.value.some(selected => selected.id === document.id)),
 )
 
 const onSelectAllChange = ({ checked }: { checked: boolean }) => {
-  checkedDocuments.value = checked ? [...deletableDocuments.value] : []
+  const pageIds = new Set(deletableDocuments.value.map(document => document.id))
+  const selectionFromOtherPages = checkedDocuments.value.filter(selected => !pageIds.has(selected.id))
+  checkedDocuments.value = checked
+    ? [...selectionFromOtherPages, ...deletableDocuments.value]
+    : selectionFromOtherPages
 }
 
 const handleRowClick = (event: DataTableRowClickEvent) => {

package/composables/app/useAppVersion.ts

@@ -3,10 +3,21 @@ import { minutesToMilliseconds } from 'date-fns'
 
 // Surfaces the running version of both services. The UI service version is baked
 // into the static bundle at build time (runtimeConfig.public.appVersion); the API
-// version is read from the health endpoint. When both match a single number is
+// version is read from the health endpoint, which reports the channel tag (e.g.
+// `latest`) the running image was pulled as. When both match a single number is
 // shown, otherwise both are shown as `UI / API`.
+//
+// A release is promoted onto the `latest` channel by retagging the exact `-rc.N`
+// build that was tested, so the baked version keeps its release-candidate suffix
+// (e.g. `v0.317.0-rc.3`) even though the artifact IS the final release. Strip that
+// suffix only on the `latest` channel, so a promoted build shows its release
+// version (`v0.317.0`). On any other channel the build is genuinely a candidate or
+// rolling build, so keep the suffix to stay identifiable.
+const RELEASE_CHANNEL = 'latest'
+const toReleaseVersion = (version: string): string => version.replace(/-rc\.\d+$/, '')
+
 export const useAppVersion = defineQuery(() => {
-  const uiVersion = useRuntimeConfig().public.appVersion as string
+  const bakedUiVersion = useRuntimeConfig().public.appVersion as string
 
   const { data: apiVersion } = useQuery<string>({
     key: () => ['app-version', 'api'],
@@ -17,10 +28,14 @@ export const useAppVersion = defineQuery(() => {
     },
   })
 
+  const uiVersion = computed(() =>
+    apiVersion.value === RELEASE_CHANNEL ? toReleaseVersion(bakedUiVersion) : bakedUiVersion,
+  )
+
   const versionDisplay = computed(() => {
     const api = apiVersion.value
-    if (!api || api === uiVersion) return uiVersion
-    return `${uiVersion} / ${api}`
+    if (!api || api === uiVersion.value) return uiVersion.value
+    return `${uiVersion.value} / ${api}`
   })
 
   return { uiVersion, apiVersion, versionDisplay }

package/composables/form/useFormKitTransform.ts

@@ -132,6 +132,43 @@ export interface TransformOptions {
   locale?: string
   labelTransform?: (label: string) => string
   optionsResolver?: (element: FormElement) => unknown[] | undefined
+  // Resolves a frontend-only warning for an element. The backend form schema carries no
+  // warning of its own, so notices about known platform issues are attached here.
+  fieldWarning?: (element: FormElement) => string | undefined
+}
+
+/**
+ * Warning notice rendered next to a field, styled like a `Message severity="warn"
+ * variant="simple"` (see `.formkit-field-warning` in FormKit/DynamicConfiguration.vue).
+ * A plain `$el` node rather than `$cmp: 'Message'`: PrimeVue components are auto-imported
+ * per component, so FormKitSchema cannot resolve them by name.
+ */
+function buildFieldWarningNode(element: FormElement, text: string): FormKitSchemaNode {
+  const base = (element.id as string | undefined) ?? (element.name as string)
+  return {
+    $el: 'div',
+    key: `${base}__warning`,
+    attrs: { class: 'formkit-field-warning' },
+    children: [
+      { $el: 'i', attrs: { class: 'pi pi-exclamation-triangle' } },
+      { $el: 'span', children: text },
+    ],
+  } as unknown as FormKitSchemaNode
+}
+
+/**
+ * Places the warning directly under the "Enable X" toggle for a nullable element — the
+ * toggle stays mounted when the section is switched off, so the warning remains readable
+ * before the user opts in. Non-nullable elements get it above their own node.
+ */
+function withFieldWarning(
+  nodes: FormKitSchemaNode | FormKitSchemaNode[],
+  warning: FormKitSchemaNode | undefined,
+  afterToggle: boolean,
+): FormKitSchemaNode | FormKitSchemaNode[] {
+  if (!warning) return nodes
+  const nodeArray = Array.isArray(nodes) ? nodes : [nodes]
+  return afterToggle ? [nodeArray[0], warning, ...nodeArray.slice(1)] : [warning, ...nodeArray]
 }
 
 function createGroupNode(
@@ -311,7 +348,11 @@ function combineConditions(toggleCondition: string, existing: string | undefined
   return `$: ${toggleCondition.slice(1)} && (${existing.slice(1)})`
 }
 
-function buildNullableToggleNode(element: FormElement, label: string | undefined): Record<string, unknown> {
+function buildNullableToggleNode(
+  element: FormElement,
+  label: string | undefined,
+  help: string | undefined,
+): Record<string, unknown> {
   const fieldName = element.name as string
   const toggleId = nullableToggleId(element)
   return {
@@ -320,17 +361,24 @@ function buildNullableToggleNode(element: FormElement, label: string | undefined
     id: toggleId,
     key: toggleId,
     label: label ? `Enable ${label}` : 'Enable',
+    ...(help ? { help } : {}),
     binary: true,
   }
 }
 
+/**
+ * `help` is only supplied for groups: a group's own node renders no help, so the toggle is the
+ * only place to hang it. Nullable leaves already render their help on the input itself — passing
+ * it here too would print the same sentence twice.
+ */
 function applyNullableToggle(
   element: FormElement,
   baseNode: FormKitSchemaNode | FormKitSchemaNode[],
   label: string | undefined,
+  help?: string,
 ): FormKitSchemaNode[] {
   const nodeArray = Array.isArray(baseNode) ? baseNode : [baseNode]
-  return [buildNullableToggleNode(element, label) as FormKitSchemaNode, ...nodeArray]
+  return [buildNullableToggleNode(element, label, help) as FormKitSchemaNode, ...nodeArray]
 }
 
 function gateElement(element: FormElement, toggleCondition: string): FormElement {
@@ -351,7 +399,7 @@ export function transformElementToSchema(
   const formkitType = getFormkitType(element)
   if (formkitType === 'repeater') return []
 
-  const { locale = 'en', labelTransform, optionsResolver } = options
+  const { locale = 'en', labelTransform, optionsResolver, fieldWarning } = options
 
   const children = (element.children as FormElement[] || []).flatMap(
     child => transformElementToSchema(child, options),
@@ -365,20 +413,26 @@ export function transformElementToSchema(
   const isNullable = element.nullable === true
   const toggleCondition = isNullable ? `$get(${nullableToggleId(element)}).value` : undefined
 
+  const warningText = fieldWarning?.(element)
+  const warningNode = warningText ? buildFieldWarningNode(element, warningText) : undefined
+
   if (formkitType === 'group') {
     const gatedElement = isNullable ? gateElement(element, toggleCondition!) : element
     const groupNode = createGroupNode(gatedElement, children, label)
-    return isNullable ? applyNullableToggle(element, groupNode, label) : groupNode
+    if (!isNullable) return withFieldWarning(groupNode, warningNode, false)
+    const toggledNodes = applyNullableToggle(element, groupNode, label, getLocalizedString(element.help, locale))
+    return withFieldWarning(toggledNodes, warningNode, true)
   }
 
   const cleanNode = buildNodeProperties(element, formkitType, label, locale, optionsResolver)
   if (children.length > 0) cleanNode.children = children
   if (isNullable) {
     cleanNode.if = combineConditions(toggleCondition!, element.if as string | undefined)
-    return applyNullableToggle(element, cleanNode as FormKitSchemaNode, label)
+    const toggledNodes = applyNullableToggle(element, cleanNode as FormKitSchemaNode, label)
+    return withFieldWarning(toggledNodes, warningNode, true)
   }
 
-  return cleanNode as FormKitSchemaNode
+  return withFieldWarning(cleanNode as FormKitSchemaNode, warningNode, false)
 }
 
 function buildLeafNodeForRepeater(
@@ -427,7 +481,8 @@ export function transformElementForRepeater(
   if (formkitType === 'group') {
     const gatedElement = isNullable ? gateElement(element, toggleCondition!) : element
     const groupNode = createGroupNode(gatedElement, children, label)
-    return isNullable ? applyNullableToggle(element, groupNode, label) : groupNode
+    if (!isNullable) return groupNode
+    return applyNullableToggle(element, groupNode, label, getLocalizedString(element.help, locale))
   }
 
   const cleanNode = buildLeafNodeForRepeater(element, formkitType, label, locale, children)

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/i18n/locales/de.yaml

@@ -790,6 +790,10 @@ process:
 form:
   locale_input:
     translate_tooltip: In alle Sprachen übersetzen
+  warnings:
+    org_memory_performance: Diese Funktion funktioniert in manchen Fällen möglicherweise
+      nicht wie erwartet. Um unerwartetes Verhalten zu vermeiden, deaktivieren Sie
+      sie bitte vorübergehend, bis eine Korrektur veröffentlicht wird.
 lib:
   vectorStore:
     label: Vektorspeicher

package/i18n/locales/en.yaml

@@ -778,6 +778,9 @@ process:
 form:
   locale_input:
     translate_tooltip: Translate to all languages
+  warnings:
+    org_memory_performance: This feature may not work as expected in some cases. To
+      avoid unexpected behavior, please temporarily disable it until a fix is released.
 lib:
   vectorStore:
     label: Vector Store

package/i18n/locales/fr.yaml

@@ -790,6 +790,10 @@ process:
 form:
   locale_input:
     translate_tooltip: Traduire dans toutes les langues
+  warnings:
+    org_memory_performance: Cette fonctionnalité peut ne pas fonctionner comme prévu
+      dans certains cas. Pour éviter tout comportement inattendu, veuillez la désactiver
+      temporairement jusqu'à la publication d'un correctif.
 lib:
   vectorStore:
     label: Stockage vectoriel

package/i18n/locales/it.yaml

@@ -786,6 +786,10 @@ process:
 form:
   locale_input:
     translate_tooltip: Traduci in tutte le lingue
+  warnings:
+    org_memory_performance: Questa funzionalità potrebbe non funzionare come previsto
+      in alcuni casi. Per evitare comportamenti inattesi, disattivala temporaneamente
+      fino al rilascio di una correzione.
 lib:
   vectorStore:
     label: Archivio vettoriale

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.317.1",
+  "version": "0.319.0",
   "description": "Swiss AI Hub - Admin & Management UI (Nuxt 3 layer)",
   "main": "./nuxt.config.ts",
   "repository": {
@@ -43,7 +43,6 @@
   "dependencies": {
     "@formkit/i18n": "^1.7.2",
     "@formkit/nuxt": "^2.0.0",
-    "@hey-api/nuxt": "^0.2.1",
     "@nuxt/fonts": "^0.14.0",
     "@nuxt/icon": "^1.15.0",
     "@nuxtjs/i18n": "10.3.0",

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

@@ -11,7 +11,7 @@
           {{ t('agent.configuration.description') }}
         </p>
         <Button
-          icon="pi pi-download"
+          icon="pi pi-file-export"
           severity="secondary"
           :label="t('agent.export.button')"
           class="shrink-0"
@@ -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/agents.vue

@@ -17,7 +17,7 @@
         />
         <div class="flex items-center gap-4">
           <Button
-            icon="pi pi-upload"
+            icon="pi pi-file-import"
             severity="secondary"
             :label="t('agent.import.button')"
             @click="triggerImport"

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,