@stacksjs/defaults 0.70.357 → 0.70.362

package/app/Actions/Dashboard/DashboardHomeAction.ts

@@ -2,6 +2,7 @@ import { Action } from '@stacksjs/actions'
 import { Order, Post, Product, Request, User } from '@stacksjs/orm'
 import { checkApplicationHealth, type ApplicationHealthCheck } from '@stacksjs/router'
 import { formatRelative, safeGet } from '../../../resources/functions/dashboard/data'
+import { dashboardOperationalIssue } from './dashboard-response'
 
 interface HttpRequestSample {
   duration: number
@@ -28,7 +29,7 @@ export function serializeHealthCheck(name: string, check: ApplicationHealthCheck
     name: name.charAt(0).toUpperCase() + name.slice(1),
     status: check.ok ? 'healthy' : 'critical',
     latency: `${check.ms}ms`,
-    detail: check.message || '',
+    detail: check.ok ? '' : 'Dependency probe failed.',
   }
 }
 
@@ -43,7 +44,11 @@ function issue(source: string, result: PromiseSettledResult<unknown>) {
   return result.status === 'rejected'
     ? {
         source,
-        message: result.reason instanceof Error ? result.reason.message : 'Query failed.',
+        message: dashboardOperationalIssue(
+          result.reason,
+          `${source} data could not be loaded.`,
+          `DashboardHomeAction.${source.toLowerCase().replaceAll(' ', '-')}`,
+        ),
       }
     : null
 }
@@ -65,7 +70,7 @@ export default new Action({
       Request.count(),
       Request.orderBy('created_at', 'desc').limit(1000).get(),
     ])
-    const health = await checkApplicationHealth()
+    const healthResult = await Promise.allSettled([checkApplicationHealth()])
 
     const [
       userCount,
@@ -93,7 +98,19 @@ export default new Action({
       })))
       : summarizeHttpRequests(0, [])
 
-    const services = Object.entries(health.checks).map(([name, check]) => serializeHealthCheck(name, check))
+    const health = healthResult[0]
+    const services = health.status === 'fulfilled'
+      ? Object.entries(health.value.checks).map(([name, check]) => {
+          if (!check.ok) {
+            dashboardOperationalIssue(
+              check.message,
+              'Dependency probe failed.',
+              `DashboardHomeAction.health.${name}`,
+            )
+          }
+          return serializeHealthCheck(name, check)
+        })
+      : []
 
     const activities = [
       ...(recentOrders.status === 'fulfilled' ? recentOrders.value : []).map((order: any) => ({
@@ -127,6 +144,16 @@ export default new Action({
     const issues = modelResults
       .map((result, index) => issue(sources[index], result))
       .filter((entry): entry is { source: string, message: string } => entry !== null)
+    if (health.status === 'rejected') {
+      issues.push({
+        source: 'System health',
+        message: dashboardOperationalIssue(
+          health.reason,
+          'System health could not be loaded.',
+          'DashboardHomeAction.health',
+        ),
+      })
+    }
 
     return { stats, httpMetrics, services, activities, issues }
   },

package/app/Actions/Dashboard/DashboardStatsAction.ts

@@ -1,5 +1,6 @@
 import { Action } from '@stacksjs/actions'
 import { Customer, Order, Post, User } from '@stacksjs/orm'
+import { dashboardOperationalIssue } from './dashboard-response'
 
 interface DashboardStatDefinition {
   title: string
@@ -61,7 +62,11 @@ export default new Action({
     const issues = results.flatMap((result, index) => result.status === 'rejected'
       ? [{
           source: definitions[index].title,
-          message: result.reason instanceof Error ? result.reason.message : 'Model query failed.',
+          message: dashboardOperationalIssue(
+            result.reason,
+            `${definitions[index].title} data could not be loaded.`,
+            `DashboardStatsAction.${definitions[index].title.toLowerCase().replaceAll(' ', '-')}`,
+          ),
         }]
       : [])
 

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

@@ -3,6 +3,7 @@ import { Action } from '@stacksjs/actions'
 import { notify } from '@stacksjs/notifications'
 import { response } from '@stacksjs/router'
 import { schema } from '@stacksjs/validation'
+import { dashboardOperationalError } from '../dashboard-response'
 
 export default new Action({
   name: 'InboxSendAction',
@@ -29,15 +30,21 @@ export default new Action({
     const subject = String(request.get('subject') || '').trim()
     const body = String(request.get('body') || '').trim()
 
-    const [result] = await notify(
-      { email: to },
-      { subject, body, data: { source: 'dashboard-inbox' } },
-      ['email'],
-      { ignorePreferences: true },
-    )
+    let result
+    try {
+      [result] = await notify(
+        { email: to },
+        { subject, body, data: { source: 'dashboard-inbox' } },
+        ['email'],
+        { ignorePreferences: true },
+      )
+    }
+    catch (error) {
+      return dashboardOperationalError(error, 'The email could not be sent.', 'InboxSendAction.provider', 502)
+    }
 
     if (!result?.success)
-      return response.json({ message: result?.error?.message || 'The email could not be sent.' }, 502)
+      return dashboardOperationalError(result?.error, 'The email could not be sent.', 'InboxSendAction.result', 502)
 
     return response.json({ success: true })
   },

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

@@ -3,6 +3,7 @@ import { existsSync, readFileSync, statSync } from 'node:fs'
 import { join, relative } from 'node:path'
 import process from 'node:process'
 import { parseCommandSource } from '../Source/source-inventory'
+import { dashboardOperationalError } from '../dashboard-response'
 
 interface CommandConfig {
   file: string
@@ -15,35 +16,40 @@ export default new Action({
   description: 'Lists commands registered by the application.',
   method: 'GET',
   async handle() {
-    const projectRoot = process.cwd()
-    const registryPath = join(projectRoot, 'app/Commands.ts')
-    const registryModule = await import(registryPath)
-    const registry = (registryModule.default || {}) as Record<string, string | CommandConfig>
-    const items = Object.entries(registry).flatMap(([signature, value]) => {
-      const config = typeof value === 'string'
-        ? { file: value, enabled: true, aliases: [] }
-        : { enabled: true, aliases: [], ...value }
-      const file = join(projectRoot, 'app/Commands', `${config.file}.ts`)
-      if (!existsSync(file))
-        return []
+    try {
+      const projectRoot = process.cwd()
+      const registryPath = join(projectRoot, 'app/Commands.ts')
+      const registryModule = await import(registryPath)
+      const registry = (registryModule.default || {}) as Record<string, string | CommandConfig>
+      const items = Object.entries(registry).flatMap(([signature, value]) => {
+        const config = typeof value === 'string'
+          ? { file: value, enabled: true, aliases: [] }
+          : { enabled: true, aliases: [], ...value }
+        const file = join(projectRoot, 'app/Commands', `${config.file}.ts`)
+        if (!existsSync(file))
+          return []
 
-      return [parseCommandSource(
-        readFileSync(file, 'utf8'),
-        relative(projectRoot, file),
-        signature,
-        config.aliases,
-        statSync(file).mtime.toISOString(),
-      )]
-    })
+        return [parseCommandSource(
+          readFileSync(file, 'utf8'),
+          relative(projectRoot, file),
+          signature,
+          config.aliases,
+          statSync(file).mtime.toISOString(),
+        )]
+      })
 
-    return {
-      items,
-      stats: {
-        total: items.length,
-        aliases: items.reduce((sum, item) => sum + (item.aliases?.length || 0), 0),
-        options: items.reduce((sum, item) => sum + (item.options?.length || 0), 0),
-        registered: Object.keys(registry).length,
-      },
+      return {
+        items,
+        stats: {
+          total: items.length,
+          aliases: items.reduce((sum, item) => sum + (item.aliases?.length || 0), 0),
+          options: items.reduce((sum, item) => sum + (item.options?.length || 0), 0),
+          registered: Object.keys(registry).length,
+        },
+      }
+    }
+    catch (error) {
+      return dashboardOperationalError(error, 'Command sources could not be loaded.', 'CommandIndexAction')
     }
   },
 })

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

@@ -1,4 +1,5 @@
 import { Action } from '@stacksjs/actions'
+import { dashboardOperationalError } from '../dashboard-response'
 import { readEnvironmentFile } from './environment-file'
 
 export default new Action({
@@ -8,9 +9,14 @@ export default new Action({
   apiResponse: true,
 
   async handle() {
-    return Response.json(
-      { environment: await readEnvironmentFile() },
-      { headers: { 'Cache-Control': 'no-store', Pragma: 'no-cache' } },
-    )
+    try {
+      return Response.json(
+        { environment: await readEnvironmentFile() },
+        { headers: { 'Cache-Control': 'no-store', Pragma: 'no-cache' } },
+      )
+    }
+    catch (error) {
+      return dashboardOperationalError(error, 'Environment file could not be loaded.', 'EnvironmentIndexAction')
+    }
   },
 })

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

@@ -1,6 +1,7 @@
 import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
 import { response } from '@stacksjs/router'
+import { dashboardOperationalError } from '../dashboard-response'
 import { updateEnvironmentFile } from './environment-file'
 
 export default new Action({
@@ -18,7 +19,13 @@ export default new Action({
     if (typeof revision !== 'string' || !/^[a-f0-9]{64}$/.test(revision))
       return response.json({ message: 'A valid environment revision is required.' }, 422)
 
-    const result = await updateEnvironmentFile(content, revision)
+    let result
+    try {
+      result = await updateEnvironmentFile(content, revision)
+    }
+    catch (error) {
+      return dashboardOperationalError(error, 'Environment file could not be saved.', 'EnvironmentUpdateAction', 500)
+    }
     if (result.issues?.length) {
       const errors = Object.fromEntries(result.issues.map((issue, index) => [
         issue.line ? String(issue.line) : `file-${index}`,

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

@@ -1,19 +1,21 @@
 import { Action } from '@stacksjs/actions'
 import { Request } from '@stacksjs/orm'
+import { dashboardOperationalError } from '../dashboard-response'
 
 export default new Action({
   name: 'RequestIndexAction',
   description: 'Returns request history data for the dashboard.',
   method: 'GET',
   async handle() {
-    const oneHourAgo = new Date(Date.now() - 60 * 60 * 1000).toISOString()
-    const [allRequests, total, errorCount, averageDurationMs, requestsLastHour] = await Promise.all([
-      Request.orderByDesc('id').limit(100).get(),
-      Request.count(),
-      Request.where('status_code', '>=', 400).count(),
-      Request.avg('duration_ms'),
-      Request.where('created_at', '>=', oneHourAgo).count(),
-    ])
+    try {
+      const oneHourAgo = new Date(Date.now() - 60 * 60 * 1000).toISOString()
+      const [allRequests, total, errorCount, averageDurationMs, requestsLastHour] = await Promise.all([
+        Request.orderByDesc('id').limit(100).get(),
+        Request.count(),
+        Request.where('status_code', '>=', 400).count(),
+        Request.avg('duration_ms'),
+        Request.where('created_at', '>=', oneHourAgo).count(),
+      ])
 
     const requests = allRequests.map(request => ({
       id: Number(request.get('id') || 0),
@@ -28,16 +30,20 @@ export default new Action({
       createdAt: String(request.get('created_at') || ''),
     }))
 
-    return {
-      requests,
-      stats: {
-        total,
-        errorCount,
-        averageDurationMs: Number(averageDurationMs || 0),
-        successRate: total > 0 ? ((total - errorCount) / total) * 100 : 0,
-        requestsLastHour,
-        requestsPerMinute: requestsLastHour / 60,
-      },
+      return {
+        requests,
+        stats: {
+          total,
+          errorCount,
+          averageDurationMs: Number(averageDurationMs || 0),
+          successRate: total > 0 ? ((total - errorCount) / total) * 100 : 0,
+          requestsLastHour,
+          requestsPerMinute: requestsLastHour / 60,
+        },
+      }
+    }
+    catch (error) {
+      return dashboardOperationalError(error, 'Request history could not be loaded.', 'RequestIndexAction')
     }
   },
 })

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

@@ -1,21 +1,27 @@
 import { Action } from '@stacksjs/actions'
 import { tsCloud } from '~/config/cloud'
 import { getDashboardCloudSnapshot } from '../Cloud/cloud-overview'
+import { dashboardOperationalError } from '../dashboard-response'
 
 export default new Action({
   name: 'ServerIndexAction',
   description: 'Returns configured servers and persisted deployment state.',
   method: 'GET',
   async handle() {
-    const snapshot = await getDashboardCloudSnapshot(tsCloud)
-    return {
-      project: snapshot.project,
-      environments: snapshot.environments,
-      servers: snapshot.serverDefinitions,
-      deployments: snapshot.deployments,
-      network: snapshot.resources.filter(resource => resource.category === 'network'),
-      events: snapshot.events,
-      generatedAt: snapshot.generatedAt,
+    try {
+      const snapshot = await getDashboardCloudSnapshot(tsCloud)
+      return {
+        project: snapshot.project,
+        environments: snapshot.environments,
+        servers: snapshot.serverDefinitions,
+        deployments: snapshot.deployments,
+        network: snapshot.resources.filter(resource => resource.category === 'network'),
+        events: snapshot.events,
+        generatedAt: snapshot.generatedAt,
+      }
+    }
+    catch (error) {
+      return dashboardOperationalError(error, 'Server state could not be loaded.', 'ServerIndexAction')
     }
   },
 })

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

@@ -3,6 +3,7 @@ import { Action } from '@stacksjs/actions'
 import { response } from '@stacksjs/router'
 import { tsCloud } from '~/config/cloud'
 import { getDashboardCloudSnapshot } from '../Cloud/cloud-overview'
+import { dashboardOperationalError } from '../dashboard-response'
 import { resolveDashboardServer } from './server-detail'
 
 export default new Action({
@@ -12,10 +13,16 @@ export default new Action({
   apiResponse: true,
   async handle(request: RequestInstance) {
     const identifier = request.getParam('id')
-    if (!identifier)
+    if (!identifier || !/^[A-Za-z0-9:_-]{1,160}$/.test(identifier))
       return response.json({ error: 'A server identifier is required.' }, 400)
 
-    const snapshot = await getDashboardCloudSnapshot(tsCloud)
+    let snapshot
+    try {
+      snapshot = await getDashboardCloudSnapshot(tsCloud)
+    }
+    catch (error) {
+      return dashboardOperationalError(error, 'Server state could not be loaded.', 'ServerShowAction')
+    }
     const detail = resolveDashboardServer(snapshot, identifier)
     if (!detail)
       return response.json({ error: 'Server state was not found.' }, 404)

package/app/Actions/Dashboard/Marketing/CampaignCancelAction.ts

@@ -1,9 +1,11 @@
 import type { RequestInstance } from '@stacksjs/types'
 import { Action } from '@stacksjs/actions'
-import { campaigns } from '@stacksjs/newsletter'
+import { campaigns, CampaignStateConflictError } from '@stacksjs/newsletter'
 import { Campaign } from '@stacksjs/orm'
 import { response } from '@stacksjs/router'
+import { dashboardOperationalError } from '../dashboard-response'
 import { validateCampaignDelivery } from './campaign-delivery'
+import { marketingRecordId } from './marketing-response'
 
 export default new Action({
   name: 'CampaignCancelAction',
@@ -11,8 +13,17 @@ export default new Action({
   method: 'POST',
 
   async handle(request: RequestInstance) {
-    const id = Number(request.getParam('id'))
-    const campaign = await Campaign.find(id)
+    const id = marketingRecordId(request)
+    if (!id)
+      return response.json({ message: 'A valid campaign id is required.' }, 400)
+
+    let campaign
+    try {
+      campaign = await Campaign.find(id)
+    }
+    catch (error) {
+      return dashboardOperationalError(error, 'Campaign could not be loaded.', 'CampaignCancelAction.read')
+    }
     if (!campaign)
       return response.json({ message: 'Campaign not found.' }, 404)
 
@@ -20,7 +31,14 @@ export default new Action({
     if (validationError)
       return response.json({ message: validationError }, 422)
 
-    await campaigns.cancel(id)
-    return response.json({ id, status: 'cancelled' })
+    try {
+      await campaigns.cancel(id)
+      return response.json({ id, status: 'cancelled' })
+    }
+    catch (error) {
+      if (error instanceof CampaignStateConflictError)
+        return response.json({ message: 'Campaign delivery state changed. Refresh and try again.' }, 409)
+      return dashboardOperationalError(error, 'Campaign could not be cancelled.', 'CampaignCancelAction.cancel', 500)
+    }
   },
 })

package/app/Actions/Dashboard/Marketing/CampaignDestroyAction.ts

@@ -3,6 +3,7 @@ import { Action } from '@stacksjs/actions'
 import { db } from '@stacksjs/database'
 import { Campaign } from '@stacksjs/orm'
 import { response } from '@stacksjs/router'
+import { marketingModelError, marketingRecordId } from './marketing-response'
 
 export default new Action({
   name: 'CampaignDestroyAction',
@@ -10,24 +11,32 @@ export default new Action({
   method: 'DELETE',
 
   async handle(request: RequestInstance) {
-    const id = Number(request.getParam('id'))
-    const campaign = await Campaign.find(id)
-    if (!campaign)
-      return response.json({ message: 'Campaign not found.' }, 404)
+    const id = marketingRecordId(request)
+    if (!id)
+      return response.json({ message: 'A valid campaign id is required.' }, 400)
 
-    const status = String(campaign.get('status') || '')
-    const sendRow = await db
-      .selectFrom('campaign_sends')
-      .select(db.fn.count('id').as('count'))
-      .where('campaign_id', '=', id)
-      .executeTakeFirst()
-    if (Number(sendRow?.count || 0) > 0 || ['scheduled', 'sending', 'sent'].includes(status)) {
-      return response.json({
-        message: 'Scheduled campaigns and campaigns with delivery history cannot be deleted.',
-      }, 409)
-    }
+    try {
+      const campaign = await Campaign.find(id)
+      if (!campaign)
+        return response.json({ message: 'Campaign not found.' }, 404)
+
+      const status = String(campaign.get('status') || '')
+      const sendRow = await db
+        .selectFrom('campaign_sends')
+        .select(db.fn.count('id').as('count'))
+        .where('campaign_id', '=', id)
+        .executeTakeFirst()
+      if (Number(sendRow?.count || 0) > 0 || ['scheduled', 'sending', 'sent'].includes(status)) {
+        return response.json({
+          message: 'Scheduled campaigns and campaigns with delivery history cannot be deleted.',
+        }, 409)
+      }
 
-    await campaign.delete()
-    return response.noContent()
+      await campaign.delete()
+      return response.noContent()
+    }
+    catch (error) {
+      return marketingModelError(error, 'Campaign could not be deleted.', 'CampaignDestroyAction')
+    }
   },
 })

package/app/Actions/Dashboard/Marketing/CampaignIndexAction.ts

@@ -2,6 +2,7 @@ import { Action } from '@stacksjs/actions'
 import { config } from '@stacksjs/config'
 import { db } from '@stacksjs/database'
 import { Campaign, EmailList } from '@stacksjs/orm'
+import { dashboardOperationalError } from '../dashboard-response'
 import { normalizeCampaigns } from './campaign-records'
 
 export default new Action({
@@ -11,49 +12,54 @@ export default new Action({
   apiResponse: true,
 
   async handle() {
-    const [
-      campaigns,
-      lists,
-      membershipRows,
-      sendRows,
-      openedRows,
-      clickedRows,
-    ] = await Promise.all([
-      Campaign.orderByDesc('id').limit(500).get(),
-      EmailList.orderBy('name', 'asc').get(),
-      db
-        .selectFrom('email_list_subscribers')
-        .select(['email_list_id', db.fn.count('id').as('count')])
-        .where('status', '=', 'subscribed')
-        .groupBy('email_list_id')
-        .execute(),
-      db
-        .selectFrom('campaign_sends')
-        .select(['campaign_id', 'status', db.fn.count('id').as('count')])
-        .groupBy(['campaign_id', 'status'])
-        .execute(),
-      db
-        .selectFrom('campaign_sends')
-        .select(['campaign_id', db.fn.count('id').as('count')])
-        .whereNotNull('opened_at')
-        .groupBy('campaign_id')
-        .execute(),
-      db
-        .selectFrom('campaign_sends')
-        .select(['campaign_id', db.fn.count('id').as('count')])
-        .whereNotNull('clicked_at')
-        .groupBy('campaign_id')
-        .execute(),
-    ])
+    try {
+      const [
+        campaigns,
+        lists,
+        membershipRows,
+        sendRows,
+        openedRows,
+        clickedRows,
+      ] = await Promise.all([
+        Campaign.orderByDesc('id').limit(500).get(),
+        EmailList.orderBy('name', 'asc').get(),
+        db
+          .selectFrom('email_list_subscribers')
+          .select(['email_list_id', db.fn.count('id').as('count')])
+          .where('status', '=', 'subscribed')
+          .groupBy('email_list_id')
+          .execute(),
+        db
+          .selectFrom('campaign_sends')
+          .select(['campaign_id', 'status', db.fn.count('id').as('count')])
+          .groupBy(['campaign_id', 'status'])
+          .execute(),
+        db
+          .selectFrom('campaign_sends')
+          .select(['campaign_id', db.fn.count('id').as('count')])
+          .whereNotNull('opened_at')
+          .groupBy('campaign_id')
+          .execute(),
+        db
+          .selectFrom('campaign_sends')
+          .select(['campaign_id', db.fn.count('id').as('count')])
+          .whereNotNull('clicked_at')
+          .groupBy('campaign_id')
+          .execute(),
+      ])
 
-    return normalizeCampaigns(
-      campaigns,
-      lists,
-      membershipRows,
-      sendRows,
-      openedRows,
-      clickedRows,
-      String((config as any).commerce?.currency || 'USD').toUpperCase(),
-    )
+      return normalizeCampaigns(
+        campaigns,
+        lists,
+        membershipRows,
+        sendRows,
+        openedRows,
+        clickedRows,
+        String((config as any).commerce?.currency || 'USD').toUpperCase(),
+      )
+    }
+    catch (error) {
+      return dashboardOperationalError(error, 'Campaigns could not be loaded.', 'CampaignIndexAction')
+    }
   },
 })