Remix.run Logo
philip1209 6 hours ago

I think the deeper need is a "self-review" flow.

People push AI-reviewed code like they wrote it. In the past, "wrote it" implies "reviewed it." With AI, that's no longer true.

I advocate for GitHub and other code review systems to add a "Require self-review" option, where people must attest that they reviewed and approved their own code. This change might seem symbolic, but it clearly sets workflows and expectations.

billbrown 3 hours ago | parent | next [-]

Yes, underthinking is rampant. Glancing at "AI" output is not reviewing code: you have to grok it (in the Heinlein sense) in order to treat it as your own.

userbinator 2 hours ago | parent [-]

You have to grok it, and not just Grok it.

Tyr42 6 hours ago | parent | prev | next [-]

Heck, doing a self review when you wrote the code catches stuff like forgetting debug prints.

nothrabannosir 5 hours ago | parent | next [-]

(tangent of the decade : prefixing your debug printfs with NOCOMMIT helps catching them before commit :) sample precommit hook and GitHub ci action I wrote is at https://github.com/nobssoftware/nocommit but it’s just a grep)

therealdrag0 2 hours ago | parent | prev [-]

Self review should also include adding guiding comments for other reviewers.

kuekacang 4 hours ago | parent | prev | next [-]

I've been lucky to discover git relatively late and sublime merge relatively soon. It seems like separating the concern of editing and reviewing code is making me consider each more as separate thing.

It also makes me more comfortable figuring out how a project's pull acceptance are like (maybe due to how fast local ui is compared to web-based git). On the other hand, I can only run some basic git cli commands and can't quickly comprehend raw text-based diff, especially when encountering some linux patches from time to time.

jeremyjh 5 hours ago | parent | prev | next [-]

We have it in a checklist in PR template. I can’t imagine a fiat class feature that would be much more meaningful. It surprised me to learn there are developers who have to be reminded to review their own code and test it, but does seem to help.

paxys 5 hours ago | parent | prev | next [-]

If someone was confident enough to push through an AI change without even reading/reviewing it themselves adding more buttons to the UI isn't going to change anything.

5 hours ago | parent [-]
[deleted]
8note 4 hours ago | parent | prev | next [-]

the tooling doesnt make it easy currently.

working at amazon, when I wanted to review code myself through the CR tool, Id still end up publishing it to the whole team and have to add some title shenanigans saying it was a self review or WIP and for others to not look at it yet

koinedad 5 hours ago | parent | prev [-]

Self review is #1

5 hours ago | parent [-]
[deleted]