[JsonReporter] Create a base JsonReporter

Author: Pierre-Sassoulas

pylint/reporters/json_reporter.py

@@ -7,19 +7,39 @@
 from __future__ import annotations
 
 import json
-from typing import TYPE_CHECKING
+from typing import TYPE_CHECKING, Any, Optional
 
-from pylint.interfaces import UNDEFINED
+from pylint.interfaces import UNDEFINED, Confidence
 from pylint.message import Message
 from pylint.reporters.base_reporter import BaseReporter
 from pylint.typing import MessageLocationTuple
 
 if TYPE_CHECKING:
+    from typing import TypedDict
+
     from pylint.lint.pylinter import PyLinter
     from pylint.reporters.ureports.nodes import Section
 
+    OldJsonExport = TypedDict(
+        "OldJsonExport",
+        {
+            "type": str,
+            "module": str,
+            "obj": str,
+            "line": int,
+            "column": int,
+            "endLine": Optional[int],
+            "endColumn": Optional[int],
+            "path": str,
+            "symbol": str,
+            "message": str,
+            # "message-id" is not a valid name, so we have to define the TypedDict like this
+            "message-id": str,
+        },
+    )
+
 
-class JSONReporter(BaseReporter):
+class BaseJSONReporter(BaseReporter):
     """Report messages and layouts in JSON."""
 
     name = "json"
@@ -30,9 +50,33 @@ def display_messages(self, layout: Section | None) -> None:
         json_dumpable = [self.serialize(message) for message in self.messages]
         print(json.dumps(json_dumpable, indent=4), file=self.out)
 
+    def display_reports(self, layout: Section) -> None:
+        """Don't do anything in this reporter."""
+
+    def _display(self, layout: Section) -> None:
+        """Do nothing."""
+
+    @staticmethod
+    def serialize(message: Message) -> Any:
+        raise NotImplementedError
+
+    @staticmethod
+    def deserialize(message_as_json: Any) -> Message:
+        raise NotImplementedError
+
+
+class JSONReporter(BaseJSONReporter):
+
+    """
+    TODO: 3.0: Remove this JSONReporter in favor of the new one handling abs-path
+    and confidence.
+
+    TODO: 2.15: Add a new JSONReporter handling abs-path, confidence and scores.
+    (Ultimately all other breaking change related to json for 3.0).
+    """
+
     @staticmethod
-    def serialize(message: Message) -> dict[str, int | str | None]:
-        # TODO: 3.0: Add abs-path and confidence or a new JSONReporter
+    def serialize(message: Message) -> OldJsonExport:
         return {
             "type": message.category,
             "module": message.module,
@@ -48,13 +92,13 @@ def serialize(message: Message) -> dict[str, int | str | None]:
         }
 
     @staticmethod
-    def deserialize(message_as_json: dict) -> Message:
+    def deserialize(message_as_json: OldJsonExport) -> Message:
         return Message(
             msg_id=message_as_json["message-id"],
             symbol=message_as_json["symbol"],
             msg=message_as_json["message"],
             location=MessageLocationTuple(
-                # TODO: 3.0: Add abs-path and confidence or a new JSONReporter
+                # TODO: 3.0: Add abs-path and confidence in a new JSONReporter
                 abspath=message_as_json["path"],
                 path=message_as_json["path"],
                 module=message_as_json["module"],
@@ -64,16 +108,10 @@ def deserialize(message_as_json: dict) -> Message:
                 end_line=message_as_json["endLine"],
                 end_column=message_as_json["endColumn"],
             ),
-            # TODO: 3.0: Make confidence available or a new JSONReporter
+            # TODO: 3.0: Make confidence available in a new JSONReporter
             confidence=UNDEFINED,
         )
 
-    def display_reports(self, layout: Section) -> None:
-        """Don't do anything in this reporter."""
-
-    def _display(self, layout: Section) -> None:
-        """Do nothing."""
-
 
 def register(linter: PyLinter) -> None:
     linter.register_reporter(JSONReporter)