fix(plan): include decision_id field in plan tree JSON output #9193

Merged
HAL9000 merged 2 commits from fix/plan-tree-json-missing-decision-id into master 2026-06-02 21:58:57 +00:00

2 Commits

Author SHA1 Message Date
controller-ci-rerun 1dc25138b3 chore: re-trigger CI [controller]
CI / push-validation (pull_request) Successful in 35s
CI / build (pull_request) Successful in 37s
CI / helm (pull_request) Successful in 37s
CI / lint (pull_request) Successful in 50s
CI / quality (pull_request) Successful in 51s
CI / typecheck (pull_request) Successful in 1m29s
CI / security (pull_request) Successful in 1m30s
CI / unit_tests (pull_request) Successful in 5m24s
CI / docker (pull_request) Successful in 1m25s
CI / integration_tests (pull_request) Successful in 8m49s
CI / coverage (pull_request) Successful in 8m38s
CI / status-check (pull_request) Successful in 3s
2026-06-02 17:43:31 -04:00
HAL9000 1301182990 fix(plan): include decision_id field in plan tree JSON output
The step_tree_json_valid BDD step was asserting a raw list from
format_output, but the function wraps all machine-readable output in
a spec-required envelope dict ({"data": [...]}). This PR fixes the
step to correctly validate the envelope structure (dict with data key)
and removes @tdd_expected_fail from the @tdd_issue_4254 scenario so
it runs as a permanent regression guard.

The code producing decision_id in tree nodes was already correct; only
the test assertion needed fixing.

ISSUES CLOSED: #9096
2026-06-02 17:43:31 -04:00