feat(mobile): Long press to copy message text

[Crypto-Virus]

copy-feature-clean.mp4

Summary

• Long press message bubbles to open a copy sheet with selectable text and a "Copy all" button
• Animated confirmation with haptic feedback, auto-dismisses after 600ms
• Extracted createExternalStore utility — refactored both messageCopySelection and reviewCommentSelection to use it, eliminating ~60 lines of duplicate boilerplate

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 66b14234-c720-4b53-a064-68d4361572d2

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[macroscopeapp]

🟡 Medium threads/MessageCopySheet.tsx:45

In onCopyAll, if the button is pressed multiple times before React re-renders with disabled={copied}, the code overwrites dismissTimerRef.current without clearing the previous timer. This leaks the previous timer ID, and if the component unmounts before all timers fire, only the last timer gets cleaned up. The orphaned timer(s) will still call router.back() potentially after the user has navigated elsewhere.

  const onCopyAll = useCallback(() => {
    if (messageText) {
      void Clipboard.setStringAsync(messageText);
      void Haptics.notificationAsync(Haptics.NotificationFeedbackType.Success);
      setCopied(true);
+      if (dismissTimerRef.current) {
+        clearTimeout(dismissTimerRef.current);
+      }
      dismissTimerRef.current = setTimeout(() => {
        dismissTimerRef.current = null;
        router.back();
      }, 600);
    }
  }, [messageText, router]);

🤖 Copy this AI Prompt to have your agent fix this:

In file apps/mobile/src/features/threads/MessageCopySheet.tsx around lines 45-55:

In `onCopyAll`, if the button is pressed multiple times before React re-renders with `disabled={copied}`, the code overwrites `dismissTimerRef.current` without clearing the previous timer. This leaks the previous timer ID, and if the component unmounts before all timers fire, only the last timer gets cleaned up. The orphaned timer(s) will still call `router.back()` potentially after the user has navigated elsewhere.

Evidence trail:
apps/mobile/src/features/threads/MessageCopySheet.tsx lines 44-54 (onCopyAll function creates timer without clearing existing one), lines 37-42 (cleanup effect only clears dismissTimerRef.current which is only the last timer), lines 49-52 (timer assignment overwrites ref without clearTimeout), line 99 (disabled={copied} relies on React re-render)

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 6bead6f. Configure here.}

[cursor]

Long-pressing image-only messages opens empty copy sheet

Medium Severity

The onLongPress handler unconditionally passes message.text to the copy sheet, even when the text is empty (e.g., image-only messages). Since OrchestrationMessage.text is Schema.String (allows ""), setMessageCopyText("") stores an empty string. In MessageCopySheet, the guard if (!messageText) treats "" as falsy and returns null, so the user sees a confusing empty form sheet (just a grabber on a blank background) that they must manually dismiss. The onLongPressMessage handler needs to skip opening the sheet when the text is empty.

Additional Locations (1)

• apps/mobile/src/features/threads/MessageCopySheet.tsx#L56-L58

 

^{Reviewed by Cursor Bugbot for commit 6bead6f. Configure here.}

[macroscopeapp]

Approvability

Verdict: Needs human review

1 blocking correctness issue found. This PR introduces a new user-facing feature (long press to copy message text) with new components, routes, and state management. Additionally, there are two unresolved medium-severity bugs identified: a timer leak on rapid button presses and an empty sheet appearing for image-only messages.

^{You can customize Macroscope's approvability policy. Learn more.}