Skip to content

Preserve tool_result ordering before reminders#58

Open
Gujiassh wants to merge 1 commit intoshareAI-lab:mainfrom
Gujiassh:fix/tool-result-ordering
Open

Preserve tool_result ordering before reminders#58
Gujiassh wants to merge 1 commit intoshareAI-lab:mainfrom
Gujiassh:fix/tool-result-ordering

Conversation

@Gujiassh
Copy link

@Gujiassh Gujiassh commented Mar 6, 2026

Summary

  • keep <reminder> blocks after tool results in s03_todo_write.py and s_full.py so Anthropic-compatible message ordering stays valid
  • add regression coverage that locks the reminder ordering behind the generated tool_result blocks

Testing

  • python3 -m unittest tests/test_tool_result_ordering.py -v
  • python3 -m py_compile agents/s03_todo_write.py agents/s_full.py tests/test_tool_result_ordering.py

Closes #55

@vercel
Copy link

vercel bot commented Mar 6, 2026

@Gujiassh is attempting to deploy a commit to the crazyboym's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

s03触发 400 BadRequest

1 participant