Skip to content

feat(integrations): Instrument PR issue resolution - #128758

Closed
giovanni-guidini wants to merge 3 commits into
gio/feat/instrument-commit-resolutionfrom
gio/feat/instrument-pr-resolution
Closed

giovanni-guidini wants to merge 3 commits into
gio/feat/instrument-commit-resolutionfrom
gio/feat/instrument-pr-resolution

Conversation

@giovanni-guidini

Copy link
Copy Markdown
Contributor

Extends the SCM resolution lifecycle introduced by the commit-resolution PR to pull-request-based issue resolution, producing provider-scoped started, success, failure, and halted outcomes without adding database state or changing resolution behavior.

Made with Cursor

Record started, successful, failed, and halted lifecycle outcomes when an SCM commit resolves an issue. Treat idempotent processing of an already-resolved issue as a halt instead of another success.
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 26, 2026
Keep terminal outcomes inside the lifecycle context so successful and halted resolutions do not also emit an automatic halt.
Extend the SCM resolution lifecycle to pull-request-based issue resolution so provider health monitoring can distinguish successful, failed, and halted PR resolution attempts.
@giovanni-guidini
giovanni-guidini force-pushed the gio/feat/instrument-pr-resolution branch from dd0faeb to e76dc25 Compare October 27, 2026 08:37
@sentry

sentry Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Sentry Snapshot Testing

Name Added Removed Changed Renamed Unchanged Skipped Status
sentry-frontend
sentry-frontend
- - - - - - ✅ 760 uploaded

Base commit c08942116253b3e5ce899bdb6a40afe51383cd74 did not produce snapshots to compare against. Did its snapshot job fail? Try rebasing this branch on a commit with a successful snapshot job.

⚙️ sentry-frontend Snapshot Settings

@giovanni-guidini
giovanni-guidini force-pushed the gio/feat/instrument-commit-resolution branch 2 times, most recently from 92e5320 to 726aa54 Compare October 27, 2026 09:47
@giovanni-guidini

Copy link
Copy Markdown
Contributor Author

Consolidated into #128757 as the second commit.

@linear-code

linear-code Bot commented Oct 27, 2026

Copy link
Copy Markdown

SCM-307

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant