doover-cli 0.5.3__tar.gz → 0.5.4__tar.gz

{doover_cli-0.5.3 → doover_cli-0.5.4}/PKG-INFO

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

{doover_cli-0.5.3 → doover_cli-0.5.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "doover-cli"
-version = "0.5.3"
+dynamic = ["version"]
 description = "CLI for Doover"
 requires-python = ">=3.11"
 dependencies = [
@@ -26,6 +26,9 @@ dependencies = [
 requires = ["hatchling"]
 build-backend = "hatchling.build"
 
+[tool.hatch.version]
+path = "src/doover_cli/__init__.py"
+
 [project.scripts]
 doover = "doover_cli:main"
 

{doover_cli-0.5.3 → doover_cli-0.5.4}/src/doover_cli/__init__.py

@@ -1,6 +1,8 @@
 from doover_cli.renderer import Renderer
 from typing import Annotated, Optional
 
+__version__ = "0.5.4"
+
 import click
 import typer
 

{doover_cli-0.5.3 → doover_cli-0.5.4}/src/doover_cli/apps/apps.py

@@ -1007,6 +1007,16 @@ def publish(
 
     client, renderer = get_state()
 
+    # The publish endpoints are organisation-scoped: without the
+    # X-Doover-Organisation header the control plane rejects the request with
+    # 401 "X-Doover-Organisation header missing" for any non-staff account.
+    # Pin the client to the app's organisation (staging-aware, resolved into the
+    # payload above) so every publish sub-call - create/partial, widget upload,
+    # processor source + version release - carries the header.
+    publish_org_id = payload.get("organisation_id")
+    if publish_org_id is not None:
+        client.organisation_id = int(publish_org_id)
+
     rich.print(
         f"Updating application on doover site ({_control_base_url() or 'unknown base URL'})...\n"
     )

{doover_cli-0.5.3 → doover_cli-0.5.4}/tests/test_apps.py

@@ -491,6 +491,51 @@ def test_app_publish_updates_existing_application(monkeypatch, tmp_path):
     assert renderer.render_calls == [{"id": 101, "published": True}]
 
 
+def test_app_publish_pins_client_to_app_organisation(monkeypatch, tmp_path):
+    # Regression: publish must set the org context on the client so the
+    # X-Doover-Organisation header is sent. Without it the control plane
+    # returns 401 "X-Doover-Organisation header missing" for non-staff accounts.
+    captured = {}
+    renderer = FakeRenderer()
+    app_config = FakeAppConfig(app_id=101)
+
+    class FakeApplicationsClient:
+        @staticmethod
+        def partial(application_id, body):
+            captured["partial"] = (application_id, body)
+            return {"id": int(application_id), "published": True}
+
+    class FakeControlClient:
+        organisation_id = None
+        applications = FakeApplicationsClient()
+
+    client = FakeControlClient()
+
+    monkeypatch.setattr(
+        "doover_cli.apps.apps.get_app_directory", lambda root=None: tmp_path
+    )
+    monkeypatch.setattr(
+        "doover_cli.apps.apps.get_app_config", lambda root_fp, app_name=None: app_config
+    )
+    monkeypatch.setattr(
+        "doover_cli.apps.apps.get_state", lambda: (client, renderer)
+    )
+    monkeypatch.setattr(
+        "doover_cli.apps.apps.export_config_command",
+        lambda ctx, app_fp, validate_: None,
+    )
+    monkeypatch.setattr(
+        "doover_cli.apps.apps.export_ui_command",
+        lambda ctx, app_fp, validate_: None,
+    )
+
+    result = runner.invoke(app, ["app", "publish", str(tmp_path)])
+
+    assert result.exit_code == 0
+    assert client.organisation_id == 17
+    assert captured["partial"][0] == "101"
+
+
 def test_app_publish_creates_then_updates_when_missing(monkeypatch, tmp_path):
     captured = {}
     renderer = FakeRenderer()

{doover_cli-0.5.3 → doover_cli-0.5.4}/uv.lock

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