fix(installer): include click runtime dependency (#1299)

Author: Q00

pyproject.toml

@@ -10,6 +10,7 @@ requires-python = ">=3.12"
 dependencies = [
     "aiosqlite>=0.20.0,<1.0.0",
     "anyio>=4.0.0,<5.0.0",
+    "click>=8.1.0,<9.0.0",
     "jsonschema>=4.21.0,<5.0.0",
     "pydantic>=2.0.0,<3.0.0",
     "prompt-toolkit>=3.0.0,<4.0.0",

scripts/install.sh

@@ -12,6 +12,7 @@
 set -euo pipefail
 
 PACKAGE_NAME="ouroboros-ai"
+CLICK_SPEC="click>=8.1.0,<9.0.0"
 MIN_PYTHON="3.12"
 IS_LOCAL=false
 RECONFIGURE="${OUROBOROS_INSTALL_RECONFIGURE:-}"
@@ -333,7 +334,9 @@ echo "Installing ${INSTALL_SPEC} ..."
 INSTALL_METHOD=""
 if [ "$HAS_UV" = true ]; then
   INSTALL_METHOD="uv"
-  UV_ARGS=(tool install --upgrade --python ">=3.12" "$PACKAGE_NAME")
+  # `click` is also declared in pyproject, but keep this explicit so the
+  # installer can repair already-published wheels whose metadata missed it.
+  UV_ARGS=(tool install --upgrade --python ">=3.12" "$PACKAGE_NAME" --with "$CLICK_SPEC")
   if [ -n "$PRE_FLAG" ]; then
     UV_ARGS+=(--prerelease=allow)
   fi
@@ -375,12 +378,14 @@ elif [ "$HAS_PIPX" = true ]; then
   else
     pipx install --force --python "$PYTHON" "$INSTALL_SPEC"
   fi
+  # The venv name is the distribution name even when installing from a local path.
+  pipx inject "ouroboros-ai" "$CLICK_SPEC"
 else
   INSTALL_METHOD="pip"
   if [ -n "$PRE_FLAG" ]; then
-    $PYTHON -m pip install --user --upgrade --pre "$INSTALL_SPEC"
+    $PYTHON -m pip install --user --upgrade --pre "$INSTALL_SPEC" "$CLICK_SPEC"
   else
-    $PYTHON -m pip install --user --upgrade "$INSTALL_SPEC"
+    $PYTHON -m pip install --user --upgrade "$INSTALL_SPEC" "$CLICK_SPEC"
   fi
 fi
 

tests/unit/scripts/test_install_runtime_selection.py

@@ -98,7 +98,7 @@ def test_preserves_opencode_backend_from_existing_config(tmp_path: Path) -> None
     assert "Runtime: opencode (preserved from" in result.stdout
     assert "Installing . ..." in result.stdout
     assert (tmp_path / "calls.log").read_text(encoding="utf-8").splitlines() == [
-        "uv tool install --upgrade --python >=3.12 .",
+        "uv tool install --upgrade --python >=3.12 . --with click>=8.1.0,<9.0.0",
         "ouroboros setup --runtime opencode --non-interactive",
     ]
 
@@ -114,7 +114,7 @@ def test_explicit_claude_installs_mcp_and_claude_extras(tmp_path: Path) -> None:
     calls = (tmp_path / "calls.log").read_text(encoding="utf-8")
     assert "Runtime: claude (from --runtime / OUROBOROS_INSTALL_RUNTIME)" in result.stdout
     assert (
-        "uv tool install --upgrade --python >=3.12 . --with mcp>=1.26.0,<2.0.0 --with claude-agent-sdk>=0.1.0,<1.0.0 --with anthropic>=0.52.0,<1.0.0"
+        "uv tool install --upgrade --python >=3.12 . --with click>=8.1.0,<9.0.0 --with mcp>=1.26.0,<2.0.0 --with claude-agent-sdk>=0.1.0,<1.0.0 --with anthropic>=0.52.0,<1.0.0"
         in calls
     )
     assert "ouroboros setup --runtime claude --non-interactive" in calls
@@ -130,7 +130,9 @@ def test_pypi_lookup_failure_stays_stable_only_for_remote_install(tmp_path: Path
 
     assert result.returncode == 0, result.stderr
     calls = (tmp_path / "calls.log").read_text(encoding="utf-8")
-    assert "uv tool install --upgrade --python >=3.12 ouroboros-ai" in calls
+    assert (
+        "uv tool install --upgrade --python >=3.12 ouroboros-ai --with click>=8.1.0,<9.0.0" in calls
+    )
     assert "--prerelease=allow" not in calls