feat: proxy claude api calls to secure auth token#849
Conversation
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Claude
AI
changed the title
|
@claude[agent] the codex agent also supports api proxying through the OPENAI_BASE_URL environment variable. Using the same architecture used to secure claude, implement api-proxying for codex via the OPENAI_BASE_URL |
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Done! The documentation now clearly shows that Codex (OpenAI) uses the identical credential isolation architecture as Claude. Key updates in commit 0f73e98: Architecture confirmation:
Documentation enhancements:
The implementation already existed—the doc now gives equal treatment to both providers. |
Claude
AI
changed the title
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ Smoke test complete - GitHub MCP, file writing, and bash tools validated. Playwright blocked by network restrictions (expected in firewall environment). Results posted to PR #849. |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
🧪 Bun Build Test Results
Overall: ✅ PASS All Bun projects successfully installed dependencies and passed their test suites.
|
|
Smoke Test Results (Claude Engine) Last 2 Merged PRs:
Test Results:
Status: PARTIAL PASS (3/4 tests passed, Playwright blocked by network restrictions)
|
Deno Build Test Results
Overall: ✅ PASS All Deno tests completed successfully.
|
Node.js Build Test Results
Overall: ✅ PASS All Node.js projects installed and tested successfully.
|
C++ Build Test Results
Overall: PASS ✅ All C++ projects built successfully.
|
Go Build Test Results
Overall: PASS ✅ All Go projects built and tested successfully.
|
Chroot Mode Version Test Results
Overall Result: ❌ Tests FAILED - Not all runtimes matched between host and chroot environments. The chroot mode successfully accessed Go from the host, but Python and Node.js versions differ between the host and chroot environments.
|
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Fixed in ee0424b. OPENAI_BASE_URL is now disabled for Codex agents. The change is temporary and can be re-enabled in the future by uncommenting lines 1012-1016 in src/docker-manager.ts. |
Claude
AI
changed the title
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded. |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
| Metric | Base | PR | Delta |
|---|---|---|---|
| Lines | 82.82% | 82.83% | 📈 +0.01% |
| Statements | 82.81% | 82.74% | 📉 -0.07% |
| Functions | 82.74% | 82.74% | ➡️ +0.00% |
| Branches | 74.87% | 74.88% | ➡️ +0.01% |
📁 Per-file Coverage Changes (1 files)
| File | Lines (Before → After) | Statements (Before → After) |
|---|---|---|
src/docker-manager.ts |
84.7% → 84.6% (-0.09%) | 84.1% → 83.8% (-0.36%) |
Coverage comparison generated by scripts/ci/compare-coverage.ts
lpcox
changed the title
C++ Build Test Results
Overall: PASS ✅ All C++ projects built successfully.
|
Go Build Test Results
Overall: PASS ✅ All Go projects tested successfully.
|
Bun Build Test Results
Overall: PASS ✅ All Bun projects built and tested successfully.
|
Build Test: Node.js - Results
Overall: PASS ✅ All Node.js projects built and tested successfully.
|
Rust Build Test Results
Overall: PASS ✅ All Rust projects built successfully and passed their test suites.
|
Deno Build Test Results
Overall: ✅ PASS All Deno tests completed successfully.
|
.NET Build Test Results ✅All .NET projects successfully restored, built, and ran.
Overall: PASS Detailshello-world:
json-parse:
|
Smoke Test ResultsLast 2 Merged PRs:
Test Results:
Status: FAIL (Playwright blocked) cc: @lpcox
|
✅ Java Build Test ResultsAll Java projects compiled and tested successfully through the AWF firewall.
Overall: PASS Both projects successfully downloaded dependencies through the Squid proxy (172.30.0.10:3128) and passed all tests.
|
|
Smoke Test Results for Claude Engine Last 2 Merged PRs:
Test Results:
Overall Status: PASS
|
Chroot Version Comparison ResultsThe chroot mode test compared runtime versions between the host and chroot environment:
Overall Result: ❌ Tests did not pass The version mismatches indicate that the chroot environment is accessing container binaries rather than the host binaries as expected. This suggests that the chroot mode transparent host binary access is not working correctly for Python and Node.js.
|
|
PR titles:
|
2 participants
Uh oh!
There was an error while loading. Please reload this page.