show installed plugin packages

Author: johnnygreco

packages/data-designer/src/data_designer/cli/README.md

@@ -336,7 +336,7 @@ data-designer plugin list
 # Search a specific catalog
 data-designer plugin --catalog research search transform
 
-# Show package metadata, compatibility, docs, and the install command
+# Show package metadata, compatibility, docs, and the install strategy
 data-designer plugin info github
 
 # Install a plugin package from a catalog and verify its runtime entry points can load
@@ -359,7 +359,7 @@ data-designer plugin catalog add research https://github.com/acme/dd-plugins
 data-designer plugin catalog list
 data-designer plugin catalog remove research
 
-# List installed runtime plugins with package and version metadata
+# List installed plugin packages with their runtime plugins
 data-designer plugin installed
 ```
 
@@ -371,10 +371,10 @@ are kept in place. This prevents a plugin dependency from upgrading,
 downgrading, or reinstalling Data Designer itself.
 
 Runtime plugin names shown by `plugin list`, `plugin search`, and
-`plugin installed` identify the registered plugin entry points.
-Install, uninstall, and info commands take the plugin package name or package
-alias. If a user passes a runtime plugin name to one of those package commands,
-the CLI points them to the owning package.
+`plugin installed` identify the registered entry points provided by a plugin
+package. Install, uninstall, and info commands take the plugin package name or
+package alias. If a user passes a runtime plugin name to one of those package
+commands, the CLI points them to the owning package.
 
 In an active virtual environment with a user `pyproject.toml`, `uv` uses
 `uv add` so the plugin package is recorded in the project. Otherwise the CLI

packages/data-designer/src/data_designer/cli/commands/plugin.py

@@ -203,8 +203,8 @@ def uninstall_command(
 
 
 def installed_command(ctx: typer.Context) -> None:
-    """List installed Data Designer runtime plugins with package metadata."""
-    _warn_if_parent_catalog_unused(ctx, "installed runtime plugins are discovered from the current Python environment")
+    """List installed Data Designer plugin packages with runtime plugin metadata."""
+    _warn_if_parent_catalog_unused(ctx, "installed plugin packages are discovered from the current Python environment")
     controller = PluginCatalogController(DATA_DESIGNER_HOME)
     controller.run_installed()
 

packages/data-designer/src/data_designer/cli/controllers/plugin_catalog_controller.py

@@ -305,11 +305,11 @@ def run_uninstall(
             )
 
     def run_installed(self) -> None:
-        """List installed runtime plugins without importing plugin modules."""
-        print_header("Installed Data Designer Runtime Plugins")
+        """List installed plugin packages without importing plugin modules."""
+        print_header("Installed Data Designer Plugin Packages")
         installed_plugins = self.catalog_service.list_installed_plugins()
         if not installed_plugins:
-            print_warning("No installed Data Designer runtime plugins were discovered")
+            print_warning("No installed Data Designer plugin packages were discovered")
             return
         self._display_installed_plugins(installed_plugins)
 
@@ -590,18 +590,16 @@ def _display_catalog_entries_vertical(
 
     @staticmethod
     def _display_installed_plugins(installed_plugins: list[InstalledPluginInfo]) -> None:
-        table = Table(title="Installed Runtime Plugins", border_style=NordColor.NORD8.value)
-        table.add_column("Runtime Plugin", style=NordColor.NORD14.value, no_wrap=True)
-        table.add_column("Package", style=NordColor.NORD9.value, no_wrap=True)
+        table = Table(title="Installed Plugin Packages", border_style=NordColor.NORD8.value)
+        table.add_column("Package", style=NordColor.NORD14.value, no_wrap=True)
         table.add_column("Version", style=NordColor.NORD13.value, no_wrap=True)
-        table.add_column("Entry Point", style=NordColor.NORD4.value)
+        table.add_column("Runtime Plugins", style=NordColor.NORD9.value)
 
-        for plugin in installed_plugins:
+        for package_name, package_version, package_plugins in _group_installed_plugins_by_package(installed_plugins):
             table.add_row(
-                _escape_markup(plugin.name),
-                _escape_markup(plugin.package_name or ""),
-                _escape_markup(plugin.package_version or ""),
-                _escape_markup(plugin.entry_point_value),
+                _escape_markup(package_name),
+                _escape_markup(package_version),
+                _escape_markup(", ".join(plugin.name for plugin in package_plugins)),
             )
         console.print(table)
 
@@ -741,6 +739,21 @@ def _package_entries_are_installed(
     )
 
 
+def _group_installed_plugins_by_package(
+    installed_plugins: list[InstalledPluginInfo],
+) -> list[tuple[str, str, list[InstalledPluginInfo]]]:
+    groups: dict[tuple[str, str], list[InstalledPluginInfo]] = {}
+    for plugin in installed_plugins:
+        package_name = plugin.package_name or "unknown"
+        package_version = plugin.package_version or ""
+        groups.setdefault((package_name, package_version), []).append(plugin)
+
+    return [
+        (package_name, package_version, sorted(package_plugins, key=lambda plugin: plugin.name))
+        for (package_name, package_version), package_plugins in sorted(groups.items())
+    ]
+
+
 def _package_alias(package_name: str) -> str | None:
     canonical_package_name = canonicalize_name(package_name)
     if not canonical_package_name.startswith(DATA_DESIGNER_PLUGIN_PACKAGE_PREFIX):

packages/data-designer/tests/cli/commands/test_plugin_command.py

@@ -174,7 +174,7 @@ def test_plugin_installed_warns_when_parent_catalog_is_unused(
 
     assert result.exit_code == 0
     mock_print_warning.assert_called_once_with(
-        "Ignoring --catalog 'research'; installed runtime plugins are discovered from the current Python environment."
+        "Ignoring --catalog 'research'; installed plugin packages are discovered from the current Python environment."
     )
     mock_ctrl.run_installed.assert_called_once_with()
 

packages/data-designer/tests/cli/controllers/test_plugin_catalog_controller.py

@@ -994,11 +994,17 @@ def test_run_installed_renders_package_and_version_metadata(
 ) -> None:
     controller.catalog_service.list_installed_plugins.return_value = [
         InstalledPluginInfo(
-            name="github",
-            package_name="data-designer-github",
+            name="document-chunker",
+            package_name="data-designer-retrieval-sdg",
             package_version="0.1.0",
-            entry_point_value="data_designer_github.plugin:plugin",
-        )
+            entry_point_value="data_designer_retrieval_sdg.plugins:document_chunker_plugin",
+        ),
+        InstalledPluginInfo(
+            name="embedding-dedup",
+            package_name="data-designer-retrieval-sdg",
+            package_version="0.1.0",
+            entry_point_value="data_designer_retrieval_sdg.plugins:embedding_dedup_plugin",
+        ),
     ]
 
     controller.run_installed()
@@ -1007,14 +1013,15 @@ def test_run_installed_renders_package_and_version_metadata(
         call.args[0] for call in mock_console.print.call_args_list if call.args and isinstance(call.args[0], Table)
     ]
     assert printed_tables
+    assert printed_tables[0].title == "Installed Plugin Packages"
     assert [column.header for column in printed_tables[0].columns] == [
-        "Runtime Plugin",
         "Package",
         "Version",
-        "Entry Point",
+        "Runtime Plugins",
     ]
-    assert list(printed_tables[0].columns[1].cells) == ["data-designer-github"]
-    assert list(printed_tables[0].columns[2].cells) == ["0.1.0"]
+    assert list(printed_tables[0].columns[0].cells) == ["data-designer-retrieval-sdg"]
+    assert list(printed_tables[0].columns[1].cells) == ["0.1.0"]
+    assert list(printed_tables[0].columns[2].cells) == ["document-chunker, embedding-dedup"]
 
 
 @patch("data_designer.cli.controllers.plugin_catalog_controller.print_error")