fix: mention draft update [WPB-12062]

[Garzas]

WPB-12062 [Android] add mentions to draft messages

---

PR Submission Checklist for internal contributors

• The PR Title

  • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
  • contains a reference JIRA issue number like SQPIT-764
  • answers the question: If merged, this PR will: ... ³

• The PR Description

  • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

---

What's new in this PR?

Issues

• Fixed a race condition in MessageCompositionHolder where mentions were not always properly saved in the message draft.
• Previously, the mention text could be inserted before selectedMentions was updated, leading to cases where the draft was saved without the correct mention data.

Causes (Optional)

• The issue was caused by an incorrect order of operations:
  • insertMentionIntoText(mention) was called before updating selectedMentions, modifying messageTextState immediately.
  • This could lead to onSaveDraft() being triggered with an outdated selectedMentions list, causing mentions to be missing from the draft.

Solutions

• Reordered the operations to ensure selectedMentions is updated before inserting the mention into the text:

  messageComposition.update {
      it.copy(
          selectedMentions = it.selectedMentions.plus(mention).sortedBy { it.start }
      )
  }
  insertMentionIntoText(mention) // Now safely modifying text after updating mentions

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.46%. Comparing base (24cdf16) to head (9d2e032).
Report is 1 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3865   +/-   ##
========================================
  Coverage    45.46%   45.46%           
========================================
  Files          491      491           
  Lines        17008    17008           
  Branches      2846     2846           
========================================
+ Hits          7732     7733    +1     
+ Misses        8489     8488    -1     
  Partials       787      787           

Files with missing lines	Coverage Δ
.../messagecomposer/state/MessageCompositionHolder.kt	46.51% <100.00%> (ø)

... and 1 file with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 24cdf16...9d2e032. Read the comment docs.

[github-actions]

Built wire-android-staging-compat-pr-3865.apk is available for download

[github-actions]

Built wire-android-dev-debug-pr-3865.apk is available for download