deepctl-cmd-plugin 0.1.11__tar.gz → 0.1.13__tar.gz

{deepctl_cmd_plugin-0.1.11 → deepctl_cmd_plugin-0.1.13}/.gitignore

@@ -129,7 +129,8 @@ dmypy.json
 
 # uv
 .uv_cache/
-uv.lock
+# uv.lock IS committed: it pins the dev/CI environment so every run tests
+# the same versions and dependency drift arrives as a reviewable PR.
 
 # IDEs
 .vscode/
@@ -157,4 +158,6 @@ Thumbs.db
 # tmps
 tmp_chatface/
 tmp_community-turbo/
-tmp_deepgram-cli/
+tmp_deepgram-cli/
+# Local Netlify folder
+.netlify

{deepctl_cmd_plugin-0.1.11 → deepctl_cmd_plugin-0.1.13}/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## [0.1.13](https://github.com/deepgram/cli/compare/deepctl-cmd-plugin-v0.1.12...deepctl-cmd-plugin-v0.1.13) (2026-09-23)
+
+
+### Bug Fixes
+
+* **plugin:** return exit code 2, rather than success, when plugin removal is cancelled ([0dcfd2c](https://github.com/deepgram/cli/commit/0dcfd2c0d68a6095f4f6651e51ba6995f2b430d3))
+* **plugin:** write the plugin-removal confirmation prompt to stderr so JSON output stays parseable ([0497ac1](https://github.com/deepgram/cli/commit/0497ac1892333c09d2a482f0b10ccd59d1f1dba2))
+
+## [0.1.12](https://github.com/deepgram/cli/compare/deepctl-cmd-plugin-v0.1.11...deepctl-cmd-plugin-v0.1.12) (2026-05-09)
+
+
+### Features
+
+* **telemetry:** full Sentry observability + per-command usage tags ([#75](https://github.com/deepgram/cli/issues/75)) ([0fe43d2](https://github.com/deepgram/cli/commit/0fe43d2e00c58d8101ef2bd4b5aaf4437db9f0cf))
+
 ## [0.1.11](https://github.com/deepgram/cli/compare/deepctl-cmd-plugin-v0.1.10...deepctl-cmd-plugin-v0.1.11) (2026-03-09)
 
 

{deepctl_cmd_plugin-0.1.11 → deepctl_cmd_plugin-0.1.13}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.4
+Metadata-Version: 2.5
 Name: deepctl-cmd-plugin
-Version: 0.1.11
+Version: 0.1.13
 Summary: Plugin management command for deepctl
 Author-email: Deepgram DevRel <devrel@deepgram.com>
 Requires-Python: >=3.10

{deepctl_cmd_plugin-0.1.11 → deepctl_cmd_plugin-0.1.13}/pyproject.toml

@@ -2,7 +2,7 @@
 name = "deepctl-cmd-plugin"
 description = "Plugin management command for deepctl"
 readme = "README.md"
-version = "0.1.11" # x-release-please-version
+version = "0.1.13" # x-release-please-version
 requires-python = ">=3.10"
 authors = [{ name = "Deepgram DevRel", email = "devrel@deepgram.com" }]
 dependencies = [

{deepctl_cmd_plugin-0.1.11 → deepctl_cmd_plugin-0.1.13}/src/deepctl_cmd_plugin/command.py

@@ -474,8 +474,17 @@ class PluginCommand(BaseGroupCommand):
         package = kwargs["package"]
         yes = kwargs.get("yes", False)
 
-        if not yes and not click.confirm(f"Are you sure you want to remove {package}?"):
-            return
+        # err=True: the prompt is interaction, not output. Keeping it off
+        # stdout is what lets `dg plugin remove ... -o json` stay parseable,
+        # and matches the sibling prompt in deepctl-cmd-keys.
+        if not yes and not click.confirm(
+            f"Are you sure you want to remove {package}?", err=True
+        ):
+            # Abort rather than return: this is a group subcommand, so there is
+            # no result for BaseCommand.EXIT_CODES to map, and a bare return
+            # exits 0 -- indistinguishable from a successful removal. main.py
+            # turns Abort into the documented exit 2 for user cancellation.
+            raise click.Abort()
 
         result = self.remove_plugin(config, auth_manager, client, package)
 

{deepctl_cmd_plugin-0.1.11 → deepctl_cmd_plugin-0.1.13}/tests/unit/test_plugin_command.py

@@ -4,6 +4,7 @@ import json
 from pathlib import Path
 from unittest.mock import MagicMock, patch
 
+import click
 import pytest
 from click.testing import CliRunner
 from deepctl_cmd_plugin.command import PluginCommand
@@ -79,12 +80,11 @@ class TestPluginCommand:
         """Test getting plugin state from existing file."""
         test_state = {"plugins": {"test-plugin": {"version": "1.0.0"}}}
 
-        with patch.object(Path, "exists", return_value=True):
-            with patch.object(
-                Path, "read_text", return_value=json.dumps(test_state)
-            ):
-                state = self.command._get_plugin_state()
-                assert state == test_state
+        with patch.object(Path, "exists", return_value=True), patch.object(
+            Path, "read_text", return_value=json.dumps(test_state)
+        ):
+            state = self.command._get_plugin_state()
+            assert state == test_state
 
     def test_save_plugin_state(self) -> None:
         """Test saving plugin state."""
@@ -140,34 +140,32 @@ class TestPluginCommand:
                 # Mock state and version operations
                 with patch.object(
                     self.command, "_get_plugin_state"
-                ) as mock_get_state:
-                    with patch.object(
-                        self.command, "_save_plugin_state"
-                    ) as mock_save_state:
-                        with patch.object(
-                            self.command,
-                            "_get_package_version",
-                            return_value="1.0.0",
-                        ):
-                            mock_get_state.return_value = {"plugins": {}}
-
-                            options = PluginInstallOptions(
-                                package="test-plugin"
-                            )
-                            result = self.command.install_plugin(
-                                self.config,
-                                self.auth_manager,
-                                self.client,
-                                options,
-                            )
-
-                            assert result.success is True
-                            assert "Successfully installed" in result.message
-                            # Strategy should use the plugin env python
-                            cmd = mock_strategy_run.call_args[0][0]
-                            assert cmd[0] == "/path/to/plugin/python"
-                            # Should save plugin state
-                            mock_save_state.assert_called_once()
+                ) as mock_get_state, patch.object(
+                    self.command, "_save_plugin_state"
+                ) as mock_save_state, patch.object(
+                    self.command,
+                    "_get_package_version",
+                    return_value="1.0.0",
+                ):
+                    mock_get_state.return_value = {"plugins": {}}
+
+                    options = PluginInstallOptions(
+                        package="test-plugin"
+                    )
+                    result = self.command.install_plugin(
+                        self.config,
+                        self.auth_manager,
+                        self.client,
+                        options,
+                    )
+
+                    assert result.success is True
+                    assert "Successfully installed" in result.message
+                    # Strategy should use the plugin env python
+                    cmd = mock_strategy_run.call_args[0][0]
+                    assert cmd[0] == "/path/to/plugin/python"
+                    # Should save plugin state
+                    mock_save_state.assert_called_once()
 
     def test_install_plugin_git_url(self) -> None:
         """Test handling git URL in install options."""
@@ -237,6 +235,54 @@ class TestPluginCommand:
             assert result.success is False
             assert "not installed" in result.message
 
+    def test_remove_declined_aborts_instead_of_exiting_zero(self) -> None:
+        """Declining the prompt must exit 2, not 0.
+
+        `_handle_remove` is a group subcommand returning None, so there is no
+        result for BaseCommand.EXIT_CODES to map to an exit code. A bare
+        return made a declined removal indistinguishable from a successful
+        one for any script branching on the exit code, contradicting the
+        contract published in the README. Abort is what main.py turns into 2.
+        """
+        with (
+            patch("click.confirm", return_value=False),
+            patch.object(self.command, "remove_plugin") as mock_remove,
+        ):
+            with pytest.raises(click.Abort):
+                self.command._handle_remove(
+                    self.config,
+                    self.auth_manager,
+                    self.client,
+                    package="test-plugin",
+                )
+
+            mock_remove.assert_not_called()
+
+    def test_remove_with_yes_skips_the_prompt(self) -> None:
+        """Positive control: --yes must not prompt and must not abort."""
+        with (
+            patch("click.confirm") as mock_confirm,
+            patch.object(self.command, "remove_plugin") as mock_remove,
+            patch.object(self.command, "_maybe_update_skills"),
+        ):
+            mock_remove.return_value = PluginOperationResult(
+                success=True,
+                action="remove",
+                package="test-plugin",
+                message="Successfully removed test-plugin",
+            )
+
+            self.command._handle_remove(
+                self.config,
+                self.auth_manager,
+                self.client,
+                package="test-plugin",
+                yes=True,
+            )
+
+            mock_confirm.assert_not_called()
+            mock_remove.assert_called_once()
+
     @patch("deepctl_cmd_plugin.command.subprocess.run")
     def test_discover_from_environment(self, mock_run: MagicMock) -> None:
         """Test discovering plugins from a specific environment."""
@@ -317,32 +363,29 @@ class TestPluginCommand:
 
         with patch.object(
             self.command, "_discover_plugins", return_value=test_plugins
-        ):
-            with patch(
-                "deepctl_cmd_plugin.command.console.print"
-            ) as mock_print:
-                with patch(
-                    "deepctl_cmd_plugin.command.print_info"
-                ) as mock_print_info:
-                    with patch.object(
-                        self.command.detector, "detect"
-                    ) as mock_detect:
-                        mock_detect.return_value.method = InstallMethod.SYSTEM
-
-                        self.command.list_plugins(
-                            self.config,
-                            self.auth_manager,
-                            self.client,
-                            verbose=True,
-                        )
-
-                        # Should print a table
-                        mock_print.assert_called_once()
-                        # Should show system installation info via print_info
-                        assert any(
-                            "system" in str(call).lower()
-                            for call in mock_print_info.call_args_list
-                        )
+        ), patch(
+            "deepctl_cmd_plugin.command.console.print"
+        ) as mock_print, patch(
+            "deepctl_cmd_plugin.command.print_info"
+        ) as mock_print_info, patch.object(
+            self.command.detector, "detect"
+        ) as mock_detect:
+            mock_detect.return_value.method = InstallMethod.SYSTEM
+
+            self.command.list_plugins(
+                self.config,
+                self.auth_manager,
+                self.client,
+                verbose=True,
+            )
+
+            # Should print a table
+            mock_print.assert_called_once()
+            # Should show system installation info via print_info
+            assert any(
+                "system" in str(call).lower()
+                for call in mock_print_info.call_args_list
+            )
 
     def test_setup_commands(self) -> None:
         """Test that all subcommands are properly set up."""
@@ -396,21 +439,20 @@ class TestPluginCommand:
                 # Test search all
                 with patch(
                     "deepctl_cmd_plugin.command.console.print"
-                ) as mock_print:
-                    with patch(
-                        "deepctl_cmd_plugin.command.print_info"
-                    ) as mock_info:
-                        self.command._handle_search(
-                            self.config, self.auth_manager, self.client
-                        )
-
-                        # Should print a table
-                        mock_print.assert_called_once()
-                        # Should show install hint
-                        assert any(
-                            "install" in str(call)
-                            for call in mock_info.call_args_list
-                        )
+                ) as mock_print, patch(
+                    "deepctl_cmd_plugin.command.print_info"
+                ) as mock_info:
+                    self.command._handle_search(
+                        self.config, self.auth_manager, self.client
+                    )
+
+                    # Should print a table
+                    mock_print.assert_called_once()
+                    # Should show install hint
+                    assert any(
+                        "install" in str(call)
+                        for call in mock_info.call_args_list
+                    )
 
                 # Test search with query
                 with patch(