Michael Glavassevich

Windows: dev mode appears to stop listening for updates to server.xml

1

I have been testing `3.8.2-SNAPSHOT` and have observed that after changing server.xml once, further updates to the file are not synch'd up by dev mode. I ran `mvn liberty:dev` with:...

org.eclipse.lsp4jakarta.commons.JavaCursorContextKind.forValue() throws an IllegalArgumentException when passed the value from its own NONE constant.

1

This was discovered while testing completion support in Liberty Tools for IntelliJ. See discussion here: https://github.com/OpenLiberty/liberty-tools-intellij/issues/681. The problem is that the `forValue()` method assumes that all of the values are...

This came up during a code review of Liberty Tools for IntelliJ which included a ported version of [JakartaProjectLabelProvider](https://github.com/eclipse/lsp4jakarta/blob/main/jakarta.jdt/org.eclipse.lsp4jakarta.jdt.core/src/main/java/org/eclipse/lsp4jakarta/jdt/internal/core/providers/JakartaProjectLabelProvider.java). See discussion here: https://github.com/OpenLiberty/liberty-tools-intellij/pull/614#discussion_r1463658593. Currently `JakartaProjectLabelProvider` is determining that a project...

Investigate whether LSP4Jakarta consistently processes fully qualified class names.

2

- Check whether LSP4Jakarta has the correct behaviour for applications if they specify a fully qualified class name instead of their simple name (e.g. `@jakarta.ws.rs.PUT` vs `@PUT`). - Check whether...

Disposer method quick fixes remove all annotations instead of mutually exclusive ones.

1

Pull request https://github.com/eclipse/lsp4jakarta/pull/511 added a quick fix which removes all three annotations from a parameter when some of the annotations are in conflict. E.g. "`@Disposes` `@Observes` `@ObservesAsync` String name" There...

Explore whether the user experience can be improved for diagnostics / quick fixes involving @Dependent

1

The screenshots I'm including here are from Liberty tools for IntelliJ, but this also applies to LSP4Jakarta. Consider a managed bean that has been annotated with multiple scopes:  The...

During a recent code review for a PR in Liberty Tools for IntelliJ (https://github.com/OpenLiberty/liberty-tools-intellij/pull/1037) we noticed a comment in the code "A single field cannot have the same" which we...

While refactoring similar code in Liberty Tools for IntelliJ I noticed that it is looping over all methods on the application's class looking for annotations that seem to just apply...

Reduce code duplication in PersistenceMapKeyDiagnosticsParticipant.

1

See discussion here: https://github.com/OpenLiberty/liberty-tools-intellij/issues/1024. The work done for IntelliJ should be ported back.