validio-cli 8.1.0__tar.gz → 8.2.1__tar.gz

{validio_cli-8.1.0 → validio_cli-8.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: validio-cli
-Version: 8.1.0
+Version: 8.2.1
 Summary: CLI tool to interact with the Validio platform
 License: Apache-2.0
 License-File: LICENSE

{validio_cli-8.1.0 → validio_cli-8.2.1}/pyproject.toml

@@ -3,7 +3,7 @@ name = "validio-cli"
 # This version does not represent the released version or any tag. For each
 # release we automatically bump this before building and publishing so this
 # should be kept at 0.0.1dev1
-version = "8.1.0"
+version = "8.2.1"
 description = "CLI tool to interact with the Validio platform"
 authors = ["Validio <support@validio.io>"]
 license = "Apache-2.0"

{validio_cli-8.1.0 → validio_cli-8.2.1}/validio_cli/bin/entities/namespaces.py

@@ -64,7 +64,7 @@ async def create(
     role: str = typer.Option(
         None,
         click_type=RoleSuggestion(),
-        help="The role to assign --api-key and --member",
+        help="The role (name or id) to assign --api-key and --member",
     ),
     api_key: list[str] = typer.Option(
         [],
@@ -113,6 +113,8 @@ async def create(
         api_keys=ns_api_keys,
     )
 
+    _raise_on_errors(namespaces)
+
     if output_format == OutputFormat.JSON:
         return output_json(namespaces, identifier)
 
@@ -133,7 +135,7 @@ async def update(
     role: str = typer.Option(
         None,
         click_type=RoleSuggestion(),
-        help="The role to assign --api-key and --member",
+        help="The role (name or id) to assign --api-key and --member",
     ),
     api_key: list[str] = typer.Option(
         [],
@@ -198,6 +200,8 @@ async def update(
         api_keys=api_keys,
     )
 
+    _raise_on_errors(namespaces)
+
     if output_format == OutputFormat.JSON:
         return output_json(namespaces, identifier)
 
@@ -234,11 +238,16 @@ async def remove(
 
 
 def _find_role_id(role_configs: list[dict[str, Any]], role_name: str) -> str:
+    # Accept either the role id or a case-insensitive role name — the server
+    # returns `role` as a scalar on existing members/apiKeys and the format is
+    # not guaranteed, so we tolerate both.
     for rc in role_configs:
-        if rc["name"].upper() == role_name.upper():
+        if rc["id"] == role_name or rc["name"].casefold() == role_name.casefold():
             return rc["id"]
 
-    raise ValidioError(f"role '{role_name}' not found")
+    available = ", ".join(sorted(rc["name"] for rc in role_configs))
+
+    raise ValidioError(f"role '{role_name}' not found. Available roles: {available}")
 
 
 def _find_admin_role_id(role_configs: list[dict[str, Any]]) -> str:
@@ -247,3 +256,18 @@ def _find_admin_role_id(role_configs: list[dict[str, Any]]) -> str:
             return rc["id"]
 
     raise ValidioError("could not find system Admin role for namespace")
+
+
+def _raise_on_errors(response: Any) -> None:
+    if not isinstance(response, dict):
+        return
+
+    errors = response.get("errors") or []
+    if not errors:
+        return
+
+    messages = "; ".join(
+        f"{e.get('code', 'UNKNOWN')}: {e.get('message', '')}" for e in errors
+    )
+
+    raise ValidioError(f"server returned errors: {messages}")