cleveragents/cleveragents-core

Fork 3

UAT: TuiCommandRouter uses space-split tokenization — colon-notation commands (e.g. session:create, plan:use) all return "Unknown command" #4735

New issue

Open

opened 2026-04-08 18:12:57 +00:00 by HAL9000 · 3 comments

HAL9000 commented

2026-04-08 18:12:57 +00:00

Owner

Bug Report

Feature area: TUI slash command tokenization and dispatch

Severity: Medium — 60+ of the 70 catalogued slash commands are silently unroutable

What was tested

src/cleveragents/tui/commands.py — TuiCommandRouter.handle() and its tokenization logic, cross-referenced against src/cleveragents/tui/slash_catalog.py.

Expected behavior (from spec)

The slash command system uses colon-notation (namespace:verb) as the canonical command format. The 70-command catalog (SLASH_COMMAND_SPECS) lists commands like session:create, persona:list, plan:use, scope:add, resource:tree, config:get, etc. The router should dispatch these colon-notation commands to their respective handlers.

Actual behavior

TuiCommandRouter.handle() tokenizes input with raw.strip().split() (space-split), then checks tokens[0] against only three hardcoded namespaces:

tokens = raw.strip().split()
if tokens[0] == "persona":
    return self._persona_command(tokens[1:], ...)
if tokens[0] == "session":
    return self._session_command(tokens[1:], ...)
if tokens[0] == "help":
    return self._help_command(tokens[1:])
return f"Unknown command: /{raw}"

The catalog uses colon-notation (session:create, persona:list) but the router expects space-separated (session create, persona list). This means:

Typing /session:create → tokens[0] is "session:create" → falls through to "Unknown command"
Typing /plan:use → "Unknown command"
Typing /scope:add → "Unknown command"
Typing /resource:list → "Unknown command"
All 60+ non-persona/non-session/non-help catalog commands → "Unknown command"

Even persona and session sub-commands are inconsistent: the catalog shows persona:list but the router handles persona list (space-separated).

Code locations

src/cleveragents/tui/commands.py lines 49–58 — handle() method with space-split tokenization
src/cleveragents/tui/slash_catalog.py — all 70 commands use colon-notation
src/cleveragents/tui/commands.py lines 96–113 — _persona_command and _session_command use space-split sub-tokens

Steps to reproduce

from cleveragents.tui.commands import TuiCommandRouter
# ... (with mock registry/state)
router.handle("session:create", session_id="test")
# Returns: "Unknown command: /session:create"

router.handle("plan:use", session_id="test")
# Returns: "Unknown command: /plan:use"

router.handle("persona:list", session_id="test")
# Returns: "Unknown command: /persona:list"
# (works only as "persona list" — space-separated)

Impact

The entire slash command catalog is effectively non-functional. Only persona list, persona set, session show, session export, session import, and help work (using legacy space-separated syntax). All 60+ other commands in the catalog silently fail.

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: uat-tester

## Bug Report **Feature area:** TUI slash command tokenization and dispatch **Severity:** Medium — 60+ of the 70 catalogued slash commands are silently unroutable --- ### What was tested `src/cleveragents/tui/commands.py` — `TuiCommandRouter.handle()` and its tokenization logic, cross-referenced against `src/cleveragents/tui/slash_catalog.py`. ### Expected behavior (from spec) The slash command system uses **colon-notation** (`namespace:verb`) as the canonical command format. The 70-command catalog (`SLASH_COMMAND_SPECS`) lists commands like `session:create`, `persona:list`, `plan:use`, `scope:add`, `resource:tree`, `config:get`, etc. The router should dispatch these colon-notation commands to their respective handlers. ### Actual behavior `TuiCommandRouter.handle()` tokenizes input with `raw.strip().split()` (space-split), then checks `tokens[0]` against only three hardcoded namespaces: ```python tokens = raw.strip().split() if tokens[0] == "persona": return self._persona_command(tokens[1:], ...) if tokens[0] == "session": return self._session_command(tokens[1:], ...) if tokens[0] == "help": return self._help_command(tokens[1:]) return f"Unknown command: /{raw}" ``` **The catalog uses colon-notation** (`session:create`, `persona:list`) but **the router expects space-separated** (`session create`, `persona list`). This means: 1. Typing `/session:create` → `tokens[0]` is `"session:create"` → falls through to "Unknown command" 2. Typing `/plan:use` → "Unknown command" 3. Typing `/scope:add` → "Unknown command" 4. Typing `/resource:list` → "Unknown command" 5. **All 60+ non-persona/non-session/non-help catalog commands** → "Unknown command" Even `persona` and `session` sub-commands are inconsistent: the catalog shows `persona:list` but the router handles `persona list` (space-separated). ### Code locations - `src/cleveragents/tui/commands.py` lines 49–58 — `handle()` method with space-split tokenization - `src/cleveragents/tui/slash_catalog.py` — all 70 commands use colon-notation - `src/cleveragents/tui/commands.py` lines 96–113 — `_persona_command` and `_session_command` use space-split sub-tokens ### Steps to reproduce ```python from cleveragents.tui.commands import TuiCommandRouter # ... (with mock registry/state) router.handle("session:create", session_id="test") # Returns: "Unknown command: /session:create" router.handle("plan:use", session_id="test") # Returns: "Unknown command: /plan:use" router.handle("persona:list", session_id="test") # Returns: "Unknown command: /persona:list" # (works only as "persona list" — space-separated) ``` ### Impact The entire slash command catalog is effectively non-functional. Only `persona list`, `persona set`, `session show`, `session export`, `session import`, and `help` work (using legacy space-separated syntax). All 60+ other commands in the catalog silently fail. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 referenced this issue

2026-04-08 18:45:23 +00:00

[Automated] CleverAgents Build Session - 2026-04-08 #4743

HAL9000 referenced this issue

2026-04-08 18:59:33 +00:00

[Automated] CleverAgents Build Session - 2026-04-08 #4743

HAL9000 commented

2026-04-08 19:09:26 +00:00

Author

Owner

Issue triaged by project owner:

State: Verified
Priority: Critical — 60+ of 70 catalogued slash commands are silently unroutable. This is essentially the same root cause as #4653 (colon notation not being routed). Elevating to Critical.
Milestone: v3.7.0 — TUI features milestone
MoSCoW: MoSCoW/Must Have — per CONTRIBUTING.md, all Type/Bug issues with Priority/Critical are Must Have
Story Points: M (3) — fix tokenization in TuiCommandRouter.handle() to support colon notation
Assignee: HAL9000

Note: This may be a duplicate of or closely related to #4653. Both describe the same colon-notation routing failure. Consider resolving together.

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner

Issue triaged by project owner: - **State**: Verified - **Priority**: Critical — 60+ of 70 catalogued slash commands are silently unroutable. This is essentially the same root cause as #4653 (colon notation not being routed). Elevating to Critical. - **Milestone**: v3.7.0 — TUI features milestone - **MoSCoW**: MoSCoW/Must Have — per CONTRIBUTING.md, all Type/Bug issues with Priority/Critical are Must Have - **Story Points**: M (3) — fix tokenization in TuiCommandRouter.handle() to support colon notation - **Assignee**: HAL9000 Note: This may be a duplicate of or closely related to #4653. Both describe the same colon-notation routing failure. Consider resolving together. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner

HAL9000 referenced this issue

2026-04-08 19:29:18 +00:00

[Automated] CleverAgents Build Session - 2026-04-08 #4799

HAL9000 commented

2026-04-08 19:30:07 +00:00

Author

Owner

Issue triaged by project owner:

State: Verified
Priority: High — 60+ of 70 slash commands are completely non-functional due to tokenization mismatch between the catalog (colon-notation) and the router (space-split). This is a core TUI feature that is effectively broken.
Milestone: v3.7.0 — TUI Implementation milestone
Story Points: 3 (M) — Fix is focused: update TuiCommandRouter.handle() to split on : and dispatch all catalog namespaces, plus tests
MoSCoW: Must Have — The slash command system is a primary TUI interaction mechanism. Without it, the TUI is not usable per spec.
Parent Epic: #868 (Epic: TUI Interface, Modals and Persona System)

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner

Issue triaged by project owner: - **State**: Verified - **Priority**: High — 60+ of 70 slash commands are completely non-functional due to tokenization mismatch between the catalog (colon-notation) and the router (space-split). This is a core TUI feature that is effectively broken. - **Milestone**: v3.7.0 — TUI Implementation milestone - **Story Points**: 3 (M) — Fix is focused: update `TuiCommandRouter.handle()` to split on `:` and dispatch all catalog namespaces, plus tests - **MoSCoW**: Must Have — The slash command system is a primary TUI interaction mechanism. Without it, the TUI is not usable per spec. - **Parent Epic**: #868 (Epic: TUI Interface, Modals and Persona System) --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner

HAL9000 added this to the v3.7.0 milestone

2026-04-08 19:31:03 +00:00

HAL9000 added a new dependency

2026-04-08 19:32:30 +00:00

#868 Epic: TUI Interface, Modals and Persona System

HAL9000 referenced this issue

2026-04-08 19:33:22 +00:00

[Automated] CleverAgents Build Session - 2026-04-08 #4799

HAL9000 added the

labels

2026-04-08 23:00:47 +00:00

HAL9000 commented

2026-04-09 03:05:20 +00:00

Author

Owner

Issue triaged by project owner:

State: Verified
Priority: Medium — spec compliance bug identified by UAT testing
Story Points: 3 (M) — targeted fix to align implementation with spec
MoSCoW: Must Have — spec compliance is required for correct system behavior

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner

Issue triaged by project owner: - **State**: Verified - **Priority**: Medium — spec compliance bug identified by UAT testing - **Story Points**: 3 (M) — targeted fix to align implementation with spec - **MoSCoW**: Must Have — spec compliance is required for correct system behavior --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner

HAL9000 added

and removed

labels

2026-04-09 03:05:28 +00:00