Skip to content

Fix TS composite btree range scans#5479

Open
JasonAtClockwork wants to merge 3 commits into
masterfrom
jlarabie/bigint-range-ts
Open

Fix TS composite btree range scans#5479
JasonAtClockwork wants to merge 3 commits into
masterfrom
jlarabie/bigint-range-ts

Conversation

@JasonAtClockwork

@JasonAtClockwork JasonAtClockwork commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Description of Changes

Closes #5454

Fixes TypeScript modules for indexes filter() and delete() methods where the final provided argument is a Range.

  • Added a helper used by filter() and delete() to choose between point scans and ranged scans
  • Fixed serializeRange so range bounds are sliced after the prefix instead of the start

API and ABI breaking changes

No breaking changes this is a bug fix

Expected complexity level and risk

2 - Limited change to the index logic for filter() and delete()

Testing

  • Added small unit tests for filter() and delete() with a composite key ending in Range
  • Ran a full e2e throwaway project with dozens of different filters and deletes using composite keys/ranges and point

@JasonAtClockwork JasonAtClockwork self-assigned this Jul 3, 2026
@JasonAtClockwork JasonAtClockwork added the bugfix Fixes something that was expected to work differently label Jul 3, 2026
@JasonAtClockwork JasonAtClockwork marked this pull request as ready for review July 6, 2026 14:59
@JasonAtClockwork JasonAtClockwork requested a review from jsdt July 6, 2026 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

asap bugfix Fixes something that was expected to work differently

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug when filtering BigInt ranges in typescript's server SDK (stdb 2.6.0)

1 participant