Skip to content

docs(readme): add critique to feature comparison #249

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
connorads wants to merge 1 commit into
base: main
Choose a base branch
from
+11 −11
Open

docs(readme): add critique to feature comparison #249

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 11 additions & 11 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,17 +84,17 @@ For the full live-session and `--agent-context` workflow guide, see [docs/agent-

## Feature comparison

| Capability | [hunk](https://github.com/modem-dev/hunk) | [lumen](https://github.com/jnsahaj/lumen) | [difftastic](https://github.com/Wilfred/difftastic) | [delta](https://github.com/dandavison/delta) | [diff-so-fancy](https://github.com/so-fancy/diff-so-fancy) | [diff](https://www.gnu.org/software/diffutils/) |
| ---------------------------------- | ----------------------------------------- | ----------------------------------------- | --------------------------------------------------- | -------------------------------------------- | ---------------------------------------------------------- | ----------------------------------------------- |
| Review-first interactive UI | ✅ | ✅ | ❌ | ❌ | ❌ | ❌ |
| Multi-file review stream + sidebar | ✅ | ✅ | ❌ | ❌ | ❌ | ❌ |
| Inline agent / AI annotations | ✅ | ❌ | ❌ | ❌ | ❌ | ❌ |
| Responsive auto split/stack layout | ✅ | ❌ | ❌ | ❌ | ❌ | ❌ |
| Mouse support inside the viewer | ✅ | ✅ | ❌ | ❌ | ❌ | ❌ |
| Runtime view toggles | ✅ | ✅ | ❌ | ❌ | ❌ | ❌ |
| Syntax highlighting | ✅ | ✅ | ✅ | ✅ | ❌ | ❌ |
| Structural diffing | ❌ | ❌ | ✅ | ❌ | ❌ | ❌ |
| Pager-compatible mode | ✅ | ❌ | ✅ | ✅ | ✅ | ✅ |
| Capability | [hunk](https://github.com/modem-dev/hunk) | [critique](https://github.com/remorses/critique) | [lumen](https://github.com/jnsahaj/lumen) | [difftastic](https://github.com/Wilfred/difftastic) | [delta](https://github.com/dandavison/delta) | [diff-so-fancy](https://github.com/so-fancy/diff-so-fancy) | [diff](https://www.gnu.org/software/diffutils/) |
| ---------------------------------- | ----------------------------------------- | ------------------------------------------------ | ----------------------------------------- | --------------------------------------------------- | -------------------------------------------- | ---------------------------------------------------------- | ----------------------------------------------- |
| Review-first interactive UI | ✅ | ✅ | ✅ | ❌ | ❌ | ❌ | ❌ |
| Multi-file review stream + sidebar | ✅ | ❌ | ✅ | ❌ | ❌ | ❌ | ❌ |
| Inline agent / AI annotations | ✅ | ❌ | ❌ | ❌ | ❌ | ❌ | ❌ |
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot May 8, 2026

Choose a reason for hiding this comment

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

P2 Potentially inaccurate ❌ for "Multi-file review stream + sidebar"

The PR author notes critique supports multi-file review streaming but lacks a sidebar. If the row's intent is the full combination (stream and sidebar), ❌ is correct. If "sidebar" is a separable sub-feature, a ✅ with a footnote or the row being split would be more accurate. Worth a quick decision from the maintainer before merging.

Prompt To Fix With AI 
This is a comment left during a code review.
Path: README.md
Line: 91

Comment:
**Potentially inaccurate ❌ for "Multi-file review stream + sidebar"**

The PR author notes `critique` supports multi-file review streaming but lacks a sidebar. If the row's intent is the full combination (stream **and** sidebar), ❌ is correct. If "sidebar" is a separable sub-feature, a ✅ with a footnote or the row being split would be more accurate. Worth a quick decision from the maintainer before merging.

How can I resolve this? If you propose a fix, please make it concise.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

| Responsive auto split/stack layout | ✅ | ✅ | ❌ | ❌ | ❌ | ❌ | ❌ |
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot May 8, 2026

Choose a reason for hiding this comment

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

P2 Potentially inaccurate ❌ for "Inline agent / AI annotations"

The PR description itself flags this as uncertain: critique has an AI review feature (per its own README), which may warrant a ✅ here. The distinction worth pinning down is whether the comparison row covers AI-generated annotations surfaced inline in the diff viewer — if critique does this, the cell should be ✅.

Prompt To Fix With AI 
This is a comment left during a code review.
Path: README.md
Line: 92

Comment:
**Potentially inaccurate ❌ for "Inline agent / AI annotations"**

The PR description itself flags this as uncertain: `critique` has an AI review feature (per its own README), which may warrant a ✅ here. The distinction worth pinning down is whether the comparison row covers AI-generated annotations surfaced inline in the diff viewer — if `critique` does this, the cell should be ✅.

How can I resolve this? If you propose a fix, please make it concise.

| Mouse support inside the viewer | ✅ | ✅ | ✅ | ❌ | ❌ | ❌ | ❌ |
| Runtime view toggles | ✅ | ❌ | ✅ | ❌ | ❌ | ❌ | ❌ |
| Syntax highlighting | ✅ | ✅ | ✅ | ✅ | ✅ | ❌ | ❌ |
| Structural diffing | ❌ | ❌ | ❌ | ✅ | ❌ | ❌ | ❌ |
| Pager-compatible mode | ✅ | ✅ | ❌ | ✅ | ✅ | ✅ | ✅ |

Hunk is optimized for reviewing a full changeset interactively.

Expand Down