Skip to content

feat: show commit message as reviewable entry for single-commit reviews#182

Merged
agavra merged 1 commit intoagavra:mainfrom
thiblahute:feat/commit-message-review
Feb 19, 2026
Merged

feat: show commit message as reviewable entry for single-commit reviews#182
agavra merged 1 commit intoagavra:mainfrom
thiblahute:feat/commit-message-review

Conversation

@thiblahute
Copy link
Copy Markdown
Contributor

@thiblahute thiblahute commented Feb 14, 2026

Summary

  • When reviewing a single commit (tuicr -r <commit>), the commit message now appears as the first entry in the file list and diff view
  • In multi-commit reviews, selecting a single commit in the inline selector also shows its commit message
  • The commit message entry supports comments, reviewed status, and folding — just like any code file
  • Added body field to CommitInfo across all VCS backends (git, hg, jj) to capture the full commit message beyond the summary line

Test plan

  • tuicr -r <single-commit> — commit message appears as first entry, can be commented on and marked reviewed
  • tuicr -r <commit-range> — selecting a single commit shows its commit message; selecting a range hides it
  • Commit message entry folds when marked reviewed
  • Regular file commenting logic unchanged
  • cargo test — all 214 tests pass

When reviewing a single commit (either via CLI or inline selector),
the commit message now appears as the first entry in the file list
and diff view. It supports comments, reviewed status, and folding
just like any code file.
@thiblahute thiblahute force-pushed the feat/commit-message-review branch from 14553e0 to 6f8a081 Compare February 14, 2026 13:36
@agavra
Copy link
Copy Markdown
Owner

agavra commented Feb 14, 2026

I like the idea, and I haven't spent any time debugging this, but it isn't working on my machine. Can you upload a screenshot of it on your machine?

@thiblahute
Copy link
Copy Markdown
Contributor Author

Weird, it doesn't work on your side, I made a quick screencapture so you can see how it is supposed to work:

Video.Feb.15.2026.11.AM.mp4

@agavra
Copy link
Copy Markdown
Owner

agavra commented Feb 19, 2026

Thanks, just got back from a trip. I'll try to debug this tomorrow to figure out why it's not working on my end.

@agavra
Copy link
Copy Markdown
Owner

agavra commented Feb 19, 2026

Ah! I just noticed it is only supposed to work when you run it with -r <revision>. After doing that it works for me. I'll merge this as is, but if you wanted to make this work even if we just select a commit via the selector screen I'm in support of that.

@agavra agavra merged commit 1933f25 into agavra:main Feb 19, 2026
4 checks passed
@thiblahute
Copy link
Copy Markdown
Contributor Author

Ah! I just noticed it is only supposed to work when you run it with -r . After doing that it works for me. I'll merge this as is, but if you wanted to make this work even if we just select a commit via the selector screen I'm in support of that.

It does work, even when you selected several commits, if you then go trhough each commit one by one, you can comment on the commit messages.

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.

2 participants