@stacksjs/defaults 0.70.354 → 0.70.356

package/app/Actions/Dashboard/Email/InboxMarkReadAction.ts

@@ -2,7 +2,7 @@ import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { emailSDK } from '@stacksjs/email'
 import { response } from '@stacksjs/router'
-import { defaultMailbox } from './mail-preference'
+import { dashboardMailbox, inboxActionError } from './inbox-request'
 
 export default new Action({
   name: 'InboxMarkReadAction',
@@ -11,16 +11,21 @@ export default new Action({
   apiResponse: true,
 
   async handle(request: RequestInstance) {
-    const mailbox = String(request.get('mailbox') || defaultMailbox())
     const messageId = String(request.get('messageId') || '')
 
     if (!messageId)
       return response.json({ message: 'messageId is required.' }, 422)
 
-    const success = await emailSDK.markAsRead(mailbox, messageId)
-    if (!success)
-      return response.json({ message: 'Email not found or its read state could not be updated.' }, 404)
+    try {
+      const mailbox = dashboardMailbox(request)
+      const success = await emailSDK.markAsRead(mailbox, messageId)
+      if (!success)
+        return response.json({ message: 'Email not found or its read state could not be updated.' }, 404)
 
-    return response.json({ success: true, mailbox, messageId })
+      return response.json({ success: true, mailbox, messageId })
+    }
+    catch (error) {
+      return inboxActionError(error, 'The email read state could not be updated.')
+    }
   },
 })

package/app/Actions/Dashboard/Email/InboxMarkUnreadAction.ts

@@ -2,7 +2,7 @@ import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { emailSDK } from '@stacksjs/email'
 import { response } from '@stacksjs/router'
-import { defaultMailbox } from './mail-preference'
+import { dashboardMailbox, inboxActionError } from './inbox-request'
 
 export default new Action({
   name: 'InboxMarkUnreadAction',
@@ -11,16 +11,21 @@ export default new Action({
   apiResponse: true,
 
   async handle(request: RequestInstance) {
-    const mailbox = String(request.get('mailbox') || defaultMailbox())
     const messageId = String(request.get('messageId') || '')
 
     if (!messageId)
       return response.json({ message: 'messageId is required.' }, 422)
 
-    const success = await emailSDK.markAsUnread(mailbox, messageId)
-    if (!success)
-      return response.json({ message: 'Email not found or its read state could not be updated.' }, 404)
+    try {
+      const mailbox = dashboardMailbox(request)
+      const success = await emailSDK.markAsUnread(mailbox, messageId)
+      if (!success)
+        return response.json({ message: 'Email not found or its read state could not be updated.' }, 404)
 
-    return response.json({ success: true, mailbox, messageId })
+      return response.json({ success: true, mailbox, messageId })
+    }
+    catch (error) {
+      return inboxActionError(error, 'The email read state could not be updated.')
+    }
   },
 })

package/app/Actions/Dashboard/Email/InboxPreferenceShowAction.ts

@@ -2,7 +2,8 @@ import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { MailPreference } from '@stacksjs/orm'
 import { response } from '@stacksjs/router'
-import { defaultMailbox, serializeMailPreference } from './mail-preference'
+import { dashboardMailbox, inboxActionError } from './inbox-request'
+import { serializeMailPreference } from './mail-preference'
 
 export default new Action({
   name: 'InboxPreferenceShowAction',
@@ -11,8 +12,13 @@ export default new Action({
   apiResponse: true,
 
   async handle(request: RequestInstance) {
-    const mailbox = String(request.get('mailbox') || defaultMailbox()).trim().toLowerCase()
-    const record = await MailPreference.where('mailbox', '=', mailbox).first()
-    return response.json({ preference: serializeMailPreference(record, mailbox) })
+    try {
+      const mailbox = dashboardMailbox(request)
+      const record = await MailPreference.where('mailbox', '=', mailbox).first()
+      return response.json({ preference: serializeMailPreference(record, mailbox) })
+    }
+    catch (error) {
+      return inboxActionError(error, 'Mail settings could not be loaded.')
+    }
   },
 })

package/app/Actions/Dashboard/Email/InboxPreferenceUpdateAction.ts

@@ -2,8 +2,8 @@ import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { MailPreference } from '@stacksjs/orm'
 import { response } from '@stacksjs/router'
+import { dashboardMailbox, inboxActionError } from './inbox-request'
 import {
-  defaultMailbox,
   type MailPreferenceInput,
   mailPreferenceAttributes,
   normalizeStringList,
@@ -132,14 +132,13 @@ export default new Action({
   apiResponse: true,
 
   async handle(request: RequestInstance) {
-    const mailboxValue = request.get('mailbox')
-    const mailbox = (mailboxValue === null || mailboxValue === undefined
-      ? defaultMailbox()
-      : typeof mailboxValue === 'string' ? mailboxValue : '')
-      .trim()
-      .toLowerCase()
-    if (!/^[^\s@<>]+@[^\s@<>]+\.[^\s@<>]+$/.test(mailbox) || mailbox.length > 320)
-      return response.json({ message: 'The mail settings are invalid.', fields: { mailbox: 'Enter a valid mailbox.' } }, 422)
+    let mailbox: string
+    try {
+      mailbox = dashboardMailbox(request)
+    }
+    catch (error) {
+      return inboxActionError(error, 'Mail settings could not be updated.')
+    }
 
     const { fields, input } = inputFromRequest(request, mailbox)
     if (Object.keys(fields).length)

package/app/Actions/Dashboard/Email/InboxShowAction.ts

@@ -1,7 +1,8 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { emailSDK } from '@stacksjs/email'
 import { response } from '@stacksjs/router'
-import { defaultMailbox } from './mail-preference'
+import { dashboardMailbox, inboxActionError } from './inbox-request'
 import { sanitizeInboxHtml } from './sanitize-inbox-html'
 
 export default new Action({
@@ -10,10 +11,10 @@ export default new Action({
   method: 'GET',
   apiResponse: true,
 
-  async handle(request: any) {
+  async handle(request: RequestInstance) {
     try {
-      const mailbox = request?.query?.mailbox || defaultMailbox()
-      const messageId = request?.params?.id
+      const mailbox = dashboardMailbox(request)
+      const messageId = String(request.getParam('id') || '')
 
       if (!messageId)
         return response.json({ message: 'A message ID is required.' }, 422)
@@ -28,12 +29,11 @@ export default new Action({
         html: sanitizeInboxHtml(email.html ?? ''),
         text: email.text ?? '',
         metadata: email.metadata,
+        attachments: email.attachments,
       }
     }
     catch (err) {
-      return response.json({
-        message: err instanceof Error ? err.message : 'Email content could not be loaded.',
-      }, 503)
+      return inboxActionError(err, 'Email content could not be loaded.')
     }
   },
 })

package/app/Actions/Dashboard/Email/InboxStatsAction.ts

@@ -1,7 +1,7 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { emailSDK } from '@stacksjs/email'
-import { response } from '@stacksjs/router'
-import { defaultMailbox } from './mail-preference'
+import { dashboardMailbox, inboxActionError } from './inbox-request'
 
 export default new Action({
   name: 'InboxStatsAction',
@@ -9,9 +9,9 @@ export default new Action({
   method: 'GET',
   apiResponse: true,
 
-  async handle(request: any) {
+  async handle(request: RequestInstance) {
     try {
-      const mailbox = request?.query?.mailbox || defaultMailbox()
+      const mailbox = dashboardMailbox(request)
       const stats = await emailSDK.getInboxStats(mailbox)
 
       return {
@@ -20,9 +20,7 @@ export default new Action({
       }
     }
     catch (err) {
-      return response.json({
-        message: err instanceof Error ? err.message : 'Inbox statistics could not be loaded.',
-      }, 503)
+      return inboxActionError(err, 'Inbox statistics could not be loaded.')
     }
   },
 })

package/app/Actions/Dashboard/Email/inbox-request.ts

@@ -0,0 +1,33 @@
+import type { RequestInstance } from '@stacksjs/types'
+import { config } from '@stacksjs/config'
+import { inboxMailboxPath, InvalidInboxMailboxError } from '@stacksjs/email'
+import { response } from '@stacksjs/router'
+import { defaultMailbox } from './mail-preference'
+
+function configuredMailDomain(): string {
+  const email = (config as any)?.email || {}
+  const fromAddress = String(email.from?.address || '').trim()
+  const fromDomain = fromAddress.includes('@') ? fromAddress.split('@').at(-1) : ''
+  return String(email.domain || fromDomain || 'stacksjs.com').trim().toLowerCase()
+}
+
+export function dashboardMailbox(request: RequestInstance): string {
+  const value = request.get('mailbox')
+  if (value !== null && value !== undefined && typeof value !== 'string')
+    throw new InvalidInboxMailboxError()
+
+  return inboxMailboxPath(value?.trim() || defaultMailbox(), configuredMailDomain()).address
+}
+
+export function inboxActionError(error: unknown, fallback: string): Response {
+  if (error instanceof InvalidInboxMailboxError) {
+    return response.json({
+      message: 'Enter a mailbox on the configured email domain.',
+      fields: { mailbox: 'Enter a mailbox on the configured email domain.' },
+    }, 422)
+  }
+
+  return response.json({
+    message: fallback,
+  }, 503)
+}

package/app/Actions/Dashboard/Infrastructure/InsightsAction.ts

@@ -6,6 +6,7 @@ import { cache } from '@stacksjs/cache'
 import { config } from '@stacksjs/config'
 import { db } from '@stacksjs/database'
 import { checkQueueHealth } from '@stacksjs/queue'
+import { dashboardOperationalIssue } from '../dashboard-response'
 import {
   compactSql,
   countValue,
@@ -23,14 +24,14 @@ interface SourceResult<T> {
   error: string
 }
 
-async function inspectSource<T>(work: Promise<T>, fallback: T): Promise<SourceResult<T>> {
+async function inspectSource<T>(work: Promise<T>, fallback: T, message: string, source: string): Promise<SourceResult<T>> {
   try {
     return { value: await work, error: '' }
   }
   catch (error) {
     return {
       value: fallback,
-      error: error instanceof Error ? error.message : 'The source could not be inspected.',
+      error: dashboardOperationalIssue(error, message, `InsightsAction.${source}`),
     }
   }
 }
@@ -135,18 +136,18 @@ export default new Action({
       filesystem,
       databaseFile,
     ] = await Promise.all([
-      inspectSource(requestSummaryQuery, undefined),
-      inspectSource(requestSuccessQuery, undefined),
-      inspectSource(slowRequestsQuery, []),
-      inspectSource(querySummaryQuery, undefined),
-      inspectSource(queryStatusesQuery, []),
-      inspectSource(slowQueriesQuery, []),
-      inspectSource(errorSummaryQuery, undefined),
-      inspectSource(recentErrorsQuery, []),
-      inspectSource(checkQueueHealth(), null),
-      inspectSource(cache.getStats(), null),
-      inspectSource(filesystemQuery, null),
-      inspectSource(databaseFileQuery, null),
+      inspectSource(requestSummaryQuery, undefined, 'Request telemetry is unavailable.', 'requests.summary'),
+      inspectSource(requestSuccessQuery, undefined, 'Request telemetry is unavailable.', 'requests.success'),
+      inspectSource(slowRequestsQuery, [], 'Request telemetry is unavailable.', 'requests.slowest'),
+      inspectSource(querySummaryQuery, undefined, 'Query telemetry is unavailable.', 'queries.summary'),
+      inspectSource(queryStatusesQuery, [], 'Query telemetry is unavailable.', 'queries.statuses'),
+      inspectSource(slowQueriesQuery, [], 'Query telemetry is unavailable.', 'queries.slowest'),
+      inspectSource(errorSummaryQuery, undefined, 'Error telemetry is unavailable.', 'errors.summary'),
+      inspectSource(recentErrorsQuery, [], 'Error telemetry is unavailable.', 'errors.recent'),
+      inspectSource(checkQueueHealth(), null, 'Queue telemetry is unavailable.', 'queue'),
+      inspectSource(cache.getStats(), null, 'Cache telemetry is unavailable.', 'cache'),
+      inspectSource(filesystemQuery, null, 'Filesystem telemetry is unavailable.', 'filesystem'),
+      inspectSource(databaseFileQuery, null, 'Database file telemetry is unavailable.', 'database-file'),
     ])
 
     const requestTotal = countValue(requestSummary.value?.total)

package/app/Actions/Dashboard/Infrastructure/LogIndexAction.ts

@@ -1,15 +1,14 @@
 import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { db } from '@stacksjs/database'
-import { request as routerRequest, response } from '@stacksjs/router'
+import { dashboardRequestValue } from '../dashboard-request'
+import { dashboardOperationalError } from '../dashboard-response'
 import { DASHBOARD_LOG_TYPES, normalizeDashboardLog, summarizeDashboardLogTypes } from './log-dashboard'
 
 const RANGES = ['1', '7', '30', '90', 'all'] as const
 
 function queryValue(request: RequestInstance, key: string): string {
-  const query = ((routerRequest as any).query || {}) as Record<string, string | string[] | undefined>
-  const value = query[key]
-  return String((Array.isArray(value) ? value[0] : value) || request.get(key) || '').trim()
+  return dashboardRequestValue(request, key)
 }
 
 export default new Action({
@@ -102,13 +101,7 @@ export default new Action({
       }
     }
     catch (error) {
-      return response.json({
-        logs: [],
-        summary: { total: 0, error: 0, warning: 0, info: 0, success: 0 },
-        pagination: { page: 1, perPage: 25, total: 0, totalPages: 1 },
-        options: { sources: [], projects: [], types: [...DASHBOARD_LOG_TYPES], ranges: [...RANGES] },
-        error: error instanceof Error ? error.message : 'Logs could not be loaded.',
-      }, 500)
+      return dashboardOperationalError(error, 'Logs could not be loaded.', 'LogIndexAction')
     }
   },
 })

package/app/Actions/Dashboard/Infrastructure/ServerShowAction.ts

@@ -1,3 +1,4 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { response } from '@stacksjs/router'
 import { tsCloud } from '~/config/cloud'
@@ -9,12 +10,8 @@ export default new Action({
   description: 'Returns one configured server or persisted deployment.',
   method: 'GET',
   apiResponse: true,
-  async handle(request) {
-    const identifier = String(
-      (request as any)?.params?.id
-      ?? (request as any)?.param?.('id')
-      ?? '',
-    )
+  async handle(request: RequestInstance) {
+    const identifier = request.getParam('id')
     if (!identifier)
       return response.json({ error: 'A server identifier is required.' }, 400)
 

package/app/Actions/Dashboard/Jobs/JobIndexAction.ts

@@ -1,26 +1,19 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { FailedJob, Job } from '@stacksjs/orm'
-import { request, response } from '@stacksjs/router'
+import { dashboardOperationalError } from '../dashboard-response'
 import { matchesJobSearch, normalizeActiveJob, normalizeFailedJob } from './job-records'
 
 export default new Action({
   name: 'JobIndexAction',
   description: 'Returns a paginated list of jobs across the active queue and the failed_jobs table.',
   method: 'GET',
-  async handle() {
-    // bun-router populates `request.query` as a Record<string, string|string[]>.
-    // Outside a request context (e.g. ad-hoc tests) it's undefined — fall back
-    // to defaults so the action stays callable.
-    const query = ((request as any).query || {}) as Record<string, string | string[] | undefined>
-    const pick = (key: string) => {
-      const v = query[key]
-      return Array.isArray(v) ? v[0] : v
-    }
-    const page = Math.max(1, Number(pick('page')) || 1)
-    const perPage = Math.min(200, Math.max(1, Number(pick('per_page')) || 25))
-    const queueFilter = String(pick('queue') || '').trim()
-    const statusFilter = String(pick('status') || '').trim().toLowerCase()
-    const search = String(pick('search') || '').trim().toLowerCase()
+  async handle(request: RequestInstance) {
+    const page = Math.max(1, Number(request.get('page', 1)) || 1)
+    const perPage = Math.min(200, Math.max(1, Number(request.get('per_page', 25)) || 25))
+    const queueFilter = String(request.get('queue', '')).trim()
+    const statusFilter = String(request.get('status', '')).trim().toLowerCase()
+    const search = String(request.get('search', '')).trim().toLowerCase()
 
     try {
       const [activeJobs, failedJobs] = await Promise.all([
@@ -50,9 +43,7 @@ export default new Action({
       return { data, total, page, perPage, queues, queueConnected: true }
     }
     catch (error) {
-      return response.json({
-        message: error instanceof Error ? error.message : 'Job history could not be loaded.',
-      }, 503)
+      return dashboardOperationalError(error, 'Job history could not be loaded.', 'JobIndexAction')
     }
   },
 })

package/app/Actions/Dashboard/Jobs/JobRetryAction.ts

@@ -1,5 +1,6 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
-import { request, response } from '@stacksjs/router'
+import { response } from '@stacksjs/router'
 import { retryFailedJob } from '@stacksjs/queue'
 import { parseJobReference } from './job-records'
 
@@ -7,8 +8,8 @@ export default new Action({
   name: 'JobRetryAction',
   description: 'Retries a single failed job by its failed_jobs id.',
   method: 'POST',
-  async handle() {
-    const reference = String((request as any).params?.id || '')
+  async handle(request: RequestInstance) {
+    const reference = request.getParam('id')
     const parsed = parseJobReference(reference)
     const id = Number(parsed.id)
     if (parsed.source === 'job') {

package/app/Actions/Dashboard/Jobs/JobShowAction.ts

@@ -1,29 +1,35 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { FailedJob, Job } from '@stacksjs/orm'
-import { request } from '@stacksjs/router'
+import { response } from '@stacksjs/router'
+import { dashboardOperationalError } from '../dashboard-response'
 import { normalizeActiveJob, normalizeFailedJob, parseJobReference } from './job-records'
 
 export default new Action({
   name: 'JobShowAction',
   description: 'Returns one native Job or FailedJob record for dashboard inspection.',
   method: 'GET',
-  async handle() {
-    const reference = String((request as any).params?.id || '')
+  async handle(request: RequestInstance) {
+    const reference = request.getParam('id')
     const parsed = parseJobReference(reference)
     const id = Number(parsed.id)
 
     if (!Number.isFinite(id) || id <= 0)
-      return { job: null, error: 'Invalid job id.' }
+      return response.json({ message: 'Invalid job id.' }, 400)
 
     try {
       if (parsed.source === 'failed') {
         const record = await FailedJob.find(id)
-        return { job: record ? normalizeFailedJob(record) : null }
+        return record
+          ? { job: normalizeFailedJob(record) }
+          : response.json({ message: 'Job not found.' }, 404)
       }
 
       if (parsed.source === 'job') {
         const record = await Job.find(id)
-        return { job: record ? normalizeActiveJob(record) : null }
+        return record
+          ? { job: normalizeActiveJob(record) }
+          : response.json({ message: 'Job not found.' }, 404)
       }
 
       const activeRecord = await Job.find(id)
@@ -31,13 +37,12 @@ export default new Action({
         return { job: normalizeActiveJob(activeRecord) }
 
       const failedRecord = await FailedJob.find(id)
-      return { job: failedRecord ? normalizeFailedJob(failedRecord) : null }
+      return failedRecord
+        ? { job: normalizeFailedJob(failedRecord) }
+        : response.json({ message: 'Job not found.' }, 404)
     }
     catch (error) {
-      return {
-        job: null,
-        error: error instanceof Error ? error.message : 'Job could not be loaded.',
-      }
+      return dashboardOperationalError(error, 'Job could not be loaded.', 'JobShowAction')
     }
   },
 })

package/app/Actions/Dashboard/Jobs/JobStatsAction.ts

@@ -1,7 +1,7 @@
 import { Action } from '@stacksjs/actions'
 import { FailedJob, Job } from '@stacksjs/orm'
 import { getGlobalMetrics } from '@stacksjs/queue'
-import { response } from '@stacksjs/router'
+import { dashboardOperationalError } from '../dashboard-response'
 
 export default new Action({
   name: 'JobStatsAction',
@@ -38,9 +38,7 @@ export default new Action({
       }
     }
     catch (error) {
-      return response.json({
-        message: error instanceof Error ? error.message : 'Job metrics could not be loaded.',
-      }, 503)
+      return dashboardOperationalError(error, 'Job metrics could not be loaded.', 'JobStatsAction')
     }
   },
 })

package/app/Actions/Dashboard/Kanban/BoardDestroyAction.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/boards/:id` (stacksjs/stacks#1846 Phase 2).
+ * `DELETE /api/dashboard/kanban/boards/:id`.
  *
  * Hard delete with cascade: removes the board, its columns, its cards,
  * its labels, and the relevant pivot rows in `card_labels` and
@@ -23,9 +24,8 @@ export default new Action({
   description: 'Hard-deletes a board and all of its columns, cards, labels, and 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 board id', 400)
     }
@@ -42,7 +42,7 @@ export default new Action({
           'DELETE FROM card_assignees WHERE card_id IN (SELECT id FROM cards WHERE board_id = ?)',
           [id],
         ).execute()
-        // Card comments (Phase 3, migration 0000000112). Same
+        // Card comments use the same
         // card-scoped pattern as the pivots.
         await qb.unsafe(
           'DELETE FROM card_comments WHERE card_id IN (SELECT id FROM cards WHERE board_id = ?)',
@@ -60,8 +60,7 @@ export default new Action({
       return { deleted: true, id }
     }
     catch (err) {
-      console.error('[dashboard/kanban] BoardDestroyAction failed:', err)
-      return kanbanError(err instanceof Error ? err.message : 'unknown error', 500)
+      return kanbanActionError(err, 'BoardDestroyAction')
     }
   },
 })

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

@@ -1,7 +1,8 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { db } from '@stacksjs/database'
 import { modelBoolean } from './kanban-model'
-import { kanbanError } from './kanban-response'
+import { kanbanActionError, kanbanError } from './kanban-response'
 
 interface BoardRow {
   id: number
@@ -56,8 +57,8 @@ interface LabelRow {
  *
  * Returns a single board with its columns + cards + label palette
  * nested in render order. Powers the kanban board page
- * (`/kanban/[id]`) — Phase 2 hits this on mount, the drag handler
- * then mutates positions via the (Phase 2) reorder endpoint and
+ * (`/kanban/[id]`). The page loads this on mount, then its drag handler
+ * mutates positions through the reorder endpoint and
  * relies on optimistic updates for snappy UX.
  *
  * Query layout (3 queries, all indexed):
@@ -67,24 +68,16 @@ interface LabelRow {
  *      `column_id, position`).
  *   4. All labels for the board.
  *
- * Card-label pivot reads land in Phase 3 alongside the card detail
- * modal — the board view shows label chips inline, hydrated from a
- * single pivot query batched with the card fetch.
+ * Card-label pivots hydrate label chips inline through a single query
+ * batched with the card fetch.
  */
 export default new Action({
   name: 'Kanban Board Show',
   description: 'Returns a single kanban board with its columns + cards + label palette.',
   method: 'GET',
   apiResponse: true,
-  async handle(request) {
-    // Route param resolution: bun-router exposes `:id` via
-    // `request.param('id')` / `request.params.id`. Both shapes appear
-    // in this codebase depending on how the route was registered;
-    // covering both keeps the action portable.
-    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 board id', 400)
     }
@@ -114,8 +107,8 @@ export default new Action({
         ).execute() as Promise<LabelRow[]>,
       ])
 
-      // Pivot lookups for label + assignee chips on card previews
-      // (stacksjs/stacks#1846 Phase 3). Two batched JOINs keyed off the
+      // Pivot lookups for label + assignee chips on card previews.
+      // Two batched JOINs keyed off the
       // board's card list — single round-trip per pivot, then group
       // client-side into `cardId → list` maps.
       const [cardLabelRows, cardAssigneeRows] = await Promise.all([
@@ -208,8 +201,7 @@ export default new Action({
       }
     }
     catch (err) {
-      console.error('[dashboard/kanban] BoardShowAction failed:', err)
-      return kanbanError(err instanceof Error ? err.message : 'unknown error', 500)
+      return kanbanActionError(err, 'BoardShowAction')
     }
   },
 })

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

@@ -1,7 +1,8 @@
+import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { db } from '@stacksjs/database'
 import { Board } from '@stacksjs/orm'
-import { kanbanError } from './kanban-response'
+import { kanbanActionError, kanbanError } from './kanban-response'
 
 interface BoardInput {
   name?: unknown
@@ -11,7 +12,7 @@ interface BoardInput {
 }
 
 /**
- * `POST /api/dashboard/kanban/boards` (stacksjs/stacks#1846 Phase 2).
+ * `POST /api/dashboard/kanban/boards`.
  *
  * Creates a new board with sensible defaults and appends it to the
  * end of the boards list (`position = max(position) + 1` so the new
@@ -28,8 +29,8 @@ export default new Action({
   description: 'Creates a new kanban board.',
   method: 'POST',
   apiResponse: true,
-  async handle(request) {
-    const body = (request as any).jsonBody as BoardInput | undefined ?? {}
+  async handle(request: RequestInstance<BoardInput>) {
+    const body = request.all()
 
     const name = typeof body.name === 'string' ? body.name.trim() : ''
     if (!name || name.length > 120) {
@@ -76,8 +77,7 @@ export default new Action({
       }
     }
     catch (err) {
-      console.error('[dashboard/kanban] BoardStoreAction failed:', err)
-      return kanbanError(err instanceof Error ? err.message : 'unknown error', 500)
+      return kanbanActionError(err, 'BoardStoreAction')
     }
   },
 })