Repository navigation
refactor: replace NitroSQLiteOnLoad with a platform context - #462
Open
chrispader wants to merge 1 commit into
Open
chrispader wants to merge 1 commit into
chrispader wants to merge 1 commit into
Conversation
chrispader
added this pull request to stack #461
October 22, 2026 18:37
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
chrispader
removed this pull request from stack #461
October 22, 2026 20:15
chrispader
force-pushed
the
nitro-v1/platform-context
branch
from
October 22, 2026 20:15
a74e32f to
66c38e0
Compare
chrispader
added this pull request to stack #464
October 22, 2026 20:16
A PlatformContext hybrid object now returns the database, temporary and legacy database directories. Kotlin reads them from the Android application context, and Objective-C++ applies the iOS Info.plist options that OnLoad.mm handled at launch. JavaScript passes them to the NitroSQLite constructor, so each root resolves databases against its own directories instead of static paths, and the Android JNI setter and the iOS +load hook go away. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
chrispader
removed this pull request from stack #464
October 23, 2026 00:18
chrispader
force-pushed
the
nitro-v1/platform-context
branch
from
October 23, 2026 00:19
66c38e0 to
537737b
Compare
chrispader
added this pull request to stack #466
October 23, 2026 00:19
This branch was previously deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
NitroSQLite sets its database directory differently on each platform. On Android, JavaScript calls
NitroSQLiteOnLoad.init(), which passes the app's files and cache directories to C++ through a hand-written JNI function. On iOS, an Objective-C+loadmethod sets the same static C++ path before JavaScript runs.A
PlatformContexthybrid object now returns the database, temporary, and legacy database directories. Android implements it in Kotlin, and iOS in Objective-C++ with the existingRNNitroSQLite_AppGroupandRNNitroSQLite_DatabaseLocationoptions. JavaScript passes the directories to theNitroSQLiteconstructor, so each root resolves databases against its own directories.🤖 Generated with Claude Code