fix(cli,upgrade): handle runtime lazy-command errors and uv tool detection

Author: djm81

src/specfact_cli/cli.py

@@ -559,9 +559,10 @@ def _invoke(args: tuple[str, ...]) -> None:
             resolved_name = resolve_command(cmd_name)
             try:
                 real_typer = CommandRegistry.get_typer(resolved_name)
-            except ValueError as exc:
+                click_cmd = get_command(real_typer)
+            except (RuntimeError, ValueError) as exc:
+                click.echo(str(exc), err=True)
                 raise click.ClickException(str(exc)) from exc
-            click_cmd = get_command(real_typer)
             # Build full prog name from root (e.g. "specfact sync") so usage shows "specfact sync bridge", not "sync sync bridge"
             parts: list[str] = []
             p = ctx.parent
@@ -626,9 +627,9 @@ def _get_real_click_group(self) -> click.Group | None:
         resolved_name = resolve_command(self._lazy_cmd_name)
         try:
             real_typer = CommandRegistry.get_typer(resolved_name)
-        except ValueError:
+            click_cmd = get_command(real_typer)
+        except (RuntimeError, ValueError):
             return None
-        click_cmd = get_command(real_typer)
         if isinstance(click_cmd, click.Group):
             return click_cmd
         return None
@@ -641,9 +642,9 @@ def format_help(self, ctx: click.Context, formatter: click.HelpFormatter) -> Non
         resolved_name = resolve_command(self._lazy_cmd_name)
         try:
             real_typer = CommandRegistry.get_typer(resolved_name)
-        except ValueError:
+            click_cmd = get_command(real_typer)
+        except (RuntimeError, ValueError):
             return
-        click_cmd = get_command(real_typer)
         prog_name = (
             f"{ctx.parent.command.name} {self._lazy_cmd_name}"
             if ctx.parent and ctx.parent.command

src/specfact_cli/modules/upgrade/src/commands.py

@@ -98,7 +98,17 @@ def _detect_uv_installation(executable_path: str) -> InstallationMethod | None:
                 command="uv pip install --upgrade specfact-cli",
                 location=str(Path(executable_path).parent.parent),
             )
-    if Path(sys.executable).name in {"uv", "uv.exe"}:
+    try:
+        result = subprocess.run(
+            ["uv", "tool", "list"],
+            capture_output=True,
+            text=True,
+            timeout=5,
+            check=False,
+        )
+    except (subprocess.TimeoutExpired, FileNotFoundError):
+        return None
+    if "specfact-cli" in result.stdout:
         return InstallationMethod(method="uv", command="uv tool upgrade specfact-cli", location=None)
     return None
 

tests/unit/commands/test_update.py

@@ -96,6 +96,35 @@ def side_effect(*args, **kwargs):
         # Should detect pipx first (before checking pip)
         assert method.method == "pipx", f"Expected pipx, got {method.method}"
 
+
+
+    @patch("specfact_cli.modules.upgrade.src.commands.subprocess.run")
+    @patch("specfact_cli.modules.upgrade.src.commands.sys.executable", "/usr/bin/python3")
+    @patch("specfact_cli.modules.upgrade.src.commands.sys.argv", ["/usr/bin/python3", "-m", "specfact_cli"])
+    def test_detect_uv_tool_installation(self, mock_subprocess: MagicMock) -> None:
+        """Test detecting uv tool installation via `uv tool list`."""
+
+        def side_effect(*args, **kwargs):
+            result = MagicMock()
+            cmd = args[0] if args else []
+            cmd_str = " ".join(cmd) if isinstance(cmd, list) else str(cmd)
+            if "pipx list" in cmd_str:
+                result.returncode = 1
+                result.stdout = ""
+            elif "uv tool list" in cmd_str:
+                result.returncode = 0
+                result.stdout = "specfact-cli v0.46.10"
+            else:
+                result.returncode = 1
+                result.stdout = ""
+            return result
+
+        mock_subprocess.side_effect = side_effect
+
+        method = detect_installation_method()
+        assert method.method == "uv"
+        assert method.command == "uv tool upgrade specfact-cli"
+
     @patch("specfact_cli.modules.upgrade.src.commands.subprocess.run")
     @patch("specfact_cli.modules.upgrade.src.commands.sys.executable", "/usr/bin/python3")
     @patch("specfact_cli.modules.upgrade.src.commands.sys.argv", ["/usr/bin/python3", "-m", "specfact_cli"])