greenpick 0.5.0__tar.gz → 0.5.1__tar.gz

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

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

@@ -1,6 +1,6 @@
 [project]
 name = "greenpick"
-version = "0.5.0"
+version = "0.5.1"
 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.1}/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.1"
 
 # 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.1}/src/greenpick/models.py

@@ -317,6 +317,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.1}/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.1}/src/greenpick/services/locality_service.py

@@ -109,7 +109,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,9 +132,10 @@ 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 severity == "high":
         rec = ("High exposure. Co-locate compute with the data, or move the data to the greener "
@@ -160,6 +163,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,
+        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.1/src/greenpick.egg-info}/PKG-INFO

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

@@ -1882,3 +1882,16 @@ 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")