Skip to content

Fix #12045: fix mvnup plugin upgrade strategy for inherited plugins from remote parent POMs#12054

Merged
gnodet merged 1 commit into
maven-4.0.xfrom
backport/maven-4.0.x/pr-12046
May 17, 2026
Merged

Fix #12045: fix mvnup plugin upgrade strategy for inherited plugins from remote parent POMs#12054
gnodet merged 1 commit into
maven-4.0.xfrom
backport/maven-4.0.x/pr-12046

Conversation

@gnodet
Copy link
Copy Markdown
Contributor

@gnodet gnodet commented May 14, 2026

Backport of #12046 to maven-4.0.x.

Summary

  • Change log level from debug to warning when effective model analysis fails in PluginUpgradeStrategy
  • Fix standalone session transport configuration: register transporter factories via @Provides methods so they properly feed into the DI Map<String, TransporterFactory>
  • Add tests for inherited plugin detection and warning on unresolvable parents

Closes #12045

Claude Code on behalf of Guillaume Nodet

…rom remote parent POMs

* Fix #12045: warn when effective model analysis fails for inherited plugins

Change the log level from debug to warning when effective model analysis
fails in PluginUpgradeStrategy, so users are aware when plugins inherited
from remote parent POMs cannot be detected for upgrade.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Fix standalone session transport for effective model resolution

Register transporter factories via @provides methods so they properly
feed into the DI Map<String, TransporterFactory>, fixing the standalone
session's ability to resolve remote parent POMs from Maven Central.

The previous approach of binding TransporterProvider directly was
overridden by the @provides method in RepositorySystemSupplier which
received an empty factory map, leaving no HTTP transport available.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Add test for inherited plugin detection from remote parent POM

Verifies that the standalone session can resolve org.apache:apache:23
from Maven Central and detect maven-enforcer-plugin:1.4.1 in the
parent's pluginManagement as needing a Maven 4 compatibility upgrade.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@gnodet gnodet merged commit 834daf2 into maven-4.0.x May 17, 2026
40 of 42 checks passed
@gnodet gnodet deleted the backport/maven-4.0.x/pr-12046 branch May 17, 2026 18:49
@github-actions github-actions Bot added this to the 4.0.0 milestone May 17, 2026
@github-actions
Copy link
Copy Markdown

@gnodet Please assign appropriate label to PR according to the type of change.

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.

1 participant