Skip to content

fix: adapter equality bug#32

Merged
olliethedev merged 5 commits intomainfrom
fix/null-eq-adapter-bug
Mar 17, 2026
Merged

fix: adapter equality bug#32
olliethedev merged 5 commits intomainfrom
fix/null-eq-adapter-bug

Conversation

@olliethedev
Copy link
Collaborator

@olliethedev olliethedev commented Mar 17, 2026

Note

Medium Risk
Changes how findMany translates eq/ne comparisons against null, which can alter query results across supported databases if callers previously relied on =/<> behavior with nulls.

Overview
Fixes adapter filtering so where clauses using operator: "eq"/"ne" with value: null generate SQL IS NULL/IS NOT NULL instead of =/<> in both the Drizzle and Kysely adapters.

Adds end-to-end adapter tests covering null comparisons for single conditions and mixed AND/OR groups, and bumps @btst/* package versions from 2.1.0 to 2.1.1.

Written by Cursor Bugbot for commit d687b07. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@olliethedev olliethedev merged commit 5318830 into main Mar 17, 2026
20 checks passed
@olliethedev olliethedev deleted the fix/null-eq-adapter-bug branch March 17, 2026 21:50
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.

1 participant