Skip to content

fix(router): prevent webhook loop from JIRA reaction fallback#426

Merged
zbigniewsobiecki merged 1 commit intodevfrom
fix/reaction-webhook-loop
Feb 19, 2026
Merged

fix(router): prevent webhook loop from JIRA reaction fallback#426
zbigniewsobiecki merged 1 commit intodevfrom
fix/reaction-webhook-loop

Conversation

@zbigniewsobiecki
Copy link
Copy Markdown
Member

Summary

  • Remove JIRA fallback comment from sendJiraReaction() — when the reactions API returned 404, it posted a 💭 comment which triggered a new comment_created webhook, creating an infinite loop that spawned containers endlessly
  • Scope acknowledge reactions to comment events only across all platforms (Trello commentCard, GitHub issue_comment/pull_request_review_comment, JIRA comment_created/comment_updated) instead of firing on every processable event
  • Update tests to verify no fallback comment is posted when the reactions API fails

Root cause

comment_created webhook → router queues job + posts 💭 fallback comment
  → JIRA fires new comment_created webhook → router queues + posts 💭
    → infinite loop

Test plan

  • All 2202 tests pass locally
  • reactions.test.ts updated: 20/20 pass
  • Build + lint + typecheck clean
  • CI passes

🤖 Generated with Claude Code

The JIRA acknowledge reaction had a fallback path that posted a 💭
comment when the reactions API returned 404. This comment triggered a
new `comment_created` webhook, which queued another job, which posted
another fallback comment — creating an infinite loop.

Changes:
- Remove the JIRA fallback comment entirely from sendJiraReaction()
- Scope acknowledge reactions to comment events only across all
  platforms (Trello commentCard, GitHub issue_comment/PR review
  comment, JIRA comment_created/comment_updated) instead of firing
  on every processable event
- Update tests to verify no fallback comment is posted

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@zbigniewsobiecki zbigniewsobiecki merged commit 19311f5 into dev Feb 19, 2026
4 checks passed
@zbigniewsobiecki zbigniewsobiecki deleted the fix/reaction-webhook-loop branch February 19, 2026 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant