fix(api-proxy): add missing JS modules to Dockerfile COPY#2950
Conversation
The OIDC refactoring PRs (#2811, #2772, #2887) added new JS modules (github-oidc.js, aws-oidc-token-provider.js, gcp-oidc-token-provider.js, oidc-refresh-utils.js) but did not update the Dockerfile COPY command. This caused the api-proxy container to crash immediately on startup with exit code 1 (Cannot find module './github-oidc'), breaking all integration tests since commit 7c25298. Fixes the api-proxy container startup crash that has been failing all integration test runs on main since 2026-05-11T15:45Z. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (1 files)
Coverage comparison generated by |
Smoke Test Results
Overall: FAIL (1 of 4 tests failed)
|
🔥 Smoke Test: Copilot BYOK (Offline) Mode
Running in BYOK offline mode ( Overall: PASS (MCP auth failure is environment-level, not BYOK-related)
|
There was a problem hiding this comment.
Pull request overview
Fixes the api-proxy container build so required OIDC-related runtime modules are actually present in the image, preventing startup crashes like Cannot find module './github-oidc' and restoring integration test stability.
Changes:
- Extends the
containers/api-proxy/DockerfileCOPYlist to include newly introduced OIDC helper/provider modules. - Ensures AWS/GCP OIDC providers and shared refresh utilities are available at runtime inside the container image.
Show a summary per file
| File | Description |
|---|---|
containers/api-proxy/Dockerfile |
Adds missing JS modules to the Docker build context so the container includes required OIDC runtime dependencies. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 1
| COPY server.js logging.js metrics.js rate-limiter.js token-tracker.js \ | ||
| model-resolver.js proxy-utils.js anthropic-transforms.js \ | ||
| proxy-request.js model-discovery.js management.js oidc-token-provider.js ./ | ||
| proxy-request.js model-discovery.js management.js oidc-token-provider.js \ | ||
| github-oidc.js aws-oidc-token-provider.js gcp-oidc-token-provider.js \ | ||
| oidc-refresh-utils.js ./ |
🔬 Smoke Test Results
Overall: FAIL Pre-step template variables ( /cc @lpcox
|
|
Smoke test Codex: FAIL Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "registry.npmjs.org"See Network Configuration for more information.
|
Smoke Test Results
Overall: FAIL —
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS
|
Bug Fix
What was the bug?
The api-proxy container has been crashing immediately on startup (exit code 1) since commit 7c25298, breaking all integration test runs on main since 2026-05-11T15:45Z.
The root cause: PR #2887 (and earlier PRs #2811, #2772) added new JS modules to
containers/api-proxy/but did not update the Dockerfile'sCOPYcommand. When the container starts, Node.js fails withCannot find module './github-oidc'.Missing files added to Dockerfile COPY
github-oidc.jsaws-oidc-token-provider.jsgcp-oidc-token-provider.jsoidc-refresh-utils.jsHow to verify
After this fix, all
tests/integration/api-proxy*.test.tsintegration tests should pass again.Impact