Some test changes.

desrosj · desrosj · commit 33f461f2ce78 · 2025-03-21T08:52:32.000-04:00
diff --git a/.github/workflows/check-built-files.yml b/.github/workflows/check-built-files.yml
@@ -6,7 +6,7 @@ on:
   # runs for pull requests.
   #
   # Other workflows that run on push will detect changes to versioned files and fail.
-  pull_request_target:
+  pull_request:
     branches:
       - trunk
       - '6.[8-9]'
@@ -31,7 +31,7 @@ on:
 concurrency:
   # The concurrency group contains the workflow name and the branch name for pull requests
   # or the commit hash for any other events.
-  group: ${{ github.workflow }}-${{ github.event_name == 'pull_request_target' && github.head_ref || github.sha }}
+  group: ${{ github.workflow }}-${{ github.event_name == 'pull_request' && github.head_ref || github.sha }}
   cancel-in-progress: true
 
 # Disable permissions for all available scopes by default.
@@ -41,11 +41,6 @@ permissions: {}
 jobs:
   update-built-files:
     name: Update built files
-    permissions:
-      contents: write
-      pull-requests: write
-    # This should never be changed to use local reference. Always reference the workflow in trunk.
-    uses: desrosj/wordpress-develop/.github/workflows/reusable-check-built-files.yml@trunk
-    secrets:
-      GH_APP_ID: ${{ secrets.GH_APP_ID }}
-      GH_APP_PRIVATE_KEY: ${{ secrets.GH_APP_PRIVATE_KEY }}
+    if: ${{ github.repository == 'WordPress/wordpress-develop' }}
+    uses: ./.github/workflows/reusable-check-built-files.yml
+
diff --git a/.github/workflows/reusable-check-built-files.yml b/.github/workflows/reusable-check-built-files.yml
@@ -1,24 +1,16 @@
 name: Lint GitHub Actions workflows
 on:
   workflow_call:
-    secrets:
-      GH_APP_ID:
-        description: 'A GitHub App ID.'
-        required: true
-      GH_APP_PRIVATE_KEY:
-        description: 'A GitHub App private key.'
-        required: true
 
 permissions: {}
 
 jobs:
   # Checks a PR for uncommitted changes to built files.
   #
-  # This job uses a GitHub App instead of $GITHUB_TOKEN because Dependabot pull requests are only granted
-  # read-only access.
+  # When changes are detected, the patch and PR number are stored as an artifact for the Commit Built File Changes
+  # workflow to commit and push the changes back for testing.
   #
   # Performs the following steps:
-  # - Generates a token for authenticating with the GitHub App.
   # - Checks out the repository.
   # - Sets up Node.js.
   # - Configures caching for Composer.
@@ -31,63 +23,22 @@ jobs:
   # - Builds WordPress.
   # - Checks for changes to versioned files.
   # - Displays the result of git diff for debugging purposes.
-  # - Configures the Git author.
-  # - Stages changes.
-  # - Commits changes.
-  # - Pushes changes.
+  # - Creates a directory for storing PR data.
+  # - Saves the patch as a file.
+  # - Saves the PR number to a file.
+  # - Uploads the PR data as an artifact.
   update-built-files:
     name: Check and update built files
     runs-on: ubuntu-24.04
     # This prevents an unnecessary second run after changes are committed back because Dependabot always rebases
     # updates and force pushes.
     if: ${{ github.actor != 'dependabot[bot]' || github.event.commits < 2 }}
     timeout-minutes: 10
-    permissions:
-      contents: write
-      pull-requests: write
     steps:
-      - name: Generate Installation Token
-        id: generate_token
-        env:
-          GH_APP_ID: ${{ secrets.GH_APP_ID }}
-          GH_APP_PRIVATE_KEY: ${{ secrets.GH_APP_PRIVATE_KEY }}
-        run: |
-          echo "$GH_APP_PRIVATE_KEY" > private-key.pem
-
-          # Generate JWT
-          JWT=$(python3 - <<EOF
-          import jwt, time
-          private_key = open("private-key.pem", "r").read()
-          payload = {
-              "iat": int(time.time()),
-              "exp": int(time.time()) + 60,  # 10-minute expiration
-              "iss": $GH_APP_ID
-          }
-          print(jwt.encode(payload, private_key, algorithm="RS256"))
-          EOF
-          )
-
-          # Get Installation ID
-          INSTALLATION_ID=$(curl -s -X GET -H "Authorization: Bearer $JWT" \
-            -H "Accept: application/vnd.github.v3+json" \
-            https://api.github.com/app/installations | jq -r '.[0].id')
-
-          # Request Installation Access Token
-          ACCESS_TOKEN=$(curl -s -X POST -H "Authorization: Bearer $JWT" \
-            -H "Accept: application/vnd.github.v3+json" \
-            "https://api.github.com/app/installations/$INSTALLATION_ID/access_tokens" | jq -r '.token')
-
-          echo "ACCESS_TOKEN=$ACCESS_TOKEN" >> "$GITHUB_ENV"
-
-          rm -f private-key.pem
-
       - name: Checkout repository
         uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
         with:
-          repository: ${{ github.event.pull_request.head.repo.full_name }}
-          ref: ${{ github.event.pull_request.head.ref }}
           show-progress: ${{ runner.debug == '1' && 'true' || 'false' }}
-          token: ${{ env.ACCESS_TOKEN }}
 
       - name: Set up Node.js
         uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0
@@ -145,21 +96,26 @@ jobs:
         if: ${{ steps.built-file-check.outputs.uncommitted_changes == 'true' }}
         run: git diff
 
-      - name: Configure git user name and email
+      - name: Create directory for PR data
         if: ${{ steps.built-file-check.outputs.uncommitted_changes == 'true' }}
-        run: |
-          git config user.name "test-wp-build-script-commit[bot]"
-          git config user.email ${{ env.GH_APP_ID }}+test-wp-build-script-commit[bot]@users.noreply.github.com
+        run: mkdir -p ./pr-data
 
-      - name: Stage changes
+      - name: Save diff to a file
         if: ${{ steps.built-file-check.outputs.uncommitted_changes == 'true' }}
-        run: git add .
+        run: git diff > ./pr-data/changes.diff
 
-      - name: Commit changes
+      - name: Save PR number
         if: ${{ steps.built-file-check.outputs.uncommitted_changes == 'true' }}
         run: |
-          git commit -m "Automation: Updating built files with changes. [dependabot skip]"
+          echo "${EVENT_NUMBER}" > ./pr-data/NR
+        env:
+          EVENT_NUMBER: ${{ github.event.number }}
 
-      - name: Push changes
+      # Uploads the PR number and diff as an artifact for the Commit Built File Changes workflow to download and then
+      # commit back for testing.
+      - name: Upload PR data as artifact
+        uses: actions/upload-artifact@65c4c4a1ddee5b72f698fdd19549f0f0fb45cf08 # v4.6.0
         if: ${{ steps.built-file-check.outputs.uncommitted_changes == 'true' }}
-        run: git push
+        with:
+          name: pr-data
+          path: pr-data/
