Skip to content

ADFA-5640: Rename Extensions Manager to Addons Manager - #2048

Merged
jimturner-adfa merged 4 commits into
stagefrom
ADFA-5640-rename-Extension-Manager-to-Addon-Manager
Oct 6, 2026
Merged

jimturner-adfa merged 4 commits into
stagefrom
ADFA-5640-rename-Extension-Manager-to-Addon-Manager

Conversation

@jimturner-adfa

@jimturner-adfa jimturner-adfa commented Sep 16, 2026 •

Copy link
Copy Markdown
Collaborator

ADFA-5640

Summary

The merged Plugins/Templates screen's Preferences entry point was still labeled "Extensions Manager", a name predating the Templates merge. Renamed to "Addons Manager" (and its summary from "Manage IDE plugins and templates" to "Manage IDE addons"), matching the parent epic's "unified addon manager" terminology.

  • plugin_manager_title: "Extensions Manager" -> "Addons Manager"
  • plugin_manager_summary: "Manage IDE plugins and templates" -> "Manage IDE addons"
  • Updated the code comments in ManagerScreen.kt and IDEApplication.kt that referred to the feature as "the Extensions Manager", for consistency.
  • The in-screen TopAppBar title ("Plugins & Templates") is a separate string describing the two tabs and is intentionally left unchanged.

Testing

Verified on a physical device: the Preferences list now shows "Addons Manager" / "Manage IDE addons", the screen still opens correctly, no crashes.

🤖 Generated with Claude Code

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@coderabbitai

coderabbitai Bot commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 20d1af8e-0c5c-4617-9a5f-40130e86ba8a
📥 Commits

Reviewing files that changed from the base of the PR and between 39677cd and 6bf1d69.

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

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 4e52eba9-f4db-42e4-8e69-8a89358f0e29
📥 Commits

Reviewing files that changed from the base of the PR and between 0f40bac and 39677cd.

📒 Files selected for processing (2)
  • app/src/main/java/com/itsaky/androidide/app/IDEApplication.kt
  • resources/src/main/res/values/strings.xml
🚧 Files skipped from review as they are similar to previous changes (2)
  • resources/src/main/res/values/strings.xml
  • app/src/main/java/com/itsaky/androidide/app/IDEApplication.kt

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.


📝 Summary
  • Renamed the Preferences entry from “Extensions Manager” to “Addons Manager.”
  • Changed its summary to “Manage IDE addons.”
  • Updated related comments in IDEApplication.kt and ManagerScreen.kt. The in-screen “Plugins & Templates” title remains unchanged.
  • The author reports that the updated labels appeared on a physical device, the screen opened, and no crashes occurred.
  • Risk and best-practice findings are unavailable because no review findings were supplied.

Walkthrough

Application comments, screen documentation, and resource strings now use “Addons Manager” terminology. Runtime behavior is unchanged.

Changes

Addons Manager terminology

Layer / File(s) Summary
Update Addons Manager terminology
app/src/main/java/com/itsaky/androidide/app/IDEApplication.kt, app/src/main/java/com/itsaky/androidide/ui/compose/ManagerScreen.kt, resources/src/main/res/values/strings.xml
Application comments and screen documentation use “Addons Manager” and “addon.” The manager title and summary describe managing IDE addons.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 39677

The Preferences labels reflect the intended Addons Manager terminology. No issue identified here prevents merging after normal checks.

🚥 Pre-merge checks | ✅ 4 | ❌ 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%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (1 skipped: 1 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: renaming the Extensions Manager to Addons Manager.
Description check ✅ Passed The description explains the label and summary changes, the updated code comments, and the unchanged in-screen title. It matches the changeset.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

A rabbit reads the labels anew,
“Addons Manager” comes into view.
The comments match the title’s name,
While runtime follows just the same.
I hop away through fields of green,
With tidy words and carrots keen.

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

The Preferences entry point and the code comments describing it
referred to the merged Plugins/Templates screen as the "Extensions
Manager", predating its current scope. Renamed to "Addons Manager"
throughout, and updated the entry's summary from "Manage IDE plugins
and templates" to "Manage IDE addons" to match.

The in-screen TopAppBar title ("Plugins & Templates") is a separate
string describing the two tabs and is left as-is.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

@Elissa-AppDevforAll Elissa-AppDevforAll left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought we had agreed on Add-ons Manager with the hyphen. This creates inconsistency BUT changing to Addons Manager and changing the display string later is much better than keeping it called Extensions Manager. So I am making myself a ticket to fix the string, and this can go in.

Image

@jimturner-adfa
jimturner-adfa merged commit a8bf746 into stage Oct 6, 2026
5 checks passed
@jimturner-adfa
jimturner-adfa deleted the ADFA-5640-rename-Extension-Manager-to-Addon-Manager branch October 6, 2026 21:32
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.

6 participants