panorama-super-cli 0.2.4__tar.gz → 0.2.5__tar.gz

{panorama_super_cli-0.2.4 → panorama_super_cli-0.2.5}/CHANGELOG.md

@@ -7,6 +7,17 @@ project will follow [Semantic Versioning](https://semver.org/). While on
 
 ## [Unreleased]
 
+## v0.2.5 — 2026-06-04
+
+### Fixed
+
+- **`psc find ip` table output now separates each target's matches with a
+  horizontal rule** (#43). Resolving several targets at once (notably with
+  `--file`) previously printed every match as one undifferentiated block, so it
+  was hard to see where one target's matches ended and the next began. The
+  table now draws a rule between per-target blocks. Machine output formats
+  (json/jsonl/yaml/csv/set) are unchanged.
+
 ## v0.2.4 — 2026-06-04
 
 ### Changed

{panorama_super_cli-0.2.4 → panorama_super_cli-0.2.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: panorama-super-cli
-Version: 0.2.4
+Version: 0.2.5
 Summary: Agent-friendly CLI for Palo Alto Panorama object management: find, dedup/merge, rename, and audit address/service objects safely.
 Project-URL: Homepage, https://github.com/thomaschristory/panorama-super-cli
 Project-URL: Documentation, https://thomaschristory.github.io/panorama-super-cli/

{panorama_super_cli-0.2.4 → panorama_super_cli-0.2.5}/psc/_version.py

@@ -4,4 +4,4 @@
 Keep this in sync with `[project].version` in `pyproject.toml`.
 """
 
-__version__ = "0.2.4"
+__version__ = "0.2.5"

{panorama_super_cli-0.2.4 → panorama_super_cli-0.2.5}/psc/cli/find_cmds.py

@@ -78,7 +78,9 @@ def ip(
         raise PscError("no matching objects", ErrorType.NOT_FOUND)
 
     model = results if len(results) != 1 else results[0]
-    render(rt.stdout, rt.output, model=model, rows=rows, table_title="find ip")
+    # Group the table by query so multi-target output (e.g. --file) draws a rule
+    # between each target's matches instead of one undifferentiated block.
+    render(rt.stdout, rt.output, model=model, rows=rows, table_title="find ip", group_by="query")
 
 
 @app.command("object")

{panorama_super_cli-0.2.4 → panorama_super_cli-0.2.5}/psc/output/format.py

@@ -104,12 +104,17 @@ def render(
     rows: list[dict[str, Any]] | None = None,
     set_lines: list[str] | None = None,
     table_title: str | None = None,
+    group_by: str | None = None,
 ) -> None:
     """Print `model`/`rows`/`set_lines` in `fmt` to `console`.
 
     The console must be the *stdout* console; errors go elsewhere. Table output
     is the only format that uses rich styling — every other format is plain text
     so pipes and captures stay clean.
+
+    `group_by` is a table-only hint: a column name whose changing value marks a
+    new block. A horizontal rule is drawn between blocks so multi-target output
+    (e.g. `find ip --file`) is easy to scan. It does not affect machine formats.
     """
     # `soft_wrap=True` is critical: without it rich wraps long lines to the
     # console width and injects newlines *inside* JSON string values, producing
@@ -135,8 +140,13 @@ def render(
         table = Table(title=table_title, header_style="bold cyan")
         for col in rows[0]:
             table.add_column(col)
-        for row in rows:
+        prev: Any = None
+        for i, row in enumerate(rows):
+            if group_by and i > 0 and row.get(group_by) != prev:
+                table.add_section()
             table.add_row(*[_cell(row.get(col)) for col in rows[0]])
+            if group_by:
+                prev = row.get(group_by)
         console.print(table)
     else:
         console.print(_json(model), markup=False, highlight=False)

{panorama_super_cli-0.2.4 → panorama_super_cli-0.2.5}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "panorama-super-cli"
-version = "0.2.4"
+version = "0.2.5"
 description = "Agent-friendly CLI for Palo Alto Panorama object management: find, dedup/merge, rename, and audit address/service objects safely."
 readme = "README.md"
 requires-python = ">=3.12"

{panorama_super_cli-0.2.4 → panorama_super_cli-0.2.5}/tests/test_cli.py

@@ -64,6 +64,14 @@ def test_strict_not_found_exit_5() -> None:
     assert json.loads(cp.stdout)["type"] == "not_found"
 
 
+def test_find_ip_table_separates_multiple_targets() -> None:
+    # Issue #43: multi-target table output must draw a rule between each
+    # target's matches. The interior divider uses box-drawing '├'.
+    cp = run("-c", str(FIXTURE), "-o", "table", "find", "ip", "10.0.0.10", "10.0.0.99")
+    assert cp.returncode == 0
+    assert "├" in cp.stdout
+
+
 _HOST_AND_NET_CONFIG = """<?xml version="1.0"?>
 <config version="11.0.0">
   <shared>

panorama_super_cli-0.2.5/tests/test_output.py

@@ -0,0 +1,65 @@
+from __future__ import annotations
+
+import io
+import json
+
+from rich.console import Console
+
+from psc.core.models import SHARED, Location
+from psc.core.refs import Reference, Target
+from psc.output.format import OutputFormat, render, to_jsonable
+
+
+def test_location_serializes_as_name() -> None:
+    assert to_jsonable(SHARED) == "shared"
+    assert to_jsonable(Location.dg("DG1")) == "DG1"
+
+
+def test_location_roundtrips_from_string() -> None:
+    loc = Location.model_validate("DG1")
+    assert loc.device_group == "DG1"
+    assert Location.model_validate("shared").is_shared
+
+
+def test_dataclass_is_jsonable() -> None:
+    ref = Reference(
+        target_name="x",
+        namespace="address",
+        referrer_kind="security-rule",
+        referrer_name="r",
+        referrer_location=SHARED,
+        field="source",
+        resolved=Target("address", "x", SHARED),
+    )
+    payload = to_jsonable(ref)
+    # round-trips through json without error
+    json.dumps(payload)
+    assert payload["resolved"]["location"] == "shared"
+    assert payload["referrer_location"] == "shared"
+
+
+def _render_table(rows: list[dict[str, object]], **kw: object) -> str:
+    buf = io.StringIO()
+    console = Console(file=buf, width=120, force_terminal=False, color_system=None)
+    render(console, OutputFormat.TABLE, rows=rows, table_title="t", **kw)
+    return buf.getvalue()
+
+
+def test_table_group_by_inserts_divider_between_groups() -> None:
+    rows = [
+        {"query": "10.0.0.1", "object": "a"},
+        {"query": "10.0.0.1", "object": "b"},
+        {"query": "10.0.0.2", "object": "c"},
+    ]
+    plain = _render_table(rows)
+    grouped = _render_table(rows, group_by="query")
+    # Grouping draws one extra horizontal rule between the two query blocks.
+    assert grouped.count("\n") == plain.count("\n") + 1
+
+
+def test_table_group_by_single_group_no_extra_divider() -> None:
+    rows = [
+        {"query": "10.0.0.1", "object": "a"},
+        {"query": "10.0.0.1", "object": "b"},
+    ]
+    assert _render_table(rows, group_by="query") == _render_table(rows)