fix: simplify workflow - remove PR comment, only upload artifacts for fork PRs

Author: JorisHeadease

.github/workflows/generate-c4-diagrams.yml

@@ -48,26 +48,10 @@ jobs:
           fi
       
       - name: Upload diagrams as artifacts
-        # Upload artifacts for PRs from forks (no write access) or for any PR for convenience
-        if: github.event_name == 'pull_request'
+        # Upload artifacts for PRs from forks (they can't push commits)
+        if: github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name != github.repository
         uses: actions/upload-artifact@v4
         with:
           name: c4-diagrams
           path: docs/images/
-          retention-days: 7
-      
-      - name: Comment on PR with generated diagrams
-        # Only comment on PRs from forks where we can't push commits
-        if: github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name != github.repository
-        uses: peter-evans/create-or-update-comment@v4
-        with:
-          token: ${{ secrets.GITHUB_TOKEN }}
-          issue-number: ${{ github.event.pull_request.number }}
-          body: |
-            ## C4 Diagrams Generated
-            
-            The C4 diagrams have been generated for this PR. Since this is from a fork, the diagrams are available as artifacts.
-            
-            **[Download diagrams from workflow artifacts](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }})**
-            
-            Once this PR is merged, the diagrams will be automatically committed to the main branch.
+          retention-days: 7