costguard-cli 2.1.0__tar.gz → 2.2.0__tar.gz

{costguard_cli-2.1.0 → costguard_cli-2.2.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: costguard-cli
-Version: 2.1.0
+Version: 2.2.0
 Summary: CostGuard CI/CD validation CLI — shift-left cost governance for cloud infrastructure
 Author-email: SKYXOPS <engineering@skyxops.com>
 License-Expression: LicenseRef-Proprietary

{costguard_cli-2.1.0 → costguard_cli-2.2.0}/costguard_cli/__init__.py

@@ -1,3 +1,3 @@
 """CostGuard CLI — shift-left cost governance for CI/CD pipelines."""
 
-__version__ = "2.1.0"
+__version__ = "2.2.0"

{costguard_cli-2.1.0 → costguard_cli-2.2.0}/costguard_cli/formatters/markdown.py

@@ -17,6 +17,11 @@ class MarkdownFormatter:
         "low": "🔵",
     }
 
+    # Wrap per-resource table in <details> when row count exceeds this.
+    # Big plans (50-100+ resources) otherwise bury budget/violations/AI
+    # narrative below an unreadable wall of rows.
+    RESOURCE_COLLAPSE_THRESHOLD = 10
+
     def format(self, response: dict, tag: str | None = None, **kwargs) -> str:
         """Format the full CostGuard API response as Markdown."""
         sections: list[str] = []
@@ -155,7 +160,7 @@ class MarkdownFormatter:
             header = "| Resource | Type | Region | Action | Past | Planned | Δ |"
             divider = "|----------|------|--------|--------|-----:|--------:|---:|"
 
-        lines = ["### Per-Resource Costs\n", header, divider]
+        rows = [header, divider]
 
         for r in resources:
             name = r.get("resource_name") or r.get("resource_id", "unknown")
@@ -179,21 +184,36 @@ class MarkdownFormatter:
 
             if pure_create:
                 cost_str = fmt(planned if planned is not None else mc)
-                lines.append(f"| `{name}` | {rtype} | {region} | {cost_str}{status} |")
+                rows.append(f"| `{name}` | {rtype} | {region} | {cost_str}{status} |")
             elif pure_delete:
                 savings_str = fmt_signed(delta) if delta is not None else fmt_signed(-(past or 0))
-                lines.append(f"| `{name}` | {rtype} | {region} | {savings_str}{status} |")
+                rows.append(f"| `{name}` | {rtype} | {region} | {savings_str}{status} |")
             else:
                 action_str = (action or "-").capitalize() if action else "-"
                 past_str    = "—" if action == "create" else fmt(past)
                 planned_str = "—" if action == "delete" else fmt(planned)
                 delta_str   = fmt_signed(delta) if delta is not None else fmt(mc)
-                lines.append(
+                rows.append(
                     f"| `{name}` | {rtype} | {region} | {action_str} | "
                     f"{past_str} | {planned_str} | {delta_str}{status} |"
                 )
 
-        return "\n".join(lines)
+        table = "\n".join(rows)
+        count = len(resources)
+
+        # Long resource lists make PR/MR comments huge and dominate the review
+        # surface. Wrap the table in <details> when it crosses the threshold so
+        # reviewers see the summary by default and can expand on demand.
+        # GitLab and GitHub both render <details>/<summary> as native folds.
+        if count > self.RESOURCE_COLLAPSE_THRESHOLD:
+            return (
+                "### Per-Resource Costs\n\n"
+                f"<details>\n"
+                f"<summary>Show {count} resources</summary>\n\n"
+                f"{table}\n\n"
+                f"</details>"
+            )
+        return f"### Per-Resource Costs\n\n{table}"
 
     def _budget_section(self, budget: dict) -> str:
         budget_name = budget.get("budget_name", "Unknown")

{costguard_cli-2.1.0 → costguard_cli-2.2.0}/costguard_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: costguard-cli
-Version: 2.1.0
+Version: 2.2.0
 Summary: CostGuard CI/CD validation CLI — shift-left cost governance for cloud infrastructure
 Author-email: SKYXOPS <engineering@skyxops.com>
 License-Expression: LicenseRef-Proprietary

{costguard_cli-2.1.0 → costguard_cli-2.2.0}/costguard_cli.egg-info/SOURCES.txt

@@ -17,4 +17,5 @@ costguard_cli/formatters/html_report.py
 costguard_cli/formatters/json_report.py
 costguard_cli/formatters/markdown.py
 costguard_cli/formatters/terminal.py
-tests/test_formatters_cost_diff.py
+tests/test_formatters_cost_diff.py
+tests/test_resource_table_collapse.py

{costguard_cli-2.1.0 → costguard_cli-2.2.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "costguard-cli"
-version = "2.1.0"
+version = "2.2.0"
 description = "CostGuard CI/CD validation CLI — shift-left cost governance for cloud infrastructure"
 requires-python = ">=3.9"
 license = "LicenseRef-Proprietary"

costguard_cli-2.2.0/tests/test_resource_table_collapse.py

@@ -0,0 +1,54 @@
+"""AI-XMOD-202: collapse per-resource cost table when > RESOURCE_COLLAPSE_THRESHOLD."""
+
+from costguard_cli.formatters.markdown import MarkdownFormatter
+
+
+def _response(n_resources: int) -> dict:
+    return {
+        "endpoint_data": {"decision": "ALLOW", "budget": {}, "violations": []},
+        "results": {
+            "total_monthly_usd": n_resources * 10,
+            "resources": [
+                {
+                    "resource_name": f"aws_instance.r{i}",
+                    "resource_type": "aws_instance",
+                    "region": "us-east-1",
+                    "monthly_cost": 10.0,
+                    "extensions": {"action": "create", "planned_monthly_cost": 10.0},
+                }
+                for i in range(n_resources)
+            ],
+        },
+        "request_id": "test",
+        "timestamp": "2026-05-24T00:00:00",
+        "processing_time_ms": 1,
+    }
+
+
+def test_short_table_renders_flat():
+    out = MarkdownFormatter().format(_response(5))
+    assert "### Per-Resource Costs" in out
+    assert "<summary>Show 5 resources" not in out
+    assert "aws_instance.r0" in out
+    assert "aws_instance.r4" in out
+
+
+def test_long_table_wrapped_in_details():
+    out = MarkdownFormatter().format(_response(15))
+    assert "### Per-Resource Costs" in out
+    assert "<details>" in out
+    assert "<summary>Show 15 resources</summary>" in out
+    assert "</details>" in out
+    assert "aws_instance.r0" in out
+    assert "aws_instance.r14" in out
+
+
+def test_threshold_boundary_exact_match_not_collapsed():
+    # At exactly the threshold, stay flat (strictly greater than collapses).
+    out = MarkdownFormatter().format(_response(MarkdownFormatter.RESOURCE_COLLAPSE_THRESHOLD))
+    assert "<summary>Show" not in out
+
+
+def test_threshold_boundary_one_over_collapses():
+    out = MarkdownFormatter().format(_response(MarkdownFormatter.RESOURCE_COLLAPSE_THRESHOLD + 1))
+    assert "<summary>Show" in out