From ed7cf00d7f405e5decf5d955b9147a6a0b451bec Mon Sep 17 00:00:00 2001 From: CleverThis Date: Tue, 5 May 2026 14:38:02 +0000 Subject: [PATCH] fix(plan): update robot helper to assert alternatives key in explain output The robot/helper_plan_explain.py integration test helper was still asserting the old field name alternatives_considered in the explain dict output. Since _build_explain_dict() now outputs alternatives (structured objects with index/description/chosen fields per spec), the assertion was failing the CI integration_tests job. Updated the assertion to check for alternatives and also verify it is a list, matching the new structured output format. --- robot/helper_plan_explain.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/robot/helper_plan_explain.py b/robot/helper_plan_explain.py index 215940953..0a416f38a 100644 --- a/robot/helper_plan_explain.py +++ b/robot/helper_plan_explain.py @@ -49,7 +49,8 @@ def _test_explain_format() -> None: assert "decision_id" in data assert "context_snapshot" in data assert "rationale" in data - assert "alternatives_considered" in data + assert "alternatives" in data + assert isinstance(data["alternatives"], list) assert data["question"] == "What to build?" print("plan-explain-ok")