Skip to content

ADFA-3459 | Add AI tooltip tag to Explain Selection action#1135

Merged
jatezzz merged 2 commits intostagefrom
feat/ADFA-3459-add-ai-tooltip-tag
Mar 31, 2026
Merged

ADFA-3459 | Add AI tooltip tag to Explain Selection action#1135
jatezzz merged 2 commits intostagefrom
feat/ADFA-3459-add-ai-tooltip-tag

Conversation

@jatezzz
Copy link
Copy Markdown
Collaborator

@jatezzz jatezzz commented Mar 30, 2026

Description

This PR introduces a new tooltip tag specifically for AI-related actions in the editor toolbar and assigns it to the ExplainSelectionAction. This ensures that the "Explain Selection" feature can now display a targeted tooltip to improve user discoverability and guidance.

Details

  • Defined EDITOR_TOOLBAR_AI constant in TooltipTag.kt.
  • Overrode retrieveTooltipTag in ExplainSelectionAction.kt to return the new AI tag.
image image

Ticket

ADFA-3459

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 30, 2026

📝 Walkthrough

Walkthrough

This pull request adds tooltip tag support to the ExplainSelectionAction by introducing a new tooltip tag constant and implementing the tooltip retrieval method to identify the action's UI context for editor toolbar AI features.

Changes

Cohort / File(s) Summary
Tooltip Tag Addition
idetooltips/src/main/java/com/itsaky/androidide/idetooltips/TooltipTag.kt
Added EDITOR_TOOLBAR_AI constant with value "editor.toolbar.ai" to the TooltipTag object.
Tooltip Tag Implementation
app/src/main/java/com/itsaky/androidide/actions/agent/ExplainSelectionAction.kt
Implemented retrieveTooltipTag(isReadOnlyContext: Boolean) override returning TooltipTag.EDITOR_TOOLBAR_AI to expose tooltip metadata for UI integration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • itsaky-adfa
  • Daniel-ADFA
  • dara-abijo-adfa

Poem

🐰 A tag for AI tips so bright,
Explaining code with helpful light,
The toolbar gleams with tooltip care,
Now users know just what goes where!
hops with glee

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.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 clearly and concisely describes the main change: adding an AI tooltip tag to the Explain Selection action, which matches the changeset.
Description check ✅ Passed The PR description clearly relates to the changeset, explaining the addition of a new tooltip tag for AI actions and its assignment to ExplainSelectionAction.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/ADFA-3459-add-ai-tooltip-tag

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

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

@jatezzz jatezzz closed this Mar 31, 2026
@jatezzz jatezzz reopened this Mar 31, 2026
@jatezzz jatezzz merged commit 304e3fd into stage Mar 31, 2026
2 checks passed
@jatezzz jatezzz deleted the feat/ADFA-3459-add-ai-tooltip-tag branch March 31, 2026 20: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.

2 participants