docs(ci): update runner registration budget#97943
Conversation
|
Codex review: needs maintainer review before merge. Reviewed June 29, 2026, 7:25 PM ET / 23:25 UTC. Summary PR surface: Docs +7. Total +7 across 2 files. Reproducibility: not applicable. for a docs/skill PR. The stale current-main text is source-verifiable in Review metrics: 1 noteworthy metric.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Risk before merge
Maintainer options:
Next step before merge
Security Review detailsBest possible solution: Land the docs/skill update only after a maintainer confirms the 10,000 live bucket and 60% target are the intended CI operating policy; keep workflow cap changes separate. Do we have a high-confidence way to reproduce the issue? Not applicable for a docs/skill PR. The stale current-main text is source-verifiable in Is this the best way to solve the issue? Yes. Updating the public CI runbook and matching maintainer skill is the narrowest maintainable fix for stale operator guidance; workflow cap changes should remain in a separate guarded PR. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 052dc59526a2. Label changesLabel changes:
Label justifications:
Evidence reviewedPR surface: Docs +7. Total +7 across 2 files. View PR surface stats
What I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
34f1c7b to
8efcfbe
Compare
Dependency graph guard clearedThis PR no longer has blocked dependency graph changes. A future dependency graph change requires a fresh
|
731dd92 to
30f68b5
Compare
289b76d to
05c0f49
Compare
|
Merged via squash.
|
What Problem This Solves
Resolves a problem where OpenClaw CI operator guidance still used the older 3,000-runner-registration bucket and 2,000-registration target after live GitHub rate-limit evidence showed the current bucket is higher.
Why This Change Was Made
The CI runbook and maintainer skill now tell operators to verify
actions_runner_registrationlive before tuning and to target about 60% of the live bucket. With the current 10,000-registration bucket, the documented operating target is 6,000 registrations per 5 minutes.User Impact
Maintainers get current, reusable guidance for right-sizing Blacksmith-backed CI without overreacting to stale quota numbers. No workflow behavior changes in this PR.
Evidence
ghx api rate_limit --jq '{core:.resources.core,graphql:.resources.graphql,actions_runner_registration:.resources.actions_runner_registration}'reportedactions_runner_registration.limit=10000andremaining=10000during this pass.node scripts/run-vitest.mjs test/scripts/ci-workflow-guards.test.tsnode scripts/check-workflows.mjsnode scripts/docs-list.js./node_modules/.bin/oxfmt --check .github/workflows/ci.yml .github/workflows/codeql-critical-quality.yml docs/ci.md test/scripts/ci-workflow-guards.test.ts .agents/skills/openclaw-ci-limits/SKILL.md .agents/skills/openclaw-ci-limits/agents/openai.yamlgit diff --check.agents/skills/autoreview/scripts/autoreview --mode localclean, no accepted/actionable findings