Skip to content

Support for androidx.sqlite's SQLiteDriver#5002

Closed
angusholder wants to merge 4 commits into
getsentry:mainfrom
angusholder:SQLiteDriver
Closed

Support for androidx.sqlite's SQLiteDriver#5002
angusholder wants to merge 4 commits into
getsentry:mainfrom
angusholder:SQLiteDriver

Conversation

@angusholder
Copy link
Copy Markdown

@angusholder angusholder commented Jan 6, 2026

📜 Description

androidx.sqlite 2.5.0 added SQLiteDriver, a new alternative to SupportSQLiteOpenHelper. The new SQLiteDriver API benefits from being Kotlin multi-platform, having a much simpler API, and supporting a bundle of the latest SQLite into your app (rather than getting whatever the Android framework has at runtime).

💡 Motivation and Context

Change is required so that users of the new androidx.sqlite driver API can get SQL breadcrumbs in Sentry

💚 How did you test it?

  • Tested with unit tests generated by Claude. I instructed it to perform the same tests as in the existing SentrySupportSQLiteStatementTest, plus test SentrySQLiteDriver is setting databaseName correctly. I've reviewed all the tests it created and believe they're correct.

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

@buenaflor buenaflor self-assigned this Mar 18, 2026
@0xadam-brown
Copy link
Copy Markdown
Member

Hey @angusholder — thanks so much for raising this and putting in the initial work (and sorry again we didn'thave the bandwidth until recently to get to it). Fyi, we ended up building an integration internally so we could align it with our span attribution patterns (#5466), but your PR was a huge help and we lifted lots from it directly.

I've added you as co-author on the so the credit will be there in git history. Thanks a bunch for the contribution 🙌

@angusholder
Copy link
Copy Markdown
Author

Hey @angusholder — thanks so much for raising this and putting in the initial work (and sorry again we didn'thave the bandwidth until recently to get to it). Fyi, we ended up building an integration internally so we could align it with our span attribution patterns (#5466), but your PR was a huge help and we lifted lots from it directly.

I've added you as co-author on the so the credit will be there in git history. Thanks a bunch for the contribution 🙌

No worries, I didn't spend too long on this PR, glad to hear support is coming soon!

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.

Support for androidx.sqlite's SQLiteDriver

3 participants