Skip to content

doc-maintainer: increase max-turns to 30, switch to weekly schedule - #6316

Merged
lpcox merged 2 commits into
mainfrom
doc-maintainer-weekly-30-turns
Jul 17, 2026
Merged

doc-maintainer: increase max-turns to 30, switch to weekly schedule#6316
lpcox merged 2 commits into
mainfrom
doc-maintainer-weekly-30-turns

Conversation

@lpcox

@lpcox lpcox commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Problem

The Documentation Maintainer workflow was failing with 429 Maximum LLM invocations exceeded (15 / 15) on every attempt (run #29538282271). The agent consumed ~452k input tokens reading context but exhausted the 15-turn limit before making any documentation changes (+0 -0 across all 4 harness attempts).

Changes

  • max-turns: 1530 — gives the agent enough headroom to read pre-computed context and actually make documentation updates
  • schedule: dailyweekly — the 7-day change gate already meant most daily runs were no-ops; weekly aligns the schedule with the gate
  • Recompiled with gh-aw v0.82.11 (latest pre-release)

The workflow was failing because the 15-turn LLM invocation limit was
exhausted before the agent could complete its work (429 errors on all
retry attempts). Doubling the limit to 30 turns gives the agent
enough headroom to read context and make documentation updates.

Also changed the schedule from daily to weekly since the 7-day change
gate already means most daily runs are no-ops.

Recompiled with gh-aw v0.82.11.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: cb31f5da-46db-42e8-93b5-9851f4f991a2
Copilot AI review requested due to automatic review settings July 16, 2026 22:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Note

This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.

- Enable copilot-requests: write so Copilot authenticates via github.token
- Recompile lock file with gh-aw v0.82.11 (actions pinned to new SHA)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

✅ Copilot review passed with no inline comments.

@lpcox Add the ready-for-aw label to this PR to trigger agentic CI smoke tests.

@lpcox
lpcox merged commit 7592957 into main Jul 17, 2026
38 of 42 checks passed
@lpcox
lpcox deleted the doc-maintainer-weekly-30-turns branch July 17, 2026 14:33
lpcox added a commit that referenced this pull request Jul 17, 2026
The doc-maintainer workflow was changed to weekly schedule with 30
max-turns in #6316 on main. Update the test assertions to match.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
lpcox added a commit that referenced this pull request Jul 17, 2026
* test: add coverage for container-lifecycle and compose-generator gaps

- container-lifecycle.ts: cover onNetworkReady topology path (lines 58-72,
  skipPull in topology mode, handleRetryStartupFailure return at line 175,
  and runAgentCommand error rethrow path (lines 313-314)
- compose-generator.ts: cover buildLocal missing containers/ guard (lines
  30-32) and api-proxy port publishing + awf-ext network attachment for
  microVM/sbx runtime mode (lines 144-158)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* test: address review feedback on lifecycle and compose-generator tests

- Remove duplicate topology tests (already in container-start.test.ts:390-461)
- Fix retry path: retry rejects, diagnostics return false, exercising line 175
- Add logger.error spy assertion to runAgentCommand test
- Delete compose-generator-gaps.test.ts (duplicates existing test files)

* fix: update doc-maintainer test to match weekly schedule change

The doc-maintainer workflow was changed to weekly schedule with 30
max-turns in #6316 on main. Update the test assertions to match.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Landon Cox <landon.cox@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants