fix(cli): add --namespace/-n option to agents plan list command #2616

Merged

freemo merged 1 commit from fix/plan-list-namespace-option into master

2026-04-03 21:40:46 +00:00

freemo commented

2026-04-03 19:48:34 +00:00

Owner

Summary

Adds the missing --namespace/-n option to the agents plan list CLI command, bringing it into alignment with the specification and achieving parity with the existing agents action list implementation. The service layer already supported namespace filtering; only the CLI layer was missing the option.

Changes

src/cleveragents/cli/commands/plan.py: Added namespace: Annotated[str | None, typer.Option("--namespace", "-n", help="Filter plans by namespace")] = None parameter to lifecycle_list_plans(). The parameter is placed as the first option after the regex argument, matching the established ordering in list_actions(). The namespace value is forwarded to service.list_plans(namespace=namespace, ...).
TUI Filters panel: Updated to display Namespace: <value> as the first filter entry when --namespace is provided, consistent with the spec output format (│ Namespace: myteam │).
CLI docstring: Updated with usage examples demonstrating the new --namespace/-n option.
features/plan_cli_spec_alignment.feature: Added 4 new Behave unit test scenarios covering: filtering by --namespace, filtering by -n (short form), combined namespace + state filtering, and namespace producing an empty result set.
robot/cli_lifecycle_e2e.robot and robot/helper_cli_lifecycle_e2e.py: Added 2 new Robot Framework integration test cases with supporting helper functions to validate end-to-end namespace filtering behaviour.

Design Decisions

Service layer was already complete: plan_lifecycle_service.py already accepted and applied a namespace keyword argument for filtering. This fix is purely a CLI-layer gap — no service or data-layer changes were required.
Parameter ordering mirrors list_actions(): To maintain consistency across the CLI surface, namespace is placed immediately after the positional regex argument, exactly as it appears in action.py. This makes the two commands symmetrical and reduces cognitive overhead for users.
Filters panel ordering: The spec explicitly shows │ Namespace: myteam │ as the leading filter entry in the TUI output. The implementation follows this ordering so that rendered output matches the specification verbatim.
No breaking changes: The parameter defaults to None, preserving all existing behaviour when --namespace is omitted. Calls without the flag are unaffected.

Testing

Unit tests (Behave): PASS — 21 scenarios in features/plan_cli_spec_alignment.feature (includes 4 new namespace scenarios)
Integration tests (Robot): PASS — robot/cli_lifecycle_e2e.robot suite passed in full (includes 2 new namespace test cases)
Coverage: ≥ 97% (pre-existing threshold maintained; new code paths covered by the 6 new test cases above)

Note: A pre-existing failure in plan_cli_commands_r2.feature (the "+2 more" assertion) was confirmed to exist on master prior to this branch and is unrelated to this change.

Modules Affected

Module	Change
`src/cleveragents/cli/commands/plan.py`	Added `--namespace`/`-n` option to `lifecycle_list_plans()`; forwarded to service call; updated docstring
TUI Filters panel (within `plan.py`)	Added `Namespace: <value>` as first filter entry in rendered output
`features/plan_cli_spec_alignment.feature`	Added 4 Behave scenarios for namespace filtering
`robot/cli_lifecycle_e2e.robot`	Added 2 Robot Framework integration test cases
`robot/helper_cli_lifecycle_e2e.py`	Added helper functions supporting new Robot test cases

Closes #2165

Automated by CleverAgents Bot
Supervisor: Implementation | Agent: ca-issue-worker

## Summary Adds the missing `--namespace`/`-n` option to the `agents plan list` CLI command, bringing it into alignment with the specification and achieving parity with the existing `agents action list` implementation. The service layer already supported namespace filtering; only the CLI layer was missing the option. ## Changes - **`src/cleveragents/cli/commands/plan.py`**: Added `namespace: Annotated[str | None, typer.Option("--namespace", "-n", help="Filter plans by namespace")] = None` parameter to `lifecycle_list_plans()`. The parameter is placed as the first option after the regex argument, matching the established ordering in `list_actions()`. The `namespace` value is forwarded to `service.list_plans(namespace=namespace, ...)`. - **TUI Filters panel**: Updated to display `Namespace: <value>` as the first filter entry when `--namespace` is provided, consistent with the spec output format (`│ Namespace: myteam │`). - **CLI docstring**: Updated with usage examples demonstrating the new `--namespace`/`-n` option. - **`features/plan_cli_spec_alignment.feature`**: Added 4 new Behave unit test scenarios covering: filtering by `--namespace`, filtering by `-n` (short form), combined namespace + state filtering, and namespace producing an empty result set. - **`robot/cli_lifecycle_e2e.robot`** and **`robot/helper_cli_lifecycle_e2e.py`**: Added 2 new Robot Framework integration test cases with supporting helper functions to validate end-to-end namespace filtering behaviour. ## Design Decisions - **Service layer was already complete**: `plan_lifecycle_service.py` already accepted and applied a `namespace` keyword argument for filtering. This fix is purely a CLI-layer gap — no service or data-layer changes were required. - **Parameter ordering mirrors `list_actions()`**: To maintain consistency across the CLI surface, `namespace` is placed immediately after the positional regex argument, exactly as it appears in `action.py`. This makes the two commands symmetrical and reduces cognitive overhead for users. - **Filters panel ordering**: The spec explicitly shows `│ Namespace: myteam │` as the leading filter entry in the TUI output. The implementation follows this ordering so that rendered output matches the specification verbatim. - **No breaking changes**: The parameter defaults to `None`, preserving all existing behaviour when `--namespace` is omitted. Calls without the flag are unaffected. ## Testing - **Unit tests (Behave):** PASS — 21 scenarios in `features/plan_cli_spec_alignment.feature` (includes 4 new namespace scenarios) - **Integration tests (Robot):** PASS — `robot/cli_lifecycle_e2e.robot` suite passed in full (includes 2 new namespace test cases) - **Coverage:** ≥ 97% (pre-existing threshold maintained; new code paths covered by the 6 new test cases above) > **Note:** A pre-existing failure in `plan_cli_commands_r2.feature` (the `"+2 more"` assertion) was confirmed to exist on `master` prior to this branch and is unrelated to this change. ## Modules Affected | Module | Change | |---|---| | `src/cleveragents/cli/commands/plan.py` | Added `--namespace`/`-n` option to `lifecycle_list_plans()`; forwarded to service call; updated docstring | | TUI Filters panel (within `plan.py`) | Added `Namespace: <value>` as first filter entry in rendered output | | `features/plan_cli_spec_alignment.feature` | Added 4 Behave scenarios for namespace filtering | | `robot/cli_lifecycle_e2e.robot` | Added 2 Robot Framework integration test cases | | `robot/helper_cli_lifecycle_e2e.py` | Added helper functions supporting new Robot test cases | ## Related Issues Closes #2165 --- **Automated by CleverAgents Bot** Supervisor: Implementation | Agent: ca-issue-worker

freemo added this to the v3.7.0 milestone

Rows
Columns

fix(cli): add --namespace/-n option to agents plan list command #2616

Summary

Changes

Design Decisions

Testing

Modules Affected

Related Issues

Code Review — APPROVED ✅

Summary

Specification Alignment ✅

Code Quality ✅

Test Quality ✅

Correctness ✅

CI Status

Minor Note