Is there any plan to support `workspace.workspaceEdit.resourceOperations`

[fortime]

Issue Prelude

Please complete these steps and check these boxes (by putting an x inside the brackets) before filing your issue:

• [x ] I have read and understood YCM's CONTRIBUTING document.
• [x] I have read and understood YCM's CODE_OF_CONDUCT document.
• [x] I have read and understood YCM's README, especially the Frequently Asked Questions section.
• [x] I have searched YCM's issue tracker to find issues similar to the one I'm about to report and couldn't find an answer to my problem. (Example Google search.)
• [ ] If filing a bug report, I have included the output of vim --version.
• [ ] If filing a bug report, I have included the output of :YcmDebugInfo.
• [ ] If filing a bug report, I have attached the contents of the logfiles using the :YcmToggleLogs command.
• [ ] If filing a bug report, I have included which OS (including specific OS version) I am using.
• [ ] If filing a bug report, I have included a minimal test case that reproduces my issue, using vim -Nu /path/to/YCM/vimrc_ycm_minimal, including what I expected to happen and what actually happened.
• [ ] If filing a installation failure report, I have included the entire output of instdeleteall.py (or cmake/make/ninja) including its invocation
• [x] I understand this is an open-source project staffed by volunteers and that any help I receive is a selfless, heartfelt gift of their free time. I know I am not entitled to anything and will be polite and courteous.
• [x] I understand my issue may be closed if it becomes obvious I didn't actually perform all of these steps.

Thank you for adhering to this process! It ensures your issue is resolved quickly and that neither your nor our time is needlessly wasted.

Issue Details

• About workspace.workspaceEdit.resourceOperations

  /**

  • The resource operations the client supports. Clients should at least
  • support 'create', 'rename' and 'delete' files and folders.
  • @since 3.13.0 */ resourceOperations?: ResourceOperationKind[];

• Why In Java, a rename operation is needed to refactor the name of a Java Class. Currently, since there is no support of rename operation, eclipse.jdt.ls won't return the rename operation. We must rename the file manually after a refactor. I think this problem may exist in other language too.

Is there any plan to support workspace.workspaceEdit.resourceOperations?

[bstaletic]

Hmm... so far, we have not implemented those for a mix of many reasons:

• No personal need for such has been encountered.
• The API by which YCM talks to ycm-core/ycmd does not currently support that either.
• These actions can be quite intrusive. What if a server decides to delete the only currently open file?
  • Also, what does delete mean? Delete file? Delete buffer?

None of which are a hard "no".

Would you mind sharing a minimal example and steps that would make jdt respond with a rename operation?

[puremourning]

Can probably just rename a class in Java as the files have to match the class name.

[fortime]

Here is the steps:

1. Apply this patch to ycmd, commit id: 8b61f198. The commit id of YouCompleteMe is 63ab13e9.

   
   diff --git a/ycmd/completers/java/java_completer.py b/ycmd/completers/java/java_completer.py
   index 33803957..03dbc1c4 100644
   --- a/ycmd/completers/java/java_completer.py
   +++ b/ycmd/completers/java/java_completer.py
   @@ -660,3 +660,13 @@ class JavaCompleter( language_server_completer.LanguageServerCompleter ):
          *language_server_completer._LspLocationToLocationAndDescription(
            request_data, item[ 'from' ] ) )
    return result
   +
   +
   +  def ExtraCapabilities( self ):
   +    return {
   +      'workspace': {
   +        'workspaceEdit': {
   +          'resourceOperations': [ 'create', 'rename', 'delete' ],
   +        }
   +      }
   +    }
   diff --git a/ycmd/completers/language_server/language_server_completer.py b/ycmd/completers/language_server/language_server_completer.py
   index 489e15d0..ebc45b11 100644
   --- a/ycmd/completers/language_server/language_server_completer.py
   +++ b/ycmd/completers/language_server/language_server_completer.py
   @@ -3587,6 +3587,8 @@ def WorkspaceEditToFixIt( request_data,
   """Converts a LSP workspace edit to a ycmd FixIt suitable for passing to
   responses.BuildFixItResponse."""

• LOGGER.info( 'workspace_edit: %s', workspace_edit )

• if not workspace_edit: return None

2. Create a maven project. The rename operation won't be returned in a single java source file.

   mvn archetype:generate -DgroupId=test -DartifactId=test-resource-op -DarchetypeArtifactId=maven-archetype-quickstart -DarchetypeVersion=1.5 -DinteractiveMode=false

3. Go into the project. And open a java file with vim.

   cd test-resource-op
   vim src/main/java/test/App.java

4. Move the cursor to the class name, and run YcmCompleter RefactorRename App1. The rename operation will appear in the log file.

BTW, there is another issue here. In WorkspaceEditToFixIt, it checks if there is changes in the response. In above case, jdt returns both changes and documentChanges, and documentChanges is being ignored.

[puremourning]

BTW, there is another issue here. In WorkspaceEditToFixIt

Hardly surprising given that is only used with the capability you just added.