@cat-factory/app 0.141.2 → 0.142.0

package/app/components/board/AddTaskModal.vue

@@ -289,8 +289,9 @@ const selectedModelPresetLabel = computed(() => {
 // Hide UI-testing pipelines (`tester-ui` / `visual-confirmation`) when the target frame has no
 // UI to exercise — they'd be refused server-side (see utils/pipeline + the backend gate). Also
 // hide `'recurring'`-only pipelines (a one-off task start of one is refused at run start) and,
-// for a `document` task, every non-document pipeline (it authors a doc — only document pipelines
-// are relevant, per the `purpose` classifier). Re-filters as the chosen task type changes.
+// for a `document` / `review` task, every pipeline whose purpose doesn't match (a doc task authors
+// a doc, a review task reviews a PR — only document / review pipelines are relevant, per the
+// `purpose` classifier). Re-filters as the chosen task type changes.
 const selectablePipelines = computed(() =>
   pipelines.pipelines.filter((p) =>
     pipelineAllowedForManualStart(p, frame.value, board.blocks, taskType.value),
@@ -302,14 +303,16 @@ const selectablePipelines = computed(() =>
 // must appear in the form BEFORE creation:
 //   - `ralph` needs its preset so the per-task validation command + iteration budget the `ralph`
 //     agent contributes surface for editing ("choose at run time" would be a dead end);
-//   - a `document` task defaults to `pl_document` so its document-only picker (the `purpose` gate
-//     hides every non-document pipeline) is never rendered empty.
-// The other typed defaults (spike/review) carry no up-front config and don't narrow their picker,
-// so the modal leaves `pipelineId` unset and `BoardService` applies the backend type-default at
-// creation. Keep these ids in step with the backend helper.
+//   - a `document` task defaults to `pl_document` and a `review` task to `pl_review` so their
+//     purpose-narrowed picker (the `purpose` gate hides every non-document / non-review pipeline)
+//     is never rendered empty.
+// The other typed default (spike) carries no up-front config and doesn't narrow its picker, so the
+// modal leaves `pipelineId` unset and `BoardService` applies the backend type-default at creation.
+// Keep these ids in step with the backend helper.
 const DEFAULT_PIPELINE_FOR_TYPE: Partial<Record<TaskTypeChoice, string>> = {
   ralph: 'pl_ralph',
   document: 'pl_document',
+  review: 'pl_review',
 }
 watch(taskType, (next) => {
   const preset = DEFAULT_PIPELINE_FOR_TYPE[next]

package/app/components/fragments/FragmentLibraryManager.vue

@@ -282,7 +282,7 @@ async function linkDocumentFragment() {
     // error surfaced) so a retry re-attempts just those, never re-linking what already went in.
     let linked = 0
     let firstError: unknown
-    for (const { path, ref } of [...stagedDocRefs.value]) {
+    for (const { path, ref } of stagedDocRefs.value) {
       try {
         await library.createDocumentFragment({ source, ref, tags })
         const i = docFilePaths.value.indexOf(path)

package/app/components/pipeline/PipelineHealthModal.vue

@@ -1,13 +1,16 @@
 <script setup lang="ts">
 // Startup advisory for unhealthy pipelines. Opened once per session from the board page when
 // `usePipelineHealth` reports any issue. Lists:
+//   • new built-in pipelines the workspace doesn't have yet (ADD them);
 //   • invalid pipelines (unknown agent kind / bad shape) — DELETE a custom one, RESEED a built-in;
 //   • outdated built-ins (a newer catalog definition is available) — RESEED to adopt it.
-// Detection is client-side (see usePipelineHealth); the actions hit the pipelines store.
+// Adding a new built-in and reseeding an existing one are the same reseed call (it creates or
+// updates by catalog id). Detection is client-side (see usePipelineHealth); the actions hit the
+// pipelines store.
 const { t } = useI18n()
 const ui = useUiStore()
 const pipelines = usePipelinesStore()
-const { invalid, outdated, hasIssues } = usePipelineHealth()
+const { invalid, outdated, newPipelines, hasIssues } = usePipelineHealth()
 const toast = useToast()
 
 const open = computed({
@@ -45,17 +48,20 @@ const reseed = (id: string) =>
 const remove = (id: string) =>
   run(id, () => pipelines.removePipeline(id), t('pipeline.health.toast.deleteFailed'))
 
-/** Reseed every reseedable pipeline (outdated built-ins + invalid built-ins) in one go. */
+/** Reseed every reseedable pipeline (new + outdated built-ins + invalid built-ins) in one go. */
 async function reseedAll() {
-  const ids = [...invalid.value.filter((h) => h.pipeline.builtin), ...outdated.value].map(
-    (h) => h.pipeline.id,
-  )
+  const ids = [
+    ...newPipelines.value.map((p) => p.id),
+    ...invalid.value.filter((h) => h.pipeline.builtin).map((h) => h.pipeline.id),
+    ...outdated.value.map((h) => h.pipeline.id),
+  ]
   for (const id of new Set(ids)) await reseed(id)
 }
 
 const reseedableCount = computed(
   () =>
     new Set([
+      ...newPipelines.value.map((p) => p.id),
       ...invalid.value.filter((h) => h.pipeline.builtin).map((h) => h.pipeline.id),
       ...outdated.value.map((h) => h.pipeline.id),
     ]).size,
@@ -71,6 +77,41 @@ const reseedableCount = computed(
       </div>
 
       <div v-else class="space-y-5">
+        <!-- New built-in pipelines the workspace can add. -->
+        <section v-if="newPipelines.length" class="space-y-2">
+          <div class="flex items-center gap-2">
+            <UIcon name="i-lucide-sparkles" class="h-4 w-4 text-emerald-400" />
+            <h3 class="text-sm font-semibold text-slate-200">
+              {{ t('pipeline.health.newHeading') }}
+            </h3>
+          </div>
+          <p class="text-[11px] text-slate-500">{{ t('pipeline.health.newDescription') }}</p>
+          <ul class="space-y-2">
+            <li
+              v-for="p in newPipelines"
+              :key="p.id"
+              class="flex items-center justify-between gap-3 rounded-lg border border-slate-800 bg-slate-900/40 p-3"
+            >
+              <div class="min-w-0">
+                <span class="truncate text-sm font-medium text-slate-100 capitalize">{{
+                  p.name
+                }}</span>
+              </div>
+              <UButton
+                size="xs"
+                color="primary"
+                variant="subtle"
+                icon="i-lucide-plus"
+                :loading="isBusy(p.id)"
+                :disabled="anyBusy"
+                @click="reseed(p.id)"
+              >
+                {{ t('pipeline.health.add') }}
+              </UButton>
+            </li>
+          </ul>
+        </section>
+
         <!-- Invalid: unknown agent kinds or a broken shape. -->
         <section v-if="invalid.length" class="space-y-2">
           <div class="flex items-center gap-2">

package/app/composables/usePipelineHealth.spec.ts

@@ -142,4 +142,26 @@ describe('usePipelineHealth', () => {
     expect(invalid.value).toHaveLength(1)
     expect(outdated.value).toHaveLength(0)
   })
+
+  it('surfaces a brand-new built-in the workspace does not have yet (offer to add)', () => {
+    // A board seeded before `pl_review` shipped: the catalog versions advertise it, but no stored
+    // pipeline has that id — so it must appear as a "new" pipeline the user can add.
+    const stored = builtin(['coder', 'reviewer'], { id: 'pl_full', version: 1 })
+    const { newPipelines, hasIssues, invalid, outdated } = scan([stored], {
+      pl_full: 1,
+      pl_review: 4,
+    })
+    expect(newPipelines.value).toEqual([{ id: 'pl_review', name: 'review' }])
+    expect(hasIssues.value).toBe(true)
+    // A brand-new built-in is not "invalid" or "outdated" — those only concern STORED pipelines.
+    expect(invalid.value).toHaveLength(0)
+    expect(outdated.value).toHaveLength(0)
+  })
+
+  it('reports no new pipelines when every catalog id is already stored', () => {
+    const stored = builtin(['coder', 'reviewer'], { id: 'pl_full', version: 1 })
+    const { newPipelines, hasIssues } = scan([stored], { pl_full: 1 })
+    expect(newPipelines.value).toHaveLength(0)
+    expect(hasIssues.value).toBe(false)
+  })
 })

package/app/composables/usePipelineHealth.ts

@@ -23,6 +23,23 @@ export interface PipelineHealth {
   outdated: boolean
 }
 
+/** A brand-new built-in pipeline that appeared in the catalog but isn't in the workspace yet. */
+export interface NewPipeline {
+  /** The catalog (built-in) id — what the reseed endpoint is keyed by (it creates the row). */
+  id: string
+  /** The built-in's display name, from the catalog versions' companion name map. */
+  name: string
+}
+
+/**
+ * A built-in's display name for the "new pipeline" advisory, humanised from its catalog id
+ * (`pl_review` -> "review", rendered capitalised) — used only until the row is reseeded into
+ * existence, at which point its real catalog name is stored. Mirrors `useRiskPolicyHealth`.
+ */
+function builtinPipelineName(id: string): string {
+  return id.replace(/^pl_/, '').replace(/_/g, ' ')
+}
+
 /** Producers a companion kind is allowed to review (inverse of {@link COMPANION_FOR_PRODUCER}). */
 function companionTargets(companion: string): string[] {
   return Object.entries(COMPANION_FOR_PRODUCER)
@@ -88,12 +105,14 @@ function shapeProblem(p: Pipeline): string | null {
 
 /**
  * Detect pipelines in an unhealthy state for the startup advisory: those referencing an unknown
- * agent kind or with an invalid shape (offer to delete a custom one / reseed a built-in), and
- * built-ins whose seeded definition has moved ahead of the stored copy (offer to reseed). Reads
- * the pipeline library + the snapshot's catalog versions from the pipelines store. Detection runs
- * entirely client-side because the canonical agent-kind catalog lives here (`AGENT_BY_KIND` +
- * `SYSTEM_AGENT_META` + registered custom kinds); the version comparison uses the catalog
- * versions the snapshot ships.
+ * agent kind or with an invalid shape (offer to delete a custom one / reseed a built-in), built-ins
+ * whose seeded definition has moved ahead of the stored copy (offer to reseed), AND brand-new
+ * built-ins that appeared in the catalog but aren't in the workspace yet (offer to ADD them — a
+ * board seeded before the built-in shipped, e.g. `pl_review`). Reads the pipeline library + the
+ * snapshot's catalog versions from the pipelines store. Detection runs entirely client-side: the
+ * canonical agent-kind catalog lives here (`AGENT_BY_KIND` + `SYSTEM_AGENT_META` + registered custom
+ * kinds), and the catalog versions the snapshot ships ARE the set of built-in ids — a catalog id
+ * with no stored pipeline is a new built-in. Mirrors `useRiskPolicyHealth` / `useModelPresetHealth`.
  */
 export function usePipelineHealth() {
   const store = usePipelinesStore()
@@ -130,11 +149,20 @@ export function usePipelineHealth() {
     return out
   })
 
+  // Brand-new built-ins: a catalog id (a `catalogVersions` key) with no stored pipeline. Adding one
+  // is the same reseed call as adopting an update (it inserts the row when absent).
+  const newPipelines = computed<NewPipeline[]>(() => {
+    const storedIds = new Set(store.pipelines.map((p) => p.id))
+    return Object.keys(store.catalogVersions)
+      .filter((id) => !storedIds.has(id))
+      .map((id) => ({ id, name: builtinPipelineName(id) }))
+  })
+
   // An invalid built-in is reseeded (not deleted) and that also clears any "outdated" flag, so
   // exclude it from the outdated list to avoid offering the same fix twice.
   const invalid = computed(() => health.value.filter((h) => h.invalid))
   const outdated = computed(() => health.value.filter((h) => h.outdated && !h.invalid))
-  const hasIssues = computed(() => health.value.length > 0)
+  const hasIssues = computed(() => health.value.length > 0 || newPipelines.value.length > 0)
 
-  return { health, invalid, outdated, hasIssues }
+  return { health, invalid, outdated, newPipelines, hasIssues }
 }

package/app/utils/pipeline.spec.ts

@@ -17,10 +17,19 @@ describe('pipelineAllowedForTaskType', () => {
     expect(pipelineAllowedForTaskType(pipeline({ purpose: undefined }), 'document')).toBe(false)
   })
 
-  it('every non-document task type is unrestricted (any purpose, and undefined type)', () => {
-    for (const type of ['feature', 'bug', 'spike', 'review', 'ralph', undefined] as const) {
+  it('a review task offers ONLY review-purpose pipelines', () => {
+    expect(pipelineAllowedForTaskType(pipeline({ purpose: 'review' }), 'review')).toBe(true)
+    expect(pipelineAllowedForTaskType(pipeline({ purpose: 'build' }), 'review')).toBe(false)
+    expect(pipelineAllowedForTaskType(pipeline({ purpose: 'document' }), 'review')).toBe(false)
+    // An unclassified pipeline is hidden from a review task (it requires the explicit classifier).
+    expect(pipelineAllowedForTaskType(pipeline({ purpose: undefined }), 'review')).toBe(false)
+  })
+
+  it('every other task type is unrestricted (any purpose, and undefined type)', () => {
+    for (const type of ['feature', 'bug', 'spike', 'ralph', undefined] as const) {
       expect(pipelineAllowedForTaskType(pipeline({ purpose: 'build' }), type)).toBe(true)
       expect(pipelineAllowedForTaskType(pipeline({ purpose: 'document' }), type)).toBe(true)
+      expect(pipelineAllowedForTaskType(pipeline({ purpose: 'review' }), type)).toBe(true)
       expect(pipelineAllowedForTaskType(pipeline({ purpose: undefined }), type)).toBe(true)
     }
   })
@@ -51,10 +60,15 @@ describe('pipelineAllowedForManualStart composes the task-type gate', () => {
   const noFrame = undefined
   const blocks: Block[] = []
 
-  it('drops a non-document pipeline for a document task, keeps it for others', () => {
+  it('drops a mismatched pipeline for a document / review task, keeps it for others', () => {
     const build = pipeline({ purpose: 'build' })
     expect(pipelineAllowedForManualStart(build, noFrame, blocks, 'document')).toBe(false)
+    expect(pipelineAllowedForManualStart(build, noFrame, blocks, 'review')).toBe(false)
     expect(pipelineAllowedForManualStart(build, noFrame, blocks, 'feature')).toBe(true)
+    // A review pipeline is offered to a review task and hidden from a document task.
+    const review = pipeline({ purpose: 'review' })
+    expect(pipelineAllowedForManualStart(review, noFrame, blocks, 'review')).toBe(true)
+    expect(pipelineAllowedForManualStart(review, noFrame, blocks, 'document')).toBe(false)
     // No task type supplied ⇒ no task-type restriction.
     expect(pipelineAllowedForManualStart(build, noFrame, blocks)).toBe(true)
   })

package/i18n/locales/de.json

@@ -3260,6 +3260,9 @@
       "invalidHeading": "Ungültige Pipelines",
       "invalidDescription": "Diese verweisen auf einen fehlenden Agenten oder sind falsch konfiguriert, sodass sie beim Start fehlschlagen (oder falsch laufen) würden. Löschen Sie eine benutzerdefinierte Pipeline oder setzen Sie eine integrierte neu auf, um ihre Katalogdefinition wiederherzustellen.",
       "builtinBadge": "integriert",
+      "newHeading": "Neue Pipelines verfügbar",
+      "newDescription": "Neue integrierte Pipelines sind verfügbar. Füge sie zur Bibliothek dieses Boards hinzu.",
+      "add": "Hinzufügen",
       "reseed": "Neu aufsetzen",
       "delete": "Löschen",
       "updatesHeading": "Updates verfügbar",

package/i18n/locales/en.json

@@ -3626,6 +3626,9 @@
       "invalidHeading": "Invalid pipelines",
       "invalidDescription": "These reference a missing agent or are misconfigured, so they would fail (or misrun) at start. Delete a custom pipeline, or reseed a built-in to restore its catalog definition.",
       "builtinBadge": "built-in",
+      "newHeading": "New pipelines available",
+      "newDescription": "New built-in pipelines have shipped. Add them to this board's library.",
+      "add": "Add",
       "reseed": "Reseed",
       "delete": "Delete",
       "updatesHeading": "Updates available",

package/i18n/locales/es.json

@@ -3527,6 +3527,9 @@
       "invalidHeading": "Pipelines no válidos",
       "invalidDescription": "Estos hacen referencia a un agente inexistente o están mal configurados, por lo que fallarían (o se ejecutarían mal) al iniciar. Elimina un pipeline personalizado o vuelve a generar uno integrado para restaurar su definición del catálogo.",
       "builtinBadge": "integrado",
+      "newHeading": "Nuevos pipelines disponibles",
+      "newDescription": "Hay nuevos pipelines integrados. Añádelos a la biblioteca de este tablero.",
+      "add": "Añadir",
       "reseed": "Regenerar",
       "delete": "Eliminar",
       "updatesHeading": "Actualizaciones disponibles",

package/i18n/locales/fr.json

@@ -3527,6 +3527,9 @@
       "invalidHeading": "Pipelines non valides",
       "invalidDescription": "Ceux-ci référencent un agent manquant ou sont mal configurés et échoueraient (ou s'exécuteraient mal) au démarrage. Supprimez un pipeline personnalisé, ou régénérez un pipeline intégré pour restaurer sa définition du catalogue.",
       "builtinBadge": "intégré",
+      "newHeading": "Nouveaux pipelines disponibles",
+      "newDescription": "De nouveaux pipelines intégrés sont arrivés. Ajoutez-les à la bibliothèque de ce tableau.",
+      "add": "Ajouter",
       "reseed": "Régénérer",
       "delete": "Supprimer",
       "updatesHeading": "Mises à jour disponibles",

package/i18n/locales/he.json

@@ -3538,6 +3538,9 @@
       "invalidHeading": "צינורות לא תקפים",
       "invalidDescription": "אלה מפנים לסוכן חסר או מוגדרים בצורה שגויה, ולכן ייכשלו (או ירוצו בצורה שגויה) בהתחלה. מחק צינור מותאם אישית, או זרע מחדש צינור מובנה כדי לשחזר את הגדרת הקטלוג שלו.",
       "builtinBadge": "מובנה",
+      "newHeading": "צינורות חדשים זמינים",
+      "newDescription": "הגיעו צינורות מובנים חדשים. הוסף אותם לספריית הלוח הזה.",
+      "add": "הוסף",
       "reseed": "זרע מחדש",
       "delete": "מחק",
       "updatesHeading": "עדכונים זמינים",

package/i18n/locales/it.json

@@ -3260,6 +3260,9 @@
       "invalidHeading": "Pipeline non valide",
       "invalidDescription": "Queste fanno riferimento a un agente mancante o sono configurate male, quindi fallirebbero (o funzionerebbero male) all'avvio. Elimina una pipeline personalizzata, oppure ripristina una integrata per recuperare la sua definizione dal catalogo.",
       "builtinBadge": "integrata",
+      "newHeading": "Nuove pipeline disponibili",
+      "newDescription": "Sono state rilasciate nuove pipeline integrate. Aggiungile alla libreria di questa board.",
+      "add": "Aggiungi",
       "reseed": "Ripristina",
       "delete": "Elimina",
       "updatesHeading": "Aggiornamenti disponibili",

package/i18n/locales/ja.json

@@ -3539,6 +3539,9 @@
       "invalidHeading": "無効なパイプライン",
       "invalidDescription": "存在しないエージェントを参照しているか設定が誤っているため、開始時に失敗(または誤動作)します。カスタムパイプラインを削除するか、ビルトインを再シードしてカタログ定義を復元してください。",
       "builtinBadge": "ビルトイン",
+      "newHeading": "新しいパイプラインがあります",
+      "newDescription": "新しい組み込みパイプラインが追加されました。このボードのライブラリに追加してください。",
+      "add": "追加",
       "reseed": "再シード",
       "delete": "削除",
       "updatesHeading": "更新あり",

package/i18n/locales/pl.json

@@ -3527,6 +3527,9 @@
       "invalidHeading": "Nieprawidłowe pipeline'y",
       "invalidDescription": "Odwołują się do brakującego agenta lub są źle skonfigurowane, więc zawiodłyby (lub wykonałyby się błędnie) przy starcie. Usuń pipeline niestandardowy albo zregeneruj wbudowany, aby przywrócić jego definicję z katalogu.",
       "builtinBadge": "wbudowany",
+      "newHeading": "Dostępne nowe pipeline'y",
+      "newDescription": "Pojawiły się nowe wbudowane pipeline'y. Dodaj je do biblioteki tej tablicy.",
+      "add": "Dodaj",
       "reseed": "Zregeneruj",
       "delete": "Usuń",
       "updatesHeading": "Dostępne aktualizacje",

package/i18n/locales/tr.json

@@ -3539,6 +3539,9 @@
       "invalidHeading": "Geçersiz pipeline'lar",
       "invalidDescription": "Bunlar eksik bir agent'a referans veriyor veya yanlış yapılandırılmış, bu yüzden başlangıçta başarısız olur (veya hatalı çalışır). Özel bir pipeline'ı silin ya da katalog tanımını geri yüklemek için bir yerleşik pipeline'ı yeniden tohumlayın.",
       "builtinBadge": "yerleşik",
+      "newHeading": "Yeni pipeline'lar mevcut",
+      "newDescription": "Yeni yerleşik pipeline'lar geldi. Bu panonun kütüphanesine ekleyin.",
+      "add": "Ekle",
       "reseed": "Yeniden tohumla",
       "delete": "Sil",
       "updatesHeading": "Güncellemeler mevcut",

package/i18n/locales/uk.json

@@ -3527,6 +3527,9 @@
       "invalidHeading": "Недійсні пайплайни",
       "invalidDescription": "Вони посилаються на відсутнього агента або неправильно налаштовані, тож зазнали б помилки (або виконалися б неправильно) під час старту. Видаліть власний пайплайн або перегенеруйте вбудований, щоб відновити його визначення з каталогу.",
       "builtinBadge": "вбудований",
+      "newHeading": "Доступні нові пайплайни",
+      "newDescription": "З'явилися нові вбудовані пайплайни. Додайте їх до бібліотеки цієї дошки.",
+      "add": "Додати",
       "reseed": "Перегенерувати",
       "delete": "Видалити",
       "updatesHeading": "Доступні оновлення",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cat-factory/app",
-  "version": "0.141.2",
+  "version": "0.142.0",
   "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.150.0"
+    "@cat-factory/contracts": "0.151.0"
   },
   "devDependencies": {
     "@toad-contracts/testing": "0.3.2",