Skip to content

Support Option values in index filters#4949

Open
Mr-Dust0 wants to merge 3 commits intoclockworklabs:masterfrom
Mr-Dust0:issue-4824-filterable-option
Open

Support Option values in index filters#4949
Mr-Dust0 wants to merge 3 commits intoclockworklabs:masterfrom
Mr-Dust0:issue-4824-filterable-option

Conversation

@Mr-Dust0
Copy link
Copy Markdown
Contributor

@Mr-Dust0 Mr-Dust0 commented May 4, 2026

Summary

  • Add `FilterableValue` support for `Option` where `T` is already filterable.
  • Add Rust compile coverage for filtering option-valued btree indexes by `None`, `Some`, and option ranges.
  • Normalize TypeScript table-cache btree comparisons so `null` and `undefined` both behave as option-none.
  • Add C# source-generator coverage for nullable btree index accessors.

Fixes #4824.

Test Plan

  • `cargo check -p module-test`
  • `pnpm --dir crates/bindings-typescript test table_cache_resolved_indexes.test.ts`
  • `dotnet test crates/bindings-csharp/Codegen.Tests/Codegen.Tests.csproj --filter NullableBTreeIndexesCompile --no-restore`

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.

Implement FilterableValue for Option<T>

1 participant