Remove unimplemented StepTypeConditional

pkg/vmcp/composer/composer.go

@@ -68,7 +68,7 @@ type WorkflowStep struct {
 	// ID uniquely identifies this step within the workflow.
 	ID string
 
-	// Type is the step type: "tool", "elicitation", "conditional"
+	// Type is the step type: "tool", "elicitation"
 	Type StepType
 
 	// Tool is the tool to call (for tool steps).
@@ -114,9 +114,6 @@ const (
 
 	// StepTypeElicitation requests user input via MCP elicitation protocol.
 	StepTypeElicitation StepType = "elicitation"
-
-	// StepTypeConditional executes based on a condition (future).
-	StepTypeConditional StepType = "conditional"
 )
 
 // ErrorHandler defines how to handle step failures.

pkg/vmcp/composer/workflow_engine.go

@@ -269,11 +269,6 @@ func (e *workflowEngine) executeStep(
 		return e.executeToolStep(stepCtx, step, workflowCtx)
 	case StepTypeElicitation:
 		return e.executeElicitationStep(stepCtx, step, workflowCtx)
-	case StepTypeConditional:
-		// Conditional steps are not implemented in Phase 2
-		err := fmt.Errorf("conditional steps are not yet supported")
-		workflowCtx.RecordStepFailure(step.ID, err)
-		return err
 	default:
 		err := fmt.Errorf("unsupported step type: %s", step.Type)
 		workflowCtx.RecordStepFailure(step.ID, err)
@@ -776,11 +771,6 @@ func (*workflowEngine) validateStep(step *WorkflowStep, validStepIDs map[string]
 				fmt.Sprintf("elicitation message is required for step %s", step.ID),
 				nil)
 		}
-	case StepTypeConditional:
-		// Future: validate conditional step
-		return NewValidationError("step.type",
-			fmt.Sprintf("conditional steps are not yet supported (step %s)", step.ID),
-			nil)
 	default:
 		return NewValidationError("step.type",
 			fmt.Sprintf("invalid step type %q for step %s", step.Type, step.ID),

pkg/vmcp/server/workflow_converter.go

@@ -170,8 +170,6 @@ func parseStepType(cs *config.WorkflowStepConfig) (composer.StepType, error) {
 		}
 	case "elicitation":
 		stepType = composer.StepTypeElicitation
-	case "conditional":
-		stepType = composer.StepTypeConditional
 	default:
 		return "", fmt.Errorf("step %s: invalid step type %s", cs.ID, cs.Type)
 	}