Repository navigation
Grok/ACP: sessionUpdate parity for live Grok agent stdio #7022
Closed
EnzoTironi
started this conversation in
Ideas
Replies: 1 comment
|
Thanks for spelling out the missing ACP updates. The orchestrator V2 work has now merged in #2829. The V2 shared ACP path now handles reasoning chunks, usage, command/config updates and session metadata. Providers consume that common update handling rather than needing the old per-adapter patches. Closing this as delivered. If a specific part is still missing in a build containing V2, please open a focused follow-up with the provider/version and the behavior you are seeing. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Before submitting
Area
apps/server
Problem or use case
Live Grok (
grok agent stdio) emits ACPsession/updatekinds and xAI extension hooks that the sharedAcpRuntimeModeldoes not fully model. Adapters either drop them or cast ad hoc.Proposed solution
Extend shared ACP parsing (
AcpRuntimeModel/AcpCoreRuntimeEvents) for:agent_thought_chunk,usage_update,available_commands_update,config_option_update,session_info_update,user_message_chunk, unknown fallback, and xAI exit_plan_mode helpers.Why this matters
Without shared parser coverage, every ACP provider invents its own casting. Grok (and Cursor) cannot surface reasoning, usage, or commands reliably.
Smallest useful scope
Parser + mock-agent fixtures only. No Grok product brand, no UI.
Alternatives considered
Per-adapter casts in GrokAdapter only — works short-term but breaks shared Cursor path and exhaustiveness.
Risks or tradeoffs
Shared streaming path; must stay additive and forward-compatible via unknown updates.
Examples or references
PR #5422. Related open usage work: #5405.
All reactions