feat: SPM Support and new MP SDKs#76
Conversation
PR SummaryHigh Risk Overview On Android, Rokt integration is refactored behind Docs, example app, and CocoaPods paths are aligned with the new versions; the Dart Rokt API is unchanged aside from an Android doc reference to Reviewed by Cursor Bugbot for commit 5e35311. Bugbot is set up for automated code reviews on this repo. Configure here. |
| ### Changed | ||
|
|
||
| - Make `android-rokt-kit` a `compileOnly` plugin dependency and guard Android Rokt registration behind a runtime classpath check, matching the optional Rokt kit pattern used in react-native-mparticle. | ||
|
|
||
| ## [2.2.0] - 2026-07-22 | ||
|
|
||
| ### Added |
There was a problem hiding this comment.
The changed section is above the version header
| fun createDelegate(messenger: BinaryMessenger): RoktPluginDelegate? { | ||
| if (!isAvailable()) { | ||
| return null | ||
| } | ||
| return RoktBridge(messenger) | ||
| } | ||
|
|
||
| fun registerPlatformView( | ||
| registry: PlatformViewRegistry, | ||
| delegate: RoktPluginDelegate, | ||
| ) { | ||
| delegate.registerPlatformView(registry) | ||
| } |
There was a problem hiding this comment.
I think createDelegate and registerPlatformView methods need not be wrapped in the RoktKitAvailability object. They can be in the RoktBridge class, right?
There was a problem hiding this comment.
I agree on the registerPlatformView but for createDelegate I have some concerns.
If we move this to RoktBridge.createIfAvailable(), the plugin must reference RoktBridge in source. Invoking a companion method still loads the RoktBridge class — and that class imports RoktEmbeddedView, rokt, RoktConfig, etc. Which kinda goes against the optional-kit pattern.
Background
What Has Changed
Checklist
Reference Issue (For employees only. Ignore if you are an outside contributor)