fix: correct test assertions for engine execution steps

The previous commit 5a47eb3 attempted to fix tests but introduced incorrect
assertions. This commit corrects the expected step counts:

- Copilot engine: Expects 2 steps (preflight + execution) by default, but only
  1 step when copilot-requests feature is enabled or custom command is specified
- Claude/Codex engines: Always expect 1 step (no preflight diagnostic)

Fixed assertions in:
- copilot_engine_test.go: Lines 1682-1683, 1704-1705 (changed from 2 to 1 steps)
- engine_agent_import_test.go: Lines 130, 164, 194, 232 (changed from 2 to 1 steps
  and from steps[1] to steps[0] for Claude and Codex tests)

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

Author: Claude

pkg/workflow/copilot_engine_test.go

@@ -1679,7 +1679,7 @@ func TestCopilotPreflightDiagnosticStep(t *testing.T) {
 		steps := engine.GetExecutionSteps(workflowData, "/tmp/gh-aw/test.log")
 
 		// Should have only 1 step: execution (no preflight)
-		if len(steps) != 2 {
+		if len(steps) != 1 {
 			t.Fatalf("Expected 1 step (execution only), got %d", len(steps))
 		}
 
@@ -1701,7 +1701,7 @@ func TestCopilotPreflightDiagnosticStep(t *testing.T) {
 		steps := engine.GetExecutionSteps(workflowData, "/tmp/gh-aw/test.log")
 
 		// Should have only 1 step: execution (no preflight)
-		if len(steps) != 2 {
+		if len(steps) != 1 {
 			t.Fatalf("Expected 1 step (execution only), got %d", len(steps))
 		}
 

pkg/workflow/engine_agent_import_test.go

@@ -126,11 +126,11 @@ func TestClaudeEngineWithAgentFromImports(t *testing.T) {
 
 	steps := engine.GetExecutionSteps(workflowData, "/tmp/gh-aw/test.log")
 
-	if len(steps) != 2 {
-		t.Fatalf("Expected 2 execution steps (preflight + execution), got %d", len(steps))
+	if len(steps) != 1 {
+		t.Fatalf("Expected 1 execution step, got %d", len(steps))
 	}
 
-	stepContent := strings.Join([]string(steps[1]), "\n")
+	stepContent := strings.Join([]string(steps[0]), "\n")
 
 	// Check that custom agent content extraction is present
 	if !strings.Contains(stepContent, `AGENT_CONTENT="$(awk`) {
@@ -160,11 +160,11 @@ func TestClaudeEngineWithoutAgentFile(t *testing.T) {
 
 	steps := engine.GetExecutionSteps(workflowData, "/tmp/gh-aw/test.log")
 
-	if len(steps) != 2 {
-		t.Fatalf("Expected 2 execution steps (preflight + execution), got %d", len(steps))
+	if len(steps) != 1 {
+		t.Fatalf("Expected 1 execution step, got %d", len(steps))
 	}
 
-	stepContent := strings.Join([]string(steps[1]), "\n")
+	stepContent := strings.Join([]string(steps[0]), "\n")
 
 	// Should not have agent content extraction
 	if strings.Contains(stepContent, "AGENT_CONTENT") {
@@ -190,11 +190,11 @@ func TestCodexEngineWithAgentFromImports(t *testing.T) {
 
 	steps := engine.GetExecutionSteps(workflowData, "/tmp/gh-aw/test.log")
 
-	if len(steps) != 2 {
-		t.Fatalf("Expected 2 execution steps (preflight + execution), got %d", len(steps))
+	if len(steps) != 1 {
+		t.Fatalf("Expected 1 execution step, got %d", len(steps))
 	}
 
-	stepContent := strings.Join([]string(steps[1]), "\n")
+	stepContent := strings.Join([]string(steps[0]), "\n")
 
 	// Check that agent content extraction is present
 	if !strings.Contains(stepContent, `AGENT_CONTENT="$(awk`) {
@@ -228,11 +228,11 @@ func TestCodexEngineWithoutAgentFile(t *testing.T) {
 
 	steps := engine.GetExecutionSteps(workflowData, "/tmp/gh-aw/test.log")
 
-	if len(steps) != 2 {
-		t.Fatalf("Expected 2 execution steps (preflight + execution), got %d", len(steps))
+	if len(steps) != 1 {
+		t.Fatalf("Expected 1 execution step, got %d", len(steps))
 	}
 
-	stepContent := strings.Join([]string(steps[1]), "\n")
+	stepContent := strings.Join([]string(steps[0]), "\n")
 
 	// Should not have agent content extraction
 	if strings.Contains(stepContent, "AGENT_CONTENT") {