Skip to content

Expose Codex standalone web search for OpenAI Responses Lite models - #67246

Merged
pelikhan merged 7 commits into
mainfrom
copilot/fix-web-search-tool-issue
Oct 9, 2026
Merged

pelikhan merged 7 commits into
mainfrom
copilot/fix-web-search-tool-issue

Conversation

Copilot AI commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Codex gives Responses Lite models a standalone web.run tool rather than hosted web_search, but the generated openai-proxy provider did not advertise support for it. As a result, models such as gpt-5.6-terra had no search tool even with hosted web enabled.

  • Provider configuration: Set supports_standalone_web_search = true for Codex’s OpenAI proxy only when network.hosted-web is enabled. Leave the flag absent otherwise.
  • Policy validation: Reject network.hosted-web.max-uses for Codex with OpenAI inference because standalone search does not support that limit.
  • Coverage and docs: Add config and validation tests, document the behavior, and regenerate the two affected workflow lock files.

The separate AWF api-proxy rejection of standalone search calls remains outside this change.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix web search tool availability for Responses Lite models Expose Codex standalone web search for OpenAI Responses Lite models Oct 9, 2026
Copilot AI requested a review from pelikhan October 9, 2026 17:44
@pelikhan
pelikhan marked this pull request as ready for review October 9, 2026 18:37
Copilot AI balanced review requested due to automatic review settings October 9, 2026 18:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The implementation, validation, tests, documentation, and generated outputs are consistent with the stated requirements.

0 open findings

What changed in this PR

Enables Codex standalone web search for OpenAI Responses Lite models when hosted web is configured.

Changes:

  • Advertises standalone search in generated Codex configuration.
  • Rejects unsupported max-uses combinations.
  • Adds tests, documentation, and regenerated workflow locks.
File Description
pkg/​workflow/​network_firewall_validation.go Validates unsupported Codex limits.
pkg/​workflow/​network_firewall_validation_test.go Covers provider-specific validation.
pkg/​workflow/​codex_config.go Adds the standalone-search provider flag.
pkg/​workflow/​codex_config_test.go Tests flag inclusion and omission.
docs/​src/​content/​docs/​engines/​codex.md Documents Responses Lite behavior.
.github/​workflows/​smoke-codex.lock.yml Regenerates Codex smoke configuration.
.github/​workflows/​daily-arxiv-researcher.lock.yml Regenerates researcher configuration.

🧠 Review effort: Balanced


💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot address the following outstanding work in one pass:

  1. Update this branch with the latest main using make merge-main, resolving any conflicts and preserving the intended changes.
  2. Fix failing check Compile queue phases (FAILURE): https://github.com/github/gh-aw/actions/runs/37966900298/job/113969493060.
  3. Fix failing check impacted-go-tests (FAILURE): https://github.com/github/gh-aw/actions/runs/37966900326/job/113969762486.
  4. Fix failing check lint-go-custom (FAILURE): https://github.com/github/gh-aw/actions/runs/37966900326/job/113969762377.
  5. Fix failing check verify-integration-build (FAILURE): https://github.com/github/gh-aw/actions/runs/37966900326/job/113969762390.

Push the necessary fixes, reply to each listed review thread and resolve it when addressed. Ignore feedback already answered or resolved. Use the pr-finisher skill and stop when only human review or CI remains; do not trigger CI.

Sous-chef head: 6123eda
Sous-chef work: 41c633c50467b0f2dc87116833139ebf287e1ad59f1a015d8d1dfedd9d1b23f1 42e5d572d9d75b7168fe7d0aeda0717cc07e65ed240f6d299e6fec661214f1ac 57b160f058c0cf0ae8470606e768b98956d9e31df7d52cc35017efdeeaeffab1 cabafb2817b103330ac0d1e2e098f214c40c4ca2be128895a8b14391f7e3306f
Sous-chef state: d65634e8b7a031398d0dff75173a0f9bcc91ca94a576b617a42105058f2290c2

Generated by 👨‍🍳 PR Sous Chef · pi · haiku45 · 7.09 AIC · ⌖ 7.65 AIC · ⊞ 1K · ◷
Comment /souschef to run again

Copilot AI and others added 2 commits October 9, 2026 19:18
…h-tool-issue

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

Copilot AI commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

@copilot address the following outstanding work in one pass:

  1. Update this branch with the latest main using make merge-main, resolving any conflicts and preserving the intended changes....

Merged the latest main and fixed the reported build/test and linter failures in commit f553cea. The CI checks still need a human-triggered rerun; this is a top-level PR comment, not a resolvable review thread.

Copilot AI requested a review from gh-aw-bot October 9, 2026 19:42
@pelikhan

pelikhan commented Oct 9, 2026

Copy link
Copy Markdown
Collaborator

@copilot the compiler should flag codex + provide copilot as not functional

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

@copilot the compiler should flag codex + provide copilot as not functional

Added a compile-time error when enabled network.hosted-web is configured for Codex using Copilot inference, with regression coverage and documentation. Committed as bc3290e.

@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot address the following outstanding work in one pass:

  1. Update this branch with the latest main using make merge-main, resolving any conflicts and preserving the intended changes.

Push the necessary fixes, reply to each listed review thread and resolve it when addressed. Ignore feedback already answered or resolved. Use the pr-finisher skill and stop when only human review or CI remains; do not trigger CI.

Sous-chef head: bc3290e
Sous-chef work:
Sous-chef state: b45d59b5d40f2cd1ef5e78ff877e9b24902370b78a8e64ba32969fcdd4b495b6

Generated by 👨‍🍳 PR Sous Chef · pi · haiku45 · 4.56 AIC · ⌖ 6.99 AIC · ⊞ 1K · ◷
Comment /souschef to run again

…h-tool-issue

# Conflicts:
#	pkg/workqueue/replay.go

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

Copilot AI commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

@copilot address the following outstanding work in one pass:

  1. Update this branch with the latest main using make merge-main, resolving any conflicts and preserving the intended changes....

Merged the latest main and resolved the duplicate test helper; commit 73c143c. The final impacted test/workflow gate passed.

@pelikhan

pelikhan commented Oct 9, 2026

Copy link
Copy Markdown
Collaborator

@copilot make it a capability of the agentic engine interface. Do not leak abstractions in the code

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

@copilot make it a capability of the agentic engine interface. Do not leak abstractions in the code

Added workflow-aware hosted-web capabilities to the engine interface and removed Codex-specific branching from generic validation. Commit: bd7b2a1.

@pelikhan
pelikhan merged commit 25c5ac9 into main Oct 9, 2026
27 checks passed
@pelikhan
pelikhan deleted the copilot/fix-web-search-tool-issue branch October 9, 2026 22:12
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.

Codex: no web search tool for Responses Lite models (openai-proxy provider lacks supports_standalone_web_search)

4 participants