Skip to content

feat(seer-infra-telemetry): Reauthenticate monitoring provider in explorer#118592

Draft
srest2021 wants to merge 14 commits into
masterfrom
srest2021/explorer-reauth-monitoring-provider
Draft

feat(seer-infra-telemetry): Reauthenticate monitoring provider in explorer#118592
srest2021 wants to merge 14 commits into
masterfrom
srest2021/explorer-reauth-monitoring-provider

Conversation

@srest2021

@srest2021 srest2021 commented Jun 26, 2026

Copy link
Copy Markdown
Member

Fixes CW-1546

Depends on #118577, https://github.com/getsentry/seer/pull/7133, #118616

Render a reconnect block, reauthenticate, and resume the Explorer run when a connected monitoring provider's refresh fails mid-run.

Datadog PAT reconnects in-place via the modal and auto-resumes the run. OAuth providers do a full-window redirect and (for now) need the explicit Resume button on return, a stopgap until CW-1557 redirects back into the Explorer and resumes automatically.

Expired at the beginning of the run:

Screen.Recording.2026-06-26.at.6.55.28.PM.mov

Expired in the middle of the run:

Screen.Recording.2026-06-26.at.6.56.56.PM.mov

srest2021 added 12 commits June 25, 2026 11:18
Relocate the Datadog PAT connect modal from getsentry into open-source
sentry/components/seer/ so it can be reused outside the billing-gated
Seer Automation settings page (e.g. the Seer Explorer reauth flow).

Pure relocation: the modal depends only on sentry/* and @sentry/scraps/*,
so it works anywhere the open-source monitoring-provider endpoints do.
The settings section (MonitoringProvidersSection) stays in getsentry and
now imports the modal from its new location.
@linear-code

linear-code Bot commented Jun 26, 2026

Copy link
Copy Markdown

CW-1546

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 26, 2026
@srest2021 srest2021 changed the title feat(seer-infra-telemetry): Reauthenticate monitoring provider in autofix and explorer feat(seer-infra-telemetry): Reauthenticate monitoring provider in explorer Jun 26, 2026
@github-actions

github-actions Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

Metric Before After Delta
Coverage 93.81% 93.81% ±0%
Typed 133,111 133,129 🟢 +18
Untyped 8,782 8,783 🔴 +1
🔍 1 new type safety issue introduced

Type assertions (as) (1 new)

File Line Detail
static/app/views/seerExplorer/hooks/usePendingUserInput.tsx 290 as ReauthMonitoringProviderDatapendingInput.data as ReauthMonitoringProviderData

This is informational only and does not block the PR.

srest2021 added a commit that referenced this pull request Jun 26, 2026
…ider endpoint (#118577)

Adds `PUT` to the monitoring provider details endpoint to
re-authenticate an existing connection. Looks up the user's existing
connection for the provider and reuses its stored `site`.

**Why:** the Seer Explorer can pause mid-run when a monitoring provider
requires reauth (refresh token or PAT expired). For providers like
Datadog that require site selection, the backend reuses the existing
site stored in the identity, and user doesn't need to pick a site from
the dropdown. Followup: #118592
Base automatically changed from srest2021/monitoring-provider-reauth-put to master June 26, 2026 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant