Skip to content

fix(tui): keep provider links readable in narrow layouts#4028

Open
roian6 wants to merge 1 commit into
Hmbown:mainfrom
roian6:fix/provider-links-narrow-layout-3991
Open

fix(tui): keep provider links readable in narrow layouts#4028
roian6 wants to merge 1 commit into
Hmbown:mainfrom
roian6:fix/provider-links-narrow-layout-3991

Conversation

@roian6

@roian6 roian6 commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes #3991.

  • render /links provider Dashboard/Docs URLs as inline code instead of bare markdown URLs
  • keep the full URL visible and copyable while avoiding oversized OSC 8 autolink payloads in narrow terminal layouts
  • add a regression covering provider link output so future provider URLs do not reintroduce bare autolinks

Testing

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets --all-features
    • Not clean locally due pre-existing lint backlog outside this diff, e.g. crates/tui/src/tools/subagent/mod.rs:3024 (unnecessary_map_or) and crates/tui/src/tui/approval.rs:1027 (collapsible_if).
  • cargo test --workspace --all-features
    • Not run; this PR is covered by the focused command/link test surface below.
  • cargo test -p codewhale-tui --bin codewhale-tui links --locked — 16 passed
  • git diff --check
  • added-line secret scan — 0 findings
  • CRLF scan for changed file — none

Checklist

  • Updated docs or comments as needed
    • Not needed; this only changes /links rendering.
  • Added or updated tests where relevant
  • Verified TUI behavior manually if UI changes
    • Not manually captured; verified with a focused /links command-output regression.
  • Harvested/co-authored credit uses a GitHub numeric noreply address
    • Not applicable; original change, no harvested/co-authored patch.

Render /links provider URLs as inline code so the markdown renderer keeps the full URL copyable without creating oversized OSC 8 autolink payloads in narrow terminal layouts.

Adds a regression covering the provider link output.

Tests:

- cargo fmt --all -- --check

- cargo test -p codewhale-tui --bin codewhale-tui links --locked
@roian6 roian6 requested a review from Hmbown as a code owner July 4, 2026 03:27
@github-actions

github-actions Bot commented Jul 4, 2026

Copy link
Copy Markdown

Thanks @roian6 for taking the time to contribute.

This repository is observing a maintainer-managed PR intake gate in dry-run mode, so this pull request is staying open. This note helps maintainers prepare the allowlist before any enforcement is considered.

Please read CONTRIBUTING.md for the expected contribution shape. A maintainer can grant recurring PR access by commenting /lgtm on a pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

v0.8.68 UX: /links provider URLs become unreadable in narrow TUI layouts

1 participant