Skip to content

Route appmod prompt to local agent provider - #12253

Merged
wangmingliang-ms merged 2 commits into
developfrom
haozhan/route-local
Jul 29, 2026
Merged

Route appmod prompt to local agent provider#12253
wangmingliang-ms merged 2 commits into
developfrom
haozhan/route-local

Conversation

@haoozhang

@haoozhang haoozhang commented Jul 28, 2026

Copy link
Copy Markdown

What does this implement/fix? Explain your changes.

This pull request updates the Java Upgrade notification service to ensure that Java upgrade prompts are always routed to the local Copilot agent, improving reliability and compatibility with custom agents. The main changes involve adding a new helper method and integrating it into the query builder logic using reflection for backward compatibility.

Copilot Agent Routing Improvements:

  • Added a new method withLocalAgentProvider that pins queries to the local Copilot agent provider using reflection, ensuring the Java Upgrade prompt uses the correct agent. This method gracefully handles older Copilot plugin versions by silently skipping if the required API is unavailable.
  • Integrated the withLocalAgentProvider method into the Copilot query builder flow, so all Java upgrade prompts are explicitly routed to the local agent.

Minor cleanup:

  • Removed an unused import of AzureTelemeter from JavaVersionNotificationService.java.

Does this close any currently open issues?

Any relevant logs, screenshots, error output, etc.?

Any other comments?

Has this been tested?

  • Tested

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

@haoozhang haoozhang changed the title Route appmod prompt to local agent Route appmod prompt to local agent provider Jul 29, 2026
@wangmingliang-ms
wangmingliang-ms merged commit 762787d into develop Jul 29, 2026
5 checks passed
@wangmingliang-ms
wangmingliang-ms deleted the haozhan/route-local branch July 29, 2026 09:11
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.

3 participants