← Back to context

Comment by realusername

8 hours ago

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.

4 comments

realusername

Reply
james_marks  8 hours 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.

  • wolletd  8 hours 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?

  • realusername  4 hours 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_me  5 hours 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