Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 101bbf48ea
ℹ️ 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".
| if (!sourceText.contains('\n') || sourceText.contains("\n\n")) { | ||
| return translatedText | ||
| } | ||
| return translatedText.replace(Regex("\\n{2,}"), "\n") |
There was a problem hiding this comment.
Respect CRLF paragraph breaks when normalizing newlines
The guard here only treats "\n\n" as an existing paragraph break, so inputs that use Windows line endings (for example "\r\n\r\n") are misclassified as having no blank lines. In that case we enter normalization and collapse "\n\n" in the translated output to a single newline, which removes real paragraph breaks and changes document formatting for CRLF-origin text. This is a regression from the previous behavior, which left those paragraph separators intact.
Useful? React with 👍 / 👎.
No description provided.