@stacksjs/defaults 0.72.6 → 0.72.7

package/app/Actions/Dashboard/Commerce/CommerceProductDetailAction.ts

@@ -24,12 +24,12 @@ export default new Action({
   async handle(request: RequestInstance) {
     const id = Number(request.getParam('id'))
     if (!Number.isSafeInteger(id) || id <= 0)
-      return response.notFound({ error: 'Product not found' })
+      return response.notFound('Product not found')
 
     try {
       const product = await Product.find(id)
       if (!product)
-        return response.notFound({ error: 'Product not found' })
+        return response.notFound('Product not found')
 
       const [categories, manufacturers, variants, units, reviews] = await Promise.all([
         Category.orderBy('name', 'asc').limit(500).get(),

package/app/Actions/Dashboard/Commerce/CommerceProductsAction.ts

@@ -22,12 +22,14 @@ export default new Action({
     try {
       const products = await Product.orderByDesc('id').limit(500).get()
       const productIds = products.map(product => commerceRecordIdentifier(product, 'Product'))
+      // Identifiers normalize to strings; `product_id` is numeric.
+      const numericProductIds = productIds.map(Number).filter(Number.isSafeInteger)
       const [categories, manufacturers, variants, units, reviews] = await Promise.all([
         Category.orderBy('name', 'asc').limit(500).get(),
         Manufacturer.orderBy('manufacturer', 'asc').limit(500).get(),
-        productIds.length > 0 ? ProductVariant.where('product_id', 'in', productIds).get() : [],
-        productIds.length > 0 ? ProductUnit.where('product_id', 'in', productIds).get() : [],
-        productIds.length > 0 ? Review.where('product_id', 'in', productIds).get() : [],
+        numericProductIds.length > 0 ? ProductVariant.whereIn('product_id', numericProductIds).get() : [],
+        numericProductIds.length > 0 ? ProductUnit.whereIn('product_id', numericProductIds).get() : [],
+        numericProductIds.length > 0 ? Review.whereIn('product_id', numericProductIds).get() : [],
       ])
       const categoryOptions = categories.map(normalizeProductOption)
       const manufacturerOptions = manufacturers.map(normalizeManufacturerOption)

package/app/Actions/Dashboard/Commerce/DeliveryRouteIndexAction.ts

@@ -16,7 +16,7 @@ export default new Action({
     try {
       const routes = await DeliveryRoute.orderByDesc('last_active').limit(500).get()
       const driverIds = [...new Set(routes.map(route => route.get('driver_id')).filter(Boolean))]
-      const drivers = driverIds.length ? await Driver.where('id', 'in', driverIds).get() : []
+      const drivers = driverIds.length ? await Driver.whereIn('id', driverIds).get() : []
       const driversById = indexDeliveryRouteDrivers(drivers)
       return routes.map(route => normalizeDeliveryRouteRecord(route, driversById))
     }

package/app/Actions/Dashboard/Commerce/DriverIndexAction.ts

@@ -13,7 +13,7 @@ export default new Action({
     try {
       const drivers = await Driver.orderBy('name', 'asc').limit(500).get()
       const userIds = [...new Set(drivers.map(driver => driver.get('user_id')).filter(Boolean))]
-      const users = userIds.length ? await User.where('id', 'in', userIds).get() : []
+      const users = userIds.length ? await User.whereIn('id', userIds).get() : []
       const usersById = indexDriverUsers(users)
       return drivers.map(driver => normalizeDriverRecord(driver, usersById))
     }

package/app/Actions/Dashboard/Commerce/LicenseKeyIndexAction.ts

@@ -21,9 +21,9 @@ export default new Action({
       const productIds = [...new Set(keys.map(key => key.get('product_id')).filter(Boolean))]
       const orderIds = [...new Set(keys.map(key => key.get('order_id')).filter(Boolean))]
       const [customers, products, orders] = await Promise.all([
-        customerIds.length ? Customer.where('id', 'in', customerIds).get() : [],
-        productIds.length ? Product.where('id', 'in', productIds).get() : [],
-        orderIds.length ? Order.where('id', 'in', orderIds).get() : [],
+        customerIds.length ? Customer.whereIn('id', customerIds).get() : [],
+        productIds.length ? Product.whereIn('id', productIds).get() : [],
+        orderIds.length ? Order.whereIn('id', orderIds).get() : [],
       ])
       const customerMap = indexLicenseKeyOptions(customers.map(normalizeLicenseKeyCustomer))
       const productMap = indexLicenseKeyOptions(products.map(normalizeLicenseKeyProduct))

package/app/Actions/Dashboard/Commerce/PaymentRefundAction.ts

@@ -15,9 +15,9 @@ export default new Action({
     const id = Number(request.getParam('id'))
     const amount = request.integer('amount')
     if (!Number.isSafeInteger(id) || id <= 0)
-      return response.notFound({ error: 'Payment not found' })
+      return response.notFound('Payment not found')
     if (!Number.isSafeInteger(amount) || amount <= 0)
-      return response.badRequest({ error: 'Refund amount must be a positive integer in minor units' })
+      return response.badRequest('Refund amount must be a positive integer in minor units')
 
     try {
       const payment = await payments.recordRefund(id, amount)
@@ -26,9 +26,9 @@ export default new Action({
     catch (error) {
       const message = error instanceof Error ? error.message : String(error)
       if (message.includes('not found'))
-        return response.notFound({ error: message })
+        return response.notFound(message)
       if (message.includes('cannot be refunded') || message.includes('exceeds the remaining'))
-        return response.badRequest({ error: message })
+        return response.badRequest(message)
 
       log.error('[dashboard:payments] Failed to record refund', { id, error })
       return response.error('Unable to record the refund.', 500)

package/app/Actions/Dashboard/Commerce/ProductUnitDefaultAction.ts

@@ -26,7 +26,7 @@ export default new Action({
       return dashboardOperationalError(error, 'Product unit default could not be updated.', 'ProductUnitDefaultAction', 500)
     }
     if (!updated)
-      return response.notFound({ error: 'Product unit not found' })
+      return response.notFound('Product unit not found')
 
     return { success: true, id: String(id), isDefault }
   },

package/app/Actions/Dashboard/Commerce/ShippingMethodIndexAction.ts

@@ -18,7 +18,7 @@ export default new Action({
       const methods = await ShippingMethod.orderBy('name', 'asc').limit(500).get()
       const methodIds = shippingMethodIds(methods)
       const zones = methodIds.size
-        ? await ShippingZone.where('shipping_method_id', 'in', [...methodIds]).orderBy('name', 'asc').get()
+        ? await ShippingZone.whereIn('shipping_method_id', [...methodIds]).orderBy('name', 'asc').get()
         : []
       const zonesByMethodId = groupShippingMethodZones(zones, methodIds)
       return methods.map(method => normalizeShippingMethodRecord(method, zonesByMethodId))

package/app/Actions/Dashboard/Commerce/ShippingRateIndexAction.ts

@@ -19,8 +19,8 @@ export default new Action({
       const methodIds = [...new Set(rates.map(rate => rate.get('shipping_method_id')).filter(Boolean))]
       const zoneIds = [...new Set(rates.map(rate => rate.get('shipping_zone_id')).filter(Boolean))]
       const [methods, zones] = await Promise.all([
-        methodIds.length ? ShippingMethod.where('id', 'in', methodIds).get() : [],
-        zoneIds.length ? ShippingZone.where('id', 'in', zoneIds).get() : [],
+        methodIds.length ? ShippingMethod.whereIn('id', methodIds).get() : [],
+        zoneIds.length ? ShippingZone.whereIn('id', zoneIds).get() : [],
       ])
       const methodsById = indexShippingRateMethods(methods)
       const zonesById = indexShippingRateZones(zones)

package/app/Actions/Dashboard/Commerce/ShippingZoneIndexAction.ts

@@ -17,7 +17,7 @@ export default new Action({
       const zones = await ShippingZone.orderBy('name', 'asc').limit(500).get()
       const methodIds = [...new Set(zones.map(zone => zone.get('shipping_method_id')).filter(Boolean))]
       const methods = methodIds.length
-        ? await ShippingMethod.where('id', 'in', methodIds).get()
+        ? await ShippingMethod.whereIn('id', methodIds).get()
         : []
       const methodsById = indexShippingZoneMethods(methods)
       return zones.map(zone => normalizeShippingZoneRecord(zone, methodsById))

package/app/Actions/Dashboard/Commerce/TaxRateDefaultAction.ts

@@ -26,7 +26,7 @@ export default new Action({
       return dashboardOperationalError(error, 'Tax rate default could not be updated.', 'TaxRateDefaultAction', 500)
     }
     if (!updated)
-      return response.notFound({ error: 'Tax rate not found' })
+      return response.notFound('Tax rate not found')
 
     return { success: true, id: String(id), isDefault }
   },

package/app/Actions/Dashboard/Commerce/commerce-dashboard.ts

@@ -198,7 +198,9 @@ function rangeWindow(range: CommerceDashboardRange, now: Date, rows: CommerceDas
     .map(row => timestamp(row.createdAt))
     .filter(Number.isFinite)
     .sort((left, right) => left - right)[0]
-  const start = Number.isFinite(earliest) ? new Date(earliest) : startOfUtcMonth(now)
+  // `earliest` is `number | undefined` (the array may be empty), and
+  // `Number.isFinite` does not narrow it for `new Date(...)`.
+  const start = earliest !== undefined && Number.isFinite(earliest) ? new Date(earliest) : startOfUtcMonth(now)
   const elapsedDays = Math.max(0, (now.getTime() - start.getTime()) / DAY_MS)
   return {
     start,
@@ -255,8 +257,11 @@ function formatCurrencyTotals(totals: Map<string, number>): { value: string, det
   const entries = [...totals.entries()].sort((left, right) => right[1] - left[1])
   if (entries.length === 0)
     return { value: formatMoney(0, 'USD'), detail: 'No recorded revenue' }
-  if (entries.length === 1)
-    return { value: formatMoney(entries[0][1], entries[0][0]), detail: entries[0][0] }
+  // Destructured rather than indexed: a `length === 1` check does not narrow
+  // `entries[0]` for the compiler, and every index below would need a guard.
+  const [only] = entries
+  if (entries.length === 1 && only)
+    return { value: formatMoney(only[1], only[0]), detail: only[0] }
   return {
     value: `${entries.length} currencies`,
     detail: entries.map(([code, amount]) => formatMoney(amount, code)).join(' | '),
@@ -269,10 +274,11 @@ function formatCurrencyAverages(totals: Map<string, { amount: number, orders: nu
     .sort((left, right) => right[1].amount - left[1].amount)
   if (entries.length === 0)
     return { value: formatMoney(0, 'USD'), detail: 'No recorded orders' }
-  if (entries.length === 1)
+  const [only] = entries
+  if (entries.length === 1 && only)
     return {
-      value: formatMoney(entries[0][1].amount / entries[0][1].orders, entries[0][0]),
-      detail: entries[0][0],
+      value: formatMoney(only[1].amount / only[1].orders, only[0]),
+      detail: only[0],
     }
   return {
     value: 'Mixed currencies',
@@ -329,7 +335,10 @@ function buildBuckets(start: Date, end: Date, bucket: RangeWindow['bucket']): Bu
 function singleCurrencyChange(current: Map<string, number>, previous: Map<string, number>): string {
   if (current.size !== 1)
     return ''
-  const [[code, amount]] = [...current.entries()]
+  const [entry] = [...current.entries()]
+  if (!entry)
+    return ''
+  const [code, amount] = entry
   return percentChange(amount, previous.get(code) || 0)
 }
 
@@ -409,12 +418,14 @@ export function buildCommerceDashboard(
     const index = bucketIndexes.get(bucketKey(createdAt, window.bucket))
     if (index === undefined)
       continue
-    orderSeries[index]++
+    // `index` came out of a Map lookup, so the compiler treats these slots as
+    // possibly missing even though the arrays are pre-filled to bucket length.
+    orderSeries[index] = (orderSeries[index] ?? 0) + 1
     if (isCancelled(order.status))
       continue
     const code = currency(order.currency)
     const values = revenueSeries.get(code) || Array.from({ length: buckets.length }, () => 0)
-    values[index] += order.totalAmount
+    values[index] = (values[index] ?? 0) + order.totalAmount
     revenueSeries.set(code, values)
   }
 

package/app/Actions/Dashboard/Content/file-manager.ts

@@ -1,3 +1,4 @@
+import type { ResponseStatus } from '@stacksjs/bun-router'
 import { statfs } from 'node:fs/promises'
 import { posix } from 'node:path'
 import type { StorageAdapter, StorageManager, UploadedFileLike } from '@stacksjs/storage'
@@ -52,10 +53,10 @@ export interface DashboardFileSnapshot {
 }
 
 export class DashboardFileError extends Error {
-  readonly status: number
+  readonly status: ResponseStatus
   readonly fields?: Record<string, string>
 
-  constructor(message: string, status = 422, fields?: Record<string, string>) {
+  constructor(message: string, status: ResponseStatus = 422, fields?: Record<string, string>) {
     super(message)
     this.name = 'DashboardFileError'
     this.status = status

package/app/Actions/Dashboard/DashboardHomeAction.ts

@@ -142,7 +142,7 @@ export default new Action({
 
     const sources = ['Users', 'Products', 'Orders', 'Posts', 'Revenue', 'Recent orders', 'Recent users', 'Request count', 'Recent requests']
     const issues = modelResults
-      .map((result, index) => issue(sources[index], result))
+      .map((result, index) => issue(sources[index] ?? 'Dashboard', result))
       .filter((entry): entry is { source: string, message: string } => entry !== null)
     if (health.status === 'rejected') {
       issues.push({

package/app/Actions/Dashboard/DashboardStatsAction.ts

@@ -57,15 +57,18 @@ export default new Action({
     const results = await Promise.allSettled(definitions.map(definition => definition.count()))
     const stats = definitions.map((definition, index) => {
       const { count: _count, ...metadata } = definition
-      return formatDashboardStat(metadata, results[index])
+      // allSettled returns one result per input, but indexed access is typed
+      // as possibly missing; a rejected placeholder keeps the shape honest.
+      const result = results[index] ?? { status: 'rejected' as const, reason: new Error('missing result') }
+      return formatDashboardStat(metadata, result)
     })
     const issues = results.flatMap((result, index) => result.status === 'rejected'
       ? [{
-          source: definitions[index].title,
+          source: definitions[index]?.title ?? 'Unknown',
           message: dashboardOperationalIssue(
             result.reason,
-            `${definitions[index].title} data could not be loaded.`,
-            `DashboardStatsAction.${definitions[index].title.toLowerCase().replaceAll(' ', '-')}`,
+            `${definitions[index]?.title ?? 'Dashboard'} data could not be loaded.`,
+            `DashboardStatsAction.${(definitions[index]?.title ?? 'unknown').toLowerCase().replaceAll(' ', '-')}`,
           ),
         }]
       : [])

package/app/Actions/Dashboard/Deployments/CreateDeploymentRollback.ts

@@ -3,7 +3,7 @@ import { Action } from '@stacksjs/actions'
 import { response } from '@stacksjs/router'
 import { trackOperatorOperation } from '../Operations/control-plane'
 import { stringValue } from '../Operations/recovery-input'
-import { deploymentRollbackInput, DeploymentRollbackError, executeDeploymentRollback } from './deployment-rollback'
+import { DeploymentRollbackError, deploymentRollbackInput, executeDeploymentRollback, rollbackAuditPayload } from './deployment-rollback'
 
 export default new Action({
   name: 'CreateDeploymentRollback',
@@ -14,7 +14,7 @@ export default new Action({
     const body = request.all() as Record<string, unknown>
     try {
       const input = deploymentRollbackInput(body)
-      const tracked = await trackOperatorOperation(request, 'dashboard.deployments.rollback', input, () => executeDeploymentRollback(input, stringValue(body.revision), stringValue(body.confirmation)))
+      const tracked = await trackOperatorOperation(request, 'dashboard.deployments.rollback', rollbackAuditPayload(input), () => executeDeploymentRollback(input, stringValue(body.revision), stringValue(body.confirmation)))
       return { success: true, ...tracked.result, operation: tracked.operation }
     }
     catch (error) {

package/app/Actions/Dashboard/Deployments/deployment-input.ts

@@ -1,11 +1,11 @@
+import type { ReadableRecord } from '@stacksjs/orm'
 export interface DeploymentCommandInput {
   environment?: unknown
   domain?: unknown
 }
 
-export interface DeploymentRecordLike {
-  get: (key: string) => unknown
-}
+/** The shared shape, kept under this name for the helpers below. */
+export type DeploymentRecordLike = ReadableRecord
 
 export function booleanValue(value: unknown): boolean {
   return value === true || value === 1 || value === '1' || value === 'true' || value === 'on'

package/app/Actions/Dashboard/Deployments/deployment-rollback.ts

@@ -6,6 +6,18 @@ export interface DeploymentRollbackInput {
   release?: string
 }
 
+/**
+ * The same input as a value the operations log can store.
+ *
+ * `JsonValue` has no room for `undefined`, so the optional fields are dropped
+ * rather than recorded as absent-but-present keys.
+ */
+export function rollbackAuditPayload(input: DeploymentRollbackInput): Record<string, string> {
+  return Object.fromEntries(
+    Object.entries(input).filter((entry): entry is [string, string] => typeof entry[1] === 'string'),
+  )
+}
+
 export class DeploymentRollbackError extends Error {
   constructor(message: string) {
     super(message)

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

@@ -181,7 +181,7 @@ export default new Action({
         averageDurationMs: finiteNumber(requestSummary.value?.average),
         maximumDurationMs: finiteNumber(requestSummary.value?.maximum),
         latestAt: requestSummary.value?.latest || '',
-        slowest: slowRequests.value.map(row => ({
+        slowest: slowRequests.value.map((row: Record<string, unknown>) => ({
           id: countValue(row.id),
           method: String(row.method || 'GET'),
           path: safeRequestPath(row.path),
@@ -200,7 +200,7 @@ export default new Action({
         averageDurationMs: finiteNumber(querySummary.value?.average),
         maximumDurationMs: finiteNumber(querySummary.value?.maximum),
         latestAt: querySummary.value?.latest || '',
-        slowest: slowQueries.value.map(row => ({
+        slowest: slowQueries.value.map((row: Record<string, unknown>) => ({
           id: countValue(row.id),
           query: compactSql(row.query),
           durationMs: finiteNumber(row.duration),
@@ -240,7 +240,7 @@ export default new Action({
       errors: {
         total: countValue(errorSummary.value?.total),
         latestAt: errorSummary.value?.latest || '',
-        recent: recentErrors.value.map(row => ({
+        recent: recentErrors.value.map((row: Record<string, unknown>) => ({
           id: countValue(row.id),
           type: String(row.type || 'Error'),
           message: String(row.message || ''),

package/app/Actions/Dashboard/Jobs/job-records.ts

@@ -1,9 +1,9 @@
+import type { ReadableRecord } from '@stacksjs/orm'
 export type JobRecordSource = 'job' | 'failed'
 export type DashboardJobStatus = 'queued' | 'processing' | 'failed'
 
-export interface ModelRecord {
-  get: (key: string) => unknown
-}
+/** The shared shape, kept under this name for the helpers below. */
+export type ModelRecord = ReadableRecord
 
 export interface NormalizedJob {
   id: string

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

@@ -36,7 +36,9 @@ export default new Action({
     if (!name || name.length > 120) {
       return kanbanError('Name is required and must be 1-120 characters.', 400)
     }
-    const description = typeof body.description === 'string' ? body.description.trim() : null
+    // `undefined` rather than `null`: the model treats the column as optional,
+    // and "no description was sent" is not the same as "set it to null".
+    const description = typeof body.description === 'string' ? body.description.trim() : undefined
     const icon = typeof body.icon === 'string' && body.icon ? body.icon : 'rectangle.stack.fill'
     const color = typeof body.color === 'string' && body.color ? body.color : 'violet'
 
@@ -71,8 +73,8 @@ export default new Action({
           position: nextPosition,
           archived: false,
           cardCount: 0,
-          createdAt: board.get('createdAt') ?? board.get('created_at') ?? null,
-          updatedAt: board.get('updatedAt') ?? board.get('updated_at') ?? null,
+          createdAt: board.get('created_at') ?? null,
+          updatedAt: board.get('updated_at') ?? null,
         },
       }
     }

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

@@ -40,7 +40,9 @@ export default new Action({
     if (!title || title.length > 300) {
       return kanbanError('`title` is required and must be 1-300 characters.', 400)
     }
-    const description = typeof body.description === 'string' ? body.description.trim() : null
+    // `undefined` rather than `null`: the model treats the column as optional,
+    // and "no description was sent" is not the same as "set it to null".
+    const description = typeof body.description === 'string' ? body.description.trim() : undefined
     const dueDate = typeof body.dueDate === 'string' && body.dueDate ? body.dueDate : null
 
     try {
@@ -84,8 +86,8 @@ export default new Action({
           createdByUserId,
           dueDate,
           archived: false,
-          createdAt: card.get('createdAt') ?? card.get('created_at') ?? null,
-          updatedAt: card.get('updatedAt') ?? card.get('updated_at') ?? null,
+          createdAt: card.get('created_at') ?? null,
+          updatedAt: card.get('updated_at') ?? null,
         },
       }
     }

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

@@ -99,7 +99,7 @@ export default new Action({
       if (boardIds.size > 1) {
         return kanbanError('All columns in a reorder request must belong to the same board.', 400)
       }
-      const boardId = colRows[0].board_id
+      const boardId = colRows[0]?.board_id
 
       // Verify all cards belong to that board (prevents a malformed
       // page from moving cards in from a sibling board).

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

@@ -1,6 +1,6 @@
-export interface KanbanModelRecord {
-  get: (key: string) => unknown
-}
+import type { ReadableRecord } from '@stacksjs/orm'
+/** The shared shape, kept under this name for the helpers below. */
+export type KanbanModelRecord = ReadableRecord
 
 export interface RefreshableKanbanModelRecord extends KanbanModelRecord {
   fresh: () => Promise<KanbanModelRecord | null>

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

@@ -1,6 +1,7 @@
+import type { ResponseStatus } from '@stacksjs/bun-router'
 import { response } from '@stacksjs/router'
 
-export function kanbanError(error: string, status: number): Response {
+export function kanbanError(error: string, status: ResponseStatus): Response {
   return response.json({ error }, status)
 }
 

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

@@ -44,6 +44,7 @@ function dependencySnapshot() {
 
   for (const wsKey of Object.keys(workspaces)) {
     const ws = workspaces[wsKey]
+    if (!ws) continue
     const allDeps = { ...ws.dependencies, ...ws.devDependencies }
     for (const depName of Object.keys(allDeps)) {
       if (declaredSet[depName]) continue

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

@@ -12,13 +12,18 @@ export default new Action({
     try {
       const allReleases = await Release.orderByDesc('id').limit(500).get()
 
+      // Only columns the Release model actually declares. `released_at`,
+      // `changes`, `change_count` and `breaking` were read here but have never
+      // existed on the model, so they resolved to '' / 0 / false on every row
+      // - the same fabricated-metric problem 9882e7ede removed from the rest
+      // of this dashboard, left behind in this one action. Nothing consumed
+      // them.
       const releases = allReleases.map(r => ({
         version: String(r.get('version') || ''),
-        date: String(r.get('created_at') || r.get('released_at') || ''),
+        date: String(r.get('created_at') || ''),
         type: String(r.get('type') || 'patch'),
         status: String(r.get('status') || 'previous'),
-        changes: Number(r.get('changes') || r.get('change_count') || 0),
-        breaking: Boolean(r.get('breaking') || false),
+        notes: String(r.get('notes') || ''),
       }))
 
       return { releases }

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

@@ -28,9 +28,9 @@ export default new Action({
         ...data,
         audience_size: 0,
         sent_count: 0,
-        open_rate: null,
-        click_rate: null,
-        conversion_rate: null,
+        // A campaign that has not been sent has no rates yet. The columns are
+        // nullable in the database but the model types them as optional
+        // numbers, so leaving them out says "not measured" in both.
       })
       return response.json({ id: campaign.get('id') }, 201)
     }

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

@@ -17,9 +17,16 @@ export default new Action({
     if (validationError)
       return response.json({ message: validationError }, 422)
 
+    // A social post belongs to a User, and the column is NOT NULL - so an
+    // author-less payload used to pass validation and then fail at the
+    // database as a 500. Say so plainly instead.
+    if (data.user_id === null)
+      return response.json({ message: 'A social post needs an author.' }, 422)
+
     try {
       const post = await SocialPost.create({
         ...data,
+        user_id: data.user_id,
         platform: data.platform as Exclude<typeof data.platform, ''>,
         likes: 0,
         shares: 0,

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

@@ -25,8 +25,12 @@ export default new Action({
       if (!post)
         return response.json({ message: 'Social post not found.' }, 404)
 
+      // Same NOT NULL author column as on create; an update that did not
+      // name one leaves the existing author alone rather than clearing it.
+      const { user_id: authorId, ...rest } = data
       await post.update({
-        ...data,
+        ...rest,
+        ...(authorId === null ? {} : { user_id: authorId }),
         platform: data.platform as Exclude<typeof data.platform, ''>,
       })
       return response.json({ id })

package/app/Actions/Dashboard/Models/ModelsIndexAction.ts

@@ -84,7 +84,7 @@ function walkModels(dir: string, recursive: boolean): Array<{ name: string, file
 function categorize(model: { source: 'userland' | 'framework', file: string }): string {
   if (model.source === 'userland') return 'userland'
   const segs = model.file.split('/Models/')[1]?.split('/') ?? []
-  if (segs.length > 1) return segs[0].toLowerCase()
+  if (segs.length > 1 && segs[0]) return segs[0].toLowerCase()
   return 'framework'
 }
 
@@ -197,19 +197,22 @@ export default new Action({
     const userlandCount = enriched.filter(m => m.source === 'userland').length
     const unavailableModels = enriched.filter(m => m.error !== null).length
 
-    const groupMap: Record<string, ModelRow[]> = {}
+    const groupMap = new Map<string, ModelRow[]>()
     for (const m of enriched) {
-      if (!groupMap[m.category]) groupMap[m.category] = []
-      groupMap[m.category].push(m)
+      const bucket = groupMap.get(m.category) ?? []
+      bucket.push(m)
+      groupMap.set(m.category, bucket)
     }
 
-    const categoryGroups: ModelGroup[] = Object.keys(groupMap)
-      .sort((a, b) => categoryOrder(a) - categoryOrder(b))
-      .map(key => ({
+    // A Map rather than a plain object: every read here is by a key that was
+    // just written, and an index signature makes the compiler doubt all four.
+    const categoryGroups: ModelGroup[] = [...groupMap.entries()]
+      .sort(([a], [b]) => categoryOrder(a) - categoryOrder(b))
+      .map(([key, models]) => ({
         key,
         label: categoryLabel(key),
-        models: groupMap[key],
-        countLabel: `${groupMap[key].length} model${groupMap[key].length === 1 ? '' : 's'}`,
+        models,
+        countLabel: `${models.length} model${models.length === 1 ? '' : 's'}`,
       }))
 
     return {

package/app/Actions/Dashboard/Operations/control-plane.ts

@@ -2,9 +2,12 @@ import type { RequestInstance } from '@stacksjs/types'
 import type {
   ControlPlaneActor,
   ControlPlaneOperation,
-  DashboardControlPlane,
   JsonValue,
 } from '@stacksjs/ts-cloud'
+// Lives on the `/deploy` entry alongside `initializeDashboardControlPlane`,
+// not on the package root - importing it from the root silently produced
+// `any`, and every caller of this helper inherited that.
+import type { DashboardControlPlane } from '@stacksjs/ts-cloud/deploy'
 import process from 'node:process'
 import { setStateDir } from '@stacksjs/ts-cloud'
 import { initializeDashboardControlPlane } from '@stacksjs/ts-cloud/deploy'
@@ -48,7 +51,16 @@ export function operationsEnvironment(controlPlane = operationsControlPlane()) {
     ?? controlPlane.environments.values().next().value
 }
 
-export async function trackOperatorOperation<T extends JsonValue>(
+/**
+ * `T` is only required to be serializable, not to BE a `JsonValue`.
+ *
+ * An interface without an index signature - `Alert`, `ReconcileResult` - never
+ * satisfies `{ [key: string]: JsonValue }` structurally, however plainly
+ * JSON-shaped its fields are. Constraining the result that way meant the
+ * honest callers were the ones that failed to typecheck, so the constraint is
+ * dropped here and the value is narrowed once, where it is actually recorded.
+ */
+export async function trackOperatorOperation<T>(
   request: RequestInstance,
   kind: string,
   input: JsonValue,
@@ -83,7 +95,7 @@ export async function trackOperatorOperation<T extends JsonValue>(
     operation = controlPlane.store.transitionOperation(operation.id, {
       to: 'succeeded',
       expectedVersion: operation.version,
-      output: result,
+      output: result as JsonValue,
     })
     controlPlane.store.appendEvent({
       organizationId: controlPlane.organization.id,
@@ -92,7 +104,7 @@ export async function trackOperatorOperation<T extends JsonValue>(
       actorId: actor.id,
       correlationId: operation.correlationId,
       type: `${kind}.succeeded`,
-      payload: result,
+      payload: result as JsonValue,
     })
     return { result, operation }
   }

package/app/Actions/Dashboard/Queries/QueryDashboardAction.ts

@@ -23,7 +23,7 @@ export default new Action({
       return {
         enabled: queryLogging?.enabled === true,
         slowThreshold: queryLogging?.slowThreshold || 100,
-        queries: rows.map(row => mapDashboardQueryLog(row as QueryLogSourceRow)),
+        queries: rows.map((row: unknown) => mapDashboardQueryLog(row as QueryLogSourceRow)),
       }
     }
     catch (error) {