I still don't like the fact that AI is adding more stuff for us to read, it's accelerating the code production but slowing down the code review.
I built my own code reviewer as well (https://pyor.review/), surfacing the important stuff first is the right track, but adding more stuff to read is daunting, but asking AI to just point you to what you need to focus on and skim the noise is what I'm leaning more towards.
I don't think your sprinkled AI hints help your pitch if they immediately show injected into the diff.
Also, I think the guided review presents the opportunity to skim the code versus read the full diff - as semantic overview of multiple files/diffs, as well as file change description, give you the opportunity to green light anything you don't really need to read into, or give you enough context that you should dive in. i.e. https://x.com/thdxr/status/2073238046296924466?s=20
I still don't like the fact that AI is adding more stuff for us to read, it's accelerating the code production but slowing down the code review. I built my own code reviewer as well (https://pyor.review/), surfacing the important stuff first is the right track, but adding more stuff to read is daunting, but asking AI to just point you to what you need to focus on and skim the noise is what I'm leaning more towards.
I don't think your sprinkled AI hints help your pitch if they immediately show injected into the diff.
Also, I think the guided review presents the opportunity to skim the code versus read the full diff - as semantic overview of multiple files/diffs, as well as file change description, give you the opportunity to green light anything you don't really need to read into, or give you enough context that you should dive in. i.e. https://x.com/thdxr/status/2073238046296924466?s=20