fix: QA fallback summary drops non-fatal harness error context

[andrewwhitecdw]

This PR addresses the following issue in go/internal/roles/coding/coding.go: QA fallback summary drops non-fatal harness error context.

Changes

• go/internal/roles/coding/coding.go: QA fallback summary drops non-fatal harness error context.

Details

--- a/go/internal/roles/coding/coding.go
+++ b/go/internal/roles/coding/coding.go
@@ -1,7 +1,11 @@
-	return &schemas.QAResult{
-		Passed:       false,
-		Summary:      fmt.Sprintf("QA agent failed for %s", issueName),
-		TestFailures: []map[string]any{},
-		CoverageGaps: []string{},
-		IterationID:  in.IterationID,
-	}, nil
+	summary := fmt.Sprintf("QA agent failed for %s", issueName)
+	if hErr != nil {
+		summary += ": " + hErr.Error()
+	}
+	return &schemas.QAResult{
+		Passed:       false,
+		Summary:      summary,
+		TestFailures: []map[string]any{},
+		CoverageGaps: []string{},
+		IterationID:  in.IterationID,
+	}, nil

Tests

Let me know if you want tests added for this fix or not.