Support "--show-error-end" from newer mypy's

kaste · kaste · commit db11a82f2f0c · 2023-02-06T16:21:30.000+01:00
We can't enable this here for everyone because it has been just
introduced in v0.981 (Sep 2022).  But we can support its output format.

Per spec, mypy might report `-1`s if it doesn't know a better value.
We convert them to `None`s as that is what SL expects.  Except for
`line` which is mandatory in SL land and hereby reported on the first
line of the buffer.
diff --git a/linter.py b/linter.py
@@ -52,7 +52,9 @@ class Mypy(PythonLinter):
     """Provides an interface to mypy."""
 
     regex = (
-        r'^(?P<filename>.+?):(?P<line>\d+):((?P<col>\d+):)?\s*'
+        r'^(?P<filename>.+?):'
+        r'(?P<line>\d+|-1):((?P<col>\d+|-1):)?'
+        r'((?P<end_line>\d+|-1):(?P<end_col>\d+|-1):)?\s*'
         r'(?P<error_type>[^:]+):\s(?P<message>.+?)(\s\s\[(?P<code>.+)\])?$'
     )
     line_col_base = (1, 1)
@@ -142,6 +144,14 @@ def find_errors(self, output):
                         previous['message'] += '\n{}'.format(error.message)
                         continue
 
+            # mypy might report `-1` for unknown values.
+            # Only `line` is mandatory within SublimeLinter
+            if error.match.group('line') == "-1":  # type: ignore[attr-defined]
+                error['line'] = 0
+            for group in ('col', 'end_line', 'end_col'):
+                if error.match.group(group) == "-1":  # type: ignore[attr-defined]
+                    error[group] = None
+
             errors.append(error)
         yield from errors
 
diff --git a/tests/test_regex.py b/tests/test_regex.py
@@ -41,6 +41,22 @@ def test_matches(self):
                 'col': None,
                 'message': '"dict" is not subscriptable, use "typing.Dict" instead'})
 
+        self.assertMatch(
+            'codespell_lib\\tests\\test_basic.py:518:5:518:13: error: Module has no attribute "mkfifo"  [attr-defined]', {
+                'line': 517,
+                'col': 4,
+                'end_line': 517,
+                'end_col': 12,
+            })
+
+        self.assertMatch(
+            'codespell_lib\\tests\\test_basic.py:-1:-1:-1:-1: error: Module has no attribute "mkfifo"  [attr-defined]', {
+                'line': 0,
+                'col': None,
+                'end_line': None,
+                'end_col': None,
+            })
+
     def test_tmp_files_that_have_no_file_extension(self):
         self.assertMatch(
             '/tmp/yoeai32h2:6:1: error: Cannot find module named \'PackageName.lib\'', {
