lukaszlenart opened a new pull request, #51:
URL: https://github.com/apache/struts-intellij-plugin/pull/51

   ## Summary
   
   - Resolve 21 critical nullability/NPE warnings identified by Qodana CI 
across 15 source files
   - Add null guards for `DomElement.getXmlTag()`, `PsiClass.getName()`, 
`PathReference.resolve()` and other `@Nullable` return values passed to 
`@NotNull` parameters
   - Replace `FileTypes.UNKNOWN.getIcon()` (nullable) with 
`AllIcons.FileTypes.Unknown` (guaranteed non-null)
   - Fix `@Nullable` annotation on `StrutsDataModel.createGroupNodeRealizer()` 
override that always returns non-null
   
   ## Test plan
   
   - [x] `./gradlew compileJava` — compilation succeeds with no new warnings
   - [x] `./gradlew test -x rat` — all existing tests pass with no regressions
   - [ ] Verify Qodana annotation count drops from 43 to ~22 (remaining: 
generated code, capitalization, deprecation, unstable API)
   
   🤖 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]

Reply via email to