Skip to content

Remove unused CopilotUsageBar - #479

Open
Lars Vogel (vogella) wants to merge 1 commit into
microsoft:mainfrom
vogellacompany:remove-copilot-usage-bar
Open

Lars Vogel (vogella) wants to merge 1 commit into
microsoft:mainfrom
vogellacompany:remove-copilot-usage-bar

Conversation

@vogella

Copy link
Copy Markdown
Contributor

CopilotUsageBar is not instantiated or referenced anywhere in the repository; the only remaining AbstractUsageBar subclass in use is ContextWindowBar. This removes the class together with the four getUsageBar*Color helpers in CssConstants, which had no other callers, dropping 84 lines of dead code.

Nothing instantiates CopilotUsageBar, so remove it together with the usage bar color helpers in CssConstants that only it called.

Assisted-by: multiple AI agents and layers of automated tooling 🤖

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

Repository searches confirm no remaining references to the deleted class or helpers.

0 open findings

What changed in this PR

Removes an unused SWT usage-bar implementation and its orphaned color helpers.

Changes:

  • Deletes CopilotUsageBar.
  • Removes its four unused color methods.
File Description
CssConstants.java Removes unused usage-bar color helpers.
CopilotUsageBar.java Deletes the unreferenced widget.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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