Skip to content

Conversation

eneufeld
Copy link
Contributor

@eneufeld eneufeld commented Jun 2, 2025

What it does

During multiple tool requests, the anthropic provider was passing all tool results over and over again. This resulted in unreasonable high token usage due to repetetition.

How to test

Using a prompt like:
@architect retrieve the directory structure and then search the workspace for "Jonas" and then search for "Philip"

With Claude before the fix and after the fix should reduce the tokens used drastically.

Follow-ups

Breaking changes

  • This PR introduces breaking changes and requires careful review. If yes, the breaking changes section in the changelog has been updated.

Attribution

Review checklist

Reminder for reviewers

@eneufeld eneufeld requested a review from JonasHelming June 2, 2025 20:00
@github-project-automation github-project-automation bot moved this to Waiting on reviewers in PR Backlog Jun 2, 2025
During multiple tool requests, the anthropic provider was
passing all tool results over and over again. This resulted in
unreasonable high token usage due to repetetition.
@eneufeld eneufeld force-pushed the fix/anthropic-toolresults branch from dce9f60 to ecc7997 Compare June 2, 2025 20:11
@JonasHelming JonasHelming requested a review from planger June 2, 2025 20:57
@github-project-automation github-project-automation bot moved this from Waiting on reviewers to Needs merge in PR Backlog Jun 2, 2025
@eneufeld eneufeld merged commit ee5e882 into master Jun 2, 2025
10 of 11 checks passed
@github-project-automation github-project-automation bot moved this from Needs merge to Done in PR Backlog Jun 2, 2025
@github-actions github-actions bot added this to the 1.63.0 milestone Jun 2, 2025
@sdirix sdirix deleted the fix/anthropic-toolresults branch June 4, 2025 09:34
@ndoschek ndoschek mentioned this pull request Jun 13, 2025
61 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants