undefined | Better HN

0 pointsrealusername1mo ago0 comments

I had a similar review by AI except my equivalent of setSomeData was stateful and needed to be there in both places, the AI just didn't understand any of it.

0 comments

james_marks1mo ago

When this happens to me it makes me question my design.

If the AI doesn’t understand it, chances are it’s counter-intuitive. Of course not all LLM’s are equal, etc, etc.

wolletd1mo ago

Then again, I have a rough idea on how I could implement this check with some (language-dependent) accuracy in a linter. With LLM's I... just hope and pray?

realusernameOP1mo ago

I'd agree with that but in the JS world, there's a lot of questionable library designs that are outside of my control.

frde_me1mo ago

My reaction in that case is that most other readers of the codebase would probably also assume this, and so it should be either made clearer that it's stateful, or it should be refactored to not be stateful

j / k navigate · click thread line to collapse