Nanda Kumar Kondreddy commited on
Commit ·
3a48cf2
1
Parent(s): a2f8fcd
PROPER FIX: Remove default /metadata route before registering custom override with tasks
Browse files- server/app.py +9 -2
server/app.py
CHANGED
|
@@ -19,7 +19,13 @@ app = create_fastapi_app(
|
|
| 19 |
ConfigDebugObservation, # observation model (inherits Observation)
|
| 20 |
)
|
| 21 |
|
| 22 |
-
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 23 |
|
| 24 |
|
| 25 |
# ---- Custom endpoints ----
|
|
@@ -55,7 +61,8 @@ def debug():
|
|
| 55 |
|
| 56 |
@app.get("/metadata")
|
| 57 |
def metadata():
|
| 58 |
-
"""OpenEnv metadata endpoint with task enumeration for validator discovery."""
|
|
|
|
| 59 |
return {
|
| 60 |
"name": "ConfigDebugEnvironment",
|
| 61 |
"description": "An RL environment for training AI agents to debug broken configuration files",
|
|
|
|
| 19 |
ConfigDebugObservation, # observation model (inherits Observation)
|
| 20 |
)
|
| 21 |
|
| 22 |
+
# ---- Override OpenEnv's default /metadata route ----
|
| 23 |
+
# Remove the built-in metadata endpoint so we can replace it with task enumeration
|
| 24 |
+
for i, route in enumerate(app.router.routes):
|
| 25 |
+
if hasattr(route, "path") and route.path == "/metadata":
|
| 26 |
+
app.router.routes.pop(i)
|
| 27 |
+
print("[APP_INIT] Removed default /metadata route for override")
|
| 28 |
+
break
|
| 29 |
|
| 30 |
|
| 31 |
# ---- Custom endpoints ----
|
|
|
|
| 61 |
|
| 62 |
@app.get("/metadata")
|
| 63 |
def metadata():
|
| 64 |
+
"""Override OpenEnv metadata endpoint with task enumeration for validator discovery."""
|
| 65 |
+
print("[VALIDATOR] GET /metadata called")
|
| 66 |
return {
|
| 67 |
"name": "ConfigDebugEnvironment",
|
| 68 |
"description": "An RL environment for training AI agents to debug broken configuration files",
|