Fix load_dotenv timing bug, remove global RNG seed, improve error handling

contract_env/env/environment.py

@@ -11,8 +11,6 @@
 from contract_env.env.models import Action, Observation
 from contract_env.env.tasks import TASKS, NegotiationTask
 
-random.seed(42)
-
 
 class ContractEnv:
     """Multi-turn contract-negotiation environment.

contract_env/server/app.py

@@ -143,6 +143,8 @@ def step(req: StepRequest) -> dict[str, Any]:
             "info": info,
         }
 
+    except RuntimeError as e:
+        raise HTTPException(status_code=400, detail=str(e))
     except ValidationError as e:
         raise HTTPException(status_code=422, detail=e.errors())
     except Exception:

hf_create.py

@@ -22,7 +22,7 @@
 
 req = urllib.request.Request(url, headers=headers, data=data)
 try:
-    with urllib.request.urlopen(req) as resp:
+    with urllib.request.urlopen(req, timeout=30) as resp:
         print(resp.read().decode("utf-8"))
 except Exception as e:
     import traceback

inference.py

@@ -40,6 +40,10 @@
     def load_dotenv(*_a: Any, **_kw: Any) -> None:  # type: ignore[misc]
         pass
 
+# Load .env BEFORE reading any environment variables so that .env-only
+# configuration is picked up by the module-level constants below.
+load_dotenv()
+
 from openai import OpenAI
 
 from contract_env.env.environment import ContractEnv
@@ -652,7 +656,6 @@ def run_episode(env, task_id: Optional[str] = None) -> tuple[float, str]:
 
 # ── MAIN ─────────────────────────────────────────────────────────────────
 def main() -> None:
-    load_dotenv()
     random.seed(42)
 
     parser = argparse.ArgumentParser(

server/app.py

@@ -5,3 +5,7 @@
 """
 
 from contract_env.server.app import app  # noqa: F401
+from contract_env.server.app import main  # noqa: F401
+
+if __name__ == "__main__":
+    main()