Use response_id to track token usage for MessageActions #6913
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
End-user friendly description of the problem this fixes or functionality that this introduces
Give a summary of what the PR does, explaining any non-trivial design decisions
@csmith49 I kept thinking about your question, and this is a proposal on what we could do to also track token usage for MessageActions. Personally, I normally hesitate to add properties to events, but the options we have here that I can think of are:
when we create actions here- not there, somewhere here plus the other places after we get theevent_id
from the stream - and save theevent_id
in it instead. Not sure this one is even possible, without returning more than the Action from the agentstep
.Link of any specific issues this addresses
Part of #6707
To run this PR locally, use the following command: