fix(cli): make plan explain accept plan_id for plan-level explanation #1057

Merged
brent.edwards merged 2 commits from bugfix/m3-plan-explain-plan-id into master 2026-03-19 22:10:23 +00:00

2 Commits

Author SHA1 Message Date
brent.edwards 59bdb4b31c Merge branch 'master' into bugfix/m3-plan-explain-plan-id
CI / lint (pull_request) Successful in 23s
CI / benchmark-publish (pull_request) Has been skipped
CI / typecheck (pull_request) Successful in 32s
CI / quality (pull_request) Successful in 30s
CI / build (pull_request) Successful in 29s
CI / security (pull_request) Successful in 55s
CI / e2e_tests (pull_request) Successful in 5m39s
CI / coverage (pull_request) Successful in 5m13s
CI / unit_tests (pull_request) Successful in 5m53s
CI / integration_tests (pull_request) Successful in 5m59s
CI / docker (pull_request) Successful in 1m8s
CI / benchmark-regression (pull_request) Successful in 41m1s
2026-03-19 22:03:12 +00:00
brent.edwards eeadb0ca61 fix(cli): make plan explain accept plan_id for plan-level explanation
CI / quality (pull_request) Successful in 34s
CI / lint (pull_request) Successful in 36s
CI / benchmark-publish (pull_request) Has been skipped
CI / typecheck (pull_request) Successful in 41s
CI / security (pull_request) Successful in 42s
CI / build (pull_request) Successful in 16s
CI / unit_tests (pull_request) Successful in 2m8s
CI / integration_tests (pull_request) Has been cancelled
CI / e2e_tests (pull_request) Has been cancelled
CI / benchmark-regression (pull_request) Has been cancelled
CI / docker (pull_request) Has been cancelled
CI / coverage (pull_request) Has been cancelled
When given an identifier, explain_decision_cmd now tries it as a
decision_id first (preserving backward compatibility) and, if not found,
falls back to treating it as a plan_id—explaining the root decision of
that plan. The parameter is renamed from decision_id to identifier and
the help text updated to reflect dual usage. The existing BDD step mock
for the "not found" path is updated to also stub list_decisions so the
fallback resolves correctly.

ISSUES CLOSED: #968
2026-03-19 22:00:31 +00:00