refactor: declare rule-id as class variable (#220)

Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>

Author: ktro2828

t4_devkit/sanity/checker.py

@@ -35,13 +35,11 @@ def is_error(self) -> bool:
 class Checker(ABC):
     """Base class for sanity checkers."""
 
+    id: RuleID
     name: RuleName
     description: str
     severity: Severity
 
-    def __init__(self, id: RuleID) -> None:
-        self.id = id
-
     def __call__(self, context: SanityContext) -> Report:
         match self.can_skip(context):
             case Some(skip):

t4_devkit/sanity/format/base.py

@@ -20,6 +20,7 @@ class FieldTypeChecker(Checker):
     """Base class for format checkers.
 
     Attributes:
+        id (RuleID): The ID of the rule.
         name (RuleName): The name of the rule.
         severity (Severity): The severity of the rule.
         description (str): The description of the rule.

t4_devkit/sanity/format/fmt001.py

@@ -9,10 +9,11 @@
 __all__ = ["FMT001"]
 
 
-@CHECKERS.register(RuleID("FMT001"))
+@CHECKERS.register()
 class FMT001(FieldTypeChecker):
     """A checker of FMT001."""
 
+    id = RuleID("FMT001")
     name = RuleName("attribute-field")
     severity = Severity.ERROR
     description = "All types of 'Attribute' fields are valid."

t4_devkit/sanity/format/fmt002.py

@@ -9,10 +9,11 @@
 __all__ = ["FMT002"]
 
 
-@CHECKERS.register(RuleID("FMT002"))
+@CHECKERS.register()
 class FMT002(FieldTypeChecker):
     """A checker of FMT002."""
 
+    id = RuleID("FMT002")
     name = RuleName("calibrated-sensor-field")
     severity = Severity.ERROR
     description = "All types of 'CalibratedSensor' fields are valid."

t4_devkit/sanity/format/fmt003.py

@@ -9,10 +9,11 @@
 __all__ = ["FMT003"]
 
 
-@CHECKERS.register(RuleID("FMT003"))
+@CHECKERS.register()
 class FMT003(FieldTypeChecker):
     """A checker of FMT003."""
 
+    id = RuleID("FMT003")
     name = RuleName("category-field")
     severity = Severity.ERROR
     description = "All types of 'Category' fields are valid."

t4_devkit/sanity/format/fmt004.py

@@ -9,10 +9,11 @@
 __all__ = ["FMT004"]
 
 
-@CHECKERS.register(RuleID("FMT004"))
+@CHECKERS.register()
 class FMT004(FieldTypeChecker):
     """A checker of FMT004."""
 
+    id = RuleID("FMT004")
     name = RuleName("ego-pose-field")
     severity = Severity.ERROR
     description = "All types of 'EgoPose' fields are valid."

t4_devkit/sanity/format/fmt005.py

@@ -9,10 +9,11 @@
 __all__ = ["FMT005"]
 
 
-@CHECKERS.register(RuleID("FMT005"))
+@CHECKERS.register()
 class FMT005(FieldTypeChecker):
     """A checker of FMT005."""
 
+    id = RuleID("FMT005")
     name = RuleName("instance-field")
     severity = Severity.ERROR
     description = "All types of 'Instance' fields are valid."

t4_devkit/sanity/format/fmt006.py

@@ -9,10 +9,11 @@
 __all__ = ["FMT006"]
 
 
-@CHECKERS.register(RuleID("FMT006"))
+@CHECKERS.register()
 class FMT006(FieldTypeChecker):
     """A checker of FMT006."""
 
+    id = RuleID("FMT006")
     name = RuleName("log-field")
     severity = Severity.ERROR
     description = "All types of 'Log' fields are valid."

t4_devkit/sanity/format/fmt007.py

@@ -9,10 +9,11 @@
 __all__ = ["FMT007"]
 
 
-@CHECKERS.register(RuleID("FMT007"))
+@CHECKERS.register()
 class FMT007(FieldTypeChecker):
     """A checker of FMT007."""
 
+    id = RuleID("FMT007")
     name = RuleName("map-field")
     severity = Severity.ERROR
     description = "All types of 'Map' fields are valid."

t4_devkit/sanity/format/fmt008.py

@@ -9,10 +9,11 @@
 __all__ = ["FMT008"]
 
 
-@CHECKERS.register(RuleID("FMT008"))
+@CHECKERS.register()
 class FMT008(FieldTypeChecker):
     """A checker of FMT008."""
 
+    id = RuleID("FMT008")
     name = RuleName("sample-field")
     severity = Severity.ERROR
     description = "All types of 'Sample' fields are valid."