zlacker

Redis has the INCR command that does this in redis without the additional round-trips (and race conditions). It also sets the value to 0 if the key doesn't exist.

So, I actually consider this to be exactly the bad behavior that people accuse Copilot of.

>>foepys+(OP)
This was just one of its suggestions, but you're right of course.. it's all based on the training data and idioms used there. If it doesn't weight more modern code higher, if it's not aware of new versions and methods, it isn't going to be super intelligent.. but it can still give you some ideas.

>>foepys+(OP)
Yes it's not very shinning there. I would also throw the error instead of printing the error to the console and returning undefined.

>>speedg+y4
You are in a callback there. Goodbye your error!

>>ponyou+bq
Oh right. I once made a layer on top of the redis client to use promises because callbacks are a pain to deal with.