Log subworkflow dependencies on install (#3871)

Mirror remove.py by logging installed components and their dependencies
during subworkflow install, and add a test for the dependency log output.

Author: xorrito624

CHANGELOG.md

@@ -28,6 +28,8 @@
 
 ### Subworkflows
 
+- Log subworkflow dependencies on install ([#4357](https://github.com/nf-core/tools/pull/4357))
+
 ### Template
 
 - New pre-commit hooks blocking large files and merge markers in pipeline template ([#3935](https://github.com/nf-core/tools/pull/3935))

nf_core/components/install.py

@@ -52,7 +52,11 @@ def __init__(
         else:
             self.installed_by = [self.component_type]
 
-    def install(self, component: str | dict[str, str], silent: bool = False) -> bool:
+    def install(
+        self,
+        component: str | dict[str, str],
+        silent: bool = False,
+    ) -> bool:
         if isinstance(component, dict):
             # Override modules_repo when the component to install is a dependency from a subworkflow.
             remote_url = component.get("git_remote", self.current_remote.remote_url)
@@ -167,6 +171,9 @@ def install(self, component: str | dict[str, str], silent: bool = False) -> bool
             self.component_type, self.modules_repo, component, version, self.installed_by, install_track
         )
 
+        installed_components = []
+        installed_components.append(component)
+
         if self.component_type == "subworkflows":
             # Under --skip-deps, don't propagate --force to transitive deps so
             # already-installed ones keep their pinned SHAs and just have
@@ -175,11 +182,17 @@ def install(self, component: str | dict[str, str], silent: bool = False) -> bool
                 original_force = self.force
                 self.force = False
                 try:
-                    self.install_included_components(component_dir)
+                    self.install_included_components(component_dir, installed_components)
                 finally:
                     self.force = original_force
             else:
-                self.install_included_components(component_dir)
+                self.install_included_components(component_dir, installed_components)
+
+        dependencies = set(installed_components) - {component}
+        if dependencies:
+            log.info(f"Installed files for '{component}' and its dependencies '{', '.join(sorted(dependencies))}'.")
+        else:
+            log.info(f"Installed files for '{component}'.")
 
         # Update container configs for the installed module. Subworkflows have no container entries of their
         # own; their included modules each trigger this when installed above.
@@ -207,27 +220,40 @@ def install(self, component: str | dict[str, str], silent: bool = False) -> bool
                     Console().print(
                         Syntax(f"includeConfig '{subworkflow_config}'", "groovy", theme="ansi_dark", padding=1)
                     )
+
         return True
 
-    def install_included_components(self, subworkflow_dir):
+    def install_included_components(self, subworkflow_dir, installed_components: list[str] | None = None):
         """
         Install included modules and subworkflows
         """
+        if installed_components is None:
+            installed_components = []
         ini_modules_repo = self.modules_repo
         modules_to_install, subworkflows_to_install = get_components_to_install(subworkflow_dir)
-        for s_install in subworkflows_to_install:
+        for subworkflow_to_install in subworkflows_to_install:
             original_installed = self.installed_by
             self.installed_by = [Path(subworkflow_dir).parts[-1]]
-            self.install(s_install, silent=True)
+            dependency_installed = self.install(subworkflow_to_install, silent=True)
             self.installed_by = original_installed
-        for m_install in modules_to_install:
+            if dependency_installed:
+                component_name = (
+                    subworkflow_to_install["name"]
+                    if isinstance(subworkflow_to_install, dict)
+                    else subworkflow_to_install
+                )
+                installed_components.append(component_name.replace("/", "_"))
+        for module_to_install in modules_to_install:
             original_component_type = self.component_type
             self.component_type = "modules"
             original_installed = self.installed_by
             self.installed_by = [Path(subworkflow_dir).parts[-1]]
-            self.install(m_install, silent=True)
+            dependency_installed = self.install(module_to_install, silent=True)
             self.component_type = original_component_type
             self.installed_by = original_installed
+            if dependency_installed:
+                component_name = module_to_install["name"] if isinstance(module_to_install, dict) else module_to_install
+                installed_components.append(component_name.replace("/", "_"))
         # self.install will have modified self.modules_repo. Restore its original value
         self.modules_repo = ini_modules_repo
 

tests/subworkflows/test_install.py

@@ -1,3 +1,4 @@
+import logging
 from pathlib import Path
 
 import pytest
@@ -65,6 +66,14 @@ def test_subworkflows_install_bam_sort_stats_samtools(self):
         assert samtools_idxstats_path.exists()
         assert samtools_flagstat_path.exists()
 
+    def test_subworkflows_install_logs_dependencies(self):
+        """Installing a subworkflow should log its module dependencies (#3871)."""
+        self.caplog.set_level(logging.INFO)
+        assert self.subworkflow_install.install("bam_sort_stats_samtools") is not False
+        assert "Installed files for 'bam_sort_stats_samtools' and its dependencies" in self.caplog.text
+        assert "samtools_index" in self.caplog.text
+        assert "bam_stats_samtools" in self.caplog.text
+
     def test_subworkflow_install_nopipeline(self):
         """Test installing a subworkflow - no pipeline given"""
         assert self.subworkflow_install.directory is not None