improve handling of validation errors

Author: landrews-amd

nodescraper/cli/inputargtypes.py

@@ -104,7 +104,7 @@ def process_file_arg(self, file_path: str) -> TModelType:
             return self.model(**data)
         except ValidationError as e:
             raise argparse.ArgumentTypeError(
-                f"Validation errors when processing {file_path}: {e.errors()}"
+                f"Validation errors when processing {file_path}: {e.errors(include_url=False)}"
             ) from e
 
 

nodescraper/interfaces/dataanalyzertask.py

@@ -76,13 +76,12 @@ def wrapper(
                     if not analyze_arg_model:
                         raise ValueError("No model defined for analysis args")
                     args = analyze_arg_model(**args)  # type: ignore
-
                 func(analyzer, data, args)
             except ValidationError as exception:
                 analyzer._log_event(
                     category=EventCategory.RUNTIME,
                     description="Validation error during analysis",
-                    data=get_exception_traceback(exception),
+                    data={"errors": exception.errors(include_url=False)},
                     priority=EventPriority.CRITICAL,
                     console_log=True,
                 )

nodescraper/interfaces/datacollectortask.py

@@ -41,7 +41,7 @@
 from nodescraper.interfaces.task import SystemCompatibilityError, Task
 from nodescraper.models import DataModel, SystemInfo, TaskResult
 from nodescraper.typeutils import TypeUtils
-from nodescraper.utils import get_exception_details, get_exception_traceback
+from nodescraper.utils import get_exception_traceback
 
 from .connectionmanager import TConnection
 from .taskresulthook import TaskResultHook
@@ -76,7 +76,7 @@ def wrapper(
                 collector._log_event(
                     category=EventCategory.RUNTIME,
                     description="Pydantic validation error",
-                    data=get_exception_details(exception),
+                    data={"errors": exception.errors(include_url=False)},
                     priority=EventPriority.CRITICAL,
                     console_log=True,
                 )