Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RUM-6189: Add Abstract and Text semantics mapper for compose SR #2292

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

ambushwork
Copy link
Member

What does this PR do?

This is the commit to set up the framework to support Jetpack compose session replay with semantics solution.
In this commit the principals changes are:

  • Add AbstractSemanticsNodeMapper as the basic mapper providing some common functions for semantics mapping
  • Add TextSemanticsNodeMapper for the compose Text composable.
  • Now ComposeView will be mapped from ComposeWireframeMapper instead of SemanticsWireframeMapper.

Demo

Sample App Session replay
Screenshot 2024-09-26 at 23 20 54 image

Additional Notes

Anything else we should know when reviewing?

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@codecov-commenter
Copy link

codecov-commenter commented Sep 26, 2024

Codecov Report

Attention: Patch coverage is 55.97015% with 59 lines in your changes missing coverage. Please review.

Project coverage is 69.52%. Comparing base (6de40db) to head (fc14db6).
Report is 2 commits behind head on feature/sr_compose.

Files with missing lines Patch % Lines
...ernal/mappers/semantics/TextSemanticsNodeMapper.kt 48.53% 19 Missing and 16 partials ⚠️
...ionreplay/compose/internal/utils/SemanticsUtils.kt 11.11% 8 Missing ⚠️
...rnal/mappers/semantics/SemanticsWireframeMapper.kt 76.67% 3 Missing and 4 partials ⚠️
...l/mappers/semantics/AbstractSemanticsNodeMapper.kt 73.68% 4 Missing and 1 partial ⚠️
...y/compose/internal/reflection/ComposeReflection.kt 42.86% 0 Missing and 4 partials ⚠️
Additional details and impacted files
@@                  Coverage Diff                   @@
##           feature/sr_compose    #2292      +/-   ##
======================================================
- Coverage               69.73%   69.52%   -0.20%     
======================================================
  Files                     746      750       +4     
  Lines                   27642    27742     +100     
  Branches                 4696     4733      +37     
======================================================
+ Hits                    19274    19287      +13     
- Misses                   7058     7120      +62     
- Partials                 1310     1335      +25     
Files with missing lines Coverage Δ
...d/sessionreplay/compose/ComposeExtensionSupport.kt 92.31% <100.00%> (ø)
...y/compose/internal/reflection/ComposeReflection.kt 15.46% <42.86%> (+2.13%) ⬆️
...l/mappers/semantics/AbstractSemanticsNodeMapper.kt 73.68% <73.68%> (ø)
...rnal/mappers/semantics/SemanticsWireframeMapper.kt 76.67% <76.67%> (ø)
...ionreplay/compose/internal/utils/SemanticsUtils.kt 11.11% <11.11%> (ø)
...ernal/mappers/semantics/TextSemanticsNodeMapper.kt 48.53% <48.53%> (ø)

... and 33 files with indirect coverage changes

@ambushwork ambushwork force-pushed the yl/compose/semenatics-mapper branch 2 times, most recently from b7b556c to 1ec890c Compare September 27, 2024 09:01
@ambushwork ambushwork marked this pull request as ready for review September 27, 2024 09:54
@ambushwork ambushwork requested review from a team as code owners September 27, 2024 09:54
@ambushwork ambushwork force-pushed the yl/compose/semenatics-mapper branch 2 times, most recently from 6c7c385 to 69681f8 Compare September 27, 2024 11:24
@ambushwork ambushwork requested a review from 0xnm September 27, 2024 11:25
0xnm
0xnm previously approved these changes Sep 27, 2024
Copy link
Member

@0xnm 0xnm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice work!

@ambushwork ambushwork merged commit c37edce into feature/sr_compose Sep 27, 2024
23 checks passed
@ambushwork ambushwork deleted the yl/compose/semenatics-mapper branch September 27, 2024 12:51
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.

4 participants