greenpick 0.5.1__tar.gz → 0.5.2__tar.gz

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: greenpick
-Version: 0.5.1
+Version: 0.5.2
 Summary: Design-time sustainability co-pilot (MCP server): greener, cheaper cloud infrastructure decisions with carbon + cost + latency tradeoffs.
 Author-email: "Data Science Advisers, LLC" <info@datascienceadvisers.com>
 License-Expression: MIT

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

@@ -1,6 +1,6 @@
 [project]
 name = "greenpick"
-version = "0.5.1"
+version = "0.5.2"
 description = "Design-time sustainability co-pilot (MCP server): greener, cheaper cloud infrastructure decisions with carbon + cost + latency tradeoffs."
 readme = "README.md"
 requires-python = ">=3.11"

{greenpick-0.5.1 → greenpick-0.5.2}/src/greenpick/__init__.py

@@ -4,7 +4,7 @@ Greener cloud-region choices with honest carbon + cost + latency tradeoffs,
 delivered inside the vibe-coding loop. Directional guidance, not audited accounting.
 """
 
-__version__ = "0.5.1"
+__version__ = "0.5.2"
 
 # httpx logs every request URL at INFO, and our live sources pass their API key / token as a URL
 # query parameter — so at the default log level the secret would land in the MCP client's logs.

{greenpick-0.5.1 → greenpick-0.5.2}/src/greenpick/models.py

@@ -303,6 +303,7 @@ class DataLocalityWarning(BaseModel):
 
     The core 'does the greener-region move backfire?' check. Primary signal is egress COST;
     transfer carbon is a secondary, deliberately-coarse figure."""
+    summary: str = ""                           # one line, FIRST key: the answer or what is missing
     compute_location: str                       # "provider/region"
     data_location: str                          # "provider/region"
     monthly_cross_boundary_gb: float

{greenpick-0.5.1 → greenpick-0.5.2}/src/greenpick/services/locality_service.py

@@ -78,6 +78,7 @@ async def data_locality_check(
             egress_rate_usd_per_gb_low=0.0, egress_rate_usd_per_gb_high=0.0,
             est_monthly_egress_usd_low=0.0, est_monthly_egress_usd_high=0.0,
             est_monthly_transfer_kg_co2=0.0,
+            summary="No cross-boundary egress: compute and data are in the same region.",
             warning="Compute and data are co-located — no cross-boundary egress for this flow.",
             recommendation="No locality penalty. Choose the region on carbon/latency/cost as usual.",
             limitations=LIMITATIONS,
@@ -137,6 +138,12 @@ async def data_locality_check(
                    "whether the move backfires on carbon. Ask the user for it, or compute it as the difference "
                    "between estimate_workload_carbon in the two regions, and call again. " + warning)
 
+    if verdict is None:
+        summary = ("NO NET VERDICT: carbon_saving_kg_co2_from_move was not provided, so this does not say "
+                   f"whether the move backfires. Estimated {cost_txt.replace('roughly ', '')}; "
+                   f"transfer ~{transfer_kg:.1f} kg CO2/mo. Supply the saving and call again.")
+    else:
+        summary = f"{verdict} Estimated {cost_txt.replace('roughly ', '')}."
     if severity == "high":
         rec = ("High exposure. Co-locate compute with the data, or move the data to the greener "
                "region too. If you can't, confirm the carbon/cost saving beats this egress penalty "
@@ -163,6 +170,6 @@ async def data_locality_check(
         est_monthly_transfer_kg_co2=transfer_kg,
         transfer_carbon_source=reading.source, transfer_carbon_quality=reading.quality,
         carbon_saving_kg_co2_from_move=carbon_saving_kg_co2_from_move,
-        net_carbon_verdict=verdict, verdict_status=verdict_status, residency_note=residency_note,
+        summary=summary, net_carbon_verdict=verdict, verdict_status=verdict_status, residency_note=residency_note,
         warning=warning, recommendation=rec, limitations=LIMITATIONS,
     )

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: greenpick
-Version: 0.5.1
+Version: 0.5.2
 Summary: Design-time sustainability co-pilot (MCP server): greener, cheaper cloud infrastructure decisions with carbon + cost + latency tradeoffs.
 Author-email: "Data Science Advisers, LLC" <info@datascienceadvisers.com>
 License-Expression: MIT

{greenpick-0.5.1 → greenpick-0.5.2}/tests/test_services.py

@@ -1895,3 +1895,17 @@ async def test_locality_without_saving_says_no_verdict():
     assert no.warning.startswith("NO NET VERDICT") and "carbon_saving_kg_co2_from_move" in no.warning
     yes = await ls.data_locality_check(MockCarbonRepository(), carbon_saving_kg_co2_from_move=40.0, **base)
     assert yes.verdict_status == "evaluated" and not yes.warning.startswith("NO NET VERDICT")
+
+
+# --- 0.5.2: the one-line summary is the FIRST key, so partial printouts still show it ---
+@pytest.mark.asyncio
+async def test_locality_summary_is_first_key_and_flags_missing_verdict():
+    from greenpick.services import locality_service as ls
+    base = dict(compute_provider="aws", compute_region="eu-north-1",
+                data_provider="aws", data_region="us-east-1", monthly_cross_boundary_gb=100)
+    no = (await ls.data_locality_check(MockCarbonRepository(), **base)).model_dump()
+    assert next(iter(no)) == "summary" and no["summary"].startswith("NO NET VERDICT")
+    yes = (await ls.data_locality_check(MockCarbonRepository(), carbon_saving_kg_co2_from_move=40.0, **base)).model_dump()
+    assert "net win" in yes["summary"].lower() and not yes["summary"].startswith("NO NET VERDICT")
+    same = (await ls.data_locality_check(MockCarbonRepository(), "aws", "us-east-1", "aws", "us-east-1", 100)).model_dump()
+    assert next(iter(same)) == "summary" and same["summary"]