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

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

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

{devnomads_cli-0.5.3 → devnomads_cli-0.5.4}/devnomads_cli.egg-info/PKG-INFO

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

{devnomads_cli-0.5.3 → devnomads_cli-0.5.4}/dncli.py

@@ -342,14 +342,14 @@ def _render_kv(data: dict[str, Any], title: str | None) -> None:
 def _auto_columns(rows: list[dict[str, Any]]) -> list[str]:
     """Choose the columns worth showing in a list view, from the data itself.
 
-    Three things get dropped, because a list is an overview and the full record
+    Two things get dropped, because a list is an overview and the full record
     is always available via the matching ``show`` command (or ``-o json``):
 
     * columns empty in every row (no information at all);
     * columns holding a list of objects (instances, mailboxes, ips, ...) -
-      these are detail, belonging in the ``show`` view, not the overview;
-    * columns whose value never changes across rows - they cost width without
-      helping tell the rows apart (the first column is kept as an anchor).
+      these are detail, belonging in the ``show`` view, not the overview.
+
+    Every other field is kept, even if its value is the same on every row.
     """
 
     cols: list[str] = []
@@ -370,17 +370,11 @@ def _auto_columns(rows: list[dict[str, Any]]) -> list[str]:
             for row in rows
         )
 
-    cols = [
+    return [
         c
         for c in cols
         if any(not empty(row.get(c)) for row in rows) and not is_object_list(c)
     ]
-    if len(rows) > 1 and cols:
-        anchor = cols[0]
-        rendered = {c: {_cell(row.get(c)) for row in rows} for c in cols}
-        if any(len(values) > 1 for values in rendered.values()):
-            cols = [c for c in cols if c == anchor or len(rendered[c]) > 1]
-    return cols
 
 
 def _render_rows(

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

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

{devnomads_cli-0.5.3 → devnomads_cli-0.5.4}/tests/test_helpers.py

@@ -231,25 +231,25 @@ def test_cell_detail_keeps_per_item_lines():
     assert _cell(insts) == "id=7 state_last_known=running"
 
 
-def test_auto_columns_drops_empty_constant_and_object_lists():
+def test_auto_columns_drops_empty_and_object_lists_only():
     rows = [
         {"id": 1, "port": 80, "registry": "same", "blank": "", "instances": [{"i": 7}]},
         {"id": 2, "port": 81, "registry": "same", "blank": "", "instances": [{"i": 8}]},
     ]
-    # registry is constant, blank is empty, instances is an object list -> all
-    # dropped; id (anchor) and port (varies) remain.
-    assert _auto_columns(rows) == ["id", "port"]
+    # blank is empty and instances is an object list -> dropped; every other
+    # scalar is kept, including the constant `registry`.
+    assert _auto_columns(rows) == ["id", "port", "registry"]
 
 
-def test_auto_columns_keeps_anchor_even_if_constant():
+def test_auto_columns_keeps_constant_scalar_columns():
     rows = [{"id": 1, "x": 9, "y": "a"}, {"id": 1, "x": 9, "y": "b"}]
-    # x is constant -> dropped; id is constant but the anchor -> kept
-    assert _auto_columns(rows) == ["id", "y"]
+    # constant columns are no longer dropped - normal fields always show
+    assert _auto_columns(rows) == ["id", "x", "y"]
 
 
 def test_auto_columns_single_row_keeps_nonempty_scalars():
     rows = [{"id": 1, "name": "x", "blank": None, "items": [{"a": 1}]}]
-    # one row: no constant-drop, but empty and object-list columns still go
+    # empty and object-list columns go, scalars stay
     assert _auto_columns(rows) == ["id", "name"]
 
 
@@ -266,8 +266,8 @@ def test_object_table_uses_auto_columns():
         {"emailaddress": "b@x", "quota": 2, "domain": "x"},
     ]
     table = _object_table(items)
-    # constant `domain` dropped; the differing fields are kept as headers
-    assert [c.header for c in table.columns] == ["emailaddress", "quota"]
+    # all scalar sub-fields are kept (empties/object-lists would be dropped)
+    assert [c.header for c in table.columns] == ["emailaddress", "quota", "domain"]
 
 
 def test_render_kv_drops_empty_and_nests_objects(capsys):