fix s03: move reminder text block after tool_result blocks#210
Open
tensaPetalum wants to merge 3 commits intoshareAI-lab:mainfrom
Open
fix s03: move reminder text block after tool_result blocks#210tensaPetalum wants to merge 3 commits intoshareAI-lab:mainfrom
tensaPetalum wants to merge 3 commits intoshareAI-lab:mainfrom
Conversation
The reminder was being inserted at index 0 of the user message content array, placing it before tool_result blocks. Claude API requires tool_result blocks to appear before any other content type when the previous assistant message contained tool_use blocks. Fix by appending the reminder after tool results instead of prepending it.
|
@tensaPetalum is attempting to deploy a commit to the crazyboym's projects Team on Vercel. A member of the Team first needs to authorize it. |
tensaPetalum
changed the title
CrazyBoyM
force-pushed
the
main
branch
2 times, most recently
from
36897b1 to
d882d01
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Problem
When the plan reminder fires, it was inserted at index 0 of the user
message content array before the
tool_resultblocks. The AnthropicAPI requires that when the previous assistant message contained
tool_useblocks, the next user message must begin withtool_resultblocks. Any other content type appearing first causes a 400 error:
Fix
Change
results.insert(0, ...)toresults.append(...)so thereminder text block always follows all tool results.
Reproduction
Run a multi-step task long enough to trigger the reminder interval
(
PLAN_REMINDER_INTERVAL = 3rounds without atodocall). The 400will fire on the next API call after the reminder is injected.