Skip to content

fix(tui): keep wrapped URLs clickable - #3823

Merged
Sayt-0 merged 1 commit into
mainfrom
fix/3821-wrapped-url-links
Jul 24, 2026
Merged

fix(tui): keep wrapped URLs clickable#3823
Sayt-0 merged 1 commit into
mainfrom
fix/3821-wrapped-url-links

Conversation

@Sayt-0

@Sayt-0 Sayt-0 commented Jul 24, 2026

Copy link
Copy Markdown
Member

Summary

  • detect complete URLs before tool argument wrapping
  • emit each visible wrapped fragment as an OSC 8 hyperlink to the complete destination
  • preserve existing visible text, wrapping, indentation, and non-link suffixes
  • add an end-to-end regression test for the fetch tool renderer

Issue expectations

Expectation Implementation
Detect URLs split across terminal lines URL boundaries are captured before wrapTextWithIndent splits the text
Treat wrapped fragments as one link Every fragment carries the same complete OSC 8 destination
Keep the entire URL clickable Click detection is verified on every rendered URL cell
Exclude surrounding text Parentheses, result summaries, and (+1 more) remain outside the link

Validation

  • task build
  • task lint
  • task test
  • go test -race ./pkg/tui/components/toolcommon ./pkg/tui/components/messages

Closes #3821

@Sayt-0
Sayt-0 requested a review from a team as a code owner July 24, 2026 08:58
@aheritier aheritier added area/tui For features/issues/fixes related to the TUI kind/fix PR fixes a bug (maps to fix:). Use on PRs only. labels Jul 24, 2026
@Sayt-0
Sayt-0 enabled auto-merge July 24, 2026 09:15
@Sayt-0
Sayt-0 merged commit 7fb6b64 into main Jul 24, 2026
14 checks passed
@Sayt-0
Sayt-0 deleted the fix/3821-wrapped-url-links branch July 24, 2026 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/tui For features/issues/fixes related to the TUI kind/fix PR fixes a bug (maps to fix:). Use on PRs only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

URLs split across multiple lines are not detected as clickable links

3 participants