try handling badly json-decoded llm queries

Author: esoteric-ephemera

mp_api/client/core/utils.py

@@ -25,8 +25,8 @@ def validate_api_key(api_key: str | None = None) -> str:
 
         api_key = SETTINGS.get("PMG_MAPI_KEY")
 
-    if not api_key or (wrong_len := len(api_key) != 32):
-        addendum = " Valid API keys are 32 characters." if wrong_len else ""
+    if not api_key or len(api_key) != 32:
+        addendum = " Valid API keys are 32 characters." if api_key else ""
         raise ValueError(
             "Please obtain a valid API key from https://materialsproject.org/api "
             f"and export it as an environment variable `MP_API_KEY`.{addendum}"

mp_api/mcp/tools.py

@@ -2,6 +2,7 @@
 from __future__ import annotations
 
 from datetime import datetime
+import json
 from typing import Literal
 
 import plotly.graph_objects as plotly_go
@@ -643,7 +644,7 @@ def get_material_data(
             efermi=efermi,
             elastic_anisotropy=elastic_anisotropy,
             elements=elements,
-            energy_above_hull=energy_above_hull,
+            energy_above_hull=tuple(float(x) for x in json.loads(energy_above_hull)),
             equilibrium_reaction_energy=equilibrium_reaction_energy,
             exclude_elements=exclude_elements,
             formation_energy=formation_energy,