Skip to content

fix(v2): align initialization and JSON-RPC batch behavior - #183

Draft
PsiACE wants to merge 1 commit into
acp-tck-evaluationfrom
fix/v2-mandatory-clauses
Draft

PsiACE wants to merge 1 commit into
acp-tck-evaluationfrom
fix/v2-mandatory-clauses

Conversation

@PsiACE

@PsiACE PsiACE commented Nov 4, 2026 •

Copy link
Copy Markdown
Member

Fix ACP v2 alpha.5 version negotiation, omitted params, and JSON-RPC batches.

Verified: make check; make test (505 passed, 1 skipped).

@PsiACE
PsiACE added this pull request to stack #184 November 5, 2026 03:28
Answer a v2 initialize request for any protocol version with the version
this runtime supports instead of rejecting it with Invalid params, and
treat an omitted `params` member as "no arguments" so all-optional
methods such as `session/list` validate.

Decode a JSON-RPC batch into a list, dispatch each entry as it would be
in its own frame, and return the collected replies as one ordered array.
Empty batches answer a single -32600, malformed entries answer in place,
and response entries are routed rather than echoed. Batches are carried
by the ndjson, SSE and WebSocket transports; Streamable HTTP POST keeps
answering a batch root with 501, so the HTTP client refuses a batch send.
@PsiACE
PsiACE force-pushed the fix/v2-mandatory-clauses branch from db456be to 6b5d9ed Compare November 7, 2026 03:38
@PsiACE PsiACE changed the title fix: align ACP v2 initialization and JSON-RPC batch behavior, and gate mandatory v2 TCK outcomes fix(v2): align initialization and JSON-RPC batch behavior Nov 7, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant