ADFA-2553 Switch from incorrect LongPress interception to Sora LongPress handler

[hal-eisen-adfa]

To correctly select text the user intended for copy/paste from Build Output and IDE Logs

Might also fix ADFA-2536

The real bug was in copyTextToClipboard - it was prepending build info to the entire buffer then extracting with original indices, causing wrong text to be copied
Fixed by extracting selected text first, then prepending build info only when needed
Removed includeDebugInfoOnCopy = true from log views so normal copy works correctly
Share/feedback still uses getShareableContent() which includes build info

[coderabbitai]

📝 Walkthrough

Walkthrough

Removed EventBus-based long-press event plumbing and an EditorLongPressEvent type; also changed copy behavior so editors no longer include debug info by default (some code paths stop setting includeDebugInfoOnCopy).

Changes

Cohort / File(s)	Summary
EventBus long-press removal app/src/main/java/com/itsaky/androidide/fragments/EmptyStateFragment.kt, editor/src/main/java/com/itsaky/androidide/editor/ui/EditorLongPressEvent.kt, editor/src/main/java/com/itsaky/androidide/editor/ui/IDEEditor.kt	Deleted EventBus imports, lifecycle register/unregister calls, and the onEditorLongPressed handler; removed the EditorLongPressEvent data class; IDEEditor long-press forwarding/posts removed.
Copy debug-info behavior changes editor/src/main/java/com/itsaky/androidide/editor/ui/IDEEditor.kt, app/src/main/java/com/itsaky/androidide/fragments/output/BuildOutputFragment.kt, app/src/main/java/com/itsaky/androidide/fragments/output/LogViewFragment.kt	Adjusted copy-to-clipboard logic in IDEEditor to include debug info only when includeDebugInfoOnCopy is true and to prepend debug info only to the selected text; removed code that enabled includeDebugInfoOnCopy in BuildOutputFragment and LogViewFragment.

Sequence Diagram(s)

(omitted)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• ADFA-2106 | Enable text selection in build logs #732 — touches long-press handling and editor selection exposure; likely related to long-press/event forwarding changes.

Suggested reviewers

• dara-abijo-adfa
• Daniel-ADFA

Poem

🐰 I hopped through code with nimble feet,
Snipped the bus that sent long-presses fleet.
Copies now carry only what you choose,
No extra traces left to bruise.
A quiet patch, a tidy feat. 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately reflects the main change: removing EventBus-based LongPress interception and related event handling from multiple files.
Description check	✅ Passed	The PR description clearly relates to the changeset, explaining the bug fix for text selection in copy/paste from Build Output and IDE Logs, and how it was resolved by fixing copyTextToClipboard() and removing incorrect LongPress interception.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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.}

[hal-eisen-adfa]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[dara-abijo-adfa]

LGTM