@pikku/inspector 0.12.28 → 0.12.31

package/src/utils/serialize-inspector-state.ts

@@ -46,12 +46,19 @@ export interface SerializableInspectorState {
   >
   wireServicesMeta: Array<[string, string[]]>
   addonRequiredParentServices: string[]
+  addonServerlessIncompatible: Array<[string, string[]]>
   configFactories: Array<
     [
       string,
       { variable: string; type: string | null; typePath: string | null }[],
     ]
   >
+  serverLifecycleFactories: Array<
+    [
+      string,
+      { variable: string; type: string | null; typePath: string | null }[],
+    ]
+  >
   filesAndMethods: InspectorState['filesAndMethods']
   filesAndMethodsErrors: Array<
     [
@@ -312,7 +319,13 @@ export function serializeInspectorState(
     wireServicesFactories: Array.from(state.wireServicesFactories.entries()),
     wireServicesMeta: Array.from(state.wireServicesMeta.entries()),
     addonRequiredParentServices: state.addonRequiredParentServices,
+    addonServerlessIncompatible: Array.from(
+      state.addonServerlessIncompatible.entries()
+    ),
     configFactories: Array.from(state.configFactories.entries()),
+    serverLifecycleFactories: Array.from(
+      state.serverLifecycleFactories.entries()
+    ),
     filesAndMethods: state.filesAndMethods,
     filesAndMethodsErrors: Array.from(
       state.filesAndMethodsErrors.entries()
@@ -491,7 +504,11 @@ export function deserializeInspectorState(
     wireServicesFactories: new Map(data.wireServicesFactories),
     wireServicesMeta: new Map(data.wireServicesMeta),
     addonRequiredParentServices: data.addonRequiredParentServices || [],
+    addonServerlessIncompatible: new Map(
+      data.addonServerlessIncompatible || []
+    ),
     configFactories: new Map(data.configFactories),
+    serverLifecycleFactories: new Map(data.serverLifecycleFactories),
     filesAndMethods: data.filesAndMethods,
     filesAndMethodsErrors: new Map(
       data.filesAndMethodsErrors.map(([key, entries]) => [

package/src/utils/workflow/derive-workflow-plan.test.ts

@@ -119,4 +119,22 @@ describe('deriveWorkflowPlan', () => {
     assert.equal(plan.deterministic, true)
     assert.deepEqual(plan.plannedSteps, [{ stepName: 'a' }])
   })
+
+  test('suspend steps appear in the plan with __workflow_suspend: key and sentence-case displayName', () => {
+    const plan = deriveWorkflowPlan([
+      rpc('build'),
+      { type: 'suspend', reason: 'building' } as WorkflowStepMeta,
+      rpc('publish'),
+      { type: 'suspend', reason: 'awaiting_approval' } as WorkflowStepMeta,
+      { type: 'suspend', reason: 'building-image' } as WorkflowStepMeta,
+    ])
+    assert.equal(plan.deterministic, true)
+    assert.deepEqual(plan.plannedSteps, [
+      { stepName: 'build' },
+      { stepName: '__workflow_suspend:building', displayName: 'Building' },
+      { stepName: 'publish' },
+      { stepName: '__workflow_suspend:awaiting_approval', displayName: 'Awaiting approval' },
+      { stepName: '__workflow_suspend:building-image', displayName: 'Building image' },
+    ])
+  })
 })

package/src/utils/workflow/derive-workflow-plan.ts

@@ -58,6 +58,16 @@ function containsConditional(steps: WorkflowStepMeta[]): boolean {
   return steps.some((step) => step.type === 'branch' || step.type === 'switch')
 }
 
+/**
+ * Turn an internal snake/kebab reason key into a sentence-case display label.
+ * e.g. "awaiting_approval" → "Awaiting approval", "building-image" → "Building image"
+ */
+function reasonToLabel(reason: string): string {
+  return reason
+    .replace(/[-_]/g, ' ')
+    .replace(/^(.)/, (c) => c.toUpperCase())
+}
+
 /** Flatten named steps (rpc/inline/sleep/parallel children) in source order. */
 function collectNamedSteps(steps: WorkflowStepMeta[]): WorkflowPlannedStep[] {
   const planned: WorkflowPlannedStep[] = []
@@ -68,6 +78,16 @@ function collectNamedSteps(steps: WorkflowStepMeta[]): WorkflowPlannedStep[] {
       case 'sleep':
         planned.push({ stepName: step.stepName })
         break
+      case 'suspend':
+        // Runtime stores the suspend step as `__workflow_suspend:${reason}`.
+        // We surface it in the planned ladder with a readable displayName so
+        // the UI shows it in the right position instead of appending it at the
+        // bottom as an unrecognised orphan.
+        planned.push({
+          stepName: `__workflow_suspend:${step.reason}`,
+          displayName: reasonToLabel(step.reason),
+        })
+        break
       case 'parallel':
         for (const child of step.children) {
           planned.push({ stepName: child.stepName })

package/src/utils/workflow/dsl/extract-dsl-workflow.ts

@@ -7,6 +7,7 @@ import type {
   ParallelGroupStepMeta,
   FanoutStepMeta,
   CancelStepMeta,
+  SuspendStepMeta,
   SetStepMeta,
   SwitchStepMeta,
   SwitchCaseMeta,
@@ -21,6 +22,7 @@ import type {
 import {
   isWorkflowDoCall,
   isWorkflowSleepCall,
+  isWorkflowSuspendCall,
   isThrowCancelException,
   extractCancelReason,
   isParallelFanout,
@@ -512,6 +514,10 @@ function extractExpressionStatement(
       return extractSleepStep(call, context)
     }
 
+    if (isWorkflowSuspendCall(call, context.checker)) {
+      return extractSuspendStep(call, context)
+    }
+
     // Check for parallel group or fanout
     if (isParallelFanout(call)) {
       return extractParallelFanout(call, context)
@@ -698,6 +704,31 @@ function extractSleepStep(
   }
 }
 
+/**
+ * Extract suspend step from workflow.suspend() call
+ */
+function extractSuspendStep(
+  call: ts.CallExpression,
+  context: ExtractionContext
+): SuspendStepMeta | null {
+  const args = call.arguments
+  if (args.length < 1) return null
+
+  try {
+    const reason = extractStringLiteral(args[0], context.checker)
+    return {
+      type: 'suspend',
+      reason,
+    }
+  } catch (error) {
+    context.errors.push({
+      message: `Failed to extract suspend step: ${error instanceof Error ? error.message : String(error)}`,
+      node: call,
+    })
+    return null
+  }
+}
+
 /**
  * Extract cancel step from throw WorkflowCancelledException statement
  */

package/src/utils/workflow/dsl/patterns.ts

@@ -42,6 +42,25 @@ export function isWorkflowSleepCall(
   )
 }
 
+/**
+ * Check if a call expression is workflow.suspend()
+ */
+export function isWorkflowSuspendCall(
+  node: ts.CallExpression,
+  _checker: ts.TypeChecker
+): boolean {
+  if (!ts.isPropertyAccessExpression(node.expression)) {
+    return false
+  }
+
+  const propAccess = node.expression
+  return (
+    propAccess.name.text === 'suspend' &&
+    ts.isIdentifier(propAccess.expression) &&
+    propAccess.expression.text === 'workflow'
+  )
+}
+
 /**
  * Check if a throw statement throws WorkflowCancelledException
  * Matches: throw new WorkflowCancelledException(...) or throw WorkflowCancelledException(...)

package/src/visit.ts

@@ -87,6 +87,12 @@ export const visitSetup = (
   )
 
   addFileWithFactory(node, checker, state.configFactories, 'CreateConfig')
+  addFileWithFactory(
+    node,
+    checker,
+    state.serverLifecycleFactories,
+    'ServerLifecycle'
+  )
 
   addRPCInvocations(node, state, logger)
   addWireAddon(node, state, logger)