fix: correct test conditions for preflight step count checks

Fix two categories of incorrectly written tests introduced with the
Copilot pre-flight diagnostic step:

1. copilot_engine_test.go: Two "skips preflight" sub-tests checked
   `len(steps) != 2` but preflight is intentionally skipped, returning
   only 1 step. Change condition to `!= 1`.

2. engine_agent_import_test.go: Claude and Codex tests expected 2 steps
   (preflight + execution) like Copilot, but those engines only return 1
   execution step. Change to `!= 1` and use `steps[0]`.

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

Author: Copilot

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") {