Nothing here yet.
Nothing here yet.
That’s a really good point — thank you for calling this out. I completely agree that the approval should be tied to the specific revision being reviewed, rather than just the stable glossary ID. The A → B race-condition test is especially useful and makes the failure mode very clear. I’ll incorporate this into the design and tests.
That's a great point. You're right that showing only input_tokens can hide a huge part of the actual input usage when prompt caching is involved. I'll look into splitting cache_read_input_tokens and cache_creation_input_tokens in the per-model breakdown. And yes, the SSE parsing is probably one of the more interesting parts of the proxy — thanks for the feedback!
