deepctl-cmd-ffprobe 0.0.2__tar.gz → 0.0.4__tar.gz

{deepctl_cmd_ffprobe-0.0.2 → deepctl_cmd_ffprobe-0.0.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepctl-cmd-ffprobe
-Version: 0.0.2
+Version: 0.0.4
 Summary: FFprobe configuration command for deepctl
 Author-email: Deepgram <devrel@deepgram.com>
 Maintainer-email: Deepgram <devrel@deepgram.com>

{deepctl_cmd_ffprobe-0.0.2 → deepctl_cmd_ffprobe-0.0.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "deepctl-cmd-ffprobe"
-version = "0.0.2" # x-release-please-version
+version = "0.0.4" # x-release-please-version
 description = "FFprobe configuration command for deepctl"
 readme = "README.md"
 license = "MIT"

{deepctl_cmd_ffprobe-0.0.2 → deepctl_cmd_ffprobe-0.0.4}/src/deepctl_cmd_ffprobe/command.py

@@ -3,20 +3,43 @@
 from __future__ import annotations
 
 import os
-import shutil
 import subprocess
 from typing import Any
 
-from deepctl_core import AuthManager, BaseCommand, BaseResult, Config, DeepgramClient
+from deepctl_core import (
+    AuthManager,
+    BaseCommand,
+    BaseResult,
+    CodedErrorResult,
+    Config,
+    DeepgramClient,
+    get_console,
+    get_output_format,
+    get_status_console,
+    is_ai_agent,
+)
+from deepctl_core import messages as msg
+from deepctl_core.output import print_error, print_hint
+from deepctl_core.tools import find_executable
 from deepctl_shared_utils import (
     get_ffprobe_path,
     print_ffprobe_install_instructions,
 )
-from rich.console import Console
 
 from .models import FfprobeResult
 
-console = Console()
+# Two channels, deliberately (#104):
+#   console        -> stdout, the human rendering of the FfprobeResult.
+#                     Written only in default (human) output mode, so
+#                     `dg -o json ffprobe | jq` sees the payload alone.
+#   status_console -> stderr, always: errors and install instructions.
+console = get_console()
+status_console = get_status_console()
+
+
+def _human_output() -> bool:
+    """True when stdout is for a person, not a parser."""
+    return get_output_format() == "default"
 
 
 class FfprobeCommand(BaseCommand):
@@ -68,13 +91,15 @@ class FfprobeCommand(BaseCommand):
         auth_manager: AuthManager,
         client: DeepgramClient,
         **kwargs: Any,
-    ) -> BaseResult:
+    ) -> BaseResult | CodedErrorResult:
         """Handle ffprobe command."""
         path = kwargs.get("path")
         reset = kwargs.get("reset", False)
 
         if path and reset:
-            console.print("[red]Error:[/red] Cannot use --path and --reset together")
+            status_console.print(
+                "[red]Error:[/red] Cannot use --path and --reset together"
+            )
             return BaseResult(
                 status="error",
                 message="Cannot use --path and --reset together",
@@ -84,6 +109,17 @@ class FfprobeCommand(BaseCommand):
             return self._handle_reset(config)
 
         if path:
+            # A stored tool path is run by later commands: an AI agent
+            # session never sets it. Scripts and CI without an agent marker
+            # still can.
+            if is_ai_agent():
+                print_error(msg.E_TOOL_PATH_AGENT)
+                print_hint(msg.E_PERSON_ONLY_AGENT_HINT)
+                return CodedErrorResult(
+                    error=msg.E_TOOL_PATH_AGENT,
+                    code=msg.CODE_CONFIRMATION_REQUIRED,
+                    hint=msg.E_PERSON_ONLY_AGENT_HINT,
+                )
             return self._handle_set_path(config, path)
 
         return self._handle_status(config)
@@ -92,21 +128,23 @@ class FfprobeCommand(BaseCommand):
         """Store a custom ffprobe path."""
         # Validate the path
         if not os.path.isfile(path):
-            console.print(f"[red]Error:[/red] File not found: {path}")
+            status_console.print(f"[red]Error:[/red] File not found: {path}")
             return BaseResult(status="error", message=f"File not found: {path}")
 
         if not os.access(path, os.X_OK):
-            console.print(f"[red]Error:[/red] File is not executable: {path}")
+            status_console.print(f"[red]Error:[/red] File is not executable: {path}")
             return BaseResult(status="error", message=f"File is not executable: {path}")
 
-        # Store in config
+        # Store in config. Only absolute paths are used, so resolve it now.
+        path = os.path.abspath(path)
         config._config.tools.ffprobe_path = path
         config.save()
 
         version = self._get_version(path)
-        console.print(f"[green]✓[/green] ffprobe path stored: {path}")
-        if version:
-            console.print(f"  Version: {version}")
+        if _human_output():
+            console.print(f"[green]✓[/green] ffprobe path stored: {path}")
+            if version:
+                console.print(f"  Version: {version}")
 
         return FfprobeResult(
             status="success",
@@ -121,14 +159,14 @@ class FfprobeCommand(BaseCommand):
         config._config.tools.ffprobe_path = None
         config.save()
 
-        console.print("[green]✓[/green] Stored ffprobe path cleared")
-
         # Show auto-detected path
-        auto_path = shutil.which("ffprobe")
-        if auto_path:
-            console.print(f"  Auto-detected: {auto_path}")
-        else:
-            console.print("  [yellow]ffprobe not found in PATH[/yellow]")
+        auto_path = find_executable("ffprobe")
+        if _human_output():
+            console.print("[green]✓[/green] Stored ffprobe path cleared")
+            if auto_path:
+                console.print(f"  Auto-detected: {auto_path}")
+            else:
+                console.print("  [yellow]ffprobe not found in PATH[/yellow]")
 
         return FfprobeResult(
             status="success",
@@ -140,20 +178,22 @@ class FfprobeCommand(BaseCommand):
     def _handle_status(self, config: Config) -> BaseResult:
         """Show ffprobe status."""
         stored_path = config.get("tools.ffprobe_path")
-        auto_path = shutil.which("ffprobe")
+        auto_path = find_executable("ffprobe")
         effective_path = get_ffprobe_path(config)
 
-        if stored_path:
-            console.print(f"  Stored path:    {stored_path}")
-        if auto_path:
-            console.print(f"  Auto-detected:  {auto_path}")
+        if _human_output():
+            if stored_path:
+                console.print(f"  Stored path:    {stored_path}")
+            if auto_path:
+                console.print(f"  Auto-detected:  {auto_path}")
 
         if effective_path:
             version = self._get_version(effective_path)
-            console.print(f"  Active path:    [green]{effective_path}[/green]")
-            if version:
-                console.print(f"  Version:        {version}")
-            console.print("\n[green]✓[/green] ffprobe is available")
+            if _human_output():
+                console.print(f"  Active path:    [green]{effective_path}[/green]")
+                if version:
+                    console.print(f"  Version:        {version}")
+                console.print("\n[green]✓[/green] ffprobe is available")
 
             return FfprobeResult(
                 status="success",

{deepctl_cmd_ffprobe-0.0.2 → deepctl_cmd_ffprobe-0.0.4}/src/deepctl_cmd_ffprobe.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepctl-cmd-ffprobe
-Version: 0.0.2
+Version: 0.0.4
 Summary: FFprobe configuration command for deepctl
 Author-email: Deepgram <devrel@deepgram.com>
 Maintainer-email: Deepgram <devrel@deepgram.com>