Add vmargs setting for qute-ls, disable JVM logging

Closes #548

Signed-off-by: David Thompson <[email protected]>

Author: datho7561

docs/qute/README.md

@@ -9,13 +9,22 @@
 
  ## Settings
 
+ * `qute.server.vmargs`: Set the arguments to pass to the JVM while launching qute-ls. Default is `-Xmx100M -XX:+UseG1GC -XX:+UseStringDeduplication -Xlog:disable`
  * `qute.trace.server`: Trace the communication between VS Code and the Qute language server in the Output view. Default is `off`.
  * `qute.native.enabled`: Enable/disable Qute validation/completion for native image mode. Default is `false`.
  * `qute.codeLens.enabled`: Enable/disable Qute CodeLens. Default is `true`.
  * `qute.inlayHint.enabled`: Enable/disable Inlay Hint. Default is `true`.
  * `qute.inlayHint.showSectionParameterType`: Show section parameter type. Default is `true`.
  * `qute.inlayHint.showMethodParameterType`: Show method parameter type. Default is `true`.
  * `qute.validation.enabled`: Enable/disable all Qute validation. Default is `false`.
- * `qute.validation.excluded`: Disable Qute validation for the given file name patterns.\n\nExample:\n```\n[\n \"**/*items.qute.*\"\n]```.
+ * `qute.validation.excluded`: Disable Qute validation for the given file name patterns.
+
+   Example:
+
+   ```json
+   [
+     "**/*items.qute.*"
+   ]
+   ```
  * `qute.validation.undefinedObject.severity`: Validation severity for undefined object in Qute template files. Default is `warning`.
  * `qute.validation.undefinedNamespace.severity`: Validation severity for undefined namespace in Qute template files. Default is `warning`.

package.json

@@ -225,6 +225,12 @@
           "description": "Action performed when detected Quarkus properties have an incorrect language.",
           "scope": "window"
         },
+        "qute.server.vmargs": {
+          "type": "string",
+          "scope": "application",
+          "markdownDescription": "The vmargs to use when launching the Qute language server",
+          "default": "-Xmx100M -XX:+UseG1GC -XX:+UseStringDeduplication -Xlog:disable"
+        },
         "qute.templates.languageMismatch": {
           "type": "string",
           "enum": [

src/qute/languageServer/client.ts

@@ -3,7 +3,7 @@ import * as requirements from './requirements';
 import { DidChangeConfigurationNotification, LanguageClientOptions } from 'vscode-languageclient';
 import { LanguageClient } from 'vscode-languageclient/node';
 import { ExtensionContext, commands, workspace, window, ConfigurationTarget, languages } from 'vscode';
-import { prepareExecutable } from './javaServerStarter';
+import { prepareExecutable } from './quteServerStarter';
 import { registerQuteExecuteWorkspaceCommand, registerVSCodeQuteCommands, synchronizeQuteValidationButton } from '../commands/registerCommands';
 import { QuteClientCommandConstants } from '../commands/commandConstants';
 import { QuteSettings } from './settings';

src/qute/languageServer/javaServerStarter.ts renamed to src/qute/languageServer/quteServerStarter.ts

@@ -30,13 +30,19 @@ function prepareParams(): string[] {
     }
   }
 
-  const vmargs = workspace.getConfiguration("quarkus.tools").get("server.vmargs", '');
+  const vmargs = workspace.getConfiguration("qute").get("server.vmargs", '');
   if (os.platform() === 'win32') {
     const watchParentProcess = '-DwatchParentProcess=';
     if (vmargs.indexOf(watchParentProcess) < 0) {
       params.push(watchParentProcess + 'false');
     }
   }
+  // Disable logging unless the user specifically sets it to a different value.
+  // Logging can cause issues, since sometimes it writes to standard out.
+  // See https://github.com/redhat-developer/vscode-java/issues/2577.
+  if (vmargs.indexOf("-Xlog:") < 0) {
+    params.push("-Xlog:disable");
+  }
   parseVMargs(params, vmargs);
   const serverHome: string = path.resolve(__dirname, '../server');
   const quteServerFound: Array<string> = glob.sync(`**/${QUTE_SERVER_NAME}`, { cwd: serverHome });