feat(workflows): add controller availability check for FSM guard

Add hasController flag to workflow context to distinguish between interactive and non-interactive steps
Update FSM guard to allow non-interactive steps without controller
Set controller state during runner initialization

Author: moazbuilds

src/workflows/runner/index.ts

@@ -215,18 +215,24 @@ export class WorkflowRunner implements RunnerContext {
   async run(): Promise<void> {
     debug('[Runner] Starting workflow');
 
-    // Load initial auto mode state
+    // Load initial auto mode and controller state
     // autoMode can work without controller for non-interactive steps (Scenarios 5-6)
-    // For interactive steps without controller, controller provider delegates to user input
+    // For interactive steps, controller is required to enter delegated state
     const controllerState = await loadControllerConfig(this.cmRoot);
     // Check for string 'true' or 'always' (autonomous mode)
     const isAutoMode = controllerState?.autonomousMode === 'true' || controllerState?.autonomousMode === 'always';
+    const hasController = controllerState?.controllerConfig != null;
+
     if (isAutoMode) {
       this.mode.enableAutoMode();
       // Sync machine context with mode state
       this.machine.context.autoMode = true;
     }
 
+    // Set controller availability for FSM guard
+    this.machine.context.hasController = hasController;
+    debug('[Runner] Controller state: autoMode=%s, hasController=%s', isAutoMode, hasController);
+
     // Start the machine
     this.machine.send({ type: 'START' });
     this.emitter.setWorkflowStatus('running');

src/workflows/state/machine.ts

@@ -124,6 +124,7 @@ export function createWorkflowMachine(initialContext: Partial<WorkflowContext> =
     steps: [],
     currentOutput: null,
     autoMode: false,
+    hasController: false,
     paused: false,
     cwd: process.cwd(),
     cmRoot: '',
@@ -154,11 +155,15 @@ export function createWorkflowMachine(initialContext: Partial<WorkflowContext> =
         on: {
           STEP_COMPLETE: [
             // Controller mode -> delegated state
+            // For interactive steps, requires a controller to be configured
+            // For non-interactive steps (interactive: false), controller is not needed (scenarios 5-6)
             {
               target: 'delegated',
               guard: (ctx) => {
                 const step = ctx.steps[ctx.currentStepIndex];
-                return ctx.autoMode && !ctx.paused && step?.interactive !== false;
+                const isInteractive = step?.interactive !== false;
+                // Interactive steps need a controller; non-interactive steps can be fully autonomous
+                return ctx.autoMode && !ctx.paused && (ctx.hasController || !isInteractive);
               },
               action: (ctx, event) => {
                 if (event.type === 'STEP_COMPLETE') {

src/workflows/state/types.ts

@@ -61,6 +61,9 @@ export interface WorkflowContext {
   // Input mode
   autoMode: boolean;
 
+  // Controller availability (for FSM guard)
+  hasController: boolean;
+
   // Pause state
   paused: boolean;