Remove search for upstream pr and create an output message step

Author: mirkokurt

.github/workflows/calculate-size-delta.yml

@@ -36,42 +36,22 @@ jobs:
         run: |
           if [ "${{ github.event_name }}" == "workflow_dispatch" ]; then
             echo "Searching for PR from branch '${{ github.ref_name }}'..."
-            UPSTREAM_REPO=$(gh api repos/${{ github.repository }} --jq '.parent.full_name // empty')
-            
-            if [ -n "$UPSTREAM_REPO" ]; then
-              echo "This is a fork. Upstream repository: $UPSTREAM_REPO"
-              # Get current repo owner and branch
-              CURRENT_USER=$(gh api repos/${{ github.repository }} --jq '.owner.login')
-              BRANCH_NAME="${{ github.ref_name }}"
-              echo "Searching in upstream for PR from $CURRENT_USER:$BRANCH_NAME"
-              
-              # Use API to search for PR with matching head
-              PR_NUMBER=$(gh api "repos/$UPSTREAM_REPO/pulls?state=open&head=$CURRENT_USER:$BRANCH_NAME" --jq '.[0].number // empty')
-              
-              if [ -z "$PR_NUMBER" ]; then
-                echo "Not found with API, trying gh pr list..."
-                PR_NUMBER=$(gh pr list --repo "$UPSTREAM_REPO" --state open --json number,headRefName,headRepositoryOwner \
-                  --jq ".[] | select(.headRefName == \"$BRANCH_NAME\" and .headRepositoryOwner.login == \"$CURRENT_USER\") | .number")
-              fi
-              TARGET_REPO="$UPSTREAM_REPO"
-            else
-              echo "This is not a fork. Searching in current repo..."
-              PR_NUMBER=$(gh pr list --state open --head "${{ github.ref_name }}" --json number --jq '.[0].number // empty')
-              TARGET_REPO="${{ github.repository }}"
+            PR_NUMBER=$(gh pr list --state open --head "${{ github.ref_name }}" --json number --jq '.[0].number // empty')
+            if [ -z "$PR_NUMBER" ]; then
+              echo "Not found in current repo, searching across forks..."
+              PR_NUMBER=$(gh pr list --state open --json number,headRefName,headRepositoryOwner --jq ".[] | select(.headRefName == \"${{ github.ref_name }}\") | .number" | head -n1)
             fi
           else
             # For issue_comment, the PR number is in the event context
             PR_NUMBER=${{ github.event.issue.number }}
-            TARGET_REPO="${{ github.repository }}"
           fi
 
           if [ -z "$PR_NUMBER" ]; then
             echo "Could not find an associated open pull request."
           else
-            echo "Found PR #$PR_NUMBER in repo $TARGET_REPO"
+            echo "Found PR #$PR_NUMBER"
           fi
           echo "pr_number=$PR_NUMBER" >> $GITHUB_OUTPUT
-          echo "target_repo=$TARGET_REPO" >> $GITHUB_OUTPUT
 
       - name: Checkout PR Branch (for comment trigger)
         if: github.event_name == 'issue_comment'
@@ -146,13 +126,12 @@ jobs:
           echo "" >> $GITHUB_STEP_SUMMARY
           echo "| Image | Size |" >> $GITHUB_STEP_SUMMARY
           echo "|-------|------|" >> $GITHUB_STEP_SUMMARY
-          echo "| app-bricks/python-base | $(docker images 'localhost:5000/app-bricks/python-base:latest' --format '{{.Size}}') |" >> $GITHUB_STEP_SUMMARY
-          echo "| app-bricks/python-apps-base | $(docker images 'localhost:5000/app-bricks/python-apps-base:latest' --format '{{.Size}}') |" >> $GITHUB_STEP_SUMMARY
+          echo "| app-bricks/python-base | ${{ steps.sizes.outputs.python_base_size }} |" >> $GITHUB_STEP_SUMMARY
+          echo "| app-bricks/python-apps-base | ${{ steps.sizes.outputs.python_apps_base_size }} |" >> $GITHUB_STEP_SUMMARY
     outputs:
       python_base_size: ${{ steps.sizes.outputs.python_base_size }}
       python_apps_base_size: ${{ steps.sizes.outputs.python_apps_base_size }}
       pr_number: ${{ steps.pr_info.outputs.pr_number }}
-      target_repo: ${{ steps.pr_info.outputs.target_repo }}
   comment-results:
     runs-on: ubuntu-latest
     needs: build
@@ -164,18 +143,26 @@ jobs:
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |
-          if gh pr comment ${{ needs.build.outputs.pr_number }} --repo "${{ needs.build.outputs.target_repo }}" --body-file - <<EOF
+          gh pr comment ${{ needs.build.outputs.pr_number }} --repo ${{ github.repository }} --body-file - <<EOF
           ## Docker Image Sizes
 
           | Image | Size |
           |-------|------|
           | app-bricks/python-base | ${{ needs.build.outputs.python_base_size }} |
           | app-bricks/python-apps-base | ${{ needs.build.outputs.python_apps_base_size }} |
           EOF
-          then
-            echo "✅ Comment posted successfully"
-          else
-            echo "⚠️ Could not post comment (likely permission issue for cross-repo commenting)"
-            echo "Image sizes are available in the job summary of the build-images job"
-            exit 0
-          fi
+  notify-no-pr:
+    runs-on: ubuntu-latest
+    needs: build
+    if: needs.build.outputs.pr_number == ''
+    permissions: {}
+    
+    steps:
+      - name: Notify to check Job Summary
+        run: |
+          echo "ℹ️ No PR found for this workflow run."
+          echo "📊 Image sizes are available in the Job Summary of the 'build' job."
+          echo ""
+          echo "To view:"
+          echo "1. Go to the 'build' job above"
+          echo "2. Scroll to the bottom to see the Job Summary"