Skip to content

feat(investigations): add investigation experience and entry points [9/4/13] - #121698

Draft
arslnb wants to merge 1 commit into
investigations/10-frontend-statefrom
investigations/10-frontend-ui
Draft

feat(investigations): add investigation experience and entry points [9/4/13]#121698
arslnb wants to merge 1 commit into
investigations/10-frontend-statefrom
investigations/10-frontend-ui

Conversation

@arslnb

@arslnb arslnb commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

This is part 4 of 4 in the Investigation execution/frontend split and is stacked on #121697.

This PR exposes the complete Investigation frontend:

  • Investigation launcher, routes, and editor
  • query/text execution controls and streamed output
  • chart/table rendering, parameters, settings, and access controls
  • breached-metric entry points in issue details and the issue list
  • Explore navigation
  • frontend integration and performance coverage

All Investigation UI remains gated by organizations:investigations. Query/text execution and breached-metric entry points additionally require organizations:investigations-query-execution.

Validation:

  • seven focused frontend suites: 59 passed
  • pre-commit eslint, stylelint, and formatting hooks passed
  • pre-push mypy and knip passed

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Aug 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Story previews

Preview the stories changed in this PR on the Vercel deployment:

Preview deployment: https://sentry-amskagvi3.sentry.dev

@github-actions

Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

Metric Before After Delta
Coverage 94.29% 94.31% 🟢 +0.02%
Typed 134,385 135,020 🟢 +635
Untyped 8,136 8,141 🔴 +5
🔍 5 new type safety issues introduced

any-typed symbols (2 new)

File Line Detail
static/app/views/seerNotebook/investigation.tsx 700 sortable (var)
static/app/views/seerNotebook/parameters.tsx 42 option (param)

Non-null assertions (!) (1 new)

File Line Detail
static/app/views/seerNotebook/investigation.tsx 996 filteredSlashCommands[ Math.min(slashCommandIndex, filteredSlashCommands.length…

Type assertions (as) (2 new)

File Line Detail
static/app/views/seerNotebook/output.tsx 355 `as 'line'
static/app/views/seerNotebook/output.tsx 391 `as
                | 'percentage'
                | 'duration'
                | 'bytes'` — `event.target.value as | 'number' | 'percentage' | 'duration' | 'bytes'` |

This is informational only and does not block the PR.

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