Skip to content

Conversation

@Daniel-Santiago-Acosta-1013

What

  • Avoid hard-wrapping URL lines in history in TUI/TUI2.
  • Fix wrapped row counts for long lines (soft wrap) in history rendering.
  • Add tests for long URL wrapping.

Why

How

  • Skip word wrapping for lines containing URLs in history insertion.
  • Use row count based on line width and viewport width.
  • Add unit tests in TUI and TUI2.

Testing

  • just fmt
  • just fix -p codex-tui
  • just fix -p codex-tui2
  • cargo test -p codex-tui
  • cargo test -p codex-tui2

@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@Daniel-Santiago-Acosta-1013
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@etraut-openai
Copy link
Collaborator

@codex review

@chatgpt-codex-connector
Copy link
Contributor

Codex Review: Didn't find any major issues. More of your lovely PRs please.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

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.

3 participants