Skip to content

fix(ci): timeout remaining llms-full and r2-pages fetches - #196

Open
SebTardif wants to merge 1 commit into
openclaw:mainfrom
SebTardif:fix/workflow-fetch-timeouts
Open

SebTardif wants to merge 1 commit into
openclaw:mainfrom
SebTardif:fix/workflow-fetch-timeouts

Conversation

@SebTardif

Copy link
Copy Markdown
Contributor

What Problem This Solves

llms-full.yml still does live fetch without AbortSignal. r2-pages.yml token verify uses curl -fsS with no --max-time, and the credential mint fetch has no abort. Sibling live-smoke and cutover scripts already bound those calls.

Evidence

Terminal output from the workflow contract checks:

$ node --test scripts/docs-site/llms-full-workflow.test.mjs scripts/docs-site/r2-pages-workflow.test.mjs
✔ every inline llms-full live fetch uses AbortSignal.timeout
✔ r2-pages credential mint curl uses --max-time
✔ r2-pages credential mint fetch uses AbortSignal.timeout
ℹ tests 3
ℹ pass 3

Real behavior proof

  • Behavior or issue addressed: Remaining docs publish fetches could hang until the job timeout.
  • Real environment tested: Windows 11, Node, docs workflow files on current main.
  • Exact steps or command run after this patch:
node --test scripts/docs-site/llms-full-workflow.test.mjs scripts/docs-site/r2-pages-workflow.test.mjs
  • Evidence after fix: terminal output from the patched tree:
✔ every inline llms-full live fetch uses AbortSignal.timeout
✔ r2-pages credential mint curl uses --max-time
✔ r2-pages credential mint fetch uses AbortSignal.timeout
  • Observed result after fix: Both llms-full live fetches use AbortSignal.timeout(30_000). Token verify curls use --max-time 30. The temp-access-credentials fetch uses the same abort signal.
  • What was not tested: A live GitHub Actions run against docs.openclaw.ai.

Summary

Bound the remaining llms-full and r2-pages network calls.

Add AbortSignal.timeout on live llms-full fetches and bound
r2-pages token verify curl plus credential mint fetch.

Signed-off-by: Sebastien Tardif <SebTardif@ncf.ca>
@SebTardif
SebTardif requested a review from a team as a code owner September 14, 2026 20:47
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.

1 participant