Hacker News
points by pksunkara 4 hours ago | 0 comments
Sorry for the shameless plug but I would really love some feedback and discussion on some of the features in the code forge I am building.

1. Hunk checkpoints - If it's a moderately sized PR (which most of the agentic PRs nowadays are), I as a developer find myself like to think in terms of hunks whether I finished reviewing that part or not. GitHub allows you to mark only files as reviewed and nothing else.

2. I am trying to add semantic diffs to code reviews. That way a developer can actually understand the changes in the structure of the code instead of relying on reading plain text changes only. For ex: a function rename can simply compress 20 lines of changes into a single line.

3. Any agentic reviews and/or automated deterministic reviews can use both of the above to mark some simple things as reviewed in order to help the reviewer more.

4. This is in addition to existing review features from GitHub and the interdiff feature from other review tools.

A small mockup is available on the marketing website at https://juju.bi

I appreciate any comments on this approach. What do you guys think?

[–] wizardmeridian 50 minutes ago
I have been thinking about a similar “semantic diff” approach to code reviews for the past several weeks. I’d love to hear more about how you’re approaching this - there’s a handful of distinct scenarios I see frequently in PRs that I think should be reviewable in a more direct form (renames, moving blocks of code between files without changing the text, whitespace changes, etc - basically any action a modern IDE can do deterministically).

I’ve played with some bookmarklets that modify the GitHub PR ui and attempt highlight these situations, but my setup is brittle (just trying to identify basic patterns in the text diff, ex “these are [oldname] replaced with [newname]”, which is not necessarily the same thing as an IDE rename-this-function action). I’ve been wondering if the answer here is in doing a diff on a compiled intermediate form of the program, visualized in some way - this would have to be language-specific though, I guess, and is well outside my expertise.

Separate from the diff aspect of reviews, I would also appreciate some more control in a PR tool over how to layout a review (what files/hunks to be shown, in what order). My team attempts to do this with carefully organized commits, so that reviewers step through them one by one, but the GitHub UI leaves a lot to be desired here (files appear in a static order top to bottom; comments and discussions can’t be stuck to one commit and show up in other commits; general bugs in the web view). I would appreciate being able to control how others see my PR if there’s room for that in your product.

Anyways, I joined your waitlist and am curious to see what you come up with - I think there are improvements to that can be made to code reviewing (that don’t involve AI - everyone is already putting AI review in their PRs), and am willing to try anything that takes a swing at that.

[–] pksunkara 8 minutes ago
> Separate from the diff aspect of reviews, I would also appreciate some more control in a PR tool over how to layout a review

I am quite interested in knowing more about this. Would you mind give me a small example? Do you want to control individual hunks order or just files?

[–] gruez 4 hours ago
>1. Hunk checkpoints

How long are your files that you can't easily "finish" a file?

[–] pksunkara 3 hours ago
I find that for even files that are less than 500 lines, if the changes are complex enough, I wish I could track viewed changes line-by-line.
[–] echelon 3 hours ago
Y'all are still reviewing code?

(Semi serious here.)