ZRANGEBYSCORE min/max argument types not complete

[mmkal]

In commands.json, ZRANGEBYSCORE has its min and max arguments as double:

redis-doc/commands.json

Lines 3806 to 3813 in e978cf5

	{
	"name": "min",
	"type": "double"
	},
	{
	"name": "max",
	"type": "double"
	},

But the value is also allowed to be +inf, -inf, (1 etc.

Likewise with ZREMRANGEBYSCORE, ZREVRANGEBYSCORE and ZCOUNT.

It'd be great if commands.json could somehow indicate that more than just double is allowed.

Note that #1232 would solve this problem with no extra effort, because this is already possible to express precisely with json schema:

name: min
schema:
  anyOf:
  - type: number
  - type: string
    enum: [-inf, +inf]
  - type: string
    pattern: ^\(\d+(\.\d+)?$

json equivalent

{
   "name": "min",
   "schema": {
      "anyOf": [
         {
            "type": "number"
         },
         {
            "type": "string",
            "enum": [
               "-inf",
               "+inf"
            ]
         },
         {
            "type": "string",
            "pattern": "^\\(\\d+(\\.\\d+)?$"
         }
      ]
   }
}

Related: mmkal/handy-redis#30

[ayruslore]

I think the issue is resolved in this pull request (mmkal/handy-redis#240). This issue can be closed

[mmkal]

That's just a workaround in a downstream library. The issue is that commands.json in this repo reports an incomplete type for the min/max arguments, and that's still the case, so it shouldn't be closed.