@cat-factory/app 0.167.0 → 0.167.1

package/app/components/settings/ConnectionWarnings.vue

@@ -0,0 +1,38 @@
+<script setup lang="ts">
+// The non-fatal GAPS a connection test reports about the config it just probed: it connects,
+// but something it never declared costs a recovery or cleanup path that only shows itself
+// during an incident (a runner-pool manifest with no `release` template leaks a runner on
+// every cancelled run).
+//
+// Rendered beside the pass/fail verdict rather than as part of it, because the two are
+// independent — a green test with a gap is the common case, and folding the gap into the
+// failure line would make a working connection look broken.
+//
+// The backend sends machine-readable codes (it does not localize prose); the copy lives in the
+// catalog behind `CONNECTION_WARNING_KEYS`. A code this SPA build predates falls back to the
+// backend's own English `message` — untranslated, but never a blank row.
+import type { ConnectionWarning } from '@cat-factory/contracts'
+import { CONNECTION_WARNING_KEYS } from '~/utils/connectionWarnings'
+
+defineProps<{ warnings?: ConnectionWarning[] }>()
+
+const { t, te } = useI18n()
+
+const copy = (warning: ConnectionWarning): string => {
+  const key = CONNECTION_WARNING_KEYS[warning.code]
+  return key && te(key) ? t(key) : warning.message
+}
+</script>
+
+<template>
+  <div
+    v-if="warnings?.length"
+    class="rounded-md border border-amber-500/40 bg-amber-950/40 px-3 py-2 text-xs text-amber-200"
+    data-testid="connection-warnings"
+  >
+    <p class="font-semibold">{{ t('settings.providerConnection.test.warningsTitle') }}</p>
+    <ul class="mt-1 list-disc space-y-1 pl-4">
+      <li v-for="warning in warnings" :key="warning.code">{{ copy(warning) }}</li>
+    </ul>
+  </div>
+</template>

package/app/components/settings/KubernetesEnvironmentForm.vue

@@ -10,6 +10,8 @@
 // driven (see docs/initiatives/descriptor-driven-infra-forms.md).
 import { computed, reactive, ref, watch } from 'vue'
 import { KUBERNETES_ENV_TOKEN_SECRET_KEY } from '@cat-factory/contracts'
+import type { ConnectionTestResult } from '@cat-factory/contracts'
+import ConnectionWarnings from '~/components/settings/ConnectionWarnings.vue'
 import SecretInput from '~/components/common/SecretInput.vue'
 import type { ProviderConnection } from '~/types/providerConnections'
 
@@ -18,7 +20,7 @@ const props = defineProps<{
   supportsTest: boolean
   testing: boolean
   busy: boolean
-  testResult: { ok: boolean; message?: string } | null
+  testResult: ConnectionTestResult | null
 }>()
 
 const emit = defineEmits<{
@@ -409,6 +411,8 @@ function optional(label: string): string {
       </span>
     </div>
 
+    <ConnectionWarnings :warnings="testResult?.warnings" />
+
     <div class="flex items-center justify-end gap-3">
       <p v-if="connectBlockedReason" class="flex-1 text-left text-xs text-rose-400">
         {{ connectBlockedReason }}

package/app/components/settings/ProviderConnectionTab.vue

@@ -12,7 +12,9 @@
 //   - a MANIFEST-driven provider (no template) → the full JSON manifest editor
 //     (ProviderManifestEditor), which replaces the old "use the API" disclaimer.
 import { computed, ref, toRaw, watch } from 'vue'
+import type { ConnectionTestResult } from '@cat-factory/contracts'
 import type { ProviderConfigField, ProviderConnectionKind } from '~/types/providerConnections'
+import ConnectionWarnings from '~/components/settings/ConnectionWarnings.vue'
 import ProvisioningLogsDrawer from '~/components/provisioning/ProvisioningLogsDrawer.vue'
 import ProviderManifestEditor from '~/components/settings/ProviderManifestEditor.vue'
 import KubernetesEnvironmentForm from '~/components/settings/KubernetesEnvironmentForm.vue'
@@ -56,7 +58,7 @@ const showLogs = ref(false)
 
 // --- Shared state -------------------------------------------------------------------
 const values = ref<Record<string, string>>({})
-const testResult = ref<{ ok: boolean; message?: string } | null>(null)
+const testResult = ref<ConnectionTestResult | null>(null)
 const testing = ref(false)
 const busy = ref(false)
 
@@ -515,6 +517,8 @@ function fieldHelp(key: string): string | undefined {
         </span>
       </div>
 
+      <ConnectionWarnings :warnings="testResult?.warnings" />
+
       <div class="flex justify-end">
         <UButton
           color="primary"

package/app/components/settings/ProviderManifestEditor.vue

@@ -17,7 +17,9 @@
 import { computed, ref, watch } from 'vue'
 import * as v from 'valibot'
 import { environmentManifestSchema, runnerPoolManifestSchema } from '@cat-factory/contracts'
+import type { ConnectionTestResult } from '@cat-factory/contracts'
 import type { ProviderConnectionKind } from '~/types/providerConnections'
+import ConnectionWarnings from '~/components/settings/ConnectionWarnings.vue'
 import SecretInput from '~/components/common/SecretInput.vue'
 
 const props = defineProps<{
@@ -34,7 +36,7 @@ const props = defineProps<{
   /** Bubbled-up busy state from the tab's store calls (so the editor shows loading). */
   testing: boolean
   busy: boolean
-  testResult: { ok: boolean; message?: string } | null
+  testResult: ConnectionTestResult | null
 }>()
 
 const emit = defineEmits<{
@@ -286,6 +288,8 @@ function onSave() {
       </span>
     </div>
 
+    <ConnectionWarnings :warnings="testResult?.warnings" />
+
     <div class="flex justify-end">
       <UButton
         color="primary"

package/app/utils/connectionWarnings.ts

@@ -0,0 +1,17 @@
+import type { ConnectionWarningCode } from '@cat-factory/contracts'
+
+// A connection test can report non-fatal GAPS in the config beside its pass/fail verdict: the
+// backend states them as machine-readable codes (it does not localize prose), and the copy the
+// operator reads lives here.
+//
+// The exhaustive `Record<ConnectionWarningCode, …>` is the tier-2 drift guard: a backend that
+// adds a warning code fails this typecheck until the SPA has copy for it, which the typed-key
+// check alone cannot catch for a runtime-assembled key.
+
+/** Config-gap warning code → i18n key. Leaf keys mirror the code verbatim. */
+export const CONNECTION_WARNING_KEYS: Record<ConnectionWarningCode, string> = {
+  runner_manifest_no_release:
+    'settings.providerConnection.test.warnings.runner_manifest_no_release',
+  runner_manifest_no_status_path:
+    'settings.providerConnection.test.warnings.runner_manifest_no_status_path',
+}

package/i18n/locales/de.json

@@ -298,7 +298,12 @@
       "test": {
         "button": "Verbindung testen",
         "ok": "Verbindung OK",
-        "failed": "Verbindung fehlgeschlagen"
+        "failed": "Verbindung fehlgeschlagen",
+        "warningsTitle": "Lücken in dieser Konfiguration",
+        "warnings": {
+          "runner_manifest_no_release": "Kein Release-Template: Beim Abbrechen eines Laufs kann dem Pool nicht mitgeteilt werden, dass er seinen Job stoppen soll. Ein verwaister Job belegt seinen Runner, bis der Pool ihn von selbst zurücknimmt.",
+          "runner_manifest_no_status_path": "Kein Statuspfad: Jede Abfrage wird als weiterhin laufend gelesen. Ein Job kann daher nur enden, wenn das Abfragebudget des Laufs aufgebraucht ist."
+        }
       },
       "toast": {
         "saved": "{title} gespeichert",

package/i18n/locales/en.json

@@ -2467,7 +2467,12 @@
       "test": {
         "button": "Test connection",
         "ok": "Connection OK",
-        "failed": "Connection failed"
+        "failed": "Connection failed",
+        "warningsTitle": "Gaps in this configuration",
+        "warnings": {
+          "runner_manifest_no_release": "No release template: cancelling a run cannot tell the pool to stop its job, so an orphaned job keeps its runner until the pool reclaims it on its own.",
+          "runner_manifest_no_status_path": "No status path: every poll reads as still running, so a job can only end by exhausting the run's poll budget."
+        }
       },
       "toast": {
         "saved": "{title} saved",

package/i18n/locales/es.json

@@ -2202,7 +2202,12 @@
       "test": {
         "button": "Probar conexión",
         "ok": "Conexión correcta",
-        "failed": "Falló la conexión"
+        "failed": "Falló la conexión",
+        "warningsTitle": "Carencias en esta configuración",
+        "warnings": {
+          "runner_manifest_no_release": "Sin plantilla de release: al cancelar una ejecución no se puede indicar al pool que detenga su trabajo, así que un trabajo huérfano ocupa su runner hasta que el pool lo recupere por su cuenta.",
+          "runner_manifest_no_status_path": "Sin ruta de estado: cada sondeo se interpreta como todavía en ejecución, así que un trabajo solo puede terminar agotando el presupuesto de sondeo de la ejecución."
+        }
       },
       "toast": {
         "saved": "{title} guardado",

package/i18n/locales/fr.json

@@ -2202,7 +2202,12 @@
       "test": {
         "button": "Tester la connexion",
         "ok": "Connexion réussie",
-        "failed": "Échec de la connexion"
+        "failed": "Échec de la connexion",
+        "warningsTitle": "Lacunes dans cette configuration",
+        "warnings": {
+          "runner_manifest_no_release": "Aucun modèle de release : annuler une exécution ne permet pas de demander au pool d'arrêter son job, donc un job orphelin occupe son runner jusqu'à ce que le pool le récupère de lui-même.",
+          "runner_manifest_no_status_path": "Aucun chemin de statut : chaque interrogation est lue comme toujours en cours, donc un job ne peut se terminer qu'en épuisant le budget d'interrogation de l'exécution."
+        }
       },
       "toast": {
         "saved": "{title} enregistré",

package/i18n/locales/he.json

@@ -2397,7 +2397,12 @@
       "test": {
         "button": "בדוק חיבור",
         "ok": "החיבור תקין",
-        "failed": "החיבור נכשל"
+        "failed": "החיבור נכשל",
+        "warningsTitle": "פערים בתצורה הזו",
+        "warnings": {
+          "runner_manifest_no_release": "אין תבנית שחרור: ביטול הרצה לא יכול להודיע למאגר להפסיק את המשימה שלו, ולכן משימה יתומה תופסת את הראנר שלה עד שהמאגר משחרר אותה בעצמו.",
+          "runner_manifest_no_status_path": "אין נתיב סטטוס: כל תשאול נקרא כאילו המשימה עדיין רצה, ולכן משימה יכולה להסתיים רק לאחר ניצול כל תקציב התשאול של ההרצה."
+        }
       },
       "toast": {
         "saved": "{title} נשמר",

package/i18n/locales/it.json

@@ -298,7 +298,12 @@
       "test": {
         "button": "Testa la connessione",
         "ok": "Connessione OK",
-        "failed": "Connessione fallita"
+        "failed": "Connessione fallita",
+        "warningsTitle": "Lacune in questa configurazione",
+        "warnings": {
+          "runner_manifest_no_release": "Nessun template di release: annullare un'esecuzione non può dire al pool di fermare il suo job, quindi un job orfano tiene occupato il suo runner finché il pool non lo recupera da solo.",
+          "runner_manifest_no_status_path": "Nessun percorso di stato: ogni polling viene letto come ancora in corso, quindi un job può terminare solo esaurendo il budget di polling dell'esecuzione."
+        }
       },
       "toast": {
         "saved": "{title} salvato",

package/i18n/locales/ja.json

@@ -2398,7 +2398,12 @@
       "test": {
         "button": "接続をテスト",
         "ok": "接続 OK",
-        "failed": "接続に失敗しました"
+        "failed": "接続に失敗しました",
+        "warningsTitle": "この設定の不足点",
+        "warnings": {
+          "runner_manifest_no_release": "release テンプレートがありません。実行をキャンセルしてもプールにジョブの停止を伝えられないため、取り残されたジョブはプールが自分で回収するまでランナーを占有し続けます。",
+          "runner_manifest_no_status_path": "ステータスパスがありません。ポーリングは常に実行中と解釈されるため、ジョブは実行のポーリング上限を使い切ることでしか終了できません。"
+        }
       },
       "toast": {
         "saved": "{title} を保存しました",

package/i18n/locales/pl.json

@@ -2202,7 +2202,12 @@
       "test": {
         "button": "Przetestuj połączenie",
         "ok": "Połączenie poprawne",
-        "failed": "Połączenie nieudane"
+        "failed": "Połączenie nieudane",
+        "warningsTitle": "Braki w tej konfiguracji",
+        "warnings": {
+          "runner_manifest_no_release": "Brak szablonu release: anulowanie przebiegu nie może przekazać puli, że ma zatrzymać zadanie, więc osierocone zadanie zajmuje swojego runnera, dopóki pula sama go nie odzyska.",
+          "runner_manifest_no_status_path": "Brak ścieżki statusu: każde odpytanie jest odczytywane jako wciąż trwające, więc zadanie może zakończyć się tylko po wyczerpaniu budżetu odpytań przebiegu."
+        }
       },
       "toast": {
         "saved": "Zapisano: {title}",

package/i18n/locales/tr.json

@@ -2398,7 +2398,12 @@
       "test": {
         "button": "Bağlantıyı test et",
         "ok": "Bağlantı tamam",
-        "failed": "Bağlantı başarısız"
+        "failed": "Bağlantı başarısız",
+        "warningsTitle": "Bu yapılandırmadaki eksikler",
+        "warnings": {
+          "runner_manifest_no_release": "Release şablonu yok: bir çalıştırma iptal edildiğinde havuza işi durdurması söylenemez, bu yüzden sahipsiz kalan iş, havuz onu kendi kendine geri alana kadar runner'ını meşgul tutar.",
+          "runner_manifest_no_status_path": "Durum yolu yok: her yoklama hâlâ çalışıyor olarak okunur, bu yüzden bir iş ancak çalıştırmanın yoklama bütçesi tükendiğinde sona erebilir."
+        }
       },
       "toast": {
         "saved": "{title} kaydedildi",

package/i18n/locales/uk.json

@@ -2202,7 +2202,12 @@
       "test": {
         "button": "Перевірити підключення",
         "ok": "Підключення в порядку",
-        "failed": "Не вдалося підключитися"
+        "failed": "Не вдалося підключитися",
+        "warningsTitle": "Прогалини в цій конфігурації",
+        "warnings": {
+          "runner_manifest_no_release": "Немає шаблону release: скасування запуску не може повідомити пулу, що завдання треба зупинити, тож осиротіле завдання займає свій раннер, доки пул не забере його сам.",
+          "runner_manifest_no_status_path": "Немає шляху до статусу: кожне опитування читається як таке, що ще триває, тож завдання може завершитися лише після вичерпання бюджету опитувань запуску."
+        }
       },
       "toast": {
         "saved": "{title} збережено",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cat-factory/app",
-  "version": "0.167.0",
+  "version": "0.167.1",
   "description": "Reusable Nuxt layer for the Agent Architecture Board SPA (components, stores, composables, pages). Consume it from a thin deployment app via `extends: ['@cat-factory/app']` and point it at your backend with NUXT_PUBLIC_API_BASE. See deploy/frontend for an example.",
   "repository": {
     "type": "git",
@@ -40,7 +40,7 @@
     "valibot": "^1.4.2",
     "vue": "3.5.40",
     "wretch": "^3.0.9",
-    "@cat-factory/contracts": "0.179.0"
+    "@cat-factory/contracts": "0.180.0"
   },
   "devDependencies": {
     "@toad-contracts/testing": "0.3.2",