-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Nest handoff history by default #1996
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Nest handoff history by default #1996
Conversation
rm-openai
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of kaz's feedback seems right. In addition, I'd say two major things:
- This setting should be on the
Handoffobject - Since you're updating the default, this is a breaking change, so please update the changelog.
jhills20
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved to Handoff, added handoff history .py file for non extension parts of handoff filter, added changelog
Summary
nest_handoff_historyflag toRunConfigand call a new helper that condenses the prior transcript into a developer-role summary when handing offTesting
uv run pytest tests/test_extension_filters.pyuv run pytest tests/test_agent_runner.py -k handoffuv run pytest tests/test_agent_runner_streamed.py -k handoffhttps://chatgpt.com/codex/tasks/task_i_68ff73bda0f4832496f3d1fa9103905f