Skip to content

fix(cache): Allow hashQuery to deal with bigint#5228

Closed
boredland wants to merge 2 commits intodrizzle-team:betafrom
boredland:patch-1
Closed

fix(cache): Allow hashQuery to deal with bigint#5228
boredland wants to merge 2 commits intodrizzle-team:betafrom
boredland:patch-1

Conversation

@boredland
Copy link
Copy Markdown

Update hashQuery to handle bigint parameters correctly.

closes #5227

Update hashQuery to handle bigint parameters correctly.

closes drizzle-team#5227
Copilot AI review requested due to automatic review settings January 7, 2026 12:24
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a bug where the hashQuery function would throw a TypeError when attempting to serialize query parameters containing bigint values. The fix adds a custom JSON replacer function to convert bigint values to strings during serialization.

  • Added bigint handling to the hashQuery function using a JSON replacer

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

Comment thread drizzle-orm/src/cache/core/cache.ts Outdated
@boredland boredland requested a review from valerii15298 January 7, 2026 21:04
@boredland
Copy link
Copy Markdown
Author

My reason for this is done, but I am not sure whether this should be fixed anyway? Anyhow, seems nobody cared...

@boredland boredland closed this Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants