Merge pull request #374 from nirmoy/patchscan-e-vs-w-and-26.04

patchscan: fix E: vs W: classification and extend to 26.04_linux-nvidia

Author: ianm-nv

.github/scripts/patchscan

@@ -30,9 +30,10 @@ def get_src_commit_strs(msg):
     # Match common backport/cherry-pick annotations:
     #   (cherry picked from commit <sha>)
     #   (backported from commit <sha>)
-    #   upstream commit <sha>
-    #   Upstream commit <sha>
-    matches = list(re.finditer(r"(?:from commit|[Uu]pstream commit) ([a-fA-F0-9]+)", msg))
+    #   Upstream commit <sha>  (only at start of line, used as a trailer)
+    # Note: "upstream commit <sha>" embedded in prose is intentionally NOT matched
+    # to avoid false positives from commits that reference upstream SHAs informally.
+    matches = list(re.finditer(r"(?:from commit|^[Uu]pstream commit) ([a-fA-F0-9]+)", msg, re.MULTILINE))
     return list(map(lambda m: m.group(1) if len(m.groups()) == 1 else None, matches))
 
 def references_upstream(commit):

.github/workflows/patchscan.yml

@@ -4,6 +4,8 @@ on:
   pull_request_target:
     branches:
       - 24.04_linux-nvidia-6.17-next
+      - 26.04_linux-nvidia-bos
+      - 26.04_linux-nvidia
   workflow_dispatch:
     inputs:
       pr_number:
@@ -106,14 +108,14 @@ jobs:
           sed 's/\x1B\[[0-9;]*[A-Za-z]//g; s/\x1B\]8;;[^\x1B]*\x1B\\//g; s/\x1B\]8;;[^\a]*\a//g' \
             patchscan_output.txt > patchscan_clean.txt
 
-          # Record runtime errors so the comment step can report them before failing
-          if [ $exit_code -ne 0 ]; then
-            echo "fixes_found=error" >> $GITHUB_OUTPUT
-          fi
-
-          # Check if missing fixes or scan errors were found
-          if grep -qE "^W:|^E:|Fixes for" patchscan_clean.txt; then
+          # Classify the outcome — order matters: W: (missing fixes) takes priority,
+          # then E: / non-zero exit (verification errors), then all-clear.
+          # Writing fixes_found twice to GITHUB_OUTPUT would make the last write win,
+          # so use a single mutually-exclusive block.
+          if grep -qE "^W:|Fixes for" patchscan_clean.txt; then
             echo "fixes_found=true" >> $GITHUB_OUTPUT
+          elif grep -qE "^E:" patchscan_clean.txt || [ $exit_code -ne 0 ]; then
+            echo "fixes_found=error" >> $GITHUB_OUTPUT
           else
             echo "fixes_found=false" >> $GITHUB_OUTPUT
           fi
@@ -142,9 +144,12 @@ jobs:
             );
 
             const body = [
-              '## :x: Patchscan: Scan Error',
+              '## :x: Patchscan: Upstream Verification Error',
               '',
-              'Patchscan encountered an error while scanning this PR:',
+              'Patchscan could not fully verify one or more commits in this PR.',
+              'This is often a false positive caused by a SAUCE commit whose message',
+              'body references an upstream SHA but has a different title.',
+              'No `Fixes:` patches appear to be missing.',
               '',
               '````',
               truncated.trim(),
@@ -260,7 +265,7 @@ jobs:
         if: steps.patchscan.outputs.fixes_found == 'true' || steps.patchscan.outputs.fixes_found == 'error'
         run: |
           if [ "${{ steps.patchscan.outputs.fixes_found }}" = "error" ]; then
-            echo "::error::Patchscan encountered a runtime error — see PR comment for details."
+            echo "::error::Patchscan upstream verification error — see PR comment for details."
           else
             echo "::warning::Missing upstream fixes detected — see PR comment for details."
           fi