feat: Optional Jet Compose#402
Merged
abelonogov-ld merged 33 commits intomainfrom Feb 28, 2026
Merged
Conversation
Updated the class name from SessionReplayEventGenerator to RRWebEventGenerator for improved clarity and consistency. Adjusted the instantiation in SessionReplayExporter accordingly.
Updated the RRWebEventGenerator to improve the generation of incremental events. Introduced new constants for DOM elements and refined the state management for image nodes. The event generation now utilizes a more structured approach for adding and removing nodes, enhancing clarity and maintainability.
Modified the RRWebEventGenerator to use a constant for the image MIME type, ensuring consistency in data URL generation for incremental events. This change enhances maintainability and clarity in the codebase.
…natureManager Updated the observability module to replace instances of CaptureEvent with the new ExportFrame class, enhancing the structure of exported frames. Introduced TileSignatureManager for improved tile-based signature computation. Adjusted related tests and export logic to accommodate these changes, ensuring consistency across the codebase.
Added CaptureManager class to facilitate capturing frames from the lowest visible window. This implementation includes functionality for emitting captured frames as ExportFrame objects, integrating session management, and applying masks for privacy. The new class enhances the observability module by providing a structured approach to frame capture, with future optimizations planned for capture quality and memory management.
Modified ExportDiffManager to accept ImageCaptureService.RawFrame instead of the previously defined RawFrame data class. This change streamlines the frame capture process and enhances integration with the ImageCaptureService, improving overall code clarity and maintainability. Additionally, removed the RawFrame data class from ExportDiffManager to reduce redundancy.
…ions Modified the TileSignatureManager to accept separate width and height parameters for tile size during signature computation. This change enhances flexibility in tile processing and improves the accuracy of tile-based signatures. Additionally, updated the ExportDiffManager to utilize the new signature computation method, ensuring consistency across the observability module.
…roved state management Updated the SessionReplayExporter to include a condition for forcing full captures based on canvas size and keyframe status. Refined the RRWebEventGenerator to manage node IDs and image handling more effectively, including the introduction of a new tile-based signature system. Adjusted the ExportFrame structure to utilize IntSize for original dimensions, enhancing clarity and consistency across the observability module.
…hashing and signature computation Modified the TileSignature data class to include separate hashLo and hashHi values, enhancing the hashing mechanism. Updated the TileSignatureManager to support multiple compute methods with preferred tile dimensions and added convenience overloads for square tiles. Adjusted internal hashing logic to reduce collision probability and improved test cases to reflect changes in tile signature handling.
(cherry picked from commit 49e4fb9)
* main: chore: release main (#396) fix: Android span e2e tests (#397) fix: improve network response capture (#379) feat: initial upload of mobile-dotnet (#393) chore: release main (#394) fix: Fix event sorting in session replay export (#392) fix: Touch move event buffering using wrong clock and mismatched constants (#391)
## Summary Fixes and removes ignore on span e2e tests. Flag Eval on the launch was intervening with tests <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Low risk: changes are limited to the Android E2E test harness and test annotations, with no impact on production SDK logic beyond skipping a test-only span trigger during tests. > > **Overview** > Fixes flaky Android span E2E tests by preventing `BaseApplication` from running `flagEvaluation()` during test runs (gated on `testUrl == null`), avoiding extra spans that interfered with assertions. > > Re-enables previously ignored span-related E2E coverage by removing `@Ignore` from several trace/export and sampling tests in `DisablingConfigOptionsE2ETest` and `SamplingE2ETest`. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 618e2b1. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
## Summary Fixes and removes ignore on span e2e tests. Flag Eval on the launch was intervening with tests <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Low risk: changes are limited to the Android E2E test harness and test annotations, with no impact on production SDK logic beyond skipping a test-only span trigger during tests. > > **Overview** > Fixes flaky Android span E2E tests by preventing `BaseApplication` from running `flagEvaluation()` during test runs (gated on `testUrl == null`), avoiding extra spans that interfered with assertions. > > Re-enables previously ignored span-related E2E coverage by removing `@Ignore` from several trace/export and sampling tests in `DisablingConfigOptionsE2ETest` and `SamplingE2ETest`. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 618e2b1. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
…ervability-sdk into andrey/tile-compression * 'andrey/tile-compression' of github.com:launchdarkly/observability-sdk: chore: add CLAUDE.md (#398)
* main: feat: Android Incremental Image Diff compression (#390)
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Vadman97
approved these changes
Feb 27, 2026
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
…ction - Updated the MaskCollector class to cache the AbstractComposeView class reference using reflection, improving compatibility checks. - Modified the traverseCompose method to accept a generic View instead of AbstractComposeView, ensuring type safety through casting. - Adjusted child view traversal logic to work with the new structure, enhancing the overall robustness of the masking functionality.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Merged
abelonogov-ld
added a commit
that referenced
this pull request
Feb 28, 2026
* main: feat: Optional Jet Compose (#402) feat: Android Incremental Image Diff compression (#390) chore: add CLAUDE.md (#398) chore: release main (#400) fix: correct react native session replay build step (#399) chore: release main (#396) fix: Android span e2e tests (#397) fix: improve network response capture (#379) # Conflicts: # sdk/@launchdarkly/mobile-dotnet/.vscode/tasks.json
abelonogov-ld
pushed a commit
that referenced
this pull request
Feb 28, 2026
🤖 I have created a release *beep* *boop* --- <details><summary>launchdarkly-observability-android: 0.27.0</summary> ## [0.27.0](launchdarkly-observability-android-0.26.1...launchdarkly-observability-android-0.27.0) (2026-02-28) ### Features * Android Incremental Image Diff compression ([#390](#390)) ([5ff93f6](5ff93f6)) * Optional Jet Compose ([#402](#402)) ([8f3a671](8f3a671)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). <!-- CURSOR_SUMMARY --> --- > [!NOTE] > <sup>[Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) is generating a summary for commit bd91847. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
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
3 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.
Summary
https://launchdarkly.atlassian.net/browse/O11Y-812
Note
Medium Risk
Touches session-replay masking traversal and dependency packaging; mistakes could break masking/Compose detection at runtime or introduce
ClassNotFoundExceptions in non-Compose apps.Overview
Makes Compose optional for the Android observability SDK. Compose UI dependencies are switched to
compileOnly, and Compose masking helpers are split into a newComposeMaskingAPI.ktso apps without Compose don’t pull in Compose UI artifacts.Hardens runtime masking traversal when Compose isn’t present.
MaskCollectornow detectsAbstractComposeViewvia reflection (cached) before traversing Compose nodes, avoiding direct type references when Compose UI isn’t on the classpath.Updates the Android E2E app to exercise both UI stacks. Adds
compose/noComposeproduct flavors with flavor-specific manifests and a new XML-basedMainActivity/layout, adjusts dependencies accordingly, and updates CI to run:app:testComposeDebugUnitTest.Written by Cursor Bugbot for commit 59b9e60. This will update automatically on new commits. Configure here.