feat(SDK-5820): Flutter Native Display SDK — complete implementation - #38
Open
CTLalit wants to merge 30 commits into
Open
feat(SDK-5820): Flutter Native Display SDK — complete implementation#38CTLalit wants to merge 30 commits into
CTLalit wants to merge 30 commits into
Conversation
Contributor
|
Important Review skippedToo many files! This PR contains 391 files, which is 291 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (38)
📒 Files selected for processing (391)
You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
CTLalit
changed the title
This was referenced May 28, 2026
Merged
Merged
CTLalit
requested review from
Sonal-Kachare,
akashvercetti,
darshanclevertap,
deeksha-rgb and
piyush-kukadiya
May 28, 2026 12:38
Create the flutter/ plugin package with pubspec.yaml, barrel export, and
the complete Dart model layer. This is the foundation for the Flutter
renderer — all subsequent steps depend on these models.
Files created:
- flutter/pubspec.yaml (sdk: >=3.0.0, flutter: >=3.10.0, cached_network_image ^3.4.1)
- flutter/lib/clevertap_native_display.dart (barrel export)
- flutter/lib/src/models/enums.dart (22 enums, lowercase JSON values, Dart 3.0 switch)
- flutter/lib/src/models/layout.dart (Dimension, NDOffset, Spacing, ChildArrangement, Layout)
- flutter/lib/src/models/text_dimension.dart (TextDimension with /1000 percent resolution)
- flutter/lib/src/models/style.dart (Style with merge() + cascadingOnly())
- flutter/lib/src/models/background.dart (Background sealed class, 10 subtypes)
- flutter/lib/src/models/action.dart (NDAction sealed class, 5 subtypes, ActionTriggers)
- flutter/lib/src/models/gallery_config.dart (GalleryConfig, PeekConfig, IndicatorStyle, ArrowStyle)
- flutter/lib/src/models/node_config.dart (ImageConfig, HtmlConfig, DividerConfig, NDAnimation)
- flutter/lib/src/models/native_display_node.dart (NativeDisplayNode sealed, Container, Element)
- flutter/lib/src/models/native_display_config.dart (NativeDisplayConfig, ResolvedConfig, NDTheme, StyleClass)
- flutter/test/models/models_test.dart (51 tests — all pass)
Key implementation details:
- Manual fromJson/toJson (no code generation)
- Dart 3.0 pattern matching throughout (switch expressions)
- Style.merge(): this > other via null coalescing per field
- Style.cascadingOnly(): text properties + opacity only
- Spacing.resolveTop/Bottom/Left/Right: top ?? vertical ?? all ?? 0.0
- TextDimension percent resolution: rootHeight * value / 1000
- Background sealed class: 5 v1 types fully implemented; 6 v2 types parsed with warning
- OpenUrlAction: handles both plain string and platform map {"ios": ..., "android": ...}
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ser, DimensionCalculator (Flutter) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…utter) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…ERY) (Flutter) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
… opacity (Flutter) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
… support (Flutter) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…g (Flutter) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- ButtonElement: GestureDetector + StyleApplier wrapping Center(Text); fires actionListener on onClick action - SpacerElement: match_parent → Spacer(); fixed dimensions → SizedBox - DividerElement: horizontal/vertical SizedBox+ColoredBox; hides Flutter Orientation to resolve name conflict with ND Orientation enum - NativeDisplayRenderer: dispatch BUTTON/SPACER/DIVIDER in _buildElement - Tests: 9 tests covering all three elements Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…uto-scroll (Flutter) - GalleryRenderer converted from stub StatelessWidget to full StatefulWidget - PageView.builder with viewportFraction from effectiveItemsPerView - PageController + Timer for auto-scroll; both disposed in dispose() - infiniteScroll: null itemCount + modulo index into children list - Page spacing via symmetric Padding around each child - Dot indicators rendered as Container rows/columns inside Stack when showIndicators is true - Hides Flutter Orientation to resolve name conflict with ND Orientation enum - container_test.dart: replaced stub test with 4 GalleryRenderer tests; upgraded wrap() with RootHeightScope + ResolvedStylesScope Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Add video_player: ^2.9.2 dependency - VideoElement: StatefulWidget initialising VideoPlayerController.networkUrl - Reads autoPlay, loop, muted, showControls from node bindings (string values) - Sets looping and volume after controller.initialize() - Shows play/pause icon overlay when showControls is true; tap toggles playback - Renders SizedBox.shrink when url is empty or controller not yet initialised - NativeDisplayRenderer: dispatch ElementType.video to VideoElement - Tests: 6 boolean-binding unit tests + 2 widget smoke tests Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Add webview_flutter: ^4.9.0 dependency - HtmlElement: StatefulWidget initialising WebViewController in initState() - html binding loaded via loadHtmlString (with optional baseUrl); url binding via loadRequest - html binding takes priority over url when both are present - javascriptEnabled / transparentBackground read from HtmlConfig - wrap_content height logs warning and falls back to 200dp; match_parent allows parent to constrain - NativeDisplayRenderer: dispatch ElementType.html to HtmlElement; remove now-unreachable wildcard case (all 7 ElementType values are now handled) - Tests: 6 configuration logic tests for bindings and HtmlConfig parsing Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- AnimationModifier: StatefulWidget with SingleTickerProviderStateMixin - Resolves AnimationType to FadeTransition, SlideTransition, ScaleTransition combinations: fadeIn → Fade; slideIn* → Slide; scaleIn → Scale; fadeScaleIn → Fade+Scale; fadeSlideIn → Fade+Slide - Slide begin offsets per direction: left=(-1,0), right=(1,0), top=(0,-1), bottom=(0,1) - Easing resolved to Flutter Curves; spring → Curves.elasticOut - Delay via Future.delayed before controller.forward() - NativeDisplayRenderer: wraps every built node in AnimationModifier when node.animation is non-null and type != none - Tests: 7 tests covering all major animation type combinations and completion Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Add url_launcher: ^6.3.0 dependency - ActionHandler: const class with optional NativeDisplayActionListener - OpenUrlAction: launches URL via launchUrl; openInBrowser=true uses externalApplication mode - CustomAction: forwards key, value, metadata to listener as flat params map - NavigateAction: forwards destination + params to listener - TrackEventAction: forwards eventName + properties to listener - CompositeAction: sequential iterates actions in order; parallel uses Future.wait - Export ActionHandler from clevertap_native_display.dart public API - Tests: 6 listener-dispatch tests (custom, navigate, event, composite, no-listener guards) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
… plugin stubs (Flutter) - NativeDisplayBridge.dart: static MethodChannel wrapper for com.clevertap.flutter.nativedisplay - fetchConfig(unitId): invokes fetchDisplayUnit, parses JSON into NativeDisplayConfig, returns null on PlatformException - pushViewedEvent(unitId): invokes pushViewedEvent - pushClickedEvent(unitId, elementId?): invokes pushClickedEvent; omits elementId key when null - Android: CleverTapNativeDisplayPlugin.kt — FlutterPlugin + MethodCallHandler; NativeDisplayPluginBridge interface for host-app delegation - iOS: CleverTapNativeDisplayPlugin.swift — FlutterPlugin; NativeDisplayPluginBridge protocol for host-app delegation - Export NativeDisplayBridge from clevertap_native_display.dart public API - Tests: 6 mock-MethodChannel tests covering fetchConfig (null, parse, PlatformException), pushViewedEvent, pushClickedEvent (with/without elementId) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
SourceKit cannot resolve 'import Flutter' outside a full Xcode/Flutter build context. The standard fix for Flutter plugin files is the canImport guard. UIKit was unused and removed. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- 4-tab bottom nav (Banners, Browser, Demos, More) mirroring Android/iOS apps - BannerShowcaseScreen: scrollable list of 10 banner cards with detail navigation - BannerDetailScreen: full-screen single banner display - ArrangementDemoScreen: interactive chip strip switching all 7 strategies - AnimationDemoScreen: 3 animation demos with info card, keyed for rebuild - HomeScreenDemo: loads home_screen.json with action/component listeners - OtherDemosScreen: tabbed view (Home, Gallery, E-commerce, Social, Dashboard) - BridgeIntegrationScreen: loads mock product/notification configs, shows API snippet - SlotDemoScreen: native display units interleaved with mock feed items - TestBrowserScreen: prev/next navigation across 181 test-config filenames - JsonViewerScreen: scrollable raw JSON with copy-to-clipboard - MoreMenuScreen: list navigation to all demo screens - Assets: 10 banner JSONs + 12 config JSONs copied from android-sample - flutter analyze: no issues Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…Flutter) Android (flutter-sample/android/): - SampleApplication: init CleverTap, bind NativeDisplayBridge, register listener - MainActivity: EventChannel (native→Dart units_updated), MethodChannel (pushEvent) - AndroidManifest: CT account ID/token/region meta-data, INTERNET permission - build.gradle.kts: CT core SDK 8.0.0 + native-display-sdk dependencies iOS (flutter-sample/ios/): - AppDelegate: CleverTap.autoIntegrate(), NativeDisplayBridge bind + fetch - Podfile: CleverTap-iOS-SDK ~>7.0, CleverTapNativeDisplay :path Dart (flutter-sample/lib/): - CleverTapIntegrationScreen: subscribe NativeDisplayBridge.eventStream, parse units_updated events, render via NativeDisplayView, Send Event → pushEvent - app.dart: bottom nav Events/Slots/Browser/More matching Android structure - SlotDemoScreen: 15 feed items + 4 dashed-border slot placeholders - Lint fixes: prefer_const_constructors / prefer_const_literals flutter/ plugin: - android/build.gradle + AndroidManifest.xml: plugin Android library scaffold - NativeDisplayBridge: pushEvent (sample MethodChannel) + eventStream (EventChannel) - gallery_renderer.dart: LayoutBuilder fallback height for unbounded PageView Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…op composite build Problem: flutter-sample referenced clevertap-native-ui-kit (compiled Kotlin 2.1.0) via includeBuild, forcing Kotlin 2.1.0 + AGP 8.9.1 + Gradle 8.11.1 on the sample. Fix: remove includeBuild and the native-display-sdk dependency entirely. Subscribe to display unit callbacks via CleverTapAPI.setDisplayUnitListener(DisplayUnitListener) from clevertap-android-sdk:8.0.0 (Java interface — no Kotlin binary compat issue). Forward CleverTapDisplayUnit.getJsonObject().toString() directly to Flutter via EventChannel. Restores original Flutter scaffold versions: Kotlin 1.8.22, AGP 8.7.0, Gradle 8.10.2. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…play Two bugs prevented display units from rendering in CleverTapIntegrationScreen: 1. Wrong JSON format: CleverTapDisplayUnit.getJsonObject() returns raw CT JSON (type:advanced-builder, slot_id, wzrk_id, etc.) not NativeDisplayConfig JSON. Flutter's NativeDisplayConfig.fromJson() cannot parse this format. Fix: extract NativeDisplayConfig from the raw CT JSON using the same 3-strategy detection as NativeDisplayConfigParser: - native_display_config top-level key (dashboard format) - custom_kv.nd_config string value - root top-level key (direct NativeDisplayConfig JSON) 2. Timing race: display units can arrive from CT server before Flutter subscribes to the EventChannel, causing them to be silently dropped (eventSink is null). Fix: cache the last received unit JSONs in SampleApplication.cachedUnitJsons and replay them immediately in EventChannel.StreamHandler.onListen(). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…utter sample platform targets ## Flutter SDK — renderer fixes - native_display_view: aspectRatio now takes precedence over width.percent — when AR is set on a node the renderer uses full available parent width and derives height = parentWidth / aspectRatio, matching Android (Compose modifier ordering) and iOS (guard check in resolveRootWidth). _effectiveWidth returns availableWidth when AR > 0; _applyRootSizing short-circuits immediately when AR is present. - native_display_view: fix _computeRootHeight for percent height roots — previously fell through to screenHeight, inflating percent-based fonts by up to 67 %. - native_display_renderer: remove debugLabel field and all _wrapWithSizing debugPrint calls. - box_container: remove _buildChild debugPrint. - vertical_container / horizontal_container: promote from v1 stubs to full arrangement implementations — all ArrangementStrategy values (spaced, space_between, space_evenly, space_around, start, center, end) now produce correct Column/Row with spacers or MainAxisAlignment. - button_element: propagate maxLines, overflow, and softWrap from resolved style. - style_applier: fix border resolution — require width > 0 before rendering; skip zero- width borders rather than rendering a phantom 1 dp default. ## Flutter sample app - clevertap_integration_screen: remove debug red border, outer LayoutBuilder, and [SAMPLE] log prints; clean up raw JSON logging from onUnitsLoaded. - test_browser_screen: add test-parity-80pct-ar.json to browser list. - Add 180+ test-config JSON assets for the in-app test browser. - Add multi-platform scaffolding (linux, macos, web, windows). - pubspec: add clevertap_plugin dependency; update flutter-sdk path reference. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…knowledge bases Codifies the cross-platform rule discovered while debugging the Flutter renderer: when aspectRatio is set on a node, it takes full available parent width and derives height = parentWidth / aspectRatio. Any width.percent (or height.percent) value is ignored. The rule is identical on all three platforms: - Android: Compose applies aspectRatio() modifier before fillMaxWidth(fraction); the AR modifier locks minW=parentWidth, making fillMaxWidth ineffective. - iOS: resolveRootWidth() guard returns parentWidth when aspectRatio > 0, regardless of percent. Documented in NativeDisplayRenderer.swift line 244. - Flutter: _effectiveWidth returns availableWidth when layout.aspectRatio > 0; _applyRootSizing short-circuits when AR is set. aspectRatio is skipped only when BOTH width AND height are simultaneously fixed (dp/sp/px). Percent dimensions are never treated as "fixed" for this check. Files updated: - CLAUDE.md — layout system section - .claude/reference/CLAUDE_CODE_REFERENCE_ACTUAL.md — layout model + priority table - .claude/reference/JSON_STRUCTURE_REFERENCE.md — How it works + Priority Rules sections - docs/JSON_STRUCTURE_REFERENCE.md — same (kept in sync) - docs/BACKEND_PAYLOAD_SPEC.md — per-platform mechanism table - .claude/agents/android-sdk/knowledge/rendering-pipeline.md — Stage 4 precedence rule - .claude/agents/android-sdk/knowledge/gotchas.md — new gotcha entry - .claude/agents/ios-sdk/knowledge/architecture.md — Swift code evidence - .claude/agents/flutter-sdk/knowledge/rendering-pipeline.md — root sizing implementation - .claude/agents/flutter-sdk/knowledge/architecture.md — NativeDisplayView description - .claude/agents/testing/knowledge/json-generation-rules.md — Section 6 Aspect Ratio Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add NativeDisplayUnit model (unitId, slotId, config, customExtras, resolvedStyles) - Add NativeDisplayConfigParser with tryParse/parseAll running off main thread via Isolate.run(); falls back to synchronous on Dart < 3.4 - Mirrors 3-strategy extraction from Android NativeDisplayConfigParser.kt: native_display_config key → custom_kv.nd_config string → root key - NativeDisplayView accepts optional pre-resolved resolvedStyles to skip redundant StyleResolver calls when using NativeDisplayConfigParser - Export both new types from the public barrel - Simplify clevertap_integration_screen.dart: remove _deepCast, _extractEntry, _UnitEntry; replace with NativeDisplayConfigParser.parseAll() (4 lines) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- NativeDisplayView: StatelessWidget → StatefulWidget; StyleResolver.resolveAll() now runs once in initState/didUpdateWidget, not on every parent rebuild; added fixed-dimension shortcut that bypasses LayoutBuilder when root has only dp dimensions (no percent, no aspectRatio) - style_applier.dart: for solid backgrounds, bake opacity into Color.withValues() instead of wrapping the whole subtree in Opacity widget (avoids saveLayer) - image_element.dart: replace Image.network() with CachedNetworkImage for static images (GIFs keep Image.network for animation fidelity); wrap in RepaintBoundary - video_element.dart: wrap in RepaintBoundary to isolate frame repaints - gallery_renderer.dart: wrap each page item in RepaintBoundary Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
CTLalit
force-pushed
the
feat/SDK-5821-flutter-models
branch
from
June 29, 2026 08:41
507575c to
cbe9f08
Compare
|
@CTLalit I think we can do without the Kotlin/Swift bridge here. Our CT Flutter We could call those directly from your dart code and delete the native kotlin/swift brridgecode, same as we did on React Native ND SDK. So the renderer stays pure Dart and we can call the existing Flutter CT plugin for Core SDK methods. Try a CT Flutter sdk version that has the ND element-clicked method. |
…acker, UnitIdScope Add the Flutter counterparts of the shipped Android/iOS attribution and logging work: - ViewedUnitsTracker: process-singleton gate for per-impression "Notification Viewed" so widget rebuilds/recycling don't refire. - NDLogger: structured, client-configurable logger mirroring Core SDK debug levels (OFF/INFO/DEBUG/VERBOSE) and the native NDLogger format. - UnitIdScope: InheritedWidget that propagates wzrk_id to element renderers so click/view attribution needs no manual threading. Wires element-click attribution through action.dart / action_handler and the button/image/video element renderers. Mirrors Android/iOS: SDK-5860 (NDLogger), SDK-5883 (per-impression Viewed), SDK-5862 (image-tap Clicked). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
CTLalit
force-pushed
the
feat/SDK-5821-flutter-models
branch
from
August 10, 2026 09:44
cbe9f08 to
0cad522
Compare
This was referenced Aug 10, 2026
…ked bridge (#77) Bring Flutter click/view attribution to parity with the shipped Android/iOS work (SDK-5793, SDK-5846). Previously only unitId + elementId reached Core SDK; the clicked action's context and the ND SDK version stamp were dropped. - ActionAttributionExtras: pure Dart helper that flattens an NDAction into a flat extras map (action_type + reserved action_* keys, spread metadata/ params/properties and CustomAction JsonObject value entries, action_value for scalars). sanitize() drops null/empty-key entries and stamps the ND SDK version (nd_lib_v_name/nd_lib_v_code). Mirrors the shipped Kotlin/Swift helper key-for-key. Client never injects wzrk_* (server-owned namespace). - NDSdkVersion: version constants kept in sync with pubspec (0.1.0 / 100), mirroring Android BuildConfig / iOS NativeDisplaySDKVersion. - NativeDisplayBridge.pushClickedEvent gains an optional extras map, forwarded as additionalProperties when non-empty; pushViewedEvent always forwards the version stamp (impressions are unit-level, no caller extras). - Button + IMAGE call sites build extras from the resolved onClick action. - Native plugins (Android Kotlin + iOS Swift): NativeDisplayPluginBridge pushClickedEvent/pushViewedEvent widened to carry additionalProperties; read from the MethodCall and forwarded to the host bridge (which forwards to the ND SDK's element-clicked / viewed-with-extras path). - Tests: 20 ActionAttributionExtras cases + bridge arg-shape assertions. flutter analyze clean; flutter test 183/183 pass. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…78) The Flutter SDK's NDLogger ignored the host Core SDK's configured debug level — NDLogger.syncFromCoreSdk existed but nothing called it. Align Flutter with the shipped Android/iOS CleverTapAutoWire behaviour (SDK-5860). - NativeDisplayBridge.syncLogLevelFromCoreSdk(): reads the Core debug level over the platform channel (new getCoreDebugLevel method) and forwards it to NDLogger.syncFromCoreSdk. No-op once the client called setLogLevel (client-wins via NDLogger.explicitlySet); graceful no-op when no bridge / Core SDK is wired (null or PlatformException leaves the level intact). - Lazily triggered once per app run from the first fetchConfig (fire-and-forget so config fetch never blocks); also public so the host can call it right after wiring. - Native plugins: NativeDisplayPluginBridge gains getCoreDebugLevel(): Int? (Android, CleverTapAPI.getDebugLevel()) / coreDebugLevel() -> NSNumber? (iOS, KVC debugLevel). iOS handles the arg-less method before the args guard. - NDLogger.resetForTest() (@VisibleForTesting) for test isolation of the process-global logger state; no change to syncFromCoreSdk/setLevel logic. - Tests: NDLogLevel.fromInt mapping (incl. unknown→DEBUG) + syncFromCoreSdk semantics + client-wins + null/exception no-op bridge cases. flutter analyze clean; flutter test 192/192 pass. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Android/iOS #54 fixed BOX hit-testing, video-controls tap reliability, numeric-boolean evaluation, and non-string JSON binding values. The Flutter renderer already meets this parity (Stack fall-through hit-testing, HitTestBehavior.opaque video overlay with no stale timer, num!=0 + "1.0" coercion). This locks it in with regression tests — NO behaviour change, zero non-test source touched. - box_hit_test_test.dart: non-interactive sibling stacked over an interactive child does not swallow the tap; topmost interactive child wins (z-order). - video_controls_tap_test.dart: controls overlay uses HitTestBehavior.opaque, tap toggles play/pause (via a fake VideoPlayerPlatform), showControls:false renders no overlay. - business_logic_test.dart: numeric-boolean literals + raw-JSON variable values coerce to bool; non-string binding JSON values coerce to string without throwing. flutter analyze clean; flutter test 215/215 pass. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Fix two adoption/packaging gaps found while auditing minimum-version requirements. - Add flutter/ios/clevertap_native_display.podspec — the plugin shipped with only Classes/*.swift and no podspec, so CocoaPods could not integrate it and iOS adoption was impossible. Minimal bridge pod: depends on Flutter only (no CleverTap SDK — the host provides Core SDK and wires the bridge), iOS 12.0 deployment target, Classes/**/* sources, Swift 5. - Add flutter/LICENSE (MIT, copied from repo root) so the pub package root and the podspec's license reference are self-contained. - pubspec.yaml: raise the declared floor from Dart 3.0 / Flutter 3.10 to Dart 3.6 / Flutter 3.27 — the honest minimum. Dominated by compileSdk 36 (kept current for Play Store) → AGP 8.7 + suppressUnsupportedCompileSdk → Gradle 8.7+/JDK 17, first available with Flutter 3.27. Prior floor let pub get succeed on toolchains that then failed at Gradle time. Kotlin 2.x confirmed NOT required (plugin pins no Kotlin; sample builds on 1.8.22). CleverTap core clevertap_plugin (Dart >=2.12 / Flutter >=1.17) is not a limiter. compileSdk stays 36 by design. flutter pub get resolves; flutter analyze clean; podspec syntax validated. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…ient (#82) Attribution now flows through the host's clevertap_plugin Dart APIs, so the ND plugin's native side is removed entirely and the plugin becomes pure Dart. Supersedes the native portions of SDK-6009/6010/6013. - Remove flutter/android/ and flutter/ios/ (native plugin + the SDK-6013 podspec); drop the flutter.plugin.platforms block from pubspec — now a regular Flutter/Dart package. Keep flutter/LICENSE. - NativeDisplayBridge slimmed to a pure-Dart helper: setLogLevel(NDLogLevel) and syncLogLevelFromCore(int) → NDLogger.syncFromCoreSdk (client-wins guard intact). All MethodChannel/EventChannel usage and fetchConfig/eventStream/ pushEvent/pushViewed/pushClicked/getCoreDebugLevel removed. - Attribution surfaced to the host via NativeDisplayView callbacks: onNotificationViewed(unitId) — fired once per impression via ViewedUnitsTracker; onNotificationClicked(unitId, elementId, extras) — extras built by ActionAttributionExtras. New NotificationClickScope (mirrors UnitIdScope) lets elements read the callback from context. Host feeds these into clevertap_plugin pushDisplayUnitViewedEvent / pushDisplayUnitElementClickedEvent. - Re-lower the version floor: with compileSdk 36 gone, the binding constraint is webview_flutter 4.9.0 → Flutter 3.24 / Dart 3.5 (down from SDK-6013's 3.27). - Tests: replace native arg-shape tests with pure-Dart log-level tests; add native_display_attribution_test.dart (viewed once/impression, click extras). - gitignore the pub-generated flutter/.flutter-plugins* registrants. flutter pub get resolves; flutter analyze clean; flutter test 213 pass. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…83) * SDK-6020: Wire flutter-sample attribution via clevertap_plugin 4.1.0 Bring the Flutter sample to parity with the Android/iOS samples using the pure-Dart SDK's client-surfaced attribution callbacks (SDK-6019). - Bump clevertap_plugin ^4.0.0 → ^4.1.0 (adds pushDisplayUnitElementClickedEvent). - clevertap_integration_screen.dart (live-unit screen): wire NativeDisplayView onNotificationViewed → CleverTapPlugin.pushDisplayUnitViewedEvent(unitId) and onNotificationClicked → pushDisplayUnitElementClickedEvent(unitId, extras), with pushDisplayUnitClickedEvent(unitId) as the no-elementId fallback. Drop the old manual unit-level click from the action listener. Event log now shows VIEWED/CLICKED attribution lines. - main.dart: NativeDisplayBridge.setLogLevel(NDLogLevel.debug) log-level demo. - bridge_integration_screen.dart: replace the deleted-API doc-snippet with the real clevertap_plugin model (getAllDisplayUnits → parse → NativeDisplayView + attribution callbacks); relabel "Bridge API" → "Integration Pattern". - slot_demo_screen.dart: fix a stale NativeDisplayBridge comment. Asset-only screens (bridge mock, test browser, slots) intentionally not wired — no real unitId. Credentials left as-is (hygiene decision pending). flutter analyze clean. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * SDK-6020: replace sample CleverTap credentials with placeholders The committed sample no longer ships live account credentials — clients populate YOUR_CLEVERTAP_ACCOUNT_ID/TOKEN/REGION. Matches the Android/iOS samples. Local development creds are kept out of VCS via skip-worktree. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
2 participants
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.
Jira
Summary
Complete Flutter implementation of the Native Display SDK — pure Dart renderer with platform channel bridge for CleverTap Core SDK integration. Mirrors the production Android (Kotlin/Compose) and iOS (Swift/SwiftUI) implementations.
This PR consolidates 15 stacked PRs. Each individual PR is listed below with its scope for reviewers who want to follow the implementation step by step.
Consolidated PRs
Architecture
Pure Dart renderer — no AndroidView/UiKitView. Platform channels are used only for the CleverTap Core SDK bridge.
Key design decisions
fromJson— no json_serializable/freezed; Dart 3.0 switch expressionsStyleResolver.resolveAll()runs once inStatefulWidget.initState(), not per rebuildNativeDisplayConfigParserusesIsolate.run()for deep-cast + 3-strategy extraction + style pre-resolution; falls back to synchronous on Dart < 3.4aspectRatio > 0, percent width is ignored and full parent width is used (matches Android/iOS)Color.withValues(alpha:)instead ofOpacitywidget for solid backgrounds (avoids saveLayer)Test plan
flutter analyze— no issues influtter/andflutter-sample/flutter test— all tests passflutter build apk --debug --target-platform android-arm64— builds cleanStyle.merge()priority (inline > styleClass > theme)NativeDisplayConfigParser3-strategy extraction matches AndroidNativeDisplayConfigParser.kt🤖 Generated with Claude Code