Skip to content

docs: clarify inter-turn input filtering#3698

Closed
maxpetrusenkoagent wants to merge 1 commit into
openai:mainfrom
maxpetrusenkoagent:hermes/oss-pr-2026-06-28-openai-agents-python-2671
Closed

docs: clarify inter-turn input filtering#3698
maxpetrusenkoagent wants to merge 1 commit into
openai:mainfrom
maxpetrusenkoagent:hermes/oss-pr-2026-06-28-openai-agents-python-2671

Conversation

@maxpetrusenkoagent

Copy link
Copy Markdown

Summary

  • document the supported call_model_input_filter pattern for injecting queued user/application input between tool turns
  • clarify the boundaries: the filter changes the next model payload, but does not mutate RunResult.input, auto-persist injected items to a client-managed session, or pause the run
  • add non-streamed and streamed characterization tests showing queued context messages injected after a tool call and before the next model turn

This follows the maintainer guidance on #2671 that RunConfig.call_model_input_filter is the existing supported hook for history compaction and queued external-message injection between tool turns, without adding a new turn hook/interceptor API.

Test plan

  • make format
  • uv run pytest tests/test_call_model_input_filter.py -q
  • uv run mypy tests/test_call_model_input_filter.py
  • uv run pyright tests/test_call_model_input_filter.py --project pyrightconfig.json
  • uv run mkdocs build
  • Second-agent review: claude -p on /tmp/oss-pr-second-agent-review.diff returned CLEAN with no blocking findings after the doc-example nit was fixed.

Full-suite note: I also ran the repo's $code-change-verification flow. make lint passed, but the full make tests/make typecheck gates are currently noisy in this local Python 3.14 environment due to unrelated optional-dependency and existing sandbox/voice/model failures outside this diff. The focused tests and file-level type checks above pass.

Issue number

Related to #2671

Checks

  • I've added new tests (if relevant)
  • I've added/updated the relevant documentation
  • I've run make lint and make format
  • I've made sure tests pass

@seratch seratch added the documentation Improvements or additions to documentation label Jun 29, 2026
@seratch

seratch commented Jun 29, 2026

Copy link
Copy Markdown
Member

Thanks for the suggestion. We don't generally encourage appending user messages this way at this moment, so let me close this PR. Regarding the queued message use case, we're aware of the need. So, we may come up with some solution for it in the future.

@seratch seratch closed this Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants