Skip to content

feat: kong proxy for codex#796

Closed
Claude wants to merge 22 commits intomainfrom
claude/enable-api-proxy-by-default
Closed

feat: kong proxy for codex#796
Claude wants to merge 22 commits intomainfrom
claude/enable-api-proxy-by-default

Conversation

@Claude
Copy link
Contributor

@Claude Claude AI commented Feb 13, 2026

  • Research how OPENAI_API_KEY flows from host to Kong
  • Document the complete flow with code references
  • Provide user with comprehensive explanation

Claude AI and others added 2 commits February 13, 2026 04:00
Changed --enable-api-proxy flag default from false to true.
Updated types.ts @default documentation to reflect new default.
All existing tests pass.

Co-authored-by: lpcox <[email protected]>
Updated documentation to clarify that API proxy is now enabled by default.
Added examples showing that the flag is optional and how to disable it.

Co-authored-by: lpcox <[email protected]>
@Claude Claude AI changed the title [WIP] Enable API proxy by default instead of CLI flag feat(cli): enable API proxy by default Feb 13, 2026
@Claude Claude AI requested a review from lpcox February 13, 2026 04:02
@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

💫 TO BE CONTINUED... Smoke Claude failed! Our hero faces unexpected challenges...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

Chroot tests failed Smoke Chroot failed - See logs for details.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

⚠️ Coverage Regression Detected

This PR decreases test coverage. Please add tests to maintain coverage levels.

Overall Coverage

Metric Base PR Delta
Lines 82.92% 82.98% 📈 +0.06%
Statements 82.91% 82.97% 📈 +0.06%
Functions 82.74% 82.74% ➡️ +0.00%
Branches 74.92% 74.87% 📉 -0.05%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/docker-manager.ts 84.8% → 85.0% (+0.22%) 84.3% → 84.5% (+0.21%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

…#798)

* Initial plan

* fix: add --build-local flag to example scripts

Co-authored-by: lpcox <[email protected]>

* fix: add --build-local to github-copilot.sh for consistency

Co-authored-by: lpcox <[email protected]>

* fix: build only api-proxy locally, pull squid and agent from GHCR

Co-authored-by: lpcox <[email protected]>

---------

Co-authored-by: anthropic-code-agent[bot] <[email protected]>
Co-authored-by: lpcox <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

💫 TO BE CONTINUED... Smoke Claude failed! Our hero faces unexpected challenges...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

Chroot tests failed Smoke Chroot failed - See logs for details.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

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

* Initial plan

* fix: add missing package-lock.json for api-proxy container

Co-authored-by: lpcox <[email protected]>

---------

Co-authored-by: anthropic-code-agent[bot] <[email protected]>
Co-authored-by: lpcox <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

💫 TO BE CONTINUED... Smoke Claude failed! Our hero faces unexpected challenges...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

Chroot tests failed Smoke Chroot failed - See logs for details.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

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

* Initial plan

* fix(api-proxy): keep containers running when no API keys present

The api-proxy container was exiting immediately when no API keys were
configured because the HTTP servers were only started conditionally.
This caused docker-compose health checks to fail and prevented
containers from starting.

Changed server.js to always start both HTTP servers (ports 10000 and
10001), with conditional middleware setup based on API key presence.
This keeps the container running and health checks passing even when
API keys are not configured.

Fixes issue where test-examples workflow failed with "dependency
failed to start: container awf-api-proxy exited (0)"

Co-authored-by: lpcox <[email protected]>

---------

Co-authored-by: anthropic-code-agent[bot] <[email protected]>
Co-authored-by: lpcox <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

💫 TO BE CONTINUED... Smoke Claude failed! Our hero faces unexpected challenges...

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

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

@github-actions
Copy link
Contributor

Deno Build Test Results

Project Tests Status
oak 1/1 ✅ PASS
std 1/1 ✅ PASS

Overall: ✅ PASS

All Deno tests completed successfully.

AI generated by Build Test Deno

@github-actions
Copy link
Contributor

Node.js Build Test Results

Project Install Tests Status
clsx PASS PASS
execa PASS PASS
p-limit PASS PASS

Overall: PASS

All Node.js projects built and tested successfully.

AI generated by Build Test Node.js

@github-actions
Copy link
Contributor

C++ Build Test Results

Project CMake Build Status
fmt PASS
json PASS

Overall: PASS

All C++ projects built successfully with CMake.

AI generated by Build Test C++

@github-actions
Copy link
Contributor

Build Test: Bun ✅

Test Results:

Project Install Tests Status
elysia 1/1 PASS ✅
hono 1/1 PASS ✅

Overall: PASS ✅

All Bun build tests completed successfully.

AI generated by Build Test Bun

@github-actions
Copy link
Contributor

Deno Build Test Results

Project Tests Status
oak 1/1 ✅ PASS
std 1/1 ✅ PASS

Overall: ✅ PASS

All Deno tests completed successfully.

AI generated by Build Test Deno

@github-actions
Copy link
Contributor

Smoke Test Results

Last 2 merged PRs:

  • feat(api-proxy): replace Node.js proxy with nginx
  • fix(api-proxy): remove anthropic credentials from api-proxy service

✅ GitHub MCP
✅ Playwright (GitHub page title verified)
✅ File Writing
✅ Bash Tool

Status: PASS

AI generated by Smoke Claude

@github-actions
Copy link
Contributor

Go Build Test Results

Project Download Tests Status
color 1/1 PASS
env 1/1 PASS
uuid 1/1 PASS

Overall: PASS

All Go projects built and tested successfully.

AI generated by Build Test Go

@github-actions
Copy link
Contributor

.NET Build Test Results

Project Restore Build Run Status
hello-world PASS
json-parse PASS

Overall: PASS

All .NET projects built and ran successfully.

AI generated by Build Test .NET

@github-actions
Copy link
Contributor

Chroot Version Comparison Test Results

Runtime Host Version Chroot Version Match?
Python 3.12.12 3.12.3 ❌ NO
Node.js v24.13.0 v20.20.0 ❌ NO
Go go1.22.12 go1.22.12 ✅ YES

Overall Result: ❌ Tests did not pass (1/3 runtimes matched)

The chroot environment has different versions for Python and Node.js compared to the host. Only Go versions match. This indicates that the chroot environment is not using the host's runtime installations as expected.

AI generated by Smoke Chroot

@lpcox lpcox changed the title feat: kang proxy for codex feat: kong proxy for codex Feb 13, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

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

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

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

@github-actions
Copy link
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 82.88% 83.22% 📈 +0.34%
Statements 82.88% 83.25% 📈 +0.37%
Functions 82.74% 82.74% ➡️ +0.00%
Branches 74.87% 75.75% 📈 +0.88%
📁 Per-file Coverage Changes (2 files)
File Lines (Before → After) Statements (Before → After)
src/squid-config.ts 99.4% → 99.5% (+0.07%) 99.4% → 99.5% (+0.07%)
src/docker-manager.ts 84.7% → 85.2% (+0.54%) 84.2% → 84.9% (+0.71%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

@github-actions
Copy link
Contributor

Smoke Test Results (Run ID: 21999937203)

Overall: PASS 🎉

cc @Mossaka

AI generated by Smoke Copilot

@github-actions
Copy link
Contributor

C++ Build Test Results

Project CMake Build Status
fmt PASS
json PASS

Overall: PASS

All C++ projects built successfully through the AWF firewall.

AI generated by Build Test C++

@github-actions
Copy link
Contributor

Build Test: Node.js - Results

Project Install Tests Status
clsx PASS PASS
execa PASS PASS
p-limit PASS PASS

Overall: PASS

All Node.js projects installed and tested successfully.

AI generated by Build Test Node.js

@github-actions
Copy link
Contributor

Smoke Test: PASS

Last 2 merged PRs:

Test Results:

  • ✅ GitHub MCP: Retrieved PR data
  • ✅ Playwright: Verified github.com page title
  • ✅ File Write: Created test file successfully
  • ✅ Bash: Verified file contents

AI generated by Smoke Claude

@github-actions
Copy link
Contributor

Bun Build Test Results

Project Install Tests Status
elysia 1/1 PASS ✅
hono 1/1 PASS ✅

Overall: PASS ✅

All Bun build tests completed successfully with Bun v1.3.9.

AI generated by Build Test Bun

@github-actions
Copy link
Contributor

Rust Build Test Results

Project Build Tests Status
fd 1/1 PASS
zoxide 1/1 PASS

Overall: PASS

All Rust projects built and tested successfully.

AI generated by Build Test Rust

@github-actions
Copy link
Contributor

Go Build Test Results

Project Download Tests Status
color 1/1 PASS
env 1/1 PASS
uuid 1/1 PASS

Overall: PASS

All Go projects successfully downloaded dependencies and passed tests.

AI generated by Build Test Go

@github-actions
Copy link
Contributor

Deno Build Test Results

Project Tests Status
oak 1/1 ✅ PASS
std 1/1 ✅ PASS

Overall: ✅ PASS

All Deno tests passed successfully!

AI generated by Build Test Deno

@github-actions
Copy link
Contributor

Java Build Test Results ✅

All Java projects successfully compiled and tested through the firewall.

Project Compile Tests Status
gson 1/1 PASS
caffeine 1/1 PASS

Overall: PASS

Configuration

  • Maven proxy configured with IP 172.30.0.10:3128
  • All dependencies downloaded through Squid proxy
  • Both projects built and tested successfully

AI generated by Build Test Java

@github-actions
Copy link
Contributor

Chroot Version Comparison Results

Runtime Host Version Chroot Version Match?
Python Python 3.12.12 Python 3.12.3 ❌ NO
Node.js v24.13.0 v20.20.0 ❌ NO
Go go1.22.12 go1.22.12 ✅ YES

Status: Tests failed - version mismatches detected for Python and Node.js.

The chroot environment is successfully accessing host binaries, but the versions differ from the host system. This may indicate that the chroot is picking up different binaries from the mounted host filesystem.

AI generated by Smoke Chroot

@github-actions
Copy link
Contributor

PR titles:
feat(ci): add api-proxy image to release pipeline
perf: batch chroot integration tests to reduce container overhead
feat: kong proxy for codex
Tests: 1✅ 2✅ 3✅ 4❌ 5✅ 6✅ 7✅ 8✅
Overall: FAIL

AI generated by Smoke Codex

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.

3 participants