Add pyright analysis support

Author: fabioz

.github/workflows/tests-pydev.yml

@@ -12,10 +12,10 @@ jobs:
           submodules: recursive
 
       - name: Set up JDK 17
-        uses: actions/setup-java@v2
+        uses: actions/setup-java@v5
         with:
           java-version: '17'
-          distribution: 'adopt'
+          distribution: 'temurin'
           cache: 'maven'
 
       - name: Installed Maven version

plugins/com.python.pydev.analysis/META-INF/MANIFEST.MF

@@ -28,6 +28,7 @@ Export-Package: com.python.pydev.analysis,
  com.python.pydev.analysis.mypy,
  com.python.pydev.analysis.pep8,
  com.python.pydev.analysis.pylint,
+ com.python.pydev.analysis.pyright,
  com.python.pydev.analysis.refactoring.changes,
  com.python.pydev.analysis.refactoring.quick_fixes,
  com.python.pydev.analysis.refactoring.refactorer,

plugins/com.python.pydev.analysis/src/com/python/pydev/analysis/additionalinfo/builders/AnalysisBuilderRunnable.java

@@ -54,6 +54,9 @@
 import com.python.pydev.analysis.ruff.OnlyRemoveMarkersRuffVisitor;
 import com.python.pydev.analysis.ruff.RuffVisitor;
 import com.python.pydev.analysis.ruff.RuffVisitorFactory;
+import com.python.pydev.analysis.pyright.OnlyRemoveMarkersPyrightVisitor;
+import com.python.pydev.analysis.pyright.PyrightVisitor;
+import com.python.pydev.analysis.pyright.PyrightVisitorFactory;
 
 /**
  * This class is used to do analysis on a thread, so that if an analysis is asked for some analysis that
@@ -80,6 +83,7 @@ public class AnalysisBuilderRunnable extends AbstractAnalysisBuilderRunnable {
     private IExternalCodeAnalysisVisitor mypyVisitor;
     private IExternalCodeAnalysisVisitor flake8Visitor;
     private IExternalCodeAnalysisVisitor ruffVisitor;
+    private IExternalCodeAnalysisVisitor pyrightVisitor;
 
     private boolean onlyRecreateCtxInsensitiveInfo;
 
@@ -131,18 +135,21 @@ private static boolean isHierarchicallyDerived(IResource curr) {
                     this.flake8Visitor = visitor;
                 } else if (visitor instanceof OnlyRemoveMarkersRuffVisitor || visitor instanceof RuffVisitor) {
                     this.ruffVisitor = visitor;
+                } else if (visitor instanceof OnlyRemoveMarkersPyrightVisitor || visitor instanceof PyrightVisitor) {
+                    this.pyrightVisitor = visitor;
                 }
             }
-            if (pyLintVisitor == null || mypyVisitor == null || flake8Visitor == null || ruffVisitor == null) {
+            if (pyLintVisitor == null || mypyVisitor == null || flake8Visitor == null || ruffVisitor == null || pyrightVisitor == null) {
                 throw new AssertionError("All visitor types must be passed.");
             }
         } else {
             this.pyLintVisitor = PyLintVisitorFactory.create(resource, document, module, internalCancelMonitor);
             this.mypyVisitor = MypyVisitorFactory.create(resource, document, module, internalCancelMonitor);
             this.flake8Visitor = Flake8VisitorFactory.create(resource, document, module, internalCancelMonitor);
             this.ruffVisitor = RuffVisitorFactory.create(resource, document, module, internalCancelMonitor);
+            this.pyrightVisitor = PyrightVisitorFactory.create(resource, document, module, internalCancelMonitor);
             this.allVisitors = new IExternalCodeAnalysisVisitor[] { this.pyLintVisitor, this.mypyVisitor,
-                    this.flake8Visitor, this.ruffVisitor };
+                    this.flake8Visitor, this.ruffVisitor, this.pyrightVisitor };
         }
 
         for (IExternalCodeAnalysisVisitor visitor : allVisitors) {

plugins/com.python.pydev.analysis/src/com/python/pydev/analysis/additionalinfo/builders/AnalysisRunner.java

@@ -49,6 +49,7 @@ public class AnalysisRunner {
     public static final String MYPY_PROBLEM_MARKER = IMiscConstants.MYPY_PROBLEM_MARKER;
     public static final String FLAKE8_PROBLEM_MARKER = IMiscConstants.FLAKE8_PROBLEM_MARKER;
     public static final String RUFF_PROBLEM_MARKER = IMiscConstants.RUFF_PROBLEM_MARKER;
+    public static final String PYRIGHT_PROBLEM_MARKER = IMiscConstants.PYRIGHT_PROBLEM_MARKER;
 
     /**
      * do we want to debug this class?
@@ -137,6 +138,14 @@ public static void deleteMarkers(IResource resource, boolean onlyPydevAnalysisMa
         } catch (CoreException e3) {
             Log.log(e3);
         }
+
+        try {
+            if (resource != null) {
+                resource.deleteMarkers(PYRIGHT_PROBLEM_MARKER, false, IResource.DEPTH_ZERO);
+            }
+        } catch (CoreException e3) {
+            Log.log(e3);
+        }
     }
 
     /**

plugins/com.python.pydev.analysis/src/com/python/pydev/analysis/pyright/OnlyRemoveMarkersPyrightVisitor.java

@@ -0,0 +1,33 @@
+/**
+ * Copyright (c) 2025 Brainwy Software Ltda. All Rights Reserved.
+ * Licensed under the terms of the Eclipse Public License (EPL).
+ * Please see the license.txt included with this distribution for details.
+ * Any modifications to this file must keep this entire header intact.
+ */
+package com.python.pydev.analysis.pyright;
+
+import org.eclipse.core.resources.IResource;
+import org.python.pydev.shared_core.IMiscConstants;
+
+import com.python.pydev.analysis.external.AbstractExternalCodeAnalysisOnlyRemoveMarkersVisitor;
+
+public class OnlyRemoveMarkersPyrightVisitor extends AbstractExternalCodeAnalysisOnlyRemoveMarkersVisitor {
+
+    public OnlyRemoveMarkersPyrightVisitor(IResource resource) {
+        super(resource);
+    }
+
+    public static final String PYRIGHT_PROBLEM_MARKER = IMiscConstants.PYRIGHT_PROBLEM_MARKER;
+    public static final String PYRIGHT_MESSAGE_ID = IMiscConstants.PYRIGHT_MESSAGE_ID;
+
+    @Override
+    public String getProblemMarkerId() {
+        return PYRIGHT_PROBLEM_MARKER;
+    }
+
+    @Override
+    public String getMessageId() {
+        return PYRIGHT_MESSAGE_ID;
+    }
+
+}