Skip to content

Commit 27b07cd

Browse files
ci(065): fix D1 readiness probe — parse data.results from index/search envelope
The retrieval-d1 readiness poll never passed: mcpproxy booted and indexed all 7 servers (45 tools), but the probe parsed the index/search response at the top level while results are nested under the `{"success":true,"data":{"results":[…]}}` envelope, so it read 0 every attempt and timed out. Fix: parse `data.results`. Verified locally — index returns 5 results for q=file within ~6s of boot. actionlint clean. Related #555 datasets; MCP-742. Co-Authored-By: Paperclip <noreply@paperclip.ing>
1 parent 0291e56 commit 27b07cd

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

.github/workflows/eval.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,8 +158,9 @@ jobs:
158158
break
159159
fi
160160
if curl -fsS -H "X-API-Key: $key" "$base/api/v1/status" >/dev/null 2>&1; then
161+
# /api/v1/index/search wraps results as {"success":true,"data":{"results":[...]}}
161162
n="$(curl -fsS -H "X-API-Key: $key" "$base/api/v1/index/search?q=file&limit=5" \
162-
| python3 -c 'import sys,json;d=json.load(sys.stdin);print(len(d.get("tools",d.get("results",[]))))' 2>/dev/null || echo 0)"
163+
| python3 -c 'import sys,json;d=json.load(sys.stdin);print(len((d.get("data") or {}).get("results", [])))' 2>/dev/null || echo 0)"
163164
echo "attempt $i: index search returned $n result(s)"
164165
[ "$n" -ge 1 ] && { ready=1; echo "Index ready."; break; }
165166
else

0 commit comments

Comments
 (0)