Thread

1. refactor_master · 1790553726 · focus · HN ↗

   In my experience, automated code review is more pointless than ever.

   We have all the linters, tests, and AI writing code for us. I don’t need the left hand to tell the right hand it did a good job. I’m very certain my code runs when I push the PR.

   What I need now is architectural, long-horizon and business perspective.

   1. RomanKornev · 1790559027 · focus · HN ↗

      What kind of code review tools did you try?

      > What I need now is architectural, long-horizon and business perspective.

      That's exactly what these tools are now good at. They have a huge gap when fixing these issues properly but they can spot these issues no problem

      1. jeremyjh · 1790564517 · focus · HN ↗

         No, no they can’t. I use AI a lot and I get a lot of value from it but they are still terrible at programming “in the large”, by which I mean slotting features into the place meant for them in the existing code base. When coding and reviewing their view is too local. They will implement a change in the first place that looks feasible when coding. When reviewing they will not look for code duplication or fit. They will review for correctness, performance, security and style but not for architectural coherence.

         They also have shockingly weak ability to identify business acceptance criteria that are completely missing in the implementation or test coverage.

         1. sfn42 · 1790580599 · focus · HN ↗

            Thing is most human developers also fit that description, so they don't see the problem.
         2. maksDanylenko · 1790593274 · focus · HN ↗

            [dead]

Unofficial Hacker News client; not affiliated with Y Combinator.