fix: ensure process exits with error code if results contain errors in reporting functions

Author: shyim

internal/verifier/reporter.go

@@ -30,10 +30,6 @@ func DoCheckReport(result *Check, reportingFormat string) error {
 		return doJUnitReport(result)
 	}
 
-	if result.HasErrors() {
-		os.Exit(1)
-	}
-
 	return nil
 }
 
@@ -72,6 +68,10 @@ func doSummaryReport(result *Check) error {
 	//nolint:forbidigo
 	fmt.Printf("\n✖ %d problems (%d errors, %d warnings)\n", totalProblems, errorCount, warningCount)
 
+	if result.HasErrors() {
+		os.Exit(1)
+	}
+
 	return nil
 }
 
@@ -85,6 +85,10 @@ func doJSONReport(result *Check) error {
 		return fmt.Errorf("failed to write JSON output: %w", err)
 	}
 
+	if result.HasErrors() {
+		os.Exit(1)
+	}
+
 	return nil
 }
 
@@ -107,6 +111,10 @@ func doGitHubReport(result *Check) error {
 		}
 	}
 
+	if result.HasErrors() {
+		os.Exit(1)
+	}
+
 	return nil
 }
 
@@ -196,6 +204,10 @@ func doJUnitReport(result *Check) error {
 		return fmt.Errorf("failed to write JUnit XML: %w", err)
 	}
 
+	if result.HasErrors() {
+		os.Exit(1)
+	}
+
 	return nil
 }
 
@@ -204,6 +216,10 @@ func doMarkdownReport(result *Check) error {
 		return fmt.Errorf("failed to write markdown output: %w", err)
 	}
 
+	if result.HasErrors() {
+		os.Exit(1)
+	}
+
 	return nil
 }