Skip to content

fix(todo): append reminders after tool results#208

Open
Gujiassh wants to merge 2 commits intoshareAI-lab:mainfrom
Gujiassh:fix/tool-result-ordering
Open

fix(todo): append reminders after tool results#208
Gujiassh wants to merge 2 commits intoshareAI-lab:mainfrom
Gujiassh:fix/tool-result-ordering

Conversation

@Gujiassh
Copy link
Copy Markdown
Contributor

Summary

  • append todo reminders after tool results instead of prepending them in the anthropic tool-result payload
  • keep tool result ordering stable for both the s03 and full agent loops
  • add focused regression coverage for the reminder ordering behavior

Testing

  • python3 -m unittest tests.test_tool_result_ordering

@Gujiassh Gujiassh force-pushed the fix/tool-result-ordering branch from 397cd51 to 35e7306 Compare April 11, 2026 12:12
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 11, 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.

@Gujiassh
Copy link
Copy Markdown
Contributor Author

Refresh complete on latest main.

  • Rebases applied to fix/tool-result-ordering and force-pushed at 8f83ca1
  • Resolved rebase conflict in agents/s03_todo_write.py while preserving tool_result-before-reminder behavior
  • Validation:
    • python3 -m unittest tests.test_tool_result_ordering
    • python3 -m py_compile agents/s03_todo_write.py agents/s_full.py tests/test_tool_result_ordering.py

Current red checks are Vercel authorization contexts; code-side CI checks are green.

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.

1 participant