Skip to content

feat(seerExplorer): Deep-link raw organization events API rows - #122082

Open
sentry-junior[bot] wants to merge 4 commits into
masterfrom
feat/seer-explorer-events-api-links
Open

feat(seerExplorer): Deep-link raw organization events API rows#122082
sentry-junior[bot] wants to merge 4 commits into
masterfrom
feat/seer-explorer-events-api-links

Conversation

@sentry-junior

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

Copy link
Copy Markdown
Contributor

When Seer Code Mode calls the raw Explore events API, the call row already has dataset and query on resolved_path, but no LINK_RULE claimed it. Only telemetry_live_search rows were clickable.

This adds an organization_events rule for:

  • GET .../events/
  • GET .../events-timeseries/

It maps the API query string onto the existing Explore deep-link builder and keeps seer's title as the label. No dataset → no link.

Companion seer title PR: getsentry/seer feat/telemetry-api-call-titles.

Checks

  • Added rule coverage + focused resolve tests in links.spec.tsx.
  • Jest was not run here (no installed frontend deps in this sandbox).

Requested by jeremy.stanley.

--

View Junior Session [Sentry]

Raw Explore list_organization_events calls already carry dataset and
query on resolved_path. Add a LINK_RULE so those rows open the matching
Explore page the same way telemetry_live_search does.

Co-Authored-By: Jeremy Stanley <jeremy.stanley@sentry.io>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Aug 14, 2026
@github-actions

github-actions Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

Metric Before After Delta
Coverage 94.36% 94.36% ±0%
Typed 134,654 134,674 🟢 +20
Untyped 8,048 8,055 🔴 +7
🔍 7 new type safety issues introduced

any-typed symbols (3 new)

File Line Detail
static/app/views/seerExplorer/links.tsx 378 params (var)
static/app/views/seerExplorer/links.tsx 543 params (var)
static/app/views/seerExplorer/links.tsx 616 project_ids (var(binding))

Type assertions (as) (4 new)

File Line Detail
static/app/views/seerExplorer/links.tsx 561 `as string
static/app/views/seerExplorer/links.tsx 566 `as string
static/app/views/seerExplorer/links.tsx 567 `as string
static/app/views/seerExplorer/links.tsx 577 `as string

This is informational only and does not block the PR.

@azulus
azulus marked this pull request as ready for review August 14, 2026 21:38
@azulus
azulus requested a review from a team as a code owner August 14, 2026 21:38

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

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

Comment thread static/app/views/seerExplorer/links.tsx
Co-Authored-By: Jeremy Stanley <jeremy.stanley@sentry.io>
@azulus azulus added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Aug 14, 2026
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 Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant