gnodet opened a new pull request, #1094: URL: https://github.com/apache/maven-plugin-tools/pull/1094
## Summary - Fix NPE in `EnhancedPluginDescriptorBuilder.build(Reader, String)` when running under Maven 4 - Maven 4 refactored `PluginDescriptorBuilder.build()` to no longer call `buildConfiguration()`, leaving the `configuration` field null - When `configuration` is null (Maven 4+), read `requiredJavaVersion`/`requiredMavenVersion` from the `PluginDescriptor` directly, since Maven 4's builder already populates them Fixes #806 ## Context This is the main blocker for #1091 (enable Maven 4 CI builds). Maven 4 changed `PluginDescriptorBuilder.build(Reader, String)` in [MNG-8084](https://github.com/apache/maven/commit/003a5bc06d) to delegate to a new `build(ReaderSupplier, String)` method that no longer calls the overridable `buildConfiguration()`. The `EnhancedPluginDescriptorBuilder` relied on that callback to capture the raw XML configuration. ## Test plan - [x] Existing unit tests pass (verified locally) - [ ] CI with Maven 3.9.14 should pass (no behavior change) - [ ] CI with Maven 4 should fix the 12 failing reporting ITs 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
