cleveragents/cleveragents-core

Fork 3

A2A facade missing `_cleveragents/session/create` and `_cleveragents/session/close` extension method routes (M6 spec gap) #8414

New issue

Open

opened 2026-04-13 18:42:53 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-13 18:42:53 +00:00

Owner

Metadata

Commit: Build: Reinforced label enforcement, and ensure implementation workers dont continue work on a mergable PR.
Branch: main
SHA: 5a9aaa79edaefb1a257114f054ea87facb8efe69
File: src/cleveragents/a2a/facade.py

Background and Context

ADR-047 specifies that all A2A operations must use the _cleveragents/ extension method prefix for spec-aligned routing. The M6 milestone (v3.5.0) acceptance criteria explicitly requires: "A2A facade session and plan lifecycle operations functional via CLI."

Session lifecycle operations (create, close) are currently only accessible via deprecated legacy names (session.create, session.close). The spec-aligned _cleveragents/session/create and _cleveragents/session/close routes are absent from both _EXTENSION_OPERATIONS and the handler dispatch map. Any client using the spec-aligned method names will receive A2aOperationNotFoundError.

Current Behavior

In facade.py, _EXTENSION_OPERATIONS contains plan, registry, context, health, sync, and namespace operations — but no session operations:

_EXTENSION_OPERATIONS: list[str] = [
    # Plan lifecycle
    "_cleveragents/plan/use",
    "_cleveragents/plan/execute",
    ...
    # Registry, Context, Health, Sync, Namespace — but NO session operations
]

Session operations are only in _LEGACY_OPERATIONS:

_LEGACY_OPERATIONS: list[str] = [
    "session.create",   # ← legacy only
    "session.close",    # ← legacy only
    ...
]

The handler map similarly only maps legacy names to session handlers.

Expected Behavior

Per ADR-047 and M6 spec, the following routes must be added:

_EXTENSION_OPERATIONS: list[str] = [
    ...
    # Session lifecycle (ADR-047 spec-aligned)
    "_cleveragents/session/create",
    "_cleveragents/session/close",
    "_cleveragents/session/list",    # optional but consistent
    "_cleveragents/session/get",     # optional but consistent
]

And the handler map must include:

"_cleveragents/session/create": self._handle_session_create,
"_cleveragents/session/close": self._handle_session_close,

Acceptance Criteria

_cleveragents/session/create is in _EXTENSION_OPERATIONS and dispatches to _handle_session_create
_cleveragents/session/close is in _EXTENSION_OPERATIONS and dispatches to _handle_session_close
list_operations() includes both new session routes
Legacy session.create and session.close routes remain for backward compatibility
BDD test: dispatch() with _cleveragents/session/create returns {"session_id": ..., "status": "created"}
BDD test: dispatch() with _cleveragents/session/close returns {"status": "closed"}
M6 acceptance criterion "A2A facade session and plan lifecycle operations functional via CLI" is satisfied

Subtasks

Add "_cleveragents/session/create" and "_cleveragents/session/close" to _EXTENSION_OPERATIONS
Add corresponding entries to _handlers() dispatch map
Consider adding _cleveragents/session/list and _cleveragents/session/get for completeness
Add BDD scenarios for spec-aligned session routes
Update list_operations() documentation to reflect new routes

Definition of Done

This issue is closed when _cleveragents/session/create and _cleveragents/session/close are routable via dispatch(), legacy routes remain functional, and BDD tests cover both spec-aligned and legacy paths.

Automated by CleverAgents Bot
Supervisor: Bug Hunt Pool | Agent: bug-hunt-pool-supervisor

## Metadata - **Commit**: `Build: Reinforced label enforcement, and ensure implementation workers dont continue work on a mergable PR.` - **Branch**: `main` - **SHA**: `5a9aaa79edaefb1a257114f054ea87facb8efe69` - **File**: `src/cleveragents/a2a/facade.py` ## Background and Context ADR-047 specifies that all A2A operations must use the `_cleveragents/` extension method prefix for spec-aligned routing. The M6 milestone (v3.5.0) acceptance criteria explicitly requires: *"A2A facade session and plan lifecycle operations functional via CLI."* Session lifecycle operations (`create`, `close`) are currently only accessible via deprecated legacy names (`session.create`, `session.close`). The spec-aligned `_cleveragents/session/create` and `_cleveragents/session/close` routes are absent from both `_EXTENSION_OPERATIONS` and the handler dispatch map. Any client using the spec-aligned method names will receive `A2aOperationNotFoundError`. ## Current Behavior In `facade.py`, `_EXTENSION_OPERATIONS` contains plan, registry, context, health, sync, and namespace operations — but **no session operations**: ```python _EXTENSION_OPERATIONS: list[str] = [ # Plan lifecycle "_cleveragents/plan/use", "_cleveragents/plan/execute", ... # Registry, Context, Health, Sync, Namespace — but NO session operations ] ``` Session operations are only in `_LEGACY_OPERATIONS`: ```python _LEGACY_OPERATIONS: list[str] = [ "session.create", # ← legacy only "session.close", # ← legacy only ... ] ``` The handler map similarly only maps legacy names to session handlers. ## Expected Behavior Per ADR-047 and M6 spec, the following routes must be added: ```python _EXTENSION_OPERATIONS: list[str] = [ ... # Session lifecycle (ADR-047 spec-aligned) "_cleveragents/session/create", "_cleveragents/session/close", "_cleveragents/session/list", # optional but consistent "_cleveragents/session/get", # optional but consistent ] ``` And the handler map must include: ```python "_cleveragents/session/create": self._handle_session_create, "_cleveragents/session/close": self._handle_session_close, ``` ## Acceptance Criteria - [ ] `_cleveragents/session/create` is in `_EXTENSION_OPERATIONS` and dispatches to `_handle_session_create` - [ ] `_cleveragents/session/close` is in `_EXTENSION_OPERATIONS` and dispatches to `_handle_session_close` - [ ] `list_operations()` includes both new session routes - [ ] Legacy `session.create` and `session.close` routes remain for backward compatibility - [ ] BDD test: `dispatch()` with `_cleveragents/session/create` returns `{"session_id": ..., "status": "created"}` - [ ] BDD test: `dispatch()` with `_cleveragents/session/close` returns `{"status": "closed"}` - [ ] M6 acceptance criterion "A2A facade session and plan lifecycle operations functional via CLI" is satisfied ## Subtasks - [ ] Add `"_cleveragents/session/create"` and `"_cleveragents/session/close"` to `_EXTENSION_OPERATIONS` - [ ] Add corresponding entries to `_handlers()` dispatch map - [ ] Consider adding `_cleveragents/session/list` and `_cleveragents/session/get` for completeness - [ ] Add BDD scenarios for spec-aligned session routes - [ ] Update `list_operations()` documentation to reflect new routes ## Definition of Done This issue is closed when `_cleveragents/session/create` and `_cleveragents/session/close` are routable via `dispatch()`, legacy routes remain functional, and BDD tests cover both spec-aligned and legacy paths. --- **Automated by CleverAgents Bot** Supervisor: Bug Hunt Pool | Agent: bug-hunt-pool-supervisor

HAL9000 added the

labels

2026-04-13 18:47:32 +00:00

HAL9000 added this to the v3.5.0 milestone

2026-04-13 18:51:06 +00:00

HAL9000 added

and removed

labels

2026-04-13 19:14:56 +00:00

HAL9000 commented

2026-04-13 19:19:02 +00:00

Author

Owner

✅ Verified — Missing _cleveragents/session/create and _cleveragents/session/close routes is an explicit M6 spec gap. These are required acceptance criteria for v3.5.0. MoSCoW: Must Have. [AUTO-OWNR-1]

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

✅ **Verified** — Missing `_cleveragents/session/create` and `_cleveragents/session/close` routes is an explicit M6 spec gap. These are required acceptance criteria for v3.5.0. **MoSCoW: Must Have**. [AUTO-OWNR-1] --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor