deepctl-cmd-update 0.2.6__tar.gz → 0.2.8__tar.gz

{deepctl_cmd_update-0.2.6 → deepctl_cmd_update-0.2.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepctl-cmd-update
-Version: 0.2.6
+Version: 0.2.8
 Summary: Update command for deepctl
 Author-email: Deepgram <devrel@deepgram.com>
 Maintainer-email: Deepgram <devrel@deepgram.com>
@@ -15,7 +15,7 @@ Classifier: Programming Language :: Python :: 3.12
 Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
-Requires-Dist: deepctl-core>=0.1.10
+Requires-Dist: deepctl-core>=0.3.1
 Requires-Dist: click>=8.0.0
 Requires-Dist: pydantic>=2.0.0
 Requires-Dist: rich>=13.0.0

{deepctl_cmd_update-0.2.6 → deepctl_cmd_update-0.2.8}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "deepctl-cmd-update"
-version = "0.2.6" # x-release-please-version
+version = "0.2.8" # x-release-please-version
 description = "Update command for deepctl"
 readme = "README.md"
 license = "MIT"
@@ -22,7 +22,7 @@ classifiers = [
 keywords = ["deepgram", "cli", "update", "self-update"]
 requires-python = ">=3.10"
 dependencies = [
-    "deepctl-core>=0.1.10",
+    "deepctl-core>=0.3.1",
     "click>=8.0.0",
     "pydantic>=2.0.0",
     "rich>=13.0.0",

{deepctl_cmd_update-0.2.6 → deepctl_cmd_update-0.2.8}/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)
@@ -136,7 +147,7 @@ class UpdateCommand(BaseCommand):
             "update.installation_method", install_info.method.value
         )
         config._set_config_value("update.installation_path", install_info.path)
-        config.save()
+        config.save(implicit=True)
 
         # Display installation info
         console.print(f"Installation method: [cyan]{install_info.method}[/cyan]")
@@ -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...")
@@ -193,7 +210,7 @@ class UpdateCommand(BaseCommand):
 
                 # Clear version cache
                 config._set_config_value("update.cached_version_info", None)
-                config.save()
+                config.save(implicit=True)
 
                 return UpdateResult(
                     success=True,
@@ -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.6 → deepctl_cmd_update-0.2.8}/src/deepctl_cmd_update/installation.py

@@ -7,6 +7,7 @@ import sys
 from enum import Enum
 from pathlib import Path
 
+from deepctl_core._launcher import get_launcher, launched_via_npm
 from pydantic import BaseModel
 
 
@@ -20,6 +21,7 @@ class InstallMethod(str, Enum):
     UV_TOOL = "uv_tool"
     UVX = "uvx"
     PIPX_RUN = "pipx_run"
+    NPM = "npm"
     SYSTEM = "system"
     DEVELOPMENT = "development"
     UNKNOWN = "unknown"
@@ -96,7 +98,7 @@ class InstallationDetector:
                 "--upgrade",
                 "deepctl",
             ],
-            InstallMethod.HOMEBREW: ["brew", "upgrade", "deepctl"],
+            InstallMethod.HOMEBREW: ["brew", "upgrade", "deepgram"],
             InstallMethod.UV_TOOL: [
                 "uv",
                 "tool",
@@ -105,6 +107,7 @@ class InstallationDetector:
             ],
             InstallMethod.UVX: None,  # Ephemeral, no update needed
             InstallMethod.PIPX_RUN: None,  # Ephemeral, no update needed
+            InstallMethod.NPM: None,  # Updated through npm, see instructions
             InstallMethod.SYSTEM: None,  # Handled separately
             InstallMethod.DEVELOPMENT: None,  # Can't auto-update
             InstallMethod.UNKNOWN: None,
@@ -121,7 +124,23 @@ class InstallationDetector:
             Human-readable update instructions
         """
         if info.method == InstallMethod.HOMEBREW:
-            return "Please update using Homebrew: brew upgrade deepctl"
+            return "Please update using Homebrew: brew update && brew upgrade deepgram"
+
+        # The launcher's venv or uv cache is private to it, so pip/uv run
+        # from inside would update the wrong thing; only npm can update it.
+        if info.method == InstallMethod.NPM:
+            if get_launcher() == "npx":
+                return (
+                    "Running via npx. Use the latest version with: "
+                    "npx @deepgram/cli@latest"
+                )
+            return (
+                "Installed via npm. Update with: "
+                "npm install -g @deepgram/cli@latest "
+                "(or run npx @deepgram/cli@latest). "
+                "If @deepgram/cli is a dependency of your project, update it "
+                "there instead: npm install @deepgram/cli@latest"
+            )
 
         if info.method == InstallMethod.UVX:
             return (
@@ -138,7 +157,9 @@ class InstallationDetector:
         if info.method == InstallMethod.SYSTEM:
             # Try to detect the system package manager
             if sys.platform.startswith("darwin"):
-                return "Please update using Homebrew: brew upgrade deepctl"
+                return (
+                    "Please update using Homebrew: brew update && brew upgrade deepgram"
+                )
             elif sys.platform.startswith("linux"):
                 # Check for common package managers
                 if Path("/etc/debian_version").exists():
@@ -261,8 +282,8 @@ class InstallationDetector:
     ) -> InstallMethod:
         """Detect the installation method.
 
-        Priority: development > oneshot > homebrew > uv_tool > pipx >
-                  uv > system > pip > unknown
+        Priority: npm > development > oneshot > homebrew > uv_tool >
+                  pipx > uv > system > pip > unknown
 
         Args:
             package_location: Path to package
@@ -272,6 +293,11 @@ class InstallationDetector:
         Returns:
             Detected installation method
         """
+        # The npm launcher runs deepctl through uv or its own venv, which
+        # would otherwise be detected as uvx or pip.
+        if launched_via_npm():
+            return InstallMethod.NPM
+
         # Development installation
         if is_editable:
             return InstallMethod.DEVELOPMENT

{deepctl_cmd_update-0.2.6 → deepctl_cmd_update-0.2.8}/src/deepctl_cmd_update/plugin_update_check.py

@@ -261,7 +261,7 @@ def print_pending_plugin_notifications() -> None:
             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"
+                f"  —  run 'dg plugin update {u['name']}' to upgrade\033[0m\n"
             )
         else:
             sys.stderr.write("\033[33mPlugin updates available:\033[0m\n")
@@ -270,7 +270,7 @@ def print_pending_plugin_notifications() -> None:
                     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"
+                "\033[33mRun 'dg plugin update <name>' to upgrade\033[0m\n"
             )
     except (BrokenPipeError, OSError, ValueError):
         # The output stream closed (e.g. an MCP host disconnected stdio

{deepctl_cmd_update-0.2.6 → deepctl_cmd_update-0.2.8}/src/deepctl_cmd_update/startup_check.py

@@ -15,6 +15,8 @@ import time
 from pathlib import Path
 from typing import Any
 
+from deepctl_core._launcher import launched_via_npm, npm_upgrade_command
+
 # Sentinel used when the background check is suppressed or has no result
 _NO_RESULT: dict[str, Any] = {}
 
@@ -42,8 +44,20 @@ def _is_ci() -> bool:
     return any(os.environ.get(v) for v in _CI_ENV_VARS)
 
 
+def upgrade_hint() -> str:
+    """Return the command a user should run to upgrade deepctl."""
+    if launched_via_npm():
+        return npm_upgrade_command()
+    return "dg update"
+
+
 def _is_oneshot() -> bool:
     """Return True if running in an ephemeral one-shot environment."""
+    # The npm launcher often runs deepctl through `uv tool run`, which looks
+    # one-shot, but the npm install itself persists and is pinned to one
+    # version, so its users do need to hear about updates.
+    if launched_via_npm():
+        return False
     # uvx
     if "UV_INTERNAL__PARENT_INTERPRETER" in os.environ:
         return True
@@ -200,7 +214,7 @@ def print_pending_notification() -> None:
         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"
+                f"  —  run \033[1m{upgrade_hint()}\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

{deepctl_cmd_update-0.2.6 → deepctl_cmd_update-0.2.8}/src/deepctl_cmd_update/version_check.py

@@ -7,6 +7,8 @@ from deepctl_core import Config
 from packaging import version
 from pydantic import BaseModel, Field
 
+from .startup_check import upgrade_hint
+
 
 class VersionInfo(BaseModel):
     """Version information from PyPI."""
@@ -245,7 +247,7 @@ class VersionChecker:
         self.config._set_config_value(
             "update.cached_version_info", info.model_dump(mode="json")
         )
-        self.config.save()
+        self.config.save(implicit=True)
 
 
 def format_version_message(info: VersionInfo) -> str:
@@ -270,5 +272,5 @@ def format_version_message(info: VersionInfo) -> str:
         else:
             message += f" (released {days_old} days ago)"
 
-    message += "\nRun 'dg update' to upgrade"
+    message += f"\nRun '{upgrade_hint()}' to upgrade"
     return message

{deepctl_cmd_update-0.2.6 → deepctl_cmd_update-0.2.8}/src/deepctl_cmd_update.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepctl-cmd-update
-Version: 0.2.6
+Version: 0.2.8
 Summary: Update command for deepctl
 Author-email: Deepgram <devrel@deepgram.com>
 Maintainer-email: Deepgram <devrel@deepgram.com>
@@ -15,7 +15,7 @@ Classifier: Programming Language :: Python :: 3.12
 Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
-Requires-Dist: deepctl-core>=0.1.10
+Requires-Dist: deepctl-core>=0.3.1
 Requires-Dist: click>=8.0.0
 Requires-Dist: pydantic>=2.0.0
 Requires-Dist: rich>=13.0.0

{deepctl_cmd_update-0.2.6 → deepctl_cmd_update-0.2.8}/src/deepctl_cmd_update.egg-info/requires.txt

@@ -1,4 +1,4 @@
-deepctl-core>=0.1.10
+deepctl-core>=0.3.1
 click>=8.0.0
 pydantic>=2.0.0
 rich>=13.0.0