fix

Author: lucasssvaz

.github/scripts/sarif_nobuild.sh

@@ -0,0 +1,43 @@
+#!/bin/bash
+
+# This script is used to process the SARIF file to rename files back to .ino and adjust line numbers
+
+# Get the SARIF file and the renamed files mapping
+sarif_file=$1
+renamed_files_file=$2
+
+# Read the renamed files mapping
+renamed_files=$(cat $renamed_files_file)
+
+# Check if files exist
+if [ ! -f "$sarif_file" ] || [ ! -f "$renamed_files_file" ]; then
+    echo "SARIF file or renamed files mapping not found, skipping processing"
+    exit 0
+fi
+
+echo "Processing SARIF file: $sarif_file"
+
+# Create a backup of the original SARIF
+cp "$sarif_file" "${sarif_file}.backup"
+
+# Process the SARIF file to rename files back to .ino and adjust line numbers
+jq --argjson renamed "$renamed_files" '
+.runs[0].results |= map(
+  if .locations[0].physicalLocation.artifactLocation.uri in $renamed then
+    .locations[0].physicalLocation.artifactLocation.uri = $renamed[.locations[0].physicalLocation.artifactLocation.uri] |
+    if .locations[0].physicalLocation.region.startLine then
+      .locations[0].physicalLocation.region.startLine = (.locations[0].physicalLocation.region.startLine - 1)
+    else .
+    end |
+    if .locations[0].physicalLocation.region.endLine then
+      .locations[0].physicalLocation.region.endLine = (.locations[0].physicalLocation.region.endLine - 1)
+    else .
+    end
+  else .
+  end
+)' "$sarif_file" > "${sarif_file}.processed"
+
+# Replace the original SARIF with the processed version
+mv "${sarif_file}.processed" "$sarif_file"
+
+echo "SARIF file processed successfully"

.github/workflows/codeql_nobuild.yml

@@ -79,41 +79,7 @@ jobs:
         if: matrix.language == 'cpp'
         run: |
           sarif_file="sarif-results/${{ matrix.language }}.sarif"
-
-          if [ -f "$sarif_file" ] && [ -f "renamed_files.json" ]; then
-            echo "Processing SARIF file: $sarif_file"
-
-            # Read the renamed files mapping
-            renamed_files=$(cat renamed_files.json)
-
-            # Create a backup of the original SARIF
-            cp "$sarif_file" "${sarif_file}.backup"
-
-            # Process the SARIF file to rename files back to .ino and adjust line numbers
-            jq --argjson renamed "$renamed_files" '
-              .runs[0].results |= map(
-                if .locations[0].physicalLocation.artifactLocation.uri in $renamed then
-                  .locations[0].physicalLocation.artifactLocation.uri = $renamed[.locations[0].physicalLocation.artifactLocation.uri] |
-                  if .locations[0].physicalLocation.region.startLine then
-                    .locations[0].physicalLocation.region.startLine = (.locations[0].physicalLocation.region.startLine - 1)
-                  else .
-                  end |
-                  if .locations[0].physicalLocation.region.endLine then
-                    .locations[0].physicalLocation.region.endLine = (.locations[0].physicalLocation.region.endLine - 1)
-                  else .
-                  end
-                else .
-                end
-              )
-            ' "$sarif_file" > "${sarif_file}.processed"
-
-            # Replace the original SARIF with the processed version
-            mv "${sarif_file}.processed" "$sarif_file"
-
-            echo "SARIF file processed successfully"
-          else
-            echo "SARIF file or renamed files mapping not found, skipping processing"
-          fi
+          .github/scripts/sarif_nobuild.sh "$sarif_file" "renamed_files.json"
 
       - name: Upload SARIF file
         uses: github/codeql-action/upload-sarif@181d5eefc20863364f96762470ba6f862bdef56b # v3.29.2