NPE on ASTNode.getParent() codeAction failure in VSCode after getData() is null

[evie-lau]

Unsure if this is related to the new cursor context code that was merged in #453.

Both of these shift focus to the Output window when the error occurs.

NPE on ASTNode.getParent()

When moving the cursor (clicking or arrow keys) to some highlighted diagnostics in a Jakarta project, this error shows up:

[Error - 5:13:30 PM] Request textDocument/codeAction failed.
  Message: Request jakarta/java/codeaction failed with message: java.lang.NullPointerException: Cannot invoke "org.eclipse.jdt.core.dom.ASTNode.getParent()" because "node" is null
  Code: -32603 

Some examples to reproduce:

• In ManagedBeanConstructor.java, move cursor to the diagnostic on the constructor definition.
  

• In ManagedBean.java, move cursor to the diagnostic on the class name, or the a field
  

• In InjectAndDisposesObservesObservesAsync.java, move cursor to any diagnostic on the method names.
  

ClassCastException moved to #455

[yeekangc]

Let us break each exception into their own issue.

[yeekangc]

Thanks for breaking out the issues, @turkeylurkey!

[yeekangc]

Need to determine how pervasive this issue is.

[rumanaHaque]

In my testing with VSCode using the latest driver dated May 29th, liberty-dev-vscode-ext-23.0.6-SNAPSHOT.vsix, I was not able to recreate this exception.

[dshimo]

Simply adding the @Resource annotation to a class consistently recreates this error. I was able to hit this with the 23.0.6 driver on Mac and Linux.

[cherylking]

Is this stop ship?

[rumanaHaque]

With @dshimo's help, I was also able to reproduce in my env.

[turkeylurkey]

Adding @resource causes a diagnostic which uses ScopeDeclarationQuickFix. This quick fix uses diagnostic.getData() which returns null and causes the NPE. Caused by microsoft/vscode#181166