Remove option for skipping message verification in validator

This also adds a short description of the functionality in PyRFLX.

Ref. eng/recordflux/RecordFlux!1456

rflx/pyrflx/pyrflx.py

@@ -24,6 +24,17 @@ def __init__(
         checksum_functions: Optional[Mapping[StrID, Mapping[str, ChecksumFunction]]] = None,
         skip_message_verification: bool = False,
     ) -> None:
+        """
+        Initialize PyRFLX.
+
+        Arguments:
+        ---------
+        model: The RecordFlux model.
+        checksum_functions: A mapping of field identifiers to checksum functions.
+        skip_message_verification: Disable some runtime checks for messages to optimize performance.
+                                   This can lead to unexpected errors if an invalid field value is
+                                   set.
+        """
         self._packages: dict[str, Package] = {}
         messages: dict[ID, MessageValue] = {}
 

rflx/validator.py

@@ -26,7 +26,6 @@ def __init__(
         files: Iterable[Union[str, Path]],
         checksum_module: Optional[str] = None,
         cache: Optional[Cache] = None,
-        skip_message_verification: bool = False,
         split_disjunctions: bool = False,
     ):
         model = self._create_model(
@@ -57,7 +56,7 @@ def __init__(
             )
 
         try:
-            self._pyrflx = PyRFLX(model, checksum_functions, skip_message_verification)
+            self._pyrflx = PyRFLX(model, checksum_functions)
         except PyRFLXError as e:
             raise ValidationError(f"invalid checksum definition: {e}") from e
 

tests/unit/cli_test.py

@@ -28,7 +28,6 @@ def validator_mock(
     files: object,  # noqa: ARG001
     checksum_module: Optional[str] = None,  # noqa: ARG001
     cache: Optional[object] = None,  # noqa: ARG001
-    skip_message_verification: bool = False,  # noqa: ARG001
     split_disjunctions: bool = False,  # noqa: ARG001
 ) -> None:
     return None