[docs] auth: Cross-reference Copilot auto model prefix-stripping in authentication-architecture - #8505
Conversation
…cation-architecture Codex's OpenAI-native auto limitation was documented, but the authentication-architecture guide did not note that Copilot's own auto selector (including the copilot/auto LiteLLM-style alias used by Codex and Pi) is unaffected and supported via api-proxy prefix stripping (stripRedundantProviderPrefix, shipped in #8500). This was already documented in docs/api-proxy-sidecar.md but missing here, causing a cross-document inconsistency. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟢 Approval recommended
The documentation accurately matches the implemented and tested prefix-stripping behavior.
Pull request overview
Clarifies that Copilot’s auto model supports copilot/auto through provider-prefix normalization.
Changes:
- Distinguishes Copilot
autofrom OpenAI-nativeauto. - Documents automatic removal of the redundant
copilot/prefix.
File summaries
| File | Description |
|---|---|
docs/authentication-architecture.md |
Adds Copilot model-routing clarification and cross-reference. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Documentation PreviewDocumentation has been built for this PR. To view locally:
Built from commit 88dede6 |
|
✅ Copilot review passed with no inline comments. @github-actions[bot] Add the |
|
❌ Smoke Copilot BYOK AOAI (api-key) reports failed. AOAI BYOK (api-key) mode investigation needed...
|
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 Warning Firewall blocked 7 domainsThe following domains were blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "accounts.google.com"
- "android.clients.google.com"
- "clients2.google.com"
- "contentautofill.googleapis.com"
- "msfeed25.pkgs.visualstudio.com"
- "www.google.com"
- "www.gstatic.com"See Network Configuration for more information.
|
|
✅ Build Test Suite completed successfully!
|
|
Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded.
|
|
❌ Smoke Gemini reports failed. Facets need polishing...
|
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤
|
|
📰 DEVELOPING STORY: Smoke Docker Sbx reports failed. Our correspondents are investigating the incident...
|
|
📡 Smoke OTel Tracing completed. All tracing scenarios validated. ✅
|
|
✅ Smoke Copilot BYOK completed. Copilot BYOK mode operational. 🔓
|
|
🚀 Security Guard has started processing this pull request |
|
❌ Smoke Copilot BYOK AOAI (Entra) reports failed. AOAI BYOK (Entra) mode investigation needed...
|
|
🔌 Smoke Services — All services reachable! ✅
|
|
✅ Smoke Claude passed Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "api.anthropic.com"See Network Configuration for more information.
|
|
🛡️ Smoke Copilot Network Isolation confirmed the egress allowlist is enforced. ✅ Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "example.com"See Network Configuration for more information.
|
|
Smoke Test: Copilot Engine —
Overall: PASS
|
|
✅ Copilot BYOK (Direct) Mode Smoke Test PASSED
Mode: Direct BYOK (COPILOT_PROVIDER_API_KEY) via api-proxy sidecar
|
|
EGRESS_RESULT allow=pass deny=pass ✅ Allowed domain (github.com) reachable: Overall: PASS cc Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "example.com"See Network Configuration for more information.
|
Smoke Test: Claude Engine Validation
Overall result: PASS Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "api.anthropic.com"See Network Configuration for more information.
|
📡 OTel Tracing Smoke Test Results
Summary: All 5 scenarios passed — OTEL module loading, span creation, GenAI usage attributes, env var/context propagation, and OTLP export through Squid are all working as expected. No regressions detected.
|
|
Smoke Test: Services Connectivity — PASS ✅
Overall: PASS
|
Warning Firewall blocked 7 domainsThe following domains were blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "accounts.google.com"
- "android.clients.google.com"
- "clients2.google.com"
- "contentautofill.googleapis.com"
- "msfeed25.pkgs.visualstudio.com"
- "www.google.com"
- "www.gstatic.com"See Network Configuration for more information.
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — PASS Note: Java's local Maven repository (
|
Chroot vs Host Version Comparison
Result: Not all tests passed (
|
Summary
containers/api-proxy/model-utils.js,model-resolver.js,model-body-rewriter.js,body-handler.js; recent PRs Support "auto" model for Pi and Codex via provider-prefix stripping #8500, fix: isolate Actions OIDC from agent container #6894, docs: update authentication and API proxy guidance #6889, feat: add Auth Doctor Updater workflow #6898; gh-aw issue #50053 (resolved by gh-aw#50054).docs/api-proxy-sidecar.mdalready documented that Copilot's ownautomodel selector (including the LiteLLM-stylecopilot/autoalias used by Codex and Pi) is supported via provider-prefix stripping (shipped in merged PR Support "auto" model for Pi and Codex via provider-prefix stripping #8500,stripRedundantProviderPrefix()incontainers/api-proxy/model-utils.js), butdocs/authentication-architecture.mdonly carried the older OpenAI-nativeautolimitation caveat without this clarification.github/gh-aw-firewall#6894(Actions OIDC agent isolation) andgithub/gh-aw#50053(runner→gateway OIDC boundary) are both merged/closed as completed; existing docs already describe them correctly as shipped, so no changes were needed there.Documentation Changes
autoOpenAI-native-limitation caveat (Example 2) clarifying that this limitation does not apply to Copilot's ownautoselector: harnesses routed through the Copilot provider (port10002), including Codex and Pi, can requestautoorcopilot/auto, and the api-proxy sidecar strips the redundantcopilot/prefix before forwarding so Copilot resolves it dynamically. This matches the existing, more complete explanation already present indocs/api-proxy-sidecar.md(Codex example section) and closes the gap between the two documents.Validation
containers/api-proxy/model-utils.js(stripRedundantProviderPrefix),model-resolver.js,model-body-rewriter.js,body-handler.js— confirmed the always-on prefix-stripping pipeline shipped in merged PR Support "auto" model for Pi and Codex via provider-prefix stripping #8500 onmain(commit 0603bd1).autoguidance indocs/api-proxy-sidecar.mdanddocs/authentication-architecture.md; wording and cross-reference link now aligned.api-proxy-sidecar.md#codex-openai-example) matches the existing anchor already used earlier in the same file.[docs] auth:— none found covering this specific cross-reference gap.Sources