agent-cli 0.82.2__tar.gz → 0.82.3__tar.gz

{agent_cli-0.82.2 → agent_cli-0.82.3}/.github/workflows/docker.yml

@@ -51,10 +51,10 @@ jobs:
           fetch-depth: 0  # Full history for version detection
 
       - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v3
+        uses: docker/setup-buildx-action@v4
 
       - name: Log in to Container Registry
-        uses: docker/login-action@v3
+        uses: docker/login-action@v4
         with:
           registry: ${{ env.REGISTRY }}
           username: ${{ github.actor }}
@@ -71,7 +71,7 @@ jobs:
 
       - name: Extract metadata
         id: meta
-        uses: docker/metadata-action@v5
+        uses: docker/metadata-action@v6
         with:
           images: ${{ env.REGISTRY }}/${{ github.repository }}-${{ matrix.image }}
           tags: |
@@ -80,7 +80,7 @@ jobs:
             type=sha,suffix=-${{ matrix.suffix }}
 
       - name: Build and push
-        uses: docker/build-push-action@v6
+        uses: docker/build-push-action@v7
         with:
           context: .
           file: ${{ matrix.dockerfile }}
@@ -116,10 +116,10 @@ jobs:
           fetch-depth: 0  # Full history for version detection
 
       - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v3
+        uses: docker/setup-buildx-action@v4
 
       - name: Log in to Container Registry
-        uses: docker/login-action@v3
+        uses: docker/login-action@v4
         with:
           registry: ${{ env.REGISTRY }}
           username: ${{ github.actor }}
@@ -136,7 +136,7 @@ jobs:
 
       - name: Extract metadata
         id: meta
-        uses: docker/metadata-action@v5
+        uses: docker/metadata-action@v6
         with:
           images: ${{ env.REGISTRY }}/${{ github.repository }}-${{ matrix.image }}
           tags: |
@@ -145,7 +145,7 @@ jobs:
             type=sha
 
       - name: Build and push
-        uses: docker/build-push-action@v6
+        uses: docker/build-push-action@v7
         with:
           context: .
           file: ${{ matrix.dockerfile }}

{agent_cli-0.82.2 → agent_cli-0.82.3}/.pre-commit-config.yaml

@@ -11,7 +11,7 @@ repos:
       - id: end-of-file-fixer
       - id: mixed-line-ending
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.15.2"
+    rev: "v0.15.5"
     hooks:
       - id: ruff
         args: ["--fix"]

{agent_cli-0.82.2 → agent_cli-0.82.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agent-cli
-Version: 0.82.2
+Version: 0.82.3
 Summary: A suite of AI-powered command-line tools for text correction, audio transcription, and voice assistance.
 Project-URL: Homepage, https://github.com/basnijholt/agent-cli
 Author-email: Bas Nijholt <bas@nijho.lt>

{agent_cli-0.82.2 → agent_cli-0.82.3}/agent_cli/agents/assistant.py

@@ -230,7 +230,7 @@ async def _async_main(
 
 
 @app.command("assistant", rich_help_panel="Voice Commands")
-@requires_extras("audio", "llm")
+@requires_extras("audio", "llm", process_name="assistant")
 def assistant(
     *,
     # --- Provider Selection ---

{agent_cli-0.82.2 → agent_cli-0.82.3}/agent_cli/agents/chat.py

@@ -377,7 +377,7 @@ async def _async_main(
 
 
 @app.command("chat", rich_help_panel="Voice Commands")
-@requires_extras("audio", "llm")
+@requires_extras("audio", "llm", process_name="chat")
 def chat(
     *,
     # --- Provider Selection ---

{agent_cli-0.82.2 → agent_cli-0.82.3}/agent_cli/agents/speak.py

@@ -81,7 +81,7 @@ async def _async_main(
 
 
 @app.command("speak", rich_help_panel="Text Commands")
-@requires_extras("audio")
+@requires_extras("audio", process_name="speak")
 def speak(
     *,
     text: str | None = typer.Argument(

{agent_cli-0.82.2 → agent_cli-0.82.3}/agent_cli/agents/transcribe.py

@@ -465,7 +465,7 @@ async def _async_main(  # noqa: PLR0912, PLR0915, C901
 
 
 @app.command("transcribe", rich_help_panel="Voice Commands")
-@requires_extras("audio", "llm")
+@requires_extras("audio", "llm", process_name="transcribe")
 def transcribe(  # noqa: PLR0912
     *,
     extra_instructions: str | None = typer.Option(

{agent_cli-0.82.2 → agent_cli-0.82.3}/agent_cli/agents/transcribe_live.py

@@ -288,7 +288,7 @@ async def _daemon_loop(cfg: DaemonConfig) -> None:  # noqa: PLR0912, PLR0915
 
 
 @app.command("transcribe-live", rich_help_panel="Voice Commands")
-@requires_extras("audio", "vad", "llm")
+@requires_extras("audio", "vad", "llm", process_name="transcribe-live")
 def transcribe_live(  # noqa: PLR0912
     *,
     # Daemon-specific options

{agent_cli-0.82.2 → agent_cli-0.82.3}/agent_cli/agents/voice_edit.py

@@ -174,7 +174,7 @@ async def _async_main(
 
 
 @app.command("voice-edit", rich_help_panel="Voice Commands")
-@requires_extras("audio", "llm")
+@requires_extras("audio", "llm", process_name="voice-edit")
 def voice_edit(
     *,
     # --- Provider Selection ---

{agent_cli-0.82.2 → agent_cli-0.82.3}/agent_cli/core/deps.py

@@ -311,8 +311,12 @@ def _try_auto_install(missing_display: list[str], extras_to_install: list[str])
 
 def _maybe_reexec_after_install() -> None:
     """Re-execute after auto-install so new packages are visible."""
-    executable = shutil.which("agent-cli") or sys.executable
-    _maybe_exec_with_marker([executable, *sys.argv[1:]], "Re-running with installed extras...")
+    argv0 = sys.argv[0] if sys.argv else ""
+    if argv0:
+        cmd = [argv0, *sys.argv[1:]]
+    else:
+        cmd = [sys.executable, "-m", "agent_cli.cli", *sys.argv[1:]]
+    _maybe_exec_with_marker(cmd, "Re-running with installed extras...")
 
 
 # -- Main Orchestration --
@@ -362,7 +366,26 @@ def _check_and_install_extras(extras: tuple[str, ...]) -> list[str]:
 # -- Decorator --
 
 
-def requires_extras(*extras: str) -> Callable[[F], F]:
+def _should_skip_extra_check_for_process_control(
+    kwargs: dict[str, object],
+    process_name: str | None,
+) -> bool:
+    """Skip dependency checks when the command only needs process control."""
+    if not process_name:
+        return False
+
+    if bool(kwargs.get("stop")) or bool(kwargs.get("status")):
+        return True
+
+    if not bool(kwargs.get("toggle")):
+        return False
+
+    from agent_cli.core import process  # noqa: PLC0415
+
+    return process.is_process_running(process_name)
+
+
+def requires_extras(*extras: str, process_name: str | None = None) -> Callable[[F], F]:
     """Decorator to declare required extras for a command.
 
     Auto-installs missing extras by default. Disable via AGENT_CLI_NO_AUTO_INSTALL=1
@@ -374,11 +397,14 @@ def requires_extras(*extras: str) -> Callable[[F], F]:
 
         @functools.wraps(func)
         def wrapper(*args: object, **kwargs: object) -> object:
+            if _should_skip_extra_check_for_process_control(kwargs, process_name):
+                return func(*args, **kwargs)
             if _check_and_install_extras(extras):
                 raise typer.Exit(1)
             return func(*args, **kwargs)
 
         wrapper._required_extras = extras  # type: ignore[attr-defined]
+        wrapper._required_process_name = process_name  # type: ignore[attr-defined]
         return wrapper  # type: ignore[return-value]
 
     return decorator

{agent_cli-0.82.2 → agent_cli-0.82.3}/agent_cli/install/hotkeys.py

@@ -4,8 +4,11 @@ from __future__ import annotations
 
 import platform
 
+import typer
+
 from agent_cli.cli import app
-from agent_cli.core.utils import print_with_style
+from agent_cli.core.deps import _check_extra_installed, install_extras_impl
+from agent_cli.core.utils import print_error_message, print_with_style
 from agent_cli.install.common import execute_installation_script, get_platform_script
 
 
@@ -39,6 +42,19 @@ def install_hotkeys() -> None:
     **Customizing hotkeys** (macOS): Edit `~/.config/skhd/skhdrc` and restart skhd:
     `skhd --restart-service`
     """
+    required_extras = ["audio", "llm"]
+    missing_extras = [extra for extra in required_extras if not _check_extra_installed(extra)]
+    if missing_extras:
+        print_with_style(
+            f"Installing required extras for hotkeys: {', '.join(missing_extras)}",
+            "cyan",
+        )
+        if not install_extras_impl(missing_extras):
+            print_error_message(
+                "Failed to install required extras for hotkeys: " + ", ".join(missing_extras),
+            )
+            raise typer.Exit(1)
+
     script_name = get_platform_script("setup-macos-hotkeys.sh", "setup-linux-hotkeys.sh")
     system = platform.system().lower()
 

{agent_cli-0.82.2 → agent_cli-0.82.3}/agent_cli/scripts/macos-hotkeys/toggle-transcription.sh

@@ -34,7 +34,7 @@ if is_transcribe_running; then
     "$NOTIFIER" -remove "$RECORDING_GROUP" >/dev/null 2>&1 || true
     notify_temp "🛑 Stopped" "Processing results..."
 else
-    "$NOTIFIER" -title "🎙️ Started" -message "Listening..." -group "$RECORDING_GROUP"
+    "$NOTIFIER" -title "🎙️ Starting" -message "Preparing transcription..." -group "$RECORDING_GROUP"
     (
         OUTPUT=$("$AGENT_CLI" transcribe --toggle --llm --quiet 2>/dev/null)
         "$NOTIFIER" -remove "$RECORDING_GROUP" >/dev/null 2>&1 || true

{agent_cli-0.82.2 → agent_cli-0.82.3}/tests/test_requires_extras.py

@@ -13,7 +13,9 @@ from agent_cli.core.deps import (
     _check_and_install_extras,
     _get_auto_install_setting,
     _get_install_hint,
+    _maybe_reexec_after_install,
     _resolve_extras_for_install,
+    _should_skip_extra_check_for_process_control,
     requires_extras,
 )
 
@@ -219,6 +221,77 @@ class TestCheckAndInstallExtras:
             assert "not visible" in mock_error.call_args[0][0]
 
 
+class TestReexecBehavior:
+    """Test re-exec behavior after auto-install."""
+
+    def test_reexec_uses_original_argv0(self) -> None:
+        """Re-exec should preserve the command path that launched agent-cli."""
+        with (
+            patch("agent_cli.core.deps._maybe_exec_with_marker") as mock_reexec,
+            patch("sys.argv", ["/custom/bin/agent-cli", "transcribe", "--toggle"]),
+        ):
+            _maybe_reexec_after_install()
+
+        mock_reexec.assert_called_once_with(
+            ["/custom/bin/agent-cli", "transcribe", "--toggle"],
+            "Re-running with installed extras...",
+        )
+
+    def test_reexec_falls_back_to_python_module_when_argv0_missing(self) -> None:
+        """Re-exec should still work if argv[0] is unavailable."""
+        with (
+            patch("agent_cli.core.deps._maybe_exec_with_marker") as mock_reexec,
+            patch("sys.argv", ["", "transcribe", "--toggle"]),
+            patch("sys.executable", "/usr/bin/python3"),
+        ):
+            _maybe_reexec_after_install()
+
+        mock_reexec.assert_called_once_with(
+            ["/usr/bin/python3", "-m", "agent_cli.cli", "transcribe", "--toggle"],
+            "Re-running with installed extras...",
+        )
+
+
+class TestProcessControlBypass:
+    """Test process-control bypasses for dependency checks."""
+
+    def test_skip_for_stop_and_status(self) -> None:
+        """Stop/status commands should not trigger extra checks."""
+        assert _should_skip_extra_check_for_process_control({"stop": True}, "transcribe") is True
+        assert _should_skip_extra_check_for_process_control({"status": True}, "transcribe") is True
+
+    def test_skip_for_toggle_when_process_running(self) -> None:
+        """Toggle should bypass extra checks when it is acting as stop."""
+        with patch("agent_cli.core.process.is_process_running", return_value=True):
+            assert (
+                _should_skip_extra_check_for_process_control({"toggle": True}, "transcribe") is True
+            )
+
+    def test_no_skip_for_toggle_when_process_not_running(self) -> None:
+        """Toggle start still needs dependency checks."""
+        with patch("agent_cli.core.process.is_process_running", return_value=False):
+            assert (
+                _should_skip_extra_check_for_process_control({"toggle": True}, "transcribe")
+                is False
+            )
+
+    def test_decorator_skips_extra_checks_for_toggle_stop(self) -> None:
+        """Decorated commands should stop an existing process without installing extras."""
+        with (
+            patch("agent_cli.core.process.is_process_running", return_value=True),
+            patch("agent_cli.core.deps._check_and_install_extras") as mock_check,
+        ):
+
+            @requires_extras("audio", process_name="transcribe")
+            def my_command(**kwargs: object) -> str:
+                assert kwargs["toggle"] is True
+                return "stopped"
+
+            assert my_command(toggle=True) == "stopped"
+
+        mock_check.assert_not_called()
+
+
 class TestDecoratorIntegration:
     """Test the requires_extras decorator end-to-end behavior."""