Skip to content

Conversation

@nornagon-openai
Copy link
Collaborator

this fixes an issue where text lines with long words would sometimes overflow.

  • the default penalties for the OptimalFit algorithm allow overflowing in some cases. this seems insane to me, and i considered just banning the OptimalFit algorithm by disabling the 'smawk' feature on textwrap, but decided to keep it and just bump the overflow penalty to ~infinity since optimal fit does sometimes produce nicer wrappings. it's not clear this is worth it, though, and maybe we should just dump the optimal fit algorithm completely.
  • user history messages weren't rendering with the same wrap algorithm as used in the composer, which sometimes resulted in wrapping messages differently in the history vs. in the composer.

@nornagon-openai nornagon-openai merged commit 4b0f5eb into main Oct 7, 2025
20 checks passed
@nornagon-openai nornagon-openai deleted the nornagon/wrap branch October 7, 2025 18:32
@github-actions github-actions bot locked and limited conversation to collaborators Oct 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants