Increase Documentation Maintainer LLM invocation budget to prevent false Copilot auth failures#5349
Conversation
There was a problem hiding this comment.
Pull request overview
This PR raises the Documentation Maintainer agent’s turn/invocation budget to avoid runs failing with a misleading Copilot “authentication failed” error when the real cause is hitting the configured max-invocations cap. It keeps the source workflow, compiled lock workflow, and the regression test expectations in sync.
Changes:
- Increased the doc-maintainer source workflow
max-turnsfrom 4 → 8. - Updated the compiled workflow to align runtime limits (
GH_AW_MAX_TURNSandapiProxy.maxRuns) from 4 → 8. - Updated the CI regression test to assert the new limits in both source and lock workflows.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/doc-maintainer.md |
Raises the doc-maintainer agent turn limit to reduce false “auth” failures caused by invocation caps. |
.github/workflows/doc-maintainer.lock.yml |
Aligns compiled/runtime settings (GH_AW_MAX_TURNS, apiProxy.maxRuns) to the new 8-turn budget. |
scripts/ci/doc-maintainer-workflow.test.ts |
Updates assertions to ensure source and lock workflows stay aligned at 8 turns. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 3/3 changed files
- Comments generated: 0
|
✅ Copilot review passed with no inline comments. @copilot Add the |
|
❌ Smoke Copilot BYOK AOAI (api-key) reports failed. AOAI BYOK (api-key) mode investigation needed... |
|
✅ Smoke Copilot BYOK completed. Copilot BYOK mode operational. 🔓 |
|
🔌 Smoke Services — All services reachable! ✅ |
|
✅ Smoke Gemini completed. All facets verified. 💎 |
|
Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded. |
|
🔑 Smoke Copilot PAT PAT auth validated. All systems operational. ✅ |
|
❌ Smoke Copilot BYOK AOAI (Entra) reports failed. AOAI BYOK (Entra) mode investigation needed... |
|
✅ Build Test Suite completed successfully! |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
✅ Smoke Claude passed |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
📡 Smoke OTel Tracing completed. All tracing scenarios validated. ✅ |
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (1 files)
Coverage comparison generated by |
🔥 Smoke Test Results — Auth mode: PAT (COPILOT_GITHUB_TOKEN)
Overall: PASS cc
|
|
✅ Smoke Test: Copilot BYOK (Direct Mode) — PASS
Running in direct BYOK mode (COPILOT_PROVIDER_API_KEY) with credentials forwarded through api-proxy sidecar.
|
Smoke Claude Result: PASS
Overall: PASS
|
|
Smoke test results:
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: PASS PR: Increase Documentation Maintainer LLM invocation budget to prevent false Copilot auth failures
|
Smoke Test: API Proxy OpenTelemetry Tracing
All 5 scenarios pass. OTEL tracing integration is fully functional.
|
Smoke Test: Gemini Engine Validation
Last 2 merged PRs:
Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "localhost"See Network Configuration for more information.
|
🧪 Chroot Version Comparison Results
Overall: ❌ Some versions differ between host and chroot. Python and Node.js versions in the chroot environment do not match the host. Go versions match correctly.
|
Smoke Test Results — ❌ FAIL
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS
|
The Documentation Maintainer workflow was failing with a Copilot “authentication failed” error, but the underlying cause was exhausting the configured LLM invocation cap (
403 Maximum LLM invocations exceeded (4 / 4)). This change aligns workflow limits so normal doc-maintainer runs don’t hit that artificial ceiling.Workflow budget increase
.github/workflows/doc-maintainer.mdmax-turns: 4→max-turns: 8Compiled workflow alignment
.github/workflows/doc-maintainer.lock.ymlGH_AW_MAX_TURNS: 4→GH_AW_MAX_TURNS: 8apiProxy.maxRuns: 4→apiProxy.maxRuns: 8Regression guard update
scripts/ci/doc-maintainer-workflow.test.ts8for both source and compiled turn limits