Skip to content

fix(opencode): include cache writes in ACP usage - #40450

Merged
rekram1-node merged 1 commit into
devfrom
acp-cache-usage
Aug 5, 2026
Merged

rekram1-node merged 1 commit into
devfrom
acp-cache-usage

Conversation

@opencode-agent

@opencode-agent opencode-agent Bot commented Aug 4, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • include cache-write tokens when reporting ACP context usage
  • use the same context-token calculation across both ACP service paths
  • cover nonzero cache reads and writes with a regression test

Testing

  • bun test test/acp/usage.test.ts
  • Prettier check for the changed files

Requested by: @rekram1-node (Aiden Cline via Slack)

Closes: #40442

@rekram1-node
rekram1-node merged commit 9f38562 into dev Aug 5, 2026
9 checks passed
@rekram1-node
rekram1-node deleted the acp-cache-usage branch August 5, 2026 02:27
Spark-Liang pushed a commit to Spark-Liang/opencode that referenced this pull request Aug 18, 2026
Co-authored-by: Aiden Cline <rekram1-node@users.noreply.github.com>
moonug pushed a commit to moonug/opencode that referenced this pull request Aug 19, 2026
Co-authored-by: Aiden Cline <rekram1-node@users.noreply.github.com>
androidand pushed a commit to androidand/opencode-skein that referenced this pull request Sep 19, 2026
Co-authored-by: Aiden Cline <rekram1-node@users.noreply.github.com>
rldona pushed a commit to rldona/FlupCode that referenced this pull request Sep 24, 2026
Co-authored-by: Aiden Cline <rekram1-node@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ACP usage_update drops cache.write from the used field

1 participant