Skip to content

fix(issues): Stop double-emitting issue activities for Seer PR created#115749

Merged
shashjar merged 2 commits into
masterfrom
shashjar/stop-double-emitting-seer-PR-created-activities-on-issues
May 18, 2026
Merged

fix(issues): Stop double-emitting issue activities for Seer PR created#115749
shashjar merged 2 commits into
masterfrom
shashjar/stop-double-emitting-seer-PR-created-activities-on-issues

Conversation

@shashjar
Copy link
Copy Markdown
Member

Resolves ID-1534.

We already have issue activity recording in place for Seer Autofix PRs created via our GitHub webhook, so we can remove the creation logic for this redundant activity type.

image

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 18, 2026

ID-1534

@github-actions github-actions Bot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels May 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

@shashjar shashjar requested a review from a team May 18, 2026 20:31
@shashjar shashjar marked this pull request as ready for review May 18, 2026 20:32
@shashjar shashjar requested a review from a team as a code owner May 18, 2026 20:32
@shashjar shashjar removed the request for review from a team May 18, 2026 20:32
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.55%

Comment thread src/sentry/seer/entrypoints/operator.py
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

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 efbeca9. Configure here.

Comment thread src/sentry/seer/entrypoints/operator.py
Comment thread src/sentry/seer/entrypoints/operator.py
@shashjar shashjar merged commit 71b4d3e into master May 18, 2026
87 of 89 checks passed
@shashjar shashjar deleted the shashjar/stop-double-emitting-seer-PR-created-activities-on-issues branch May 18, 2026 21:43
shashjar added a commit that referenced this pull request May 26, 2026
… from timeline (#116233)

Reverts #115749 to restore
`SEER_PR_CREATED` activity creation in the Seer operator pipeline (fed
by webhook events received into Sentry from Seer).

Hides `SEER_PR_CREATED` activities from the frontend activity timeline
on the issue details page - we continue to rely on
`SET_RESOLVED_IN_PULL_REQUEST` activities for the UI display instead (to
avoid duplicate "PR created" entries).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants