Skip to content

feat(tui): add fullscreen local diff viewer dialog - #15710

Closed
marcusschiesser wants to merge 11 commits into
anomalyco:devfrom
marcusschiesser:feat/tui-diff-viewer
Closed

marcusschiesser wants to merge 11 commits into
anomalyco:devfrom
marcusschiesser:feat/tui-diff-viewer

Conversation

@marcusschiesser

@marcusschiesser marcusschiesser commented Mar 2, 2026 •

Copy link
Copy Markdown
Contributor

Issue for this PR

Closes #5102

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

  • add a dedicated local diff viewer dialog in TUI with fast file-to-file navigation
  • render selected file diffs using the internal OpenCode diff renderer with unified/split toggle
  • add fullscreen dialog support and expose the viewer via /diff, command palette, and <leader>d
  • improve dialog UX with clear file status legend and reliable esc close behavior

How did you verify your code works?

I tested my changes locally

Screenshots / recordings

image
opencode.diff.viewer.mov

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.

@github-actions github-actions Bot added the needs:compliance This means the issue will auto-close after 2 hours. label Mar 2, 2026
@github-actions

github-actions Bot commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

Based on my search, the only related PR I found is:

PR #12188: feat(tui): add interactive code review system for reviewing agent changes

This PR is related as it also deals with TUI diff viewing/code review functionality, though it focuses on reviewing agent changes rather than a general local diff viewer. However, they could be complementary features rather than duplicates.

All other searches returned only the current PR #15710 itself. No other PRs directly addressing the same fullscreen local diff viewer feature were found.

@github-actions github-actions Bot removed the needs:compliance This means the issue will auto-close after 2 hours. label Mar 2, 2026
@github-actions

github-actions Bot commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

Thanks for updating your PR! It now meets our contributing guidelines. 👍

@rekram1-node

Copy link
Copy Markdown
Collaborator

We have some ideas in this area too but this is cool. I'm not 100% sure if this will get merged cause normally bigger UI things are kinda opinionated by some members of the team but I will say some of this kinda goes against existing client server paradigm, the tui is the client, and the opencode server is where the "repo" / "code" will be so running git diff on the client doesn't make a whole lot of sense in this case

@marcusschiesser

Copy link
Copy Markdown
Contributor Author

@rekram1-node Thanks for your repo. Makes sense running git diff on the server. Happy to take a look into this

@gosantos

Copy link
Copy Markdown

This looks great. I miss this feature so much. I believe this will be better than codex/claude.

@marcusschiesser marcusschiesser closed this by deleting the head repository May 15, 2026
@abhisheksoni27

Copy link
Copy Markdown

why did this get closed?

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.

[FEATURE]: Session or Per-chat File Diff Viewer

4 participants