Skip to content

fix(postgrest): narrow column types after not(column, is, null)#2264

Merged
mandarini merged 3 commits intomasterfrom
fix/postgrest-not-null-narrowing
Apr 24, 2026
Merged

fix(postgrest): narrow column types after not(column, is, null)#2264
mandarini merged 3 commits intomasterfrom
fix/postgrest-not-null-narrowing

Conversation

@mandarini
Copy link
Copy Markdown
Contributor

@mandarini mandarini commented Apr 21, 2026

When filtering with .not(col, 'is', null), the resulting row type now marks that column as non-nullable in both the Row (for filter chaining) and Result (for the returned data type).

Closes #1360

Thanks to @DevanshTyagi04 and #2028 for the motivation to address the issue.

@mandarini mandarini requested review from a team as code owners April 21, 2026 14:41
@github-actions github-actions Bot added the postgrest-js Related to the postgrest-js library. label Apr 21, 2026
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 21, 2026

Open in StackBlitz

@supabase/auth-js

npm i https://pkg.pr.new/@supabase/auth-js@2264

@supabase/functions-js

npm i https://pkg.pr.new/@supabase/functions-js@2264

@supabase/postgrest-js

npm i https://pkg.pr.new/@supabase/postgrest-js@2264

@supabase/realtime-js

npm i https://pkg.pr.new/@supabase/realtime-js@2264

@supabase/storage-js

npm i https://pkg.pr.new/@supabase/storage-js@2264

@supabase/supabase-js

npm i https://pkg.pr.new/@supabase/supabase-js@2264

commit: e47215b

@mandarini mandarini requested a review from steve-chavez April 21, 2026 15:02
Copy link
Copy Markdown
Member

@steve-chavez steve-chavez left a comment

Choose a reason for hiding this comment

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

LGTM

@mandarini mandarini merged commit d72bd9f into master Apr 24, 2026
21 checks passed
@mandarini mandarini deleted the fix/postgrest-not-null-narrowing branch April 24, 2026 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

postgrest-js Related to the postgrest-js library.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inferred types for nullable columns after not null checks should be non-nullable

2 participants