doover-cli 0.4.1__tar.gz → 0.4.2__tar.gz

{doover_cli-0.4.1 → doover_cli-0.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: doover-cli
-Version: 0.4.1
+Version: 0.4.2
 Summary: CLI for Doover
 License-File: LICENSE
 Requires-Python: >=3.11

{doover_cli-0.4.1 → doover_cli-0.4.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "doover-cli"
-version = "0.4.1"
+version = "0.4.2"
 description = "CLI for Doover"
 requires-python = ">=3.11"
 dependencies = [

{doover_cli-0.4.1 → doover_cli-0.4.2}/src/doover_cli/apps/apps.py

@@ -104,6 +104,21 @@ def _resolve_staging(staging: bool | None) -> bool:
     return ".staging." in _control_base_url()
 
 
+def _detect_git_commit(root_fp: Path) -> str:
+    """Best-effort current git commit SHA for the app dir (empty if unavailable)."""
+    try:
+        result = subprocess.run(
+            ["git", "rev-parse", "HEAD"],
+            cwd=root_fp,
+            capture_output=True,
+            text=True,
+            check=False,
+        )
+    except OSError:
+        return ""
+    return result.stdout.strip() if result.returncode == 0 else ""
+
+
 def _build_container(
     root_fp: Path, *, buildx: bool, build_args: str, image_name: str
 ) -> None:
@@ -1087,7 +1102,13 @@ def release(
     ] = None,
     tag: Annotated[
         str | None,
-        typer.Option(help="Optional version label (e.g. git sha / semver)."),
+        typer.Option(help="Optional version label (e.g. semver / image tag)."),
+    ] = None,
+    commit: Annotated[
+        str | None,
+        typer.Option(
+            help="Source git commit SHA (for browsing the code). Defaults to the repo's HEAD."
+        ),
     ] = None,
     notes: Annotated[str, typer.Option(help="Optional release notes.")] = "",
     staging: Annotated[
@@ -1124,12 +1145,15 @@ def release(
         )
         raise typer.Exit(1)
 
+    resolved_commit = commit if commit is not None else _detect_git_commit(root_fp)
+
     client, renderer = get_state()
     with renderer.loading("Creating application version..."):
         version = client.create_application_version(
             application_id,
             digest=digest,
             tag=tag or "",
+            commit=resolved_commit or "",
             notes=notes or "",
         )
     rich.print("[green]Released a new application version.[/green]")

{doover_cli-0.4.1 → doover_cli-0.4.2}/src/doover_cli/utils/apps.py

@@ -306,16 +306,20 @@ def get_app_directory(root: Path | None = None) -> Path:
 
 
 def get_uv_path() -> Path:
-    brew = Path("/opt/homebrew/bin/uv")
-    if brew.exists():
-        return brew
-
-    uv_path = Path.home() / ".local" / "bin" / "uv"
-    if not uv_path.exists():
-        raise RuntimeError(
-            "uv not found in your PATH. Please install it and try again."
-        )
-    return uv_path
+    # Prefer uv on PATH (covers CI where uv is installed to a tool cache dir),
+    # then common fixed locations.
+    found = shutil.which("uv")
+    if found:
+        return Path(found)
+
+    for candidate in (
+        Path("/opt/homebrew/bin/uv"),
+        Path.home() / ".local" / "bin" / "uv",
+    ):
+        if candidate.exists():
+            return candidate
+
+    raise RuntimeError("uv not found in your PATH. Please install it and try again.")
 
 
 def call_with_uv(

{doover_cli-0.4.1 → doover_cli-0.4.2}/tests/test_apps.py

@@ -662,7 +662,9 @@ def test_app_publish_processor_type_without_ui_skips_ui_export(monkeypatch, tmp_
     assert "export_ui_called" not in captured
     assert captured["build_script"] == ("./build.sh", tmp_path)
     assert captured["processor_source"] == ("303", {"file": package_fp})
-    assert renderer.render_calls == [{"id": 303, "versioned": True}]
+    # publish uploads the package but no longer cuts a version (that's `release`)
+    assert "processor_version" not in captured
+    assert renderer.render_calls == [{"id": 303}]
 
 
 def test_app_publish_skips_ui_export_when_config_disables_generation(
@@ -769,9 +771,7 @@ def test_app_publish_honours_explicit_staging(monkeypatch, tmp_path):
         lambda ctx, app_fp, validate_: None,
     )
 
-    result = runner.invoke(
-        app, ["app", "publish", str(tmp_path), "--staging"]
-    )
+    result = runner.invoke(app, ["app", "publish", str(tmp_path), "--staging"])
 
     assert result.exit_code == 0
     assert captured["body"]["deployment_data"] == "staging-data"
@@ -872,9 +872,7 @@ def test_app_publish_skips_build_when_requested_by_config(monkeypatch, tmp_path)
     assert renderer.render_calls == [{"id": 101}]
 
 
-def test_app_publish_processor_builds_package_and_releases_version(
-    monkeypatch, tmp_path
-):
+def test_app_publish_processor_builds_package_without_release(monkeypatch, tmp_path):
     captured = {}
     renderer = FakeRenderer()
     app_config = FakeAppConfig(app_id=303)
@@ -938,10 +936,11 @@ def test_app_publish_processor_builds_package_and_releases_version(
     assert captured["build_script"] == ("./build.sh", tmp_path)
     assert captured["application_id"] == "303"
     assert captured["body"] == {"file": package_fp}
-    assert captured["version"][0] == "303"
+    # publish uploads the package; the version is cut by `doover app release`
+    assert "version" not in captured
     assert "build_called" not in captured
     assert "push_called" not in captured
-    assert renderer.render_calls == [{"id": 303, "versioned": True}]
+    assert renderer.render_calls == [{"id": 303}]
 
 
 def test_publish_processor_package_uses_package_zip(tmp_path):

{doover_cli-0.4.1 → doover_cli-0.4.2}/uv.lock

@@ -544,7 +544,7 @@ wheels = [
 
 [[package]]
 name = "doover-cli"
-version = "0.4.0"
+version = "0.4.2"
 source = { editable = "." }
 dependencies = [
     { name = "click" },