Skip to content

fix(agent): emit cumulative ReAct usage updates - #341

Merged
yuluo-yx merged 1 commit into
agentic-ai-java:mainfrom
alei-jia:codex/test-279-cumulative-usage
Nov 3, 2026
Merged

yuluo-yx merged 1 commit into
agentic-ai-java:mainfrom
alei-jia:codex/test-279-cumulative-usage

Conversation

@alei-jia

@alei-jia alei-jia commented Nov 1, 2026

Copy link
Copy Markdown

I acknowledge this contribution to ARGI; emit invocation-cumulative ReAct UsageUpdate snapshots for streaming and non-streaming model calls, project repeated current-turn snapshots without double counting, publish final response usage, and preserve unknown fields and invocation isolation; add 15 offline regression cases covering overwrite consumers, multiple turns, repeated and response-only stream usage, partial and unknown usage, overlapping runs, and failures; Agent Core: 142 tests passed, with upstream API and runtime-contract modules also passing; Fixes #279.

@yuluo-yx yuluo-yx left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The cumulative snapshots look correct, including repeated stream updates and unknown usage fields. CI is failing in NeutralToolRuntimeTest.interruptionCancelsSourceAndPreservesInterruptFlag, outside this change; please rerun the failed job before merging.

@yuluo-yx yuluo-yx left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yuluo-yx
yuluo-yx merged commit 59a5e9f into agentic-ai-java:main Nov 3, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG][P2] 独立 ReAct 的 UsageUpdate 应发送调用级累计用量而非单轮用量

2 participants