Node Client: zadd does not support -inf and inf for score

[woodne]

Describe the bug

The type parameters of the nodejs client do not allow the special cases of -inf and +inf as described in the valkey zadd command documentation. Is this an intentional omission, or a miss?

Typescript playground: https://www.typescriptlang.org/play/?#code/JYWwDg9gTgLgBAbzgcQDbACYFMDC6sB28AvnAGZQQhwDkAAgG4CGqA1lgJ4D0zbnAtAHN02GgG4AUBIDGEAgGd40-ETgBeOEwDuTYPDSZcKmADppULExhHghGAAoEEuC80YMF+fKzyAXHABtAF0AGgliAEpJCW1dJWMTAC8md3sAIhAOAGlONJDEOD0sEABGfxL8opAAJn80-mACMjS4SKA

Expected Behavior

Expect to be able to set score to -inf or +inf for zadd command

Current Behavior

The types for zadd do not allow it, and get a compilation error: https://github.com/valkey-io/valkey-glide/blob/main/node/src/BaseClient.ts#L4021-L4025

Reproduction Steps

https://www.typescriptlang.org/play/?#code/JYWwDg9gTgLgBAbzgcQDbACYFMDC6sB28AvnAGZQQhwDkAAgG4CGqA1lgJ4D0zbnAtAHN02GgG4AUBIDGEAgGd40-ETgBeOEwDuTYPDSZcKmADppULExhHghGAAoEEuC80YMF+fKzyAXHABtAF0AGgliAEpJCW1dJWMTAC8md3sAIhAOAGlONJDEOD0sEABGfxL8opAAJn80-mACMjS4SKA

Possible Solution

No response

Additional Information/Context

No response

Client version used

1.3.0

Engine type and version

Valkey 7.0

OS

macOS

Language

TypeScript

Language Version

TypeScript 5.6.3

Cluster information

No response

Logs

No response

Other information

No response

[jamesx-improving]

Thank you for reporting this issue! We are prioritizing it and will have a fix soon.

[shohamazon]

Hello @woodne 🙂

Thanks for pointing this out! This was an unintended omission, and we will address it in the upcoming release. 🚀

Is this currently blocking you? If so, we can provide a patch or a release candidate to unblock you sooner. Let us know what works best for you!

[woodne]

I do not believe it is actively blocking me, I just came across it as I am adapting some code to port from ioredis to glide. I do not think I have actual an actual use for it, so I have a workaround, but just thought I'd point it out.

[shohamazon]

I understand, thank you so much!

[jamesx-improving]

Hi @woodne , while we are still working on the fix, there's a confirmed workaround if you really need it:

client.zadd(key1, {infMember: Infinity})

You can use Infinity or -Infinity, which are TS/JS built-in values equal to Number.POSITIVE_INFINITY and Number.NEGATIVE_INFINITY respectively.