Address Qodo code review suggestions

Improve PG version check robustness and efficiency per Qodo feedback:

1. Struct detection: Use sed/grep pipeline to better filter comments and
   whitespace, reducing false positives. Isolates struct body boundaries
   more reliably before checking for modifications.

2. File iteration: Use while/read loop instead of for loop to correctly
   handle filenames that contain spaces.

3. Workflow efficiency: Capture script output once and pass between steps
   using GITHUB_OUTPUT multiline strings, eliminating redundant execution.

Changes reduce false positives and improve compatibility while maintaining
the same detection logic.

Author: sensei-hacker

.github/scripts/check-pg-versions.sh

@@ -95,7 +95,13 @@ check_file_for_pg_changes() {
 
             # Check if this struct's typedef was modified
             local struct_pattern="typedef struct ${struct_type%_t}_s"
-            local struct_changes=$(echo "$diff_output" | grep -A50 "$struct_pattern" | grep -E "^[-+]" | grep -v "^[-+]typedef struct" | grep -v "^[-+]}" | grep -v "^[-+]//" | grep -v "^[-+] \*" || true)
+            # Isolate struct body from diff, remove comments and empty lines, then check for remaining changes
+            local struct_body_diff=$(echo "$diff_output" | sed -n "/${struct_pattern}/,/\}.*${struct_type};/p")
+            local struct_changes=$(echo "$struct_body_diff" | grep -E "^[-+]" \
+                | grep -v -E "^[-+]\s*(typedef struct|}|//|\*)" \
+                | sed -E 's://.*$::' \
+                | sed -E 's:/\*.*\*/::' \
+                | tr -d '[:space:]')
 
             if [ -n "$struct_changes" ]; then
                 echo "    ⚠️  Struct definition modified"
@@ -145,9 +151,9 @@ EOF
 }
 
 # Check each changed file
-for file in $CHANGED_FILES; do
+while IFS= read -r file; do
     check_file_for_pg_changes "$file"
-done
+done <<< "$CHANGED_FILES"
 
 # Check if any issues were found
 if [ -s $ISSUES_FILE ]; then

.github/workflows/pg-version-check.yml

@@ -31,8 +31,14 @@ jobs:
         id: pg_check
         run: |
           set +e  # Don't fail the workflow, just capture exit code
-          bash .github/scripts/check-pg-versions.sh
-          echo "exit_code=$?" >> $GITHUB_OUTPUT
+          # Run script and capture output. Exit code 1 is expected for issues.
+          # The output is captured and encoded to be passed between steps.
+          output=$(bash .github/scripts/check-pg-versions.sh 2>&1)
+          exit_code=$?
+          echo "exit_code=${exit_code}" >> $GITHUB_OUTPUT
+          echo "output<<EOF" >> $GITHUB_OUTPUT
+          echo "$output" >> $GITHUB_OUTPUT
+          echo "EOF" >> $GITHUB_OUTPUT
         env:
           GITHUB_BASE_REF: ${{ github.base_ref }}
           GITHUB_HEAD_REF: ${{ github.head_ref }}
@@ -42,16 +48,11 @@ jobs:
         uses: actions/github-script@v7
         with:
           script: |
-            const fs = require('fs');
-
-            // Find the issues file (created by the script)
-            const { execSync } = require('child_process');
+            // Use the captured output from the previous step
+            const output = `${{ steps.pg_check.outputs.output }}`;
             let issuesContent = '';
 
             try {
-              // Re-run script to get issues output
-              const output = execSync('bash .github/scripts/check-pg-versions.sh 2>&1 || true', { encoding: 'utf-8' });
-
               // Extract issues from output (everything after the warning line)
               const lines = output.split('\n');
               let capturing = false;