fix(ci): improve JSONL file handling in test validation workflow

Author: mrz1836

.github/workflows/fortress-test-validation.yml

@@ -119,7 +119,7 @@ jobs:
           if find ci-results/ -name "*.jsonl" 2>/dev/null | grep -q .; then
             echo "✅ Found CI results JSONL files"
 
-            for jsonl_file in $(find ci-results/ -name "*.jsonl" 2>/dev/null); do
+            while IFS= read -r -d '' jsonl_file; do
               # Extract artifact directory name from JSONL file path
               # Supported directory structures:
               #   1. Expected: ci-results/ARTIFACT_NAME/.mage-x/ci-results.jsonl
@@ -207,12 +207,12 @@ jobs:
                   TOTAL_FAILURES=$((TOTAL_FAILURES + FAILURE_COUNT))
                 fi
               fi
-            done
+            done < <(find ci-results/ -name "*.jsonl" -print0 2>/dev/null)
           else
             echo "⚠️ No JSONL files found - checking for test-output.log files..."
 
             # Fallback: check test-output.log files for exit codes
-            for log_file in $(find ci-results/ -name "test-output.log" 2>/dev/null); do
+            while IFS= read -r -d '' log_file; do
               echo "📄 Checking: $log_file"
 
               # Look for FAIL indicators
@@ -222,7 +222,7 @@ jobs:
                 FAIL_COUNT=$(grep -c "^--- FAIL:" "$log_file" 2>/dev/null || echo "1")
                 TOTAL_FAILURES=$((TOTAL_FAILURES + FAIL_COUNT))
               fi
-            done
+            done < <(find ci-results/ -name "test-output.log" -print0 2>/dev/null)
           fi
 
           # Final validation result
@@ -266,7 +266,7 @@ jobs:
           TOTAL_UNIQUE=0
           TOTAL_TESTS=0
 
-          for jsonl_file in $(find ci-results/ -name "*.jsonl" 2>/dev/null); do
+          while IFS= read -r -d '' jsonl_file; do
             SUMMARY=$(grep '"type":"summary"' "$jsonl_file" 2>/dev/null | head -1 || echo "")
             if [[ -n "$SUMMARY" ]]; then
               PASSED=$(echo "$SUMMARY" | jq -r '.summary.passed // 0')
@@ -280,7 +280,7 @@ jobs:
               TOTAL_UNIQUE=$((TOTAL_UNIQUE + UNIQUE))
               TOTAL_TESTS=$((TOTAL_TESTS + TOTAL))
             fi
-          done
+          done < <(find ci-results/ -name "*.jsonl" -print0 2>/dev/null)
 
           echo "- **Unique Tests**: $TOTAL_UNIQUE" >> $GITHUB_STEP_SUMMARY
           echo "- **Test Runs**: $TOTAL_TESTS" >> $GITHUB_STEP_SUMMARY
@@ -300,7 +300,7 @@ jobs:
             echo "### Test Matrix Results" >> $GITHUB_STEP_SUMMARY
             echo "" >> $GITHUB_STEP_SUMMARY
 
-            for jsonl_file in $(find ci-results/ -name "*.jsonl" 2>/dev/null); do
+            while IFS= read -r -d '' jsonl_file; do
               # Extract artifact directory name from JSONL file path
               # Supported directory structures:
               #   1. Expected: ci-results/ARTIFACT_NAME/.mage-x/ci-results.jsonl
@@ -332,7 +332,7 @@ jobs:
                   echo "- ❌ **$TEST_LABEL**: $FAILED failures" >> $GITHUB_STEP_SUMMARY
                 fi
               fi
-            done
+            done < <(find ci-results/ -name "*.jsonl" -print0 2>/dev/null)
           fi
 
           # Add detailed failure section if there are failures
@@ -344,7 +344,7 @@ jobs:
             echo "" >> $GITHUB_STEP_SUMMARY
 
             FAILURE_COUNT=0
-            for jsonl_file in $(find ci-results/ -name "*.jsonl" 2>/dev/null); do
+            while IFS= read -r -d '' jsonl_file; do
               while read -r line; do
                 # Limit total failures shown
                 FAILURE_COUNT=$((FAILURE_COUNT + 1))
@@ -406,7 +406,7 @@ jobs:
                 echo "</details>" >> $GITHUB_STEP_SUMMARY
                 echo "" >> $GITHUB_STEP_SUMMARY
               done < <(grep '"type":"failure"' "$jsonl_file" 2>/dev/null)
-            done
+            done < <(find ci-results/ -name "*.jsonl" -print0 2>/dev/null)
           fi
 
       # --------------------------------------------------------------------