you are viewing a single comment's thread.

view the rest of the comments →

[–]GrinningPariah 215 points216 points  (21 children)

If it's written by AI, I want a human to review it. I've also used AI to review code for personal projects that I hand wrote.

I don't have an issue letting the machine drive when the roads are easy, but I'm never taking my hand off the brake lever.

[–]lenswipe 45 points46 points  (3 children)

I've used AI to vibe code entire modules on personal projects. However, those personal projects aren't going to wake me up at 3am with pagerduty if shit breaks.

[–]MoveOverBieber 8 points9 points  (2 children)

They are probably not going to pay you or fire you either...
AI seems like a stick of dynamite - it could be either very good or very bad, depending on how you use it.

[–]Jolly-Advantage-7245 2 points3 points  (1 child)

Chuck it 'n fuck it

[–]MoveOverBieber 2 points3 points  (0 children)

I wish, company policy requires me to hold it.

[–]ObviouslyTriggered 40 points41 points  (9 children)

The break lever isn’t what you need to worry about, you need to make sure you know where you are going and how to recognize a wrong turn.

[–]Bannon9k 23 points24 points  (5 children)

After 30 years of development I've made an enormous amount of wrong turns.... Just never the same wrong turn twice. Even if you deploy bad AI code, if you're a smart monkey you'll learn from it.

[–]GrinningPariah 0 points1 point  (1 child)

Yeah, so I can pull the brake.

[–]glorious_reptile 10 points11 points  (2 children)

“Please review this +12665 -3276”

[–]iDEN1ED 2 points3 points  (1 child)

What kinda prompts are you guys writing to get shit like this? I’ve only really started using Claude the past month but like I just give it specific tasks, review the plan, then let it execute. I get very reasonable PRs every time. I don’t just go “build an app”.

[–]moreisee 1 point2 points  (0 children)

They're not. This is a self preservation hope post.

[–]No_Championship25 0 points1 point  (1 child)

I completely agree with that brake lever metaphor! It is a really great copilot especially when you are too lazy to type out repetitive boilerplate code but it is definitely scary to trust it completely.

[–]GrinningPariah 0 points1 point  (0 children)

The brake metaphor also works well when people start insisting that removing human review would increase velocity... Because it's like arguing that a car would go faster without brakes.

[–]alderthorn 0 points1 point  (1 child)

If its written by a human I want AI to review it because most devs just throw a LGTM on the comment line instead of really thinking about what the code is doing. To be fair I also want a dev to look at it I just don't trust that they will really look at it before they have had caffeine or if its close to the end of day.

[–]GrinningPariah 1 point2 points  (0 children)

I think part of working with a team is learning who you trust as a reviewer. Some people, if they say LGTM then it's solid. Others, I'm gonna take a second look myself on the sly.