Skip to content

[codex] Exclude external tool output from memories#26821

Merged
rka-oai merged 4 commits into
mainfrom
codex/standalone-web-search-memory
Jun 8, 2026
Merged

[codex] Exclude external tool output from memories#26821
rka-oai merged 4 commits into
mainfrom
codex/standalone-web-search-memory

Conversation

@rka-oai

@rka-oai rka-oai commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add contains_external_context() to tool output so other tools can be opted out of influencing memory when disable_on_external_context=true
  • Classify standalone web-search output as external context (to match behavior as hosted web search)
  • Verify with integration test

@rka-oai rka-oai changed the title [codex] Exclude standalone web search from memories [codex] Add func to mark tool output as external context (to not include in memory when disable_on_external_context on) Jun 6, 2026
@rka-oai rka-oai marked this pull request as ready for review June 6, 2026 20:18
@rka-oai rka-oai requested a review from a team as a code owner June 6, 2026 20:18
@rka-oai rka-oai requested a review from sayan-oai June 6, 2026 20:19
@rka-oai rka-oai changed the title [codex] Add func to mark tool output as external context (to not include in memory when disable_on_external_context on) [codex] Exclude external tool output from memories Jun 6, 2026
Base automatically changed from codex/v1-search-code-mode to main June 8, 2026 06:18
@rka-oai rka-oai force-pushed the codex/standalone-web-search-memory branch from 7a7bbf3 to d9c87f3 Compare June 8, 2026 06:24
@rka-oai rka-oai enabled auto-merge (squash) June 8, 2026 06:25
@rka-oai rka-oai merged commit 26d9329 into main Jun 8, 2026
31 checks passed
@rka-oai rka-oai deleted the codex/standalone-web-search-memory branch June 8, 2026 16:53
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants