Skip to content

fix(github): Deliver repository-scoped PR events - #1216

Merged
dcramer merged 2 commits into
mainfrom
fix/repo-scoped-pr-events
Aug 4, 2026
Merged

fix(github): Deliver repository-scoped PR events#1216
dcramer merged 2 commits into
mainfrom
fix/repo-scoped-pr-events

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Publish every pull request webhook event to both the exact PR identifier and the repository identifier, so repository-scoped watches and event tasks can match PR activity.

Repository subscriptions

Repository resources and the plugin resource-type catalog now advertise the supported PR event set alongside issue events. Repository copies of terminal PR events omit terminal, so a single merged or closed PR does not complete a repository-wide temporary watch.

The webhook, repository-tool, and plugin catalog tests cover the dual routing contract, and the GitHub plugin docs now describe both issue and PR repository scopes.

Fixes #1214

Co-Authored-By: David Cramer <david@sentry.io>
@vercel

vercel Bot commented Aug 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
junior-docs Ready Ready Preview Aug 4, 2026 6:03am

Request Review

@dcramer
dcramer marked this pull request as ready for review August 4, 2026 03:48
@github-actions github-actions Bot added the risk: high PR risk score: high label Aug 4, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9d0a34e. Configure here.

Comment thread packages/junior-github/src/resource-events/repository.ts
Event task and watch validation use the plugin resourceTypes catalog, so repository-scoped PR events still failed without this.

Co-Authored-By: David Cramer <david@sentry.io>
@dcramer
dcramer merged commit beb2a0c into main Aug 4, 2026
31 checks passed
@dcramer
dcramer deleted the fix/repo-scoped-pr-events branch August 4, 2026 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: high PR risk score: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Repo-scoped PR event tasks never match

1 participant