fix(tui): make /help command list all catalogued slash commands from SLASH_COMMAND_SPECS
CI / lint (pull_request) Successful in 47s
CI / helm (pull_request) Successful in 26s
CI / build (pull_request) Successful in 41s
CI / quality (pull_request) Successful in 53s
CI / typecheck (pull_request) Successful in 1m8s
CI / push-validation (pull_request) Successful in 42s
CI / security (pull_request) Successful in 1m55s
CI / unit_tests (pull_request) Successful in 6m49s
CI / docker (pull_request) Successful in 1m41s
CI / coverage (pull_request) Successful in 10m42s
CI / integration_tests (pull_request) Failing after 27m21s
CI / status-check (pull_request) Failing after 3s
CI / lint (pull_request) Successful in 47s
CI / helm (pull_request) Successful in 26s
CI / build (pull_request) Successful in 41s
CI / quality (pull_request) Successful in 53s
CI / typecheck (pull_request) Successful in 1m8s
CI / push-validation (pull_request) Successful in 42s
CI / security (pull_request) Successful in 1m55s
CI / unit_tests (pull_request) Successful in 6m49s
CI / docker (pull_request) Successful in 1m41s
CI / coverage (pull_request) Successful in 10m42s
CI / integration_tests (pull_request) Failing after 27m21s
CI / status-check (pull_request) Failing after 3s
Replace the hardcoded help string in TuiCommandRouter.handle() with a dynamic lookup against SLASH_COMMAND_SPECS from slash_catalog.py. Changes: - Add _help_command(), _help_list_all(), _help_for_command() methods to TuiCommandRouter - /help (no args): iterates SLASH_COMMAND_SPECS, groups commands by namespace (sorted alphabetically), renders all 70 commands with descriptions in colon-namespaced format (e.g. persona:list) - /help <command>: looks up the given command in SLASH_COMMAND_SPECS and renders its full help (group, description) - /help <unknown>: returns 'Unknown command: /<cmd>' message - /help /persona:list (with leading slash): strips the slash and resolves correctly - Import defaultdict and SLASH_COMMAND_SPECS at module level Tests: - Update tui_commands_coverage.feature: replace old exact-match scenario for help text with new dynamic-listing assertions - Add tui_commands_coverage_steps.py: new 'should contain' step definition - Add tui_help_command_full_catalog.feature: 12 BDD scenarios covering /help no-args, /help <command>, /help <unknown>, namespace grouping, colon-namespaced format, and regression against old hardcoded string - Add tui_help_command_full_catalog_steps.py: step definitions for the new feature (all-commands check, not-equal assertion) - Add robot/tui_help_command.robot: 5 Robot Framework integration tests verifying the help command via direct Python invocation and headless TUI startup Closes #3434 --- **Automated by CleverAgents Bot** Supervisor: Implementation | Agent: ca-issue-worker
This commit is contained in:
@@ -26,7 +26,6 @@ TUI Help Command Lists All Catalogued Commands
|
||||
|
||||
TUI Help Command Groups By Namespace
|
||||
[Documentation] /help output must include group headers for Session, Persona, Plan, Utility.
|
||||
[Tags] tdd_issue tdd_issue_4298 tdd_expected_fail
|
||||
${script}= Catenate SEPARATOR=\n
|
||||
... from cleveragents.tui.commands import TuiCommandRouter
|
||||
... from unittest.mock import MagicMock
|
||||
|
||||
Reference in New Issue
Block a user