wide-doctor-37830
03/03/2026, 4:16 PMcalm-artist-7206
03/03/2026, 7:39 PMview , rather than on state._events ?calm-artist-7206
03/03/2026, 7:40 PMview doesn't get to the LLM anywaywide-doctor-37830
03/03/2026, 7:50 PMview properties assume the event list is well-formed by the time the view is made, that way they can just focus on maintaining those properties instead of enforcing them. That way the views don't need all the events every time they're made/updated.wide-doctor-37830
03/03/2026, 7:52 PMview objects are just views.calm-artist-7206
03/03/2026, 8:03 PMcalm-artist-7206
03/03/2026, 8:05 PMtransform method on events; and because it creates a new Event which has to be in the right order. Maybe it's the right thing - because it's necessary, but it's interesting that it needs to. I don't know if we can assume that the event log is workablecalm-artist-7206
03/03/2026, 8:06 PMwide-doctor-37830
03/03/2026, 8:16 PMViewProperty. Unfortunately those properties can only be used to define contiguous groups (you have to limit their expressivity or the condenser has an impossible job), and the failure case might have the AgentErrorEvent pretty far from the actual tool call observation.
So I'm not surprised that the transform in the events_to_messages function is where Xingyao landed.wide-doctor-37830
03/03/2026, 8:19 PMwide-doctor-37830
03/03/2026, 8:20 PMAgentErrorEvent and the actual observation. Maybe we should work to ensure that error event is only added if the tool is actually broken?calm-artist-7206
03/03/2026, 8:24 PMcalm-artist-7206
03/03/2026, 8:24 PMwide-doctor-37830
03/03/2026, 8:35 PM