Skip to content

fix: show setup links in full instead of cutting them off with "…" - #22

Merged
eukosh merged 1 commit into
mainfrom
fix/setup-links-whole
Oct 25, 2026
Merged

eukosh merged 1 commit into
mainfrom
fix/setup-links-whole

Conversation

@eukosh

@eukosh eukosh commented Oct 25, 2026

Copy link
Copy Markdown
Owner

Summary

Rich's table cells crop any word wider than the cell, so long setup URLs ended in …. The guide now wraps the text itself and never splits a word. A URL wider than the terminal stays one line, which the terminal soft-wraps, so copying it gives the whole link.

 _show(task, bullet)
-  Table.grid(bullet | Group(do, url, Table.grid(field | value)))   # crops with "…"
+  _hang(do,    "  " + bullet)
+  _hang(url,   indent, link style)
+  _hang(value, dim "field  ")    # per value row

+_hang(text, first, style)
+  textwrap.wrap(text, width, break_long_words=False) or [""]
+  print line 1 after `first`, the rest under it, soft_wrap=True

Evidence

  • Before (COLUMNS=60, step 4):
         Application home page            https://github.com/…
         Application privacy policy link  https://github.com/…
    
    After:
         Application home page            https://github.com/eukosh/youtube-kit
         Application privacy policy link  https://github.com/eukosh/youtube-kit/blob/main/PRIVACY.md
    
  • test_links_stay_whole_in_a_narrow_terminal runs the full setup with a 40-column console and checks that every URL appears whole in the visible text. It fails on the old code (with color on too) and passes now.

Merge Danger

Door: two-way

Blast Radius: cosmetic

Only the stderr text of ytk auth login setup and sign-in tips changes. JSON output and exit codes don't change.

🤖 Generated with Claude Code

In a narrow terminal, `ytk auth login` cropped long URLs in the setup
steps, so copying one gave a broken link. URLs now stay on one line the
terminal soft-wraps; prose still wraps under its bullet.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@eukosh
eukosh merged commit bf97653 into main Oct 25, 2026
7 checks passed
@eukosh
eukosh deleted the fix/setup-links-whole branch October 25, 2026 15:22
eukosh pushed a commit that referenced this pull request Oct 25, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.7.2](v0.7.1...v0.7.2)
(2026-10-25)


### Fixed

* show setup links in full instead of cutting them off with "…"
([#22](#22))
([bf97653](bf97653))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

1 participant