Replace paranoidWarnings with errorReportingMode

Signed-off-by: Ben Sherman <bentshermann@gmail.com>

Author: bentsherman

modules/language-server/src/main/java/nextflow/lsp/NextflowLanguageServer.java

@@ -28,6 +28,7 @@
 import nextflow.lsp.file.PathUtils;
 import nextflow.lsp.util.JsonUtils;
 import nextflow.lsp.util.Logger;
+import nextflow.lsp.services.ErrorReportingMode;
 import nextflow.lsp.services.LanguageServerConfiguration;
 import nextflow.lsp.services.LanguageService;
 import nextflow.lsp.services.SemanticTokensVisitor;
@@ -444,24 +445,24 @@ public void didChangeConfiguration(DidChangeConfigurationParams params) {
         if( debug != null )
             Logger.setDebugEnabled(debug);
 
+        var errorReportingMode = errorReportingMode(params);
+        if( errorReportingMode != null && configuration.errorReportingMode() != errorReportingMode )
+            shouldInitialize = true;
         var excludePatterns = JsonUtils.getStringArray(params.getSettings(), "nextflow.files.exclude");
         if( !DefaultGroovyMethods.equals(configuration.excludePatterns(), excludePatterns) )
             shouldInitialize = true;
         var extendedCompletion = JsonUtils.getBoolean(params.getSettings(), "nextflow.completion.extended");
         var harshilAlignment = JsonUtils.getBoolean(params.getSettings(), "nextflow.formatting.harshilAlignment");
         var maheshForm = JsonUtils.getBoolean(params.getSettings(), "nextflow.formatting.maheshForm");
         var maxCompletionItems = JsonUtils.getInteger(params.getSettings(), "nextflow.completion.maxItems");
-        var paranoidWarnings = JsonUtils.getBoolean(params.getSettings(), "nextflow.paranoidWarnings");
-        if( paranoidWarnings != null && configuration.paranoidWarnings() != paranoidWarnings )
-            shouldInitialize = true;
 
         configuration = new LanguageServerConfiguration(
+            errorReportingMode != null ? errorReportingMode : configuration.errorReportingMode(),
             excludePatterns,
             extendedCompletion != null ? extendedCompletion : configuration.extendedCompletion(),
             harshilAlignment != null ? harshilAlignment : configuration.harshilAlignment(),
             maheshForm != null ? maheshForm : configuration.maheshForm(),
-            maxCompletionItems != null ? maxCompletionItems : configuration.maxCompletionItems(),
-            paranoidWarnings != null ? paranoidWarnings : configuration.paranoidWarnings()
+            maxCompletionItems != null ? maxCompletionItems : configuration.maxCompletionItems()
         );
 
         if( shouldInitialize ) {
@@ -487,6 +488,13 @@ public void didChangeConfiguration(DidChangeConfigurationParams params) {
         }
     }
 
+    private ErrorReportingMode errorReportingMode(DidChangeConfigurationParams params) {
+        var string = JsonUtils.getString(params.getSettings(), "nextflow.errorReportingMode");
+        if( string == null )
+            return null;
+        return ErrorReportingMode.valueOf(string.toUpperCase());
+    }
+
     private void progressCreate(String token) {
         client.createProgress(new WorkDoneProgressCreateParams(Either.forLeft(token)));
     }

modules/language-server/src/main/java/nextflow/lsp/services/ErrorReportingMode.java

@@ -0,0 +1,37 @@
+/*
+ * Copyright 2024-2025, Seqera Labs
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package nextflow.lsp.services;
+
+import nextflow.script.control.FutureWarning;
+import org.codehaus.groovy.control.messages.WarningMessage;
+import org.codehaus.groovy.syntax.SyntaxException;
+
+public enum ErrorReportingMode {
+    OFF,
+    ERRORS,
+    WARNINGS,
+    PARANOID;
+
+    public boolean isRelevant(SyntaxException error) {
+        return this != OFF;
+    }
+
+    public boolean isRelevant(WarningMessage warning) {
+        if( warning instanceof FutureWarning )
+            return this == PARANOID;
+        return compareTo(WARNINGS) >= 0;
+    }
+}

modules/language-server/src/main/java/nextflow/lsp/services/LanguageServerConfiguration.java

@@ -19,22 +19,22 @@
 import java.util.List;
 
 public record LanguageServerConfiguration(
+    ErrorReportingMode errorReportingMode,
     List<String> excludePatterns,
     boolean extendedCompletion,
     boolean harshilAlignment,
     boolean maheshForm,
-    int maxCompletionItems,
-    boolean paranoidWarnings
+    int maxCompletionItems
 ) {
 
     public static LanguageServerConfiguration defaults() {
         return new LanguageServerConfiguration(
+            ErrorReportingMode.WARNINGS,
             Collections.emptyList(),
             false,
             false,
             false,
-            100,
-            false
+            100
         );
     }
 }

modules/language-server/src/main/java/nextflow/lsp/services/LanguageService.java

@@ -41,7 +41,6 @@
 import nextflow.lsp.util.LanguageServerUtils;
 import nextflow.lsp.util.Logger;
 import nextflow.lsp.util.Positions;
-import nextflow.script.control.FutureWarning;
 import nextflow.script.control.RelatedInformationAware;
 import nextflow.script.formatter.FormattingOptions;
 import org.codehaus.groovy.runtime.DefaultGroovyMethods;
@@ -382,6 +381,9 @@ protected void publishDiagnostics(Set<URI> changedUris) {
         changedUris.forEach((uri) -> {
             var diagnostics = new ArrayList<Diagnostic>();
             for( var error : astCache.getErrors(uri) ) {
+                if( !configuration.errorReportingMode().isRelevant(error) )
+                    continue;
+
                 var message = error.getOriginalMessage();
                 var range = LanguageServerUtils.errorToRange(error);
                 if( range == null ) {
@@ -396,7 +398,7 @@ protected void publishDiagnostics(Set<URI> changedUris) {
             }
 
             for( var warning : astCache.getWarnings(uri) ) {
-                if( !configuration.paranoidWarnings() && warning instanceof FutureWarning )
+                if( !configuration.errorReportingMode().isRelevant(warning) )
                     continue;
 
                 var message = warning.getMessage();

modules/language-server/src/main/java/nextflow/lsp/util/JsonUtils.java

@@ -68,6 +68,18 @@ public static Integer getInteger(Object json, String path) {
         }
     }
 
+    public static String getString(Object json, String path) {
+        var value = getObjectPath(json, path);
+        if( value == null )
+            return null;
+        try {
+            return value.getAsString();
+        }
+        catch( ClassCastException e ) {
+            return null;
+        }
+    }
+
     public static String getString(Object json) {
         return json instanceof JsonPrimitive jp ? jp.getAsString() : null;
     }