Skip to content

feat(server): ACP sessionUpdate parity for Grok clients - #5422

Closed
EnzoTironi wants to merge 2 commits into
pingdotgg:mainfrom
EnzoTironi:pr/grok-acp-parser
Closed

EnzoTironi wants to merge 2 commits into
pingdotgg:mainfrom
EnzoTironi:pr/grok-acp-parser

fix(server): wire ACP UsageUpdated through makeAcpTokenUsageEvent

a45fb3d
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Approvability Check completed Aug 6, 2026 in 2m 47s

Not eligible (click on check for details)

  • Eligibility
  • Correctness

Note

More information about how approvability works can be found in our Help Center.

Details

Eligibility
This PR adds substantial new feature logic for parsing multiple ACP session update event types. Additionally, there's an unresolved review comment identifying that ConfigOptionsUpdated events are parsed but never applied to the config state, potentially causing stale state issues in subsequent operations.

Notes:
No code owners found in target merge base de592a0

Correctness
1 blocking correctness issue found