11–20 of 32 posts
But in cases where context lives in peoples heads, its hard for ai to know whats going on snd whats the main goal of this code chanhe.
if the context is given (or documented somewhere claude has access to) + context of company is not too big ti consume it, there is no use in human review imo.
wow I think /review and /code-review just waste of token
If something highly sensitive, yes, and in general, a tendency to review with more scrutiny on backend changes.
But for frontend, less and less every week.
I wonder how codebases for important products (not demos or short-living ones) end up without anyone reviewing the code. Some claim engineers should become more of "architects" and "designers", but not coders/reviewers anymore. Wish to hear people's stories, if much documentation and reviewing from the high-level only is enough to keep the project from becoming a failure
If someone’s best practice,optimization or security check is that important, automate it. Automation is more scalable and consistent . It would make more sense to me to build purpose built agents to look for any patterns that matter. Yes, humans are needed to sanity check and design the agents and rules. Combine that with a well designed build and test harness , human code review seems to me to be obsolete.
Here’s the interesting part: each PR gets reviewed by more than 3 people, each from their own perspective. Based on the questions and the affected paths, the reviewer directs the agent to focus on different points. Someone who is already familiar with the codebase can also guide the agent in the right direction.
Recently, we also started automating UI reviews. The agent runs all the services, verifies the feature through the UI, takes screenshots, and uploads them directly to the GitHub PR. We call this UI Review. + Test coverage also improtant.
Nope. I occasionally do human review of my own code and very occasionally of other peoples', but for the most part it's all 100% automated. I use Codex's Github code review integration and it does a pretty decent job of spotting P1's and P2's.
The speed you're as able to move at with no to lightweight manual code reviews is just so much higher, but there are cases where broken code could mess up critical prod systems so I don't see how you'd get rid of human code reviews there just yet.
For personal usage I've found quick DeepSeek flash reviews a massive boon to finding obvious bugs though.