Skip to content

refactor: remove experimental CreateBeanPropertyFixes dependency#71

Open
lukaszlenart wants to merge 1 commit intomainfrom
refactor/remove-experimental-CreateBeanPropertyFixes
Open

refactor: remove experimental CreateBeanPropertyFixes dependency#71
lukaszlenart wants to merge 1 commit intomainfrom
refactor/remove-experimental-CreateBeanPropertyFixes

Conversation

@lukaszlenart
Copy link
Copy Markdown
Member

Summary

  • Remove usage of IntelliJ's experimental CreateBeanPropertyFixes API from BeanPropertyPathReference.getQuickFixes() and the delegating quick-fix aggregation in ResultParamNameCustomConverter.MergingBeanPropertyPathReference.
  • Eliminates the plugin verifier warning: "Experimental API class CreateBeanPropertyFixes is referenced in BeanPropertyPathReference.getQuickFixes(). This class can be changed in a future release leading to incompatibilities."
  • Bean-property resolution, completion, highlighting, rename, and bind behavior are all preserved.

Test plan

  • StrutsHighlightingTest passes (covers unresolved bean-property diagnostics in struts.xml params)
  • ValidationHighlightingTest passes (covers unresolved field names in validation XML)
  • Full compileJava and compileTestJava succeed with zero new warnings
  • No remaining references to CreateBeanPropertyFixes in src/
  • Zero linter diagnostics on edited files

Made with Cursor

Drop usage of the experimental `CreateBeanPropertyFixes` API from
`BeanPropertyPathReference` and the delegating quick-fix aggregation
in `ResultParamNameCustomConverter.MergingBeanPropertyPathReference`.

This eliminates the plugin verifier warning about referencing an
`@ApiStatus.Experimental` class that may change in future IntelliJ
Platform releases. Bean-property resolution, completion, highlighting,
and rename/bind behavior are unaffected.

🤖 Generated by AI Assistant

Made-with: Cursor
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

🔌 Plugin artifact ready for testing!

Download from Actions artifacts

Artifact: struts2-261.18970.1

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