Combine reports from all @phlexbot format workflows

.github/actions/handle-fix-commit/action.yaml

@@ -39,6 +39,10 @@ inputs:
     description: "The number of times to retry pushing the commit."
     required: false
     default: "6"
+  skip-comment:
+    description: "Skip posting PR comments (for workflow_call usage)"
+    required: false
+    default: "false"
 
 runs:
   using: "composite"
@@ -55,7 +59,7 @@ runs:
         fi
 
     - name: No changes to apply
-      if: steps.check_changes.outputs.changes == 'false'
+      if: steps.check_changes.outputs.changes == 'false' && inputs.skip-comment != 'true'
       uses: thollander/actions-comment-pull-request@24bffb9b452ba05a4f3f77933840a6a841d1b32b # v3.0.1
       with:
         message: "No automatic ${{ inputs.tool }} fixes were necessary."
@@ -125,7 +129,9 @@ runs:
         exit 1
 
     - name: Notify of commit
-      if: steps.commit_and_push.conclusion == 'success' && steps.commit_and_push.outputs.pushed == 'true'
+      if:
+        steps.commit_and_push.conclusion == 'success' && steps.commit_and_push.outputs.pushed == 'true' &&
+        inputs.skip-comment != 'true'
       uses: thollander/actions-comment-pull-request@24bffb9b452ba05a4f3f77933840a6a841d1b32b # v3.0.1
       with:
         # yamllint disable rule:line-length
@@ -153,7 +159,7 @@ runs:
         path: ${{ inputs.working-directory }}/${{ steps.create_patch.outputs.patch_name }}
 
     - name: Comment with patch instructions
-      if: steps.create_patch.outputs.patch_name
+      if: steps.create_patch.outputs.patch_name && inputs.skip-comment != 'true'
       uses: thollander/actions-comment-pull-request@24bffb9b452ba05a4f3f77933840a6a841d1b32b # v3.0.1
       with:
         # yamllint disable rule:line-length

.github/workflows/clang-format-fix.yaml

@@ -8,35 +8,58 @@ run-name: "${{ github.actor }} fixing C++ code format"
   workflow_dispatch:
     inputs:
       ref:
-        description: "The branch, ref, or SHA to checkout. Defaults to the repository's default branch."
+        description: "The branch name to checkout and push fixes to (must be a branch, not a commit SHA). Defaults to the repository's default branch."
         required: false
         type: string
+  workflow_call:
+    inputs:
+      checkout-path:
+        description: "Path to check out code to"
+        required: false
+        type: string
+      ref:
+        description: "The branch name to checkout and push fixes to (must be a branch, not a commit SHA)"
+        required: true
+        type: string
+      repo:
+        description: "The repository to checkout from"
+        required: true
+        type: string
+      skip-comment:
+        description: "Skip posting PR comments"
+        required: false
+        type: string
+        default: "false"
 
 permissions:
   pull-requests: write
   contents: write
 
+env:
+  local_checkout_path:
+    ${{ (github.event_name == 'workflow_call' && inputs.checkout-path) || format('{0}-src',
+    github.event.repository.name) }}
+
 jobs:
   pre-check:
     runs-on: ubuntu-latest
     name: Parse command
     if: >
-      github.event_name == 'workflow_dispatch' || (
+      github.event_name == 'workflow_dispatch' || github.event_name == 'workflow_call' || (
         github.event_name == 'issue_comment' &&
         github.event.issue.pull_request &&
         contains(fromJSON('["OWNER", "COLLABORATOR", "MEMBER"]'), github.event.comment.author_association) &&
-        (
-          startsWith(github.event.comment.body, format('@{0}bot format', github.event.repository.name)) ||
-          startsWith(github.event.comment.body, format('@{0}bot clang-fix', github.event.repository.name))
-        )
+        startsWith(github.event.comment.body, format('@{0}bot clang-fix', github.event.repository.name))
       )
     # Authorization: Only OWNER, COLLABORATOR, or MEMBER can trigger via comments.
     # This covers repo owners, invited collaborators, and all org members.
     outputs:
       ref:
-        ${{ (github.event_name == 'workflow_dispatch' && (github.event.inputs.ref || github.ref)) ||
-        steps.get_pr.outputs.ref }}
-      repo: ${{ steps.get_pr.outputs.repo || github.repository }}
+        ${{ (github.event_name == 'workflow_call' && inputs.ref) || (github.event_name == 'workflow_dispatch' &&
+        (github.event.inputs.ref || github.ref_name)) || steps.get_pr.outputs.ref }}
+      repo:
+        ${{ (github.event_name == 'workflow_call' && inputs.repo) || (github.event_name == 'workflow_dispatch' &&
+        github.repository) || steps.get_pr.outputs.repo }}
 
     steps:
       - name: Get PR Info
@@ -49,26 +72,66 @@ jobs:
     name: Apply formatting
     needs: pre-check
     if: ${{ needs.pre-check.result == 'success' }}
+    outputs:
+      changes: ${{ steps.handle_commit.outputs.changes }}
+      pushed: ${{ steps.handle_commit.outputs.pushed }}
+      commit_sha: ${{ steps.handle_commit.outputs.commit_sha }}
+      commit_sha_short: ${{ steps.handle_commit.outputs.commit_sha_short }}
+      patch_name: ${{ steps.handle_commit.outputs.patch_name }}
     steps:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
         with:
-          path: phlex-src
+          path: ${{ env.local_checkout_path }}
           ref: ${{ needs.pre-check.outputs.ref }}
           repository: ${{ needs.pre-check.outputs.repo }}
           token: ${{ secrets.WORKFLOW_PAT }}
 
       - uses: DoozyX/clang-format-lint-action@bcb4eb2cb0d707ee4f3e5cc3b456eb075f12cf73 # v0.20
         with:
-          source: "./phlex-src"
+          source: "./${{ env.local_checkout_path }}"
           clangFormatVersion: 20
           inplace: "True"
           extensions: cpp,hpp,cpp.in,hpp.in
 
       - name: Handle fix commit
+        id: handle_commit
         uses: Framework-R-D/phlex/.github/actions/handle-fix-commit@main
         with:
           tool: clang-format
-          working-directory: phlex-src
+          working-directory: ${{ env.local_checkout_path }}
           token: ${{ secrets.WORKFLOW_PAT }}
           pr-info-ref: ${{ needs.pre-check.outputs.ref }}
           pr-info-repo: ${{ needs.pre-check.outputs.repo }}
+          skip-comment: ${{ github.event_name == 'workflow_call' && inputs.skip-comment || 'false' }}
+
+      - name: Remove eyes reaction
+        if: github.event_name == 'issue_comment'
+        uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
+        with:
+          script: |
+            const reactions = await github.rest.reactions.listForIssueComment({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              comment_id: context.payload.comment.id
+            });
+            const eyesReaction = reactions.data.find(r => r.content === 'eyes' && r.user.type === 'Bot');
+            if (eyesReaction) {
+              await github.rest.reactions.deleteForIssueComment({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                comment_id: context.payload.comment.id,
+                reaction_id: eyesReaction.id
+              }).catch(() => {});
+            }
+
+      - name: Add completion reaction
+        if: github.event_name == 'issue_comment'
+        uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
+        with:
+          script: |
+            await github.rest.reactions.createForIssueComment({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              comment_id: context.payload.comment.id,
+              content: 'rocket'
+            });

.github/workflows/clang-tidy-fix.yaml

@@ -8,7 +8,7 @@ name: Clang-Tidy Fix
   workflow_dispatch:
     inputs:
       ref:
-        description: "The branch, ref, or SHA to checkout. Defaults to the repository's default branch."
+        description: "The branch name to checkout and push fixes to (must be a branch, not a commit SHA). Defaults to the repository's default branch."
         required: false
         type: string
       tidy-checks:
@@ -35,7 +35,7 @@ jobs:
     # This covers repo owners, invited collaborators, and all org members.
     outputs:
       ref:
-        ${{ (github.event_name == 'workflow_dispatch' && (github.event.inputs.ref || github.ref)) ||
+        ${{ (github.event_name == 'workflow_dispatch' && (github.event.inputs.ref || github.ref_name)) ||
         steps.get_pr.outputs.ref }}
       repo: ${{ steps.get_pr.outputs.repo || github.repository }}
       tidy_checks:

.github/workflows/cmake-format-fix.yaml

@@ -8,7 +8,7 @@ run-name: "${{ github.actor }} fixing CMake format"
   workflow_dispatch:
     inputs:
       ref:
-        description: "The branch, ref, or SHA to checkout. Defaults to the repository's default branch."
+        description: "The branch name to checkout and push fixes to (must be a branch, not a commit SHA). Defaults to the repository's default branch."
         required: false
         type: string
   workflow_call:
@@ -18,13 +18,18 @@ run-name: "${{ github.actor }} fixing CMake format"
         required: false
         type: string
       ref:
-        description: "The branch, ref, or SHA to checkout"
+        description: "The branch name to checkout and push fixes to (must be a branch, not a commit SHA)"
         required: true
         type: string
       repo:
         description: "The repository to checkout from"
         required: true
         type: string
+      skip-comment:
+        description: "Skip posting PR comments"
+        required: false
+        type: string
+        default: "false"
 
 permissions:
   pull-requests: write
@@ -44,17 +49,14 @@ jobs:
         github.event_name == 'issue_comment' &&
         github.event.issue.pull_request &&
         contains(fromJSON('["OWNER", "COLLABORATOR", "MEMBER"]'), github.event.comment.author_association) &&
-        (
-          startsWith(github.event.comment.body, format('@{0}bot format', github.event.repository.name)) ||
-          startsWith(github.event.comment.body, format('@{0}bot cmake-fix', github.event.repository.name))
-        )
+        startsWith(github.event.comment.body, format('@{0}bot cmake-fix', github.event.repository.name))
       )
     # Authorization: Only OWNER, COLLABORATOR, or MEMBER can trigger via comments.
     # This covers repo owners, invited collaborators, and all org members.
     outputs:
       ref:
         ${{ (github.event_name == 'workflow_call' && inputs.ref) || (github.event_name == 'workflow_dispatch' &&
-        (github.event.inputs.ref || github.ref)) || steps.get_pr.outputs.ref }}
+        (github.event.inputs.ref || github.ref_name)) || steps.get_pr.outputs.ref }}
       repo:
         ${{ (github.event_name == 'workflow_call' && inputs.repo) || (github.event_name == 'workflow_dispatch' &&
         github.repository) || steps.get_pr.outputs.repo }}
@@ -70,6 +72,12 @@ jobs:
     name: Apply CMake formatting
     needs: pre-check
     if: needs.pre-check.result == 'success'
+    outputs:
+      changes: ${{ steps.handle_commit.outputs.changes }}
+      pushed: ${{ steps.handle_commit.outputs.pushed }}
+      commit_sha: ${{ steps.handle_commit.outputs.commit_sha }}
+      commit_sha_short: ${{ steps.handle_commit.outputs.commit_sha_short }}
+      patch_name: ${{ steps.handle_commit.outputs.patch_name }}
     steps:
       - name: Checkout code
         uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
@@ -93,10 +101,44 @@ jobs:
           gersemi -i ${{ env.local_checkout_path }}
 
       - name: Handle fix commit
+        id: handle_commit
         uses: Framework-R-D/phlex/.github/actions/handle-fix-commit@main
         with:
           tool: cmake-format
           working-directory: ${{ env.local_checkout_path }}
           token: ${{ secrets.WORKFLOW_PAT }}
           pr-info-ref: ${{ needs.pre-check.outputs.ref }}
           pr-info-repo: ${{ needs.pre-check.outputs.repo }}
+          skip-comment: ${{ github.event_name == 'workflow_call' && inputs.skip-comment || 'false' }}
+
+      - name: Remove eyes reaction
+        if: github.event_name == 'issue_comment'
+        uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
+        with:
+          script: |
+            const reactions = await github.rest.reactions.listForIssueComment({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              comment_id: context.payload.comment.id
+            });
+            const eyesReaction = reactions.data.find(r => r.content === 'eyes' && r.user.type === 'Bot');
+            if (eyesReaction) {
+              await github.rest.reactions.deleteForIssueComment({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                comment_id: context.payload.comment.id,
+                reaction_id: eyesReaction.id
+              }).catch(() => {});
+            }
+
+      - name: Add completion reaction
+        if: github.event_name == 'issue_comment'
+        uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
+        with:
+          script: |
+            await github.rest.reactions.createForIssueComment({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              comment_id: context.payload.comment.id,
+              content: 'rocket'
+            });

.github/workflows/dependabot-auto-merge.yaml

@@ -20,14 +20,10 @@ jobs:
   dependabot:
     runs-on: ubuntu-latest
     if: >-
-      (github.event_name == 'pull_request_target' &&
-      github.event.pull_request.user.login == 'dependabot[bot]' &&
-      github.event.pull_request.base.ref == 'main') ||
-      (github.event_name == 'pull_request_review' &&
-      github.event.pull_request.user.login == 'dependabot[bot]' &&
-      github.event.pull_request.base.ref == 'main') ||
-      (github.event_name == 'check_suite' &&
-      github.event.check_suite.pull_requests[0] != null &&
+      (github.event_name == 'pull_request_target' && github.event.pull_request.user.login == 'dependabot[bot]' &&
+      github.event.pull_request.base.ref == 'main') || (github.event_name == 'pull_request_review' &&
+      github.event.pull_request.user.login == 'dependabot[bot]' && github.event.pull_request.base.ref == 'main') ||
+      (github.event_name == 'check_suite' && github.event.check_suite.pull_requests[0] != null &&
       startsWith(github.event.check_suite.head_branch, 'dependabot/'))
     steps:
       - name: Get PR details
@@ -76,6 +72,5 @@ jobs:
             fi
           fi
         # yamllint enable
-
         env:
           GH_TOKEN: ${{ secrets.WORKFLOW_PAT }}