Skip to content

feat: Use new Nitro null types#251

Merged
chrispader merged 18 commits intomainfrom
feat/use-nitro-null-types
Feb 4, 2026
Merged

feat: Use new Nitro null types#251
chrispader merged 18 commits intomainfrom
feat/use-nitro-null-types

Conversation

@chrispader
Copy link
Member

@chrispader chrispader commented Dec 26, 2025

Makes use of the newly introduced NullType in Nitro. Replaces all occurences of the custom singelton SQLiteNullValue type with margelo::nitro::NullType


Note

Medium Risk
Updates the native/JSI boundary and public TS types around nulls and query results, which can affect runtime behavior and downstream type expectations. Core DB execution paths are touched but changes are largely mechanical/spec-driven.

Overview
Switches NitroSQLite null handling to Nitro’s built-in NullType, removing the custom SQLiteNullValue/NITRO_SQLITE_NULL plumbing and making null a first-class SQLiteValue across TS and C++.

Regenerates/renames Nitrogen specs and C++ hybrids (e.g. HybridNitroSQLiteQueryResultSpec), updates execute/executeBatch signatures to use the new types, and adds a native rows getter (NitroSQLiteQueryResultRows) for TypeORM-style access.

Updates the example app to accept the new null behavior (tests use null directly and drop simple-null-handling coverage), tweaks the unit test UI to use FlatList, and relaxes react-native-nitro-modules to * in the example lock/package.

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

@chrispader chrispader marked this pull request as ready for review February 4, 2026 16:30
@chrispader chrispader merged commit ed20a11 into main Feb 4, 2026
5 of 6 checks passed
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