Greptile Summary

Moves the Patterns.WEB_URL.matcher() regex computation in LinkifiedMessageText off the main thread by replacing a synchronous remember block with a LaunchedEffect + withContext(Dispatchers.Default) pattern, addressing Sentry issue COLUMBA-3C. The initial render shows plain text while links are computed asynchronously, then updates the composable state with the linkified AnnotatedString.

• Core change: Regex-based URL detection now runs on Dispatchers.Default instead of blocking the main thread during composition
• UX trade-off: Single-frame delay before links appear — imperceptible in practice since message text is stable and the regex completes quickly
• Formatting: Minor import reordering (alphabetical sort) and indentation fixes for LifecycleEventObserver and LazyColumn modifier chains
• No issues found: The remember and LaunchedEffect keys are correctly synchronized, pointerInput restarts when annotatedText updates, and buildAnnotatedString is safe to call from background threads

Confidence Score: 5/5

• This PR is safe to merge — it's a straightforward performance optimization with correct coroutine/state management and no behavioral regressions.
• The change is well-scoped: it moves a single regex computation off the main thread using standard Compose coroutine patterns. The remember and LaunchedEffect keys are correctly aligned, the pointerInput handler properly restarts on state changes, and the fallback behavior (plain text during async computation) is graceful. The remaining changes are purely cosmetic (import sorting, indentation).
• No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant C as Composition (Main Thread)
    participant S as State (annotatedText)
    participant BG as Dispatchers.Default
    participant UI as Text Composable

    C->>S: remember(text, linkColor) → plain AnnotatedString
    S->>UI: Render plain text (no links)
    C->>BG: LaunchedEffect → withContext(Dispatchers.Default)
    BG->>BG: Patterns.WEB_URL.matcher(text)
    BG->>BG: buildAnnotatedString with link styles
    BG->>S: Update annotatedText with linkified result
    S->>UI: Recompose with clickable links
    Note over UI: pointerInput restarts with new annotatedText key

_{Last reviewed commit: b111c78}

Codecov Report

❌ Patch coverage is 39.47368% with 23 lines in your changes missing coverage. Please review.

📢 Thoughts on this report? Let us know!