Skip to content

ref(seer): Remove Langfuse URL plumbing from Seer Explorer - #121186

Merged
aliu39 merged 2 commits into
masterfrom
andrewliu/remove-langfuse-url
Aug 4, 2026
Merged

ref(seer): Remove Langfuse URL plumbing from Seer Explorer#121186
aliu39 merged 2 commits into
masterfrom
andrewliu/remove-langfuse-url

Conversation

@aliu39

@aliu39 aliu39 commented Aug 4, 2026

Copy link
Copy Markdown
Member

Langfuse tracing is being killed in favor of Sentry's own AI tracing, so the Langfuse deep links scattered through Seer Explorer no longer point anywhere useful. This removes the getLangfuseUrl util and every consumer of it.

What's removed

  • getLangfuseUrl() in static/app/views/seerExplorer/utils.tsx (hardcoded langfuse.getsentry.net session URL)
  • langfuse_url from the seer.explorer.feedback_submitted analytics event definition and its call site
  • The langfuse_url tag attached to Seer Explorer user-feedback submissions
  • The employee-only /langfuse slash command, plus the now-unused onLangfuse handler prop on SlashCommandHandlers and the handleOpenLangfuse callback

Why it's safe

The /conversations slash command and the conversations_url analytics property/feedback tag already cover the same debugging need via the Sentry AI trace view, so nothing is left without an equivalent. langfuse_url was write-only telemetry — dropping the property just stops sending a dead link.

Frontend-only; no backend changes.

Langfuse tracing is being killed in favor of Sentry's own AI tracing, so
the Langfuse deep links no longer point anywhere useful. Deletes the
getLangfuseUrl util and every consumer: the langfuse_url analytics
property on seer.explorer.feedback_submitted, the langfuse_url feedback
tag, and the employee-only /langfuse slash command (along with its now
unused onLangfuse handler prop).

The /conversations slash command already covers this use case with the
Sentry AI trace view.
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Aug 4, 2026
@aliu39
aliu39 marked this pull request as ready for review August 4, 2026 17:25
@aliu39
aliu39 requested review from a team as code owners August 4, 2026 17:25

@natemoo-re natemoo-re left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙌 nice!

@aliu39
aliu39 merged commit 58bb5a9 into master Aug 4, 2026
79 checks passed
@aliu39
aliu39 deleted the andrewliu/remove-langfuse-url branch August 4, 2026 17:37
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.

2 participants