cleveragents/cleveragents-core
4
0
Fork 3
Code Issues 6.0k Pull Requests 115 Actions Packages Projects Releases Wiki Activity

fix(a2a): close session_id validation bypass in _handle_session_close

Browse Source 
Removed unreachable duplicate code left over after moving session_id validation
to the top of _handle_session_close(). Updated BDD test scenario in
a2a_facade_wiring.feature to cover the no-service + empty session_id path.

PR-CLOSED: #9250
This commit is contained in:
HAL 9000
2026-05-09 12:55:32 +00:00
committed by drew
parent 863be6780a
commit 901e3e360b
2 changed files with 14 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
features/a2a_facade_wiring.feature
+9
View File
@@ -34,6 +34,15 @@ Feature: A2A local facade wiring to live services
When I dispatch wired operation "session.close" with params {}
Then the wired response status should be "error"
# ---------------------------------------------------------------
# Validate session_id at entry — no service scenario (PR #9250)
# ---------------------------------------------------------------
Scenario: session.close without session_id and no service returns error
Given a wired A2aLocalFacade with no services
When I dispatch wired operation "session.close" with params {"session_id": ""}
Then the wired response status should be "error"
# ---------------------------------------------------------------
# Plan lifecycle wiring
# ---------------------------------------------------------------