Silent Exit

Author: Behemyth

cppython/build/prepare.py

@@ -78,12 +78,6 @@ def prepare(self) -> BuildPreparationResult:
 
         pyproject_data = self._load_pyproject()
 
-        # Check if CPPython is configured
-        tool_data = pyproject_data.get('tool', {})
-        if 'cppython' not in tool_data:
-            self.logger.info('CPPython: No [tool.cppython] configuration found, skipping preparation')
-            return BuildPreparationResult()
-
         # Get version from pyproject if available
         project_data = pyproject_data.get('project', {})
         version = project_data.get('version')

cppython/plugins/pdm/plugin.py

@@ -50,23 +50,21 @@ def on_post_install(self, project: Project, dry_run: bool, **_kwargs: Any) -> No
             _kwargs: Sink for unknown arguments
         """
         root = project.root.absolute()
+        pdm_pyproject = project.pyproject.open_for_read()
 
         # Attach configuration for CPPythonPlugin callbacks
         version = project.pyproject.metadata.get('version')
         verbosity = project.core.ui.verbosity
 
         project_configuration = ProjectConfiguration(project_root=root, verbosity=verbosity, version=version)
 
-        self.logger.info("CPPython: Entered 'on_post_install'")
-
-        if (pdm_pyproject := project.pyproject.open_for_read()) is None:
-            self.logger.info('CPPython: Project data was not available')
-            return
-
-        cppython_project = CPPythonProject(project_configuration, self, pdm_pyproject)
+        try:
+            cppython_project = CPPythonProject(project_configuration, self, pdm_pyproject)
 
-        if not dry_run:
-            cppython_project.install()
+            if not dry_run:
+                cppython_project.install()
+        except Exception:
+            self.logger.debug('CPPython: Error during post-install hook', exc_info=True)
 
 
 class CPPythonCommand(BaseCommand):

cppython/project.py

@@ -28,6 +28,11 @@ def __init__(
         self._interface = interface
         self.logger = logging.getLogger('cppython')
 
+        # Early exit: if no CPPython configuration table, do nothing silently
+        tool_data = pyproject_data.get('tool')
+        if not tool_data or not isinstance(tool_data, dict) or not tool_data.get('cppython'):
+            return
+
         builder = Builder(project_configuration, self.logger)
 
         self.logger.info('Initializing project')

pyproject.toml

@@ -88,6 +88,9 @@ select = [
 [tool.ruff.lint.pydocstyle]
 convention = "google"
 
+[tool.ruff.lint.per-file-ignores]
+"cppython/console/entry.py" = ["PT"] # CLI commands, not pytest tests
+
 [tool.ruff.format]
 docstring-code-format = true
 indent-style = "space"

tests/unit/test_project.py

@@ -4,6 +4,7 @@
 import tomllib
 from importlib import metadata
 from pathlib import Path
+from typing import Any
 
 import pytest
 from pytest_mock import MockerFixture
@@ -46,6 +47,31 @@ def test_self_construction(request: pytest.FixtureRequest) -> None:
         # Doesn't have the cppython table
         assert not project.enabled
 
+    @staticmethod
+    def test_missing_tool_table_raw_dict(tmp_path: Path, caplog: pytest.LogCaptureFixture) -> None:
+        """Constructing Project with a raw dict lacking tool.cppython should produce zero log output.
+
+        This simulates input from a host tool like PDM that passes raw pyproject data
+        rather than a model_dump() result.
+
+        Args:
+            tmp_path: Temporary directory for dummy data
+            caplog: Pytest fixture for capturing logs
+        """
+        project_configuration = ProjectConfiguration(project_root=tmp_path, version=None)
+        interface = MockInterface()
+
+        # Raw dict as PDM would provide — no tool table at all
+        raw_data: dict[str, Any] = {'project': {'name': 'some-other-project', 'version': '1.0.0'}}
+
+        with caplog.at_level(logging.DEBUG):
+            project = Project(project_configuration, interface, raw_data)
+
+        # Absolutely no log output for projects without CPPython configuration
+        assert len(caplog.records) == 0
+
+        assert not project.enabled
+
     @staticmethod
     def test_missing_tool_table(tmp_path: Path, caplog: pytest.LogCaptureFixture) -> None:
         """The project type should be constructable without the tool table