@cat-factory/app 0.261.2 → 0.261.4

package/app/stores/board/context.ts

@@ -31,8 +31,8 @@ export const UNDO_WINDOW_MS = 6000
  * the `board` store setup and threaded into {@link createBoardMutations} / {@link createBoardRemoval}
  * so the split operations stay behaviourally identical to the original single-closure store — the
  * factories are cohesive extractions purely to keep each function within the size budget, not new
- * seams. `api`/`toast`/`tr` are the store's own resolved handles (a store runs outside a component
- * `setup`, so `tr` bridges to the Nuxt app's global i18n instance).
+ * seams. `api`/`toast`/`tr`/`present` are the store's own resolved handles (a store runs outside a
+ * component `setup`, so `tr` bridges to the Nuxt app's global i18n instance).
  */
 export interface BoardWriteContext {
   blocks: Ref<Block[]>
@@ -43,4 +43,11 @@ export interface BoardWriteContext {
   api: ReturnType<typeof useApi>
   toast: ReturnType<typeof useToast>
   tr: (key: string, params?: Record<string, unknown>) => string
+  /**
+   * The SPA's one failure-toast funnel (`usePipelineErrorToast().present`), threaded like `toast`
+   * because it is resolved in the store setup. Every rolled-back write reports through it rather
+   * than building its own `{ title, description: err.message }`, so a board failure reads the same
+   * as every other one: translated copy, and the raw detail copyable behind the disclosure.
+   */
+  present: (error: unknown, titleKey?: string) => void
 }

package/app/stores/board/dependencies.ts

@@ -8,7 +8,7 @@ import type { BoardWriteContext } from './context'
  * the split is purely to keep every function within the size budget.
  */
 export function createBoardDependencies(ctx: BoardWriteContext) {
-  const { getBlock, upsert, api, toast, tr } = ctx
+  const { getBlock, upsert, api, present } = ctx
 
   /**
    * Toggle a dependency edge target -> source (target dependsOn source). The backend
@@ -20,12 +20,7 @@ export function createBoardDependencies(ctx: BoardWriteContext) {
     try {
       upsert(await api.toggleDependency(useWorkspaceStore().requireId(), targetId, { sourceId }))
     } catch (e) {
-      toast.add({
-        title: tr('board.toast.linkFailed'),
-        description: e instanceof Error ? e.message : String(e),
-        icon: 'i-lucide-triangle-alert',
-        color: 'error',
-      })
+      present(e, 'board.toast.linkFailed')
     }
   }
 
@@ -39,12 +34,7 @@ export function createBoardDependencies(ctx: BoardWriteContext) {
     } catch (e) {
       // Mirror `toggleDependency`: a failure must surface (and leave the edge visible) rather
       // than rejecting unhandled with no feedback.
-      toast.add({
-        title: tr('board.toast.unlinkFailed'),
-        description: e instanceof Error ? e.message : String(e),
-        icon: 'i-lucide-triangle-alert',
-        color: 'error',
-      })
+      present(e, 'board.toast.unlinkFailed')
     }
   }
 

package/app/stores/board/mutations.ts

@@ -17,7 +17,7 @@ import type { BoardWriteContext } from './context'
  * {@link createBoardPlacement}.
  */
 export function createBoardMutations(ctx: BoardWriteContext) {
-  const { getBlock, upsert, api, toast, tr } = ctx
+  const { getBlock, upsert, api, present } = ctx
 
   async function addBlock(type: BlockType, position: { x: number; y: number }): Promise<Block> {
     const block = await api.addFrame(useWorkspaceStore().requireId(), { type, position })
@@ -127,12 +127,7 @@ export function createBoardMutations(ctx: BoardWriteContext) {
       upsert(await api.assignToEpic(useWorkspaceStore().requireId(), taskId, { epicId }))
     } catch (e) {
       t.epicId = prev
-      toast.add({
-        title: tr('board.toast.epicFailed'),
-        description: e instanceof Error ? e.message : String(e),
-        icon: 'i-lucide-triangle-alert',
-        color: 'error',
-      })
+      present(e, 'board.toast.epicFailed')
     }
   }
 

package/app/stores/board/placement.ts

@@ -34,7 +34,7 @@ function blockAsPatchable(block: Block): PatchSnapshot {
  * in-closure functions, and the split is purely to keep every function within the size budget.
  */
 export function createBoardPlacement(ctx: BoardWriteContext) {
-  const { blocks, getBlock, upsert, api, toast, tr } = ctx
+  const { blocks, getBlock, upsert, api, toast, tr, present } = ctx
 
   /**
    * Move a block into a new container at a new local position. Drag-reparent commits
@@ -94,12 +94,14 @@ export function createBoardPlacement(ctx: BoardWriteContext) {
       // can act on rather than a fault. The backend sends the machine-readable reason and no
       // translated prose, so map it here; anything else keeps the raw message as the last resort.
       const refusal = moveRefusalKey(e)
-      toast.add({
-        title: tr('board.toast.moveFailed'),
-        description: refusal ? tr(refusal) : e instanceof Error ? e.message : String(e),
-        icon: 'i-lucide-triangle-alert',
-        color: 'error',
-      })
+      if (refusal) {
+        toast.add({
+          title: tr('board.toast.moveFailed'),
+          description: tr(refusal),
+          icon: 'i-lucide-triangle-alert',
+          color: 'error',
+        })
+      } else present(e, 'board.toast.moveFailed')
     }
   }
 
@@ -137,12 +139,7 @@ export function createBoardPlacement(ctx: BoardWriteContext) {
       // Restore the pre-drag position — a rejected move must not leave the block at a
       // spot the server never stored (a lie that survives until the next re-hydrate).
       b.position = prevPosition
-      toast.add({
-        title: tr('board.toast.moveFailed'),
-        description: e instanceof Error ? e.message : String(e),
-        icon: 'i-lucide-triangle-alert',
-        color: 'error',
-      })
+      present(e, 'board.toast.moveFailed')
     }
   }
 
@@ -198,12 +195,7 @@ export function createBoardPlacement(ctx: BoardWriteContext) {
       upsert(await api.resizeBlock(useWorkspaceStore().requireId(), id, bounds))
     } catch (e) {
       previewResize(id, from.position, from.size)
-      toast.add({
-        title: tr('board.toast.resizeFailed'),
-        description: e instanceof Error ? e.message : String(e),
-        icon: 'i-lucide-triangle-alert',
-        color: 'error',
-      })
+      present(e, 'board.toast.resizeFailed')
     }
   }
 
@@ -244,12 +236,7 @@ export function createBoardPlacement(ctx: BoardWriteContext) {
           if (cur[key] === patch[key]) cur[key] = prev[key]
         }
       }
-      toast.add({
-        title: tr('board.toast.updateFailed'),
-        description: e instanceof Error ? e.message : String(e),
-        icon: 'i-lucide-triangle-alert',
-        color: 'error',
-      })
+      present(e, 'board.toast.updateFailed')
       return false
     }
   }

package/app/stores/board/removal.ts

@@ -11,7 +11,7 @@ import { UNDO_WINDOW_MS } from './context'
  * stays internal — it is only wired into the delete toast's undo action.
  */
 export function createBoardRemoval(ctx: BoardWriteContext) {
-  const { blocks, getBlock, pendingRemovals, pendingDoomed, api, toast, tr } = ctx
+  const { blocks, getBlock, pendingRemovals, pendingDoomed, api, toast, tr, present } = ctx
 
   /**
    * Optimistically drop a block and its descendants from the cache, returning a
@@ -142,12 +142,7 @@ export function createBoardRemoval(ctx: BoardWriteContext) {
         // switch must not inject the old subtree onto the workspace now on screen (it
         // re-hydrates from the server there on the next refresh anyway).
         if (useWorkspaceStore().workspaceId === pending.wsId) reattach(pending.snap)
-        toast.add({
-          title: tr('board.toast.deleteFailed'),
-          description: e instanceof Error ? e.message : String(e),
-          icon: 'i-lucide-triangle-alert',
-          color: 'error',
-        })
+        present(e, 'board.toast.deleteFailed')
       }
     }
     const timer = setTimeout(() => void finalize(), UNDO_WINDOW_MS)

package/app/stores/board.spec.ts

@@ -246,11 +246,15 @@ describe('board store read getters', () => {
     expect(() => store.previewMove('missing', { x: 1, y: 1 })).not.toThrow()
   })
 
-  it('updateBlock restores the patched fields and toasts when the write fails', async () => {
-    // Capture the toast the store surfaces on failure. Re-stub before creating the store so it
-    // binds this spy (the store resolves `useToast()` once at setup).
+  it('updateBlock restores the patched fields and reports the failure when the write fails', async () => {
+    // Capture what the store surfaces on failure. Every rolled-back write drains into the shared
+    // failure funnel (`usePipelineErrorToast().present`) rather than building its own toast, so the
+    // assertion is on the funnel and on the TITLE KEY it was given. Re-stub before creating the
+    // store: it resolves both handles once at setup.
     const addSpy = vi.fn()
+    const presentSpy = vi.fn()
     vi.stubGlobal('useToast', () => ({ add: addSpy }))
+    vi.stubGlobal('usePipelineErrorToast', () => ({ present: presentSpy }))
     setActivePinia(createPinia())
     const s = useBoardStore()
     s.hydrate([frame('f1', { title: 'Original', description: 'orig' })])
@@ -263,7 +267,7 @@ describe('board store read getters', () => {
     )
     expect(s.getBlock('f1')?.title).toBe('Original')
     expect(s.getBlock('f1')?.description).toBe('orig')
-    expect(addSpy).toHaveBeenCalledWith(expect.objectContaining({ color: 'error' }))
+    expect(presentSpy).toHaveBeenCalledWith(expect.any(Error), 'board.toast.updateFailed')
   })
 
   it('updateBlock reports a no-op for a block that is not on the board', async () => {
@@ -428,6 +432,7 @@ describe('board store deferred delete + undo', () => {
   function setup(removeImpl: () => Promise<void>) {
     const removeSpy = vi.fn(removeImpl)
     const addSpy = vi.fn()
+    const presentSpy = vi.fn()
     const actions: ToastAction[] = []
     vi.stubGlobal('useApi', () => ({ removeBlock: removeSpy }))
     vi.stubGlobal('useToast', () => ({
@@ -436,9 +441,10 @@ describe('board store deferred delete + undo', () => {
         if (t.actions) actions.push(...t.actions)
       },
     }))
+    vi.stubGlobal('usePipelineErrorToast', () => ({ present: presentSpy }))
     setActivePinia(createPinia())
     useWorkspaceStore().workspaceId = 'ws1'
-    return { store: useBoardStore(), removeSpy, addSpy, actions }
+    return { store: useBoardStore(), removeSpy, addSpy, presentSpy, actions }
   }
 
   beforeEach(() => {
@@ -500,14 +506,15 @@ describe('board store deferred delete + undo', () => {
     expect(removeSpy).toHaveBeenCalledWith('ws1', 'f1')
   })
 
-  it('restores the subtree and toasts an error if the deferred delete fails', async () => {
-    const { store, addSpy } = setup(() => Promise.reject(new Error('boom')))
+  it('restores the subtree and reports the failure if the deferred delete fails', async () => {
+    const { store, presentSpy } = setup(() => Promise.reject(new Error('boom')))
     store.hydrate([frame('f1'), task('t1', 'f1')])
     store.removeBlock('f1')
     await vi.runAllTimersAsync()
     expect(store.getBlock('f1')?.id).toBe('f1')
     expect(store.getBlock('t1')?.id).toBe('t1')
-    expect(addSpy).toHaveBeenCalledWith(expect.objectContaining({ color: 'error' }))
+    // Through the shared funnel, named by its title key (see `updateBlock`'s case above).
+    expect(presentSpy).toHaveBeenCalledWith(expect.any(Error), 'board.toast.deleteFailed')
   })
 
   it('does not reattach a failed deferred delete onto a different workspace', async () => {

package/app/stores/board.ts

@@ -141,7 +141,17 @@ export const useBoardStore = defineStore('board', () => {
   // The write operations, split into cohesive factories sharing the state above (a size-only
   // extraction — behaviour is identical to the former in-closure functions). `undoRemove` stays
   // internal to the removal factory (only its delete toast wires it), so it is NOT re-exported.
-  const context = { blocks, getBlock, upsert, pendingRemovals, pendingDoomed, api, toast, tr }
+  const context = {
+    blocks,
+    getBlock,
+    upsert,
+    pendingRemovals,
+    pendingDoomed,
+    api,
+    toast,
+    tr,
+    present: usePipelineErrorToast().present,
+  }
   const mutations = createBoardMutations(context)
   const placement = createBoardPlacement(context)
   const { detach, reattach, removeBlock } = createBoardRemoval(context)

package/app/stores/recurringPipelines.ts

@@ -12,11 +12,10 @@ import { useBoardStore } from '~/stores/board'
  */
 export const useRecurringPipelinesStore = defineStore('recurringPipelines', () => {
   const api = useApi()
-  const toast = useToast()
-  // Resolve translations through the Nuxt app's global i18n instance — a store runs outside a
-  // component `setup`, so `useI18n()` is unavailable (see the board store for the same pattern).
-  const nuxtApp = useNuxtApp()
-  const tr = (key: string): string => (nuxtApp.$i18n as { t: (k: string) => string }).t(key)
+  // The failure funnel resolves its own translations through the Nuxt app's global i18n instance,
+  // which is what a store (running outside a component `setup`) needs, so this store no longer
+  // carries a `tr` bridge or a raw toast handle of its own.
+  const { present } = usePipelineErrorToast()
 
   const schedules = ref<PipelineSchedule[]>([])
   /** Lazily-loaded run history, keyed by schedule id. */
@@ -71,12 +70,7 @@ export const useRecurringPipelinesStore = defineStore('recurringPipelines', () =
     } catch (e) {
       schedules.value = prevSchedules
       if (blockSnap) board.reattach(blockSnap)
-      toast.add({
-        title: tr('board.toast.recurringDeleteFailed'),
-        description: e instanceof Error ? e.message : String(e),
-        icon: 'i-lucide-triangle-alert',
-        color: 'error',
-      })
+      present(e, 'board.toast.recurringDeleteFailed')
     }
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cat-factory/app",
-  "version": "0.261.2",
+  "version": "0.261.4",
   "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.41",
     "wretch": "^3.0.9",
-    "@cat-factory/contracts": "0.292.2"
+    "@cat-factory/contracts": "0.293.0"
   },
   "devDependencies": {
     "@toad-contracts/testing": "0.3.2",