deepctl-cmd-update 0.2.5__tar.gz → 0.2.7__tar.gz

{deepctl_cmd_update-0.2.5 → deepctl_cmd_update-0.2.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepctl-cmd-update
-Version: 0.2.5
+Version: 0.2.7
 Summary: Update command for deepctl
 Author-email: Deepgram <devrel@deepgram.com>
 Maintainer-email: Deepgram <devrel@deepgram.com>

{deepctl_cmd_update-0.2.5 → deepctl_cmd_update-0.2.7}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "deepctl-cmd-update"
-version = "0.2.5" # x-release-please-version
+version = "0.2.7" # x-release-please-version
 description = "Update command for deepctl"
 readme = "README.md"
 license = "MIT"

{deepctl_cmd_update-0.2.5 → deepctl_cmd_update-0.2.7}/src/deepctl_cmd_update/command.py

@@ -69,8 +69,16 @@ class UpdateCommand(BaseCommand):
         auth_manager: Any,  # Not used for update command
         client: Any,  # Not used for update command
         **kwargs: Any,
-    ) -> dict[str, Any]:
-        """Handle the update command execution."""
+    ) -> UpdateResult | dict[str, Any]:
+        """Handle the update command execution.
+
+        Returns the model rather than a dict wherever the exit code has to
+        be non-zero: BaseCommand.exit_code_for reads `.status` off the
+        result, and a dict has none, so a dict always exits 0. The three
+        failure paths carry status="error" (exit 1) and the decline path
+        status="cancelled" (exit 2). output_result unwraps either, so the
+        JSON payload is the same shape from both.
+        """
         console = get_console()
 
         # Extract arguments from kwargs
@@ -93,10 +101,13 @@ class UpdateCommand(BaseCommand):
                 version_info = asyncio.run(version_checker.check_version(force=True))
             except Exception as e:
                 print_error(f"Failed to check for updates: {e}")
+                # status="error" on the model, not a dict: see the docstring.
+                # A command that failed has to exit 1, as the README says.
                 return UpdateResult(
+                    status="error",
                     success=False,
                     message=f"Failed to check for updates: {e}",
-                ).model_dump()
+                )
 
         # Display version info
         message = format_version_message(version_info)
@@ -167,14 +178,20 @@ class UpdateCommand(BaseCommand):
         # Confirm update
         if not yes and not Confirm.ask("\nDo you want to proceed with the update?"):
             print_info("Update cancelled")
+            # Return the model, not .model_dump(): BaseCommand.exit_code_for
+            # reads `.status` off the result, and a plain dict has none -- a
+            # declined update used to exit 0 while its own payload said
+            # "Update cancelled by user". status="cancelled" is the documented
+            # exit 2.
             return UpdateResult(
+                status="cancelled",
                 success=False,
                 message="Update cancelled by user",
                 current_version=version_info.current_version,
                 latest_version=version_info.latest_version,
                 update_available=version_info.update_available,
                 installation_method=install_info.method.value,
-            ).model_dump()
+            )
 
             # Execute update
         print_info("Updating deepctl...")
@@ -212,13 +229,14 @@ class UpdateCommand(BaseCommand):
                 console.print(f"[yellow]{shlex.join(update_command)}[/yellow]")
 
                 return UpdateResult(
+                    status="error",
                     success=False,
                     message=f"Update failed: {error_msg}",
                     current_version=version_info.current_version,
                     latest_version=version_info.latest_version,
                     update_available=version_info.update_available,
                     installation_method=install_info.method.value,
-                ).model_dump()
+                )
 
         except Exception as e:
             print_error(f"Failed to execute update: {e}")
@@ -228,10 +246,11 @@ class UpdateCommand(BaseCommand):
             console.print(f"[yellow]{update_command}[/yellow]")
 
             return UpdateResult(
+                status="error",
                 success=False,
                 message=f"Failed to execute update: {e}",
                 current_version=version_info.current_version,
                 latest_version=version_info.latest_version,
                 update_available=version_info.update_available,
                 installation_method=install_info.method.value,
-            ).model_dump()
+            )

{deepctl_cmd_update-0.2.5 → deepctl_cmd_update-0.2.7}/src/deepctl_cmd_update/plugin_update_check.py

@@ -256,18 +256,23 @@ def print_pending_plugin_notifications() -> None:
     if not updates:
         return
 
-    if len(updates) == 1:
-        u = updates[0]
-        sys.stderr.write(
-            f"\033[33mPlugin update available: {u['name']} {u['current']} → {u['latest']}"
-            f"  —  run 'deepctl plugin update {u['name']}' to upgrade\033[0m\n"
-        )
-    else:
-        sys.stderr.write("\033[33mPlugin updates available:\033[0m\n")
-        for u in updates:
+    try:
+        if len(updates) == 1:
+            u = updates[0]
             sys.stderr.write(
-                f"\033[33m  {u['name']} {u['current']} → {u['latest']}\033[0m\n"
+                f"\033[33mPlugin update available: {u['name']} {u['current']} → {u['latest']}"
+                f"  —  run 'deepctl plugin update {u['name']}' to upgrade\033[0m\n"
             )
-        sys.stderr.write(
-            "\033[33mRun 'deepctl plugin update <name>' to upgrade\033[0m\n"
-        )
+        else:
+            sys.stderr.write("\033[33mPlugin updates available:\033[0m\n")
+            for u in updates:
+                sys.stderr.write(
+                    f"\033[33m  {u['name']} {u['current']} → {u['latest']}\033[0m\n"
+                )
+            sys.stderr.write(
+                "\033[33mRun 'deepctl plugin update <name>' to upgrade\033[0m\n"
+            )
+    except (BrokenPipeError, OSError, ValueError):
+        # The output stream closed (e.g. an MCP host disconnected stdio
+        # after `dg mcp` finished). These notifications are purely advisory.
+        pass

{deepctl_cmd_update-0.2.5 → deepctl_cmd_update-0.2.7}/src/deepctl_cmd_update/startup_check.py

@@ -197,7 +197,13 @@ def print_pending_notification() -> None:
     latest = _result.get("latest")
     current = _result.get("current")
     if latest and current:
-        sys.stderr.write(
-            f"\n\033[33m  Update available:  {current} → {latest}"
-            f"  —  run \033[1mdg update\033[0m\033[33m to upgrade\033[0m\n\n"
-        )
+        try:
+            sys.stderr.write(
+                f"\n\033[33m  Update available:  {current} → {latest}"
+                f"  —  run \033[1mdg update\033[0m\033[33m to upgrade\033[0m\n\n"
+            )
+        except (BrokenPipeError, OSError, ValueError):
+            # The output stream closed (e.g. an MCP host disconnected stdio
+            # after `dg mcp` finished). Nothing useful to do — the notification
+            # is purely advisory.
+            pass

{deepctl_cmd_update-0.2.5 → deepctl_cmd_update-0.2.7}/src/deepctl_cmd_update.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepctl-cmd-update
-Version: 0.2.5
+Version: 0.2.7
 Summary: Update command for deepctl
 Author-email: Deepgram <devrel@deepgram.com>
 Maintainer-email: Deepgram <devrel@deepgram.com>