Skip to content

ref(seer-agent): Use Link component to enable cmd clicking tool links#114494

Merged
aliu39 merged 2 commits intomasterfrom
jb/seer/tool-link-cmd-click
Apr 30, 2026
Merged

ref(seer-agent): Use Link component to enable cmd clicking tool links#114494
aliu39 merged 2 commits intomasterfrom
jb/seer/tool-link-cmd-click

Conversation

@aliu39
Copy link
Copy Markdown
Member

@aliu39 aliu39 commented Apr 30, 2026

Summary

Screen.Recording.2026-04-30.at.3.15.07.PM.mov
  • Replaces the styled button + manual navigate() pattern on tool call links with Link from @sentry/scraps/link
  • cmd/ctrl+click now works natively (opens in a new tab) since Link renders a proper <a> with href
  • Click handler is simplified to only stop propagation and fire the analytics event — all navigation is deferred to the Link component

Test plan

  • Click a tool call link — should navigate client-side as before
  • cmd+click / ctrl+click a tool call link — should open in a new tab

Replace the custom button+navigate pattern with the Link component from
@sentry/scraps/link. This gives cmd/ctrl+click to open in a new tab for
free, removes the manual modifier-key handling, and simplifies the click
handler to only fire analytics.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 30, 2026
@aliu39 aliu39 changed the title ref(seer-explorer): Use Link component for tool call links ref(seer-agent): Use Link component to enable cmd clicking tool links Apr 30, 2026
@aliu39 aliu39 marked this pull request as ready for review April 30, 2026 22:14
@aliu39 aliu39 requested a review from a team as a code owner April 30, 2026 22:14
@aliu39 aliu39 enabled auto-merge (squash) April 30, 2026 22:15
@aliu39 aliu39 merged commit 28e76ad into master Apr 30, 2026
74 checks passed
@aliu39 aliu39 deleted the jb/seer/tool-link-cmd-click branch April 30, 2026 22:18
cleptric pushed a commit that referenced this pull request May 5, 2026
…#114494)

## Summary


https://github.com/user-attachments/assets/de723eb3-3130-4484-aaed-75a1998fdb5d



- Replaces the styled `button` + manual `navigate()` pattern on tool
call links with `Link` from `@sentry/scraps/link`
- cmd/ctrl+click now works natively (opens in a new tab) since `Link`
renders a proper `<a>` with `href`
- Click handler is simplified to only stop propagation and fire the
analytics event — all navigation is deferred to the `Link` component

## Test plan

- [x] Click a tool call link — should navigate client-side as before
- [x] cmd+click / ctrl+click a tool call link — should open in a new tab

---------

Co-authored-by: Claude <noreply@anthropic.com>
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