greenpick 0.5.0__tar.gz → 0.5.2__tar.gz

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: greenpick
-Version: 0.5.0
+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.0 → greenpick-0.5.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "greenpick"
-version = "0.5.0"
+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.0 → 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.0"
+__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.0 → 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
@@ -317,6 +318,7 @@ class DataLocalityWarning(BaseModel):
     transfer_carbon_quality: Optional[str] = None
     carbon_saving_kg_co2_from_move: Optional[float] = None # echoed back if provided
     net_carbon_verdict: Optional[str] = None    # set only when a saving was supplied
+    verdict_status: str = "not_evaluated"       # 'evaluated' only when a saving was supplied
     residency_note: Optional[str] = None        # neutral cross-jurisdiction heads-up, if any
     warning: str = ""
     recommendation: str = ""

{greenpick-0.5.0 → greenpick-0.5.2}/src/greenpick/server.py

@@ -358,7 +358,9 @@ async def data_locality_egress_check(
     compute runs, where the data lives (provider+region), and monthly GB crossing between them.
     Returns a directional egress-COST ballpark, a coarse transfer-carbon figure, and — if you
     pass the carbon saving the greener-region move buys — whether it's still a net win. Use this
-    before accepting a greener-region suggestion when your data stays put."""
+    before accepting a greener-region suggestion when your data stays put. Use the user's real GB and
+    data location; if unknown, ask rather than assume. If verdict_status is 'not_evaluated' there is NO
+    answer yet to 'does it backfire?': say so, and get carbon_saving_kg_co2_from_move first."""
     advice = await locality_service.data_locality_check(
         _repo, compute_provider, compute_region, data_provider, data_region,  # type: ignore[arg-type]
         monthly_cross_boundary_gb=monthly_cross_boundary_gb,

{greenpick-0.5.0 → 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,
@@ -109,7 +110,9 @@ async def data_locality_check(
 
     # Net verdict only if the caller told us what the move saves.
     verdict = None
+    verdict_status = "not_evaluated"
     if carbon_saving_kg_co2_from_move is not None:
+        verdict_status = "evaluated"
         net = carbon_saving_kg_co2_from_move - transfer_kg
         if net > 0:
             verdict = (f"Still a net win: the move saves ~{carbon_saving_kg_co2_from_move:.1f} kg "
@@ -130,10 +133,17 @@ async def data_locality_check(
                f"cross the boundary, costing {cost_txt}, plus ~{transfer_kg:.1f} kg CO2/mo of extra "
                f"transfer. A greener-region move can be undone by this.")
     if verdict is None:
-        # Point the caller at the optional input that turns this into a net win/backfire verdict.
-        warning += (" For a net verdict, pass carbon_saving_kg_co2_from_move (the kg CO2/mo the move "
-                    "saves, e.g. the difference between estimate_workload_carbon in the two regions).")
+        # Say up front that the question 'does it backfire?' is NOT answered yet, and what is missing.
+        warning = ("NO NET VERDICT: you did not pass carbon_saving_kg_co2_from_move, so this does NOT say "
+                   "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 "
@@ -160,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, 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.0 → greenpick-0.5.2/src/greenpick.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: greenpick
-Version: 0.5.0
+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.0 → greenpick-0.5.2}/tests/test_services.py

@@ -1882,3 +1882,30 @@ def test_tool_greenest_regions_filters_and_errors():
     assert r["ranking"] and all(x["provider"] == "aws" for x in r["ranking"])
     assert any(x["quality"] == "annual_average" for x in _tool("greenest_regions_on_average")(month=1)["ranking"])
     assert "error" in _tool("greenest_regions_on_average")(month=13)
+
+
+# --- 0.5.1: a missing carbon saving is called out, not silently skipped ---
+@pytest.mark.asyncio
+async def test_locality_without_saving_says_no_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)
+    assert no.net_carbon_verdict is None and no.verdict_status == "not_evaluated"
+    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"]