deepctl-core 0.2.11__tar.gz → 0.2.13__tar.gz

{deepctl_core-0.2.11 → deepctl_core-0.2.13}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepctl-core
-Version: 0.2.11
+Version: 0.2.13
 Summary: Core components for deepctl
 Author-email: Deepgram <devrel@deepgram.com>
 Maintainer-email: Deepgram <devrel@deepgram.com>

{deepctl_core-0.2.11 → deepctl_core-0.2.13}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "deepctl-core"
-version = "0.2.11" # x-release-please-version
+version = "0.2.13" # x-release-please-version
 description = "Core components for deepctl"
 readme = "README.md"
 license = "MIT"

{deepctl_core-0.2.11 → deepctl_core-0.2.13}/src/deepctl_core/base_command.py

@@ -39,6 +39,7 @@ class BaseCommand(ABC):
             raise ValueError("Command must have a name")
         if not self.help:
             raise ValueError("Command must have help text")
+        self._guided: bool = True
 
     def execute(self, ctx: click.Context, **kwargs: Any) -> None:
         """Execute the command with Click context.
@@ -104,26 +105,112 @@ class BaseCommand(ABC):
                         )
                         raise click.ClickException("Project ID required")
 
+            self._guided = self.is_guided(ctx)
+            self._tag_telemetry_start(ctx)
+
             # Execute the command
             try:
                 with TimingContext(f"command_{self.name}_handler"):
                     result = self.handle(config, auth_manager, client, **kwargs)
 
+                status = "ok"
+                if result is not None and hasattr(result, "status"):
+                    status = str(result.status)
+                self._tag_telemetry_status(status)
+
                 # Handle command result
                 if result is not None:
                     with TimingContext("output_processing"):
-                        self.output_result(result, config)
+                        try:
+                            self.output_result(result, config)
+                        except (BrokenPipeError, OSError):
+                            # Downstream stream closed (e.g. an MCP host disconnected
+                            # stdio after `dg mcp` finished). Nothing useful to log
+                            # here because the logger writes to the same closed stream.
+                            pass
+                        except ValueError as exc:
+                            if "closed file" not in str(exc):
+                                raise
 
             except KeyboardInterrupt:
+                self._tag_telemetry_status("cancelled")
                 stderr_console.print("\n[yellow]Command cancelled by user[/yellow]")
                 raise click.Abort()
 
             except Exception as e:
+                self._tag_telemetry_status("error")
                 print_error(f"Command failed: {e}")
                 if config.get("output.verbose", False):
                     stderr_console.print_exception()
                 raise click.ClickException(str(e))
 
+    def is_guided(self, ctx: click.Context) -> bool:
+        """True only when the user invoked this command with no input at all.
+
+        'No input' = no positional arg, no flag, no option value, no env-var
+        override. The single rule for every command: any user-provided signal
+        means scripting intent — skip prompts. Only the bare invocation gets
+        the guided/interactive flow.
+
+        Returns False whenever telemetry's `_agentic` heuristic fires
+        (CI=1, --agent-friendly, --non-interactive, AI tool env vars, etc.).
+        """
+        if _agentic:
+            return False
+        for param in ctx.command.params:
+            if not param.name:
+                continue
+            src = ctx.get_parameter_source(param.name)
+            if src is not None and src.name in ("COMMANDLINE", "ENVIRONMENT"):
+                return False
+        return True
+
+    def _tag_telemetry_start(self, ctx: click.Context) -> None:
+        """Annotate the active Sentry transaction with command-level usage signal.
+
+        Sets the transaction name to the full Click command path
+        (e.g. 'deepctl debug audio' instead of just 'debug') and tags it with
+        the user-provided flag NAMES (never values), the requested output
+        format, and the auth method. All values are bounded enums or
+        already-public flag identifiers — no PII risk.
+
+        Wrapped in a bare except so a Sentry hiccup, missing scope, or unknown
+        Click parameter-source enum value can never crash the user's command.
+        """
+        try:
+            import sentry_sdk
+
+            scope = sentry_sdk.get_current_scope()
+            transaction = getattr(scope, "transaction", None)
+            if transaction is not None and ctx.command_path:
+                transaction.name = ctx.command_path
+
+            used: list[str] = []
+            for param in ctx.command.params:
+                if not param.name:
+                    continue
+                src = ctx.get_parameter_source(param.name)
+                if src is not None and src.name == "COMMANDLINE":
+                    used.append(param.name)
+            scope.set_tag("cmd.flags", ",".join(sorted(used)) or "(none)")
+            scope.set_tag("cmd.output_format", ctx.params.get("output") or "default")
+        except Exception:
+            pass
+
+    def _tag_telemetry_status(self, status: str) -> None:
+        """Tag the active Sentry transaction with the command outcome.
+
+        Status is a bounded enum: 'ok', 'cancelled', 'error', or whatever the
+        command's BaseResult.status returned (also bounded by the result
+        model). Bare except so telemetry can't crash the command teardown.
+        """
+        try:
+            import sentry_sdk
+
+            sentry_sdk.get_current_scope().set_tag("cmd.status", status)
+        except Exception:
+            pass
+
     @abstractmethod
     def handle(
         self,
@@ -294,7 +381,7 @@ class BaseCommand(ABC):
         Returns:
             True if confirmed, False otherwise
         """
-        if _agentic or not self.ci_friendly:
+        if _agentic or not self.ci_friendly or not getattr(self, "_guided", True):
             return default
 
         try:
@@ -318,7 +405,9 @@ class BaseCommand(ABC):
         Returns:
             User input
         """
-        if (_agentic or not self.ci_friendly) and default is not None:
+        if (
+            _agentic or not self.ci_friendly or not getattr(self, "_guided", True)
+        ) and default is not None:
             return default
 
         try:

{deepctl_core-0.2.11 → deepctl_core-0.2.13}/src/deepctl_core.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepctl-core
-Version: 0.2.11
+Version: 0.2.13
 Summary: Core components for deepctl
 Author-email: Deepgram <devrel@deepgram.com>
 Maintainer-email: Deepgram <devrel@deepgram.com>