Right, I do use AI for review and find it helpful most of the time (assuming it doesn't decide to be unnecessarily pedantic). I have found it struggles to reason through architectural implications or subtle performance problems when there isn't a specific linter for it, though.
For example, I'm instantiating something N times with this change. Each instantiation is fast, but this is a hot path. It's not clear whether that's safe or if it needs to be gated which is something a reviewer would ideally flag.
I suppose the counter-argument most people would make now is that if the AI didn't call it out, it's probably not likely enough of an issue to focus on -- even if it does wind up becoming a problem later.
For example, I'm instantiating something N times with this change. Each instantiation is fast, but this is a hot path. It's not clear whether that's safe or if it needs to be gated which is something a reviewer would ideally flag.
I suppose the counter-argument most people would make now is that if the AI didn't call it out, it's probably not likely enough of an issue to focus on -- even if it does wind up becoming a problem later.