Fix #1096: print runtime errors in context of failing subproject

Replace the pattern of collecting RuntimeErrors across all subprojects
and printing them at the end with immediate in-context logging via
`logger.print_warning_line(project.name, str(exc))` at the point of
failure. This ensures errors like "svn not available on system" appear
next to the subproject they belong to instead of at the end of the run.

The final `raise RuntimeError()` (no message) still produces a non-zero
exit code without double-printing.

Also bumps version to 0.14.0 and updates CHANGELOG, feature tests.

https://claude.ai/code/session_01RTgPqrX37jFkK843dq65Bu

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: spoorcc

CHANGELOG.rst

@@ -10,6 +10,7 @@ Release 0.14.0 (unreleased)
 * Allow ``dfetch freeze`` to accept project names to freeze only specific projects (#1063)
 * Edit manifest in-place when freezing inside a git or SVN superproject, preserving comments and layout (#1063)
 * Add new ``remove`` command to remove projects from manifest and disk (#26)
+* Print runtime errors (e.g. ``svn not available on system``) directly in context of the failing subproject instead of collecting and showing them at the end (#1096)
 
 Release 0.13.0 (released 2026-03-30)
 ====================================

dfetch/__init__.py

@@ -1,5 +1,5 @@
 """Dfetch."""
 
-__version__ = "0.13.0"
+__version__ = "0.14.0"
 
 DEFAULT_MANIFEST_NAME: str = "dfetch.yaml"

dfetch/commands/check.py

@@ -45,7 +45,7 @@
 from dfetch.reporting.check.reporter import CheckReporter
 from dfetch.reporting.check.sarif_reporter import SarifReporter
 from dfetch.reporting.check.stdout_reporter import CheckStdoutReporter
-from dfetch.util.util import catch_runtime_exceptions, in_directory
+from dfetch.util.util import in_directory
 
 logger = get_logger(__name__)
 
@@ -102,13 +102,16 @@ def __call__(self, args: argparse.Namespace) -> None:
         reporters = self._get_reporters(args, superproject.manifest)
 
         with in_directory(superproject.root_directory):
-            exceptions: list[str] = []
+            had_errors: bool = False
             for project in superproject.manifest.selected_projects(args.projects):
-                with catch_runtime_exceptions(exceptions) as exceptions:
+                try:
                     dfetch.project.create_sub_project(project).check_for_update(
                         reporters,
                         files_to_ignore=superproject.ignored_files(project.destination),
                     )
+                except RuntimeError as exc:
+                    logger.print_warning_line(project.name, str(exc))
+                    had_errors = True
 
                 if not args.no_recommendations and os.path.isdir(project.destination):
                     with in_directory(project.destination):
@@ -117,8 +120,8 @@ def __call__(self, args: argparse.Namespace) -> None:
             for reporter in reporters:
                 reporter.dump_to_file()
 
-        if exceptions:
-            raise RuntimeError("\n".join(exceptions))
+        if had_errors:
+            raise RuntimeError()
 
     @staticmethod
     def _get_reporters(

dfetch/commands/format_patch.py

@@ -38,7 +38,6 @@
 from dfetch.project.subproject import SubProject
 from dfetch.project.svnsubproject import SvnSubProject
 from dfetch.util.util import (
-    catch_runtime_exceptions,
     check_no_path_traversal,
     in_directory,
 )
@@ -84,7 +83,7 @@ def __call__(self, args: argparse.Namespace) -> None:
         """Perform the format patch."""
         superproject = create_super_project()
 
-        exceptions: list[str] = []
+        had_errors: bool = False
 
         output_dir_path = pathlib.Path(args.output_directory).resolve()
 
@@ -94,7 +93,7 @@ def __call__(self, args: argparse.Namespace) -> None:
 
         with in_directory(superproject.root_directory):
             for project in superproject.manifest.selected_projects(args.projects):
-                with catch_runtime_exceptions(exceptions) as exceptions:
+                try:
                     subproject = dfetch.project.create_sub_project(project)
 
                     # Check if the project has a patch, maybe suggest creating one?
@@ -139,9 +138,12 @@ def __call__(self, args: argparse.Namespace) -> None:
                             project.name,
                             f"formatted patch written to {output_patch_file.relative_to(os.getcwd())}",
                         )
+                except RuntimeError as exc:
+                    logger.print_warning_line(project.name, str(exc))
+                    had_errors = True
 
-        if exceptions:
-            raise RuntimeError("\n".join(exceptions))
+        if had_errors:
+            raise RuntimeError()
 
 
 def _determine_target_patch_type(subproject: SubProject) -> PatchType:

dfetch/commands/freeze.py

@@ -68,7 +68,7 @@
 from dfetch.log import get_logger
 from dfetch.project import create_super_project
 from dfetch.project.superproject import NoVcsSuperProject
-from dfetch.util.util import catch_runtime_exceptions, in_directory
+from dfetch.util.util import in_directory
 
 logger = get_logger(__name__)
 
@@ -99,7 +99,6 @@ def __call__(self, args: argparse.Namespace) -> None:
         superproject = create_super_project()
         make_backup = isinstance(superproject, NoVcsSuperProject)
 
-        exceptions: list[str] = []
         manifest_updated = False
 
         with in_directory(superproject.root_directory):
@@ -110,7 +109,7 @@ def __call__(self, args: argparse.Namespace) -> None:
                 shutil.copyfile(manifest_path, manifest_path + ".backup")
 
             for project in projects_to_freeze:
-                with catch_runtime_exceptions(exceptions) as exceptions:
+                try:
                     sub_project = dfetch.project.create_sub_project(project)
                     on_disk_version = sub_project.on_disk_version()
 
@@ -133,6 +132,8 @@ def __call__(self, args: argparse.Namespace) -> None:
                         )
                         superproject.manifest.update_project_version(project)
                         manifest_updated = True
+                except RuntimeError as exc:
+                    logger.print_warning_line(project.name, str(exc))
 
             if manifest_updated:
                 superproject.manifest.dump()

dfetch/commands/update.py

@@ -28,7 +28,6 @@
 from dfetch.log import get_logger
 from dfetch.project import create_super_project
 from dfetch.util.util import (
-    catch_runtime_exceptions,
     check_no_path_traversal,
     in_directory,
 )
@@ -78,20 +77,25 @@ def __call__(self, args: argparse.Namespace) -> None:
         """Perform the update."""
         superproject = create_super_project()
 
-        exceptions: list[str] = []
+        had_errors: bool = False
         destinations: list[str] = [
             os.path.realpath(project.destination)
             for project in superproject.manifest.projects
         ]
         with in_directory(superproject.root_directory):
             for project in superproject.manifest.selected_projects(args.projects):
-                with catch_runtime_exceptions(exceptions) as exceptions:
+                try:
                     self._check_destination(project, destinations)
-                    destination = project.destination
+                except RuntimeError:
+                    had_errors = True
+                    continue
 
-                    def _ignored(dst: str = destination) -> list[str]:
-                        return list(superproject.ignored_files(dst))
+                destination = project.destination
 
+                def _ignored(dst: str = destination) -> list[str]:
+                    return list(superproject.ignored_files(dst))
+
+                try:
                     dfetch.project.create_sub_project(project).update(
                         force=args.force,
                         ignored_files_callback=_ignored,
@@ -102,9 +106,12 @@ def _ignored(dst: str = destination) -> list[str]:
                     ):
                         with in_directory(project.destination):
                             check_sub_manifests(superproject.manifest, project)
+                except RuntimeError as exc:
+                    logger.print_warning_line(project.name, str(exc))
+                    had_errors = True
 
-        if exceptions:
-            raise RuntimeError("\n".join(exceptions))
+        if had_errors:
+            raise RuntimeError()
 
     @staticmethod
     def _check_destination(

dfetch/util/util.py

@@ -176,18 +176,6 @@ def in_directory(path: str | Path) -> Generator[str, None, None]:
         os.chdir(pwd)
 
 
-@contextmanager
-def catch_runtime_exceptions(
-    exc_list: list[str] | None = None,
-) -> Generator[list[str], None, None]:
-    """Catch all runtime errors and add it to list of strings."""
-    exc_list = exc_list or []
-    try:
-        yield exc_list
-    except RuntimeError as exc:
-        exc_list += [str(exc)]
-
-
 @contextmanager
 def prefix_runtime_exceptions(
     prefix: str,

features/check-archive.feature

@@ -24,7 +24,7 @@ Feature: Checking dependencies from an archive
         When I run "dfetch check" in MyProject
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               SomeProject:
               > up-to-date (some-remote-server/SomeProject.tar.gz)
             """
@@ -48,7 +48,7 @@ Feature: Checking dependencies from an archive
         When I run "dfetch check" in MyProject
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               SomeProject:
               > up-to-date (sha256:<archive-sha256>)
             """
@@ -69,7 +69,7 @@ Feature: Checking dependencies from an archive
         When I run "dfetch check" in MyProject
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               SomeProject:
               > wanted (some-remote-server/SomeProject.tar.gz), available (some-remote-server/SomeProject.tar.gz)
             """
@@ -87,7 +87,7 @@ Feature: Checking dependencies from an archive
         When I run "dfetch check"
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               non-existent-archive:
               > wanted (https://dfetch.invalid/does-not-exist.tar.gz), but not available at the upstream.
             """
@@ -113,7 +113,7 @@ Feature: Checking dependencies from an archive
         When I run "dfetch check SomeProject" in MyProject
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               SomeProject:
               > up-to-date (some-remote-server/SomeProject.tar.gz)
             """
@@ -136,7 +136,7 @@ Feature: Checking dependencies from an archive
         When I run "dfetch check SomeProject" in MyProject
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               SomeProject:
               > Local changes were detected, please generate a patch using 'dfetch diff SomeProject' and add it to your manifest using 'patch:'. Alternatively overwrite the local changes with 'dfetch update --force SomeProject'
               > up-to-date (some-remote-server/SomeProject.tar.gz)

features/check-git-repo.feature

@@ -26,7 +26,7 @@ Feature: Checking dependencies from a git repository
         When I run "dfetch check"
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               ext/test-repo-rev-only:
               > wanted (e1fda19a57b873eb8e6ae37780594cbb77b70f1a), available (e1fda19a57b873eb8e6ae37780594cbb77b70f1a)
               ext/test-rev-and-branch:
@@ -52,7 +52,7 @@ Feature: Checking dependencies from a git repository
         When I run "dfetch check"
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               ext/test-repo-tag-v1:
               > wanted (v1), available (v2.0)
             """
@@ -82,7 +82,7 @@ Feature: Checking dependencies from a git repository
         When I run "dfetch check"
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               ext/test-repo-rev-only:
               > up-to-date (e1fda19a57b873eb8e6ae37780594cbb77b70f1a)
               ext/test-rev-and-branch:
@@ -116,7 +116,7 @@ Feature: Checking dependencies from a git repository
         And I run "dfetch check"
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               ext/test-repo-tag:
               > wanted (v2.0), current (v1), available (v2.0)
             """
@@ -138,7 +138,7 @@ Feature: Checking dependencies from a git repository
         When I run "dfetch check SomeProject"
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               SomeProject:
               > Local changes were detected, please generate a patch using 'dfetch diff SomeProject' and add it to your manifest using 'patch:'. Alternatively overwrite the local changes with 'dfetch update --force SomeProject'
               > up-to-date (master - 90be799b58b10971691715bdc751fbe5237848a0)
@@ -159,7 +159,7 @@ Feature: Checking dependencies from a git repository
         When I run "dfetch check SomeProject"
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               SomeProject:
               > up-to-date (master - 90be799b58b10971691715bdc751fbe5237848a0)
             """
@@ -179,7 +179,7 @@ Feature: Checking dependencies from a git repository
         When I run "dfetch check"
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
             >>>git ls-remote --heads https://giiiiiidhub.com/i-do-not-exist/broken<<< failed!
             'https://giiiiiidhub.com/i-do-not-exist/broken' is not a valid URL or unreachable:
             fatal: unable to access 'https://giiiiiidhub.com/i-do-not-exist/broken/': Could not resolve host: giiiiiidhub.com
@@ -208,7 +208,7 @@ Feature: Checking dependencies from a git repository
         When I run "dfetch check"
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               SomeProjectMissingTag:
               > wanted (i-dont-exist), but not available at the upstream.
               SomeProjectNonExistentBranch:
@@ -230,7 +230,7 @@ Feature: Checking dependencies from a git repository
         When I run "dfetch check"
         Then the output starts with:
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
             >>>git ls-remote --heads --tags https://github.com/dfetch-org/test-repo-private.git<<< returned 128:
             """
 
@@ -247,6 +247,6 @@ Feature: Checking dependencies from a git repository
         When I run "dfetch check"
         Then the output starts with:
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
             >>>git ls-remote --heads --tags git@github.com:dfetch-org/test-repo-private.git<<< returned 128:
             """

features/check-specific-projects.feature

@@ -27,7 +27,7 @@ Feature: Checking specific projects
         When I run "dfetch check ext/test-rev-and-branch"
         Then the output shows
             """
-            Dfetch (0.13.0)
+            Dfetch (0.14.0)
               ext/test-rev-and-branch:
               > wanted (main - 8df389d0524863b85f484f15a91c5f2c40aefda1), available (main - e1fda19a57b873eb8e6ae37780594cbb77b70f1a)
             """