Fix: zremrangebyscore should accept string | number for min/max scores

[mvonschledorn]

Description

Fixes #1395

The zremrangebyscore command had incorrect TypeScript type definitions that only accepted number for min and max parameters, but the Redis protocol and Upstash REST API accept string values like '-inf' and '+inf'.

This PR aligns zremrangebyscore with the existing zcount command, which already uses number | string for score range parameters.

Changes

• Updated min parameter type from number to number | string
• Updated max parameter type from number to number | string

Before

export class ZRemRangeByScoreCommand extends Command<number, number> {
  constructor(cmd: [key: string, min: number, max: number], opts?: CommandOptions<number, number>)
}

After

export class ZRemRangeByScoreCommand extends Command<number, number> {
  constructor(cmd: [key: string, min: number | string, max: number | string], opts?: CommandOptions<number, number>)
}

Justification

1. Redis Protocol Support: Redis ZREMRANGEBYSCORE accepts special string values like '-inf', '+inf', and '(value' for exclusive ranges

2. Consistency: The ZCountCommand already uses this pattern (see zcount.ts)

3. Runtime vs Types: The Upstash REST API already accepts these string values at runtime, this PR just fixes the TypeScript types to match

Testing

This change is purely a type definition update and doesn't affect runtime behavior. Users can now use the following without type errors:

// Now works without type assertions!
await redis.zremrangebyscore('mykey', '-inf', Date.now() - 900000);
await redis.zremrangebyscore('mykey', 0, '+inf');
await redis.zremrangebyscore('mykey', '(10', '(20'); // Exclusive ranges

References

• Redis ZREMRANGEBYSCORE command: https://redis.io/commands/zremrangebyscore/
• Upstash TypeScript SDK docs: https://upstash.com/docs/redis/sdks/ts/commands/zset/zremrangebyscore
• Consistent implementation in zcount.ts: https://github.com/upstash/redis-js/blob/main/pkg/commands/zcount.ts

[Copilot]

Pull request overview

This PR fixes TypeScript type definitions for the zremrangebyscore command to properly support string values like '-inf', '+inf', and exclusive ranges (e.g., '(10') for min/max score parameters.

• Updated the ZRemRangeByScoreCommand constructor to accept number | (${number} | "-inf" | "+inf" for min and max parameters
• Added test coverage for "+inf" string value
• Refactored existing test structure to use beforeEach/afterEach for better test isolation

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
pkg/commands/zremrangebyscore.ts	Updated type definitions to support string score ranges including infinity values and exclusive ranges with template literal types
pkg/commands/zremrangebyscore.test.ts	Refactored test structure with beforeEach/afterEach hooks and added test for "+inf" parameter

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.