Update modernize-java plugin#1763
Open
leonard520 wants to merge 1 commit into
Open
Conversation
…ure-java-migration-copilot-vscode-extension@4657622 Source commit: 4657622462e4904f9619e35b4f2919bfa56c454e Source branch: fix/sync-from-release-tag Triggered by: haital_microsoft Workflow: https://github.com/devdiv-azure-service-dmitryr/azure-java-migration-copilot-vscode-extension/actions/runs/26074926939
Contributor
🔍 Skill Validator Results✅ All checks passed
Summary
Full validator output```text Found 1 agent(s) Validated 1 agent(s)✅ All checks passed (1 agent(s)) |
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the modernize-java plugin to align with upstream release 1.17.0, including MCP server invocation tweaks and a significant refresh of the modernize-java agent’s workflow/plan-generation guidance.
Changes:
- Bumped plugin version to
1.17.0and pinned the MCP server package to@1.17.0, adding--callerType plugin. - Updated the
modernize-javaagent guidance (plan format spec, event reporting details, precheck behavior changes). - Added a
PostToolUsehook to the agent front matter to run telemetry scripts after tool usage.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| plugins/modernize-java/.github/plugin/plugin.json | Bumps plugin version and adjusts MCP server npx args (pin + callerType). |
| agents/modernize-java.agent.md | Expands/changes upgrade workflow instructions and adds a PostToolUse hook command. |
Comment on lines
+18
to
+19
| command: "bash .github/java-upgrade/hooks/scripts/recordToolUse.sh" | ||
| windows: "powershell -NonInteractive -File .github/java-upgrade/hooks/scripts/recordToolUse.ps1" |
Contributor
|
Is this in a public github repo? Would it be worth migrating to the external plugin model? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…ure-java-migration-copilot-vscode-extension@4657622
Source commit: 4657622462e4904f9619e35b4f2919bfa56c454e
Source branch: fix/sync-from-release-tag
Triggered by: haital_microsoft
Workflow: https://github.com/devdiv-azure-service-dmitryr/azure-java-migration-copilot-vscode-extension/actions/runs/26074926939
Pull Request Checklist
npm startand verified thatREADME.mdis up to date.stagedbranch for this pull request.Description
Type of Contribution
Additional Notes
By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.