Skip to content

Disable LLM gateway for Codex engine#15748

Merged
pelikhan merged 5 commits intomainfrom
copilot/update-codex-agentic-engine
Feb 14, 2026
Merged

Disable LLM gateway for Codex engine#15748
pelikhan merged 5 commits intomainfrom
copilot/update-codex-agentic-engine

Conversation

Copy link
Contributor

Copilot AI commented Feb 14, 2026

The Codex engine no longer requires LLM gateway proxying for API requests. Updated SupportsLLMGateway() to return -1, signaling no gateway support.

Changes

  • pkg/workflow/codex_engine.go: Changed SupportsLLMGateway() return value from constants.CodexLLMGatewayPort (10001) to -1
  • Test updates: Updated 4 test files to reflect that Codex no longer:
    • Includes API proxy Docker images in collected images
    • Adds --enable-api-proxy flag to AWF commands
    • Gets special treatment in strict mode validation (now treated like Copilot)
  • Workflow recompilation: Recompiled 10 Codex workflow lock files to reflect the LLM gateway changes:
    • Removed api-proxy Docker image from download steps
    • Removed --enable-api-proxy flag from AWF commands

Example

// Before
func (e *CodexEngine) SupportsLLMGateway() int {
    return constants.CodexLLMGatewayPort
}

// After  
func (e *CodexEngine) SupportsLLMGateway() int {
    return -1
}

The convention is: positive port numbers indicate LLM gateway support, -1 indicates no support. This change aligns Codex with Copilot and Custom engines, which also return -1.

Affected Workflows

The following Codex workflows were recompiled with the updated engine behavior:

  • changeset.lock.yml
  • codex-github-remote-mcp-test.lock.yml
  • daily-fact.lock.yml
  • daily-issues-report.lock.yml
  • daily-observability-report.lock.yml
  • daily-performance-summary.lock.yml
  • deep-report.lock.yml
  • duplicate-code-detector.lock.yml
  • issue-arborist.lock.yml
  • smoke-codex.lock.yml

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.


Changeset

  • Type: patch
  • Description: Disable the Codex LLM gateway proxy and update related workflows/tests to match the simplified engine behavior.

Generated by Changeset Generator

@pelikhan pelikhan added the smoke label Feb 14, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 14, 2026

Smoke Copilot SDK failed. Please review the logs for details.

@github-actions
Copy link
Contributor

🧪 Smoke Project is now testing project operations...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 14, 2026

Changeset Generator failed. Please review the logs for details.

@github-actions
Copy link
Contributor

🧪 Smoke Temporary ID is now testing temporary ID functionality...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 14, 2026

🎬 THE ENDSmoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

@github-actions
Copy link
Contributor

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 14, 2026

🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation...

Copilot AI changed the title [WIP] Update codex agentic engine to disable llm gateway Disable LLM gateway for Codex engine Feb 14, 2026
Copilot AI requested a review from pelikhan February 14, 2026 19:13
@github-actions
Copy link
Contributor

Smoke Temporary ID completed successfully. Temporary ID validation passed.

@github-actions
Copy link
Contributor

Agent Container Tool Check

Tool Status Version
bash 5.2.21
sh available
git 2.52.0
jq 1.7
yq 4.52.2
curl 8.5.0
gh 2.86.0
node 20.20.0
python3 3.12.3
go 1.24.13
java 21.x
dotnet 10.0.102

Result: 12/12 tools available ✅

All required development tools are present and accessible in the agent container environment.

AI generated by Agent Container Smoke Test

@pelikhan
Copy link
Contributor

@copilot recompile

@github-actions
Copy link
Contributor

Smoke Project completed successfully. All project operations validated.

@github-actions
Copy link
Contributor

🎯 Smoke Test Results: PARTIAL

PRs Reviewed (GitHub MCP):

PRs Queried (safeinputs-gh):

Test Results:

Overall Status: PARTIAL (8/8 core tests passed, 2 MCP tests skipped due to unavailable tools)


cc @pelikhan

AI generated by Smoke Copilot

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Smoke test review complete. Changes correctly disable LLM gateway for Codex engine by returning -1 from SupportsLLMGateway(). All related tests properly updated.

📰 BREAKING: Report filed by Smoke Copilot for issue #15748

Copy link
Contributor Author

Copilot AI commented Feb 14, 2026

@copilot recompile

Workflows recompiled. Updated 10 Codex workflow lock files that now:

  • No longer include api-proxy Docker image downloads
  • No longer add --enable-api-proxy flag to AWF commands

Commit: c7466cc

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/actions/download-artifact/git/ref/tags/v6
    • Triggering command: /usr/bin/gh gh api /repos/actions/download-artifact/git/ref/tags/v6 --jq .object.sha (http block)
  • https://api.github.com/repos/actions/github-script/git/ref/tags/v7
    • Triggering command: /usr/bin/gh gh api /repos/actions/github-script/git/ref/tags/v7 --jq .object.sha (http block)
  • https://api.github.com/repos/actions/setup-go/git/ref/tags/v5
    • Triggering command: /usr/bin/gh gh api /repos/actions/setup-go/git/ref/tags/v5 --jq .object.sha (http block)
  • https://api.github.com/repos/actions/setup-go/git/ref/tags/v6
    • Triggering command: /usr/bin/gh gh api /repos/actions/setup-go/git/ref/tags/v6 --jq .object.sha (http block)
  • https://api.github.com/repos/actions/setup-node/git/ref/tags/v6
    • Triggering command: /usr/bin/gh gh api /repos/actions/setup-node/git/ref/tags/v6 --jq .object.sha (http block)
  • https://api.github.com/repos/actions/upload-artifact/git/ref/tags/v6
    • Triggering command: /usr/bin/gh gh api /repos/actions/upload-artifact/git/ref/tags/v6 --jq .object.sha (http block)
    • Triggering command: /usr/bin/gh gh api /repos/actions/upload-artifact/git/ref/tags/v6 --jq .object.sha -c=4 -nolocalimports -importcfg /tmp/go-build1086959685/b001/importcfg -pack /home/REDACTED/work/gh-aw/gh-aw/cmd/gh-aw/main.go (http block)
  • https://api.github.com/user
    • Triggering command: /usr/bin/gh gh api user --jq .login (http block)

If you need me to access, download, or install something from one of these locations, you can either:

@github-actions
Copy link
Contributor

🧪 Smoke Temporary ID is now testing temporary ID functionality...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 14, 2026

Smoke Copilot SDK failed. Please review the logs for details.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 14, 2026

🎬 THE ENDSmoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

@github-actions
Copy link
Contributor

github-actions bot commented Feb 14, 2026

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions
Copy link
Contributor

github-actions bot commented Feb 14, 2026

Changeset Generator completed successfully!

@github-actions
Copy link
Contributor

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Contributor

🧪 Smoke Project is now testing project operations...

@github-actions
Copy link
Contributor

Smoke Temporary ID completed successfully. Temporary ID validation passed.

@github-actions
Copy link
Contributor

Agent Container Tool Check ✅

All required development tools are available in the agent container environment.

Tool Status Version
bash 5.2.21
sh available
git available
jq 1.7
yq 4.52.2
curl available
gh 2.86.0
node available
python3 available
go 1.24.13
java available
dotnet available

Result: 12/12 tools available ✅ PASS


Smoke test completed successfully - all development tools are accessible.

AI generated by Agent Container Smoke Test

@github-actions
Copy link
Contributor

Smoke Project completed successfully. All project operations validated.

@github-actions
Copy link
Contributor

⚠️ Security scanning failed for Smoke Copilot. Review the logs for details.

@github-actions
Copy link
Contributor

Smoke test (Codex) summary:
PRs: Add max limit enforcement to add_comment handler (SEC-003); Fix IMP-003: Document schema generation architecture and update conformance check
GitHub MCP: ✅
Serena MCP (activate+find_symbol): ❌ (find_symbol EOF)
Playwright: ❌ (tool unavailable)
File write + cat: ✅
Build (make build): ✅
Overall: FAIL

AI generated by Smoke Codex

@pelikhan pelikhan marked this pull request as ready for review February 14, 2026 19:54
Copilot AI review requested due to automatic review settings February 14, 2026 19:54
@pelikhan pelikhan merged commit b666255 into main Feb 14, 2026
@pelikhan pelikhan deleted the copilot/update-codex-agentic-engine branch February 14, 2026 19:54
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

💥 Automated smoke test review - all systems nominal!

💥 [THE END] — Illustrated by Smoke Claude for issue #15748

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR disables LLM gateway support for the Codex engine, simplifying its architecture by removing the API proxy layer. The change aligns Codex with other engines that don't require LLM gateway proxying (like Copilot and Custom engines).

Changes:

  • Modified Codex engine's SupportsLLMGateway() to return -1 instead of port 10001
  • Updated 4 test files to verify Codex no longer uses api-proxy images or --enable-api-proxy flags
  • Recompiled 10 Codex workflow lock files to remove api-proxy Docker images and --enable-api-proxy flags from AWF commands

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/workflow/codex_engine.go Changed SupportsLLMGateway() to return -1, indicating no gateway support
pkg/workflow/strict_mode_llm_gateway_test.go Updated test names and assertions to reflect Codex lacking LLM gateway support
pkg/workflow/enable_api_proxy_test.go Updated test to verify Codex AWF commands exclude --enable-api-proxy flag
pkg/workflow/docker_api_proxy_test.go Updated test to verify Codex workflows exclude api-proxy Docker image
.github/workflows/smoke-codex.lock.yml Removed api-proxy image and --enable-api-proxy flag from compiled workflow
.github/workflows/issue-arborist.lock.yml Removed api-proxy image and --enable-api-proxy flag from compiled workflow
.github/workflows/duplicate-code-detector.lock.yml Removed api-proxy image and --enable-api-proxy flag from compiled workflow
.github/workflows/deep-report.lock.yml Removed api-proxy image and --enable-api-proxy flag from compiled workflow
.github/workflows/daily-performance-summary.lock.yml Removed api-proxy image and --enable-api-proxy flag from compiled workflow
.github/workflows/daily-observability-report.lock.yml Removed api-proxy image and --enable-api-proxy flag from compiled workflow
.github/workflows/daily-issues-report.lock.yml Removed api-proxy image and --enable-api-proxy flag from compiled workflow
.github/workflows/daily-fact.lock.yml Removed api-proxy image and --enable-api-proxy flag from compiled workflow
.github/workflows/codex-github-remote-mcp-test.lock.yml Removed api-proxy image and --enable-api-proxy flag from compiled workflow
.github/workflows/changeset.lock.yml Removed api-proxy image and --enable-api-proxy flag from compiled workflow
.changeset/patch-disable-codex-llm-gateway.md Added changeset documenting the patch-level change

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants