Merge branch 'mavlink:master' into Stable_V1.0

Author: igoutta

.github/workflows/auto-merge.yml

@@ -1,4 +1,4 @@
-name: Size Check
+name: Build Results
 
 on:
   workflow_run:
@@ -11,12 +11,15 @@ permissions:
   pull-requests: write
 
 jobs:
-  check-size:
+  check-sizes:
     name: Check Artifact Sizes
     runs-on: ubuntu-latest
     if: github.event.workflow_run.conclusion == 'success'
     timeout-minutes: 15
 
+    outputs:
+      sizes-json: ${{ steps.sizes.outputs.json }}
+
     steps:
       - name: Checkout
         uses: actions/checkout@v6
@@ -39,7 +42,7 @@ jobs:
             --output sizes.json \
             --summary "$GITHUB_STEP_SUMMARY" \
             --no-fail
-          python tools/check-sizes.py artifacts/ --markdown > sizes.md
+          echo "json=$(jq -c . sizes.json)" >> $GITHUB_OUTPUT
 
       - name: Upload size report
         uses: actions/upload-artifact@v5
@@ -68,10 +71,10 @@ jobs:
             echo "::warning::Artifact size thresholds exceeded. See size report for details."
           fi
 
-  comment-pr:
-    name: Post Size Comment
+  post-pr-comment:
+    name: Post Build Results
     runs-on: ubuntu-latest
-    needs: check-size
+    needs: check-sizes
     if: github.event.workflow_run.event == 'pull_request'
     timeout-minutes: 10
 
@@ -99,7 +102,67 @@ jobs:
             });
             return prs.data.length > 0 ? prs.data[0].number : null;
 
-      - name: Download PR artifacts
+      - name: Collect build status
+        if: steps.pr.outputs.result != 'null'
+        id: builds
+        uses: actions/github-script@v7
+        with:
+          script: |
+            const headSha = context.payload.workflow_run.head_sha;
+
+            const runs = await github.rest.actions.listWorkflowRunsForRepo({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              head_sha: headSha,
+              per_page: 100
+            });
+
+            const platforms = {
+              'Linux': { status: '⏳', conclusion: 'pending', url: '' },
+              'Windows': { status: '⏳', conclusion: 'pending', url: '' },
+              'MacOS': { status: '⏳', conclusion: 'pending', url: '' },
+              'Android': { status: '⏳', conclusion: 'pending', url: '' }
+            };
+
+            for (const run of runs.data.workflow_runs) {
+              if (platforms[run.name]) {
+                if (run.status === 'completed') {
+                  platforms[run.name].status = run.conclusion === 'success' ? '✅' : '❌';
+                  platforms[run.name].conclusion = run.conclusion;
+                  platforms[run.name].url = run.html_url;
+                } else if (run.status === 'in_progress') {
+                  platforms[run.name].status = '🔄';
+                  platforms[run.name].conclusion = 'in_progress';
+                  platforms[run.name].url = run.html_url;
+                }
+              }
+            }
+
+            let table = '| Platform | Status | Details |\n|----------|--------|--------|\n';
+            for (const [name, info] of Object.entries(platforms)) {
+              const link = info.url ? `[View](${info.url})` : '-';
+              table += `| ${name} | ${info.status} | ${link} |\n`;
+            }
+
+            const allComplete = Object.values(platforms).every(p =>
+              ['success', 'failure', 'cancelled'].includes(p.conclusion)
+            );
+            const allSuccess = Object.values(platforms).every(p => p.conclusion === 'success');
+
+            let summary = '';
+            if (allComplete) {
+              summary = allSuccess
+                ? '✅ All builds passed!'
+                : '❌ Some builds failed.';
+            } else {
+              summary = '⏳ Some builds still in progress...';
+            }
+
+            core.setOutput('table', table);
+            core.setOutput('summary', summary);
+            core.setOutput('all_complete', allComplete);
+
+      - name: Download PR artifacts for size comparison
         if: steps.pr.outputs.result != 'null'
         uses: actions/download-artifact@v5
         with:
@@ -115,12 +178,27 @@ jobs:
           path: baseline-sizes.json
           key: artifact-sizes-baseline-latest
 
-      - name: Compare sizes
+      - name: Generate combined report
         if: steps.pr.outputs.result != 'null'
-        id: compare
         run: |
-          python tools/check-sizes.py artifacts-pr/ --json --output pr-sizes.json --no-fail
-          python3 << 'EOF'
+          cat > comment.md << 'HEADER'
+          ## 🏗️ Build Results
+
+          ### Platform Status
+
+          ${{ steps.builds.outputs.table }}
+          **${{ steps.builds.outputs.summary }}**
+
+          HEADER
+
+          # Generate size report
+          python tools/check-sizes.py artifacts-pr/ --json --output pr-sizes.json --no-fail 2>/dev/null || true
+
+          if [[ -f "pr-sizes.json" ]]; then
+            echo "### 📦 Artifact Sizes" >> comment.md
+            echo "" >> comment.md
+
+            python3 << 'EOF'
           import json
           import os
 
@@ -133,7 +211,7 @@ jobs:
                   baseline_data = json.load(f)
                   baseline = {a['name']: a for a in baseline_data.get('artifacts', [])}
 
-          lines = ['## 📦 Artifact Size Report', '']
+          lines = []
 
           if baseline:
               lines.extend([
@@ -147,10 +225,10 @@ jobs:
               ])
 
           total_delta = 0
-          for a in pr_data['artifacts']:
+          for a in pr_data.get('artifacts', []):
               name = a['name']
               size = a['size_human']
-              status = '⚠️' if a['exceeds_threshold'] else '✅'
+              status = '⚠️' if a.get('exceeds_threshold') else '✅'
 
               if baseline and name in baseline:
                   old_size = baseline[name]['size_bytes']
@@ -167,25 +245,33 @@ jobs:
 
                   lines.append(f'| {name} | {size} | {delta_str} | {status} |')
               else:
-                  threshold = f"{a['threshold_mb']} MB" if a['threshold_mb'] else '-'
+                  threshold = f"{a.get('threshold_mb', '-')} MB" if a.get('threshold_mb') else '-'
                   lines.append(f'| {name} | {size} | {threshold} | {status} |')
 
           lines.append('')
           if baseline and total_delta != 0:
               direction = 'increased' if total_delta > 0 else 'decreased'
               lines.append(f'**Total size {direction} by {abs(total_delta) / 1024 / 1024:.2f} MB**')
           elif not baseline:
-              lines.append('*No baseline available for comparison (first build or cache expired)*')
+              lines.append('*No baseline available for comparison*')
 
-          with open('size-comment.md', 'w') as f:
+          with open('sizes-section.md', 'w') as f:
               f.write('\n'.join(lines))
           EOF
-          cat size-comment.md
 
-      - name: Post PR comment
+            cat sizes-section.md >> comment.md
+          fi
+
+          echo "" >> comment.md
+          echo "---" >> comment.md
+          echo "<sub>Updated: $(date -u '+%Y-%m-%d %H:%M:%S UTC')</sub>" >> comment.md
+
+          cat comment.md
+
+      - name: Post combined comment
         if: steps.pr.outputs.result != 'null'
         uses: thollander/actions-comment-pull-request@v3
         with:
           pr-number: ${{ steps.pr.outputs.result }}
-          comment-tag: artifact-sizes
-          file-path: size-comment.md
+          comment-tag: build-results
+          file-path: comment.md