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

{deepctl_cmd_plugin-0.1.12 → 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/

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

@@ -1,5 +1,13 @@
 # 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)
 
 

{deepctl_cmd_plugin-0.1.12 → 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.12
+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.12 → 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.12" # 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.12 → 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.12 → 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
@@ -234,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."""