Switch to artifact-based approach for PR summary comments

wstest.yml:
- Remove git commit/push that was failing due to permissions
- Upload summary file as workflow artifact instead
- Use standard artifact naming: conformance-summary-{TEST_MODE}

post-summary.yml:
- Replace git checkout with artifact download
- Find summary file dynamically in downloaded artifact
- Use artifact file path for PR commenting
- This avoids fork permission issues while maintaining functionality

Author: oberstet

.github/workflows/post-summary.yml

@@ -71,42 +71,38 @@ jobs:
             const summary_filename = `${head_repo}-${head_ref}-${pr_number}-wstest-summary.md`;
             core.setOutput('summary_filename', summary_filename);
 
-      - name: Checkout PR branch
-        uses: actions/checkout@v4
+      - name: Download summary artifact
+        uses: actions/download-artifact@v4
         with:
-          repository: ${{ steps.pr-info.outputs.head_repo }}
-          ref: ${{ steps.pr-info.outputs.head_ref }}
-          fetch-depth: 2  # Need at least 2 commits to detect changes
+          name: conformance-summary-${{ github.event.workflow_run.conclusion == 'success' && 'quick' || env.TEST_MODE || 'quick' }}
+          path: summary-artifact/
+        continue-on-error: true
 
-      - name: Check if summary file exists and was changed
+      - name: Check if summary file exists
         id: file-check
         run: |
-          SUMMARY_FILE=".audit/${{ steps.pr-info.outputs.summary_filename }}"
-          echo "Checking file: $SUMMARY_FILE"
+          echo "==> Looking for summary file in artifact..."
           
-          if [[ ! -f "$SUMMARY_FILE" ]]; then
-            echo "❌ Summary file not found: $SUMMARY_FILE"
+          # Find the summary file in the downloaded artifact
+          SUMMARY_FILE=$(find summary-artifact/ -name "*wstest-summary.md" | head -1)
+          
+          if [[ -z "$SUMMARY_FILE" || ! -f "$SUMMARY_FILE" ]]; then
+            echo "❌ Summary file not found in artifact"
             echo "should_comment=false" >> $GITHUB_OUTPUT
+            ls -la summary-artifact/ || echo "No artifact directory"
             exit 0
           fi
           
-          echo "✅ Summary file found"
-          
-          # Check if file was changed in the last commit (from wstest workflow)
-          if git diff HEAD~1 HEAD --name-only | grep -q "$SUMMARY_FILE"; then
-            echo "✅ Summary file was updated in latest commit"
-            echo "should_comment=true" >> $GITHUB_OUTPUT
-          else
-            echo "ℹ️  Summary file exists but wasn't changed"
-            echo "should_comment=false" >> $GITHUB_OUTPUT
-          fi
+          echo "✅ Summary file found: $SUMMARY_FILE"
+          echo "summary_file_path=$SUMMARY_FILE" >> $GITHUB_OUTPUT
+          echo "should_comment=true" >> $GITHUB_OUTPUT
 
       - name: Comment on PR
         if: steps.file-check.outputs.should_comment == 'true'
         uses: thollander/actions-comment-pull-request@v3
         with:
           pr_number: ${{ steps.pr-info.outputs.pr_number }}
-          filePath: .audit/${{ steps.pr-info.outputs.summary_filename }}
+          filePath: ${{ steps.file-check.outputs.summary_file_path }}
           comment_tag: wstest-conformance-summary
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 

.github/workflows/wstest.yml

@@ -363,38 +363,12 @@ jobs:
           path: docs/_static/websocket/conformance/
           retention-days: 30
 
-      - name: Commit summary to .audit/ directory
-        run: |
-          echo "==> Committing summary file to .audit/ directory..."
-
-          # Check if file exists and if it has changed
-          if git ls-files --error-unmatch $SUMMARY_FILE >/dev/null 2>&1; then
-            # File exists, check if it changed
-            if git diff --quiet HEAD $SUMMARY_FILE; then
-              echo "Summary file unchanged, skipping commit"
-              exit 0
-            else
-              echo "Summary file changed, will update"
-            fi
-          else
-            echo "New summary file, will add"
-          fi
-
-          # Configure git
-          git config user.name "github-actions[bot]"
-          git config user.email "github-actions[bot]@users.noreply.github.com"
-
-          # Create and checkout the branch locally (from detached HEAD)
-          git checkout -b ${{ github.head_ref }}
-
-          # Add and commit the file
-          git add $SUMMARY_FILE
-          git commit -m "Add/update WebSocket conformance summary (Test mode: ${{ env.TEST_MODE }}, Generated: $(date -u '+%Y-%m-%d %H:%M:%S UTC')) for PR #${{ github.event.number }}"
-
-          # Push to the PR branch
-          git push origin ${{ github.head_ref }}
-
-          echo "✅ Summary file committed to $SUMMARY_FILE"
+      - name: Upload summary report for PR comment
+        uses: actions/upload-artifact@v4
+        with:
+          name: conformance-summary-${{ env.TEST_MODE }}
+          path: ${{ env.SUMMARY_FILE }}
+          retention-days: 30
 
       - name: Report summary
         run: |