Skip to content

Commit 65c77a9

Browse files
committed
chore: remove confused reaction on thread-build failure
The confused emoji reaction was confusing users when thread context building failed. Remove the step entirely - failures are already logged in the workflow run.
1 parent 5fff4b6 commit 65c77a9

File tree

1 file changed

+0
-17
lines changed

1 file changed

+0
-17
lines changed

.github/workflows/review-pr.yml

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -657,20 +657,3 @@ jobs:
657657
mistral-api-key: ${{ secrets.MISTRAL_API_KEY }}
658658
github-token: ${{ steps.app-token.outputs.token || github.token }}
659659
skip-auth: "true" # Org membership already verified above
660-
661-
- name: React on thread-build failure
662-
if: >-
663-
always() &&
664-
steps.check.outputs.is_agent == 'true' &&
665-
steps.auth.outputs.authorized == 'true' &&
666-
steps.thread.outcome == 'failure'
667-
continue-on-error: true
668-
shell: bash
669-
env:
670-
GH_TOKEN: ${{ steps.app-token.outputs.token || github.token }}
671-
REPO: ${{ github.repository }}
672-
COMMENT_ID: ${{ github.event.comment.id }}
673-
run: |
674-
gh api "repos/$REPO/pulls/comments/$COMMENT_ID/reactions" \
675-
-f content="confused" --silent || true
676-
echo "😕 Thread context build failed — added confused reaction"

0 commit comments

Comments
 (0)