@stacksjs/defaults 0.70.354 → 0.70.356

package/app/Actions/Dashboard/Kanban/ColumnDestroyAction.ts

@@ -1,9 +1,10 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { db } from '@stacksjs/database'
-import { kanbanError } from './kanban-response'
+import { kanbanActionError, kanbanError } from './kanban-response'
 
 /**
- * `DELETE /api/dashboard/kanban/columns/:id` (stacksjs/stacks#1846 Phase 2).
+ * `DELETE /api/dashboard/kanban/columns/:id`.
  *
  * Hard-deletes the column and cascade-cleans every card inside it
  * (plus the pivot rows referencing those cards). Same convention as
@@ -19,9 +20,8 @@ export default new Action({
   description: 'Hard-deletes a column and its cards.',
   method: 'DELETE',
   apiResponse: true,
-  async handle(request) {
-    const rawId = (request as any)?.params?.id ?? (request as any)?.param?.('id') ?? null
-    const id = Number(rawId)
+  async handle(request: RequestInstance) {
+    const id = Number(request.getParam('id'))
     if (!Number.isFinite(id) || id <= 0) {
       return kanbanError('Invalid column id', 400)
     }
@@ -37,7 +37,7 @@ export default new Action({
           'DELETE FROM card_assignees WHERE card_id IN (SELECT id FROM cards WHERE column_id = ?)',
           [id],
         ).execute()
-        // Card comments (Phase 3, stacksjs/stacks#1846).
+        // Card comments are card-scoped children.
         await qb.unsafe(
           'DELETE FROM card_comments WHERE card_id IN (SELECT id FROM cards WHERE column_id = ?)',
           [id],
@@ -48,8 +48,7 @@ export default new Action({
       return { deleted: true, id }
     }
     catch (err) {
-      console.error('[dashboard/kanban] ColumnDestroyAction failed:', err)
-      return kanbanError(err instanceof Error ? err.message : 'unknown error', 500)
+      return kanbanActionError(err, 'ColumnDestroyAction')
     }
   },
 })

package/app/Actions/Dashboard/Kanban/ColumnStoreAction.ts

@@ -1,7 +1,8 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { db } from '@stacksjs/database'
 import { Board, BoardColumn } from '@stacksjs/orm'
-import { kanbanError } from './kanban-response'
+import { kanbanActionError, kanbanError } from './kanban-response'
 
 interface ColumnInput {
   boardId?: unknown
@@ -11,7 +12,7 @@ interface ColumnInput {
 }
 
 /**
- * `POST /api/dashboard/kanban/columns` (stacksjs/stacks#1846 Phase 2).
+ * `POST /api/dashboard/kanban/columns`.
  *
  * Appends a new column to the end of the target board. Validates that
  * `boardId` points at a real board so a typo doesn't silently create
@@ -22,8 +23,8 @@ export default new Action({
   description: 'Creates a new column on a board.',
   method: 'POST',
   apiResponse: true,
-  async handle(request) {
-    const body = (request as any).jsonBody as ColumnInput | undefined ?? {}
+  async handle(request: RequestInstance<ColumnInput>) {
+    const body = request.all()
 
     const boardId = Number(body.boardId)
     if (!Number.isFinite(boardId) || boardId <= 0) {
@@ -74,8 +75,7 @@ export default new Action({
       }
     }
     catch (err) {
-      console.error('[dashboard/kanban] ColumnStoreAction failed:', err)
-      return kanbanError(err instanceof Error ? err.message : 'unknown error', 500)
+      return kanbanActionError(err, 'ColumnStoreAction')
     }
   },
 })

package/app/Actions/Dashboard/Kanban/ColumnUpdateAction.ts

@@ -1,7 +1,8 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { BoardColumn } from '@stacksjs/orm'
 import { modelNullableString, modelNumber, modelString, modelValue, refreshModel } from './kanban-model'
-import { kanbanError } from './kanban-response'
+import { kanbanActionError, kanbanError } from './kanban-response'
 
 interface ColumnInput {
   name?: unknown
@@ -10,7 +11,7 @@ interface ColumnInput {
 }
 
 /**
- * `PATCH /api/dashboard/kanban/columns/:id` (stacksjs/stacks#1846 Phase 2).
+ * `PATCH /api/dashboard/kanban/columns/:id`.
  *
  * Partial update for name / color / card limit. `position` and
  * `boardId` are NOT updatable here — moving a column between boards
@@ -23,14 +24,13 @@ export default new Action({
   description: 'Partial update of a board column.',
   method: 'PATCH',
   apiResponse: true,
-  async handle(request) {
-    const rawId = (request as any)?.params?.id ?? (request as any)?.param?.('id') ?? null
-    const id = Number(rawId)
+  async handle(request: RequestInstance<ColumnInput>) {
+    const id = Number(request.getParam('id'))
     if (!Number.isFinite(id) || id <= 0) {
       return kanbanError('Invalid column id', 400)
     }
 
-    const body = (request as any).jsonBody as ColumnInput | undefined ?? {}
+    const body = request.all()
     const set: Record<string, unknown> = {}
 
     if (typeof body.name === 'string') {
@@ -80,8 +80,7 @@ export default new Action({
       }
     }
     catch (err) {
-      console.error('[dashboard/kanban] ColumnUpdateAction failed:', err)
-      return kanbanError(err instanceof Error ? err.message : 'unknown error', 500)
+      return kanbanActionError(err, 'ColumnUpdateAction')
     }
   },
 })

package/app/Actions/Dashboard/Kanban/ColumnsReorderAction.ts

@@ -1,6 +1,7 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { db } from '@stacksjs/database'
-import { kanbanError } from './kanban-response'
+import { kanbanActionError, kanbanError } from './kanban-response'
 
 interface ReorderInput {
   boardId?: unknown
@@ -8,7 +9,7 @@ interface ReorderInput {
 }
 
 /**
- * `POST /api/dashboard/kanban/columns/reorder` (stacksjs/stacks#1846 Phase 2).
+ * `POST /api/dashboard/kanban/columns/reorder`.
  *
  * Bulk rewrite column positions for a board. The submitted `order`
  * is an array of column ids in their new display order; index becomes
@@ -27,8 +28,8 @@ export default new Action({
   description: 'Bulk-rewrite `position` on a board\'s columns.',
   method: 'POST',
   apiResponse: true,
-  async handle(request) {
-    const body = (request as any).jsonBody as ReorderInput | undefined ?? {}
+  async handle(request: RequestInstance<ReorderInput>) {
+    const body = request.all()
 
     const boardId = Number(body.boardId)
     if (!Number.isFinite(boardId) || boardId <= 0) {
@@ -74,8 +75,7 @@ export default new Action({
       return { reordered: ids.length }
     }
     catch (err) {
-      console.error('[dashboard/kanban] ColumnsReorderAction failed:', err)
-      return kanbanError(err instanceof Error ? err.message : 'unknown error', 500)
+      return kanbanActionError(err, 'ColumnsReorderAction')
     }
   },
 })

package/app/Actions/Dashboard/Kanban/LabelDestroyAction.ts

@@ -1,9 +1,10 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { db } from '@stacksjs/database'
-import { kanbanError } from './kanban-response'
+import { kanbanActionError, kanbanError } from './kanban-response'
 
 /**
- * `DELETE /api/dashboard/kanban/labels/:id` (stacksjs/stacks#1846 Phase 3).
+ * `DELETE /api/dashboard/kanban/labels/:id`.
  *
  * Removes a label and detaches it from every card that carries it.
  * The label row goes from the `labels` table; every pivot row in
@@ -15,9 +16,8 @@ export default new Action({
   description: 'Hard-deletes a label and its card_labels pivot rows.',
   method: 'DELETE',
   apiResponse: true,
-  async handle(request) {
-    const rawId = (request as any)?.params?.id ?? (request as any)?.param?.('id') ?? null
-    const id = Number(rawId)
+  async handle(request: RequestInstance) {
+    const id = Number(request.getParam('id'))
     if (!Number.isFinite(id) || id <= 0)
       return kanbanError('Invalid label id', 400)
 
@@ -30,8 +30,7 @@ export default new Action({
       return { deleted: true, id }
     }
     catch (err) {
-      console.error('[dashboard/kanban] LabelDestroyAction failed:', err)
-      return kanbanError(err instanceof Error ? err.message : 'unknown error', 500)
+      return kanbanActionError(err, 'LabelDestroyAction')
     }
   },
 })

package/app/Actions/Dashboard/Kanban/LabelStoreAction.ts

@@ -1,6 +1,7 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { Board, Label } from '@stacksjs/orm'
-import { kanbanError } from './kanban-response'
+import { kanbanActionError, kanbanError } from './kanban-response'
 
 interface LabelInput {
   boardId?: unknown
@@ -9,7 +10,7 @@ interface LabelInput {
 }
 
 /**
- * `POST /api/dashboard/kanban/labels` (stacksjs/stacks#1846 Phase 3).
+ * `POST /api/dashboard/kanban/labels`.
  *
  * Creates a label scoped to a board. Labels are unique per
  * `(board_id, name)` — the migration enforces this via a unique
@@ -21,8 +22,8 @@ export default new Action({
   description: 'Creates a label on a board.',
   method: 'POST',
   apiResponse: true,
-  async handle(request) {
-    const body = (request as any).jsonBody as LabelInput | undefined ?? {}
+  async handle(request: RequestInstance<LabelInput>) {
+    const body = request.all()
 
     const boardId = Number(body.boardId)
     if (!Number.isFinite(boardId) || boardId <= 0) {
@@ -50,8 +51,7 @@ export default new Action({
       }
     }
     catch (err) {
-      console.error('[dashboard/kanban] LabelStoreAction failed:', err)
-      return kanbanError(err instanceof Error ? err.message : 'unknown error', 500)
+      return kanbanActionError(err, 'LabelStoreAction')
     }
   },
 })

package/app/Actions/Dashboard/Kanban/LabelUpdateAction.ts

@@ -1,7 +1,8 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { Label } from '@stacksjs/orm'
 import { modelNumber, modelString, refreshModel } from './kanban-model'
-import { kanbanError } from './kanban-response'
+import { kanbanActionError, kanbanError } from './kanban-response'
 
 interface LabelInput {
   name?: unknown
@@ -9,7 +10,7 @@ interface LabelInput {
 }
 
 /**
- * `PATCH /api/dashboard/kanban/labels/:id` (stacksjs/stacks#1846 Phase 3).
+ * `PATCH /api/dashboard/kanban/labels/:id`.
  *
  * Partial update for label name + color. `board_id` is not movable —
  * a label on board A renaming to "Bug" must not collide with a
@@ -21,14 +22,13 @@ export default new Action({
   description: 'Partial update of a label name / color.',
   method: 'PATCH',
   apiResponse: true,
-  async handle(request) {
-    const rawId = (request as any)?.params?.id ?? (request as any)?.param?.('id') ?? null
-    const id = Number(rawId)
+  async handle(request: RequestInstance<LabelInput>) {
+    const id = Number(request.getParam('id'))
     if (!Number.isFinite(id) || id <= 0) {
       return kanbanError('Invalid label id', 400)
     }
 
-    const body = (request as any).jsonBody as LabelInput | undefined ?? {}
+    const body = request.all()
     const set: Record<string, unknown> = {}
     let renamingTo: string | null = null
 
@@ -74,8 +74,7 @@ export default new Action({
       }
     }
     catch (err) {
-      console.error('[dashboard/kanban] LabelUpdateAction failed:', err)
-      return kanbanError(err instanceof Error ? err.message : 'unknown error', 500)
+      return kanbanActionError(err, 'LabelUpdateAction')
     }
   },
 })

package/app/Actions/Dashboard/Kanban/UsersListAction.ts

@@ -1,9 +1,9 @@
 import { Action } from '@stacksjs/actions'
 import { db } from '@stacksjs/database'
-import { kanbanError } from './kanban-response'
+import { kanbanActionError, kanbanError } from './kanban-response'
 
 /**
- * `GET /api/dashboard/kanban/users` (stacksjs/stacks#1846 Phase 3).
+ * `GET /api/dashboard/kanban/users`.
  *
  * Lightweight user list for the assignee picker on the card detail
  * modal. Returns `{ id, name, email }` only — no roles, no profile,
@@ -30,8 +30,7 @@ export default new Action({
       }
     }
     catch (err) {
-      console.error('[dashboard/kanban] UsersListAction failed:', err)
-      return kanbanError(err instanceof Error ? err.message : 'unknown error', 500)
+      return kanbanActionError(err, 'UsersListAction')
     }
   },
 })

package/app/Actions/Dashboard/Kanban/kanban-comment.ts

@@ -0,0 +1,14 @@
+import { modelNullableNumber, modelNullableString, modelNumber, modelString } from './kanban-model'
+
+export function cardCommentResponse(record: object, author?: object | null) {
+  return {
+    id: modelNumber(record, 'id'),
+    uuid: modelNullableString(record, 'uuid'),
+    userId: modelNullableNumber(record, 'userId', 'user_id'),
+    body: modelString(record, 'body'),
+    authorName: author ? modelNullableString(author, 'name') : null,
+    authorEmail: author ? modelNullableString(author, 'email') : null,
+    createdAt: modelNullableString(record, 'createdAt', 'created_at'),
+    updatedAt: modelNullableString(record, 'updatedAt', 'updated_at'),
+  }
+}

package/app/Actions/Dashboard/Kanban/kanban-model.test.ts

@@ -1,6 +1,8 @@
 import { describe, expect, test } from 'bun:test'
+import { cardCommentResponse } from './kanban-comment'
 import {
   modelBoolean,
+  modelNullableNumber,
   modelNullableString,
   modelNumber,
   modelString,
@@ -31,12 +33,26 @@ describe('kanban model response values', () => {
   })
 
   test('normalizes nullable strings and persisted booleans', () => {
+    expect(modelNullableNumber(record({ value: null }), 'value')).toBeNull()
+    expect(modelNullableNumber(record({ value: '42' }), 'value')).toBe(42)
     expect(modelNullableString(record({ value: null }), 'value')).toBeNull()
     expect(modelBoolean(record({ enabled: 1 }), 'enabled')).toBe(true)
     expect(modelBoolean(record({ enabled: 'true' }), 'enabled')).toBe(true)
     expect(modelBoolean(record({ enabled: 0 }), 'enabled')).toBe(false)
   })
 
+  test('reads raw query rows as well as ORM model records', () => {
+    const row = {
+      board_id: 4,
+      archived: 1,
+      created_at: '2026-08-10 12:00:00',
+    }
+
+    expect(modelNumber(row, 'boardId', 'board_id')).toBe(4)
+    expect(modelBoolean(row, 'archived')).toBe(true)
+    expect(modelString(row, 'createdAt', 'created_at')).toBe('2026-08-10 12:00:00')
+  })
+
   test('refreshes generated database values after a write', async () => {
     const stale = {
       ...record({ updated_at: null }),
@@ -46,4 +62,24 @@ describe('kanban model response values', () => {
     const refreshed = await refreshModel(stale)
     expect(modelString(refreshed, 'updatedAt', 'updated_at')).toBe('2026-07-31 04:00:00')
   })
+
+  test('serializes model-backed comments with their author', () => {
+    expect(cardCommentResponse(record({
+      id: 7,
+      uuid: 'comment-uuid',
+      userId: 3,
+      body: 'Updated note',
+      createdAt: '2026-08-10 20:00:00',
+      updatedAt: '2026-08-10 21:00:00',
+    }), record({ name: 'Chris', email: 'chris@example.com' }))).toEqual({
+      id: 7,
+      uuid: 'comment-uuid',
+      userId: 3,
+      body: 'Updated note',
+      authorName: 'Chris',
+      authorEmail: 'chris@example.com',
+      createdAt: '2026-08-10 20:00:00',
+      updatedAt: '2026-08-10 21:00:00',
+    })
+  })
 })

package/app/Actions/Dashboard/Kanban/kanban-model.ts

@@ -10,28 +10,35 @@ export async function refreshModel(record: RefreshableKanbanModelRecord): Promis
   return await record.fresh() ?? record
 }
 
-export function modelValue(record: KanbanModelRecord, camelKey: string, snakeKey = camelKey): unknown {
-  const camelValue = record.get(camelKey)
+export function modelValue(record: object, camelKey: string, snakeKey = camelKey): unknown {
+  const model = record as Partial<KanbanModelRecord>
+  const row = record as Record<string, unknown>
+  const camelValue = typeof model.get === 'function' ? model.get(camelKey) : row[camelKey]
   if (camelValue !== undefined)
     return camelValue
-  return record.get(snakeKey)
+  return typeof model.get === 'function' ? model.get(snakeKey) : row[snakeKey]
 }
 
-export function modelNumber(record: KanbanModelRecord, camelKey: string, snakeKey = camelKey): number {
+export function modelNumber(record: object, camelKey: string, snakeKey = camelKey): number {
   return Number(modelValue(record, camelKey, snakeKey))
 }
 
-export function modelString(record: KanbanModelRecord, camelKey: string, snakeKey = camelKey): string {
+export function modelNullableNumber(record: object, camelKey: string, snakeKey = camelKey): number | null {
+  const value = modelValue(record, camelKey, snakeKey)
+  return value === undefined || value === null ? null : Number(value)
+}
+
+export function modelString(record: object, camelKey: string, snakeKey = camelKey): string {
   const value = modelValue(record, camelKey, snakeKey)
   return value === undefined || value === null ? '' : String(value)
 }
 
-export function modelNullableString(record: KanbanModelRecord, camelKey: string, snakeKey = camelKey): string | null {
+export function modelNullableString(record: object, camelKey: string, snakeKey = camelKey): string | null {
   const value = modelValue(record, camelKey, snakeKey)
   return value === undefined || value === null ? null : String(value)
 }
 
-export function modelBoolean(record: KanbanModelRecord, camelKey: string, snakeKey = camelKey): boolean {
+export function modelBoolean(record: object, camelKey: string, snakeKey = camelKey): boolean {
   const value = modelValue(record, camelKey, snakeKey)
   return value === true || value === 1 || value === '1' || value === 'true'
 }

package/app/Actions/Dashboard/Kanban/kanban-response.ts

@@ -3,3 +3,8 @@ import { response } from '@stacksjs/router'
 export function kanbanError(error: string, status: number): Response {
   return response.json({ error }, status)
 }
+
+export function kanbanActionError(error: unknown, action: string): Response {
+  console.error(`[dashboard/kanban] ${action} failed:`, error)
+  return kanbanError('The Kanban request could not be completed.', 500)
+}

package/app/Actions/Dashboard/Library/CreateComponent.ts

@@ -3,12 +3,14 @@ import { existsSync } from 'node:fs'
 import { Action, createComponent as scaffoldComponent } from '@stacksjs/actions'
 import { userComponentsPath } from '@stacksjs/path'
 import { response } from '@stacksjs/router'
+import { dashboardOperationalError } from '../dashboard-response'
 import { componentSourceRows } from './library-source'
 
 export default new Action({
   name: 'CreateComponent',
   description: 'Creates a new STX component.',
   method: 'POST',
+  apiResponse: true,
 
   async handle(request: RequestInstance) {
     const name = String(request.get('name') || '').trim()
@@ -16,13 +18,20 @@ export default new Action({
     if (!/^[A-Z][A-Za-z0-9]*$/.test(name))
       return response.json({ message: 'Use a PascalCase component name.' }, 422)
 
-    const destination = userComponentsPath(`${name}.stx`)
-    if (existsSync(destination))
-      return response.json({ message: 'A component with that name already exists.' }, 409)
+    try {
+      const destination = userComponentsPath(`${name}.stx`)
+      if (existsSync(destination))
+        return response.json({ message: 'A component with that name already exists.' }, 409)
 
-    await scaffoldComponent({ name })
+      await scaffoldComponent({ name })
 
-    const created = componentSourceRows().find(component => component.name === name)
-    return response.json({ component: created }, 201)
+      const created = componentSourceRows().find(component => component.name === name)
+      if (!created)
+        throw new Error(`Scaffolded component ${name} could not be found in resources/components.`)
+      return response.json({ component: created }, 201)
+    }
+    catch (error) {
+      return dashboardOperationalError(error, 'The component could not be created.', 'CreateComponent', 500)
+    }
   },
 })

package/app/Actions/Dashboard/Library/CreateFunction.ts

@@ -3,12 +3,14 @@ import { existsSync } from 'node:fs'
 import { Action, createFunction as scaffoldFunction } from '@stacksjs/actions'
 import { userFunctionsPath } from '@stacksjs/path'
 import { response } from '@stacksjs/router'
+import { dashboardOperationalError } from '../dashboard-response'
 import { functionSourceRows } from './library-source'
 
 export default new Action({
   name: 'CreateFunction',
   description: 'Creates a new function.',
   method: 'POST',
+  apiResponse: true,
 
   async handle(request: RequestInstance) {
     const name = String(request.get('name') || '').trim().toLowerCase()
@@ -16,13 +18,20 @@ export default new Action({
     if (!/^[a-z][a-z0-9]*(?:-[a-z0-9]+)*$/.test(name))
       return response.json({ message: 'Use a lowercase kebab-case function name.' }, 422)
 
-    const destination = userFunctionsPath(`${name}.ts`)
-    if (existsSync(destination))
-      return response.json({ message: 'A function with that name already exists.' }, 409)
+    try {
+      const destination = userFunctionsPath(`${name}.ts`)
+      if (existsSync(destination))
+        return response.json({ message: 'A function with that name already exists.' }, 409)
 
-    await scaffoldFunction({ name })
+      await scaffoldFunction({ name })
 
-    const created = functionSourceRows().find(fn => fn.name === name)
-    return response.json({ function: created }, 201)
+      const created = functionSourceRows().find(fn => fn.name === name)
+      if (!created)
+        throw new Error(`Scaffolded function ${name} could not be found in resources/functions.`)
+      return response.json({ function: created }, 201)
+    }
+    catch (error) {
+      return dashboardOperationalError(error, 'The function could not be created.', 'CreateFunction', 500)
+    }
   },
 })