Update Pydantic Logging

Author: Behemyth

cppython/core/resolution.py

@@ -1,11 +1,12 @@
 """Data conversion routines"""
 
+import logging
 from pathlib import Path
 from typing import Any, cast
 
 from pydantic import BaseModel, DirectoryPath, ValidationError
 
-from cppython.core.exception import ConfigError, ConfigException
+from cppython.core.exception import ConfigException
 from cppython.core.plugin_schema.generator import Generator, GeneratorPluginGroupData
 from cppython.core.plugin_schema.provider import Provider, ProviderPluginGroupData
 from cppython.core.plugin_schema.scm import SCM, SCMPluginGroupData
@@ -257,7 +258,12 @@ def resolve_model[T: BaseModel](model: type[T], data: dict[str, Any]) -> T:
         # BaseModel is setup to ignore extra fields
         return model(**data)
     except ValidationError as e:
-        new_errors: list[ConfigError] = []
-        for error in e.errors():
-            new_errors.append(ConfigError(message=error['msg']))
-        raise ConfigException('The input project failed', new_errors) from e
+        # Log the raw ValidationError for debugging
+        logging.getLogger('cppython').debug('ValidationError details: %s', e.errors())
+
+        if e.errors():
+            formatted_errors = '\n'.join(f"Field '{error['loc'][0]}': {error['msg']}" for error in e.errors())
+        else:
+            formatted_errors = 'An unknown validation error occurred.'
+
+        raise ConfigException(f'The input project failed validation:\n{formatted_errors}', []) from e

cppython/project.py

@@ -9,6 +9,7 @@
 from cppython.core.resolution import resolve_model
 from cppython.core.schema import Interface, ProjectConfiguration, PyProject
 from cppython.schema import API
+from cppython.utility.exception import ProcessError
 
 
 class Project(API):
@@ -29,8 +30,9 @@ def __init__(
         try:
             pyproject = resolve_model(PyProject, pyproject_data)
         except ConfigException as error:
-            self.logger.error(error, exc_info=True)
-            raise SystemExit(1) from None
+            # Log the exception message explicitly
+            self.logger.error('Configuration error:\n%s', error, exc_info=False)
+            raise SystemExit('Error: Invalid configuration. Please check your pyproject.toml.') from None
 
         if not pyproject.tool or not pyproject.tool.cppython:
             self.logger.info("The pyproject.toml file doesn't contain the `tool.cppython` table")
@@ -69,9 +71,12 @@ def install(self) -> None:
 
         try:
             self._data.plugins.provider.install()
+        except ProcessError as error:
+            self.logger.error('Installation failed: %s', error.error)
+            raise SystemExit('Error: Provider installation failed. Please check the logs.') from None
         except Exception as exception:
-            self.logger.error('Provider %s failed to install', self._data.plugins.provider.name())
-            raise exception
+            self.logger.error('Unexpected error during installation: %s', str(exception))
+            raise SystemExit('Error: An unexpected error occurred during installation.') from None
 
         self._data.sync()
 
@@ -93,8 +98,11 @@ def update(self) -> None:
 
         try:
             self._data.plugins.provider.update()
+        except ProcessError as error:
+            self.logger.error('Update failed: %s', error.error)
+            raise SystemExit('Error: Provider update failed. Please check the logs.') from None
         except Exception as exception:
-            self.logger.error('Provider %s failed to update', self._data.plugins.provider.name())
-            raise exception
+            self.logger.error('Unexpected error during update: %s', str(exception))
+            raise SystemExit('Error: An unexpected error occurred during update.') from None
 
         self._data.sync()