devnomads-cli 0.5.1__tar.gz → 0.5.2__tar.gz

{devnomads_cli-0.5.1 → devnomads_cli-0.5.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devnomads-cli
-Version: 0.5.1
+Version: 0.5.2
 Summary: Manage your DevNomads services from the command line
 Author-email: DevNomads <support@devnomads.nl>
 License: MIT

{devnomads_cli-0.5.1 → devnomads_cli-0.5.2}/devnomads_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devnomads-cli
-Version: 0.5.1
+Version: 0.5.2
 Summary: Manage your DevNomads services from the command line
 Author-email: DevNomads <support@devnomads.nl>
 License: MIT

{devnomads_cli-0.5.1 → devnomads_cli-0.5.2}/dncli.py

@@ -211,16 +211,44 @@ def render(
         out_console.print(str(data), soft_wrap=True)
 
 
+def _summarize_item(item: dict[str, Any]) -> str:
+    """Compact one-line ``key=value`` view of an object's scalar fields for a
+    table cell. Nested lists/objects collapse to a count so the line stays
+    short; None and empty fields are dropped."""
+
+    parts = []
+    for key, val in item.items():
+        if val is None or val == "":
+            continue
+        if isinstance(val, bool):
+            parts.append(f"{key}={'yes' if val else 'no'}")
+        elif isinstance(val, dict):
+            if val:
+                parts.append(f"{key}={{{len(val)}}}")
+        elif isinstance(val, list):
+            if val:
+                parts.append(f"{key}=[{len(val)}]")
+        else:
+            parts.append(f"{key}={val}")
+    return " ".join(parts)
+
+
 def _cell(value: Any) -> str:
     if value is None:
         return ""
     if isinstance(value, bool):
         return "yes" if value else "no"
-    if isinstance(value, list) and not any(
-        isinstance(item, (dict, list)) for item in value
-    ):
-        return ", ".join(str(item) for item in value)
-    if isinstance(value, (dict, list)):
+    if isinstance(value, dict):
+        return _summarize_item(value)
+    if isinstance(value, list):
+        if not value:
+            return ""
+        if all(isinstance(item, dict) for item in value):
+            # a list of objects (instances, mailboxes, ips, ...) -> one
+            # readable line per item instead of an unreadable JSON blob.
+            return "\n".join(_summarize_item(item) for item in value)
+        if not any(isinstance(item, (dict, list)) for item in value):
+            return ", ".join(str(item) for item in value)
         return json.dumps(value)
     return str(value)
 

{devnomads_cli-0.5.1 → devnomads_cli-0.5.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "devnomads-cli"
-version = "0.5.1"
+version = "0.5.2"
 description = "Manage your DevNomads services from the command line"
 readme = "README.md"
 requires-python = ">=3.10"

{devnomads_cli-0.5.1 → devnomads_cli-0.5.2}/tests/test_helpers.py

@@ -167,11 +167,42 @@ def test_cell():
     assert _cell(True) == "yes"
     assert _cell(False) == "no"
     assert _cell(42) == "42"
-    assert _cell({"a": 1}) == '{"a": 1}'
-    # scalar lists read as comma-separated values, nested ones stay JSON
+    # a bare object renders as a compact key=value summary
+    assert _cell({"a": 1}) == "a=1"
+    # scalar lists read as comma-separated values
     assert _cell(["instance_7:80", "instance_8:80"]) == "instance_7:80, instance_8:80"
     assert _cell([]) == ""
-    assert _cell([{"a": 1}]) == '[{"a": 1}]'
+
+
+def test_cell_list_of_objects_is_summarized():
+    # one readable line per item, not a raw JSON blob
+    ips = [
+        {"type": "v4", "address": "185.223.163.238"},
+        {"type": "v6", "address": "2a10:8c80:0:138::1"},
+    ]
+    assert _cell(ips) == (
+        "type=v4 address=185.223.163.238\ntype=v6 address=2a10:8c80:0:138::1"
+    )
+
+
+def test_cell_summary_drops_empty_and_counts_nested():
+    item = {
+        "id": 7,
+        "image": "geleijn-it/website",
+        "ended_at": None,
+        "has_pending_changes": 0,
+        "volumes": [{"id": 10}],
+        "meta": {},
+    }
+    # None dropped, 0 kept, non-empty nested list shown as a count, empty dict dropped
+    assert (
+        _cell([item])
+        == "id=7 image=geleijn-it/website has_pending_changes=0 volumes=[1]"
+    )
+
+
+def test_cell_mixed_list_stays_json():
+    assert _cell([{"a": 1}, "scalar"]) == '[{"a": 1}, "scalar"]'
 
 
 def test_flatten_row_merges_nested_objects():