Skip to content

git_ui: Strip ANSI escape codes from git command output#53444

Merged
Veykril merged 3 commits intozed-industries:mainfrom
zaniluca:fix/escape-ansi-in-git-output
Apr 9, 2026
Merged

git_ui: Strip ANSI escape codes from git command output#53444
Veykril merged 3 commits intozed-industries:mainfrom
zaniluca:fix/escape-ansi-in-git-output

Conversation

@zaniluca
Copy link
Copy Markdown
Contributor

@zaniluca zaniluca commented Apr 8, 2026

When git commands (push, pull, hooks...) produce output containing ANSI escape sequences for colors or formatting, Zed was displaying them as raw escape codes in the output buffer, making the output hard to read. This simply escapes ANSI from the git output

Before and After

Before After
Screenshot 2026-04-08 at 21 13 07 Screenshot 2026-04-08 at 21 15 14

Self-Review Checklist:

  • I've reviewed my own diff for quality, security, and reliability
  • Unsafe blocks (if any) have justifying comments
  • The content is consistent with the UI/UX checklist
  • Tests cover the new/changed behavior
  • Performance impact has been considered and is acceptable

Related to #43817. This PR only addresses the escaping of the ANSI codes; colors and other stuff are not handled

Release Notes:

  • Fixed ANSI escape codes being displayed as raw text in git command output

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Apr 8, 2026
@zed-community-bot zed-community-bot Bot added the first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions label Apr 8, 2026
@zaniluca zaniluca marked this pull request as ready for review April 8, 2026 19:59
@zed-codeowner-coordinator zed-codeowner-coordinator Bot requested review from a team, Veykril and kubkon and removed request for a team April 8, 2026 19:59
Copy link
Copy Markdown
Member

@Veykril Veykril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Veykril Veykril enabled auto-merge (squash) April 9, 2026 06:57
@Veykril Veykril merged commit 7e1b636 into zed-industries:main Apr 9, 2026
31 checks passed
@Bertie690
Copy link
Copy Markdown
Contributor

Bertie690 commented Apr 18, 2026

Since this was merged, I request @esthertrapadoux to close #43817.

piper-of-dawn pushed a commit to piper-of-dawn/zed that referenced this pull request Apr 25, 2026
…es#53444)

When git commands (push, pull, hooks...) produce output containing ANSI
escape sequences for colors or formatting, Zed was displaying them as
raw escape codes in the output buffer, making the output hard to read.
This simply escapes ANSI from the git output

### Before and After

  <table>
    <tr>
      <th align="center">Before</th>
<th align="center">After</th>
    </tr>
    <tr>
  <td>
<img width="882" height="862" alt="Screenshot 2026-04-08 at 21 13 07"
src="https://github.com/user-attachments/assets/58731e80-d864-47ca-8983-d0e86e924843"
/>
</td>
      <td>
<img width="882" height="862" alt="Screenshot 2026-04-08 at 21 15 14"
src="https://github.com/user-attachments/assets/7649200a-2d82-4442-88da-e231304911a8"
/>
</td>
    </tr>         
  </table>        

Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [ ] Unsafe blocks (if any) have justifying comments
- [x] The content is consistent with the [UI/UX
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
- [x] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

Related to zed-industries#43817. This PR only addresses the escaping of the ANSI
codes; colors and other stuff are not handled

Release Notes:

- Fixed ANSI escape codes being displayed as raw text in git command
output
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants