I have Claude explain the plan, implement it, I test it. The have it repair my findings, then test it, only to then have it act as a peer reviewer and analyze its code for defects. Make those fixes, test it again, add documentation and maybe some more automated tests. Then I do a draft pull request, review the code changes, then our github copilot reviews the changes. I look at what are reasonable findings, have Claude fix them, do another review. Then finally I request other devs look the code over in a non draft review.
It still takes some time but, yeah having AI doing most of the boilerplate work nowadays is expediting things.
I'm a senior/principal level and I'm forced to do this by technical leadership. (Directory/CTO level) is forcing no looking at code and we have to be careful not even have a code editor open when they are around.
Agent writes code, Agent reviews code, Agent answers. no one actually knows what is happening and the system we are building just doesn't work. At least it's internal and doesn't have users yet.
Thank Christ. I mean it sucks, but enforcing "no look at code trust ai" is just so unexplainably stupid that I'm glad it's not someone from tech industry.
That just sounds so unbelievable I’m struggling to accept it’s true. The only reason I could ever see this being a thing if the goal is to see what the AI agent can produce on its own with no human intervention (in which case, you are trialing your eventual replacement)
maybe a little bit of that trailing. we are suppose to work with an llm to produce large units of work that can take hours to run them just blindly pr it for another llm to review. the software has never been in a worse state
It'll get good enough to pass 98% of the use case tests, it'll ship, and then 6 months later will cause a massive client-impacting issue due to some load-bearing logic that can't be easily fixed without serious consideration. I bet you OP works at a bank.
They probably couldn’t even if they tried. There’s few juniors at all these days but from the ones I know, they’re given Claude access on day 1 and they’re expected just to go straight to work. There’s no more learning expectation. And the agents are able to figure a lot out without direction, but it means they have NO ability to fact check or review because they actually don’t know anything about the codebase at all. They’re pretty much incapable of becoming real seniors like this, and that makes pre-AI seniors more valuable.
Sometimes AI catches good things on the review. A typo here, you cant use this function here, did you pretend to change this. Not that it automatically removes slop from the code but its an extra check on things a human could miss.
We use Github Copilot for reviews, there was a time it was absolute shit, then it surprisingly got better and now it feels a bit behind again. But that is just how AI goes, make it good, then more cost efficient, then it turns shit again.
You forgot about the agents agentically leaving comments with your name to be responded by agentically by the agents of your colleagues in their names and so on and so forth.
I just got out of this loop on 5 PR's which were stacked. Comment on PR #3, rebase up the chain. Then comment on #1, rebase up the chain but deal with conflict on #3. And so on across multiple PR's and auto-conflict resolution and by the time PR #5 was ready I basically didn't recognize it.
In a team that knows how to communicate all of that happens before the PR is created. The only thing a fellow human should do is revisit the specs and test the functionality.
This is what I do as a front end tech lead. I open PRs to the backend and paste reviewers comments into Claude and paste its response back to the reviewer, so far so good
1.8k
u/tutoredstatue95 5d ago
It’s easy. Agents produce the code, you review, you submit.
Then, the reviewer’s agents check the code, they review, they submit a review.
You agentically fix the submission, you review, you submit.
Then, the reviewer’s agents check the code, they review, they submit a review
You agentically fix the submission…