Fix warnings parsing

Author: ipavlic

pkg/executor/executor.go

@@ -86,9 +86,12 @@ func (e *CLIExecutor) Run(apexCode string, org string) (string, error) {
 		return "", fmt.Errorf("sf apex run failed: %w\nOutput: %s", err, string(output))
 	}
 
+	// Extract JSON from output (skip warning messages)
+	jsonOutput := extractJSON(string(output))
+
 	// Parse JSON response
 	var response ApexRunResponse
-	if err := json.Unmarshal(output, &response); err != nil {
+	if err := json.Unmarshal([]byte(jsonOutput), &response); err != nil {
 		return "", fmt.Errorf("failed to parse sf apex run JSON output: %w\nOutput: %s", err, string(output))
 	}
 
@@ -159,6 +162,17 @@ func (e *CLIExecutor) ExecuteParallel(apexCode string, runs int, maxConcurrent i
 	return results, nil
 }
 
+// extractJSON finds the first JSON object in the output string
+// This handles cases where sf CLI outputs warnings before the JSON
+func extractJSON(output string) string {
+	// Find the first '{' character
+	start := strings.Index(output, "{")
+	if start == -1 {
+		return output
+	}
+	return output[start:]
+}
+
 // createTempApexFile writes Apex code to a temporary file
 func createTempApexFile(apexCode string) (string, error) {
 	tmpFile, err := os.CreateTemp("", "apex-bench-*.apex")

pkg/generator/templates.go

@@ -21,15 +21,15 @@ for (Integer i = 0; i < warmupIterations; i++) {
 // Measurement phase
 Long totalWallTime = 0;
 Long totalCpuTime = 0;
-Long minWallTime = Long.MAX_VALUE;
-Long maxWallTime = 0;
-Integer minCpuTime = Integer.MAX_VALUE;
-Integer maxCpuTime = 0;
+Long minWallTime = null;
+Long maxWallTime = null;
+Integer minCpuTime = null;
+Integer maxCpuTime = null;
 
 {{if .TrackHeap}}
 Long totalHeapUsed = 0;
-Long minHeapUsed = Long.MAX_VALUE;
-Long maxHeapUsed = 0;
+Long minHeapUsed = null;
+Long maxHeapUsed = null;
 {{end}}
 
 {{if .TrackDB}}
@@ -54,8 +54,8 @@ for (Integer i = 0; i < measurementIterations; i++) {
     Long heapAfter = Limits.getHeapSize();
     Long heapDelta = heapAfter - heapBefore;
     totalHeapUsed += heapDelta;
-    if (heapDelta < minHeapUsed) minHeapUsed = heapDelta;
-    if (heapDelta > maxHeapUsed) maxHeapUsed = heapDelta;
+    if (minHeapUsed == null || heapDelta < minHeapUsed) minHeapUsed = heapDelta;
+    if (maxHeapUsed == null || heapDelta > maxHeapUsed) maxHeapUsed = heapDelta;
     {{end}}
 
     Long wallDelta = wallEnd - wallStart;
@@ -64,10 +64,10 @@ for (Integer i = 0; i < measurementIterations; i++) {
     totalWallTime += wallDelta;
     totalCpuTime += cpuDelta;
 
-    if (wallDelta < minWallTime) minWallTime = wallDelta;
-    if (wallDelta > maxWallTime) maxWallTime = wallDelta;
-    if (cpuDelta < minCpuTime) minCpuTime = cpuDelta;
-    if (cpuDelta > maxCpuTime) maxCpuTime = cpuDelta;
+    if (minWallTime == null || wallDelta < minWallTime) minWallTime = wallDelta;
+    if (maxWallTime == null || wallDelta > maxWallTime) maxWallTime = wallDelta;
+    if (minCpuTime == null || cpuDelta < minCpuTime) minCpuTime = cpuDelta;
+    if (maxCpuTime == null || cpuDelta > maxCpuTime) maxCpuTime = cpuDelta;
 }
 
 {{if .TrackDB}}