Code action 'Create method...' does not work

[loralei28]

Environment

• Operating System: Windows
• JDK version:17
• Visual Studio Code version:1.93.1
• Java extension version: redhat-1.35.0

Steps To Reproduce

1. Create a new Java file, for example Test.java
2. In Test.java file, invoke class and main code snippet to generate code
3. Type bar();
4. Click ‘Quick fix..’ in the java file or click 'Show fix' in the error window
5. Select code action "Create method 'bar()'"

Current Result

”Create method 'bar()'“ does not work and will delete "bar()"

[rgrunber]

Screencast.from.2024-09-26.08-52-55.mp4

Seems to be working for me. @loralei28 , @testforstephen what am I missing here. Also, if you activate Java: Open Java Language Server Log File, are there any errors in that log ? Does the "Extension Host" logs (from the dropdown) of the Output view indicate any errors ?

Could definitely see it being a regression of eclipse-jdtls/eclipse.jdt.ls#3229 though.

[datho7561]

I reproduced this issue, it seems like it's Windows-specific

[datho7561]

Here is a sample file:

package functions;

import io.quarkus.funqy.Funq;

/**
 * Your Function class
 */
public class Function {

    /**
     * Use the Quarkus Funqy extension for our function. This function simply echoes its input
     * @param input a Java bean
     * @return a Java bean
     */
    @Funq
    public Output function(Input input) {

        // Add business logic here
        bar();

        return null;
    }

}

Here is a trace of what the server returns for the quickfix:

[Trace - 10:45:24 AM] Received response 'codeAction/resolve - (119)' in 5ms.
Result: {
    "title": "Create method 'bar()'",
    "kind": "quickfix",
    "diagnostics": [
        {
            "range": {
                "start": {
                    "line": 18,
                    "character": 8
                },
                "end": {
                    "line": 18,
                    "character": 11
                }
            },
            "severity": 1,
            "code": "67108964",
            "source": "Java",
            "message": "The method bar() is undefined for the type Function"
        }
    ],
    "edit": {
        "changes": {},
        "documentChanges": [
            {
                "textDocument": {
                    "version": null,
                    "uri": "file:///C:/Users/david/Documents/Programming/TestProjects/quarkus-func/src/main/java/functions/Function.java"
                },
                "edits": [
                    {
                        "snippet": {
                            "value": "${1:bar}();\n\n        return null;\n    }\n\n    ${2|private,public,protected|} ${3:void} ${1:bar}() {\n        // TODO Auto-generated method stub\n        throw new UnsupportedOperationException(\"Unimplemented method 'bar'\");\n    }"
                        },
                        "range": {
                            "start": {
                                "line": 18,
                                "character": 8
                            },
                            "end": {
                                "line": 21,
                                "character": 5
                            }
                        },
                        "newText": ""
                    }
                ]
            }
        ]
    },
    "command": {
        "title": "refresh Diagnostics",
        "command": "java.project.refreshDiagnostics",
        "arguments": [
            "file:///c%3A/Users/david/Documents/Programming/TestProjects/quarkus-func/src/main/java/functions/Function.java",
            "thisFile",
            false,
            true
        ]
    }
}

[rgrunber]

I'm seeing the exact same response, only file URI is different (I'm on Linux). When you have time, i would try to validate whether

--- yours.txt	2024-09-26 11:17:54.751100419 -0400
+++ mine.txt	2024-09-26 11:23:02.727259008 -0400
@@ -1,4 +1,4 @@
-[Trace - 10:45:24 AM] Received response 'codeAction/resolve - (119)' in 5ms.
+[Trace - 11:22:36] Received response 'codeAction/resolve - (458)' in 39ms.
 Result: {
     "title": "Create method 'bar()'",
     "kind": "quickfix",
@@ -26,7 +26,7 @@
             {
                 "textDocument": {
                     "version": null,
-                    "uri": "file:///C:/Users/david/Documents/Programming/TestProjects/quarkus-func/src/main/java/functions/Function.java"
+                    "uri": "file:///home/rgrunber/sample-projects/my-quarkus-example/src/main/java/functions/Function.java"
                 },
                 "edits": [
                     {
@@ -53,7 +53,7 @@
         "title": "refresh Diagnostics",
         "command": "java.project.refreshDiagnostics",
         "arguments": [
-            "file:///c%3A/Users/david/Documents/Programming/TestProjects/quarkus-func/src/main/java/functions/Function.java",
+            "file:///home/rgrunber/sample-projects/my-quarkus-example/src/main/java/functions/Function.java",
             "thisFile",
             false,
             true

Definitely seems like something on the client side that's treated differently.

Update: Maybe the LSP text document URI is slightly different than the converted to VS Code text document URI ?

vscode-java/src/extension.ts

Line 334 in 29329d8

if (documentUris.includes(uri.toString())) {

.

[testforstephen]

Looks like snippet text of "codeAction/resolve" response is not respected on Windows, and it still applies newText, which is "" and behaves as delete.

[fp024]

Hello.

After upgrading the extension to version 1.35.0 in Windows 11 environment, Create local Variable '???' does not work properly.

However, there was no problem in Ubuntu 22.04 environment.

Windows 11 + Ext Ver. 1.35.0 (It became "")

win-do-not-work-create-val.mp4

Ubuntu 22.04 + Ext Ver. 1.35.0

ubuntu-do-work-create-val.mp4

[rgrunber]

@testforstephen , can you try the PR and let me know if it resolves it for you ? It should prevent newText from being populated.

If someone could try out java-win32-x64-1.36.0-179.vsix (on windows) from https://github.com/rgrunber/vscode-java/releases/tag/v1.36.0 and let me know if that fixes things for them, that would help.

[fp024]

In my Windows 11 environment, installing 1.36.0 did not solve the problem.

[rgrunber]

@testforstephen , I think we should just disable the feature ( snippetEditSupport in extendedClientCapabilities ) until there's some kind of fix upstream / we find a proper solution for windows. Could we do a bug-fix release for Monday ?

[testforstephen]

@testforstephen , I think we should just disable the feature ( snippetEditSupport in extendedClientCapabilities ) until there's some kind of fix upstream / we find a proper solution for windows. Could we do a bug-fix release for Monday ?

+1