fix: Make action attempt error and result nullable

codegen/lib/layouts/resources.ts

@@ -1,4 +1,9 @@
-import type { Blueprint, Resource } from '@seamapi/blueprint'
+import type {
+  ActionAttempt,
+  Blueprint,
+  Property,
+  Resource,
+} from '@seamapi/blueprint'
 import { kebabCase, pascalCase } from 'change-case'
 
 export interface ResourceLayoutContext {
@@ -39,7 +44,7 @@ export const getResourceLayoutContexts = (
       ({ resourceType }) => !discriminatedResourceTypes.has(resourceType),
     ),
     ...blueprint.events,
-    ...blueprint.actionAttempts,
+    ...blueprint.actionAttempts.map(normalizeActionAttempt),
   ]
   const resourceTypes = [
     ...new Set(resources.map(({ resourceType }) => resourceType)),
@@ -57,6 +62,27 @@ export const getResourceLayoutContexts = (
   }))
 }
 
+// The blueprint merges the per-status variants and drops the nullability.
+const nullWhilePendingDoc =
+  'Null while the action attempt is pending or when this value does not apply.'
+
+const normalizeActionAttempt = (resource: ActionAttempt): ActionAttempt => ({
+  ...resource,
+  properties: resource.properties.map((property) =>
+    property.name === 'error' || property.name === 'result'
+      ? toStatusDependentProperty(property)
+      : property,
+  ),
+})
+
+const toStatusDependentProperty = (property: Property): Property => ({
+  ...property,
+  isNullable: true,
+  description: [property.description, nullWhilePendingDoc]
+    .filter((part) => part !== '')
+    .join(' '),
+})
+
 const getBatchResourceLayoutContexts = (
   resources: Resource[],
 ): BatchResourceLayoutContext[] => {

src/lib/resolve-action-attempt.ts

@@ -123,9 +123,10 @@ export class SeamActionAttemptFailedError<
   code: string
 
   constructor(actionAttempt: FailedActionAttempt<T>) {
-    super(actionAttempt.error.message, actionAttempt)
+    const { error } = actionAttempt as T
+    super(error?.message ?? 'Action attempt failed', actionAttempt)
     this.name = this.constructor.name
-    this.code = actionAttempt.error.type
+    this.code = error?.type ?? 'unknown'
   }
 }
 
@@ -167,11 +168,15 @@ const isFailedActionAttempt = <T extends ActionAttempt>(
  */
 export type SucceededActionAttempt<T extends ActionAttempt> = T & {
   status: 'success'
+  result: NonNullable<T['result']>
+  error: null
 }
 
 /**
  * An action attempt that has failed.
  */
 export type FailedActionAttempt<T extends ActionAttempt> = T & {
   status: 'error'
+  error: NonNullable<T['error']>
+  result: null
 }